Skip to content

[codex] parse tsconfig JSONC without stripping globs#503

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

[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

@chaochaoweb3 chaochaoweb3 marked this pull request as ready for review June 30, 2026 05:24
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