perf: optimize doc-maintainer token usage (~27% reduction)

[lpcox]

Problem

The doc-maintainer workflow costs ~$3.91/run and uses ~3,003K tokens. Analysis from the token optimization advisor (#1643) identified that:

• 22 GitHub MCP tools are loaded but never called — the agent uses bash for all git/file operations
• The agent spends 3-5 turns on discovery (running git log, find) that can be pre-computed

Changes

1. Remove unused github: toolset (−14% tokens)

The agent never calls any GitHub MCP tools. It uses bash (git/find/cat) for discovery and safeoutputs.create_pull_request for the PR. Each tool schema adds ~600 tokens to every turn's system prompt.

Before: 22 GitHub MCP tools loaded → ~13,200 tokens/turn × 32 turns = ~422K tokens/run
After: 0 GitHub MCP tools loaded

2. Pre-compute git changes in steps: block (−8% tokens)

Added two pre-agent steps:

• git-changes: Runs git log --since="7 days ago" --name-only and injects results into the prompt via ${{ steps.git-changes.outputs.RECENT_COMMITS }}
• doc-files: Lists all docs via find docs/ -name "*.md" and injects via ${{ steps.doc-files.outputs.DOC_FILES }}

This eliminates the agent's discovery phase and lets it jump straight to analyzing diffs.

3. max-turns guard (not implemented)

The copilot engine does not support max-turns. The existing timeout-minutes: 15 serves as the cost guard.

Expected Impact

Metric	Before	After	Savings
Total tokens/run	3,003K	~2,200K	−27%
Cost/run	$3.91	~$2.55	−35%
GitHub tools loaded	22	0	−100%

Closes #1643

[Copilot]

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

[github-actions]

✅ Coverage Check Passed

Overall Coverage

Metric	Base	PR	Delta
Lines	86.11%	86.21%	📈 +0.10%
Statements	85.98%	86.08%	📈 +0.10%
Functions	87.41%	87.41%	➡️ +0.00%
Branches	78.50%	78.56%	📈 +0.06%

📁 Per-file Coverage Changes (1 files)

File	Lines (Before → After)	Statements (Before → After)
src/docker-manager.ts	86.4% → 86.8% (+0.39%)	85.9% → 86.3% (+0.38%)

---

Coverage comparison generated by scripts/ci/compare-coverage.ts

[github-actions]

Smoke Test Results

• ✅ GitHub MCP: feat(smoke-claude): trim unused tools to reduce token spend ~8% / fix: token analyzers should always run and close older issues
• ✅ Playwright: github.com title contains "GitHub"
• ✅ File write: /tmp/gh-aw/agent/smoke-test-claude-23957630045.txt created
• ✅ Bash: file content verified

Overall: PASS

💥 [THE END] — Illustrated by Smoke Claude

[github-actions]

Smoke Test Results

PR Titles: fix: fast-kill agent container on SIGTERM/SIGINT before full cleanup, feat(smoke-claude): trim unused tools to reduce token spend ~8%

• GitHub MCP (last 2 merged PRs): ✅
• safeinputs-gh PR query: ❌
• Playwright github.com title contains GitHub: ❌
• Tavily search result exists: ❌
• File write /tmp/gh-aw/agent/smoke-test-codex-23957630051.txt: ✅
• Bash cat file verification: ✅
• Discussion query + mystical comment: ❌
• npm ci && npm run build: ✅
  Overall status: FAIL

🔮 The oracle has spoken through Smoke Codex

[github-actions]

🤖 Smoke Test Results

PR: perf: optimize doc-maintainer token usage (~27% reduction) — @lpcox

Test	Result
GitHub MCP connectivity	✅ PR data fetched
GitHub.com HTTP	✅ (pre-step: file confirms pass)
File write/read	✅ /tmp/gh-aw/agent/smoke-test-copilot-23957630066.txt verified

Overall: PASS

📰 BREAKING: Report filed by Smoke Copilot

[github-actions]

Smoke Test: GitHub Actions Services Connectivity ✅

Check	Result
Redis PING @ host.docker.internal:6379	✅ PONG
PostgreSQL pg_isready @ host.docker.internal:5432	✅ accepting connections
PostgreSQL SELECT 1 (db: smoketest, user: postgres)	✅ returned 1

All checks passed.

🔌 Service connectivity validated by Smoke Services

[github-actions]

🏗️ Build Test Suite Results

Ecosystem	Project	Build/Install	Tests	Status
Bun	elysia	✅	1/1 passed	✅ PASS
Bun	hono	✅	1/1 passed	✅ PASS
C++	fmt	✅	N/A	✅ PASS
C++	json	✅	N/A	✅ PASS
Deno	oak	N/A	1/1 passed	✅ PASS
Deno	std	N/A	1/1 passed	✅ PASS
.NET	hello-world	✅	N/A	✅ PASS
.NET	json-parse	✅	N/A	✅ PASS
Go	color	✅	ok	✅ PASS
Go	env	✅	ok	✅ PASS
Go	uuid	✅	ok	✅ PASS
Java	gson	✅	1/1 passed	✅ PASS
Java	caffeine	✅	1/1 passed	✅ PASS
Node.js	clsx	✅	All passed	✅ PASS
Node.js	execa	✅	All passed	✅ PASS
Node.js	p-limit	✅	All passed	✅ PASS
Rust	fd	✅	1/1 passed	✅ PASS
Rust	zoxide	✅	1/1 passed	✅ PASS

Overall: 8/8 ecosystems passed — ✅ PASS

Generated by Build Test Suite for issue #1645 · ◷