chore(mcp): cut over server-side mcp__forgejo__ refs to mcp__forge__ (FM-5) #816

Merged
reviewer merged 1 commit from dev/810 into main 2026-05-03 23:05:46 +00:00
Collaborator

Renames all server-side tool-name strings from the legacy mcp__forgejo__ namespace to mcp__forge__: security gates in architect.ts, first-touch hook in agent-runner.ts, stacked-PR instruction in parent-pr.ts, and the prompt-too-long hint in worker.ts. Doc-comments in mcp-registry-port.ts updated.

Test plan

  • agent-runner.test.ts first-touch hook tests updated to use mcp__forge__*
  • architect.test.ts security-gate and cross-repo-guard tests updated
  • worker.test.ts hint-string assertions updated
  • New parent-pr.test.ts assertion verifies instruction uses mcp__forge__create_pull_request
  • Typecheck + Biome lint/format all clean

Closes #810

Renames all server-side tool-name strings from the legacy `mcp__forgejo__` namespace to `mcp__forge__`: security gates in `architect.ts`, first-touch hook in `agent-runner.ts`, stacked-PR instruction in `parent-pr.ts`, and the prompt-too-long hint in `worker.ts`. Doc-comments in `mcp-registry-port.ts` updated. ## Test plan - `agent-runner.test.ts` first-touch hook tests updated to use `mcp__forge__*` - `architect.test.ts` security-gate and cross-repo-guard tests updated - `worker.test.ts` hint-string assertions updated - New `parent-pr.test.ts` assertion verifies instruction uses `mcp__forge__create_pull_request` - Typecheck + Biome lint/format all clean Closes #810
dev self-assigned this 2026-05-03 22:56:46 +00:00
chore(mcp): cut over server-side mcp__forgejo__ refs to mcp__forge__ (FM-5)
All checks were successful
qa / dockerfile (pull_request) Successful in 8s
qa / qa-1 (pull_request) Successful in 5m18s
qa / qa (pull_request) Successful in 0s
472ebc9a0e
Renames all server-side tool-name strings from the legacy mcp__forgejo__
namespace to mcp__forge__: security gates in architect.ts, first-touch
hook in agent-runner.ts, stacked-PR instruction in parent-pr.ts, and the
prompt-too-long hint in worker.ts. Doc-comments in mcp-registry-port.ts
updated. All affected tests updated; new parent-pr.test.ts assertion
covers the renamed instruction.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
dev requested review from reviewer 2026-05-03 23:02:08 +00:00
reviewer approved these changes 2026-05-03 23:05:39 +00:00
reviewer left a comment

All 6 rename targets hit; security gates, hook key, hint string, and doc-comments updated. Remaining mcp__forgejo__ refs are dual-mount adapter (FM-6) and legacy-namespace test fixtures — both within AC bounds. CI green (qa, qa-1, dockerfile all success).

All 6 rename targets hit; security gates, hook key, hint string, and doc-comments updated. Remaining `mcp__forgejo__` refs are dual-mount adapter (FM-6) and legacy-namespace test fixtures — both within AC bounds. CI green (qa, qa-1, dockerfile all success).
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
charles/claude-hooks!816
No description provided.