FM-3 migrate agents to forge-mcp, drop forgejo-mcp + patches #687

Merged

charles merged 1 commit from boss/654 into main

2026-05-01 21:23:52 +00:00

code-lead commented

2026-05-01 20:55:23 +00:00

Collaborator

Cuts container-mode agents over from mcp__forgejo__* (upstream Go binary) to mcp__forge__* (our forge-neutral wrapper) and removes the upstream binary + patches/ directory from the image.

Test plan

git grep mcp__forgejo__ under /skills/ returns zero hits
bun x turbo run typecheck clean
bun x @biomejs/biome check . exit 0
bun x turbo run test passes (2983 server + 51 forge-mcp tests)
New tests cover the per-dispatch FORGE_TYPE/FORGE_REPO/FORGE_TOKEN/FORGE_TOKENS injection in renderForInstance
Manual: image rebuild on claude-hooks:dev to confirm the Go toolchain is gone and forge-mcp --help runs (release.yml smoke step covers this)

Closes #654

Cuts container-mode agents over from `mcp__forgejo__*` (upstream Go binary) to `mcp__forge__*` (our forge-neutral wrapper) and removes the upstream binary + `patches/` directory from the image. ## Test plan - `git grep mcp__forgejo__` under `/skills/` returns zero hits - `bun x turbo run typecheck` clean - `bun x @biomejs/biome check .` exit 0 - `bun x turbo run test` passes (2983 server + 51 forge-mcp tests) - New tests cover the per-dispatch FORGE_TYPE/FORGE_REPO/FORGE_TOKEN/FORGE_TOKENS injection in `renderForInstance` - Manual: image rebuild on `claude-hooks:dev` to confirm the Go toolchain is gone and `forge-mcp --help` runs (release.yml smoke step covers this) Closes #654

code-lead self-assigned this

2026-05-01 20:55:23 +00:00

code-lead added 1 commit

2026-05-01 20:55:23 +00:00

chore(forge-mcp): FM-3 migrate agents to forge-mcp, drop forgejo-mcp + patches

qa / dockerfile (pull_request) Successful in 6s

Details

qa / qa (pull_request) Successful in 3m31s

Details

7b58eadd62

- Skills: rewrite all `/skills/*.md` to call `mcp__forge__*` (rename
  table from spec). `git grep mcp__forgejo__` under /skills returns
  zero hits.
- Dockerfile: drop the multi-stage `forgejo-mcp` Go builder + the
  `patches/` directory; the image now ships only the compiled
  `forge-mcp` binary at /usr/local/bin/forge-mcp.
- config/mcp-builtin{,-defs}.json: rename the legacy `forgejo` entry
  to `forge`, switch env to FORGE_TOKEN (FORGE_FORGEJO_URL stays).
- agent-env-sync.renderForInstance: optional dispatch-context arg
  layered on top of the static MCP env writes FORGE_TYPE / FORGE_REPO
  / FORGE_TOKEN (and FORGE_TOKENS for cross-forge dispatches) into
  `.claude.json::mcpServers.forge.env` per dispatch.
- registry.ts pre-dispatch render passes the request's repo + forge
  + token through.
- New `docs/forge-mcp.md` (env contract, tool surface, rename table,
  per-forge differences). Update `docs/credentials.md` (FORGE_TOKEN
  replaces FORGEJO_ACCESS_TOKEN), `docs/design-review.md`,
  `docs/runner-setup.md`, project CLAUDE.md.
- release.yml smoke + justfile install recipe: forgejo-mcp →
  forge-mcp.

Closes #654

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>

code-lead requested review from reviewer

2026-05-01 20:58:57 +00:00

reviewer approved these changes

2026-05-01 21:02:05 +00:00

reviewer left a comment

Correct — Go builder stage gone, patches/ deleted, skills fully on mcp__forge__*, renderForInstance dispatch context wired with tests for single-forge / multi-forge / boot-time cases, docs complete. CI green.

Correct — Go builder stage gone, patches/ deleted, skills fully on `mcp__forge__*`, `renderForInstance` dispatch context wired with tests for single-forge / multi-forge / boot-time cases, docs complete. CI green.

code-lead force-pushed boss/654 from 7b58eadd62

qa / dockerfile (pull_request) Successful in 6s

Details

qa / qa (pull_request) Successful in 3m31s

Details

to 070b635981

qa / dockerfile (pull_request) Successful in 5s

Details

qa / qa (pull_request) Successful in 2m14s

Details

2026-05-01 21:07:41 +00:00

Compare