Skip to content

Improve Code Health of the AI corpus synchronizer - #338

Merged
woksin merged 4 commits into
mainfrom
cli-326-code-health
Oct 12, 2026
Merged

woksin merged 4 commits into
mainfrom
cli-326-code-health

Conversation

@woksin

@woksin woksin commented Oct 12, 2026

Copy link
Copy Markdown
Collaborator

Changed

@woksin woksin self-assigned this Oct 12, 2026
@woksin

woksin commented Oct 12, 2026

Copy link
Copy Markdown
Collaborator Author

Reviewer notes

Behaviour-preserving refactor of a CodeScene hotspot (project 85913), batch 2. Tests, docs and config are untouched.

  • Code Health: AiCorpusSynchronizer.cs 5.42 -> 9.09; new files 9.38-10.
  • Local Tier 1 (CI mirror) passed on the merged tree. Docker/packaging steps run in CI only.
  • Full cross-provider review passed. A point it raised about deferred enumeration of a caller-supplied rule list was then made strictly equivalent to main, and that fix was reviewed separately (passed).

@woksin
woksin merged commit 64b5851 into main Oct 12, 2026
@woksin
woksin deleted the cli-326-code-health branch October 12, 2026 07:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant