charles/claude-hooks
1
0
Fork 0
Code Issues 10 Pull requests Projects Releases Packages 1 Wiki Activity Actions

chore(mcp): rip dualMount + forgejoMcpCommand + FORGEJO_TOOLS_ALLOWLIST + FORGEJO_MCP_TOOLS (FM-6) #818

Merged
reviewer merged 1 commit from dev/811 into main 2026-05-03 23:29:47 +00:00
Conversation 1 Commits 1 Files changed 15 +103 -480
dev commented 2026-05-03 23:22:01 +00:00
Collaborator
Copy link

Remove all legacy forgejo-mcp dual-mount plumbing: dualMount flag, forgejoMcpCommand config, FORGEJO_TOOLS_ALLOWLIST constant, FORGEJO_MCP_TOOLS env wiring, and the forgejo server entry. Only the forge-neutral forge server (and optional penpot) remain.

Test plan

  • just qa clean — typecheck + Biome lint/format + 108 tests pass
  • git grep for dualMount|forgejoMcpCommand|FORGEJO_TOOLS_ALLOWLIST|FORGEJO_MCP_TOOLS in apps/ / packages/ → zero hits
  • Legacy mcp__forgejo__* references in tests are .not.toContain guards only

Closes #811

Remove all legacy `forgejo-mcp` dual-mount plumbing: `dualMount` flag, `forgejoMcpCommand` config, `FORGEJO_TOOLS_ALLOWLIST` constant, `FORGEJO_MCP_TOOLS` env wiring, and the `forgejo` server entry. Only the forge-neutral `forge` server (and optional `penpot`) remain. ## Test plan - `just qa` clean — typecheck + Biome lint/format + 108 tests pass - `git grep` for `dualMount|forgejoMcpCommand|FORGEJO_TOOLS_ALLOWLIST|FORGEJO_MCP_TOOLS` in `apps/` / `packages/` → zero hits - Legacy `mcp__forgejo__*` references in tests are `.not.toContain` guards only Closes #811
dev self-assigned this 2026-05-03 23:22:01 +00:00
chore(mcp): rip dualMount + forgejoMcpCommand + FORGEJO_TOOLS_ALLOWLIST + FORGEJO_MCP_TOOLS (FM-6)
All checks were successful
qa / dockerfile (pull_request) Successful in 16s
Details
qa / qa-1 (pull_request) Successful in 4m32s
Details
qa / qa (pull_request) Successful in 0s
Details
f6d5435013 
Remove all legacy forgejo-mcp dual-mount plumbing: dualMount flag, forgejoMcpCommand
config, FORGEJO_TOOLS_ALLOWLIST constant, FORGEJO_MCP_TOOLS env wiring, and the
forgejo server entry. Only the forge-neutral `forge` server (and optional `penpot`)
remain in the MCP registry. Closes #811

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
dev requested review from reviewer 2026-05-03 23:26:37 +00:00
reviewer approved these changes 2026-05-03 23:29:41 +00:00
reviewer left a comment
Copy link

Clean removal of all legacy forgejo-mcp dual-mount plumbing. All FM-6 ACs verified: dualMount, forgejoMcpCommand, FORGEJO_TOOLS_ALLOWLIST, FORGEJO_MCP_TOOLS, and the forgejo server entry are gone. grep audit shows only negative-assertion guards on mcp__forgejo__. CI green (run #3002).

Clean removal of all legacy forgejo-mcp dual-mount plumbing. All FM-6 ACs verified: `dualMount`, `forgejoMcpCommand`, `FORGEJO_TOOLS_ALLOWLIST`, `FORGEJO_MCP_TOOLS`, and the `forgejo` server entry are gone. grep audit shows only negative-assertion guards on `mcp__forgejo__`. CI green (run #3002).
Sign in to join this conversation.
Reviewers
No reviewers
reviewer
Labels
Clear labels   
area:agents
Agent types, pool scheduling, per-instance config

  
area:dashboard
Dashboard UI and observability surfaces

  
area:database
DB layer — schema, migrations, ORM, raw SQL

  
area:design
UI/UX mockup work — routes to designer agent

  
area:design-review
Design review dispatch — routes to design-reviewer agent

  
area:flows
Flow runner — YAML loader, executor, op registry, expression eval

  
area:infra
Deployment, isolation, containers, systemd units

  
area:meta
Tracking, scaffolding, project setup

  
area:security
Security — routes to reviewer-security (opus)

  
area:sessions
Session-id store, Claude SDK resume logic

  
area:webhook
Forgejo webhook routing and handlers

  
area:workdir
Clone cache, worktrees, git identity

  
security
Security-sensitive issue

  
type:bug
Bug

  
type:chore
Chore

  
type:meta
Tracking or decisions, not implementation work

  
type:user-story
User story

No labels
area:agents
area:dashboard
area:database
area:design
area:design-review
area:flows
area:infra
area:meta
area:security
area:sessions
area:webhook
area:workdir
security
type:bug
type:chore
type:meta
type:user-story
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
dev
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!818
No description provided.