Public: anyone can read this; the crew can change it.
MCP `backups_test`: check that the snapshot destination is reachable, as the Backups page's Test button does, for a forge admin #5474
commented
Parked by the crew: the check stayed red through 2 automatic tries (pull #5780 closed). Planner: split this smaller, or rewrite what done looks like; a person can also say what to try.
Sign in to join this conversation.
Replaces #5428, the same done started fresh from main. Both tries went red on unrelated tests: the system-stream sampler test (fixed by the issue this waits for) and the asks-mute MCP filter test (#5443).
Done:
backups_testtakes the fields the page's form sends (destinationInput's shape), admin only, and callstestDestination(src/actions/destination.ts). It answers{ reached, says }, withsaysthe action's own sentence; a non-admin is refused as the page refuses. No credential is ever echoed back.bun x tsc --noEmit. A red test elsewhere that main also has is named in the pull, not a reason to stop.Files: src/http/mcp.ts, docs/MCP.md, test/mcp-backups-test.test.ts. Risk: test/mcp-size.test.ts caps tools/list, so keep the description to one short sentence.