Skip to content

Stabilize bounded regex extraction under load - #2972

Merged
Widthdom merged 3 commits into
mainfrom
fix-issue2945-2943
Jun 3, 2026
Merged

Stabilize bounded regex extraction under load#2972
Widthdom merged 3 commits into
mainfrom
fix-issue2945-2943

Conversation

@Widthdom

@Widthdom Widthdom commented Jun 2, 2026

Copy link
Copy Markdown
Owner

Summary

Validation

  • dotnet run --project tools/CodeIndex.Changelog -- check
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName~ReferenceExtractorTests.Extract_FSharp_DetectsWhenGuardApplicationCalls|FullyQualifiedName~QueryCommandRunnerTests.RunReferences_ExactJson_SqlTruncateTargetsHandleOnlyAndMultipleTargets|FullyQualifiedName~QueryCommandRunnerTests.RunReferences_ExactJson_SqlLineEndCommentsKeepUnfinishedTargetPrefixes|FullyQualifiedName~QueryCommandRunnerTests.RunReferences_ExactJson_SqlMergeTempTargetWithMultilineHintResolvesTargetAndSource|FullyQualifiedName~BoundedRegexTests.DefaultMatchTimeout_MatchesRuntimeSafetyTimeout"
  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj -c Release --framework net9.0 --filter "FullyQualifiedName~ExportImportCommandRunnerTests|FullyQualifiedName~ReportCommandRunnerTests|FullyQualifiedName~QueryCommandRunnerTests.RunReferences_ExactJson_SqlMergeTempTargetWithMultilineHintResolvesTargetAndSource|FullyQualifiedName~QueryCommandRunnerTests.RunReferences_ExactJson_SqlUsingSourceMatcherSkipsDdlUsingClauses"
  • git diff --check
  • dotnet build
  • dotnet ./src/CodeIndex/bin/Debug/net8.0/cdidx.dll status --check --json
  • Codex adversarial review: no blocking/actionable issues found.

Notes

  • No AGENT.md, CLAUDE.md, or AGENT_GUIDE.md changes.
  • No additional follow-up issues were identified.

Fixes #2943
Fixes #2945

@Widthdom
Widthdom marked this pull request as ready for review June 2, 2026 21:41
Resolve conflicts with the bounded regex timeout guard added on main by keeping the runtime safety timeout alignment and preserving the scheduler-contention regression test.
@Widthdom
Widthdom merged commit bd737a5 into main Jun 3, 2026
10 checks passed
@Widthdom
Widthdom deleted the fix-issue2945-2943 branch June 3, 2026 00:07
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.

Flaky full dotnet test run can fail reference extraction tests QueryCommandRunner SQL reference JSON test is flaky under grouped net9 runs

1 participant