From c0f24f56b5e773bcc3e8821e26da5b86d152e1ca Mon Sep 17 00:00:00 2001 From: jochen Date: Fri, 23 Oct 2026 16:35:05 +0200 Subject: [PATCH] Drop the view's use of the build seats: it reads their workers and publishes no ask (issue 1887 A8a) --- modules/mesh-delivery-view/manifest_test.go | 6 +++--- modules/mesh-delivery-view/module.json | 5 ----- 2 files changed, 3 insertions(+), 8 deletions(-) diff --git a/modules/mesh-delivery-view/manifest_test.go b/modules/mesh-delivery-view/manifest_test.go index de423f5..871bca3 100644 --- a/modules/mesh-delivery-view/manifest_test.go +++ b/modules/mesh-delivery-view/manifest_test.go @@ -94,9 +94,9 @@ func TestManifestIsTakenAndReadsOnly(t *testing.T) { // 1134/AC7 t.Errorf("the manifest declares %q: the view writes only its answers", k) } } - // The view reads the build seats' streams for the backlog of asks (issue 1887 A8a): the three seats it uses. - if want := []string{"node-build-go", "node-build-image", "node-build-node"}; !reflect.DeepEqual(m.Uses, want) { - t.Errorf("uses = %v, want %v", m.Uses, want) + // The view uses no build seat: it reads their workers by the broker's named grant and publishes no ask (ADR 0525 ยง2). + if len(m.Uses) != 0 { + t.Errorf("uses = %v, want none", m.Uses) } // The one state the view writes: the operator's answers to the asks that need no proof (ADR 0528 clause 6, 1297/B9), which the teller reads. if want := []state{{Name: "answers", History: 5}}; !reflect.DeepEqual(m.State, want) { diff --git a/modules/mesh-delivery-view/module.json b/modules/mesh-delivery-view/module.json index 0d5ed50..61ed274 100644 --- a/modules/mesh-delivery-view/module.json +++ b/modules/mesh-delivery-view/module.json @@ -28,11 +28,6 @@ "mesh-delivery-watchman.stuck", "node-engine.have" ], - "uses": [ - "node-build-go", - "node-build-image", - "node-build-node" - ], "own-secrets": { "broker": "${dir:state}/broker" },