summaryrefslogtreecommitdiff
path: root/.github
diff options
context:
space:
mode:
authorYuriy Andamasov <yuriy@vyos.io>2026-05-11 09:23:15 +0300
committerYuriy Andamasov <yuriy@vyos.io>2026-05-11 09:23:15 +0300
commita0a9c004aa521eb4a90d71c25e5bb1dd9e5cde6f (patch)
tree00af4dcab9ff32ad7039e3488f1ee554c00d3b09 /.github
parent82a900681a50eecfd67fa1fb82e9ea973e04ffd6 (diff)
downloadvyos-documentation-a0a9c004aa521eb4a90d71c25e5bb1dd9e5cde6f.tar.gz
vyos-documentation-a0a9c004aa521eb4a90d71c25e5bb1dd9e5cde6f.zip
ci(ai-validation): align fail-closed-gate comment + bump REVIEWER_REF to v1.0.2
Two small follow-ups now that the DB-pin revert wave (#1971/#1972/#1973 + canonical #15) has settled and `reviewer-v1.0.2` is tagged: 1. Comment alignment. The fail-closed-gate rationale comment that landed in #1971/#1972 says "the new code will refuse to load an older-format DB at the fail-closed gate". A Copilot finding on #1973 pointed out this is imprecise: the workflow's fail-closed gate is a presence check (`[ ! -d .reference-db/extracted ]`); it does not load or inspect the DB. The actual schema-mismatch protection lives inside the pinned reviewer Python code at `Pass 1 — deterministic checks` (loaded via `uv pip install ./reviewer-src` from `env.REVIEWER_REF`). The fix landed on sagitta (#1973 / b09ea673). This commit ports the same wording to this branch so the workflow file is byte-identical across rolling/circinus/sagitta again. 2. REVIEWER_REF bump v1.0.1 -> v1.0.2. The canonical at `VyOS-Networks/vyos-docs-opus-reviewer/scripts/ai-validation.yml` already defaults to v1.0.2 since the canonical-sync PR #15 merged and the `reviewer-v1.0.2` tag was pushed. v1.0.2 is functionally identical to v1.0.1 — the Python package and `branches.json` are unchanged; the bump just aligns this deployed copy with the canonical default for hygiene. No behavioral change.
Diffstat (limited to '.github')
-rw-r--r--.github/workflows/ai-validation.yml9
1 files changed, 6 insertions, 3 deletions
diff --git a/.github/workflows/ai-validation.yml b/.github/workflows/ai-validation.yml
index c19f7737..c0f70b9c 100644
--- a/.github/workflows/ai-validation.yml
+++ b/.github/workflows/ai-validation.yml
@@ -13,7 +13,7 @@ concurrency:
cancel-in-progress: true
env:
- REVIEWER_REF: reviewer-v1.0.1
+ REVIEWER_REF: reviewer-v1.0.2
# Force JavaScript actions to run on Node 24. Some pinned action SHAs
# we rely on still ship with Node 20 ABI; this env var opts the whole
# workflow into Node 24 without per-action version churn.
@@ -331,8 +331,11 @@ jobs:
# DB assets). The reference DB schema is intentionally backwards
# compatible across reviewer Python releases; the freshest DB is
# the right choice. If the DB schema ever changes incompatibly,
- # bump REVIEWER_REF in the consuming workflow and the new code
- # will refuse to load an older-format DB at the fail-closed gate.
+ # bump REVIEWER_REF in the consuming workflow — the newer
+ # reviewer code will error at DB-load time inside `Pass 1 —
+ # deterministic checks`. The workflow's own fail-closed gate is
+ # a presence check (`[ -s diff-md.patch ] && [ ! -d
+ # .reference-db/extracted ]`); it does not inspect DB schema.
latest: true
fileName: reference-db-${{ steps.branch.outputs.vyos1x }}.tar.gz
out-file-path: .reference-db