@dogfood-lab/verify 1.5.0 → 1.6.0

package/README.md

@@ -83,7 +83,7 @@ Provenance fields (`github_run_id`, `github_workflow_ref`) are required when `pr
 
 ### Prefix taxonomy
 
-The verifier emits two prefix classes:
+The verifier emits rejection-reason strings under stable prefixes, each mapping to one of four routing classes:
 
 Discrimination happens by **class**, surfaced by `parseRejectionReason` (below). Every prefix maps to one of four classes: **submission-bad** (the submitter fixes the payload), **operational** (the verifier/tooling faulted), **ingest** (an ingest-side load fault), or **unknown** (unrecognized prefix).
 
@@ -94,7 +94,7 @@ Discrimination happens by **class**, surfaced by `parseRejectionReason` (below).
 | `schema:` | `validators/schema.js` | JSON Schema check on the submission/record envelope failed. The rest of the string carries the AJV path + message. |
 | `policy:` | `validators/policy.js` | Per-repo policy gate failed (forbidden tags, missing required fields, surface evidence/CI requirements, etc.). |
 | `steps[<id>]:` | `validators/steps.js` | Step-level contract check failed on a specific step id (gate accumulation, ordering, evidence shape). |
-| `provenance:` | `validators/provenance.js` | The GitHub run-id confirmation could not match the submitted commit/repo at the GitHub API. |
+| `provenance:` | `validators/provenance.js` | The run was genuinely **absent / not confirmable** — a 404 from the provider API, or the run head did not match the submitted commit/repo. The submitter's payload points at a run that does not exist or does not bind. (Operational provider faults — 429/5xx/401/403 — are NOT this class; see `provenance-fault:` below.) |
 | `repo:` | `index.js` cross-field guard | `submission.repo` does not match the owner/repo encoded in `source.run_url` (anti-forgery guard). Emitted as `repo:mismatch: …`. |
 | `submission-contains-verifier-field:` | `index.js` | The submission carried a verifier-owned field (`policy_version`, `verification`, or an object `overall_verdict`) it must not author. |
 | `CONTRACT_SCHEMA_TOO_NEW:` | `validators/schema-version.js` | The submission's `schema_version` declares a MAJOR **above** what this build supports (see `SUPPORTED_SCHEMA_VERSIONS` in `@dogfood-lab/schemas`). This build cannot understand a future contract — **the operator must upgrade testing-os**, but the routing class stays submission-bad (the payload as-shipped cannot be accepted by THIS build). |
@@ -109,6 +109,7 @@ Discrimination happens by **class**, surfaced by `parseRejectionReason` (below).
 | `VALIDATOR_FAULT_STEPS:` | `runValidator('steps', …)` catch | Internal exception inside the steps validator. |
 | `VALIDATOR_FAULT_CONTRACT_SCHEMA_VERSION:` | `runValidator('contract_schema_version', …)` catch | The version gate was called with an unknown contract key (a programmer error at the call site, not a submission fault). |
 | `submission-malformed:` | `index.js` null/non-object early-return | The submission itself was `null` or not an object — a malfunctioning **dispatcher** sent garbage, not a submitter who authored a bad-but-shaped payload. Page ops / inspect the dispatch pipeline; do NOT bounce it to a submitter. |
+| `provenance-fault:` | `index.js` provenance catch | The provenance adapter THREW an operational error confirming the run — a provider **429 rate-limit, 5xx outage, or 401/403 token** fault (`validators/provenance.js` throws these on purpose for non-404 responses). The submitter's payload is fine; the verifier could not reach a verdict. Page ops / retry; do NOT bounce it to a submitter. Distinct from the submission-bad `provenance:` (genuine absence/404). |
 
 Any future `VALIDATOR_FAULT_<NEW>:` prefix is classified `operational` by family — `parseRejectionReason` matches the `VALIDATOR_FAULT_` head, so a new validator class needs no parser edit. The `submission-malformed:` prefix is matched literally (it is not part of the `VALIDATOR_FAULT_` family).
 
@@ -153,6 +154,10 @@ for (const r of result.rejection_reasons) {
 
 Persistence note: every entry above is round-tripped verbatim through `verification.rejection_reasons` in the persisted-record JSON; the schema enforces `array of string` so any consumer of the audit-DB ground truth sees the same prefix vocabulary.
 
+### Warnings channel (accepted-with-warning)
+
+Not every policy signal is a rejection. A policy rule declared `severity: warn` produces a `policy: <id>: <message>` entry on `verification.warnings` (an optional `array of string` on the persisted record) **without** flipping the verdict to `rejected` — the submission is accepted and recorded, the warning rides alongside it. (`severity: info` rules are logged only and never persisted; `severity: reject` rules go to `rejection_reasons` as above.) Consumers that want advisory signals read `verification.warnings`; the routing decision (`parseRejectionReason`) only concerns `rejection_reasons`. A clean accepted submission carries no `warnings` key at all.
+
 ## Docs
 
 📖 Full handbook: **<https://dogfood-lab.github.io/testing-os/handbook/>**

package/cli.js

@@ -316,7 +316,10 @@ export function renderExplain(record) {
   const accepted = v.status === 'accepted';
   const lines = [];
 
-  lines.push(accepted ? 'VERDICT: accepted' : 'VERDICT: rejected');
+  // Uppercase the rendered state word to match the sibling verdict banners
+  // (findings-render.js). This is display-only; the underlying enum
+  // (v.status, --json output) stays lowercase.
+  lines.push(`VERDICT: ${(v.status || (accepted ? 'accepted' : 'rejected')).toUpperCase()}`);
   lines.push('');
   lines.push(`  schema_valid:         ${v.schema_valid}`);
   lines.push(`  policy_valid:         ${v.policy_valid}`);

package/index.js

@@ -209,9 +209,15 @@ export async function verify(submission, options) {
         refCommitSha: submission.ref?.commit_sha
       });
     } catch (err) {
-      reasons.push(`provenance: verification failed: ${err.message}`);
+      // verify-A-002: the adapter THROWS on operational provider faults
+      // (429 rate-limit, 5xx outage, 401/403 token) and returns false only for
+      // a genuinely-absent run (404/transport). Emit a DISTINCT `provenance-fault:`
+      // prefix here so parseRejectionReason routes the incident to ops instead of
+      // bouncing an outage back to the submitter as submission-bad. The not-confirmed
+      // case below keeps the bare `provenance:` prefix (still submission-bad).
+      reasons.push(`provenance-fault: verification failed: ${err.message}`);
     }
-    if (!provenanceConfirmed && !reasons.some(r => r.startsWith('provenance:'))) {
+    if (!provenanceConfirmed && !reasons.some(r => r.startsWith('provenance'))) {
       reasons.push('provenance: source run could not be confirmed');
     }
   }
@@ -243,6 +249,12 @@ export async function verify(submission, options) {
   // BEFORE handing the (broken) policy to `validatePolicy`, so the
   // operator sees a real "policy:" rejection in `rejection_reasons`.
   let policyValid = false;
+  // VERIFY-F4: severity:warn / severity:info policy rules surface here as an
+  // accepted-with-warning channel. Warnings NEVER enter `reasons` (which become
+  // rejection_reasons and flip status to 'rejected') — they land on
+  // verification.warnings at assembly so an operator sees the advisory without
+  // the submission being bounced.
+  const warnings = [];
   if (schemaResult.valid) {
     if (repoPolicy && repoPolicy.__torn === true) {
       const detail = repoPolicy.reason || 'repo policy YAML failed to parse';
@@ -253,6 +265,7 @@ export async function verify(submission, options) {
       if (policyRun.ok) {
         policyValid = policyRun.result.valid;
         reasons.push(...policyRun.result.errors.map(e => `policy: ${e}`));
+        warnings.push(...(policyRun.result.warnings || []).map(w => `policy: ${w}`));
       } else {
         reasons.push(policyRun.faultReason);
       }
@@ -300,7 +313,8 @@ export async function verify(submission, options) {
       provenance_confirmed: provenanceConfirmed,
       schema_valid: schemaResult.valid,
       policy_valid: policyValid,
-      rejection_reasons: reasons
+      rejection_reasons: reasons,
+      ...(warnings.length ? { warnings } : {})
     },
     ...(submission.notes ? { notes: submission.notes } : {})
   };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dogfood-lab/verify",
-  "version": "1.5.0",
+  "version": "1.6.0",
   "type": "module",
   "description": "Central verifier for testing-os. Validates submissions against schema and policy, produces persisted records.",
   "main": "index.js",

package/parse-rejection.js

@@ -24,7 +24,9 @@
  * The prefix vocabulary below is enumerated from the ACTUAL emitters — it is
  * NOT invented:
  *   - verify/index.js:                schema:, policy:, steps[<id>]:,
- *                                     provenance:, repo:,
+ *                                     provenance: (run absent → submission-bad),
+ *                                     provenance-fault: (provider 429/5xx/401/403
+ *                                       → operational), repo:,
  *                                     submission-contains-verifier-field:,
  *                                     submission-malformed:,
  *                                     VALIDATOR_FAULT_<NAME>:
@@ -61,6 +63,14 @@ const LITERAL_PREFIXES = [
   // submission-bad
   { match: 'schema:', prefix: 'schema:', class: 'submission-bad' },
   { match: 'policy:', prefix: 'policy:', class: 'submission-bad' },
+  // operational — a provider-side provenance FAULT (429/5xx/401/403). The
+  // adapter THROWS these (verify-A-002); index.js catches the throw and emits
+  // this distinct `provenance-fault:` prefix so the incident pages ops. Ordered
+  // before the bare `provenance:` literal — it is a distinct token, but keeping
+  // the longer, more-specific match first preserves the most-specific-first
+  // invariant the array is sorted by. The genuine not-confirmed case
+  // (`provenance: source run could not be confirmed`) stays submission-bad below.
+  { match: 'provenance-fault:', prefix: 'provenance-fault:', class: 'operational' },
   { match: 'provenance:', prefix: 'provenance:', class: 'submission-bad' },
   { match: 'repo:', prefix: 'repo:', class: 'submission-bad' },
   {

package/validators/policy.js

@@ -29,6 +29,31 @@ function deepMerge(target, source) {
   return result;
 }
 
+/**
+ * Global reject-rule ids the build can account for. Two groups:
+ *   - HANDLED HERE — enforced by the switch below.
+ *   - ENFORCED ELSEWHERE — owned by another validator or by verify() itself
+ *     (the switch's `default` arm intentionally no-ops them).
+ *
+ * PROACT-VERIFY-002: any `severity: reject` rule whose id is OUTSIDE this set is
+ * an operator-added gate the build does not enforce. Silently no-op'ing it (the
+ * old `default: break`) meant the rule looked active in global-policy.yaml but
+ * never ran. We now surface an actionable diagnostic instead. Keep this set in
+ * sync when a new reject rule gains real enforcement.
+ */
+const KNOWN_REJECT_RULE_IDS = new Set([
+  // handled by the switch in validatePolicy
+  'scenario-minimum',
+  'attested-if-human',
+  'blocked-needs-reason',
+  // enforced by other validators or by verify() itself (default-arm no-op is correct)
+  'schema-valid',
+  'provenance-confirmed',
+  'step-results-present',
+  'step-verdict-consistent',
+  'no-verdict-upgrade',
+]);
+
 /**
  * Resolve the effective surface policy for a given product surface.
  * Repo policy overrides global defaults per surface.
@@ -55,16 +80,32 @@ function resolveSurfacePolicy(surface, globalPolicy, repoPolicy) {
  * @param {object} options
  * @param {object} options.globalPolicy
  * @param {object|null} options.repoPolicy
- * @returns {{ valid: boolean, errors: string[] }}
+ * @returns {{ valid: boolean, errors: string[], warnings: string[] }}
  */
 export function validatePolicy(submission, { globalPolicy, repoPolicy }) {
   const errors = [];
+  // VERIFY-F4: warn-severity rules record an accepted-with-warning note here
+  // instead of being silently dropped. A populated warnings[] never affects
+  // `valid` — the caller (index.js) routes it to verification.warnings.
+  const warnings = [];
 
   // --- Global rules (non-overridable) ---
 
   const globalRules = globalPolicy.global_rules || [];
 
   for (const rule of globalRules) {
+    // VERIFY-F4: a global warn-rule is accepted-with-warning; an info-rule logs
+    // only. Neither may reject. Mirrors the PROACT-VERIFY-002 discipline of not
+    // dropping operator-authored rules on the floor: a declared rule that the
+    // build does nothing with is invisible to the operator who wrote it.
+    if (rule.severity === 'warn') {
+      warnings.push(`${rule.id}: ${rule.description || 'policy warning'}`);
+      continue;
+    }
+    if (rule.severity === 'info') {
+      // Logged only — info rules are intentionally non-surfacing in the record.
+      continue;
+    }
     if (rule.severity !== 'reject') continue;
 
     switch (rule.id) {
@@ -95,8 +136,20 @@ export function validatePolicy(submission, { globalPolicy, repoPolicy }) {
         break;
 
       // schema-valid, provenance-confirmed, step-results-present, step-verdict-consistent,
-      // no-verdict-upgrade are enforced by other validators or the main verify() function
+      // no-verdict-upgrade are enforced by other validators or the main verify() function.
+      // PROACT-VERIFY-002: a reject rule the build neither handles here NOR enforces
+      // elsewhere would otherwise pass SILENTLY — the operator's new gate never runs.
+      // Reject the submission with a diagnostic naming the unenforced rule so the gap
+      // is visible instead of failing open.
       default:
+        if (!KNOWN_REJECT_RULE_IDS.has(rule.id)) {
+          // No `policy:` prefix here — index.js prepends it to every policy
+          // error (mirrors the `[rule.id]` / `surface[...]` messages above).
+          errors.push(
+            `rule "${rule.id}" is declared severity:reject but has no enforcement in this build — ` +
+              `add an enforcement arm in validators/policy.js or register it in KNOWN_REJECT_RULE_IDS`
+          );
+        }
         break;
     }
   }
@@ -134,6 +187,33 @@ export function validatePolicy(submission, { globalPolicy, repoPolicy }) {
           }
         }
       }
+
+      // VERIFY-F2: tag gating. forbidden_tags rejects a scenario_result carrying
+      // any listed tag; required_tags rejects one missing any listed tag (a
+      // tagless scenario fails every required_tags rule). Tags are optional on
+      // the scenario_result, so an absent `tags` array trips required_tags but
+      // never forbidden_tags.
+      const tags = new Set(sr.tags || []);
+
+      if (evidenceReqs.forbidden_tags) {
+        for (const tag of evidenceReqs.forbidden_tags) {
+          if (tags.has(tag)) {
+            errors.push(
+              `surface[${surface}]: scenario "${sr.scenario_id}" carries forbidden tag "${tag}"`
+            );
+          }
+        }
+      }
+
+      if (evidenceReqs.required_tags) {
+        for (const tag of evidenceReqs.required_tags) {
+          if (!tags.has(tag)) {
+            errors.push(
+              `surface[${surface}]: scenario "${sr.scenario_id}" is missing required tag "${tag}"`
+            );
+          }
+        }
+      }
     }
   }
 
@@ -176,5 +256,5 @@ export function validatePolicy(submission, { globalPolicy, repoPolicy }) {
     }
   }
 
-  return { valid: errors.length === 0, errors };
+  return { valid: errors.length === 0, errors, warnings };
 }

package/validators/provenance-gitlab.test.js

@@ -212,6 +212,7 @@ describe('gitlabProvenance distinguishes ops failures from missing runs', () =>
     it(`throws an operational error on HTTP ${status} (not a submission-bad false)`, async () => {
       const adapter = gitlabProvenance('token', {
         timeoutMs: 1000,
+        retries: 0,
         fetchImpl: fetchWithStatus(status)
       });
       await assert.rejects(
@@ -298,3 +299,77 @@ describe('gitlabProvenance input guards', () => {
     assert.equal(ok, false);
   });
 });
+
+// ── Bounded retry over transient faults (PROACT-VERIFY-001, mirrors GitHub) ──
+//
+// MEDIUM/resilience: a single 429/5xx blip used to fail the submission as an
+// operational incident. The adapter now retries 429/5xx within a bounded budget
+// (opts.retries, default 2) with exponential backoff (honoring Retry-After), and
+// still THROWS on exhaustion so a genuinely-down provider surfaces as
+// operational — never a false 'confirmed'. 404 is NOT retried.
+
+describe('gitlabProvenance bounded retry (PROACT-VERIFY-001)', () => {
+  it('retries a 429 then confirms on the following 200 (retry worked)', async () => {
+    let calls = 0;
+    const fetchImpl = async () => {
+      calls++;
+      if (calls === 1) {
+        return { ok: false, status: 429, headers: { get: () => null }, json: async () => ({}) };
+      }
+      return { ok: true, status: 200, json: async () => mockPipeline() };
+    };
+    const adapter = gitlabProvenance('token', { timeoutMs: 1000, backoffMs: 1, fetchImpl });
+    const ok = await adapter.confirm(pipelineSource(), { refCommitSha: RUN_HEAD });
+    assert.equal(ok, true, '429-then-200 must confirm — the retry succeeded');
+    assert.equal(calls, 2, 'expected exactly one retry (2 total requests)');
+  });
+
+  it('honors Retry-After (numeric seconds) before the retry', async () => {
+    let calls = 0;
+    const waited = [];
+    const fetchImpl = async () => {
+      calls++;
+      if (calls === 1) {
+        return { ok: false, status: 503, headers: { get: h => (h === 'retry-after' ? '2' : null) }, json: async () => ({}) };
+      }
+      return { ok: true, status: 200, json: async () => mockPipeline() };
+    };
+    const adapter = gitlabProvenance('token', {
+      timeoutMs: 1000,
+      sleepImpl: ms => { waited.push(ms); return Promise.resolve(); },
+      fetchImpl
+    });
+    const ok = await adapter.confirm(pipelineSource(), { refCommitSha: RUN_HEAD });
+    assert.equal(ok, true);
+    assert.deepEqual(waited, [2000], 'Retry-After: 2 must drive a 2000ms wait');
+  });
+
+  it('THROWS on exhausted 5xx retries (a genuinely-down provider still surfaces)', async () => {
+    let calls = 0;
+    const fetchImpl = async () => {
+      calls++;
+      return { ok: false, status: 500, headers: { get: () => null }, json: async () => ({}) };
+    };
+    const adapter = gitlabProvenance('token', { timeoutMs: 1000, retries: 2, backoffMs: 1, fetchImpl });
+    await assert.rejects(
+      adapter.confirm(pipelineSource(), { refCommitSha: RUN_HEAD }),
+      err => {
+        assert.match(err.message, /provenance: GitLab API returned 500/);
+        return true;
+      }
+    );
+    assert.equal(calls, 3, 'expected 1 initial + 2 retries = 3 requests before throwing');
+  });
+
+  it('does NOT retry a 404 (run genuinely absent → single immediate false)', async () => {
+    let calls = 0;
+    const fetchImpl = async () => {
+      calls++;
+      return { ok: false, status: 404, headers: { get: () => null }, json: async () => ({}) };
+    };
+    const adapter = gitlabProvenance('token', { timeoutMs: 1000, retries: 2, backoffMs: 1, fetchImpl });
+    const ok = await adapter.confirm(pipelineSource(), { refCommitSha: RUN_HEAD });
+    assert.equal(ok, false);
+    assert.equal(calls, 1, '404 must not be retried');
+  });
+});

package/validators/provenance.js

@@ -31,6 +31,66 @@ export const GITHUB_PROVENANCE_TIMEOUT_MS = 30000;
  */
 export const GITLAB_PROVENANCE_TIMEOUT_MS = 30000;
 
+/**
+ * Default RETRY budget for a transient provider fault (HTTP 429 rate-limit or a
+ * 5xx provider outage). PROACT-VERIFY-001: a single 429/5xx during a momentary
+ * blip used to fail the whole submission as an operational incident even though
+ * one retry would have confirmed the run. We retry a BOUNDED number of times
+ * with exponential backoff (honoring `Retry-After` when the provider sends it),
+ * then THROW on exhaustion so a genuinely-down provider still surfaces as an
+ * operational signal — never a false 'confirmed'. 404 is NOT retried (the run is
+ * genuinely absent, a single immediate rejection).
+ *
+ * `retries` is the number of ADDITIONAL attempts after the first, so the default
+ * 2 means up to 3 total requests. It is an opts field (like `timeoutMs`) so it
+ * stays test-injectable and the offline stub path is unaffected.
+ */
+export const PROVENANCE_RETRIES = 2;
+
+/**
+ * Base backoff between retry attempts, in ms. Attempt N waits
+ * `PROVENANCE_BACKOFF_MS * 2^(N-1)` (250 → 500 → …), unless the provider sent a
+ * `Retry-After` header, which takes precedence. Injectable via opts so tests run
+ * without real delay.
+ */
+export const PROVENANCE_BACKOFF_MS = 250;
+
+/** HTTP statuses worth retrying: 429 rate-limit + the 5xx provider-outage band. */
+function isRetryableStatus(status) {
+  return status === 429 || (status >= 500 && status <= 599);
+}
+
+/**
+ * Resolve the wait before the next attempt. A `Retry-After` header (delay in
+ * seconds, or an HTTP-date) is honored when present and parseable; otherwise
+ * fall back to exponential backoff. `attempt` is 1-based (1 = wait before the
+ * 2nd request).
+ *
+ * @param {Response} resp - The non-ok response carrying a possible Retry-After.
+ * @param {number} attempt - 1-based retry index.
+ * @param {number} backoffMs - Base backoff.
+ * @returns {number} Milliseconds to wait (never negative).
+ */
+function nextBackoffMs(resp, attempt, backoffMs) {
+  const header = resp?.headers?.get?.('retry-after');
+  if (header != null && header !== '') {
+    const asSeconds = Number(header);
+    if (Number.isFinite(asSeconds) && asSeconds >= 0) {
+      return Math.round(asSeconds * 1000);
+    }
+    const asDate = Date.parse(header);
+    if (!Number.isNaN(asDate)) {
+      return Math.max(0, asDate - Date.now());
+    }
+  }
+  return backoffMs * 2 ** (attempt - 1);
+}
+
+/** Default sleep: a real timer. Injectable via `opts.sleepImpl` for tests. */
+function defaultSleep(ms) {
+  return new Promise(resolve => setTimeout(resolve, ms));
+}
+
 /**
  * Stub provenance adapter. Always confirms.
  * Use in tests and local development.
@@ -62,6 +122,9 @@ export const rejectingProvenance = {
 export function githubProvenance(token, opts = {}) {
   const timeoutMs = opts.timeoutMs ?? GITHUB_PROVENANCE_TIMEOUT_MS;
   const fetchImpl = opts.fetchImpl ?? fetch;
+  const retries = opts.retries ?? PROVENANCE_RETRIES;
+  const backoffMs = opts.backoffMs ?? PROVENANCE_BACKOFF_MS;
+  const sleep = opts.sleepImpl ?? defaultSleep;
   return {
     /**
      * @param {object} source - submission.source (provider-scoped run claim)
@@ -96,54 +159,57 @@ export function githubProvenance(token, opts = {}) {
 
       const apiUrl = `https://api.github.com/repos/${owner}/${repo}/actions/runs/${provider_run_id}`;
 
