Skip to content

Improve Code Health of hotspot source files - #246

Merged
woksin merged 3 commits into
mainfrom
chronicle-typescript-244-code-health
Oct 11, 2026
Merged

woksin merged 3 commits into
mainfrom
chronicle-typescript-244-code-health

Conversation

@woksin

@woksin woksin commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

Changed

Behaviour-preserving extract-method refactoring guided by CodeScene findings (#244).
Keep skipped reducer events synchronous and serialize redaction causation inside the observed operation.
@woksin woksin self-assigned this Oct 11, 2026
@woksin

woksin commented Oct 11, 2026

Copy link
Copy Markdown
Contributor Author

Reviewer notes

Behaviour-preserving refactor of CodeScene hotspot source files (CodeScene project 85913). Tests, specs, docs and config are untouched; the existing suites are the safety net.

  • Code Health (baseline -> now): EventSequence.ts 6.56 -> 9.38; ChronicleClient.ts 7.97 -> 10; Reducers.ts 7.74 -> 9.68
  • Local Tier 1 (mirrors CI) passed on the final tree: yarn ci (3319 tests); yarn test:integration not run locally (needs kernel container)
  • Reviewed by a cross-provider review (Claude Opus + GPT) with adversarial verification; behaviour changes found during iteration (evaluation order, async-context flow, resource lifetimes, visibility) were fixed before this PR.
  • Integration jobs run in CI only (Docker is not available locally).

@woksin
woksin merged commit a09604d into main Oct 11, 2026
@woksin
woksin deleted the chronicle-typescript-244-code-health branch October 11, 2026 11:14
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