fix(ci): preserve browser evidence and pin source freshness - #95
Conversation
Run documentation browser certification independently of source freshness, while preserving the canonical fail-closed release rollup. Compare watched paths in pinned Git trees rather than timestamp-filtered moving refs. Add 83 dependency-free CLI and workflow regressions. Refs #89; this does not advance the source lock or close the outstanding public-source review.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Exact-head hosted verification receiptRun 34814168620 exercised head
Downloaded and independently checked both artifact ZIP SHA-256 digests:
This supersedes the initial PR body's pending hosted-verification status, not its source-review limit. The topology repair now has real browser execution evidence and real fail-closed rollup evidence. #89 still needs substantive review of those four source paths against affected public pages, followed by a truthful lock update and fresh successful canonical CI. Do not merge this PR by ignoring that remaining gate. |
There was a problem hiding this comment.
🔵 Needs a closer look
Full frozen-install, browser, and live drift gates were not run in this sandbox.
Pull request overview
This PR repairs documentation CI evidence and source-freshness detection while preserving a strict release gate.
Changes:
- Separates freshness and browser verification jobs.
- Pins drift checks to immutable Git tree identities.
- Adds regression coverage and documents evidence semantics.
File summaries
| File | Summary |
|---|---|
scripts/docs-release-gate.test.mjs |
Tests release-gate outcomes and workflow wiring. |
scripts/docs-release-gate.mjs |
Requires both verification jobs to succeed. |
scripts/check-source-drift.test.mjs |
Adds source-drift regression coverage. |
scripts/check-source-drift.mjs |
Implements pinned, fail-closed tree comparison. |
scripts/check-automation-syntax.mjs |
Runs automation regression tests. |
docs/e2e-source-freshness.md |
Documents evidence and freshness semantics. |
.github/workflows/docs.yml |
Adds independent verification jobs and canonical rollup. |
Review details
- Files reviewed: 7/7 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Scope
Refs #89; closes no issue. Repair the documentation E2E/CI evidence path without weakening source freshness or publication acceptance.
Base:
e0860cd68030248fb829b00bc32d684fd1720ff2Head:
44f10e33604c2abfb659baca965006a0c2e545f7Observed failures
103474663562, installed dependencies successfully, then failed source freshness. Chrome installation andpnpm verifywere skipped. That run does not establish a browser regression in dependency PR chore(deps): bump the next-react group across 1 directory with 2 updates #90.verifiedAtas a commit-history filter and repeatedly queried movingmain. A late-merged old-dated change can evade that timestamp filter; separate reads can mix revisions.comparison.head_commitalso does not reliably establish the target identity.Changes
success. Missing, failed, cancelled, skipped, or pending outcomes fail. No continue-on-error, weakened assertion, new quarantine, or release waiver.changesnow represents current path-identity differences, not exhaustive historical chronology. A reverted path with exactly the reviewed identity is unchanged for current docs purposes; this does not certify deployed behavior or historical events.check:automation/pnpm verify.Exactly seven changed files. No dependency manifests, source-lock values, public product claims, runtime behavior, branch protections, deployment controls, or release credentials changed.
Verification actually executed
c224c745753472c82ea2893b511999c95f3db86d.node --test scripts/check-source-drift.test.mjs scripts/docs-release-gate.test.mjs: 83 passed, 0 failed, 0 skipped/cancelled on Node 22.16.0.node --checkfor all changed/new JavaScript modules: passed.These are real subprocess executions against fixture-controlled GitHub responses, not live upstream verification. The 83 cases comprise 33 source-drift cases, all 49 pairings of seven gate outcomes, and one workflow wiring contract.
Remaining gates and review
Full frozen install,
pnpm verify, Chrome/browser certification and live GitHub drift detection have not run in this sandbox; package/network access is unavailable. Inspect exact-head CI before claiming those outcomes. Source freshness is expected to stay red until #89's substantive review is completed.Do not merge by bypassing the canonical gate. Review the workflow topology and the shift from timestamp-history to current pinned-tree semantics. Then complete the separately scoped source-to-page review and truthful
verifiedCommit/verifiedAtupdate; this PR does not supply or fabricate that review.This is an API-authored scoped review branch under Val's instruction to patch E2E blockers. It does not modify any existing agent worktree or claim a canonical Bead registration, host census, release, or deployment. Existing operational/security gates remain open.