pumuki-ast-hooks 5.3.28 → 5.3.30

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pumuki-ast-hooks",
-  "version": "5.3.28",
+  "version": "5.3.30",
   "description": "Enterprise-grade AST Intelligence System with multi-platform support (iOS, Android, Backend, Frontend) and Feature-First + DDD + Clean Architecture enforcement. Includes dynamic violations API for intelligent querying.",
   "main": "index.js",
   "bin": {
@@ -123,4 +123,4 @@
     "./skills": "./skills/skill-rules.json",
     "./hooks": "./hooks/index.js"
   }
-}
+}

package/scripts/hooks-system/application/services/installation/McpConfigurator.js

@@ -48,6 +48,45 @@ class McpConfigurator {
         this.logger = logger;
     }
 
+    resolveAutomationEntrypoint() {
+        // Prefer dependency-installed hook system (node_modules) to avoid requiring scripts/hooks-system in consumer repos
+        // Fallback to repo-local hook system if it exists.
+        const candidates = [
+            this.hookSystemRoot
+                ? path.join(this.hookSystemRoot, 'infrastructure', 'mcp', 'ast-intelligence-automation.js')
+                : null,
+            path.join(this.targetRoot, 'scripts', 'hooks-system', 'infrastructure', 'mcp', 'ast-intelligence-automation.js')
+        ].filter(Boolean);
+
+        for (const c of candidates) {
+            try {
+                if (fs.existsSync(c)) return c;
+            } catch {
+                // ignore
+            }
+        }
+
+        // Last resort: keep previous behaviour
+        return path.join(this.targetRoot, 'scripts/hooks-system/infrastructure/mcp/ast-intelligence-automation.js');
+    }
+
+    disableDuplicateServersForRepo(existing, currentServerId) {
+        if (!existing || !existing.mcpServers) return;
+
+        Object.entries(existing.mcpServers).forEach(([id, server]) => {
+            if (!server || id === currentServerId) return;
+            const envObj = server.env || {};
+            const repoRoot = envObj.REPO_ROOT;
+            if (repoRoot !== this.targetRoot) return;
+
+            const args0 = Array.isArray(server.args) ? String(server.args[0] || '') : '';
+            // Disable legacy server pointing to repo-local scripts/hooks-system when we already configure the canonical one
+            if (args0.includes('/scripts/hooks-system/') || args0.includes('\\scripts\\hooks-system\\')) {
+                server.disabled = true;
+            }
+        });
+    }
+
     getGlobalWindsurfConfigPath() {
         return path.join(os.homedir(), '.codeium', 'windsurf', 'mcp_config.json');
     }
@@ -65,12 +104,13 @@ class McpConfigurator {
         }
 
         const serverId = computeServerIdForRepo(this.targetRoot);
+        const entrypoint = this.resolveAutomationEntrypoint();
         const mcpConfig = {
             mcpServers: {
                 [serverId]: {
                     command: nodePath,
                     args: [
-                        path.join(this.targetRoot, 'scripts/hooks-system/infrastructure/mcp/ast-intelligence-automation.js')
+                        entrypoint
                     ],
                     env: {
                         REPO_ROOT: this.targetRoot,
@@ -97,6 +137,9 @@ class McpConfigurator {
                 const existing = JSON.parse(fs.readFileSync(globalConfigPath, 'utf8'));
                 if (!existing.mcpServers) existing.mcpServers = {};
 
+                // Prevent duplicate MCP servers for the same repoRoot by disabling legacy entries.
+                this.disableDuplicateServersForRepo(existing, serverId);
+
                 existing.mcpServers[serverId] = mcpConfig.mcpServers[serverId];
 
                 fs.writeFileSync(globalConfigPath, JSON.stringify(existing, null, 2));

package/scripts/hooks-system/infrastructure/ast/ast-intelligence.js

@@ -368,18 +368,36 @@ function generateOutput(findings, context, project, root) {
 
   // Top violations
   const grouped = {};
+  const levelRank = { LOW: 1, MEDIUM: 2, HIGH: 3, CRITICAL: 4 };
+  const emojiForLevel = (level) => (level === 'CRITICAL' || level === 'HIGH' ? '🔴' : '🔵');
+
   findings.forEach(f => {
-    grouped[f.ruleId] = (grouped[f.ruleId] || 0) + 1;
+    if (!f || !f.ruleId) return;
+    const level = mapToLevel(f.severity);
+    if (!grouped[f.ruleId]) {
+      grouped[f.ruleId] = { count: 0, worstLevel: level };
+    }
+    grouped[f.ruleId].count += 1;
+    if ((levelRank[level] || 0) > (levelRank[grouped[f.ruleId].worstLevel] || 0)) {
+      grouped[f.ruleId].worstLevel = level;
+    }
+  });
+
+  const entries = Object.entries(grouped)
+    .map(([ruleId, data]) => ({ ruleId, count: data.count, worstLevel: data.worstLevel }))
+    .sort((a, b) => b.count - a.count);
+
+  const blockers = entries.filter(e => e.worstLevel === 'CRITICAL' || e.worstLevel === 'HIGH');
+  const nonBlockers = entries.filter(e => e.worstLevel !== 'CRITICAL' && e.worstLevel !== 'HIGH');
+
+  blockers.forEach(({ ruleId, count, worstLevel }) => {
+    console.error(`${emojiForLevel(worstLevel)} ${ruleId} - ${count} violations`);
   });
 
-  Object.entries(grouped)
-    .sort((a, b) => b[1] - a[1])
-    .slice(0, 20)
-    .forEach(([ruleId, count]) => {
-      const severity = ruleId.includes("types.any") || ruleId.includes("security.") || ruleId.includes("architecture.") ? "error" :
-        ruleId.includes("performance.") || ruleId.includes("debug.") ? "warning" : "info";
-      const emoji = severity === "error" ? "🔴" : severity === "warning" ? "🟡" : "🔵";
-      console.error(`${emoji} ${ruleId} - ${count} violations`);
+  nonBlockers
+    .slice(0, Math.max(0, 20 - blockers.length))
+    .forEach(({ ruleId, count, worstLevel }) => {
+      console.error(`${emojiForLevel(worstLevel)} ${ruleId} - ${count} violations`);
     });
 
   // Summary