[Bug] Quota consumed on /ultrareview crash before findings generation

Bug Description
Issue: /ultrareview consumed quota despite crashing before producing findings. Session: 01EtMn6g2o3aH55K9d5HJULK Branch: chore/nfk-340-security-hardening Output received: {"error":"Review crashed before producing findings. See session logs for details."} API Error: Server is temporarily limiting requests · Rate limited Expected: free quota counter unchanged when review crashes pre-findings. Actual: counter decremented anyway.

Environment Info

• Platform: linux
• Terminal: vscode
• Version: 2.1.126
• Feedback ID: 06da578d-7f23-4d42-a4cf-f4c1779fb5d5

Errors

[{"error":"Error: Plugin MCP server error - mcp-config-invalid: MCP server pinecone invalid: Missing environment variables: PINECONE_API_KEY\n    at JZH (/$bunfs/root/src/entrypoints/cli.js:2771:34881)\n    at processTicksAndRejections (native:7:39)","timestamp":"2026-05-03T09:35:49.334Z"},{"error":"Error: Plugin MCP server error - mcp-config-invalid: MCP server pinecone invalid: Missing environment variables: PINECONE_API_KEY\n    at JZH (/$bunfs/root/src/entrypoints/cli.js:2771:34881)\n    at async V (/$bunfs/root/src/entrypoints/cli.js:6351:11810)\n    at processTicksAndRejections (native:7:39)","timestamp":"2026-05-03T09:35:49.920Z"},{"error":"Error: Plugin MCP server error - mcp-config-invalid: MCP server pinecone invalid: Missing environment variables: PINECONE_API_KEY\n    at JZH (/$bunfs/root/src/entrypoints/cli.js:2771:34881)\n    at async N (/$bunfs/root/src/entrypoints/cli.js:6351:12696)\n    at processTicksAndRejections (native:7:39)","timestamp":"2026-05-03T09:35:49.921Z"},{"error":"Error: Plugin MCP server error - mcp-config-invalid: MCP server pinecone invalid: Missing environment variables: PINECONE_API_KEY\n    at JZH (/$bunfs/root/src/entrypoints/cli.js:2771:34881)\n    at async V (/$bunfs/root/src/entrypoints/cli.js:6351:11810)\n    at processTicksAndRejections (native:7:39)","timestamp":"2026-05-03T09:35:49.971Z"},{"error":"Error: Plugin MCP server error - mcp-config-invalid: MCP server pinecone invalid: Missing environment variables: PINECONE_API_KEY\n    at JZH (/$bunfs/root/src/entrypoints/cli.js:2771:34881)\n    at async N (/$bunfs/root/src/entrypoints/cli.js:6351:12696)\n    at processTicksAndRejections (native:7:39)","timestamp":"2026-05-03T09:35:49.972Z"},{"error":"The operation timed out.","timestamp":"2026-05-03T09:37:39.074Z"},{"error":"The operation timed out.","timestamp":"2026-05-03T09:38:45.631Z"},{"error":"SyntaxError: JSON Parse error: Unterminated string\n    at <parse> (:0)\n    at parse (unknown)\n    at dKq (/$bunfs/root/src/entrypoints/cli.js:172:10030)\n    at _ (/$bunfs/root/src/entrypoints/cli.js:132:18031)\n    at <anonymous> (/$bunfs/root/src/entrypoints/cli.js:174:1422)\n    at <anonymous> (/$bunfs/root/src/entrypoints/cli.js:5161:741)\n    at map (native:1:11)\n    at rw$ (/$bunfs/root/src/entrypoints/cli.js:5161:559)\n    at Xj4 (/$bunfs/root/src/entrypoints/cli.js:8906:10530)\n    at processTicksAndRejections (native:7:39)","timestamp":"2026-05-03T10:41:31.577Z"},{"error":"Error: Error normalizing tool input: ZodError: [\n  {\n    \"expected\": \"string\",\n    \"code\": \"invalid_type\",\n    \"path\": [\n      \"file_path\"\n    ],\n    \"message\": \"Invalid input: expected string, received undefined\"\n  },\n  {\n    \"expected\": \"string\",\n    \"code\": \"invalid_type\",\n    \"path\": [\n      \"old_string\"\n    ],\n    \"message\": \"Invalid input: expected string, received undefined\"\n  },\n  {\n    \"expected\": \"string\",\n    \"code\": \"invalid_type\",\n    \…

Note: Content was truncated.