Skip to content

Commit b3b3dd2

Browse files
authored
Merge pull request #6957 from Systemorph/docs/finish-autonomously
docs(agents): one fleet-wide 'finish autonomously' rule block
2 parents 125245a + f119dc2 commit b3b3dd2

2 files changed

Lines changed: 25 additions & 0 deletions

File tree

‎.github/shared-rules.json‎

Lines changed: 21 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -377,6 +377,27 @@
377377
"Landing in the three change sets this register's header describes: step 1 (this entry) names",
378378
"the hub only; step 2 lands the six spoke copies; step 3 widens required-in to all seven."
379379
]
380+
},
381+
{
382+
"id": "finish-autonomously",
383+
"hub": "Systemorph/MeshWeaver",
384+
"required-in": [
385+
"Systemorph/MeshWeaver"
386+
],
387+
"slots": [],
388+
"why": [
389+
"Maintainer directives, 2026-09-09 ('ALWAYS bring requested fixes to production … Do not ask for a second",
390+
"ship-it instruction') and 2026-10-18 ('see that it is all being merged automatically. see that in central",
391+
"place it is clearly stated to finish autonomously'). Before this block the rule existed as three",
392+
"differently-worded passages — core's Git Workflow and GitHub PR Operations paragraphs and Plugins'",
393+
"'Ship it' / 'At the end' sections — so each repo's agents stopped at a different stage.",
394+
"",
395+
"No slots: 'the supported pipeline' and 'the target instance' are written generically so the text reads",
396+
"identically in a content repo with no image and in core.",
397+
"",
398+
"Landing in the three change sets this register's header describes: step 1 (this entry) names the hub",
399+
"only; step 2 lands the six spoke copies; step 3 widens required-in to all seven."
400+
]
380401
}
381402
]
382403
}

‎AGENTS.md‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,10 @@ second tree. New skills become available to both clients through the directory l
3131

3232
**When the task's goal is reached, automatically follow the [`pullrequest` skill](.claude/skills/pullrequest/SKILL.md)** — update the DOC PAGE the change belongs to (🚨 **not** a What's New file: those are written per RELEASE, not per change — policy `whatsnew-cadence`; see `conserve-work-products` below, which this clause used to contradict), commit, push, open the PR, wait for green CI, merge. "Goal reached" means implemented, verified, and the touched projects build clean with CI's flags; don't stop to ask permission at that point. Everything short of that stays manual: never commit or push half-done or unverified work, and **never merge with CI red or pending**.
3333

34+
<!-- shared-rule:begin finish-autonomously -->
35+
🚨 **Finish AUTONOMOUSLY — a task ends MERGED, DEPLOYED and VERIFIED, with no second "go" from anyone** (maintainer, 2026-09-09 and 2026-10-18: *"see that it is all being merged automatically"*, *"clearly stated to finish autonomously"*). A request to change something authorizes the whole delivery: worktree → change → local gates → push → pull request → every review thread answered → green → merged → deployed through the supported pipeline → the changed behaviour verified on the target instance → worktrees removed. Never hand back "shall I merge?", a bare pull-request link, or "green and waiting". Once the review is answered, arm auto-merge (it lands when the required checks pass, never before) and leave ONE background REST watcher (a script polling at ≥ 60 s, never a model loop) that re-arms a disarmed pull request, lands the halves of a cross-repo change in the order the pair gates demand (a platform ADDITION before the dependent that uses it, a DELETING half only after the dependent adaptation has merged), and wakes you only for what a script cannot decide: a thread to answer, a red to fix at the root. A red the workflow itself calls infra — no runner, no verdict in time, a timeout — is waited out or re-run, never a reason to stop. Stop and ask ONLY when the next step is **fatal** (a required gate red on your own change would ship broken), **system-changing** beyond the normal deploy (branch protection, credentials, migrations, a hand-roll), or **destructive** (deleting data, rewriting a shared branch); then state the blocker, the last completed stage, and what production actually runs now — unchanged, or which deploy steps already landed. Autonomy never licenses bypassing a check, an access control or the CD pipeline.
36+
<!-- shared-rule:end finish-autonomously -->
37+
3438
### 🚨🚨🚨 ABSOLUTE: NEVER work on the primary checkout — it stays on `main`, untouched. EVERYONE creates a worktree.
3539

3640
**The primary checkout (`/Users/roland/code/MeshWeaver`) is a READ-ONLY reference, not a workspace. It must stay parked on `main` and untouched — never edit, build, commit, `checkout`, `switch`, `reset`, or `stash` there, and never leave it on a feature branch.** It is the base every session's worktree is cut from; mutating it can clobber every concurrent session's uncommitted WIP.

0 commit comments

Comments
 (0)