novox/mesh-tools
18
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Put tool-runner back in node-tools' manifest (L3 final, issue 774)

Browse Source 
The judge that knows the word runs now.

Claude-Session: https://claude.ai/code/session_01JG73KG7AfSdh6FP8VrBqCn
This commit is contained in:
jschoubben
2026-11-03 23:09:34 +01:00
parent b4819dc1fb
commit 7da4c55752
2 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
node-tools/internal/provide/provide_test.go
+4 -6
View File
@@ -206,14 +206,12 @@ func TestTheManifestNamesNothingOfTheController(t *testing.T) {
if err != nil {
t.Fatal(err)
}
// Read as text, as mesh-catalog's controllerdeps check reads its own: mesh-sdk names no module (issue 774).
// Read as text, as mesh-catalog's controllerdeps check reads its own: mesh-sdk names no module (issue 774), and the
// manifest says it is the tool runner rather than being known by its name.
if strings.Contains(string(raw), "mesh-controller") {
t.Error("node-tools' manifest names the ended controller")
}
// The manifest says nothing of being the tool runner until the indexer whose judge knows the word runs (issue 774):
// the running judge refuses a manifest with "tool-runner", and while one manifest is refused the grants part holds
// the bus's user list, so the build of the indexer that knows it is never granted. The word comes back then.
if catalogue.IsToolRunner(m) {
t.Error("node-tools' manifest says \"tool-runner\" before the judge that reads it runs")
if !catalogue.IsToolRunner(m) {
t.Error("node-tools' manifest does not say it is the tool runner (\"tool-runner\": true)")
}
}