claude-flow-novice 2.14.6 → 2.14.8

package/claude-assets/skills/hook-pipeline/python-subprocess-safety.py

@@ -0,0 +1,77 @@
+#!/usr/bin/env python3
+import ast
+import sys
+
+class SubprocessSafetyVisitor(ast.NodeVisitor):
+    def __init__(self):
+        self.unsafe_calls = []
+        self.allowed_stderr_values = {'Attribute': ['PIPE', 'DEVNULL', 'STDOUT'],
+                                      'Name': ['subprocess.PIPE', 'subprocess.DEVNULL', 'subprocess.STDOUT']}
+
+    def is_safe_stderr(self, node):
+        if isinstance(node, ast.Attribute):
+            return node.attr in self.allowed_stderr_values['Attribute']
+        if isinstance(node, ast.Name):
+            return node.id in self.allowed_stderr_values['Name']
+        return False
+
+    def visit_Call(self, node):
+        if isinstance(node.func, ast.Attribute):
+            func_name = node.func.attr
+            if func_name in ['run', 'Popen', 'check_output']:
+                # Check if from subprocess module
+                module_name = None
+                if isinstance(node.func.value, ast.Name):
+                    module_name = node.func.value.id
+
+                if module_name == 'subprocess':
+                    # Strategies for safe subprocess calls
+                    stderr_found = False
+                    capture_output_found = False
+
+                    for kw in node.keywords:
+                        if kw.arg == 'stderr' and self.is_safe_stderr(kw.value):
+                            stderr_found = True
+                        if kw.arg == 'capture_output' and isinstance(kw.value, ast.NameConstant) and kw.value.value is True:
+                            capture_output_found = True
+
+                    if not (stderr_found or capture_output_found):
+                        self.unsafe_calls.append(
+                            f"Line {node.lineno}: Subprocess call '{func_name}' without stderr parameter or capture_output"
+                        )
+
+        self.generic_visit(node)
+
+def validate_subprocess_safety(file_path):
+    try:
+        with open(file_path, 'r') as f:
+            tree = ast.parse(f.read(), filename=file_path)
+    except SyntaxError as e:
+        print(f"Syntax error in {file_path}: {e}")
+        return 1
+    except FileNotFoundError:
+        print(f"File not found: {file_path}")
+        return 1
+
+    visitor = SubprocessSafetyVisitor()
+    visitor.visit(tree)
+
+    if visitor.unsafe_calls:
+        print("Subprocess safety warnings:")
+        for call in visitor.unsafe_calls:
+            print(call)
+        return 2
+
+    return 0
+
+def main():
+    if len(sys.argv) < 2:
+        print("Usage: python3 python-subprocess-safety.py <python_file>")
+        return 1
+
+    file_path = sys.argv[1]
+    result = validate_subprocess_safety(file_path)
+    sys.exit(result)
+
+if __name__ == '__main__':
+    main()

package/claude-assets/skills/hook-pipeline/rust-command-safety.sh

