@claude-flow/cli 3.7.0-alpha.81 → 3.7.0-alpha.82

package/dist/src/mcp-tools/hooks-tools.js

@@ -1394,44 +1394,14 @@ export const hooksPretrain = {
         // (readdirSync/statSync already imported statically at the top.)
         const extCounts = {};
         let filesAnalyzed = 0;
-        // #1953: separate budget for code files. The old code gated the
-        // import-pattern extraction on `filesAnalyzed <= 50`, which counts
-        // EVERY directory entry (including .md/.yaml/.db/.log). In any
-        // markdown/docs-heavy repo, the depth-first walker burned through the
-        // 50-file budget on non-code files before reaching any source — so
-        // `patternsExtracted: 0` even when hundreds of `.ts`/`.js` files existed.
-        let codeFilesScanned = 0;
         let totalLines = 0;
         const maxDepth = depth === 'shallow' ? 2 : depth === 'deep' ? 6 : 4;
         const patterns = [];
-        // #1953: recurse into directories that typically contain code first
-        // (`src/`, `apps/`, `packages/`, `lib/`, `crates/`, `workers/`, `server/`)
-        // before docs / specs / planning dirs, so the import-extraction budget
-        // is spent on the highest-signal directories even in mixed repos.
-        const CODE_DIR_PREFIXES = new Set([
-            'src', 'apps', 'packages', 'lib', 'crates', 'workers',
-            'server', 'backend', 'frontend', 'app', 'cli', 'core',
-        ]);
-        const scoreEntry = (name) => {
-            if (CODE_DIR_PREFIXES.has(name))
-                return 0;
-            // Deprioritise common docs / output directories.
-            if (/^(docs?|specs?|_.*|examples?|samples?|out|build|target|coverage|tests?)$/.test(name))
-                return 2;
-            return 1;
-        };
         const scan = (dir, currentDepth) => {
             if (currentDepth > maxDepth)
                 return;
             try {
                 const entries = readdirSync(dir, { withFileTypes: true });
-                // Sort: code-likely dirs first, files mixed in by name, deprioritised
-                // dirs last. Stable for deterministic test behaviour.
-                entries.sort((a, b) => {
-                    const sa = a.isDirectory() ? scoreEntry(a.name) : 1;
-                    const sb = b.isDirectory() ? scoreEntry(b.name) : 1;
-                    return sa - sb;
-                });
                 for (const entry of entries) {
                     if (entry.name.startsWith('.') || entry.name === 'node_modules' || entry.name === 'dist')
                         continue;
@@ -1445,18 +1415,15 @@ export const hooksPretrain = {
                             extCounts[ext] = (extCounts[ext] || 0) + 1;
                         filesAnalyzed++;
                         // For code files, count lines and extract imports
-                        if (['.ts', '.js', '.tsx', '.jsx', '.mjs', '.cjs', '.py', '.go', '.rs', '.java'].includes(ext)) {
+                        if (['.ts', '.js', '.py', '.go', '.rs', '.java'].includes(ext)) {
                             try {
                                 const content = readFileSync(full, 'utf-8');
                                 const lines = content.split('\n');
                                 totalLines += lines.length;
-                                // #1953: gate on the code-file count, not every-file count.
-                                // Also widened the per-file scan window from 30 → 80 lines:
-                                // modern TS files often have license headers + JSDoc + type
-                                // imports before the first `import` statement.
-                                if (++codeFilesScanned <= 50) {
-                                    for (const line of lines.slice(0, 80)) {
-                                        if (line.startsWith('import ') || line.startsWith('from ') || (line.startsWith('const ') && line.includes('require('))) {
+                                // Extract import patterns (first 50 files max for performance)
+                                if (filesAnalyzed <= 50) {
+                                    for (const line of lines.slice(0, 30)) {
+                                        if (line.startsWith('import ') || line.startsWith('from ') || line.startsWith('const ') && line.includes('require(')) {
                                             const trimmed = line.trim();
                                             if (trimmed.length < 120 && !patterns.includes(trimmed))
                                                 patterns.push(trimmed);
@@ -1491,7 +1458,7 @@ export const hooksPretrain = {
         // #1847: when the corpus contains files but no patterns were extracted
         // (typical for Markdown vaults), make the source-code-only extraction
         // contract explicit so users don't conclude the hook system is broken.
-        const SUPPORTED_EXTRACTION_EXTS = ['.ts', '.js', '.tsx', '.jsx', '.mjs', '.cjs', '.py', '.go', '.rs', '.java'];
+        const SUPPORTED_EXTRACTION_EXTS = ['.ts', '.js', '.py', '.go', '.rs', '.java'];
         let note;
         if (filesAnalyzed > 0 && patterns.length === 0) {
             const codeFileCount = SUPPORTED_EXTRACTION_EXTS.reduce((sum, ext) => sum + (extCounts[ext] ?? 0), 0);
@@ -3832,69 +3799,8 @@ export const hooksWorkerCancel = {
         };
     },
 };
-// #1916: the `ruflo hooks teammate-idle` / `ruflo hooks task-completed` CLI
-// subcommands (Agent Teams hooks) referenced unregistered tools. Minimal
-// acknowledgement handlers with the shapes the CLI expects — auto-assignment
-// and pattern-learning are delegated to the task-queue consumer / intelligence
-// pipeline (a tracked #1916 follow-up).
-export const hooksTeammateIdle = {
-    name: 'hooks_teammate-idle',
-    description: 'Agent Teams hook — fired when a teammate agent finishes its turn; reports whether a pending task can be auto-assigned. Use when native Task is wrong because you have a persistent multi-agent team with a shared task list and want idle workers picked up automatically rather than re-spawning subagents. For a one-shot Task, native Task is fine. (Auto-assignment is delegated to the task-queue consumer — this acknowledges the event today.)',
-    category: 'hooks',
-    inputSchema: {
-        type: 'object',
-        properties: {
-            teammateId: { type: 'string', description: 'ID of the idle teammate' },
-            teamName: { type: 'string', description: 'Team name' },
-            autoAssign: { type: 'boolean', description: 'Auto-assign a pending task if available' },
-            checkTaskList: { type: 'boolean', description: 'Consult the shared task list' },
-            timestamp: { type: 'number', description: 'Event timestamp (ms)' },
-        },
-    },
-    handler: async (input) => {
-        const teammateId = String(input.teammateId ?? '');
-        return {
-            success: true,
-            teammateId,
-            action: 'waiting',
-            pendingTasks: 0,
-            message: 'teammate-idle acknowledged; auto-assignment requires the task-queue consumer (#1916 follow-up)',
-        };
-    },
-};
-export const hooksTaskCompleted = {
-    name: 'hooks_task-completed',
-    description: 'Agent Teams hook — fired when a task is marked complete; records completion and (eventually) trains patterns + notifies the team lead. Use when native TodoWrite is wrong because the work was a persisted, agent-assigned task whose outcome should feed cross-session learning and team coordination. For an in-session checklist tick, native TodoWrite is fine. (Pattern-learning is delegated to the intelligence pipeline — this records the completion today.)',
-    category: 'hooks',
-    inputSchema: {
-        type: 'object',
-        properties: {
-            taskId: { type: 'string', description: 'ID of the completed task' },
-            teammateId: { type: 'string', description: 'Teammate that completed it' },
-            success: { type: 'boolean', description: 'Whether the task succeeded' },
-            quality: { type: 'number', description: 'Quality score 0-1' },
-            trainPatterns: { type: 'boolean', description: 'Feed the outcome to the learning pipeline' },
-            notifyLead: { type: 'boolean', description: 'Notify the team lead' },
-        },
-        required: ['taskId'],
-    },
-    handler: async (input) => {
-        const taskId = String(input.taskId ?? '');
-        const quality = typeof input.quality === 'number' ? input.quality : (input.success === false ? 0 : 1);
-        return {
-            success: true,
-            taskId,
-            patternsLearned: 0,
-            leadNotified: input.notifyLead === true,
-            metrics: { duration: 0, quality, learningUpdates: 0 },
-            note: 'completion recorded; pattern-learning is delegated to the intelligence pipeline (#1916 follow-up)',
-        };
-    },
-};
 // Export all hooks tools
 export const hooksTools = [
-    hooksTeammateIdle,
-    hooksTaskCompleted,
     hooksPreEdit,
     hooksPostEdit,
     hooksPreCommand,