Receipt: the most recent smoke-install run on main (https://github.com/wave-av/mcp-server/actions/runs/33905925774) fails with FAIL expected 18 tools, got 19 because the voice tool added a 19th tool and the constant in .github/workflows/smoke-install.yml was not bumped.
PR #121 (design tools, 4 more) bumps the constant to 23 on its branch, so merging #121 fixes main by side effect; if #121 is not merged first, main needs its own bump to 19.
Better fix (follow-up): derive the expected count from the registered tool list (e.g. scripts/smoke-mcp.mjs reads it from the built server, or the workflow greps src/tools/index.ts) so the constant can never go stale again. The --all step's separate rows.length !== 18 assertion covers only the 18 gateway-backed tools and is intentionally unrelated.
Type: bug · Area: ci · Priority: P2
Receipt: the most recent smoke-install run on main (https://github.com/wave-av/mcp-server/actions/runs/33905925774) fails with
FAIL expected 18 tools, got 19because the voice tool added a 19th tool and the constant in .github/workflows/smoke-install.yml was not bumped.PR #121 (design tools, 4 more) bumps the constant to 23 on its branch, so merging #121 fixes main by side effect; if #121 is not merged first, main needs its own bump to 19.
Better fix (follow-up): derive the expected count from the registered tool list (e.g. scripts/smoke-mcp.mjs reads it from the built server, or the workflow greps src/tools/index.ts) so the constant can never go stale again. The
--allstep's separaterows.length !== 18assertion covers only the 18 gateway-backed tools and is intentionally unrelated.Type: bug · Area: ci · Priority: P2