Skip to content

agent-team: wire Context7 MCP into planner + implementer#67

Open
verkyyi wants to merge 14 commits intomainfrom
mcp/context7-planner-implementer
Open

agent-team: wire Context7 MCP into planner + implementer#67
verkyyi wants to merge 14 commits intomainfrom
mcp/context7-planner-implementer

Conversation

@verkyyi
Copy link
Copy Markdown
Owner

@verkyyi verkyyi commented Apr 22, 2026

Summary

  • Adds mcp-servers.context7 (stdio, npx -y @upstash/context7-mcp, tools: resolve-library-id, get-library-docs) to both the planner and implementer agent-team workflows.
  • Widens planner network from defaults[defaults, node, context7.com]. Implementer already had node; just appended context7.com.
  • Prompt bodies unchanged — Claude discovers MCP tools on its own; explicit nudges tend to cause over-lookup.

Why

Planner and implementer both benefit from current framework docs when grounding file paths / writing API calls. Spec and reviewer don't touch APIs directly, so they stay untouched.

Test plan

  • Re-install agent-team into verkyyi/agent-team-playground (/install-agent-team) to recompile the lockfiles.
  • Open a toy issue that references a framework; label agent-team.
  • Watch planner + implementer runs: confirm MCP gateway loads the context7 server; confirm no firewall denial on context7.com (widen to *.context7.com if so).
  • Confirm the run still dispatches downstream and completes (no regression on the base pipeline).

🤖 Generated with Claude Code

Gives both agents fresh library/API docs via Upstash's Context7 MCP
(stdio, npx-launched). Planner uses it to ground plan file paths
against real framework surfaces; implementer uses it to write calls
against current APIs instead of training-data ones.

Planner: widened network from `defaults` to include the `node` preset
(for the npm fetch of @upstash/context7-mcp) + context7.com.
Implementer: `node` already allowed, just appended context7.com.

Prompt bodies unchanged — Claude discovers MCP tools automatically;
explicit "check Context7 first" nudges tend to cause over-lookup.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
verkyyi and others added 13 commits April 23, 2026 00:05
Closes the "PR conflicts after human leaves" gap: implementer rebases at
start of its own runs, and a scheduled sweep rebases open agent-team PRs
that fall behind main. Mechanical conflicts resolve silently; semantic
ones escalate via state:blocked.
Eight-task breakdown: mode input + rebase-at-start (T1), rebase-only mode
(T2), sweep-agent (T3), install-skill updates (T4), README (T5), install
behavior tests (T6), e2e install test (T7), playground smoke (T8). Each
task scoped to a single file or test file and ends in a commit.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- CR step 5 clarifies push-vs-proceed across modes
- Inputs paragraph lists mode
- Step 3 defers escalation to CR's template
- Skip condition uses ancestry check
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Iteration guard explicitly skipped for rebase mode
- Rebase-only mode stops on non-draft PRs
- Test-failure cleanup uses reset (not rebase --abort)
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Fetch main once before the loop (was per-PR)
- Tighten idempotency wording to 're-entrant safe'
- Remove dashboard-issue comment path + unused add-comment safe-output
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Without this label the sweep never finds PRs to rebase — the implementer
already tags new PRs with agent-team:pr but the install skill wasn't
creating the label, so gh pr list --label agent-team:pr returned empty.

Also gitignore stray catalog/**/*.lock.yml artifacts left by local
gh aw compile during development.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

1 participant