token-pilot 0.14.2 → 0.16.0

package/dist/ast-index/client.js

@@ -1,9 +1,12 @@
 import { execFile } from 'node:child_process';
 import { promisify } from 'node:util';
-import { stat } from 'node:fs/promises';
-import { createHash } from 'node:crypto';
-import { readFile } from 'node:fs/promises';
 import { findBinary, installBinary } from './binary-manager.js';
+import { parseFileCount, parseOutlineText, parseImportsText, parseImplementationsText, parseHierarchyText, parseAgrepText, parseTodoText, parseDeprecatedText, parseAnnotationsText, parseModuleListText, parseModuleDepText, parseUnusedDepsText, parseModuleApiText, } from './parser.js';
+import { buildFileStructure } from './enricher.js';
+import { parseTypeScriptRegex } from './regex-parser.js';
+import { parsePythonRegex } from './regex-parser-python.js';
+const TS_JS_EXTENSIONS = new Set(['ts', 'tsx', 'js', 'jsx', 'mjs', 'cjs']);
+const PYTHON_EXTENSIONS = new Set(['py', 'pyw']);
 const execFileAsync = promisify(execFile);
 export class AstIndexClient {
     static MAX_INDEX_FILES = 50_000;
@@ -51,12 +54,10 @@ export class AstIndexClient {
     async ensureIndex() {
         if (this.indexed)
             return;
-        // Project root is too broad (/, home dir) — refuse to build
         if (this.indexDisabled) {
             throw new Error('ast-index: index build disabled — project root is too broad (e.g. /). ' +
                 'Configure mcpServers with "args": ["/path/to/project"] to set the correct project root.');
         }
-        // If a previous build found >50k files, don't retry
         if (this.indexOversized) {
             throw new Error('ast-index disabled: previous build indexed >50k files (likely node_modules). ' +
                 'Ensure node_modules is in .gitignore, then restart the MCP server.');
@@ -73,14 +74,12 @@ export class AstIndexClient {
         }
     }
     async buildIndex() {
-        // Check if index already exists and has files
         let existingFileCount = 0;
         try {
             const stats = await this.exec(['--format', 'json', 'stats']);
-            existingFileCount = this.parseFileCount(stats);
+            existingFileCount = parseFileCount(stats);
         }
         catch { /* no index yet */ }
-        // Guard: existing index is oversized (node_modules leak from previous build)
         if (existingFileCount > AstIndexClient.MAX_INDEX_FILES) {
             console.error(`[token-pilot] ast-index: existing index has ${existingFileCount} files (>${AstIndexClient.MAX_INDEX_FILES}) — likely includes node_modules. Clearing.`);
             try {
@@ -88,19 +87,15 @@ export class AstIndexClient {
             }
             catch { /* best effort */ }
             existingFileCount = 0;
-            // Fall through to rebuild — maybe .gitignore was fixed
         }
         if (existingFileCount > 0) {
-            // Index exists — use incremental update (fast)
             console.error(`[token-pilot] ast-index: updating index (${existingFileCount} files)...`);
             try {
                 await this.exec(['update'], 30000);
-                // Re-check count after update
                 try {
-                    existingFileCount = this.parseFileCount(await this.exec(['--format', 'json', 'stats']));
+                    existingFileCount = parseFileCount(await this.exec(['--format', 'json', 'stats']));
                 }
                 catch { /* keep previous count */ }
-                // Guard: update may have grown index beyond limit
                 if (existingFileCount > AstIndexClient.MAX_INDEX_FILES) {
                     return this.handleOversizedIndex(existingFileCount);
                 }
@@ -112,12 +107,10 @@ export class AstIndexClient {
                 console.error(`[token-pilot] ast-index: update failed, falling back to rebuild — ${updateErr instanceof Error ? updateErr.message : updateErr}`);
             }
         }
-        // No index or update failed — full rebuild
         console.error('[token-pilot] ast-index: building index (this may take a moment)...');
         try {
             await this.exec(['rebuild'], 120000);
-            const fileCount = this.parseFileCount(await this.exec(['--format', 'json', 'stats']).catch(() => ''));
-            // Guard: rebuild produced oversized index
+            const fileCount = parseFileCount(await this.exec(['--format', 'json', 'stats']).catch(() => ''));
             if (fileCount > AstIndexClient.MAX_INDEX_FILES) {
                 return this.handleOversizedIndex(fileCount);
             }
@@ -125,10 +118,9 @@ export class AstIndexClient {
             console.error(`[token-pilot] ast-index: index built (${fileCount} files)`);
         }
         catch (buildErr) {
-            // If rebuild failed due to lock, check if index is usable anyway
             const errMsg = buildErr instanceof Error ? buildErr.message : String(buildErr);
             if (errMsg.includes('lock') || errMsg.includes('already running')) {
-                const count = this.parseFileCount(await this.exec(['--format', 'json', 'stats']).catch(() => ''));
+                const count = parseFileCount(await this.exec(['--format', 'json', 'stats']).catch(() => ''));
                 if (count > 0 && count <= AstIndexClient.MAX_INDEX_FILES) {
                     this.indexed = true;
                     console.error(`[token-pilot] ast-index: using existing index (${count} files, rebuild skipped due to lock)`);
@@ -142,7 +134,6 @@ export class AstIndexClient {
             throw buildErr;
         }
     }
-    /** Mark index as oversized — disables index-dependent tools, outline still works */
     async handleOversizedIndex(fileCount) {
         this.indexOversized = true;
         this.indexed = false;
@@ -156,117 +147,52 @@ export class AstIndexClient {
             `  → Tools disabled: find_unused, find_usages, related_files, project_overview\n` +
             `  → Tools still working: outline, smart_read, smart_read_many, read_symbol`);
     }
-    /** Extract file count from stats output (JSON or text) */
-    parseFileCount(statsText) {
-        // Try JSON first (--format json)
-        try {
-            const json = JSON.parse(statsText);
-            if (json?.stats?.file_count !== undefined)
-                return json.stats.file_count;
-        }
-        catch { /* not JSON, fall through */ }
-        // Fallback: text format
-        const match = statsText.match(/Files:\s*(\d+)/);
-        return match ? parseInt(match[1], 10) : 0;
-    }
     async outline(filePath) {
-        // outline parses a single file — try directly without requiring full index
         try {
             const result = await this.exec(['outline', filePath]);
-            const entries = this.parseOutlineText(result);
+            const entries = parseOutlineText(result);
             if (entries.length === 0)
                 return null;
-            return await this.buildFileStructure(filePath, entries);
+            return await buildFileStructure(filePath, entries);
         }
         catch {
-            // Direct call failed — try building index first (unless disabled/oversized)
             if (this.indexDisabled || this.indexOversized)
-                return null;
+                return this.regexFallback(filePath);
             try {
                 await this.ensureIndex();
                 const result = await this.exec(['outline', filePath]);
-                const entries = this.parseOutlineText(result);
+                const entries = parseOutlineText(result);
                 if (entries.length === 0)
                     return null;
-                return await this.buildFileStructure(filePath, entries);
+                return await buildFileStructure(filePath, entries);
             }
             catch (err) {
                 console.error(`[token-pilot] ast-index outline failed for ${filePath}: ${err instanceof Error ? err.message : err}`);
-                return null;
+                return this.regexFallback(filePath);
             }
         }
     }
-    /**
-     * Parse text output from `ast-index outline`:
-     *   Outline of src/file.ts:
-     *     :10 ClassName [class]
-     *     :11 propName [property]
-     *     :14 methodName [function]
-     */
-    parseOutlineText(text) {
-        const lines = text.split('\n');
-        const entries = [];
-        const classStack = [];
-        for (const line of lines) {
-            // Match: optional whitespace, :LINE_NUM, SYMBOL_NAME, [KIND]
-            const match = line.match(/^(\s*):(\d+)\s+(\S+)\s+\[(\w+)\]/);
-            if (!match)
-                continue;
-            const indent = match[1].length;
-            const entry = {
-                name: match[3],
-                kind: match[4],
-                start_line: parseInt(match[2], 10),
-                end_line: 0, // computed later
-            };
-            // Pop stack until we find a parent with less indent
-            while (classStack.length > 0 && classStack[classStack.length - 1].indent >= indent) {
-                classStack.pop();
-            }
-            if (classStack.length > 0) {
-                // This is a child of the top of stack
-                const parent = classStack[classStack.length - 1].entry;
-                if (!parent.children)
-                    parent.children = [];
-                parent.children.push(entry);
-            }
-            else {
-                entries.push(entry);
-            }
-            // Push classes/interfaces onto stack as potential parents
-            if (['class', 'interface', 'struct', 'enum', 'impl', 'trait', 'namespace', 'module'].includes(entry.kind.toLowerCase())) {
-                classStack.push({ entry, indent });
-            }
+    /** Regex-based fallback for TS/JS/Python when ast-index binary is unavailable. */
+    async regexFallback(filePath) {
+        const ext = filePath.split('.').pop()?.toLowerCase() ?? '';
+        const parser = TS_JS_EXTENSIONS.has(ext) ? parseTypeScriptRegex
+            : PYTHON_EXTENSIONS.has(ext) ? parsePythonRegex
+                : null;
+        if (!parser)
+            return null;
+        try {
+            const { readFile } = await import('node:fs/promises');
+            const content = await readFile(filePath, 'utf-8');
+            const entries = parser(content);
+            if (entries.length === 0)
+                return null;
+            return await buildFileStructure(filePath, entries);
         }
-        // Compute end_line for all entries
-        this.computeEndLines(entries);
-        return entries;
-    }
-    /** Compute end_line from sequential start positions */
-    computeEndLines(entries) {
-        for (let i = 0; i < entries.length; i++) {
-            // Children first (recursive)
-            if (entries[i].children?.length) {
-                this.computeEndLines(entries[i].children);
-            }
-            if (i < entries.length - 1) {
-                // end = next sibling's start - 1
-                entries[i].end_line = entries[i + 1].start_line - 1;
-            }
-            else {
-                // last entry: estimate based on children or use start + reasonable default
-                const children = entries[i].children;
-                if (children?.length) {
-                    entries[i].end_line = children[children.length - 1].end_line + 1;
-                }
-                else {
-                    entries[i].end_line = entries[i].start_line + 10; // estimated
-                }
-            }
+        catch {
+            return null;
         }
     }
     async symbol(name) {
-        // Try directly first (works if index exists from a previous session)
         try {
             const result = await this.exec(['symbol', name, '--format', 'json']);
             const raw = JSON.parse(result);
@@ -276,7 +202,6 @@ export class AstIndexClient {
             }
         }
         catch { /* fall through to ensureIndex path */ }
-        // Direct call failed — try building index (unless disabled/oversized)
         if (this.indexDisabled || this.indexOversized)
             return null;
         try {
@@ -317,7 +242,6 @@ export class AstIndexClient {
                 })) : []),
                 ...(Array.isArray(parsed.references) ? parsed.references : []),
             ];
-            // Fallback: if parsed is an array directly
             const matches = all.length > 0 ? all : (Array.isArray(parsed) ? parsed : []);
             const mapped = matches
                 .map((m) => ({
@@ -326,7 +250,7 @@ export class AstIndexClient {
                 text: m.content ?? m.text ?? m.signature ?? '',
             }))
                 .filter(r => r.file !== '' && r.text !== '');
-            // Deduplicate by file:line (merge of 4 categories creates dupes)
+            // Deduplicate by file:line
             const seen = new Set();
             return mapped.filter(r => {
                 const key = `${r.file}:${r.line}`;
@@ -348,12 +272,7 @@ export class AstIndexClient {
             const raw = JSON.parse(result);
             if (!Array.isArray(raw))
                 return [];
-            return raw.map(u => ({
-                file: u.path,
-                line: u.line,
-                text: u.context,
-                kind: 'reference',
-            }));
+            return raw.map(u => ({ file: u.path, line: u.line, text: u.context, kind: 'reference' }));
         }
         catch (err) {
             console.error(`[token-pilot] ast-index usages failed: ${err instanceof Error ? err.message : err}`);
@@ -368,8 +287,7 @@ export class AstIndexClient {
                 return JSON.parse(result);
             }
             catch {
-                // JSON parse failed — parse text format as fallback
-                return this.parseImplementationsText(result);
+                return parseImplementationsText(result);
             }
         }
         catch (err) {
@@ -385,8 +303,7 @@ export class AstIndexClient {
                 return JSON.parse(result);
             }
             catch {
-                // JSON parse failed — parse text format as fallback
-                return this.parseHierarchyText(result, name);
+                return parseHierarchyText(result, name);
             }
         }
         catch (err) {
@@ -394,62 +311,6 @@ export class AstIndexClient {
             return null;
         }
     }
-    parseImplementationsText(text) {
-        const results = [];
-        // Parse lines like: "class ClassName (file.php:42)"
-        for (const line of text.split('\n')) {
-            const m = line.match(/^\s*(class|interface|trait|struct|impl)\s+(\S+)\s+\((.+):(\d+)\)/);
-            if (m) {
-                results.push({ kind: m[1], name: m[2], file: m[3], line: parseInt(m[4], 10) });
-            }
-        }
-        return results;
-    }
-    parseHierarchyText(text, rootName) {
-        if (!text.trim())
-            return null;
-        // Parse ast-index hierarchy text output:
-        //   Hierarchy for 'ClassName':
-        //     Parents:
-        //       ParentClass (extends)
-        //     Children:
-        //       ChildClass (implements)  (file.ts:42)
-        const lines = text.split('\n');
-        const parents = [];
-        const childNodes = [];
-        let section = 'none';
-        for (const line of lines) {
-            const trimmed = line.trim();
-            if (trimmed === 'Parents:') {
-                section = 'parents';
-                continue;
-            }
-            if (trimmed === 'Children:') {
-                section = 'children';
-                continue;
-            }
-            if (trimmed.startsWith('Hierarchy for') || !trimmed)
-                continue;
-            // Match: SymbolName (relationship)  (file:line) — file:line is optional
-            const m = trimmed.match(/^(\S+)\s+\((\w+)\)(?:\s+\((.+):(\d+)\))?/);
-            if (m && section !== 'none') {
-                const node = {
-                    name: m[1],
-                    kind: m[2], // extends, implements, etc.
-                    children: [],
-                    file: m[3],
-                    line: m[4] ? parseInt(m[4], 10) : undefined,
-                };
-                if (section === 'parents')
-                    parents.push(node);
-                else
-                    childNodes.push(node);
-            }
-        }
-        if (parents.length === 0 && childNodes.length === 0)
-            return null;
-        return { name: rootName, kind: 'class', children: childNodes, parents };
-    }
     async stats() {
         try {
             return await this.exec(['stats']);
@@ -458,10 +319,6 @@ export class AstIndexClient {
             return null;
         }
     }
-    /**
-     * List all files known to the ast-index.
-     * Parses the `files` command output which lists one file per line.
-     */
     async listFiles() {
         try {
             await this.ensureIndex();
@@ -473,10 +330,6 @@ export class AstIndexClient {
             return [];
         }
     }
-    /**
-     * Cross-references: definitions + imports + usages in one call.
-     * Replaces separate symbol() + usages() calls.
-     */
     async refs(symbolName, limit = 20) {
         await this.ensureIndex();
         try {
@@ -488,9 +341,6 @@ export class AstIndexClient {
             return { definitions: [], imports: [], usages: [] };
         }
     }
-    /**
-     * Project map: directory structure with file counts and symbol kinds.
-     */
     async map(options) {
         await this.ensureIndex();
         try {
@@ -507,9 +357,6 @@ export class AstIndexClient {
             return null;
         }
     }
-    /**
-     * Detect project conventions: architecture, frameworks, naming patterns.
-     */
     async conventions() {
         await this.ensureIndex();
         try {
@@ -521,9 +368,6 @@ export class AstIndexClient {
             return null;
         }
     }
-    /**
-     * Find callers of a function.
-     */
     async callers(functionName, limit = 50) {
         await this.ensureIndex();
         try {
@@ -536,9 +380,6 @@ export class AstIndexClient {
             return [];
         }
     }
-    /**
-     * Show call hierarchy tree (callers tree up).
-     */
     async callTree(functionName, depth = 3) {
         await this.ensureIndex();
         try {
@@ -550,9 +391,6 @@ export class AstIndexClient {
             return null;
         }
     }
-    /**
-     * Show changed symbols since base branch (git diff).
-     */
     async changed(base) {
         await this.ensureIndex();
         try {
@@ -568,9 +406,6 @@ export class AstIndexClient {
             return [];
         }
     }
-    /**
-     * Find potentially unused symbols.
-     */
     async unusedSymbols(options) {
         await this.ensureIndex();
         try {
@@ -590,72 +425,27 @@ export class AstIndexClient {
             return [];
         }
     }
-    /**
-     * Get imports for a specific file.
-     * Parses text output: "  { X, Y } from 'source';"
-     */
     async fileImports(filePath) {
         await this.ensureIndex();
         try {
             const result = await this.exec(['imports', filePath]);
-            return this.parseImportsText(result);
+            return parseImportsText(result);
         }
         catch (err) {
             console.error(`[token-pilot] ast-index imports failed: ${err instanceof Error ? err.message : err}`);
             return [];
         }
     }
-    parseImportsText(text) {
-        const entries = [];
-        for (const line of text.split('\n')) {
-            const trimmed = line.trim();
-            if (!trimmed || trimmed.startsWith('Imports in') || trimmed.startsWith('Total:'))
-                continue;
-            // Match: { X, Y } from 'source'
-            const braceMatch = trimmed.match(/^\{\s*(.+?)\s*\}\s+from\s+['"](.+?)['"]/);
-            if (braceMatch) {
-                entries.push({
-                    specifiers: braceMatch[1].split(',').map(s => s.trim()),
-                    source: braceMatch[2],
-                });
-                continue;
-            }
-            // Match: * as X from 'source'
-            const nsMatch = trimmed.match(/^\*\s+as\s+(\S+)\s+from\s+['"](.+?)['"]/);
-            if (nsMatch) {
-                entries.push({
-                    specifiers: [nsMatch[1]],
-                    source: nsMatch[2],
-                    isNamespace: true,
-                });
-                continue;
-            }
-            // Match: X from 'source' (default import)
-            const defaultMatch = trimmed.match(/^(\w+)\s+from\s+['"](.+?)['"]/);
-            if (defaultMatch) {
-                entries.push({
-                    specifiers: [defaultMatch[1]],
-                    source: defaultMatch[2],
-                    isDefault: true,
-                });
-                continue;
-            }
-        }
-        return entries;
-    }
     // --- Code audit commands ---
-    /** Check if ast-grep (sg) is available for structural pattern search */
     async checkAstGrep() {
         if (this.astGrepAvailable !== null)
             return this.astGrepAvailable;
-        // Try system PATH first
         try {
             await execFileAsync('sg', ['--version'], { timeout: 3000 });
             this.astGrepAvailable = true;
             return true;
         }
         catch { /* not in PATH */ }
-        // Try node_modules/.bin/sg (from optionalDependencies or source installs)
         try {
             const localBinDir = new URL('../../node_modules/.bin', import.meta.url).pathname;
             await execFileAsync(localBinDir + '/sg', ['--version'], { timeout: 3000 });
@@ -667,7 +457,6 @@ export class AstIndexClient {
         this.astGrepAvailable = false;
         return false;
     }
-    /** Structural pattern search via ast-grep. Requires ast-grep (sg) installed. */
     async agrep(pattern, options) {
         if (this.indexDisabled || this.indexOversized)
             return [];
@@ -684,129 +473,52 @@ export class AstIndexClient {
             args.push('--lang', options.lang);
         try {
             const result = await this.exec(args, 15000);
-            return this.parseAgrepText(result).slice(0, limit);
+            return parseAgrepText(result).slice(0, limit);
         }
         catch (err) {
             console.error(`[token-pilot] ast-index agrep failed: ${err instanceof Error ? err.message : err}`);
             return [];
         }
     }
-    parseAgrepText(text) {
-        const results = [];
-        for (const line of text.split('\n')) {
-            if (!line.trim())
-                continue;
-            // Format: file:line:matched_text  OR  file:line: matched_text
-            const match = line.match(/^(.+?):(\d+):(.*)$/);
-            if (match) {
-                results.push({
-                    file: match[1],
-                    line: parseInt(match[2], 10),
-                    text: match[3].trim(),
-                });
-            }
-        }
-        return results;
-    }
-    /** Find TODO/FIXME/HACK comments in the project */
     async todo() {
         if (this.indexDisabled || this.indexOversized)
             return [];
         await this.ensureIndex();
         try {
             const result = await this.exec(['todo'], 15000);
-            return this.parseTodoText(result);
+            return parseTodoText(result);
         }
         catch (err) {
             console.error(`[token-pilot] ast-index todo failed: ${err instanceof Error ? err.message : err}`);
             return [];
         }
     }
-    parseTodoText(text) {
-        const results = [];
-        for (const line of text.split('\n')) {
-            if (!line.trim())
-                continue;
-            // Try format: file:line: KIND: message  OR  file:line: KIND message
-            const match = line.match(/^(.+?):(\d+):\s*(TODO|FIXME|HACK|XXX|NOTE|WARN(?:ING)?)[:\s]+(.*)$/i);
-            if (match) {
-                results.push({
-                    file: match[1],
-                    line: parseInt(match[2], 10),
-                    kind: match[3].toUpperCase(),
-                    text: match[4].trim(),
-                });
-            }
-        }
-        return results;
-    }
-    /** Find @Deprecated symbols in the project */
     async deprecated() {
         if (this.indexDisabled || this.indexOversized)
             return [];
         await this.ensureIndex();
         try {
             const result = await this.exec(['deprecated'], 15000);
-            return this.parseDeprecatedText(result);
+            return parseDeprecatedText(result);
         }
         catch (err) {
             console.error(`[token-pilot] ast-index deprecated failed: ${err instanceof Error ? err.message : err}`);
             return [];
         }
     }
-    parseDeprecatedText(text) {
-        const results = [];
-        for (const line of text.split('\n')) {
-            if (!line.trim())
-                continue;
-            // Try format: kind name (file:line) - message  OR  kind name (file:line)
-            const match = line.match(/^(\w+)\s+(\S+)\s+\((.+?):(\d+)\)(?:\s*-\s*(.+))?$/);
-            if (match) {
-                results.push({
-                    kind: match[1],
-                    name: match[2],
-                    file: match[3],
-                    line: parseInt(match[4], 10),
-                    message: match[5]?.trim(),
-                });
-            }
-        }
-        return results;
-    }
-    /** Find symbols with a specific annotation/decorator */
     async annotations(name) {
         if (this.indexDisabled || this.indexOversized)
             return [];
         await this.ensureIndex();
         try {
             const result = await this.exec(['annotations', name], 15000);
-            return this.parseAnnotationsText(result, name);
+            return parseAnnotationsText(result, name);
         }
         catch (err) {
             console.error(`[token-pilot] ast-index annotations failed: ${err instanceof Error ? err.message : err}`);
             return [];
         }
     }
-    parseAnnotationsText(text, annotationName) {
-        const results = [];
-        for (const line of text.split('\n')) {
-            if (!line.trim())
-                continue;
-            // Try format: kind name (file:line)  OR  @Annotation kind name (file:line)
-            const match = line.match(/^(?:@\S+\s+)?(\w+)\s+(\S+)\s+\((.+?):(\d+)\)$/);
-            if (match) {
-                results.push({
-                    kind: match[1],
-                    name: match[2],
-                    file: match[3],
-                    line: parseInt(match[4], 10),
-                    annotation: annotationName,
-                });
-            }
-        }
-        return results;
-    }
-    /** Trigger incremental index update (called by file watcher after edits) */
     async incrementalUpdate() {
         if (!this.indexed || this.indexDisabled || this.indexOversized)
             return;
@@ -817,8 +529,7 @@ export class AstIndexClient {
             console.error(`[token-pilot] ast-index incremental update failed: ${err instanceof Error ? err.message : err}`);
         }
     }
-    // --- Module analysis methods (ast-index v3.27.0) ---
-    /** List project modules matching optional pattern */
+    // --- Module analysis methods ---
     async modules(pattern) {
         if (this.indexDisabled || this.indexOversized)
             return [];
@@ -826,207 +537,89 @@ export class AstIndexClient {
         try {
             const cmdArgs = pattern ? ['module', pattern] : ['module'];
             const result = await this.exec(cmdArgs, 15000);
-            return this.parseModuleListText(result);
+            return parseModuleListText(result);
         }
         catch (err) {
             console.error(`[token-pilot] ast-index module failed: ${err instanceof Error ? err.message : err}`);
             return [];
         }
     }
-    /** Get dependencies of a module */
     async moduleDeps(module) {
         if (this.indexDisabled || this.indexOversized)
             return [];
         await this.ensureIndex();
         try {
             const result = await this.exec(['deps', module], 15000);
-            return this.parseModuleDepText(result);
+            return parseModuleDepText(result);
         }
         catch (err) {
             console.error(`[token-pilot] ast-index deps failed: ${err instanceof Error ? err.message : err}`);
             return [];
         }
     }
-    /** Get modules that depend on this module */
     async moduleDependents(module) {
         if (this.indexDisabled || this.indexOversized)
             return [];
         await this.ensureIndex();
         try {
             const result = await this.exec(['dependents', module], 15000);
-            return this.parseModuleDepText(result);
+            return parseModuleDepText(result);
         }
         catch (err) {
             console.error(`[token-pilot] ast-index dependents failed: ${err instanceof Error ? err.message : err}`);
             return [];
         }
     }
-    /** Find unused dependencies of a module */
     async unusedDeps(module) {
         if (this.indexDisabled || this.indexOversized)
             return [];
         await this.ensureIndex();
         try {
             const result = await this.exec(['unused-deps', module], 15000);
-            return this.parseUnusedDepsText(result);
+            return parseUnusedDepsText(result);
         }
         catch (err) {
             console.error(`[token-pilot] ast-index unused-deps failed: ${err instanceof Error ? err.message : err}`);
             return [];
         }
     }
-    /** Get public API of a module */
     async moduleApi(module) {
         if (this.indexDisabled || this.indexOversized)
             return [];
         await this.ensureIndex();
         try {
             const result = await this.exec(['api', module], 15000);
-            return this.parseModuleApiText(result);
+            return parseModuleApiText(result);
         }
         catch (err) {
             console.error(`[token-pilot] ast-index api failed: ${err instanceof Error ? err.message : err}`);
             return [];
         }
     }
-    // Parsers for module commands (text format — JSON may not be supported for all)
-    parseModuleListText(text) {
-        const results = [];
-        for (const line of text.split('\n')) {
-            if (!line.trim())
-                continue;
-            // Try JSON first
-            try {
-                const parsed = JSON.parse(line);
-                if (Array.isArray(parsed))
-                    return parsed;
-            }
-            catch { /* not JSON, parse as text */ }
-            // Format: name (path) — N files  OR  name (path)  OR  path
-            const match = line.match(/^(\S+)\s+\((.+?)\)(?:\s*—\s*(\d+)\s+files?)?$/);
-            if (match) {
-                results.push({
-                    name: match[1],
-                    path: match[2],
-                    file_count: match[3] ? parseInt(match[3], 10) : undefined,
-                });
-            }
-            else {
-                // Fallback: treat entire line as a path-based module
-                const trimmed = line.trim();
-                if (trimmed && !trimmed.startsWith('#') && !trimmed.startsWith('─')) {
-                    const name = trimmed.split('/').pop() ?? trimmed;
-                    results.push({ name, path: trimmed });
-                }
-            }
-        }
-        return results;
-    }
-    parseModuleDepText(text) {
-        const results = [];
-        for (const line of text.split('\n')) {
-            if (!line.trim())
-                continue;
-            // Try JSON first
-            try {
-                const parsed = JSON.parse(line);
-                if (Array.isArray(parsed))
-                    return parsed;
-            }
-            catch { /* not JSON, parse as text */ }
-            // Format: → name (path)  OR  ← name (path)  OR  name (path)  OR  name
-            const match = line.match(/^[→←\-\s]*(\S+)(?:\s+\((.+?)\))?(?:\s+\[(direct|transitive)\])?$/);
-            if (match) {
-                results.push({
-                    name: match[1],
-                    path: match[2] ?? match[1],
-                    type: match[3],
-                });
-            }
-        }
-        return results;
-    }
-    parseUnusedDepsText(text) {
-        const results = [];
-        for (const line of text.split('\n')) {
-            if (!line.trim())
-                continue;
-            // Try JSON first
-            try {
-                const parsed = JSON.parse(line);
-                if (Array.isArray(parsed))
-                    return parsed;
-            }
-            catch { /* not JSON, parse as text */ }
-            // Format: ⚠ name (path) — reason  OR  name (path)  OR  name — reason
-            const match = line.match(/^[⚠!\s]*(\S+)(?:\s+\((.+?)\))?(?:\s*[—\-]+\s*(.+))?$/);
-            if (match) {
-                results.push({
-                    name: match[1],
-                    path: match[2] ?? match[1],
-                    reason: match[3]?.trim(),
-                });
-            }
-        }
-        return results;
-    }
-    parseModuleApiText(text) {
-        const results = [];
-        for (const line of text.split('\n')) {
-            if (!line.trim())
-                continue;
-            // Try JSON first
-            try {
-                const parsed = JSON.parse(line);
-                if (Array.isArray(parsed))
-                    return parsed;
-            }
-            catch { /* not JSON, parse as text */ }
-            // Format: kind name (file:line)  OR  kind name signature (file:line)
-            const match = line.match(/^(\w+)\s+(\S+)(?:\s+(.*?))?\s+\((.+?):(\d+)\)$/);
-            if (match) {
-                results.push({
-                    kind: match[1],
-                    name: match[2],
-                    signature: match[3]?.trim() || undefined,
-                    file: match[4],
-                    line: parseInt(match[5], 10),
-                });
-            }
-        }
-        return results;
-    }
     // --- Utility methods ---
     isAvailable() {
         return this.binaryPath !== null;
     }
-    /** Returns true if the index was built but found >50k files (node_modules leak) */
     isOversized() {
         return this.indexOversized;
     }
-    /** Returns true if index building is disabled (dangerous root like /) */
     isDisabled() {
         return this.indexDisabled;
     }
-    /** Disable index building (e.g. project root is / or home dir) */
     disableIndex() {
         this.indexDisabled = true;
     }
-    /** Re-enable index building after auto-detecting a valid project root */
     enableIndex() {
         this.indexDisabled = false;
     }
-    /** Update project root (e.g. after auto-detecting from file path) */
     updateProjectRoot(newRoot) {
         this.projectRoot = newRoot;
-        this.indexed = false; // Force re-check
+        this.indexed = false;
     }
     async exec(args, timeoutMs) {
         if (!this.binaryPath) {
             throw new Error('ast-index not initialized. Call init() first.');
         }
-        // If ast-grep was found in node_modules/.bin, inject it into PATH
-        // so ast-index can find sg when running agrep
         const env = this.astGrepBinDir
             ? { ...process.env, PATH: `${this.astGrepBinDir}:${process.env.PATH ?? ''}` }
             : undefined;
@@ -1041,291 +634,5 @@ export class AstIndexClient {
         }
         return stdout;
     }
-    async buildFileStructure(filePath, entries) {
-        const content = await readFile(filePath, 'utf-8');
-        const lines = content.split('\n');
-        const fileStat = await stat(filePath);
-        // Fix last entry end_line to use actual file line count
-        this.fixLastEndLine(entries, lines.length);
-        // Enrich classes that ast-index returned without children (language-specific)
-        const lang = this.detectLanguage(filePath);
-        if (lang === 'Python') {
-            this.enrichPythonClassMethods(entries, lines);
-        }
-        else if (lang === 'PHP') {
-            this.enrichPHPClassMethods(entries, lines);
-        }
-        // Enrich entries with signatures from file content
-        this.enrichSignatures(entries, lines);
-        return {
-            path: filePath,
-            language: lang,
-            meta: {
-                lines: lines.length,
-                bytes: fileStat.size,
-                lastModified: fileStat.mtimeMs,
-                contentHash: createHash('sha256').update(content).digest('hex'),
-            },
-            imports: [],
-            exports: [],
-            symbols: entries.map(e => this.mapOutlineEntry(e)),
-        };
-    }
-    /**
-     * Python: ast-index doesn't return methods inside classes.
-     * Parse file content to extract `def` methods for classes without children.
-     */
-    enrichPythonClassMethods(entries, lines) {
-        for (const entry of entries) {
-            if (entry.kind.toLowerCase() !== 'class')
-                continue;
-            if (entry.children && entry.children.length > 0)
-                continue;
-            const classStartIdx = entry.start_line - 1; // 0-based
-            const classEndIdx = entry.end_line - 1;
-            // Detect class body indent: look for first `def ` inside class range
-            let bodyIndent = -1;
-            for (let i = classStartIdx + 1; i <= classEndIdx && i < lines.length; i++) {
-                const defMatch = lines[i].match(/^(\s+)def\s/);
-                if (defMatch) {
-                    bodyIndent = defMatch[1].length;
-                    break;
-                }
-            }
-            if (bodyIndent < 0)
-                continue; // no methods found
-            const methods = [];
-            for (let i = classStartIdx + 1; i <= classEndIdx && i < lines.length; i++) {
-                const line = lines[i];
-                // Match `def method_name(` at the detected indent level
-                const match = line.match(new RegExp(`^\\s{${bodyIndent}}def\\s+(\\w+)\\s*\\(`));
-                if (!match)
-                    continue;
-                const methodName = match[1];
-                const methodLine = i + 1; // 1-based
-                // Check for async/static/decorators
-                const isAsync = line.includes('async def');
-                const isStatic = i > 0 && /^\s*@staticmethod/.test(lines[i - 1]);
-                const isClassMethod = i > 0 && /^\s*@classmethod/.test(lines[i - 1]);
-                // Collect decorators above
-                const decorators = [];
-                for (let d = i - 1; d >= classStartIdx; d--) {
-                    const decMatch = lines[d].match(new RegExp(`^\\s{${bodyIndent}}@(\\w+)`));
-                    if (decMatch) {
-                        decorators.unshift(`@${decMatch[1]}`);
-                    }
-                    else {
-                        break;
-                    }
-                }
-                // Determine visibility from name
-                const visibility = methodName.startsWith('__') && !methodName.endsWith('__')
-                    ? 'private'
-                    : methodName.startsWith('_')
-                        ? 'protected'
-                        : 'public';
-                methods.push({
-                    name: methodName,
-                    kind: isStatic || isClassMethod ? 'function' : 'method',
-                    start_line: methodLine,
-                    end_line: 0, // computed below
-                    signature: line.trim(),
-                    visibility,
-                    is_async: isAsync,
-                    is_static: isStatic,
-                    decorators: decorators.length > 0 ? decorators : undefined,
-                });
-            }
-            // Compute end_lines for methods
-            for (let m = 0; m < methods.length; m++) {
-                if (m < methods.length - 1) {
-                    // End before next method (or its first decorator)
-                    const nextStart = methods[m + 1].start_line;
-                    // Walk back from next method to skip decorators/blank lines
-                    let endLine = nextStart - 1;
-                    for (let k = nextStart - 2; k >= methods[m].start_line; k--) {
-                        const l = lines[k];
-                        if (l.trim() === '' || new RegExp(`^\\s{${bodyIndent}}@`).test(l)) {
-                            endLine = k; // 0-based → will be used as 1-based below
-                        }
-                        else {
-                            break;
-                        }
-                    }
-                    methods[m].end_line = endLine;
-                }
-                else {
-                    // Last method ends at class end
-                    methods[m].end_line = entry.end_line;
-                }
-            }
-            entry.children = methods;
-        }
-    }
-    /**
-     * PHP: ast-index doesn't return methods inside classes.
-     * Parse file content to extract `function` methods for classes without children.
-     */
-    enrichPHPClassMethods(entries, lines) {
-        for (const entry of entries) {
-            if (entry.kind.toLowerCase() !== 'class')
-                continue;
-            if (entry.children && entry.children.length > 0)
-                continue;
-            const classStartIdx = entry.start_line - 1;
-            const classEndIdx = entry.end_line - 1;
-            // Detect class body indent: look for first `function ` inside class range
-            let bodyIndent = -1;
-            for (let i = classStartIdx + 1; i <= classEndIdx && i < lines.length; i++) {
-                const fnMatch = lines[i].match(/^(\s+)(?:public|private|protected|static|\s)*function\s/);
-                if (fnMatch) {
-                    bodyIndent = fnMatch[1].length;
-                    break;
-                }
-            }
-            if (bodyIndent < 0)
-                continue;
-            const methods = [];
-            for (let i = classStartIdx + 1; i <= classEndIdx && i < lines.length; i++) {
-                const line = lines[i];
-                // Match PHP method: [visibility] [static] function name(
-                const match = line.match(new RegExp(`^\\s{${bodyIndent}}(?:(public|private|protected)\\s+)?(?:(static)\\s+)?function\\s+(\\w+)\\s*\\(`));
-                if (!match)
-                    continue;
-                const visibility = match[1] ?? 'public';
-                const isStatic = !!match[2];
-                const methodName = match[3];
-                const methodLine = i + 1;
-                methods.push({
-                    name: methodName,
-                    kind: isStatic ? 'function' : 'method',
-                    start_line: methodLine,
-                    end_line: 0,
-                    signature: line.trim(),
-                    visibility,
-                    is_static: isStatic,
-                });
-            }
-            // Compute end_lines
-            for (let m = 0; m < methods.length; m++) {
-                if (m < methods.length - 1) {
-                    methods[m].end_line = methods[m + 1].start_line - 1;
-                }
-                else {
-                    methods[m].end_line = entry.end_line;
-                }
-            }
-            entry.children = methods;
-        }
-    }
-    /** Fix the last entry's end_line to use actual file line count */
-    fixLastEndLine(entries, totalLines) {
-        if (entries.length === 0)
-            return;
-        const last = entries[entries.length - 1];
-        last.end_line = totalLines;
-        // Recursively fix children
-        if (last.children?.length) {
-            this.fixLastEndLine(last.children, last.end_line - 1);
-        }
-    }
-    /** Read actual signature lines from file content */
-    enrichSignatures(entries, lines) {
-        for (const entry of entries) {
-            if (!entry.signature) {
-                const lineIdx = entry.start_line - 1;
-                if (lineIdx >= 0 && lineIdx < lines.length) {
-                    entry.signature = lines[lineIdx].trim();
-                }
-            }
-            if (entry.children?.length) {
-                this.enrichSignatures(entry.children, lines);
-            }
-        }
-    }
-    mapOutlineEntry(entry) {
-        return {
-            name: entry.name,
-            qualifiedName: entry.name, // Will be enriched with parent context
-            kind: this.mapKind(entry.kind),
-            signature: entry.signature ?? entry.name,
-            location: {
-                startLine: entry.start_line,
-                endLine: entry.end_line,
-                lineCount: entry.end_line - entry.start_line + 1,
-            },
-            visibility: this.mapVisibility(entry.visibility),
-            async: entry.is_async ?? false,
-            static: entry.is_static ?? false,
-            decorators: entry.decorators ?? [],
-            children: (entry.children ?? []).map(c => this.mapOutlineEntry(c)),
-            doc: entry.doc ?? null,
-            references: [],
-        };
-    }
-    mapKind(kind) {
-        const map = {
-            function: 'function',
-            class: 'class',
-            method: 'method',
-            property: 'property',
-            variable: 'variable',
-            type: 'type',
-            interface: 'interface',
-            enum: 'enum',
-            constant: 'constant',
-            namespace: 'namespace',
-            struct: 'class',
-            trait: 'interface',
-            impl: 'class',
-            module: 'namespace',
-        };
-        return map[kind.toLowerCase()] ?? 'function';
-    }
-    mapVisibility(vis) {
-        if (!vis)
-            return 'default';
-        const map = {
-            public: 'public',
-            private: 'private',
-            protected: 'protected',
-            pub: 'public',
-            export: 'public',
-        };
-        return map[vis.toLowerCase()] ?? 'default';
-    }
-    detectLanguage(filePath) {
-        const ext = filePath.split('.').pop()?.toLowerCase() ?? '';
-        const map = {
-            ts: 'TypeScript', tsx: 'TypeScript',
-            js: 'JavaScript', jsx: 'JavaScript', mjs: 'JavaScript',
-            py: 'Python',
-            go: 'Go',
-            rs: 'Rust',
-            java: 'Java',
-            kt: 'Kotlin', kts: 'Kotlin',
-            swift: 'Swift',
-            cs: 'C#',
-            cpp: 'C++', cc: 'C++', cxx: 'C++', hpp: 'C++',
-            c: 'C', h: 'C',
-            php: 'PHP',
-            rb: 'Ruby',
-            scala: 'Scala',
-            dart: 'Dart',
-            lua: 'Lua',
-            sh: 'Bash', bash: 'Bash',
-            sql: 'SQL',
-            r: 'R',
-            vue: 'Vue',
-            svelte: 'Svelte',
-            pl: 'Perl', pm: 'Perl',
-            ex: 'Elixir', exs: 'Elixir',
-            groovy: 'Groovy',
-            m: 'Objective-C',
-            proto: 'Protocol Buffers',
-            bsl: 'BSL',
-        };
-        return map[ext] ?? 'Unknown';
-    }
 }
 //# sourceMappingURL=client.js.map