docs: add subagent tool-budget guardrails for MCP tasks (Closes #259) #284

2026-07-06T23:17:57-05:00

jcwalker3 commented

2026-07-06 23:17:57 -05:00

Summary

Closes #259.

Documents subagent tool-budget defaults for deterministic MCP tasks (commit/review/merge), forbids delegating gitea_commit_files to subagents when the main author session can commit natively, and pins the guidance in runbooks, portable skill, operator guide, and doc-contract tests.

Changes

docs/llm-workflow-runbooks.md — new Subagent Tool-Budget Guardrails section with budget table and forbidden detours
skills/llm-project-workflow/SKILL.md — portable subagent budget rules
gitea_mcp_server.py — subagent_tool_budget operator-guide rule
tests/test_subagent_tool_budget_docs.py — doc-contract tests
tests/test_operator_guide.py — assert guide includes new rule key

Validation

cd branches/docs-issue-259-subagent-tool-budget
/opt/homebrew/bin/pytest tests/ -q
994 passed, 6 skipped, 18 subtests passed

LLM Handoff Metadata

LLM-Agent-SHA: llm-7c4e2a9f1b83
Branch: docs/issue-259-subagent-tool-budget
HEAD: 82e6d3a
Worktree: branches/docs-issue-259-subagent-tool-budget
Author identity: jcwalker3 (prgs-author)

Controller Handoff

Task: Implement #259 subagent tool-budget guardrails
Repo: Scaled-Tech-Consulting/Gitea-Tools (prgs)
Role: author
Identity: jcwalker3 / prgs-author
Issue/PR: Add subagent tool-budget guardrails for deterministic MCP tasks (#259)
Branch/SHA: docs/issue-259-subagent-tool-budget @ 82e6d3a
Files changed: docs/llm-workflow-runbooks.md, skills/llm-project-workflow/SKILL.md, gitea_mcp_server.py, tests/test_subagent_tool_budget_docs.py, tests/test_operator_guide.py
Validation: pytest tests/ -q → 994 passed, 6 skipped
Mutations: claim #259, branch push, PR opened
Current status: PR open; independent review needed
Blockers: none
Next: non-author reviewer validates and merges if approved
Safety: no self-review; no self-merge; no tags; no secrets; no prod

## Summary Closes #259. Documents subagent tool-budget defaults for deterministic MCP tasks (commit/review/merge), forbids delegating `gitea_commit_files` to subagents when the main author session can commit natively, and pins the guidance in runbooks, portable skill, operator guide, and doc-contract tests. ## Changes - `docs/llm-workflow-runbooks.md` — new **Subagent Tool-Budget Guardrails** section with budget table and forbidden detours - `skills/llm-project-workflow/SKILL.md` — portable subagent budget rules - `gitea_mcp_server.py` — `subagent_tool_budget` operator-guide rule - `tests/test_subagent_tool_budget_docs.py` — doc-contract tests - `tests/test_operator_guide.py` — assert guide includes new rule key ## Validation ``` cd branches/docs-issue-259-subagent-tool-budget /opt/homebrew/bin/pytest tests/ -q 994 passed, 6 skipped, 18 subtests passed ``` ## LLM Handoff Metadata - LLM-Agent-SHA: llm-7c4e2a9f1b83 - Branch: docs/issue-259-subagent-tool-budget - HEAD: 82e6d3a - Worktree: branches/docs-issue-259-subagent-tool-budget - Author identity: jcwalker3 (prgs-author) ## Controller Handoff - Task: Implement #259 subagent tool-budget guardrails - Repo: Scaled-Tech-Consulting/Gitea-Tools (prgs) - Role: author - Identity: jcwalker3 / prgs-author - Issue/PR: #259 - Branch/SHA: docs/issue-259-subagent-tool-budget @ 82e6d3a - Files changed: docs/llm-workflow-runbooks.md, skills/llm-project-workflow/SKILL.md, gitea_mcp_server.py, tests/test_subagent_tool_budget_docs.py, tests/test_operator_guide.py - Validation: pytest tests/ -q → 994 passed, 6 skipped - Mutations: claim #259, branch push, PR opened - Current status: PR open; independent review needed - Blockers: none - Next: non-author reviewer validates and merges if approved - Safety: no self-review; no self-merge; no tags; no secrets; no prod

jcwalker3 added 1 commit 2026-07-06 23:17:57 -05:00

docs: add subagent tool-budget guardrails for MCP tasks (Closes #259 ) 82e6d3a3f4

Document default tool-call and wall-time budgets for deterministic MCP
work, forbid subagent commit delegation when gitea_commit_files is
available, and pin the rules in runbooks, portable skill, operator guide,
and doc-contract tests.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>

sysadmin added 1 commit 2026-07-07 01:00:40 -05:00

Merge branch 'master' into docs/issue-259-subagent-tool-budget 52b729e86a

jcwalker3 referenced this pull request

2026-07-07 01:30:27 -05:00

Reviewer reports must include exact conflict proof for skipped non-mergeable PRs #322

jcwalker3 added 2 commits 2026-07-07 03:36:06 -05:00

merge: resolve master conflicts for #259 subagent tool-budget docs edf26410a0

Integrate split-workflow router SKILL.md (#333) with subagent tool-budget
guardrails (#259). Keep shell spawn hard-stop (#258) and subagent
delegation (#266) operator-guide rules alongside subagent_tool_budget.

merge: integrate remote branch history 3c50db61eb

sysadmin approved these changes 2026-07-07 03:43:36 -05:00

sysadmin left a comment

All tests pass successfully. Reviewed the subagent tool-budget guardrails documentation and rule changes. The budget rules are now integrated into the guide, SKILL.md, and runbooks, with complete test coverage in tests/test_subagent_tool_budget_docs.py. Tested inside the review worktree, and the single failing test is pre-existing on master.

sysadmin merged commit fd6f7ff22c into master

2026-07-07 03:43:48 -05:00

jcwalker3 referenced this pull request

2026-07-07 03:44:16 -05:00

feat: add create-issue final-report verifier (Closes #337) #341

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Scaled-Tech-Consulting/Gitea-Tools#284