Skip to content

fix(adk): the last runbook lines that said test jobs run now say none does yet (115f follow-up) - #504

Draft
LamaSu wants to merge 1 commit into
feat/adk-runbookfrom
fix/adk-runbook-stale-test-job-lines
Draft

LamaSu wants to merge 1 commit into
feat/adk-runbookfrom
fix/adk-runbook-stale-test-job-lines

Conversation

@LamaSu

@LamaSu LamaSu commented Oct 3, 2026

Copy link
Copy Markdown
Owner

The tracked follow-up to verdict 115f on #464 (SHIP, with one LOW). Stacked on #464.

The LOW: three lines still implied that test jobs run:

  • 00-prerequisites.md and intake's asks-about-pay event said the runbook "runs test jobs only";
  • 05-register.md said phase 6 "exercises the device".

Phase 6 now drills the emergency stop and runs no job until the gateway offers a queue-only submission, so these lines now say that. No job carries money, and on the current gateway no job runs at all.

Test: the round-5 no-summary check also refuses "runs test jobs only" and "phase 6 exercises". It fails at 9d4653b (#464's head) and passes here. The starter tests pass.

Review: pack 115g (delta from 9d4653b).

🤖 Generated with Claude Code

… does yet (verdict 115f LOW)

00-prerequisites.md and intake's asks-about-pay event said the runbook
"runs test jobs only", and 05-register.md said phase 6 "exercises the
device". Phase 6 now drills the emergency stop and runs no job until the
gateway offers a queue-only submission, so they say that: no job carries
money, and on the current gateway no job runs at all.

Test: the no-summary check also refuses "runs test jobs only" and
"phase 6 exercises"; it fails at 9d4653b and passes here.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant