--- collectCustomProviders tests ---

TestCollectPendingComments_AwaitsPool covers the worker-pool drain branch of CollectPendingComments.

TestCommentWorkerPool_AwaitKeyConcurrentSubmitOtherKey is the regression test for the review-path race: one file's goroutine draining its async commen

TestCommentWorkerPool_AwaitKeyUnknown verifies waiting on a key with no submissions returns immediately.

TestCommentWorkerPool_AwaitKeyWaitsForOwnKey verifies that AwaitKey blocks until the units submitted under its key complete, without waiting for units

TestCommitDiffMergeCommitReviewsFirstParentDiff covers `ocr review --commit <merge>`: plain `git show` renders merge commits as a combined diff ("diff

TestCommitDiffSurvivesExternalDiffTool covers the ModeCommit call site (git show <commit>), which likewise must pass --no-ext-diff so that a user's ex

TestCompleteEnum verifies the closure returns the provided values with the no-file-completion directive.

TestCompleteSessionIDs_WithSession drives the success loop of the shell completion helper against a real fixture session, covering the prefix-match an

--- ResumeState.CompletedCount ---

Concurrent registration and transitions on distinct items must be race-free (run with -race) and produce exactly-once coverage.

TestConfigRoundTripPreservesTimeoutSec guards against silent config loss: the resolver reads providers.<name>.timeout_sec / llm.timeout_sec (the docs

TestConfirmDeleteCustomModel_SaveFailureRollback drives the custom-model delete handler down its save-failure branch and asserts the entry is rolled b

TestConfirmDeleteCustomProviderModel_Guards covers the not-user-added and nil-cfg early returns.

TestConfirmDeleteOfficialModel covers deleting a user-added model from an official (preset) provider entry, plus the not-user-added guard.

TestConfirmDeleteOfficialModelActiveClear covers the modelTUIModel handler where the deleted model is also the active config model: deletion clears th

TestConfirmDeleteOfficialModel_SaveFailureRollback drives the official-model delete handler down its save-failure branch for a user-added model and as

The first terminal state wins and a conflicting later transition returns an error: a late error must not demote a completed item.

The CSP must stay strict: no 'unsafe-inline'/'unsafe-eval' relaxation, since the formerly-inline session script now lives in static/session.js.

--- copyLlmComments ---

Coverage arrays are sorted by item_id and marshal as [] (never null).

--- protocol normalization / openai-responses support ---