@bike4mind/cli 0.2.17-ja-feat-slack-entity-extraction.17685 → 0.2.17

package/dist/index.js

@@ -3022,6 +3022,39 @@ SUBAGENT DELEGATION:
 - Subagents keep the main conversation clean and run faster with optimized models
 - Delegate when you need to search extensively or analyze code structure
 
+CODE SEARCH BEST PRACTICES:
+When searching code, follow this hierarchy for speed and efficiency:
+
+1. Start Narrow \u2192 Go Broad
+   \u2705 Efficient: ${TOOL_GLOB_FILES}("src/**/*.test.ts") \u2192 ${TOOL_GREP_SEARCH}("test('user login'") \u2192 ${TOOL_FILE_READ}("src/auth/login.test.ts")
+   \u274C Inefficient: ${TOOL_GREP_SEARCH}("login") \u2192 Read 50 files individually \u2192 Repeat with different term
+
+2. Leverage Git Information
+   Before searching, check recent changes:
+   - ${TOOL_BASH_EXECUTE}("git log --name-only --oneline -20")
+   - ${TOOL_BASH_EXECUTE}("git log --oneline -10 -- src/auth/")
+
+3. File Patterns
+   Use specific patterns instead of broad searches:
+   \u2705 Good: ${TOOL_GLOB_FILES}("**/*.{ts,tsx}"), ${TOOL_GLOB_FILES}("src/components/**/Button*")
+   \u274C Bad: ${TOOL_GLOB_FILES}("**/*"), ${TOOL_GREP_SEARCH}("auth")
+
+4. Test Files as Documentation
+   When learning about a feature, check tests first:
+   ${TOOL_GLOB_FILES}("**/*.test.{ts,tsx}") \u2192 ${TOOL_GREP_SEARCH}("describe('AuthProvider'") \u2192 ${TOOL_FILE_READ} test file \u2192 Read implementation
+
+5. Batch Operations
+   Prefer glob patterns over multiple calls:
+   \u2705 ${TOOL_GLOB_FILES}("src/**/*.{ts,tsx,js,jsx}") (one call)
+   \u274C 4 separate ${TOOL_GLOB_FILES} calls for each extension
+
+6. Tool Selection Decision Tree
+   Goal: Find where "AuthProvider" is defined
+   \u2192 ${TOOL_GLOB_FILES}("**/*Auth*.{ts,tsx}") (narrow the search)
+   \u2192 ${TOOL_GREP_SEARCH}("(class|interface|type) AuthProvider") (find exact location)
+   \u2192 ${TOOL_FILE_READ}("src/auth/AuthProvider.tsx") (read only that file)
+   Result: 3 tool calls instead of 10-15
+
 FOR GENERAL TASKS:
 - Use available tools to get information (weather, web search, calculations, etc.)
 - When user asks follow-up questions, use conversation context to understand what they're referring to
@@ -8509,40 +8542,39 @@ var globFilesTool = {
 };
 
 // ../../b4m-core/packages/services/dist/src/llm/tools/implementation/grepSearch/index.js
-import { globStream } from "glob";
-import { readFile, stat as stat2 } from "fs/promises";
+import { stat as stat2 } from "fs/promises";
 import path9 from "path";
