Skip to content

fix(runbooks): use rendered agent ids in rosters - #1056

Open
rahul05ranjan wants to merge 1 commit into
msitarzewski:mainfrom
rahul05ranjan:fix/runbook-rendered-agent-ids
Open

rahul05ranjan wants to merge 1 commit into
msitarzewski:mainfrom
rahul05ranjan:fix/runbook-rendered-agent-ids

Conversation

@rahul05ranjan

Copy link
Copy Markdown

Summary

 engineering/engineering-frontend-developer.md
- runbook id: engineering-frontend-developer (filename stem)
+ runbook id: frontend-developer (rendered catalog id)

Align all four NEXUS runbooks with the agent IDs produced from name: frontmatter. Update check-runbooks.sh to validate against those IDs so this mismatch is caught in CI. Fixes #1027.

Evidence

  • Before: 62 of 64 runbook references differed from the rendered IDs.
  • After: scripts/check-runbooks.sh passes: 4 runbooks, 64 references resolve. A separate JSON comparison confirmed that only the 62 agent IDs and the explanatory _note changed; runbook and group structure stayed the same. git diff --check passes.

Merge Danger

Door: Two-way

Blast Radius: Runbook rosters. Consumers that expect filename stems instead of rendered IDs would need to follow the catalog ID convention; the converter and installer already use the rendered IDs.

TheRealVitja pushed a commit to TheRealVitja/agency-agents that referenced this pull request Oct 6, 2026
…nts)

Curated sync of the open upstream pull requests as of 2026-10-06:

- Script and CI fixes: msitarzewski#1055, msitarzewski#1030, msitarzewski#865, msitarzewski#860, msitarzewski#967, msitarzewski#870, msitarzewski#869, msitarzewski#889,
  msitarzewski#1056, msitarzewski#755, msitarzewski#868, msitarzewski#771, msitarzewski#867; ported msitarzewski#523, msitarzewski#512 and the permissions
  part of msitarzewski#790.
- Existing-agent fixes: msitarzewski#1033-msitarzewski#1052, msitarzewski#1053, msitarzewski#1054, msitarzewski#1023, msitarzewski#756-msitarzewski#759, msitarzewski#799,
  msitarzewski#805, msitarzewski#715, msitarzewski#752, msitarzewski#784, msitarzewski#793, msitarzewski#858, msitarzewski#812, msitarzewski#789, msitarzewski#1007.
- New agents: msitarzewski#702, msitarzewski#707, msitarzewski#731, msitarzewski#732, msitarzewski#764, msitarzewski#848, msitarzewski#859, msitarzewski#862, msitarzewski#863, msitarzewski#886,
  msitarzewski#908, msitarzewski#982-msitarzewski#985, msitarzewski#1031, msitarzewski#1032.
- Docs: msitarzewski#577, msitarzewski#743, msitarzewski#762, msitarzewski#785, msitarzewski#786, msitarzewski#815, msitarzewski#816.

Fixes found while integrating: Bash 3.2 guard for the msitarzewski#755 worker argv,
Windsurf re-conversion over a stale .windsurfrules, locale-independent
check-divisions.sh, agency- prefix handling in the outputs eval, and the
India Business Navigator's YAML and headings. Resolves upstream issues
msitarzewski#229, msitarzewski#763, msitarzewski#821 and msitarzewski#1027.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01NwgfpJ9tGbUh5g84u5VSgv
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

runbooks.json: 62/64 roster slug refs don't resolve against rendered agent ids (file-stem vs name-derived mismatch)

1 participant