fix: fixed binary to be executable with npx

Author: wayzeek

bin/cli.js

@@ -1,22 +1,52 @@
 #!/usr/bin/env node
 
-import { startServer } from '../build/index.js';
+import { fileURLToPath } from 'url';
+import { dirname, resolve } from 'path';
+import { spawn } from 'child_process';
+import { createRequire } from 'module';
 
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
+const require = createRequire(import.meta.url);
+
+// Parse command line arguments
 const args = process.argv.slice(2);
-const mode = args[0]?.toLowerCase();
-
-if (mode === 'http') {
-  // If the HTTP server module exists, import and run it
-  import('../build/http-server.js')
-    .catch(error => {
-      console.error('Error starting HTTP server:', error);
-      process.exit(1);
-    });
-} else {
-  // Default to stdio server
-  startServer()
-    .catch(error => {
-      console.error('Error starting stdio server:', error);
-      process.exit(1);
-    });
+const httpMode = args.includes('--http') || args.includes('-h');
+
+console.log(`Starting Starknet MCP Server in ${httpMode ? 'HTTP' : 'stdio'} mode...`);
+
+// Determine which file to execute
+const scriptPath = resolve(__dirname, '../build', httpMode ? 'http-server.js' : 'index.js');
+
+try {
+  // Check if the built files exist
+  require.resolve(scriptPath);
+  
+  // Execute the server
+  const server = spawn('node', [scriptPath], {
+    stdio: 'inherit',
+    shell: false
+  });
+
+  server.on('error', (err) => {
+    console.error('Failed to start server:', err);
+    process.exit(1);
+  });
+
+  // Handle clean shutdown
+  const cleanup = () => {
+    if (!server.killed) {
+      server.kill();
+    }
+  };
+
+  process.on('SIGINT', cleanup);
+  process.on('SIGTERM', cleanup);
+  process.on('exit', cleanup);
+
+} catch (error) {
+  console.error('Error: Server files not found. The package may not be built correctly.');
+  console.error('Please try reinstalling the package or contact the maintainers.');
+  console.error(error);
+  process.exit(1);
 } 

src/index.ts

@@ -1,23 +1,20 @@
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
-import startServerImpl from "./server/server.js";
+import startServer from "./server/server.js";
 
-export const startServer = async () => {
+// Start the server
+async function main() {
   try {
-    const server = await startServerImpl();
+    const server = await startServer();
     const transport = new StdioServerTransport();
     await server.connect(transport);
     console.error("Starknet MCP Server running on stdio");
-    return server;
   } catch (error) {
     console.error("Error starting Starknet MCP server:", error);
     process.exit(1);
   }
-};
+}
 
-// Start the server when this file is run directly
-if (import.meta.url === `file://${process.argv[1]}`) {
-  startServer().catch((error) => {
-    console.error("Fatal error in main():", error);
-    process.exit(1);
-  });
-}
+main().catch((error) => {
+  console.error("Fatal error in main():", error);
+  process.exit(1);
+});