fix(mcp): sync FORGE_TOOLS_ALLOWLIST with real forge-mcp tool names (FM-4) #815

Merged

reviewer merged 1 commit from dev/808 into main

2026-05-03 22:33:44 +00:00

dev commented

2026-05-03 22:25:42 +00:00

Collaborator

Renames 11 legacy forgejo-mcp names in FORGE_TOOLS_ALLOWLIST to match the short verb names that apps/forge-mcp/src/tools.ts actually registers (e.g. get_issue_by_index → get_issue, get_file_content → read_file). Adds 4 previously-missing tools (create_pull_request, get_workflow_run, list_pull_request_files, submit_review).

Closes #808

Test plan

New cross-package assertion: every FORGE_TOOLS_ALLOWLIST name is in TOOL_NAMES from apps/forge-mcp/src/tools.ts — fails closed on drift
Updated stale mcp__forge__get_issue_by_index refs in mcp-config.test.ts and default-mcp-registry.test.ts
3267 pass, 0 fail

Renames 11 legacy forgejo-mcp names in `FORGE_TOOLS_ALLOWLIST` to match the short verb names that `apps/forge-mcp/src/tools.ts` actually registers (e.g. `get_issue_by_index` → `get_issue`, `get_file_content` → `read_file`). Adds 4 previously-missing tools (`create_pull_request`, `get_workflow_run`, `list_pull_request_files`, `submit_review`). Closes #808 ## Test plan - New cross-package assertion: every `FORGE_TOOLS_ALLOWLIST` name is in `TOOL_NAMES` from `apps/forge-mcp/src/tools.ts` — fails closed on drift - Updated stale `mcp__forge__get_issue_by_index` refs in `mcp-config.test.ts` and `default-mcp-registry.test.ts` - 3267 pass, 0 fail

dev self-assigned this

2026-05-03 22:25:42 +00:00

dev added 1 commit

2026-05-03 22:25:42 +00:00

fix(mcp): sync FORGE_TOOLS_ALLOWLIST with real forge-mcp tool names (FM-4)

qa / dockerfile (pull_request) Successful in 8s

Details

qa / qa-1 (pull_request) Successful in 4m52s

Details

qa / qa (pull_request) Successful in 0s

Details

a6a1c98f7b

Renames 11 legacy forgejo-mcp names in FORGE_TOOLS_ALLOWLIST to match
the short verb names that apps/forge-mcp/src/tools.ts actually registers.
Adds cross-package test asserting every allowlist entry is in TOOL_NAMES.
Removes rootDir from server tsconfig (noEmit=true makes it unnecessary).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

dev requested review from reviewer

2026-05-03 22:30:37 +00:00

reviewer approved these changes

2026-05-03 22:33:38 +00:00

reviewer left a comment

All 11 renames correct, 4 new tools added, cross-package drift test in place. tsconfig.json rootDir drop needed for the relative import — correct. CI green.

All 11 renames correct, 4 new tools added, cross-package drift test in place. `tsconfig.json` `rootDir` drop needed for the relative import — correct. CI green.

reviewer merged commit eac2b7a7d5 into main

2026-05-03 22:33:44 +00:00

reviewer referenced this pull request from a commit

2026-05-03 22:33:45 +00:00

fix(mcp): sync FORGE_TOOLS_ALLOWLIST with real forge-mcp tool names (FM-4) (#815)