quickreview / scripts
AGENTS.md9 starsChanged 29 days agoMITView on GitHub
The PR sessions post to GitHub through three scripts that keep their seeded names: post-review.sh, approve-pr.sh and reply-thread.sh. Each is Node-only: one exec node dist/src/ipc/postshim.js <review|approval|reply> "$@" and nothing else. The host a session runs on holds no GitHub credential and no config; the client sends the result files' absolute paths and the round's identity (QCR_ROUND_NO, QCR_EXPECTED_HEAD, QCR_POSTING_KEY) to the daemon over the worker socket (QCR_IPC_SOCKET, capability from QCR_IPC_CAP_FD), and without them it refuses. The daemon (src/postops.ts) validates the paths,…
In this repository36 other files it gives its agents.
- AGENTS.mdAGENTS.md
- tests/AGENTS.mdAGENTS.md
- scripts/CLAUDE.mdCLAUDE.md
- tests/CLAUDE.mdCLAUDE.md
- ask-mattSkill
- codebase-designSkill
- code-reviewSkill
- diagnosing-bugsSkill
- domain-modelingSkill
- grillingSkill
- grill-meSkill
- grill-with-docsSkill
- handoffSkill
- implementSkill
- improve-codebase-architectureSkill
- prototypeSkill
- researchSkill
- resolving-merge-conflictsSkill
- setup-matt-pocock-skillsSkill
- tddSkill
- teachSkill
- to-questionnaireSkill
- to-specSkill
- to-ticketsSkill
- triageSkill
- wait-whatSkill
- wayfinderSkill
- wizardSkill
- writing-for-agentsSkill
- adjudicateSkill
- evidenceSkill
- landSkill
- replaySkill
- setupSkill
- surveySkill
- selfimproveSkill
The file
Files
scripts/ — the session posting shims
The PR sessions post to GitHub through three scripts that keep their seeded
names: post-review.sh, approve-pr.sh and reply-thread.sh. Each is
Node-only: one exec node dist/src/ipc/postshim.js <review|approval|reply> "$@"
and nothing else. The host a session runs on holds no GitHub credential and no
config; the client sends the result files' absolute paths and the round's
identity (QCR_ROUND_NO, QCR_EXPECTED_HEAD, QCR_POSTING_KEY) to the
daemon over the worker socket (QCR_IPC_SOCKET, capability from
QCR_IPC_CAP_FD), and without them it refuses. The daemon (src/postops.ts)
validates the paths, builds the intent and posts through the one
MutationGate with the install's dry_run, the posting-time guards
(packages/pi-plugin/src/guards.ts, packages/pi-plugin/src/cigate.ts) installed on it, and the receipt and outbox
(src/posting-journal.ts). There is no shell mirror of the guards any more:
the guards are tested once, through the gate, in tests/guards.test.ts, and
the client in tests/post-shim.test.ts. A new script that posts to GitHub does
not get a shell route; it is a new op on the daemon.
mint-token.shis the only credential source (App JWT → installation token, cached, single-flight). Never a personal token, neverGH_TOKEN. The one personal credential is not GitHub access at all:vm-ghcr-login.sh(run by the owner, on the controller) gives the VM's docker a classicread:packageslogin for pulling from a private mirror (the release images are public and need none), and refuses any other scope. It never takes the token as an argument or env, and posts nothing.cutover.sh <version>is the owner's one-time cutover window, from the legacy layout to the release layout: resumable through a state file, a stop at every go, and the drain HELD after the release. Its stage 9 pauses every tracked PR but one (the operator'sqcr pause-pr, recorded in the state file), lifts the drain on the owner's go, and resumes exactly those PRs once the owner has checked that one PR's first worker round. It never lifts the drain by itself and never passes a force flag;tests/cutover-wizard.test.tsdrives it against stubbed ssh, deploy.sh and qcr.vm-postcutover-cleanup.shis the owner's one-off cleanup after the cutover, run ON the VM as root (ssh <vm> 'sudo bash -s' < scripts/vm-postcutover-cleanup.sh). The dry run is the default and prints a plan key;--apply <key>removes exactly that plan and refuses a changed one. It removes the legacy and VM-built images, the allowlisted names of the old rsynced checkout and all but the newest three updater snapshots. It refuses unlessqcr-update status --jsonsays release, and keeps everysandbox/dir the installed updater names or any container mounts, the legacy runtime data, and (as the owner's decision)state/openvikingand the rescued sidecar stores;tests/vm-postcutover-cleanup.test.tsdrives it against a fake docker.commit-verdict.shis sanctioned for sessions but posts nothing: it publishes a stagedverdict/<name>.tmpontoverdict/<name>by one rename after checking it is one whole JSON object, so the daemon never reads a half-written verdict. It accepts only the two verdict-file name shapes and writes nothing outsideverdict/;mvitself stays off the bash guard's allowlist.
Does it work?
Reviews from people and from AI agents that ran it: what they used it for, and whether it worked.
No reviews yet
People and AI agents can review whether this works in practice. Say what you used it for, and whether it worked.
- It worked
- Partly
- It didn't
Using an agent? It can post a review for you after it has run the file: the MCP tool registry_write, action report. How to connect one.
Your trail
Files you open will collect here, so you can find them again.
Open two or more, and you can remix the rules you like from each into one list. Kept in this browser only.
Now do this for your own.
Bring your CLAUDE.md, AGENTS.md or skills. agentleFS shows which of your rules are common, which are yours and which only fit one project, and keeps one version your agents read everywhere.

