Skip to content

fix: scope agent-beads doctor routes to the selected rig#3471

Open
Bella-Giraffety wants to merge 1 commit intogastownhall:mainfrom
Bella-Giraffety:polecat/mayor-gs-8h3-agent-routes-scope
Open

fix: scope agent-beads doctor routes to the selected rig#3471
Bella-Giraffety wants to merge 1 commit intogastownhall:mainfrom
Bella-Giraffety:polecat/mayor-gs-8h3-agent-routes-scope

Conversation

@Bella-Giraffety
Copy link
Copy Markdown

@Bella-Giraffety Bella-Giraffety commented Apr 1, 2026

Summary

  • gt doctor --rig was still loading route prefixes from other rigs in agent-beads-exist.
  • That caused missing-agent reports, and --fix behavior, to include unrelated rigs.
  • Scope route-derived rig expectations to the selected rig and add focused Run()/Fix() regression coverage.

Why

Fixes #3046.

agent-beads-exist was still traversing every registered rig route even when doctor was explicitly scoped to one rig. That made a healthy selected rig look broken, and could make --fix do unrelated work outside the requested scope.

Validation

  • GOTOOLCHAIN=auto go test ./internal/doctor -run 'TestAgentBeadsExistCheck|TestRoutesCheck_DirectLayoutRig|TestDetermineRigBeadsPath_Containment'

Notes

  • This PR intentionally scopes only route-derived rig agents.
  • It does not change town-level mayor/deacon handling under --rig.

@github-actions github-actions bot added the status/needs-triage Inbox — we haven't looked at it yet label Apr 1, 2026
@Bella-Giraffety
Copy link
Copy Markdown
Author

This PR has been rebuilt on a clean upstream main base and force-pushed so it now contains only the route-scoping fix and focused tests. The previous branch accidentally included unrelated local workspace artifacts and fork-only history.

@Bella-Giraffety Bella-Giraffety force-pushed the polecat/mayor-gs-8h3-agent-routes-scope branch from f1dbb37 to b2c135e Compare April 1, 2026 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status/needs-triage Inbox — we haven't looked at it yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

agent-beads-exist Unfixable

2 participants