@lcv-ideas-software/cross-review 4.3.0 → 4.3.1

package/CHANGELOG.md

@@ -7,6 +7,24 @@ standard `v00.00.00`; npm package versions remain SemVer.
 
 ## [Unreleased]
 
+## [v04.03.01] — 2026-06-05
+
+**Patch — provider skip classification hotfix.** This release follows up on a
+real hard-gate incident where Claude/Anthropic was skipped after provider
+overload. The immediate provider cause was Anthropic `overloaded_error`; the
+runtime issue was that any `provider_error`, including non-retryable provider
+400-style failures, could be treated as skippable.
+
+### Changed
+
+- `provider_error` is now skippable only when classified as retryable, so
+  non-retryable provider payload/schema rejections block convergence instead of
+  being silently removed from the panel.
+- Anthropic `overloaded_error` without preserved HTTP status text is now treated
+  as retryable, matching HTTP 529 overload behavior.
+- `session.peer_skipped_unavailable` events now include retryability, recovery
+  hint, and a redacted provider error preview in event data.
+
 ## [v04.03.00] — 2026-06-05
 
 **Minor — P1/P2/P3 audit follow-up.** This release closes the first concrete

package/README.md

@@ -24,7 +24,7 @@ npm install -g @lcv-ideas-software/cross-review
 npm install -g @lcv-ideas-software/cross-review --registry=https://npm.pkg.github.com
 ```
 
-**Status.** Stable. Current release: **v04.03.00** (npm package `4.3.0`). See [CHANGELOG.md](./CHANGELOG.md) for the full release history.
+**Status.** Stable. Current release: **v04.03.01** (npm package `4.3.1`). See [CHANGELOG.md](./CHANGELOG.md) for the full release history.
 
 > **Project renamed 2026-05-15.** This project was previously published as
 > [`@lcv-ideas-software/cross-review-v2`](https://www.npmjs.com/package/@lcv-ideas-software/cross-review-v2)
@@ -38,6 +38,7 @@ The version history at a glance:
 
 | Release              | Scope                                                                                                                                                                                                              |
 | -------------------- | ------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------ |
+| **`v04.03.01`**      | Patch — tighten skip-peer classification so non-retryable provider errors block, while Anthropic overload events remain retryable and better surfaced in skip diagnostics.                                         |
 | **`v04.03.00`**      | Minor — P1/P2/P3 follow-up with unresolved-evidence close-out visibility, an offline fixture eval harness, and a read-only peer reliability report.                                                                |
 | **`v04.02.05`**      | Patch — harden session auditability with terminal events, cost split reporting, `not_resurfaced` visibility, and relator provenance checks for session IDs/GitHub URLs.                                            |
 | **`v04.02.04`**      | Patch — harden truthfulness preflight auditability, add a read-only preflight retest tool, and reduce false parser warnings for attached/log evidence.                                                             |

package/dist/scripts/smoke.js

@@ -22,6 +22,7 @@ import { SWEEP_MIN_IDLE_MS } from "../src/core/session-store.js";
 import { parsePeerStatus } from "../src/core/status.js";
 import { PEERS } from "../src/core/types.js";
 import { getCallerCandidatesFromClientInfo, lockCallerPeerSelection, pruneCompletedJobs, SessionIdSchema, setHostTokensRecord, verifyCallerIdentity, } from "../src/mcp/server.js";
+import { classifyProviderError } from "../src/peers/errors.js";
 import { selectFromCandidates } from "../src/peers/model-selection.js";
 import { StubAdapter } from "../src/peers/stub.js";
 import { redact } from "../src/security/redact.js";
@@ -488,13 +489,13 @@ assert.equal(checkConvergence(["codex", "claude"], "READY", [fakeReady("codex"),
 // taxonomy + the skip-gated quorum floor + the zero-skip non-regression
 // invariant.
 {
-    const fakeFailure = (peer, failureClass) => ({
+    const fakeFailure = (peer, failureClass, retryable = true) => ({
         peer,
         provider: "stub",
         model: "stub",
         failure_class: failureClass,
         message: `stub ${failureClass}`,
-        retryable: true,
+        retryable,
         attempts: 3,
         latency_ms: 0,
     });
@@ -509,8 +510,12 @@ assert.equal(checkConvergence(["codex", "claude"], "READY", [fakeReady("codex"),
         "fallback_exhausted",
     ]) {
         assert.equal(isSkippableFailure(fakeFailure("grok", fc)), true, `v3.7.3 / skip-peer: ${fc} must be skippable (infra unavailability)`);
-        assert.equal(SKIPPABLE_FAILURE_CLASSES.has(fc), true, `v3.7.3 / skip-peer: ${fc} must be in SKIPPABLE_FAILURE_CLASSES`);
     }
+    assert.equal(SKIPPABLE_FAILURE_CLASSES.has("provider_error"), true, "v3.7.3 / skip-peer: provider_error remains a known skip candidate when retryable");
+    assert.equal(isSkippableFailure(fakeFailure("grok", "provider_error", false)), false, "v4.3.1 / skip-peer: non-retryable provider_error (e.g. provider 400 payload/schema rejection) must block instead of being skipped");
+    const anthropicOverloaded = classifyProviderError("claude", "anthropic", "claude-opus-4-8", new Error('{"type":"error","error":{"details":null,"type":"overloaded_error","message":"Overloaded"},"request_id":"req_fixture"}'), 1, Date.now());
+    assert.equal(anthropicOverloaded.failure_class, "provider_error");
+    assert.equal(anthropicOverloaded.retryable, true, "v4.3.1 / provider-errors: Anthropic overloaded_error without HTTP status text must still be retryable");
     for (const fc of [
         "schema",
         "unparseable_after_recovery",