Address PR review feedback

Author: data-douser

CHANGELOG.md

@@ -16,7 +16,7 @@ _Changes on `main` since the latest tagged release that have not yet been includ
 
 ### Fixed
 
-- **`query_results_cache_retrieve` rejected by GitHub Copilot Chat (HTTP 400 invalid schema)** — The `lineRange` and `resultIndices` parameters were defined with `z.tuple([...])`, which the MCP SDK serialized to a bare-array JSON Schema value (e.g. `[{"type":"integer"}, {"type":"integer"}]`). GitHub Copilot Chat enforces strict JSON Schema validation and rejected the entire `ql-mcp` server with `"... is not of type 'object', 'boolean'"`. Both parameters now use `z.object({ start, end })` so they serialize to a valid `type: "object"` JSON Schema. Tool callers must now pass `{ "lineRange": { "start": 1, "end": 10 } }` instead of `{ "lineRange": [1, 10] }`. ([#NNN](https://github.com/advanced-security/codeql-development-mcp-server/pull/NNN))
+- **`query_results_cache_retrieve` rejected by GitHub Copilot Chat (HTTP 400 invalid schema)** — The `lineRange` and `resultIndices` parameters were defined with `z.tuple([...])`, which the MCP SDK serialized to a bare-array JSON Schema value (e.g. `[{"type":"integer"}, {"type":"integer"}]`). GitHub Copilot Chat enforces strict JSON Schema validation and rejected the entire `ql-mcp` server with `"... is not of type 'object', 'boolean'"`. Both parameters now use `z.object({ start, end })` so they serialize to a valid `type: "object"` JSON Schema. Tool callers must now pass `{ "lineRange": { "start": 1, "end": 10 } }` instead of `{ "lineRange": [1, 10] }`. ([#263](https://github.com/advanced-security/codeql-development-mcp-server/pull/263))
 
 ## [v2.25.2] — 2026-04-15
 

server/test/src/tools/tool-schema-validation.test.ts

@@ -34,7 +34,7 @@ vi.mock('../../../src/lib/session-data-manager', () => ({
       clearCacheEntries: vi.fn(),
     }),
     getConfig: vi.fn().mockReturnValue({
-      storageLocation: '/tmp/test',
+      storageLocation: '.tmp/test',
       autoTrackSessions: false,
       retentionDays: 90,
       includeCallParameters: false,