Skip to content

fix: code scanning alert 8 in dependency detection#94

Draft
jbeckwith-oai wants to merge 1 commit intomainfrom
codex/fix-codeql-alert-8-config-imports
Draft

fix: code scanning alert 8 in dependency detection#94
jbeckwith-oai wants to merge 1 commit intomainfrom
codex/fix-codeql-alert-8-config-imports

Conversation

@jbeckwith-oai
Copy link
Copy Markdown
Contributor

@jbeckwith-oai jbeckwith-oai commented Apr 13, 2026

Summary

  • replace regex-based import parsing with a small line-oriented import reader
  • preserve existing dependency detection behavior for package imports
  • add a regression test through writeProviderFile package generation

Root Cause

CodeQL flagged the provider dependency regex as polynomial-time on crafted import strings. The parser only needs a single import specifier per line, so a direct string scan is simpler and safer.

Validation

  • npm test -- src/generator/config-imports.test.ts
  • npm run build

@jbeckwith-oai jbeckwith-oai changed the title [codex] Fix code scanning alert 8 in dependency detection fix: code scanning alert 8 in dependency detection Apr 13, 2026
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.

1 participant