smart-context-mcp 1.7.7 → 1.8.0

package/README.md

@@ -105,6 +105,29 @@ See [Task Runner Workflows](https://github.com/Arrayo/smart-context-mcp/blob/mai
 
 ---
 
+## When to Use (and When Not To)
+
+**Use devctx when:**
+- You're exploring an unfamiliar codebase
+- The task spans multiple sessions (checkpoints save context)
+- You need to understand how files relate to each other (graph/imports)
+- The context is too large to manage manually
+- You're doing complex multi-file refactors or debugging across layers
+
+**Skip devctx when:**
+- You already know exactly which files to touch
+- It's a single-file or surgical change (2-3 edits max)
+- You have the full mental map from a recent exploration
+- Native tools (Grep, Read, StrReplace) are more direct for the task
+
+**Honest verdict from real users:**
+
+> "The MCP shines in long, multi-session tasks or when you don't know the codebase. For contained refactors where you already know what to touch, native tools are just as fast or faster. The real value was `smart_read(outline)` for the initial analysis and checkpoints to not lose the thread between sessions."
+
+The 90% token savings are real, but they require the right task type to materialize.
+
+---
+
 ## 📊 Real Metrics
 
 **Production use on this project:**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "smart-context-mcp",
-  "version": "1.7.7",
+  "version": "1.8.0",
   "description": "MCP server that reduces agent token usage by 90% with intelligent context compression, task checkpoint persistence, and workflow-aware agent guidance.",
   "author": "Francisco Caballero Portero <fcp1978@hotmail.com>",
   "type": "module",

package/src/config/ignored-paths.js

@@ -19,3 +19,9 @@ export const IGNORED_FILE_NAMES = [
   'bun.lockb',
   'npm-shrinkwrap.json',
 ];
+
+export const IGNORED_FILE_PATTERNS = [
+  /\.min\.(js|css)$/,
+  /\.(map|snap)$/,
+  /^(questions|answers|fixtures|seed|dump|data)\.(json|jsonl|ndjson)$/i,
+];

package/src/index-manager.js

@@ -33,7 +33,7 @@ const saveIndexMetadata = (meta, root = projectRoot) => {
     }
     fs.writeFileSync(metaPath, JSON.stringify(meta, null, 2), 'utf8');
   } catch (error) {
-    console.warn('Failed to save index metadata:', error.message);
+    process.stderr.write(`[devctx] Failed to save index metadata: ${error.message}\n`);
   }
 };
 
@@ -65,31 +65,12 @@ const timeout = (ms, message) => {
   });
 };
 
-const isTestEnvironment = () => {
-  return process.env.NODE_ENV === 'test' || 
-         typeof process.env.NODE_TEST_CONTEXT !== 'undefined' ||
-         process.argv.some(arg => arg.includes('--test'));
-};
-
-const isMcpEnvironment = () => {
-  return process.env.MCP_SERVER === 'true' || 
-         process.argv.some(arg => arg.includes('devctx-server'));
-};
-
-const log = (message, level = 'info') => {
-  if (isTestEnvironment() || isMcpEnvironment()) {
-    return;
-  }
-  
-  if (level === 'warn') {
-    console.warn(message);
-  } else {
-    console.log(message);
-  }
+const log = (message) => {
+  process.stderr.write(`[devctx] ${message}\n`);
 };
 
 export const ensureIndexReady = async (options = {}) => {
-  const { force = false, timeoutMs = INDEX_BUILD_TIMEOUT_MS, root = projectRoot, silent = false } = options;
+  const { force = false, timeoutMs = INDEX_BUILD_TIMEOUT_MS, root = projectRoot } = options;
   
   if (!force) {
     const existingIndex = loadIndex(root);
@@ -101,9 +82,7 @@ export const ensureIndexReady = async (options = {}) => {
     }
   }
   
-  if (!silent) {
-    log('📦 Building search index (this may take 30-60s)...');
-  }
+  log('Building search index...');
   
   try {
     const buildPromise = buildIndexCore({ root, incremental: true });
@@ -119,14 +98,10 @@ export const ensureIndexReady = async (options = {}) => {
       version: result?.version
     }, root);
     
-    if (!silent) {
-      log('✅ Index ready');
-    }
+    log('Index ready');
     return { status: 'built', cached: false, fileCount: result?.files?.length || 0 };
   } catch (error) {
-    if (!silent) {
-      log('⚠️ Index build failed, search will use fallback mode', 'warn');
-    }
+    log(`Index build failed: ${error.message}`);
     return { status: 'fallback', error: error.message };
   }
 };

package/src/tools/smart-search.js

@@ -11,7 +11,7 @@ import { truncate } from '../utils/text.js';
 import { recordToolUsage } from '../usage-feedback.js';
 import { recordDecision, DECISION_REASONS, EXPECTED_BENEFITS } from '../decision-explainer.js';
 import { recordDevctxOperation } from '../missed-opportunities.js';
-import { IGNORED_DIRS, IGNORED_FILE_NAMES } from '../config/ignored-paths.js';
+import { IGNORED_DIRS, IGNORED_FILE_NAMES, IGNORED_FILE_PATTERNS } from '../config/ignored-paths.js';
 import { buildMetricsDisplay } from '../utils/metrics-display.js';
 import { createProgressReporter } from '../streaming.js';
 import { ensureIndexReady } from '../index-manager.js';
@@ -44,7 +44,12 @@ export const intentWeights = {
 
 const defaultWeights = intentWeights.explore;
 
-const shouldIgnoreFile = (filePath) => ignoredFileNames.has(path.basename(filePath));
+const shouldIgnoreFile = (filePath) => {
+  const base = path.basename(filePath);
+  if (ignoredFileNames.has(base)) return true;
+  if (IGNORED_FILE_PATTERNS.some((p) => p.test(base))) return true;
+  return false;
+};
 
 const isSearchableFile = (entryName, fullPath) => fallbackExtensions.has(path.extname(entryName)) || isDockerfile(fullPath);
 
@@ -92,6 +97,8 @@ const parseRgLine = (line, root) => {
   };
 };
 
+const MAX_FILE_SIZE = '1M';
+
 const searchWithRipgrep = async (root, query) => {
   const args = [
     '--line-number',
@@ -100,10 +107,12 @@ const searchWithRipgrep = async (root, query) => {
     'never',
     '--smart-case',
     '--fixed-strings',
+    '--max-filesize', MAX_FILE_SIZE,
   ];
 
   for (const dir of ignoredDirs) {
     args.push('--glob', `!${dir}/**`);
+    args.push('--glob', `!**/${dir}/**`);
   }
 
   for (const fileName of ignoredFileNames) {