@@ -0,0 +1,38 @@
+#!/bin/bash
+# Rust Command Safety Validator
+set -o pipefail
+
+# Exit codes:
+# 0 - No issues found
+# 1 - File not found or cannot be read
+# 2 - Command safety warnings found
+
+if [ $# -eq 0 ]; then
+    echo "Usage: $0 <rust_file>"
+    exit 1
+fi
+
+file_path="$1"
+
+if [ ! -f "$file_path" ]; then
+    echo "File not found: $file_path"
+    exit 1
+fi
+
+# Use grep to find Command::new() calls and check for stderr in next 5 lines
+unsafe_commands=$(grep -n "Command::new(" "$file_path" | while read -r line; do
+    line_num=$(echo "$line" | cut -d':' -f1)
+    next_lines=$(tail -n +"$line_num" "$file_path" | head -n 5)
+
+    if ! echo "$next_lines" | grep -q "\.stderr("; then
+        echo "Line $line_num: Command::new() without stderr() configuration in next 5 lines"
+    fi
+done)
+
+if [ -n "$unsafe_commands" ]; then
+    echo "Command safety warnings:"
+    echo "$unsafe_commands"
+    exit 2
+fi
+
+exit 0

package/claude-assets/skills/hook-pipeline/rust-dependency-checker.sh

@@ -0,0 +1,50 @@
+#!/bin/bash
+# Rust Dependency Checker
+set -o pipefail
+
+# Exit codes:
+# 0 - No issues found
+# 1 - Cargo.toml not found or cannot be read
+# 2 - Dependency warnings found
+
+if [ $# -eq 0 ]; then
+    echo "Usage: $0 <rust_file>"
+    exit 1
+fi
+
+file_path="$1"
+project_dir=$(dirname "$file_path")
+cargo_toml="${project_dir}/Cargo.toml"
+
+if [ ! -f "$cargo_toml" ]; then
+    echo "Cargo.toml not found in project directory"
+    exit 1
+fi
+
+# Skip list for built-in/standard libraries
+skip_modules="std|core|alloc|proc_macro"
+
+# Find local crate dependencies in the file
+local_deps=$(grep -o "use crate::[a-zA-Z_][a-zA-Z0-9_:]*" "$file_path" |
+    sed 's/use crate:://g' |
+    cut -d':' -f1 |
+    sort -u |
+    grep -vE "^(${skip_modules})$")
+
+# Check dependencies in Cargo.toml
+if [ -n "$local_deps" ]; then
+    warnings=$(echo "$local_deps" | while read -r dep; do
+        if ! grep -q "path = \".*${dep}\"" "$cargo_toml"; then
+            echo "Undeclared local crate dependency: ${dep}"
+        fi
+    done)
+
+    if [ -n "$warnings" ]; then
+        echo "Dependency warnings:"
+        echo "$warnings"
+        echo "Recommendation: Declare local crate dependencies in Cargo.toml"
+        exit 2
+    fi
+fi
+
+exit 0

package/claude-assets/skills/hook-pipeline/rust-future-safety.sh

@@ -0,0 +1,50 @@
+#!/bin/bash
+# Rust Future Safety Validator
+set -o pipefail
+
+# Exit codes:
+# 0 - No issues found
+# 1 - File not found or cannot be read
+# 2 - Future safety warnings found
+
+if [ $# -eq 0 ]; then
+    echo "Usage: $0 <rust_file>"
+    exit 1
+fi
+
+file_path="$1"
+
+if [ ! -f "$file_path" ]; then
+    echo "File not found: $file_path"
+    exit 1
+fi
+
+# Find async function definitions
+async_functions=$(grep -n "async fn" "$file_path")
+
+if [ -n "$async_functions" ]; then
+    warnings=$(echo "$async_functions" | while read -r async_line; do
+        line_num=$(echo "$async_line" | cut -d':' -f1)
+        func_name=$(echo "$async_line" | sed -n 's/.*async fn \([a-zA-Z_][a-zA-Z0-9_]*\).*/\1/p')
+
+        # Search for function calls without .await
+        calls_without_await=$(grep -n "\\b${func_name}(" "$file_path" |
+            grep -v ".await" |
+            grep -v "async fn" |
+            grep -v "fn ${func_name}")
+
+        if [ -n "$calls_without_await" ]; then
+            echo "Lines related to async function '${func_name}':"
+            echo "$calls_without_await"
+            echo "Recommendation: Use .await for async function calls. Run cargo clippy for more details."
+        fi
+    done)
+
+    if [ -n "$warnings" ]; then
+        echo "Future safety warnings:"
+        echo "$warnings"
+        exit 2
+    fi
+fi
+
+exit 0

package/dist/agents/agent-loader.js

@@ -1,145 +1,12 @@
-"use strict";
 /**
  * Dynamic Agent Loader - Reads agent definitions from .claude/agents/ directory
  * Single source of truth for agent types in the system
- */ var __awaiter = this && this.__awaiter || function(thisArg, _arguments, P, generator) {
-    function adopt(value) {
-        return value instanceof P ? value : new P(function(resolve) {
-            resolve(value);
-        });
-    }
-    return new (P || (P = Promise))(function(resolve, reject) {
-        function fulfilled(value) {
-            try {
-                step(generator.next(value));
-            } catch (e) {
-                reject(e);
-            }
-        }
-        function rejected(value) {
-            try {
-                step(generator["throw"](value));
-            } catch (e) {
-                reject(e);
-            }
-        }
-        function step(result) {
-            result.done ? resolve(result.value) : adopt(result.value).then(fulfilled, rejected);
-        }
-        step((generator = generator.apply(thisArg, _arguments || [])).next());
-    });
-};
-var __generator = this && this.__generator || function(thisArg, body) {
-    var _ = {
-        label: 0,
-        sent: function() {
-            if (t[0] & 1) throw t[1];
-            return t[1];
-        },
-        trys: [],
-        ops: []
-    }, f, y, t, g = Object.create((typeof Iterator === "function" ? Iterator : Object).prototype);
-    return g.next = verb(0), g["throw"] = verb(1), g["return"] = verb(2), typeof Symbol === "function" && (g[Symbol.iterator] = function() {
-        return this;
-    }), g;
-    function verb(n) {
-        return function(v) {
-            return step([
-                n,
-                v
-            ]);
-        };
-    }
-    function step(op) {
-        if (f) throw new TypeError("Generator is already executing.");
-        while(g && (g = 0, op[0] && (_ = 0)), _)try {
-            if (f = 1, y && (t = op[0] & 2 ? y["return"] : op[0] ? y["throw"] || ((t = y["return"]) && t.call(y), 0) : y.next) && !(t = t.call(y, op[1])).done) return t;
-            if (y = 0, t) op = [
-                op[0] & 2,
-                t.value
-            ];
-            switch(op[0]){
-                case 0:
-                case 1:
-                    t = op;
-                    break;
-                case 4:
-                    _.label++;
-                    return {
-                        value: op[1],
-                        done: false
-                    };
-                case 5:
-                    _.label++;
-                    y = op[1];
-                    op = [
-                        0
-                    ];
-                    continue;
-                case 7:
-                    op = _.ops.pop();
-                    _.trys.pop();
-                    continue;
-                default:
-                    if (!(t = _.trys, t = t.length > 0 && t[t.length - 1]) && (op[0] === 6 || op[0] === 2)) {
-                        _ = 0;
-                        continue;
-                    }
-                    if (op[0] === 3 && (!t || op[1] > t[0] && op[1] < t[3])) {
-                        _.label = op[1];
-                        break;
-                    }
-                    if (op[0] === 6 && _.label < t[1]) {
-                        _.label = t[1];
-                        t = op;
-                        break;
-                    }
-                    if (t && _.label < t[2]) {
-                        _.label = t[2];
-                        _.ops.push(op);
-                        break;
-                    }
-                    if (t[2]) _.ops.pop();
-                    _.trys.pop();
-                    continue;
-            }
-            op = body.call(thisArg, _);
-        } catch (e) {
-            op = [
-                6,
-                e
-            ];
-            y = 0;
-        } finally{
-            f = t = 0;
-        }
-        if (op[0] & 5) throw op[1];
-        return {
-            value: op[0] ? op[1] : void 0,
-            done: true
-        };
-    }
-};
-var __spreadArray = this && this.__spreadArray || function(to, from, pack) {
-    if (pack || arguments.length === 2) for(var i = 0, l = from.length, ar; i < l; i++){
-        if (ar || !(i in from)) {
-            if (!ar) ar = Array.prototype.slice.call(from, 0, i);
-            ar[i] = from[i];
-        }
-    }
-    return to.concat(ar || Array.prototype.slice.call(from));
-};
-Object.defineProperty(exports, "__esModule", {
-    value: true
-});
-exports.refreshAgents = exports.getAgentsByCategory = exports.isValidAgentType = exports.searchAgents = exports.getAgentCategories = exports.getAllAgents = exports.getAgent = exports.getAvailableAgentTypes = exports.agentLoader = exports.AgentLoader = void 0;
-exports.resolveLegacyAgentType = resolveLegacyAgentType;
-var node_fs_1 = require("node:fs");
-var glob_1 = require("glob");
-var node_path_1 = require("node:path");
-var yaml_1 = require("yaml");
+ */ import { readFileSync, existsSync } from 'node:fs';
+import { glob } from 'glob';
+import { resolve, dirname } from 'node:path';
+import { parse as parseYaml } from 'yaml';
 // Legacy agent type mapping for backward compatibility
-var LEGACY_AGENT_MAPPING = {
+const LEGACY_AGENT_MAPPING = {
     analyst: 'code-analyzer',
     coordinator: 'hierarchical-coordinator',
     optimizer: 'perf-analyzer',
@@ -150,40 +17,38 @@ var LEGACY_AGENT_MAPPING = {
 };
 /**
  * Resolve legacy agent types to current equivalents
- */ function resolveLegacyAgentType(legacyType) {
+ */ export function resolveLegacyAgentType(legacyType) {
     return LEGACY_AGENT_MAPPING[legacyType] || legacyType;
 }
-var AgentLoader = /** @class */ function() {
-    function AgentLoader() {
-        this.agentCache = new Map();
-        this.categoriesCache = [];
-        this.lastLoadTime = 0;
-        this.CACHE_EXPIRY = 60000; // 1 minute cache
-    }
-    AgentLoader.prototype.getAgentsDirectory = function() {
-        var currentDir = process.cwd();
+export class AgentLoader {
+    agentCache = new Map();
+    categoriesCache = [];
+    lastLoadTime = 0;
+    CACHE_EXPIRY = 60_000;
+    getAgentsDirectory() {
+        let currentDir = process.cwd();
         while(currentDir !== '/'){
-            var claudeAgentsPath = (0, node_path_1.resolve)(currentDir, '.claude', 'agents');
-            if ((0, node_fs_1.existsSync)(claudeAgentsPath)) {
+            const claudeAgentsPath = resolve(currentDir, '.claude', 'agents');
+            if (existsSync(claudeAgentsPath)) {
                 return claudeAgentsPath;
             }
-            currentDir = (0, node_path_1.dirname)(currentDir);
+            currentDir = dirname(currentDir);
         }
-        return (0, node_path_1.resolve)(process.cwd(), '.claude', 'agents');
-    };
-    AgentLoader.prototype.parseAgentFile = function(filePath) {
+        return resolve(process.cwd(), '.claude', 'agents');
+    }
+    parseAgentFile(filePath) {
         try {
-            var content = (0, node_fs_1.readFileSync)(filePath, 'utf-8');
-            var frontmatterMatch = content.match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n([\s\S]*)$/);
+            const content = readFileSync(filePath, 'utf-8');
+            const frontmatterMatch = content.match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n([\s\S]*)$/);
             if (!frontmatterMatch) {
-                console.warn("No frontmatter found in ".concat(filePath));
+                console.warn(`No frontmatter found in ${filePath}`);
                 return null;
             }
-            var yamlContent = frontmatterMatch[1], markdownContent = frontmatterMatch[2];
-            var frontmatter = (0, yaml_1.parse)(yamlContent);
-            var description = frontmatter.description;
+            const [, yamlContent, markdownContent] = frontmatterMatch;
+            const frontmatter = parseYaml(yamlContent);
+            const description = frontmatter.description;
             if (!frontmatter.name || !description) {
-                console.warn("Missing required fields (name, description) in ".concat(filePath));
+                console.warn(`Missing required fields (name, description) in ${filePath}`);
                 return null;
             }
             return {
@@ -204,13 +69,129 @@ var AgentLoader = /** @class */ function() {
                 content: markdownContent.trim()
             };
         } catch (error) {
-            console.error("Error parsing agent file ".concat(filePath, ":"), error);
+            console.error(`Error parsing agent file ${filePath}:`, error);
             return null;
         }
-    };
-    AgentLoader.prototype.parseTools = function(frontmatter) {
-        var extractTools = function(input) {
+    }
+    parseTools(frontmatter) {
+        const extractTools = (input)=>{
             if (Array.isArray(input)) return input.map(String);
+            if (typeof input === 'string') {
+                return input.split(/[,\s]+/).map((t)=>t.trim()).filter((t)=>t.length > 0);
+            }
+            return [];
+        };
+        // Safely handle tools and capabilities.tools
+        const toolsFromFrontmatter = frontmatter.tools ? extractTools(frontmatter.tools) : [];
+        const toolsFromCapabilities = frontmatter.capabilities && typeof frontmatter.capabilities === 'object' ? extractTools(Object(frontmatter.capabilities).tools) : [];
+        return [
+            ...toolsFromFrontmatter,
+            ...toolsFromCapabilities
+        ];
+    }
+    async loadAgents() {
+        const agentsDir = this.getAgentsDirectory();
+        if (!existsSync(agentsDir)) {
+            console.warn(`Agents directory not found: ${agentsDir}`);
+            return;
+        }
+        const agentFiles = await new Promise((resolve, reject)=>{
+            glob('**/*.md', {
+                cwd: agentsDir,
+                ignore: [
+                    '**/README.md',
+                    '**/MIGRATION_SUMMARY.md'
+                ],
+                absolute: true
+            }, (err, matches)=>{
+                if (err) reject(err);
+                else resolve(matches);
+            });
+        });
+        this.agentCache.clear();
+        this.categoriesCache = [];
+        const categoryMap = new Map();
+        for (const filePath of agentFiles){
+            const agent = this.parseAgentFile(filePath);
+            if (agent) {
+                this.agentCache.set(agent.name, agent);
+                const relativePath = filePath.replace(agentsDir, '');
+                const pathParts = relativePath.split('/');
+                const category = pathParts[1] || 'uncategorized';
+                if (!categoryMap.has(category)) {
+                    categoryMap.set(category, []);
+                }
+                categoryMap.get(category).push(agent);
+            }
+        }
+        this.categoriesCache = Array.from(categoryMap.entries()).map(([name, agents])=>({
+                name,
+                agents: agents.sort((a, b)=>a.name.localeCompare(b.name))
+            }));
+        this.lastLoadTime = Date.now();
+    }
+    // Rest of the methods remain similar to the original implementation
+    needsRefresh() {
+        return Date.now() - this.lastLoadTime > this.CACHE_EXPIRY;
+    }
+    async ensureLoaded() {
+        if (this.agentCache.size === 0 || this.needsRefresh()) {
+            await this.loadAgents();
+        }
+    }
+    async getAvailableAgentTypes() {
+        await this.ensureLoaded();
+        const currentTypes = Array.from(this.agentCache.keys());
+        const legacyTypes = Object.keys(LEGACY_AGENT_MAPPING);
+        return Array.from(new Set([
+            ...currentTypes,
+            ...legacyTypes
+        ])).sort();
+    }
+    async getAgent(name) {
+        await this.ensureLoaded();
+        return this.agentCache.get(name) || this.agentCache.get(resolveLegacyAgentType(name)) || null;
+    }
+    async getAllAgents() {
+        await this.ensureLoaded();
+        return Array.from(this.agentCache.values()).sort((a, b)=>a.name.localeCompare(b.name));
+    }
+    async getAgentCategories() {
+        await this.ensureLoaded();
+        return this.categoriesCache;
+    }
+    async searchAgents(query) {
+        await this.ensureLoaded();
+        const lowerQuery = query.toLowerCase();
+        return Array.from(this.agentCache.values()).filter((agent)=>agent.name.toLowerCase().includes(lowerQuery) || agent.description.toLowerCase().includes(lowerQuery) || agent.capabilities?.some((cap)=>cap.toLowerCase().includes(lowerQuery)));
+    }
+    async isValidAgentType(name) {
+        await this.ensureLoaded();
+        return this.agentCache.has(name) || this.agentCache.has(resolveLegacyAgentType(name));
+    }
+    async getAgentsByCategory(category) {
+        const categories = await this.getAgentCategories();
+        const found = categories.find((cat)=>cat.name === category);
+        return found?.agents || [];
+    }
+    async refresh() {
+        this.lastLoadTime = 0;
+        await this.loadAgents();
+    }
+}
+// Singleton instance
+export const agentLoader = new AgentLoader();
+// Convenience exports for use in other modules
+export const getAvailableAgentTypes = ()=>agentLoader.getAvailableAgentTypes();
+export const getAgent = (name)=>agentLoader.getAgent(name);
+export const getAllAgents = ()=>agentLoader.getAllAgents();
+export const getAgentCategories = ()=>agentLoader.getAgentCategories();
+export const searchAgents = (query)=>agentLoader.searchAgents(query);
+export const isValidAgentType = (name)=>agentLoader.isValidAgentType(name);
+export const getAgentsByCategory = (category)=>agentLoader.getAgentsByCategory(category);
+export const refreshAgents = ()=>agentLoader.refresh();
+
+//# sourceMappingURL=agent-loader.js.map.isArray(input)) return input.map(String);
             if (typeof input === 'string') {
                 return input.split(/[,\s]+/).map(function(t) {
                     return t.trim();