Skip to content

Split the testing package's largest modules into focused helpers - #290

Merged
woksin merged 7 commits into
mainfrom
chore/code-health-testing
Nov 11, 2026
Merged

woksin merged 7 commits into
mainfrom
chore/code-health-testing

Conversation

@woksin

@woksin woksin commented Nov 11, 2026

Copy link
Copy Markdown
Contributor

Changed

@woksin woksin self-assigned this Nov 11, 2026
@woksin

woksin commented Nov 11, 2026

Copy link
Copy Markdown
Contributor Author

Notes for reviewers:

  • Behavior-preserving refactor. 27 characterization specs were added before each refactor to pin error messages, validation and rejection order, cloning/defaults, append failure latching, seed rollback and subject fallback.
  • Code Health: ProjectionCapabilities 2.85 -> 10, ProjectionValueConverter 5.52 -> 10, InProcessEventSequence 3.44 -> 9.09; every new helper module scores 10 (two interface-only modules have no score). The remaining InProcessEventSequence findings concern its public method signatures and are left for the next major.
  • Local checks: yarn install --immutable, yarn ci (builds, typechecks, ESM smoke test, 3931 specs passed, 1 skipped) and yarn test in Source. Integration tests against a kernel run in CI only.
  • Independent cross-provider review of the whole change: passed, no blocking findings.

@woksin
woksin merged commit f159c39 into main Nov 11, 2026
8 of 9 checks passed
@woksin
woksin deleted the chore/code-health-testing branch November 11, 2026 19:52
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