add npx support

Author: Larkooo

bin/sensei-mcp.js

@@ -1,11 +1,16 @@
 #!/usr/bin/env node
 
-// Use CommonJS require for better compatibility with npx
-const path = require('path');
-const { spawn } = require('child_process');
+// Use ES modules since package.json has "type": "module"
+import { fileURLToPath } from 'url';
+import { dirname, join } from 'path';
+import { spawn } from 'child_process';
+
+// Get the directory of the current module
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = dirname(__filename);
 
 // Path to the built main.js file
-const mainPath = path.join(__dirname, '../build/src/main.js');
+const mainPath = join(__dirname, '../build/src/main.js');
 
 // Log startup information
 console.log('Starting Sensei MCP server...');

eslint.config.mjs

@@ -63,4 +63,15 @@ export default tseslint.config(
       },
     },
   },
+  // Configuration for bin directory
+  {
+    files: ['bin/**/*.js'],
+    languageOptions: {
+      globals: {
+        ...globals.node,
+        console: 'readonly',
+        process: 'readonly',
+      },
+    },
+  },
 );

package.json

@@ -6,6 +6,12 @@
   "bin": {
     "sensei-mcp": "./bin/sensei-mcp.js"
   },
+  "files": [
+    "bin/",
+    "build/",
+    "prompts/",
+    "resources/"
+  ],
   "devDependencies": {
     "@eslint/js": "~9.17",
     "@types/eslint__js": "~8.42",
@@ -39,8 +45,8 @@
     "prettier": "prettier \"{src,__{tests}__}/**/*.{ts,mts}\" --config .prettierrc --write",
     "prettier:check": "prettier \"{src,__{tests}__}/**/*.{ts,mts}\" --config .prettierrc --check",
     "test:watch": "vitest unit",
-    "prepare": "npm run build",
-    "postinstall": "npm run prepare"
+    "prepare": "npm run build && chmod +x bin/sensei-mcp.js",
+    "postinstall": "chmod +x bin/sensei-mcp.js"
   },
   "license": "Apache-2.0",
   "dependencies": {

src/prompts.ts

@@ -8,10 +8,30 @@ import {
   CallToolResult,
   GetPromptResult,
 } from '@modelcontextprotocol/sdk/types.js';
+import { fileURLToPath } from 'url';
+
+// Get the directory of the current module
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
 
 // Configuration
+// Use the package directory if it exists, otherwise use the current working directory
 export const PROMPTS_DIR = path.join(process.cwd(), 'prompts');
 
+// Try to use the package directory for prompts if it exists
+export async function getPromptsDir(): Promise<string> {
+  const packagePromptsDir = path.join(__dirname, '../../prompts');
+  try {
+    await fs.access(packagePromptsDir);
+    return packagePromptsDir;
+  } catch (
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    _error
+  ) {
+    return PROMPTS_DIR;
+  }
+}
+
 // Resource reference regex pattern (e.g., {{resource:path/to/resource}})
 export const RESOURCE_REF_PATTERN = /\{\{resource:(.*?)\}\}/g;
 
@@ -299,12 +319,14 @@ export async function loadPrompts(
   const span = Logger.span('loadPrompts');
 
   try {
-    await fs.mkdir(PROMPTS_DIR, { recursive: true });
-    Logger.debug(`Ensuring prompts directory exists`, { path: PROMPTS_DIR });
+    // Get the prompts directory
+    const promptsDir = await getPromptsDir();
+    await fs.mkdir(promptsDir, { recursive: true });
+    Logger.debug(`Ensuring prompts directory exists`, { path: promptsDir });
 
-    const files = await fs.readdir(PROMPTS_DIR);
+    const files = await fs.readdir(promptsDir);
     Logger.info(`Found ${files.length} potential prompt files`, {
-      directory: PROMPTS_DIR,
+      directory: promptsDir,
     });
 
     let loadedCount = 0;
@@ -319,7 +341,7 @@ export async function loadPrompts(
         continue;
       }
 
-      const filePath = path.join(PROMPTS_DIR, file);
+      const filePath = path.join(promptsDir, file);
       const stats = await fs.stat(filePath);
 
       if (stats.isDirectory()) {

src/resources.ts

@@ -5,10 +5,29 @@ import {
   ResourceTemplate,
 } from '@modelcontextprotocol/sdk/server/mcp.js';
 import Logger from './logger.js';
+import { fileURLToPath } from 'url';
+
+// Get the directory of the current module
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = path.dirname(__filename);
 
 // Configuration
 export const RESOURCES_DIR = path.join(process.cwd(), 'resources');
 
+// Try to use the package directory for resources if it exists
+export async function getResourcesDir(): Promise<string> {
+  const packageResourcesDir = path.join(__dirname, '../../resources');
+  try {
+    await fs.access(packageResourcesDir);
+    return packageResourcesDir;
+  } catch (
+    // eslint-disable-next-line @typescript-eslint/no-unused-vars
+    _error
+  ) {
+    return RESOURCES_DIR;
+  }
+}
+
 /**
  * Load a file's content from disk
  */
@@ -20,9 +39,9 @@ export async function loadFile(filePath: string): Promise<string> {
     span.end('success');
     return content;
   } catch (error) {
-    Logger.error(`Failed to load file: ${filePath}`, error);
+    Logger.error(`Failed to load file`, error, { path: filePath });
     span.end('error');
-    throw new Error(`Failed to load file: ${filePath}`);
+    throw error;
   }
 }
 
@@ -36,34 +55,39 @@ export async function loadResources(
   const resourceMap = new Map<string, string>();
 
   try {
-    await fs.mkdir(RESOURCES_DIR, { recursive: true });
-    Logger.debug(`Ensuring resources directory exists`, {
-      path: RESOURCES_DIR,
-    });
+    // Get the resources directory
+    const resourcesDir = await getResourcesDir();
+    await fs.mkdir(resourcesDir, { recursive: true });
+    Logger.debug(`Ensuring resources directory exists`, { path: resourcesDir });
 
-    const files = await fs.readdir(RESOURCES_DIR, { recursive: true });
+    const files = await fs.readdir(resourcesDir);
     Logger.info(`Found ${files.length} potential resource files`, {
-      directory: RESOURCES_DIR,
+      directory: resourcesDir,
     });
 
     let loadedCount = 0;
 
     for (const file of files) {
-      const fileSpan = Logger.span('processResourceFile', { file });
+      const resourceSpan = Logger.span('processResourceFile', { file });
+
+      if (!file.endsWith('.txt')) {
+        Logger.trace(`Skipping non-txt file`, { file });
+        resourceSpan.end('skipped_non_txt');
+        continue;
+      }
 
-      // Skip directories and non-text files
-      const filePath = path.join(RESOURCES_DIR, file);
+      const filePath = path.join(resourcesDir, file);
       const stats = await fs.stat(filePath);
 
       if (stats.isDirectory()) {
         Logger.trace(`Skipping directory`, { path: filePath });
-        fileSpan.end('skipped_directory');
+        resourceSpan.end('skipped_directory');
         continue;
       }
 
       // Convert file path to resource URI
       // e.g., resources/docs/intro.txt -> docs/intro
-      const relPath = path.relative(RESOURCES_DIR, filePath);
+      const relPath = path.relative(resourcesDir, filePath);
       const resourcePath = relPath.replace(/\.[^/.]+$/, ''); // Remove extension
       const resourceUri = `file://${resourcePath}`;
 
@@ -91,12 +115,12 @@ export async function loadResources(
           uri: resourceUri,
           size: content.length,
         });
-        fileSpan.end('success');
+        resourceSpan.end('success');
       } catch (error) {
         Logger.error(`Failed to process resource file`, error, {
           path: filePath,
         });
-        fileSpan.end('error');
+        resourceSpan.end('error');
       }
     }
 
@@ -110,7 +134,7 @@ export async function loadResources(
         const span = Logger.span('dynamicResourceLoad', { path: params.path });
         try {
           const resourcePath = params.path;
-          const filePath = path.join(RESOURCES_DIR, resourcePath + '.txt');
+          const filePath = path.join(resourcesDir, resourcePath + '.txt');
           const content = await loadFile(filePath);
           span.end('success');
           return {

src/server.ts

@@ -5,7 +5,7 @@ import { loadResources } from './resources.js';
 import { loadPrompts } from './prompts.js';
 import fs from 'fs/promises';
 import path from 'path';
-import { PROMPTS_DIR, parseMetadata } from './prompts.js';
+import { getPromptsDir, parseMetadata } from './prompts.js';
 
 // Server configuration
 const SERVER_CONFIG = {
@@ -20,8 +20,11 @@ export async function startServer(): Promise<void> {
   const span = Logger.span('startServer');
 
   try {
+    // Get the prompts directory
+    const promptsDir = await getPromptsDir();
+    
     // Load the sensei prompt content to use as server instructions
-    const senseiPromptPath = path.join(PROMPTS_DIR, 'sensei.txt');
+    const senseiPromptPath = path.join(promptsDir, 'sensei.txt');
     let senseiInstructions = '';
 
     try {