fix: set ignoreTryCatch: false in rollup commonjs plugin for Windows

Browse source

minimatch 3.x uses `try { return require('path') } catch (e) {}` to
get the path module, falling back to `{ sep: '/' }`. With rollup's
commonjs plugin, `ignoreTryCatch: true` (the default) leaves
require('path') unconverted inside try blocks. At runtime in the ESM
bundle, require() is unavailable, so the catch fires and minimatch
uses '/' as the path separator — breaking all glob matching on Windows
where paths use backslashes.

Setting ignoreTryCatch: false ensures require('path') is converted to
an ESM import, giving minimatch the correct path.sep ('\\' on Windows).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

...

This commit is contained in:

jdx 2026-04-11 17:28:10 +00:00

parent 45d1740529

commit e8a0f70f2f

No known key found for this signature in database

GPG key ID: 584DADE86724B407

4 changed files with 108 additions and 97 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

192

dist/index.js generated vendored

View file

File diff suppressed because one or more lines are too long

2

dist/index.js.map generated vendored

View file

File diff suppressed because one or more lines are too long