-var MAX_FILE_SIZE3 = 10 * 1024 * 1024;
-var DEFAULT_IGNORE_PATTERNS3 = [
-  "**/node_modules/**",
-  "**/.git/**",
-  "**/dist/**",
-  "**/build/**",
-  "**/.next/**",
-  "**/coverage/**",
-  "**/*.min.js",
-  "**/*.min.css",
-  "**/package-lock.json",
-  "**/pnpm-lock.yaml",
-  "**/yarn.lock"
-];
-async function isBinaryFile2(filePath) {
+import { execFile } from "child_process";
+import { promisify } from "util";
+import { createRequire } from "module";
+var execFileAsync = promisify(execFile);
+var require2 = createRequire(import.meta.url);
+function getRipgrepPath() {
   try {
-    const { openSync, readSync, closeSync } = await import("fs");
-    const buffer = Buffer.alloc(8192);
-    const fd = openSync(filePath, "r");
-    const bytesRead = readSync(fd, buffer, 0, 8192, 0);
-    closeSync(fd);
-    const chunk = buffer.slice(0, bytesRead);
-    return chunk.includes(0);
-  } catch {
-    return true;
+    const ripgrepPath = require2.resolve("@vscode/ripgrep");
+    const ripgrepDir = path9.dirname(ripgrepPath);
+    const rgBinary = path9.join(ripgrepDir, "..", "bin", "rg" + (process.platform === "win32" ? ".exe" : ""));
+    return rgBinary;
+  } catch (error) {
+    throw new Error("ripgrep is not available. Please install @vscode/ripgrep: pnpm add @vscode/ripgrep --filter @bike4mind/services");
   }
 }
 function isPathWithinWorkspace(targetPath, baseCwd) {
   const resolvedTarget = path9.resolve(targetPath);
   const resolvedBase = path9.resolve(baseCwd);
-  return resolvedTarget.startsWith(resolvedBase);
+  const relativePath = path9.relative(resolvedBase, resolvedTarget);
+  return !relativePath.startsWith("..") && !path9.isAbsolute(relativePath);
+}
+function convertGlobToRipgrepGlobs(globPattern) {
+  if (!globPattern || globPattern === "**/*") {
+    return [];
+  }
+  const braceMatch = globPattern.match(/\*\.{([^}]+)}/);
+  if (braceMatch) {
+    const extensions = braceMatch[1].split(",");
+    return extensions.map((ext) => `*.${ext.trim()}`);
+  }
+  return [globPattern];
 }
 async function searchFiles2(params) {
   const { pattern, dir_path, include } = params;
@@ -8562,59 +8594,69 @@ async function searchFiles2(params) {
     }
     throw error;
   }
-  let searchRegex;
+  const rgPath = getRipgrepPath();
+  const rgArgs = [
+    "--json",
+    // Machine-readable output
+    "--max-count",
+    "500",
+    // Limit matches per file
+    "--ignore-case",
+    // Case-insensitive search
+    "--max-filesize",
+    "10M"
+    // Skip files larger than 10MB
+  ];
+  const globs = convertGlobToRipgrepGlobs(include || "");
+  if (globs.length > 0) {
+    globs.forEach((glob2) => {
+      rgArgs.push("--glob", glob2);
+    });
+  }
+  rgArgs.push(pattern, targetDir);
+  let stdout;
   try {
-    searchRegex = new RegExp(pattern, "i");
+    const result2 = await execFileAsync(rgPath, rgArgs, {
+      maxBuffer: 50 * 1024 * 1024
+      // 50MB buffer for large results
+    });
+    stdout = result2.stdout;
   } catch (error) {
-    throw new Error(`Invalid regex pattern: ${pattern}`);
+    const execError = error;
+    if (execError.code === 1 && execError.stdout) {
+      stdout = execError.stdout;
+    } else if (execError.code === 2) {
+      const errorMsg = execError.stderr || "Unknown ripgrep error";
+      throw new Error(`Ripgrep error: ${errorMsg}`);
+    } else {
+      throw error;
+    }
   }
-  const globPattern = include || "**/*";
-  const filesStream = globStream(globPattern, {
-    cwd: targetDir,
-    dot: false,
-    // Skip hidden files
-    ignore: DEFAULT_IGNORE_PATTERNS3,
-    absolute: true,
-    nodir: true
-    // Only match files, not directories
-  });
+  const lines = stdout.split("\n").filter(Boolean);
   const allMatches = [];
   let filesSearched = 0;
