Skip to content

feat: add module-sync catchall tracing#584

Open
jeffsee55 wants to merge 2 commits into
mainfrom
jeff/module-sync-fallbacks
Open

feat: add module-sync catchall tracing#584
jeffsee55 wants to merge 2 commits into
mainfrom
jeff/module-sync-fallbacks

Conversation

@jeffsee55
Copy link
Copy Markdown
Contributor

Summary

  • Add a moduleSyncCatchall trace option to include both module-sync and normal fallback export targets.
  • Preserve existing module-sync fallback behavior while extending catchall coverage to wildcard export patterns.
  • Document the new option and add focused unit coverage.

Test plan

  • npm run build
  • npx prettier --check readme.md src/node-file-trace.ts src/resolve-dependency.ts src/types.ts test/unit.test.js test/unit/imports-module-sync/output.js test/unit/imports-module-sync-cjs/output.js test/unit/module-sync-condition-cjs/output.js test/unit/module-sync-condition-es/output.js test/unit/module-sync-condition-es-default/output.js test/unit/module-sync-condition-es-nested/output.js test/unit/self-reference-module-sync/output.js test/unit/module-sync-catchall/input.js test/unit/module-sync-catchall/package.json test/unit/module-sync-catchall/sync/feature.js test/unit/module-sync-catchall/fallback/feature.js test/unit/module-sync-catchall/output.js test/unit/module-sync-catchall/test-opts.json
  • npx jest test/unit.test.js --runInBand --testNamePattern="module-sync"

Made with Cursor

Co-authored-by: Cursor <cursoragent@cursor.com>
@jeffsee55 jeffsee55 force-pushed the jeff/module-sync-fallbacks branch from ad81f3f to 91d6e80 Compare May 20, 2026 21:36
@jeffsee55 jeffsee55 marked this pull request as ready for review May 20, 2026 21:37
@jeffsee55 jeffsee55 requested review from a team, icyJoseph, ijjk and styfle as code owners May 20, 2026 21:37
Comment thread src/resolve-dependency.ts Outdated
Comment thread src/resolve-dependency.ts Outdated
Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants