[polewskm]

Slice B of the #95 refactor: module moves and shared helpers, with no change in behavior.

• The C# contract checks move from gather.ts to contracts.ts, stoppedBy to checks.ts (with NoProviderError in its own no-provider.ts, so the acknowledgement still loads no provider code), and reading a finding comment's title and text to threads.ts.
• review() takes an options object, runs its reviewer calls in runReviewers(), finishes in finishReview(), and runs a light review in its own reviewLight(); every caller and test passes the same arguments as before.
• The eval output file has one EvalOutput type its writer and readers share, one helper counts the findings that belong upstream, and the workflows read the Node version from .node-version.
• Merges main in (merge commit, so this must merge with a merge commit, not by rebasing) to resolve the changelog and test/acknowledge.test.ts.

Refs #95

A first review round dropped the leftover NoProviderError re-export.

[ncode-review]

🤖 NCode Review at 6784e11: 1 new finding on the changed lines.

[ncode-review]

_{Superseded by the review at 3a08a59.}

[polewskm]

/review

[ncode-review]

🤖 Review complete

Verdict: ✅ Approve — Re-review of what changed since the last review at 6784e11: 2 hunks in 1 file and changes taken back in 2 files, by correctness only. Earlier findings: F1 fixed. No further findings above nits.

1 reviewer run — 0 findings (run details)

Reviewer	Model	Findings	Top
correctness	sonnet-5-5	0	🟢

_{Trial: findings are posted as review threads and in the check run. $0.03 on Amazon Bedrock, 9s, at 3a08a59. (how to interact)}

📋 Outcomes

Merged at 3a08a59: 1 fixed, 0 dismissed, 0 acknowledged, 0 ignored.

Finding	Reviewer	Severity	Outcome
F1	hygiene	🟡 Nit	✅ fixed