-  const maxMatches = 500;
-  for await (const filePath of filesStream) {
-    if (allMatches.length >= maxMatches) {
-      break;
-    }
-    const fileAbsPath = filePath;
+  for (const line of lines) {
     try {
-      const stats = await stat2(fileAbsPath);
-      if (stats.size > MAX_FILE_SIZE3) {
-        continue;
-      }
-      if (await isBinaryFile2(fileAbsPath)) {
-        continue;
-      }
-      filesSearched++;
-      const content = await readFile(fileAbsPath, "utf8");
-      const lines = content.split(/\r?\n/);
-      lines.forEach((line, index) => {
-        if (allMatches.length < maxMatches && searchRegex.test(line)) {
-          allMatches.push({
-            filePath: path9.relative(targetDir, fileAbsPath) || path9.basename(fileAbsPath),
-            lineNumber: index + 1,
-            line
-          });
-        }
-      });
-    } catch (readError) {
-      const errCode = readError.code;
-      if (errCode !== "ENOENT" && errCode !== "EACCES") {
-        console.debug(`Could not read ${fileAbsPath}: ${readError.message}`);
+      const item = JSON.parse(line);
+      if (item.type === "match") {
+        const match = item;
+        allMatches.push({
+          filePath: path9.relative(targetDir, match.data.path.text) || path9.basename(match.data.path.text),
+          lineNumber: match.data.line_number,
+          line: match.data.lines.text.trimEnd()
+          // Remove trailing newline
+        });
+      } else if (item.type === "summary") {
+        const stats = item;
+        filesSearched = stats.data.stats.searches || 0;
       }
+    } catch {
+      continue;
     }
   }
   const searchDirDisplay = dir_path || ".";
   const filterInfo = include ? ` (filter: "${include}")` : "";
+  const maxMatches = 500;
   if (allMatches.length === 0) {
     return `No matches found for pattern "${pattern}" in "${searchDirDisplay}"${filterInfo}.
 Searched ${filesSearched} file(s).`;
@@ -8624,7 +8666,6 @@ Searched ${filesSearched} file(s).`;
       acc[match.filePath] = [];
     }
     acc[match.filePath].push(match);
-    acc[match.filePath].sort((a, b) => a.lineNumber - b.lineNumber);
     return acc;
   }, {});
   const matchCount = allMatches.length;
@@ -9504,7 +9545,7 @@ var ToolErrorType;
 
 // src/utils/diffPreview.ts
 import * as Diff from "diff";
-import { readFile as readFile2 } from "fs/promises";
+import { readFile } from "fs/promises";
 import { existsSync as existsSync7 } from "fs";
 async function generateFileDiffPreview(args) {
   try {
@@ -9519,7 +9560,7 @@ ${preview}${hasMore ? `
 
 ... (${lines2.length - 20} more lines)` : ""}`;
     }
-    const currentContent = await readFile2(args.path, "utf-8");
+    const currentContent = await readFile(args.path, "utf-8");
     const patch = Diff.createPatch(
       args.path,
       currentContent,
@@ -11430,7 +11471,7 @@ import { isAxiosError as isAxiosError2 } from "axios";
 // package.json
 var package_default = {
   name: "@bike4mind/cli",
-  version: "0.2.17-ja-feat-slack-entity-extraction.17685+63c62fb5c",
+  version: "0.2.17",
   type: "module",
   description: "Interactive CLI tool for Bike4Mind with ReAct agents",
   license: "UNLICENSED",
@@ -11536,11 +11577,11 @@ var package_default = {
     zustand: "^4.5.4"
   },
   devDependencies: {
-    "@bike4mind/agents": "0.1.0",
-    "@bike4mind/common": "2.43.1-ja-feat-slack-entity-extraction.17685+63c62fb5c",
-    "@bike4mind/mcp": "1.22.3-ja-feat-slack-entity-extraction.17685+63c62fb5c",
-    "@bike4mind/services": "2.40.1-ja-feat-slack-entity-extraction.17685+63c62fb5c",
-    "@bike4mind/utils": "2.2.1-ja-feat-slack-entity-extraction.17685+63c62fb5c",
+    "@bike4mind/agents": "workspace:*",
+    "@bike4mind/common": "workspace:*",
+    "@bike4mind/mcp": "workspace:*",
+    "@bike4mind/services": "workspace:*",
+    "@bike4mind/utils": "workspace:*",
     "@types/better-sqlite3": "^7.6.13",
     "@types/diff": "^5.0.9",
     "@types/jsonwebtoken": "^9.0.4",
@@ -11554,7 +11595,9 @@ var package_default = {
     typescript: "^5.9.3",
     vitest: "^3.2.4"
   },
-  gitHead: "63c62fb5c81c2b1ff06c988e0edc9ae6d4445566"
+  optionalDependencies: {
+    "@vscode/ripgrep": "^1.17.0"
+  }
 };
 
 // src/config/constants.ts
@@ -11754,6 +11797,14 @@ Focus on:
 
 You have read-only access. Use file_read, grep_search, and glob_files to explore.
 
+CODE SEARCH EFFICIENCY:
+1. Start narrow with glob_files using specific patterns (e.g., "**/*Auth*.ts")
+2. Then use grep_search with precise regex on narrowed results
+3. Finally use file_read only on relevant files
+4. Leverage git log to find recent changes
+5. Check test files first to understand feature usage
+6. Batch glob operations instead of multiple separate calls
+
 When you find what you're looking for, provide a clear summary including:
 1. What you found (files, functions, patterns)
 2. Key insights or observations

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bike4mind/cli",
-  "version": "0.2.17-ja-feat-slack-entity-extraction.17685+63c62fb5c",
+  "version": "0.2.17",
   "type": "module",
   "description": "Interactive CLI tool for Bike4Mind with ReAct agents",
   "license": "UNLICENSED",
@@ -30,16 +30,6 @@
     "dist",
     "bin"
   ],
-  "scripts": {
-    "dev": "tsx src/index.tsx",
-    "build": "tsup",
-    "typecheck": "tsc --noEmit",
-    "test": "vitest run",
-    "test:watch": "vitest",
-    "start": "node dist/index.js",
-    "prepublishOnly": "pnpm build",
-    "postinstall": "node -e \"try { require('better-sqlite3') } catch(e) { if(e.message.includes('bindings')) { console.log('\\n⚠️  Rebuilding better-sqlite3 native bindings...'); require('child_process').execSync('pnpm rebuild better-sqlite3', {stdio:'inherit'}) } }\""
-  },
   "dependencies": {
     "@anthropic-ai/sdk": "^0.22.0",
     "@aws-sdk/client-apigatewaymanagementapi": "3.654.0",
@@ -106,11 +96,6 @@
     "zustand": "^4.5.4"
   },
   "devDependencies": {
-    "@bike4mind/agents": "0.1.0",
-    "@bike4mind/common": "2.43.1-ja-feat-slack-entity-extraction.17685+63c62fb5c",
-    "@bike4mind/mcp": "1.22.3-ja-feat-slack-entity-extraction.17685+63c62fb5c",
-    "@bike4mind/services": "2.40.1-ja-feat-slack-entity-extraction.17685+63c62fb5c",
-    "@bike4mind/utils": "2.2.1-ja-feat-slack-entity-extraction.17685+63c62fb5c",
     "@types/better-sqlite3": "^7.6.13",
     "@types/diff": "^5.0.9",
     "@types/jsonwebtoken": "^9.0.4",
@@ -122,7 +107,23 @@
     "tsup": "^8.5.1",
     "tsx": "^4.21.0",
     "typescript": "^5.9.3",
-    "vitest": "^3.2.4"
+    "vitest": "^3.2.4",
+    "@bike4mind/agents": "0.1.0",
+    "@bike4mind/common": "2.43.0",
+    "@bike4mind/mcp": "1.22.3",
+    "@bike4mind/services": "2.41.0",
+    "@bike4mind/utils": "2.2.0"
+  },
+  "optionalDependencies": {
+    "@vscode/ripgrep": "^1.17.0"
   },
-  "gitHead": "63c62fb5c81c2b1ff06c988e0edc9ae6d4445566"
-}
+  "scripts": {
+    "dev": "tsx src/index.tsx",
+    "build": "tsup",
+    "typecheck": "tsc --noEmit",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "start": "node dist/index.js",
+    "postinstall": "node -e \"try { require('better-sqlite3') } catch(e) { if(e.message.includes('bindings')) { console.log('\\n⚠️  Rebuilding better-sqlite3 native bindings...'); require('child_process').execSync('pnpm rebuild better-sqlite3', {stdio:'inherit'}) } }\""
+  }
+}