Repository navigation
feat(harness): add PR template and REVIEW.md - #30
Merged
Merged
Conversation
The reviewer reads REVIEW.md when a repository has one and uses it as the single reference for Important, Nit, and Pre-existing. Otherwise it keeps the default tag rules. /setup-harness offers the pull request template (Risk, How to revert, Tier) to projects that do not have one. Co-authored-by: Florian <fmflurry@users.noreply.github.com>
fmflurry
added a commit
that referenced
this pull request
Oct 18, 2026
* docs(readme): document shipped harness changes and ship Record #25 through #30 from the tree, including both no-progress checks, and state how /ship is launched in each harness. Co-authored-by: Florian <fmflurry@users.noreply.github.com> * docs(readme): record the rules diet in recent changes #32 cut always-applied Cursor rules from 21 files to 7. The verification skill and the socratic spike stay unshipped. Co-authored-by: Florian <fmflurry@users.noreply.github.com> --------- Co-authored-by: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Florian <fmflurry@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Step 3 of the harness action plan. When a repository has
REVIEW.md, the reviewer reads it and uses it as the single reference for the tag mapping (Important, Nit, Pre-existing). The report cites that file. WhenREVIEW.mdis absent, the reviewer says so and uses the default tag rules incode-reviewer..github/pull_request_template.mdadds Risk, How to revert, and Tier./prand/create-pull-requestreadREVIEW.mdand fill that template./setup-harnessoffers the same template to a project that does not already have one. Claude, OpenCode, and the regenerated Cursor plugin all carry the read rule. Plugin version is 0.4.2 (patch only).Always-loaded Cursor rules: 1438 → 1438 lines (delta 0), still 21 always-applied files. No always-on rule was added or edited. The new text is in the code-reviewer agent, the
/prcommand, and the setup-harness skill.Risk
A repository that adds its own
REVIEW.mdreplaces the default tag table. A shorter or different file changes what blocks a merge. This repository'sREVIEW.mdrestates the current default, so reviews here keep the same tags and must cite the file.How to revert
Revert
617d782. That restores plugin 0.4.1 and removesREVIEW.md,.github/pull_request_template.md, and the/prcommand.Tier
T2. The diff is harness instructions and the generated plugin. It does not touch application code, secrets, or a protected path. T3 would need a cross-check and a human review; this does not.
Checks
node scripts/cursor-plugin/check.mjs:cursor-plugin check OKbash scripts/agent-harness/test/run.sh:tests: 1269 passed, 0 failednpm test: 68 passed, 2 skipped, 1 failed. The failure is the known pre-existing one,OpenCode local install syncs the repository config payload exactly.npx tsc --noEmit --pretty false --project tsconfig.strict.json: exit 0