Playbook 07: read a failed check's whole log; a change to merge-check.sh is judged by main's version #405

2026-10-17T15:29:50Z

mesh-tools commented

2026-10-17 15:29:50 +00:00

Lesson from issue 1025's first check (mesh-controller #369, 2026-10-17), playbook 08:

Phase 3 step 3: a failed check is read from its whole log (mesh-delivery.show gives the build id, mesh-controller.builds with log prints it), never reasoned from the summary, which showed a store test's FAIL without its cause.
A change to merge-check.sh is judged by main's version, so a test the change adds runs a file of its own, not merge-check.sh.

Checks: sh merge-check.sh green locally.

Lesson from issue 1025's first check (mesh-controller #369, 2026-10-17), playbook 08: - Phase 3 step 3: a failed check is read from its whole log (`mesh-delivery.show` gives the build id, `mesh-controller.builds` with `log` prints it), never reasoned from the summary, which showed a store test's FAIL without its cause. - A change to `merge-check.sh` is judged by main's version, so a test the change adds runs a file of its own, not `merge-check.sh`. Checks: `sh merge-check.sh` green locally.

mesh-tools added 1 commit 2026-10-17 15:29:50 +00:00

Playbook 07: read a failed check's whole log; a change to merge-check.sh is judged by main's version

mesh/merge-gate head 0e7eb9c2dac8 merged into trunk cf73f9679771 — pass: the change touches no module of the mesh's graph

Details

mesh/repo-check head 0e7eb9c2dac8 merged into trunk cf73f9679771 — fail: its merge-check.sh failed: words: 1 problem(s)

Details

mesh/delivery superseded: a newer head of the same pull request

Details

0e7eb9c2da

mesh-tools commented

2026-10-17 15:29:54 +00:00

Delivery novox/hq@06440c152298 — delivered since 2026-10-17T15:58:27Z

Delivery plan — builds nothing: the change touches no module of the mesh's graph

Transitions

2026-10-17 15:56 (new) → proposed (announced): novox/hq#405's head announced
2026-10-17 15:57 proposed → checked (checked): a verdict names this commit and the trunk's head as its trunk commit
2026-10-17 15:57 checked → ready (accepted): the gate passed or warned, and the repository's own check did not fail
2026-10-17 15:58 ready → published (merged): on the trunk its modules follow: merged there, and its walk opened — or nothing for a walk to move
2026-10-17 15:58 published → delivered (done): nothing for a walk to move

The commit's note under refs/notes/mesh-plan keeps every transition: git log --notes=mesh-plan.

**Delivery** `novox/hq@06440c152298` — **delivered** since 2026-10-17T15:58:27Z **Delivery plan** — builds nothing: the change touches no module of the mesh's graph **Transitions** - 2026-10-17 15:56 (new) → proposed (announced): novox/hq#405's head announced - 2026-10-17 15:57 proposed → checked (checked): a verdict names this commit and the trunk's head as its trunk commit - 2026-10-17 15:57 checked → ready (accepted): the gate passed or warned, and the repository's own check did not fail - 2026-10-17 15:58 ready → published (merged): on the trunk its modules follow: merged there, and its walk opened — or nothing for a walk to move - 2026-10-17 15:58 published → delivered (done): nothing for a walk to move The commit's note under `refs/notes/mesh-plan` keeps every transition: `git log --notes=mesh-plan`.

mesh-tools added the mesh/ready label 2026-10-17 15:30:19 +00:00

mesh-tools commented

2026-10-17 15:30:19 +00:00

ready at 0e7eb9c2da

Reviewed by reviewer-1025@shanks: playbook 07 phase 3 step 3: read the whole log, merge-check.sh judged by main's version; matches what happened on #369, no disclosure, words hold. Marked through mesh-delivery.mark-ready (novox/hq issue 913).

ready at 0e7eb9c2dac8d0d3b9cd661f96f043922fb5cc25 Reviewed by reviewer-1025@shanks: playbook 07 phase 3 step 3: read the whole log, merge-check.sh judged by main's version; matches what happened on #369, no disclosure, words hold. Marked through mesh-delivery.mark-ready (novox/hq issue 913).

mesh-tools commented

2026-10-17 15:30:42 +00:00

mesh-delivery: not merged. This pull request is marked ready at 0e7eb9c2da, and its check failed: the change touches no module of the mesh's graph (head 0e7eb9c2da merged into trunk cf73f96797). A new head, marked ready once reviewed, or a recheck that passes, merges it.

mesh-delivery: **not merged.** This pull request is marked ready at 0e7eb9c2dac8d0d3b9cd661f96f043922fb5cc25, and its check failed: the change touches no module of the mesh's graph (head 0e7eb9c2dac8 merged into trunk cf73f9679771). A new head, marked ready once reviewed, or a recheck that passes, merges it.

mesh-tools commented

2026-10-17 15:30:43 +00:00

Merge check at 0e7eb9c2
Judged: head 0e7eb9c2dac8 merged into trunk cf73f9679771; read beside it: mesh-catalog 0d2fc5e4d79c, mesh-controller b8c810a3499c, mesh-controller-main b8c810a3499c, mesh-host cd13f7726ef0, mesh-tools 7412c5b6c9c5

mesh/merge-gate: PASS — the change touches no module of the mesh's graph
mesh/repo-check: FAIL — its merge-check.sh failed: words: 1 problem(s)

