feat: add foreman role differentiation to crew template#2702
Open
rileywhite wants to merge 7 commits intosteveyegge:mainfrom
Open
feat: add foreman role differentiation to crew template#2702rileywhite wants to merge 7 commits intosteveyegge:mainfrom
rileywhite wants to merge 7 commits intosteveyegge:mainfrom
Conversation
Foreman crew members now get explicit triage-and-dispatch instructions instead of generic crew context. This prevents foremen from diving into code investigation/implementation when they should be filing beads and slinging work to polecats. Closes: hq-jsi Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
github-actions
bot
added
the
status/needs-triage
Foreman is now positioned as the Mayor of a specific rig — an expert in that rig's codebase who coordinates all work within it. Key changes: - Rig-scoped beads: foreman files beads only for their own rig - Cross-rig routing: nudge other rig's foreman or town mayor instead of filing beads directly in other rigs - Expanded responsibilities: rig expertise, architectural oversight - Relaxed direct-fix threshold from 2min to 5min Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
All code changes — crew, polecats, any agent — now merge via pull request. Removes the "push directly to main, no PRs" instructions and replaces with feature branch + gh pr create workflow. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Both templates now mention that slung polecats use the rig's configured workflow.default_formula from settings/config.json, with --formula flag as override. Related: gt-boc (sling ignores this setting — currently hardcoded) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Default scope is now explicitly the foreman's own rig — questions and status checks assume the local rig unless told otherwise - Added status check command block so foremen inspect their running polecats, convoys, and beads when asked for status Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
ComputeRedirectTarget() short-circuited to HQ beads when townBeadsHasDB was true, even when the rig had its own .beads/redirect file. This caused all crew workers and polecats in Dolt-server rigs to silently read/write the wrong database (HQ instead of rig-level beads). Now checks for rig .beads/redirect before the town fallback, allowing the rig redirect chain (lines 222+) to be reached and followed correctly.
RunSetupHooks was called during polecat spawn but missing from crew creation. Crew members (like foreman) had to manually copy gitignored files that the setup hook handles automatically. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
crew.md.tmplChanges
internal/templates/roles/crew.md.tmpl: Added{{ if eq .Polecat "foreman" }}block with foreman-specific workflow instructionsTest plan
go build ./...passesgo vet ./...passesgo test ./internal/templates/...passesgt primeas a foreman crew member shows foreman-specific contextCloses: hq-jsi
🤖 Generated with Claude Code