agent-dispatch: test, then push; a local-only commit is not done

[asubbarao]

Replaces the house rule "nothing pushed, no PR, nothing another human can see, without his approval" with Test, then push: run the tests first, push only a green branch, open a ready PR with before/after test evidence. A commit that exists only locally is not done, and an untested push or PR is not allowed. Comments on other people's repos still need approval.

Docs-only change (skills/agent-dispatch/SKILL.md); no server file changes.

— Opus 5.5 (Claude Code), for Alok Subbarao

🤖 Generated with Claude Code

[asubbarao]

duckstack/review: failure

• skills/agent-dispatch/SKILL.md:138 Section 4 of this file plus the CLAUDE.md house rule 'Evidence a teammate may see comes from CI/GitHub, never from local logfiles'; RULES.md 26-27: The new rule asks for before/after test evidence in the PR body but does not say where it comes from. Read as a pasted local run, it contradicts section 4 of the same file: local runs are for iteration only, a PR body is something a teammate sees, and section 4 requires false-first evidence (mutant, failing output, restored passing output) read from CI.
  Rewrite: - Test, then push. Run the tests locally first and push only a branch whose tests passed. Then open a ready (never draft) PR into the owner's repo. For before/after evidence in the body, link the CI runs and checks (gh run view --json read through shellfs) and give the false-first triple from section 4: the mutant, its failing CI output and the restored passing output. Never paste a local pytest | tail. The PR reaches main only through the review queue. A commit that exists only on this laptop is not done, and an untested push or PR is not allowed. Comments on other people's repos still need his yes.

Opus 5.5, duckstack review queue (duckstack-review skill)

[asubbarao]

duckstack/review: success

No findings.

Opus 5.5, duckstack review queue (duckstack-review skill)