fix: Gitea dogfood — Node without setup-node mirror + PyYAML venv #2

Merged
layue13 merged 2 commits from cursor/dogfood-runner-fixes-165a into main 2026-10-27 08:06:31 +00:00
Owner

Unblocks self-review on this forge: setup-node is not mirrored (404); pip --user hits PEP 668. Optional setup-node + require runner Node; install PyYAML via workspace venv.

Unblocks self-review on this forge: setup-node is not mirrored (404); pip --user hits PEP 668. Optional setup-node + require runner Node; install PyYAML via workspace venv.
layue13 added 1 commit 2026-10-27 08:02:19 +00:00
fix: run without setup-node mirror; install PyYAML via venv
Gatekeep evals / eval-floor (push) Successful in 6s
Gatekeep evals / eval-floor (pull_request) Successful in 3s
gatekeep / security PR risk high — no open security blockers
Gatekeep / gatekeep (pull_request) Failing after 1m33s
7b25fefdaa
Gitea dogfood failed because actions/setup-node is not mirrored
(404 on git.potatolab.uk/actions/setup-node) and pip --user is
blocked by PEP 668. Prefer runner Node with an optional mirrored
setup-node, and install PyYAML into a workspace venv.

Gatekeep review

Hey — here's the multi-model pass. Severity first so you can skim Critical → Low in seconds.

PR risk tier high
Policy floor low
Agreement bar ≥2 models
Agreed / disputed 0 / 0
Critical / High (agreed) 0 / 0

Needs 2 human approvals. Bots never fill those seats.

Models (parallel)

Model Their PR tier Findings
model-1-openrouter_google_gemma-4-31b-it_free.json high 0 (err: http)
model-2-openrouter_google_gemma-4-26b-a4b-it_free.json high 0 (err: http)
openrouter:cohere/north-mini-code:free low 0

Agreed findings

Sorted by severity. Each one spells out severity, where, and why it matters.
Fix or dismiss with dismissed: <reason> (then resolve the thread). Gatekeep has no chat / @-command surface.

Nothing the models agreed on — nice, or spooky. Glance at disputed below.

Disputed (summary only — not posted inline)

One model saw it; others didn't. Useful signal, not a merge blocker by itself.

No disagreements worth parking.

Deterministic security scan

No hard-fail hits.

Policy floor signals

  • none

Still on humans

  • Ensure .gatekeep/.venv is ignored by git to keep the repo clean.

Missed something real? Open a gatekeep-incident issue — that's how it becomes an eval.
Inline comments only when models agree. PR tier is fail-closed. Findings lead with severity + location + why.

<!-- gatekeep-review --> ## Gatekeep review Hey — here's the multi-model pass. **Severity first** so you can skim Critical → Low in seconds. | | | | --- | --- | | **PR risk tier** | `high` | | Policy floor | `low` | | Agreement bar | ≥2 models | | Agreed / disputed | 0 / 0 | | Critical / High (agreed) | 0 / 0 | Needs **2 human** approvals. Bots never fill those seats. ### Models (parallel) | Model | Their PR tier | Findings | | --- | --- | --- | | `model-1-openrouter_google_gemma-4-31b-it_free.json` | `high` | 0 (err: http) | | `model-2-openrouter_google_gemma-4-26b-a4b-it_free.json` | `high` | 0 (err: http) | | `openrouter:cohere/north-mini-code:free` | `low` | 0 | ### Agreed findings Sorted by severity. Each one spells out **severity**, **where**, and **why it matters**. Fix or dismiss with `dismissed: <reason>` (then resolve the thread). Gatekeep has **no** chat / @-command surface. _Nothing the models agreed on — nice, or spooky. Glance at disputed below._ ### Disputed (summary only — not posted inline) One model saw it; others didn't. Useful signal, not a merge blocker by itself. _No disagreements worth parking._ ### Deterministic security scan _No hard-fail hits._ ### Policy floor signals - none ### Still on humans - Ensure .gatekeep/.venv is ignored by git to keep the repo clean. --- Missed something real? Open a **gatekeep-incident** issue — that's how it becomes an eval. _Inline comments only when models agree. PR tier is fail-closed. Findings lead with severity + location + why._
layue13 added 1 commit 2026-10-27 08:06:30 +00:00
fix: drop setup-node uses; fix GITHUB_OUTPUT python quoting
Gatekeep evals / eval-floor (push) Successful in 2s
Gatekeep evals / eval-floor (pull_request) Successful in 2s
gatekeep / security PR risk high — no open security blockers
Gatekeep / gatekeep (pull_request) Successful in 1m12s
7a581254bf
Remove the optional setup-node step (still 404s as ##[error] on this
forge). Fix review.sh output writer — escaped quotes in a one-liner
caused SyntaxError after an otherwise successful publish.
layue13 merged commit a4fcfee1b3 into main 2026-10-27 08:06:31 +00:00
Sign in to join this conversation.
No Reviewers
No labels
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: layue13/gatekeep#2