fix: mcp.json endpoint + README stale-claim cleanup (SKILL.md untouched) #1

Merged
basilensis merged 2 commits from fix/mcp-endpoint-and-stale-docs into main 2026-09-06 02:47:09 +00:00
Owner

What

Corrects the distribution package's user-facing entry points to match the shipped product:

  • mcp.json (the critical fix): https://mcp.spaghetti.gg → https://mcp.spaghetti.gg/api/mcp/agentic-engine/mcp. The bare host root does not serve MCP (Caddy proxies the whole host without a path rewrite), so every client installed via npx plugins add Tinirau/unreal-workbench currently gets a server URL that cannot connect.
  • README.md: factual corrections — tool list updated to the shipped set (Log Analyzer/analyze_logs, Asset Janitor/tidy_assets, research tools, Tool-Maker), dropped "GoTrue", removed the nonexistent spaghetti.gg/dashboard link and tm_/prj_ id prefixes (ids are UUIDs), /signup link → sign-in, engine-version claim tightened (connector needs UE 5.8; cloud tools are engine-agnostic), .spaghetti documented as optional with the personal-account fallback.
  • plugin.json: 0.1.0 → 0.1.1, description updated to the real tool set.

Out of scope, deliberately: skills/ue5-unreal/SKILL.md is left untouched — it serves the end user's UE5 Claude Code session and the UE5 MCP plugin integration, and reworking it belongs to that track. (Heads-up for that track: it currently references tool names that no longer exist — analyze_crash, audit_assets, search_kb — and the unimplemented .spaghetti auto-creation directive.)

Why now

The v1 user documentation (Tinirau/spaghetti#399) recommends this package as the primary install path and carries a "Known limitation" callout about the broken URL — this PR is the fix that callout is waiting on. Merge this and the docs marker can be removed.

Fact sources: apps/mastra/src/mcp/server.ts (tool registrations), packages/db-schema migrations 0021/0023 (pricing), infra/roles/service_proxy/templates/Caddyfile.j2 (host routing, no rewrite), docs/mcp-auth-architecture.md (verified endpoint).

## What Corrects the distribution package's user-facing entry points to match the shipped product: - **`mcp.json` (the critical fix):** `https://mcp.spaghetti.gg` → `https://mcp.spaghetti.gg/api/mcp/agentic-engine/mcp`. The bare host root does not serve MCP (Caddy proxies the whole host without a path rewrite), so every client installed via `npx plugins add Tinirau/unreal-workbench` currently gets a server URL that cannot connect. - **`README.md`:** factual corrections — tool list updated to the shipped set (Log Analyzer/`analyze_logs`, Asset Janitor/`tidy_assets`, research tools, Tool-Maker), dropped "GoTrue", removed the nonexistent `spaghetti.gg/dashboard` link and `tm_`/`prj_` id prefixes (ids are UUIDs), `/signup` link → sign-in, engine-version claim tightened (connector needs UE 5.8; cloud tools are engine-agnostic), `.spaghetti` documented as optional with the personal-account fallback. - **`plugin.json`:** 0.1.0 → 0.1.1, description updated to the real tool set. **Out of scope, deliberately:** `skills/ue5-unreal/SKILL.md` is left untouched — it serves the end user's UE5 Claude Code session and the UE5 MCP plugin integration, and reworking it belongs to that track. (Heads-up for that track: it currently references tool names that no longer exist — `analyze_crash`, `audit_assets`, `search_kb` — and the unimplemented `.spaghetti` auto-creation directive.) ## Why now The v1 user documentation (Tinirau/spaghetti#399) recommends this package as the primary install path and carries a "Known limitation" callout about the broken URL — this PR is the fix that callout is waiting on. Merge this and the docs marker can be removed. Fact sources: `apps/mastra/src/mcp/server.ts` (tool registrations), `packages/db-schema` migrations 0021/0023 (pricing), `infra/roles/service_proxy/templates/Caddyfile.j2` (host routing, no rewrite), `docs/mcp-auth-architecture.md` (verified endpoint).
basilensis added 1 commit 2026-09-06 02:33:11 +00:00
- mcp.json: bare https://mcp.spaghetti.gg ->
  https://mcp.spaghetti.gg/api/mcp/agentic-engine/mcp (the host root does
  not serve MCP; Caddy proxies the whole host without a path rewrite)
- SKILL.md/README: replace tools that no longer exist (analyze_crash,
  audit_assets, search_kb) with the shipped set (analyze_logs,
  tidy_assets, make_tool, topup, the five research tools); document
  credit costs and the approval flow
- drop stale claims: GoTrue, spaghetti.gg/dashboard, tm_/prj_ id
  prefixes, '.spaghetti auto-created on first tool call' (unimplemented)
- plugin.json: 0.1.0 -> 0.1.1
basilensis added 1 commit 2026-09-06 02:42:27 +00:00
The SKILL.md serves the end user's UE5 Claude Code gamedev session and the
UE5 MCP plugin integration; reworking it belongs to that track, not to this
endpoint/docs fix. Restored verbatim from main.
basilensis changed title from fix: mcp.json endpoint + replace stale tool docs (analyze_logs / tidy_assets / research tools) to fix: mcp.json endpoint + README stale-claim cleanup (SKILL.md untouched) 2026-09-06 02:43:06 +00:00
basilensis merged commit 9a67a811ad into main 2026-09-06 02:47:09 +00:00
Sign in to join this conversation.
No Reviewers
No labels
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Tinirau/unreal-workbench#1