-      // Per-request timeout. Without this, a hung GitHub API call (rate-limit
-      // throttle, regional outage, slow connection) blocks ingest indefinitely.
-      // AbortController fires AbortError on timeout — we re-throw with a clear
-      // message so the verifier records it in rejection_reasons instead of
-      // silently treating it as 'provenance returned false.'
-      const controller = new AbortController();
-      const timer = setTimeout(() => controller.abort(), timeoutMs);
-
+      // PROACT-VERIFY-001: bounded retry over transient provider faults (429
+      // rate-limit, 5xx outage). Each attempt carries its own per-request
+      // AbortController timeout — without it a hung GitHub API call blocks ingest
+      // indefinitely. We retry up to `retries` extra times with exponential
+      // backoff (honoring Retry-After), then THROW on exhaustion so a
+      // genuinely-down provider surfaces as an operational signal — never a
+      // false 'confirmed'. 404 is NOT retried (run genuinely absent). The
+      // earlier non-retry behavior (single 429/5xx → immediate throw) failed a
+      // submission on a momentary blip that one retry would have confirmed.
       let run;
-      try {
-        const resp = await fetchImpl(apiUrl, {
-          headers: {
-            Authorization: `Bearer ${token}`,
-            Accept: 'application/vnd.github+json',
-            'X-GitHub-Api-Version': '2022-11-28'
-          },
-          signal: controller.signal
-        });
-
-        if (!resp.ok) {
-          // verify-A-002: only 404 means the run is genuinely absent — a real
-          // submission-bad rejection. Every other non-2xx is an OPERATIONAL
-          // signal (401/403 expired/insufficient token, 429 rate limit, 5xx
-          // GitHub outage). Collapsing those into `return false` made the
-          // verifier record 'source run could not be confirmed' and routing
-          // bounced an ops incident to submitters. Throw so the existing catch
-          // in index.js records it as a provenance verification failure
-          // (operational), mirroring the timeout fix above.
-          if (resp.status === 404) return false;
-          throw new Error(`provenance: GitHub API returned ${resp.status}`);
+      for (let attempt = 0; ; attempt++) {
+        const controller = new AbortController();
+        const timer = setTimeout(() => controller.abort(), timeoutMs);
+
+        let resp;
+        try {
+          resp = await fetchImpl(apiUrl, {
+            headers: {
+              Authorization: `Bearer ${token}`,
+              Accept: 'application/vnd.github+json',
+              'X-GitHub-Api-Version': '2022-11-28'
+            },
+            signal: controller.signal
+          });
+        } catch (err) {
+          if (err && (err.name === 'AbortError' || err.code === 'ABORT_ERR')) {
+            throw new Error(`provenance: GitHub API timeout after ${timeoutMs}ms`);
+          }
+          // Genuine transport error (DNS, connection refused) — no run to
+          // confirm. Reserve `return false` for this case (mirrors GitLab).
+          return false;
+        } finally {
+          clearTimeout(timer);
         }
 
-        run = await resp.json();
-      } catch (err) {
-        if (err && (err.name === 'AbortError' || err.code === 'ABORT_ERR')) {
-          throw new Error(`provenance: GitHub API timeout after ${timeoutMs}ms`);
+        if (resp.ok) {
+          run = await resp.json();
+          break;
         }
-        // verify-A-002: operational signals we raised ourselves (non-2xx HTTP)
-        // already carry the 'provenance:' prefix — re-throw them so they reach
-        // index.js as a verification failure rather than being swallowed into
-        // 'return false'. Reserve `return false` for genuine transport errors
-        // (DNS, connection refused) where there is no run to confirm.
-        if (err && typeof err.message === 'string' && err.message.startsWith('provenance:')) {
-          throw err;
+
+        // verify-A-002: only 404 means the run is genuinely absent — a real
+        // submission-bad rejection, never retried. 429/5xx are transient
+        // OPERATIONAL signals: retry within budget. 401/403 (expired/insufficient
+        // token) are NOT transient — throw immediately. On exhausted retries the
+        // last 429/5xx throws so a real outage still surfaces (operational).
+        if (resp.status === 404) return false;
+        if (isRetryableStatus(resp.status) && attempt < retries) {
+          await sleep(nextBackoffMs(resp, attempt + 1, backoffMs));
+          continue;
         }
-        return false;
-      } finally {
-        clearTimeout(timer);
+        throw new Error(`provenance: GitHub API returned ${resp.status}`);
       }
 
       if (run.id !== Number(provider_run_id)) return false;
@@ -211,6 +277,9 @@ export function gitlabProvenance(token, opts = {}) {
   const timeoutMs = opts.timeoutMs ?? GITLAB_PROVENANCE_TIMEOUT_MS;
   const fetchImpl = opts.fetchImpl ?? fetch;
   const apiBase = (opts.apiBase ?? 'https://gitlab.com').replace(/\/+$/, '');
+  const retries = opts.retries ?? PROVENANCE_RETRIES;
+  const backoffMs = opts.backoffMs ?? PROVENANCE_BACKOFF_MS;
+  const sleep = opts.sleepImpl ?? defaultSleep;
   return {
     /**
      * @param {object} source - submission.source (provider-scoped run claim)
@@ -252,46 +321,50 @@ export function gitlabProvenance(token, opts = {}) {
         ? `${apiBase}/api/v4/projects/${projectId}/jobs/${provider_run_id}`
         : `${apiBase}/api/v4/projects/${projectId}/pipelines/${provider_run_id}`;
 
-      // Per-request timeout — identical discipline to githubProvenance. Without
-      // it a hung gitlab.com/api call blocks ingest until the surrounding runner
-      // times out. AbortError → re-thrown with a clear 'provenance:' message so
-      // the verifier records it instead of silently returning false.
-      const controller = new AbortController();
-      const timer = setTimeout(() => controller.abort(), timeoutMs);
-
+      // PROACT-VERIFY-001: bounded retry over transient provider faults — same
+      // discipline as githubProvenance. Each attempt carries its own per-request
+      // AbortController timeout. 429/5xx retry within budget (exponential backoff,
+      // honoring Retry-After), then THROW on exhaustion so a real outage still
+      // surfaces as operational — never a false 'confirmed'. 404 is a single
+      // immediate rejection; 401/403 throw immediately (not transient).
       let run;
-      try {
-        const resp = await fetchImpl(endpoint, {
-          headers: {
-            'PRIVATE-TOKEN': token,
-            Accept: 'application/json'
-          },
-          signal: controller.signal
-        });
-
-        if (!resp.ok) {
-          // Mirror verify-A-002: only 404 means the pipeline/job is genuinely
-          // absent (submission-bad). 401/403/429/5xx are OPERATIONAL — throw so
-          // the catch in index.js records a 'provenance:' verification failure
-          // (paged to ops) instead of bouncing an outage to the submitter.
-          if (resp.status === 404) return false;
-          throw new Error(`provenance: GitLab API returned ${resp.status}`);
+      for (let attempt = 0; ; attempt++) {
+        const controller = new AbortController();
+        const timer = setTimeout(() => controller.abort(), timeoutMs);
+
+        let resp;
+        try {
+          resp = await fetchImpl(endpoint, {
+            headers: {
+              'PRIVATE-TOKEN': token,
+              Accept: 'application/json'
+            },
+            signal: controller.signal
+          });
+        } catch (err) {
+          if (err && (err.name === 'AbortError' || err.code === 'ABORT_ERR')) {
+            throw new Error(`provenance: GitLab API timeout after ${timeoutMs}ms`);
+          }
+          // Genuine transport error (DNS, connection refused) — no run to confirm.
+          return false;
+        } finally {
+          clearTimeout(timer);
         }
 
-        run = await resp.json();
-      } catch (err) {
-        if (err && (err.name === 'AbortError' || err.code === 'ABORT_ERR')) {
-          throw new Error(`provenance: GitLab API timeout after ${timeoutMs}ms`);
+        if (resp.ok) {
+          run = await resp.json();
+          break;
         }
-        // Operational signals we raised ourselves already carry the
-        // 'provenance:' prefix — re-throw. Reserve `return false` for genuine
-        // transport errors (DNS, connection refused) where there is no run.
-        if (err && typeof err.message === 'string' && err.message.startsWith('provenance:')) {
-          throw err;
+
+        // Mirror verify-A-002: 404 = pipeline/job genuinely absent (submission-bad,
+        // never retried). 429/5xx = transient operational fault, retry within
+        // budget. 401/403 = non-transient operational, throw immediately.
+        if (resp.status === 404) return false;
+        if (isRetryableStatus(resp.status) && attempt < retries) {
+          await sleep(nextBackoffMs(resp, attempt + 1, backoffMs));
+          continue;
         }
-        return false;
-      } finally {
-        clearTimeout(timer);
+        throw new Error(`provenance: GitLab API returned ${resp.status}`);
       }
 
       // id in the response must match the claimed run id.