Fixes for PR review comments

Author: data-douser

client/src/ql-mcp-client.js

@@ -809,7 +809,8 @@ async function main() {
 }
 
 // Run if called directly
-if (import.meta.url === pathToFileURL(process.argv[1]).href) {
+const cliPath = process.argv[1] ? path.resolve(process.argv[1]) : undefined;
+if (cliPath && import.meta.url === pathToFileURL(cliPath).href) {
   main().catch((error) => {
     console.error("Fatal error:", error);
     process.exit(1);

server/dist/ql-mcp-server.js

@@ -9,7 +9,7 @@ import { logger } from '../utils/logger';
 import { evaluateQueryResults, QueryEvaluationResult, extractQueryMetadata } from './query-results-evaluator';
 import { getOrCreateLogDirectory } from './log-directory-manager';
 import { writeFileSync, rmSync, existsSync, mkdirSync } from 'fs';
-import { join, dirname, resolve, basename } from 'path';
+import { basename, dirname, join, resolve } from 'path';
 import { createProjectTempDir } from '../utils/temp-dir';
 import { fileURLToPath } from 'url';
 

server/dist/ql-mcp-server.js.map

@@ -9,6 +9,7 @@ import { StreamableHTTPServerTransport } from '@modelcontextprotocol/sdk/server/
 import express from 'express';
 import cors from 'cors';
 import dotenv from 'dotenv';
+import { resolve } from 'path';
 import { pathToFileURL } from 'url';
 import { registerCodeQLTools, registerCodeQLResources } from './tools';
 import { registerLanguageResources } from './resources/language-resources';
@@ -141,6 +142,7 @@ async function main(): Promise<void> {
 }
 
 // Start the server if this file is run directly
-if (import.meta.url === pathToFileURL(process.argv[1]).href) {
+const scriptPath = process.argv[1] ? resolve(process.argv[1]) : undefined;
+if (scriptPath && import.meta.url === pathToFileURL(scriptPath).href) {
   main();
 }