Skip to content

[codex] parse tsconfig JSONC without stripping globs#503

Draft
chaochaoweb3 wants to merge 1 commit into
Egonex-AI:mainfrom
chaochaoweb3:codex/fix-tsconfig-jsonc-globs
Draft

[codex] parse tsconfig JSONC without stripping globs#503
chaochaoweb3 wants to merge 1 commit into
Egonex-AI:mainfrom
chaochaoweb3:codex/fix-tsconfig-jsonc-globs

Conversation

@chaochaoweb3

Copy link
Copy Markdown

Summary

  • replace the naive tsconfig JSONC comment stripping with a string-aware scanner
  • preserve comment-looking tokens inside string literals such as **/*.ts
  • add a regression test that keeps path aliases when include globs and block comments are both present

Fixes #502

Validation

  • npm exec --yes --package=pnpm@10.6.2 -- pnpm install --frozen-lockfile
  • npm exec --yes --package=pnpm@10.6.2 -- pnpm vitest run understand-anything-plugin/src/tests/extract-import-map.test.mjs
  • npm exec --yes --package=pnpm@10.6.2 -- pnpm eslint understand-anything-plugin/skills/understand/extract-import-map.mjs understand-anything-plugin/src/tests/extract-import-map.test.mjs
  • node --check understand-anything-plugin/skills/understand/extract-import-map.mjs
  • git diff --check

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.

tsconfig with glob include + path aliases → all aliases silently dropped

1 participant