Playbooks 07 and 08: read the forge branch's head before trusting an agent's push #479

Merged

mesh-tools merged 1 commits from pb/read-the-pushed-head into main 2026-10-26 10:03:59 +00:00

Conversation 1 Commits 1 Files Changed 3

+3 -3

mesh-tools commented 2026-10-26 10:03:11 +00:00

Member

Copy Link

Copy Source

A lesson from 2026-10-26 (issue 1906's fix, mesh-delivery #60): the fixing agent committed on a detached HEAD and reported its push; the first try of the reviewed head failed 'not our ref' because the forge never had it. Playbook 07 step 3 item 1 now says a session reads the forge branch's head and compares it with the reviewed commit before it starts a try or opens a pull request; playbook 08 step 5 points to it; playbooks.html's step line follows. hq's checks pass (go test ./...).

Reviewed-by: lead-47@shanks at be1bbe0a65

A lesson from 2026-10-26 (issue 1906's fix, mesh-delivery #60): the fixing agent committed on a detached HEAD and reported its push; the first try of the reviewed head failed 'not our ref' because the forge never had it. Playbook 07 step 3 item 1 now says a session reads the forge branch's head and compares it with the reviewed commit before it starts a try or opens a pull request; playbook 08 step 5 points to it; playbooks.html's step line follows. hq's checks pass (go test ./...). Reviewed-by: lead-47@shanks at be1bbe0a652abf1342d0d6048355dd04e65c3f70

mesh-tools added 1 commit 2026-10-26 10:03:11 +00:00

Playbooks 07 and 08: read the forge head before trusting a reported push, so a try never runs on a push that did not happen be1bbe0a65

mesh-tools commented 2026-10-26 10:03:14 +00:00

Author

Member

Copy Link

Copy Source

Delivery novox/hq@be1bbe0a652a — merged since 2026-10-26T10:04:01Z

Source — source.main moved to novox/hq@122cda84b3. Whether it reached the machines is mesh-delivery-view's, derived from source.main, the wants and the haves.

Transitions

• 2026-10-26 10:03 (new) → proposed (announced): novox/hq#479's head announced
• 2026-10-26 10:03 proposed → ready (accepted): a verdict names this commit and the trunk's head as its trunk commit, the gate passed or warned, and the repository's own check did not fail; or the check gates no merge (check-gates-merge false)
• 2026-10-26 10:04 ready → merged (merged): on the trunk its modules follow: merged there

The commit's note under refs/notes/mesh-plan keeps every transition of this change up to its merge: git log --notes=mesh-plan.

<!-- mesh-delivery:view --> **Delivery** `novox/hq@be1bbe0a652a` — **merged** since 2026-10-26T10:04:01Z **Source** — source.main moved to novox/hq@122cda84b302. Whether it reached the machines is `mesh-delivery-view`'s, derived from `source.main`, the wants and the haves. **Transitions** - 2026-10-26 10:03 (new) → proposed (announced): novox/hq#479's head announced - 2026-10-26 10:03 proposed → ready (accepted): a verdict names this commit and the trunk's head as its trunk commit, the gate passed or warned, and the repository's own check did not fail; or the check gates no merge (check-gates-merge false) - 2026-10-26 10:04 ready → merged (merged): on the trunk its modules follow: merged there The commit's note under `refs/notes/mesh-plan` keeps every transition of this change up to its merge: `git log --notes=mesh-plan`.

mesh-tools merged commit 122cda84b3 into main 2026-10-26 10:03:59 +00:00

mesh-tools deleted branch pb/read-the-pushed-head 2026-10-26 10:03:59 +00:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

mesh/ready

Reviewed and ready: mesh-delivery merges at the marked head once its checks pass (hq ADR 0364)

No labels

Milestone

No items

No Milestone

Assignees

Clear assignees

agent-ace agent-g14 agent-novox agent-shanks jschoubben mesh-tools miel.truyen nox-agent

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: novox/hq#479