From 24d4f6426f984127706f8c8dd5627e399a7cc52f Mon Sep 17 00:00:00 2001 From: pbean Date: Fri, 19 Jun 2026 17:07:43 -0700 Subject: [PATCH] test(bmad-module): track fixture .mcp.json so CI install test passes The acme-devlog fixture's .mcp.json was swallowed by the global .gitignore '.mcp.json' rule, so it was never committed. The skill's install test asserts _bmad/devlog/.mcp.json exists, which passed locally (file on disk) but failed on CI's clean checkout. Add a negation for the test-fixtures path and commit the fixture. Co-Authored-By: Claude Opus 4.8 --- .gitignore | 3 +++ .../examples/comprehensive/acme-devlog/.mcp.json | 11 +++++++++++ 2 files changed, 14 insertions(+) create mode 100644 src/core-skills/bmad-module/tests/fixtures/examples/comprehensive/acme-devlog/.mcp.json diff --git a/.gitignore b/.gitignore index 99e48d9ab..534d83a64 100644 --- a/.gitignore +++ b/.gitignore @@ -48,6 +48,9 @@ CLAUDE.local.md .junie/ .agents/ +# Test fixtures need a committed .mcp.json (the rule above ignores real projects') +!src/core-skills/bmad-module/tests/fixtures/**/.mcp.json + z*/ !docs/zh-cn/ diff --git a/src/core-skills/bmad-module/tests/fixtures/examples/comprehensive/acme-devlog/.mcp.json b/src/core-skills/bmad-module/tests/fixtures/examples/comprehensive/acme-devlog/.mcp.json new file mode 100644 index 000000000..a73e9e07a --- /dev/null +++ b/src/core-skills/bmad-module/tests/fixtures/examples/comprehensive/acme-devlog/.mcp.json @@ -0,0 +1,11 @@ +{ + "mcpServers": { + "devlog-history": { + "command": "node", + "args": ["${CLAUDE_PLUGIN_ROOT}/scripts/mcp-server.js"], + "env": { + "DEVLOG_PATH": "${CLAUDE_PLUGIN_DATA}/devlog" + } + } + } +}