Skip to content

Cap TypeScript path alias parsing work - #3302

Merged
Widthdom merged 1 commit into
mainfrom
fix-issue3069
Jun 5, 2026
Merged

Cap TypeScript path alias parsing work#3302
Widthdom merged 1 commit into
mainfrom
fix-issue3069

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 5, 2026

Copy link
Copy Markdown
Owner

Summary

  • Cap TypeScript/jsconfig compilerOptions.paths alias rule counts, per-rule and total target counts, and alias/target string lengths.
  • Cap oversized module specifiers and wildcard-substituted target lengths before path probing, with bounded warnings for skipped work.
  • Add focused SymbolExtractor coverage for truncation, ignored long strings, oversized module specifiers, and oversized wildcard substitution.

Documentation / Changelog

  • Added changelog fragment: changelog.d/unreleased/3069.security.md.
  • No README/DEVELOPER/TESTING guide changes were needed because this is a bounded internal hardening of existing TypeScript/jsconfig alias resolution behavior.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~SymbolExtractorTests.Extract_TypeScript_"
  • dotnet build CodeIndex.sln -c Release -p:UseSharedCompilation=false
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release --no-build --filter "FullyQualifiedName~SymbolExtractorTests.Extract_TypeScript_"
  • dotnet format CodeIndex.sln --verify-no-changes
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Final Codex adversarial review: No blocking/actionable issues found.

Known validation notes:

Fixes #3069

@Widthdom
Widthdom force-pushed the fix-issue3069 branch 3 times, most recently from aa1ebc2 to c143f28 Compare June 5, 2026 13:15
@Widthdom
Widthdom merged commit f05e186 into main Jun 5, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue3069 branch June 5, 2026 14:59
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.

TypeScript path alias parsing should cap paths rule count, target count, and string lengths

1 participant