From 34a607a1f5cf44888f588d0dfca423b1b69b5e33 Mon Sep 17 00:00:00 2001 From: jochen Date: Mon, 2 Nov 2026 23:02:17 +0100 Subject: [PATCH] Give the checker back its compose-candidate invoke, now that the new indexer runs MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The invoke was left out (mesh-delivery #142) only while the indexer built before the verb ran; the new indexer and the node-engine's compose-candidate run on every machine since 22:57, so a check composes on the machines again instead of passing with a warning (ADR 0593 §7, §9). Claude-Session: https://claude.ai/code/session_01NcbgfJ9Gum47zkFKWRaovL --- modules/mesh-delivery-checker/manifest_test.go | 12 +++++------- modules/mesh-delivery-checker/module.json | 3 ++- 2 files changed, 7 insertions(+), 8 deletions(-) diff --git a/modules/mesh-delivery-checker/manifest_test.go b/modules/mesh-delivery-checker/manifest_test.go index 691686b..7d995a3 100644 --- a/modules/mesh-delivery-checker/manifest_test.go +++ b/modules/mesh-delivery-checker/manifest_test.go @@ -111,13 +111,11 @@ func TestManifestDeclarations(t *testing.T) { // 1678/AC1 } // 1887/A5b: the checker reads nothing of the controller's plans-building (ADR 0483 superseded by ADR 0525 §7), calls no // build module's tool and holds no capacity: it puts check asks on the Go and Node build seats (ADR 0525 §1, §4). - // The forge's reads for a mesh-sdk pin (ADR 0484 clause 9). Not yet the node seat node-engine's compose-candidate - // (ADR 0593 §7): the indexer that runs while this delivers was built before the verb existed and refuses a manifest - // invoking it, which held the grants part's whole list (issue 1126, 2026-11-02); a follow-up adds it back once the - // new indexer runs. - if want := []string{"gitea.gitea_commits", "gitea.gitea_compare", "gitea.gitea_get_file", - "gitea.gitea_list_branches"}; !reflect.DeepEqual(sorted(m.Invokes), want) { - t.Errorf("invokes = %v, want the forge's reads", m.Invokes) + // The forge's reads for a mesh-sdk pin (ADR 0484 clause 9), and of the node seat node-engine its one verb + // compose-candidate, asked of every placed node (ADR 0593 §7), never the seat's other verbs. + if want := []string{"gitea.gitea_commits", "gitea.gitea_compare", "gitea.gitea_get_file", "gitea.gitea_list_branches", + "seat:node-engine.compose-candidate"}; !reflect.DeepEqual(sorted(m.Invokes), want) { + t.Errorf("invokes = %v, want the forge's reads and compose-candidate", m.Invokes) } if want := []string{"node-build-go", "node-build-node"}; !reflect.DeepEqual(sorted(m.Uses), want) { t.Errorf("uses = %v, want the Go and Node build seats", m.Uses) diff --git a/modules/mesh-delivery-checker/module.json b/modules/mesh-delivery-checker/module.json index 1b3b00f..767884c 100644 --- a/modules/mesh-delivery-checker/module.json +++ b/modules/mesh-delivery-checker/module.json @@ -65,7 +65,8 @@ "gitea.gitea_commits", "gitea.gitea_compare", "gitea.gitea_get_file", - "gitea.gitea_list_branches" + "gitea.gitea_list_branches", + "seat:node-engine.compose-candidate" ], "own-secrets": {