Fix Windows integration test failure: Node.js shell compatibility

[Copilot]

Integration Testing Improvement PR: MCP Server Primitives

Related Issue: Job ID 62804875673

⚠ Scope Note: This PR fixes the client test harness to enable integration tests on Windows. Changes are in client/src/ql-mcp-client.js, not test cases.

Summary

Integration tests failed on windows-latest with "CodeQL CLI not found in PATH" despite CodeQL being installed. Root cause: Node.js execSync defaults to cmd.exe on Windows, which cannot execute the bash-based CodeQL stub from gh codeql install-stub.

Primitive Metadata

• Primitive name(s): All integration tests (client harness fix)
• Type(s): Client infrastructure
• Related resources or prompts: N/A (harness-level fix)

Test Additions / Modifications

No test files modified. Fix enables existing tests to run on Windows.

Before vs After File States

// Before: client/src/ql-mcp-client.js
execSync("codeql version", {
  encoding: "utf8",
  stdio: ["pipe", "pipe", "pipe"],
  timeout: 5000
})

// After: client/src/ql-mcp-client.js  
execSync("codeql version", {
  encoding: "utf8",
  stdio: ["pipe", "pipe", "pipe"],
  timeout: 5000,
  shell: process.platform === "win32" ? "bash" : undefined
})

Assertions Added

• Tool result validation (CodeQL CLI detection now succeeds on Windows)

Edge & Negative Cases

Windows-specific shell behavior when invoking bash scripts via Node.js child_process.execSync.

Non-Goals / Deferred Follow-ups

None. Single-line fix with no additional scope.

Local Verification

Cannot reproduce Windows-specific failure locally on Unix. Fix validated via:

• Server unit test suite (345 tests pass)
• Lint/format checks pass
• Code review clean

CI will validate on windows-latest.

Checklist

• Scope limited to client harness fix
• All new tests pass locally (N/A - no new tests)
• Does not introduce unrelated refactors
• Documentation/comments updated where helpful
• Re-ran tests to confirm deterministic results
• Added negative test (N/A - infrastructure fix)
• Followed directory structure (N/A - client code, not tests)
• Each file in before has matching file in after (N/A)
• No binary or non-diffable files added
• Tests focus on a single primitive concern (N/A)
• Ran npm run lint:fix and npm run build-and-test successfully
• Checked for accidental changes outside allowed path

Screenshots / Logs (Optional)

From failing workflow logs:

[2026-02-06T21:51:55.747Z] [ERROR] CodeQL CLI not found in PATH

Expected after fix: CLI check succeeds, integration tests execute.

Additional Notes

Cross-platform execSync behavior: Windows defaults to cmd.exe, Unix defaults to /bin/sh. CodeQL stub requires bash. Pattern stored in repo memory for future reference.

Original prompt

Fix the failing GitHub Actions workflow Integration Tests (windows-latest)
Analyze the workflow logs, identify the root cause of the failure, and implement a fix.
Job ID: 62804875673
Job URL: https://github.com/advanced-security/codeql-development-mcp-server/actions/runs/21766880957/job/62804875673

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.