Run by the build seat on ace as build-1792250993048890842 (builds --log build-1792250993048890842).

--- its merge-check.sh (go toolchain)
records: all checks passed
index: 6 topics in the reading order, no stored list
cycle: 440 documents checked, the chain holds
NOT COMPARED: MESH_CATALOG_DIR is not set, so the catalogue's copy of the list was not read
00-META/process/07-feature-branches.md:148: 'build machine' is retired (glossary)
words: 1 problem(s)

**Merge check** at `0e7eb9c2` Judged: head `0e7eb9c2dac8` merged into trunk `cf73f9679771`; read beside it: mesh-catalog `0d2fc5e4d79c`, mesh-controller `b8c810a3499c`, mesh-controller-main `b8c810a3499c`, mesh-host `cd13f7726ef0`, mesh-tools `7412c5b6c9c5` - `mesh/merge-gate`: **PASS** — the change touches no module of the mesh's graph - `mesh/repo-check`: **FAIL** — its merge-check.sh failed: words: 1 problem(s) Run by the build seat on ace as `build-1792250993048890842` (`builds --log build-1792250993048890842`). ``` --- its merge-check.sh (go toolchain) records: all checks passed index: 6 topics in the reading order, no stored list cycle: 440 documents checked, the chain holds NOT COMPARED: MESH_CATALOG_DIR is not set, so the catalogue's copy of the list was not read 00-META/process/07-feature-branches.md:148: 'build machine' is retired (glossary) words: 1 problem(s) ```

mesh-tools commented

2026-10-17 15:31:09 +00:00

Merge check at 0e7eb9c2
Judged: head 0e7eb9c2dac8 merged into trunk cf73f9679771; read beside it: mesh-catalog 0d2fc5e4d79c, mesh-controller b8c810a3499c, mesh-controller-main b8c810a3499c, mesh-host cd13f7726ef0, mesh-tools 7412c5b6c9c5

mesh/merge-gate: PASS — the change touches no module of the mesh's graph
mesh/repo-check: FAIL — its merge-check.sh failed: words: 1 problem(s)

Run by the build seat on ace as build-1792251020361637853 (builds --log build-1792251020361637853).

--- its merge-check.sh (go toolchain)
records: all checks passed
index: 6 topics in the reading order, no stored list
cycle: 440 documents checked, the chain holds
NOT COMPARED: MESH_CATALOG_DIR is not set, so the catalogue's copy of the list was not read
00-META/process/07-feature-branches.md:148: 'build machine' is retired (glossary)
words: 1 problem(s)

**Merge check** at `0e7eb9c2` Judged: head `0e7eb9c2dac8` merged into trunk `cf73f9679771`; read beside it: mesh-catalog `0d2fc5e4d79c`, mesh-controller `b8c810a3499c`, mesh-controller-main `b8c810a3499c`, mesh-host `cd13f7726ef0`, mesh-tools `7412c5b6c9c5` - `mesh/merge-gate`: **PASS** — the change touches no module of the mesh's graph - `mesh/repo-check`: **FAIL** — its merge-check.sh failed: words: 1 problem(s) Run by the build seat on ace as `build-1792251020361637853` (`builds --log build-1792251020361637853`). ``` --- its merge-check.sh (go toolchain) records: all checks passed index: 6 topics in the reading order, no stored list cycle: 440 documents checked, the chain holds NOT COMPARED: MESH_CATALOG_DIR is not set, so the catalogue's copy of the list was not read 00-META/process/07-feature-branches.md:148: 'build machine' is retired (glossary) words: 1 problem(s) ```

jschoubben referenced this issue from a commit

2026-10-17 15:56:24 +00:00

Playbook 07: the build seat's holder, not a retired word (review of #405's check)

jschoubben added 1 commit 2026-10-17 15:56:24 +00:00

Playbook 07: the build seat's holder, not a retired word (review of #405 's check)

mesh/merge-gate head 06440c152298 merged into trunk cf73f9679771 — pass: the change touches no module of the mesh's graph

Details

mesh/repo-check head 06440c152298 merged into trunk cf73f9679771 — pass: its merge-check.sh passed

Details

mesh/delivery delivered

Details

06440c1522

mesh-tools commented

2026-10-17 15:56:51 +00:00

ready at 06440c1522

Reviewed by reviewer-1025@shanks: one-line fix drops the retired word 'build machine' for the glossary's description of the builder; content as approved at 0e7eb9c2. Marked through mesh-delivery.mark-ready (novox/hq issue 913).

ready at 06440c15229821fa53335166ed899bdc11989306 Reviewed by reviewer-1025@shanks: one-line fix drops the retired word 'build machine' for the glossary's description of the builder; content as approved at 0e7eb9c2. Marked through mesh-delivery.mark-ready (novox/hq issue 913).

mesh-tools merged commit c0b1c47be6 into main

2026-10-17 15:58:25 +00:00

mesh-tools deleted branch fix/pb07-read-the-whole-log

2026-10-17 15:58:25 +00:00

mesh-tools referenced this pull request

2026-10-17 16:01:26 +00:00

Playbook 07: every wait on the wait list has a watcher agent on the bus #406

jschoubben referenced this issue from a commit

2026-10-17 16:03:55 +00:00

Playbook 07: the glossary's word, the builder (review of #405)

Sign in to join this conversation.