project-compass 4.3.2 → 4.3.3

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "project-compass",
-  "version": "4.3.2",
-  "description": "🧭 Futuristic TUI workspace navigator & runner - AI-powered project detection for Node, Python, Rust, Go, Java, PHP, Ruby, .NET",
+  "version": "4.3.3",
+  "description": "\ud83e\udded Futuristic TUI workspace navigator & runner - AI-powered project detection for Node, Python, Rust, Go, Java, PHP, Ruby, .NET",
   "main": "src/cli.js",
   "bin": {
     "project-compass": "src/cli.js"

package/src/detectors/python.js

@@ -23,27 +23,38 @@ function getPythonPackageManager(projectPath) {
 
 function gatherPythonDependencies(projectPath) {
   const deps = new Set();
-  const addFromRequirements = (filePath) => {
-    if (!fs.existsSync(filePath)) return;
-    const raw = fs.readFileSync(filePath, 'utf-8');
+  
+  // Only read requirements.txt files - NOT .py files!
+  const reqPath = path.join(projectPath, 'requirements.txt');
+  if (fs.existsSync(reqPath)) {
+    const raw = fs.readFileSync(reqPath, 'utf-8');
     raw.split(/\r?\n/).forEach((line) => {
       const clean = line.trim().split('#')[0].trim();
       if (!clean || clean.startsWith('-') || clean.startsWith('"') || clean.startsWith("'")) return;
       const match = clean.match(/^([a-zA-Z0-9_.-]+)/);
       if (match) deps.add(match[1].toLowerCase());
     });
-  };
-
-  addFromRequirements(path.join(projectPath, 'requirements.txt'));
-  addFromRequirements(path.join(projectPath, 'requirements-dev.txt'));
-
+  }
+  
+  const reqDevPath = path.join(projectPath, 'requirements-dev.txt');
+  if (fs.existsSync(reqDevPath)) {
+    const raw = fs.readFileSync(reqDevPath, 'utf-8');
+    raw.split(/\r?\n/).forEach((line) => {
+      const clean = line.trim().split('#')[0].trim();
+      if (!clean || clean.startsWith('-') || clean.startsWith('"') || clean.startsWith("'")) return;
+      const match = clean.match(/^([a-zA-Z0-9_.-]+)/);
+      if (match) deps.add(match[1].toLowerCase());
+    });
+  }
+  
+  // Only read pyproject.toml dependencies section - NOT .py files!
   const pyproject = path.join(projectPath, 'pyproject.toml');
   if (fs.existsSync(pyproject)) {
     const content = fs.readFileSync(pyproject, 'utf-8');
     const depSection = content.match(/(?:dependencies|requires)\s*=\s*\[([^\]]+)\]/g);
     if (depSection) {
       depSection.forEach((section) => {
-        const matches = section.match(/["']([^"']+)["']/g);
+        const matches = section.match(/["']([^"']+)/g);
         if (matches) {
           matches.forEach((m) => {
             const dep = m.replace(/["']/g, '').split(/[>=<=~!]/)[0].trim();
@@ -53,27 +64,16 @@ function gatherPythonDependencies(projectPath) {
       });
     }
   }
-
-  const pipfile = path.join(projectPath, 'Pipfile');
-  if (fs.existsSync(pipfile)) {
-    const content = fs.readFileSync(pipfile, 'utf-8');
-    const matches = content.match(/["']([^"']+)["']/g);
-    if (matches) {
-      matches.forEach((m) => {
-        const dep = m.replace(/["']/g, '').split(/[>=<=~!]/)[0].trim();
-        if (dep) deps.add(dep.toLowerCase());
-      });
-    }
-  }
-
+  
   return Array.from(deps);
 }
 
+// Uses EXACT matching - NOT substring!
 function detectPythonFramework(deps) {
   const frameworks = [];
   
   const hasDep = (pattern) => 
-    deps.some(dep => {
+    deps.some((dep) => {
       const depLower = dep.toLowerCase();
       return depLower === pattern.toLowerCase() ||
              depLower.startsWith(pattern.toLowerCase() + '==') ||
@@ -82,23 +82,23 @@ function detectPythonFramework(deps) {
              depLower.startsWith(pattern.toLowerCase() + '~=');
     });
   
-  if (hasDep('fastapi')) frameworks.push({ name: 'FastAPI', icon: '\u26A1' });
-  if (hasDep('flask')) frameworks.push({ name: 'Flask', icon: '\uD83C\uDF36\uFE0F' });
-  if (hasDep('django')) frameworks.push({ name: 'Django', icon: '\uD83C\uDF3F' });
-  if (hasDep('tornado')) frameworks.push({ name: 'Tornado', icon: '\uD83C\uDF2A\uFE0F' });
-  if (hasDep('aiohttp')) frameworks.push({ name: 'AioHTTP', icon: '\uD83D\uDD04' });
-  if (hasDep('sanic')) frameworks.push({ name: 'Sanic', icon: '\uD83D\uDE80' });
-  if (hasDep('pyramid')) frameworks.push({ name: 'Pyramid', icon: '\uD83D\uDE3A' });
-  if (hasDep('falcon')) frameworks.push({ name: 'Falcon', icon: '\uD83D\uDC05' });
-  if (hasDep('starlette')) frameworks.push({ name: 'Starlette', icon: '\u2B50' });
-  if (hasDep('pandas')) frameworks.push({ name: 'Pandas', icon: '\uD83D\uDC3C' });
-  if (hasDep('numpy')) frameworks.push({ name: 'NumPy', icon: '\uD83D\uDD22' });
-  if (hasDep('scipy')) frameworks.push({ name: 'SciPy', icon: '\uD83D\uDD2C' });
-  if (hasDep('torch') || hasDep('pytorch')) frameworks.push({ name: 'PyTorch', icon: '\uD83D\uDD25' });
-  if (hasDep('tensorflow')) frameworks.push({ name: 'TensorFlow', icon: '\uD83E\uDD20' });
-  if (hasDep('sqlalchemy')) frameworks.push({ name: 'SQLAlchemy', icon: '\uD83D\uDCC4\uFE0F' });
-  if (hasDep('pytest')) frameworks.push({ name: 'Pytest', icon: '\u2705' });
-  if (hasDep('celery')) frameworks.push({ name: 'Celery', icon: '\uD83D\uDE2C' });
+  if (hasDep('fastapi')) frameworks.push({ name: 'FastAPI', icon: '⚡' });
+  if (hasDep('flask')) frameworks.push({ name: 'Flask', icon: '🌶️' });
+  if (hasDep('django')) frameworks.push({ name: 'Django', icon: '🌿' });
+  if (hasDep('tornado')) frameworks.push({ name: 'Tornado', icon: '🌪️' });
+  if (hasDep('aiohttp')) frameworks.push({ name: 'AioHTTP', icon: '🔄' });
+  if (hasDep('sanic')) frameworks.push({ name: 'Sanic', icon: '🚀' });
+  if (hasDep('pyramid')) frameworks.push({ name: 'Pyramid', icon: '🔺' });
+  if (hasDep('falcon')) frameworks.push({ name: 'Falcon', icon: '🦅' });
+  if (hasDep('starlette')) frameworks.push({ name: 'Starlette', icon: '⭐' });
+  if (hasDep('pandas')) frameworks.push({ name: 'Pandas', icon: '🐼' });
+  if (hasDep('numpy')) frameworks.push({ name: 'NumPy', icon: '🔢' });
+  if (hasDep('scipy')) frameworks.push({ name: 'SciPy', icon: '🔬' });
+  if (hasDep('torch') || hasDep('pytorch')) frameworks.push({ name: 'PyTorch', icon: '🔥' });
+  if (hasDep('tensorflow')) frameworks.push({ name: 'TensorFlow', icon: '🧠' });
+  if (hasDep('sqlalchemy')) frameworks.push({ name: 'SQLAlchemy', icon: '🗄️' });
+  if (hasDep('pytest')) frameworks.push({ name: 'Pytest', icon: '✅' });
+  if (hasDep('celery')) frameworks.push({ name: 'Celery', icon: '🥬' });
   
   return frameworks;
 }
@@ -117,8 +117,10 @@ export default {
     const isPoetry = pkgManager === 'poetry';
     const isPipenv = pkgManager === 'pipenv';
     
+    const allDeps = gatherPythonDependencies(projectPath);
+    const detectedFrameworks = detectPythonFramework(allDeps);
+    
     const commands = {};
-
     if (isUV) {
       commands.install = { label: 'UV Sync', command: ['uv', 'sync'], source: 'builtin' };
       commands.add = { label: 'UV Add', command: ['uv', 'add'], source: 'builtin' };
@@ -134,8 +136,8 @@ export default {
       commands.install = { label: 'Pip Install', command: ['pip', 'install', '-r', 'requirements.txt'], source: 'builtin' };
     }
 
-    if (hasProjectFile(projectPath, 'pyproject.toml') || hasProjectFile(projectPath, 'setup.py')) {
-      commands.test = { label: 'Pytest', command: [isUV ? 'uv' : 'python', ...(isUV ? ['run'] : []), 'pytest'], source: 'builtin' };
+    if (isUV || isPoetry || isPipenv) {
+      commands.test = { label: 'Pytest', command: [isUV ? 'uv' : isPoetry ? 'poetry' : 'pipenv', ...(isUV ? ['run'] : []), 'pytest'], source: 'builtin' };
     } else {
       commands.test = { label: 'Unittest', command: ['python', '-m', 'unittest', 'discover'], source: 'builtin' };
     }
@@ -151,28 +153,16 @@ export default {
 
     if (hasProjectFile(projectPath, 'manage.py')) {
       const djangoCmd = isUV ? ['uv', 'run', 'python', 'manage.py'] :
-                        isPoetry ? ['poetry', 'run', 'python', 'manage.py'] :
-                        ['python', 'manage.py'];
+                           ['python', 'manage.py'];
       commands['runserver'] = { label: 'Django Runserver', command: [...djangoCmd, 'runserver'], source: 'builtin' };
       commands['migrate'] = { label: 'Django Migrate', command: [...djangoCmd, 'migrate'], source: 'builtin' };
       commands['test'] = { label: 'Django Test', command: [...djangoCmd, 'test'], source: 'builtin' };
     }
 
-    if (hasProjectFile(projectPath, 'fastapi') || hasProjectFile(projectPath, 'main.py')) {
-      if (hasProjectFile(projectPath, 'main.py')) {
-        const fastapiCmd = isUV ? ['uv', 'run', 'uvicorn', 'main:app', '--reload'] :
-                          ['uvicorn', 'main:app', '--reload'];
-        commands['dev'] = { label: 'FastAPI Dev', command: fastapiCmd, source: 'builtin' };
-      }
-    }
-
-    const allDeps = gatherPythonDependencies(projectPath);
-    const detectedFrameworks = detectPythonFramework(allDeps);
-
     const metadata = {
       dependencies: allDeps,
-      packageManager: pkgManager,
-      frameworks: detectedFrameworks
+      frameworks: detectedFrameworks,
+      packageManager: pkgManager
     };
 
     const setupHints = [];