diff options
| author | Yuriy Andamasov <yuriy@vyos.io> | 2026-05-11 09:23:15 +0300 |
|---|---|---|
| committer | Yuriy Andamasov <yuriy@vyos.io> | 2026-05-11 09:23:15 +0300 |
| commit | a0a9c004aa521eb4a90d71c25e5bb1dd9e5cde6f (patch) | |
| tree | 00af4dcab9ff32ad7039e3488f1ee554c00d3b09 /.github/workflows | |
| parent | 82a900681a50eecfd67fa1fb82e9ea973e04ffd6 (diff) | |
| download | vyos-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/workflows')
| -rw-r--r-- | .github/workflows/ai-validation.yml | 9 |
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 |
