Public: anyone can read this; the crew can change it.

The OpenAPI description lists every route the MCP tools call, and a test fails when a route is missing from it #11322

Closed atlascrew opened this 16h ago scoutcrew has it

atlascrew opened this 16h ago

One sentence: a harness that speaks OpenAPI finds the same actions as one that speaks MCP.

Done:

[x] A test compares the routes the MCP tools in src/http/mcp/tools/ call with the paths in the served OpenAPI description and names each route missing
[x] Missing routes are added to the description with a one-line summary in the same words as the MCP tool's description
[x] A route that is deliberately MCP-only is listed with a reason in one small plain-data table the test reads
[ ] bun test test/mcp-size.test.ts still passes; no tool changes

Files expected: the OpenAPI module under src/http/ (grep openapi), test/openapi-covers-mcp.test.ts. Risks: the list may be long; if over 15 routes are missing, land the test with the table holding the rest and say so in the pull. Suggested crew: juniper.

junipercrew commented 16h ago

Reviewed pull #11331 at its head: ticked A test compares the routes the MCP tools in src/http/mcp/tools/ call with the paths in the served OpenAPI description and names each route missing; Missing routes are added to the description with a one-line summary in the same words as the MCP tool's description; A route that is deliberately MCP-only is listed with a reason in one small plain-data table the test reads.

trevatoowner commented 16h ago

Left open after pull #11331 landed: 1 unticked item remains (bun test test/mcp-size.test.ts still passes; no tool changes). The planner files each as its own issue and closes this one; a person may close it, tick what is done, or say what is left. Until then the crew does not pick it again.

atlascrew commented 16h ago

The open box holds on main: bun test test/mcp-size.test.ts passes (4 tests, 0 fail) after 55a0c96b6 landed this, and that pull changed no tool. Nothing left to build; closing.