wave-agent-sdk 0.7.1 → 0.8.0

package/dist/managers/permissionManager.js

@@ -12,6 +12,28 @@ import { splitBashCommand, stripEnvVars, stripRedirections, getSmartPrefix, DANG
 import { isPathInside } from "../utils/pathSafety.js";
 import { BASH_TOOL_NAME, EDIT_TOOL_NAME, WRITE_TOOL_NAME, READ_TOOL_NAME, LS_TOOL_NAME, } from "../constants/tools.js";
 const SAFE_COMMANDS = ["cd", "ls", "pwd", "true", "false"];
+const DEFAULT_ALLOWED_RULES = [
+    "Bash(git status*)",
+    "Bash(git diff*)",
+    "Bash(git log*)",
+    "Bash(git show*)",
+    "Bash(git branch*)",
+    "Bash(git tag*)",
+    "Bash(git remote*)",
+    "Bash(git ls-files*)",
+    "Bash(git rev-parse*)",
+    "Bash(git config --list*)",
+    "Bash(git config -l*)",
+    "Bash(git cat-file*)",
+    "Bash(git count-objects*)",
+    "Bash(echo*)",
+    "Bash(which*)",
+    "Bash(type*)",
+    "Bash(hostname*)",
+    "Bash(whoami*)",
+    "Bash(date*)",
+    "Bash(uptime*)",
+];
 import { logger } from "../utils/globalLogger.js";
 export class PermissionManager {
     constructor(container, options = {}) {
@@ -39,69 +61,71 @@ export class PermissionManager {
      */
     updateConfiguredDefaultMode(defaultMode) {
         const oldEffectiveMode = this.getCurrentEffectiveMode();
-        logger?.debug("Updating configured default permission mode", {
-            previous: this.configuredDefaultMode,
-            new: defaultMode,
-        });
         this.configuredDefaultMode = defaultMode;
         const newEffectiveMode = this.getCurrentEffectiveMode();
         if (oldEffectiveMode !== newEffectiveMode &&
             this.onConfiguredDefaultModeChange) {
-            logger?.debug("Effective permission mode changed due to configuration update", {
-                oldMode: oldEffectiveMode,
-                newMode: newEffectiveMode,
-            });
             this.onConfiguredDefaultModeChange(newEffectiveMode);
         }
     }
     /**
-     * Get all currently allowed rules
+     * Get the configured default mode
+     */
+    getConfiguredDefaultMode() {
+        return this.configuredDefaultMode;
+    }
+    /**
+     * Get all currently allowed rules (user-defined)
      */
     getAllowedRules() {
         return [...this.allowedRules];
     }
+    /**
+     * Get all currently denied rules
+     */
+    getDeniedRules() {
+        return [...this.deniedRules];
+    }
+    /**
+     * Get all additional directories
+     */
+    getAdditionalDirectories() {
+        return [...this.additionalDirectories];
+    }
+    /**
+     * Get all default allowed rules
+     */
+    getDefaultAllowedRules() {
+        return [...DEFAULT_ALLOWED_RULES];
+    }
     /**
      * Update the allowed rules (e.g., when configuration reloads)
      */
     updateAllowedRules(rules) {
-        logger?.debug("Updating allowed permission rules", {
-            count: rules.length,
-        });
         this.allowedRules = rules;
     }
     /**
      * Update the denied rules (e.g., when configuration reloads)
      */
     updateDeniedRules(rules) {
-        logger?.debug("Updating denied permission rules", {
-            count: rules.length,
-        });
         this.deniedRules = rules;
     }
     /**
      * Add temporary rules for the current session
      */
     addTemporaryRules(rules) {
-        logger?.debug("Adding temporary permission rules", {
-            count: rules.length,
-            rules,
-        });
         this.temporaryRules.push(...rules);
     }
     /**
      * Clear all temporary rules
      */
     clearTemporaryRules() {
-        logger?.debug("Clearing temporary permission rules");
         this.temporaryRules = [];
     }
     /**
      * Update the additional directories (e.g., when configuration reloads)
      */
     updateAdditionalDirectories(directories) {
-        logger?.debug("Updating additional directories", {
-            count: directories.length,
-        });
         this.additionalDirectories = directories.map((dir) => {
             if (this.workdir && !path.isAbsolute(dir)) {
                 return path.resolve(this.workdir, dir);
@@ -113,16 +137,12 @@ export class PermissionManager {
      * Update the working directory
      */
     updateWorkdir(workdir) {
-        logger?.debug("Updating working directory", {
-            workdir,
-        });
         this.workdir = workdir;
     }
     /**
      * Set the current plan file path
      */
     setPlanFilePath(path) {
-        logger?.debug("Setting plan file path", { path });
         this.planFilePath = path;
     }
     /**
@@ -150,11 +170,6 @@ export class PermissionManager {
                 return { isInside: true, resolvedPath: absolutePath };
             }
         }
-        logger?.debug("Path is outside Safe Zone", {
-            absolutePath,
-            workdir: effectiveWorkdir,
-            additionalDirectories: this.additionalDirectories,
-        });
         return { isInside: false, resolvedPath: absolutePath };
     }
     /**
@@ -169,21 +184,13 @@ export class PermissionManager {
     resolveEffectivePermissionMode(cliPermissionMode) {
         // CLI override takes highest precedence
         if (cliPermissionMode !== undefined) {
-            logger?.debug("Using CLI permission mode override", {
-                cliMode: cliPermissionMode,
-                configuredDefault: this.configuredDefaultMode,
-            });
             return cliPermissionMode;
         }
         // Use configured default mode if available
         if (this.configuredDefaultMode !== undefined) {
-            logger?.debug("Using configured default permission mode", {
-                configuredDefault: this.configuredDefaultMode,
-            });
             return this.configuredDefaultMode;
         }
         // Fall back to system default
-        logger?.debug("Using system default permission mode");
         return "default";
     }
     /**
@@ -191,11 +198,6 @@ export class PermissionManager {
      * Called by individual tools after validation/diff, before real operation
      */
     async checkPermission(context) {
-        logger?.debug("Checking permission for tool", {
-            toolName: context.toolName,
-            permissionMode: context.permissionMode,
-            hasCallback: !!context.canUseToolCallback,
-        });
         // 0. Check denied rules first - Deny always takes precedence
         for (const rule of this.deniedRules) {
             if (this.matchesRule(context, rule)) {
@@ -211,9 +213,6 @@ export class PermissionManager {
         }
         // 1. If bypassPermissions mode, always allow
         if (context.permissionMode === "bypassPermissions") {
-            logger?.debug("Permission bypassed for tool", {
-                toolName: context.toolName,
-            });
             return { behavior: "allow" };
         }
         // 1.1 If acceptEdits mode, allow Edit, Write
@@ -234,9 +233,6 @@ export class PermissionManager {
                         // Fall through to normal permission check flow to trigger confirmation prompt
                     }
                     else {
-                        logger?.debug("Permission automatically accepted for tool in acceptEdits mode", {
-                            toolName: context.toolName,
-                        });
                         return { behavior: "allow" };
                     }
                 }
@@ -244,9 +240,6 @@ export class PermissionManager {
         }
         // 1.2 Check if tool call is allowed by persistent or temporary rules
         if (this.isAllowedByRule(context)) {
-            logger?.debug("Permission allowed by persistent rule", {
-                toolName: context.toolName,
-            });
             return { behavior: "allow" };
         }
         // 1.3 If plan mode, allow Read-only tools and Edit/Write for plan file
@@ -258,10 +251,6 @@ export class PermissionManager {
                     const absoluteTargetPath = path.resolve(targetPath);
                     const absolutePlanPath = path.resolve(this.planFilePath);
                     if (absoluteTargetPath === absolutePlanPath) {
-                        logger?.debug("Allowing write to plan file in plan mode", {
-                            toolName: context.toolName,
-                            targetPath,
-                        });
                         return { behavior: "allow" };
                     }
                 }
@@ -273,22 +262,18 @@ export class PermissionManager {
         }
         // 2. If not a restricted tool, always allow
         if (!this.isRestrictedTool(context.toolName)) {
-            logger?.debug("Tool is not restricted, allowing", {
-                toolName: context.toolName,
-            });
             return { behavior: "allow" };
         }
         // 3. If custom callback provided, call it and return result
         if (context.canUseToolCallback) {
             try {
-                logger?.debug("Calling custom permission callback for tool", {
-                    toolName: context.toolName,
-                });
                 const decision = await context.canUseToolCallback(context);
-                logger?.debug("Custom callback returned decision", {
-                    toolName: context.toolName,
-                    decision,
-                });
+                if (decision.behavior !== "allow") {
+                    logger?.debug("Custom callback returned decision", {
+                        toolName: context.toolName,
+                        decision,
+                    });
+                }
                 return decision;
             }
             catch (error) {
@@ -317,12 +302,7 @@ export class PermissionManager {
      * Determine if a tool requires permission checks based on its name
      */
     isRestrictedTool(toolName) {
-        const isRestricted = RESTRICTED_TOOLS.includes(toolName);
-        logger?.debug("Checking if tool is restricted", {
-            toolName,
-            isRestricted,
-        });
-        return isRestricted;
+        return RESTRICTED_TOOLS.includes(toolName);
     }
     /**
      * Helper method to create a permission context for CLI integration
@@ -388,13 +368,6 @@ export class PermissionManager {
                 context.hidePersistentOption = true;
             }
         }
-        logger?.debug("Created permission context", {
-            toolName,
-            permissionMode,
-            hasCallback: !!callback,
-            hasToolInput: !!toolInput,
-            suggestedPrefix,
-        });
         return context;
     }
     /**
@@ -424,8 +397,9 @@ export class PermissionManager {
             const regexPattern = pattern
                 .replace(/[.+^${}()|[\]\\?]/g, "\\$&") // Escape regex special chars including ?
                 .replace(/\*/g, ".*"); // Replace * with .*
-            const regex = new RegExp(`^${regexPattern}$`);
-            return regex.test(processedPart);
+            const regex = new RegExp(`^${regexPattern}$`, "s");
+            const matched = regex.test(processedPart);
+            return matched;
         }
         // Handle path-based rules (e.g., "Read(**/*.env)")
         const pathTools = [
@@ -490,7 +464,9 @@ export class PermissionManager {
                         ...ctx,
                         toolInput: { ...ctx.toolInput, command: processedPart },
                     };
-                    const allowedByRule = rules.some((rule) => this.matchesRule(partContext, rule));
+                    const allowedByRule = rules.some((rule) => {
+                        return this.matchesRule(partContext, rule);
+                    });
                     if (allowedByRule)
                         return true;
                     return !this.isRestrictedTool(ctx.toolName);
@@ -504,7 +480,11 @@ export class PermissionManager {
             return true;
         }
         // Check persistent allowed rules
-        return isAllowedByRuleList(context, this.allowedRules);
+        if (isAllowedByRuleList(context, this.allowedRules)) {
+            return true;
+        }
+        // Check default allowed rules
+        return isAllowedByRuleList(context, DEFAULT_ALLOWED_RULES);
     }
     /**
      * Expand a bash command into individual permission rules, filtering out safe commands.
@@ -580,4 +560,45 @@ export class PermissionManager {
         }
         return rules;
     }
+    /**
+     * Add a persistent permission rule
+     * @param rule - The rule to add (e.g., "Bash(ls)")
+     */
+    async addPermissionRule(rule) {
+        if (!this.workdir) {
+            throw new Error("Working directory not set in PermissionManager");
+        }
+        // 1. Expand rule if it's a Bash command
+        let rulesToAdd = [rule];
+        const bashMatch = rule.match(/^Bash\((.*)\)$/);
+        if (bashMatch) {
+            const command = bashMatch[1];
+            rulesToAdd = this.expandBashRule(command, this.workdir);
+        }
+        const configurationService = this.container.get("ConfigurationService");
+        for (const ruleToAdd of rulesToAdd) {
+            // 2. Update PermissionManager state
+            const currentRules = this.getAllowedRules();
+            const defaultRules = this.getDefaultAllowedRules();
+            if (!currentRules.includes(ruleToAdd) &&
+                !defaultRules.includes(ruleToAdd)) {
+                this.updateAllowedRules([...currentRules, ruleToAdd]);
+                // 3. Persist to settings.local.json
+                try {
+                    if (configurationService) {
+                        await configurationService.addAllowedRule(this.workdir, ruleToAdd);
+                        this._logger?.debug("Persistent permission rule added", {
+                            rule: ruleToAdd,
+                        });
+                    }
+                }
+                catch (error) {
+                    this._logger?.error("Failed to persist permission rule", {
+                        rule: ruleToAdd,
+                        error: error instanceof Error ? error.message : String(error),
+                    });
+                }
+            }
+        }
+    }
 }

package/dist/managers/planManager.d.ts

@@ -1,4 +1,5 @@
 import { Container } from "../utils/container.js";
+import type { PermissionMode } from "../types/permissions.js";
 export declare class PlanManager {
     private container;
     private planDir;
@@ -15,5 +16,10 @@ export declare class PlanManager {
      * Returns the directory where plan files are stored
      */
     getPlanDir(): string;
+    /**
+     * Handle plan mode transition, generating or clearing plan file path
+     * @param mode - The current effective permission mode
+     */
+    handlePlanModeTransition(mode: PermissionMode): void;
 }
 //# sourceMappingURL=planManager.d.ts.map

package/dist/managers/planManager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"planManager.d.ts","sourceRoot":"","sources":["../../src/managers/planManager.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAOlD,qBAAa,WAAW;IAIV,OAAO,CAAC,SAAS;IAH7B,OAAO,CAAC,OAAO,CAAS;IACxB,OAAO,CAAC,mBAAmB,CAAuB;gBAE9B,SAAS,EAAE,SAAS;IAIxC;;OAEG;IACU,yBAAyB,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC;QAC7D,IAAI,EAAE,MAAM,CAAC;QACb,IAAI,EAAE,MAAM,CAAC;KACd,CAAC;IA+BF;;OAEG;IACI,UAAU,IAAI,MAAM;CAG5B"}
+{"version":3,"file":"planManager.d.ts","sourceRoot":"","sources":["../../src/managers/planManager.ts"],"names":[],"mappings":"AAKA,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAGlD,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,yBAAyB,CAAC;AAO9D,qBAAa,WAAW;IAIV,OAAO,CAAC,SAAS;IAH7B,OAAO,CAAC,OAAO,CAAS;IACxB,OAAO,CAAC,mBAAmB,CAAuB;gBAE9B,SAAS,EAAE,SAAS;IAIxC;;OAEG;IACU,yBAAyB,CAAC,IAAI,CAAC,EAAE,MAAM,GAAG,OAAO,CAAC;QAC7D,IAAI,EAAE,MAAM,CAAC;QACb,IAAI,EAAE,MAAM,CAAC;KACd,CAAC;IA+BF;;OAEG;IACI,UAAU,IAAI,MAAM;IAI3B;;;OAGG;IACI,wBAAwB,CAAC,IAAI,EAAE,cAAc,GAAG,IAAI;CAkB5D"}

package/dist/managers/planManager.js

@@ -46,4 +46,25 @@ export class PlanManager {
     getPlanDir() {
         return this.planDir;
     }
+    /**
+     * Handle plan mode transition, generating or clearing plan file path
+     * @param mode - The current effective permission mode
+     */
+    handlePlanModeTransition(mode) {
+        const permissionManager = this.container.get("PermissionManager");
+        const messageManager = this.container.get("MessageManager");
+        if (mode === "plan") {
+            this.getOrGeneratePlanFilePath(messageManager?.getRootSessionId())
+                .then(({ path }) => {
+                logger?.debug("Plan file path generated", { path });
+                permissionManager?.setPlanFilePath(path);
+            })
+                .catch((error) => {
+                logger?.error("Failed to generate plan file path", error);
+            });
+        }
+        else {
+            permissionManager?.setPlanFilePath(undefined);
+        }
+    }
 }

package/dist/managers/skillManager.d.ts

@@ -24,6 +24,10 @@ export declare class SkillManager {
      * Get all available skills metadata
      */
     getAvailableSkills(): SkillMetadata[];
+    /**
+     * Get metadata for a specific skill by name
+     */
+    getSkillMetadata(name: string): SkillMetadata | undefined;
     /**
      * Load a specific skill by name
      * Returns the skill content that was loaded during initialization
@@ -47,11 +51,12 @@ export declare class SkillManager {
     executeSkill(args: SkillToolArgs): Promise<{
         content: string;
         context?: SkillInvocationContext;
+        allowedTools?: string[];
     }>;
     /**
-     * Format skill content for output
+     * Process skill content with arguments and bash commands
      */
-    private formatSkillContent;
+    private processSkillContent;
     /**
      * Format available skills list for error messages
      */

package/dist/managers/skillManager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"skillManager.d.ts","sourceRoot":"","sources":["../../src/managers/skillManager.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EACV,mBAAmB,EACnB,aAAa,EACb,KAAK,EAGL,aAAa,EACb,sBAAsB,EACvB,MAAM,mBAAmB,CAAC;AAG3B,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAGlD;;GAEG;AACH,qBAAa,YAAY;IAUrB,OAAO,CAAC,SAAS;IATnB,OAAO,CAAC,kBAAkB,CAAS;IACnC,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,OAAO,CAAS;IAExB,OAAO,CAAC,aAAa,CAAoC;IACzD,OAAO,CAAC,YAAY,CAA4B;IAChD,OAAO,CAAC,WAAW,CAAS;gBAGlB,SAAS,EAAE,SAAS,EAC5B,OAAO,GAAE,mBAAwB;IAQnC;;OAEG;IACG,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC;IAoCjC;;OAEG;IACH,aAAa,IAAI,OAAO;IAIxB;;OAEG;IACH,kBAAkB,IAAI,aAAa,EAAE;IAQrC;;;OAGG;IACG,SAAS,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,KAAK,GAAG,IAAI,CAAC;IAgBzD;;OAEG;YACW,cAAc;IAkB5B;;OAEG;YACW,uBAAuB;IAiFrC;;OAEG;YACW,oBAAoB;IAkBlC;;OAEG;IACG,YAAY,CAChB,IAAI,EAAE,aAAa,GAClB,OAAO,CAAC;QAAE,OAAO,EAAE,MAAM,CAAC;QAAC,OAAO,CAAC,EAAE,sBAAsB,CAAA;KAAE,CAAC;IAqCjE;;OAEG;IACH,OAAO,CAAC,kBAAkB;IAY1B;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAc7B;;OAEG;IACH,oBAAoB,CAAC,MAAM,EAAE,KAAK,EAAE,GAAG,IAAI;CAc5C"}
+{"version":3,"file":"skillManager.d.ts","sourceRoot":"","sources":["../../src/managers/skillManager.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EACV,mBAAmB,EACnB,aAAa,EACb,KAAK,EAGL,aAAa,EACb,sBAAsB,EACvB,MAAM,mBAAmB,CAAC;AAS3B,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAGlD;;GAEG;AACH,qBAAa,YAAY;IAUrB,OAAO,CAAC,SAAS;IATnB,OAAO,CAAC,kBAAkB,CAAS;IACnC,OAAO,CAAC,WAAW,CAAS;IAC5B,OAAO,CAAC,OAAO,CAAS;IAExB,OAAO,CAAC,aAAa,CAAoC;IACzD,OAAO,CAAC,YAAY,CAA4B;IAChD,OAAO,CAAC,WAAW,CAAS;gBAGlB,SAAS,EAAE,SAAS,EAC5B,OAAO,GAAE,mBAAwB;IAQnC;;OAEG;IACG,UAAU,IAAI,OAAO,CAAC,IAAI,CAAC;IAoCjC;;OAEG;IACH,aAAa,IAAI,OAAO;IAIxB;;OAEG;IACH,kBAAkB,IAAI,aAAa,EAAE;IAQrC;;OAEG;IACH,gBAAgB,CAAC,IAAI,EAAE,MAAM,GAAG,aAAa,GAAG,SAAS;IAQzD;;;OAGG;IACG,SAAS,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO,CAAC,KAAK,GAAG,IAAI,CAAC;IAgBzD;;OAEG;YACW,cAAc;IAkB5B;;OAEG;YACW,uBAAuB;IA8ErC;;OAEG;YACW,oBAAoB;IAkBlC;;OAEG;IACG,YAAY,CAAC,IAAI,EAAE,aAAa,GAAG,OAAO,CAAC;QAC/C,OAAO,EAAE,MAAM,CAAC;QAChB,OAAO,CAAC,EAAE,sBAAsB,CAAC;QACjC,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;KACzB,CAAC;IA4CF;;OAEG;YACW,mBAAmB;IAyBjC;;OAEG;IACH,OAAO,CAAC,qBAAqB;IAc7B;;OAEG;IACH,oBAAoB,CAAC,MAAM,EAAE,KAAK,EAAE,GAAG,IAAI;CAe5C"}

package/dist/managers/skillManager.js

@@ -2,6 +2,8 @@ import { readdir, stat } from "fs/promises";
 import { join } from "path";
 import { homedir } from "os";
 import { parseSkillFile, formatSkillError } from "../utils/skillParser.js";
+import { substituteCommandParameters } from "../utils/commandArgumentParser.js";
+import { parseBashCommands, replaceBashCommandsWithOutput, executeBashCommands, } from "../utils/markdownParser.js";
 import { logger } from "../utils/globalLogger.js";
 /**
  * Manages skill discovery and loading
@@ -64,6 +66,15 @@ export class SkillManager {
         }
         return Array.from(this.skillMetadata.values());
     }
+    /**
+     * Get metadata for a specific skill by name
+     */
+    getSkillMetadata(name) {
+        if (!this.initialized) {
+            throw new Error("SkillManager not initialized. Call initialize() first.");
+        }
+        return this.skillMetadata.get(name);
+    }
     /**
      * Load a specific skill by name
      * Returns the skill content that was loaded during initialization
@@ -129,10 +140,7 @@ export class SkillManager {
                         };
                         // Create full skill object with content
                         const skill = {
-                            name: parsed.skillMetadata.name,
-                            description: parsed.skillMetadata.description,
-                            type: type, // Use the collection type
-                            skillPath: parsed.skillMetadata.skillPath,
+                            ...skillMetadata,
                             content: parsed.content,
                             frontmatter: parsed.frontmatter,
                             isValid: parsed.isValid,
@@ -185,8 +193,8 @@ export class SkillManager {
      * Execute a skill by name
      */
     async executeSkill(args) {
-        const { skill_name } = args;
-        logger?.debug(`Invoking skill: ${skill_name}`);
+        const { skill_name, args: skillArgs } = args;
+        logger?.debug(`Invoking skill: ${skill_name} with args: ${skillArgs}`);
         try {
             // Load the skill
             const skill = await this.loadSkill(skill_name);
@@ -201,12 +209,15 @@ export class SkillManager {
                     content: `❌ **Skill validation failed**:\n\n\`\`\`\n${errorMsg}\n\`\`\``,
                 };
             }
+            // Process skill content with arguments and bash commands
+            const processedContent = await this.processSkillContent(skill, skillArgs || "");
             // Return skill content with context
             return {
-                content: this.formatSkillContent(skill),
+                content: processedContent,
                 context: {
                     skillName: skill_name,
                 },
+                allowedTools: skill.allowedTools,
             };
         }
         catch (error) {
@@ -217,15 +228,23 @@ export class SkillManager {
         }
     }
     /**
-     * Format skill content for output
+     * Process skill content with arguments and bash commands
      */
-    formatSkillContent(skill) {
+    async processSkillContent(skill, argsString) {
         const header = `🧠 **${skill.name}** (${skill.type} skill)\n\n`;
         const description = `*${skill.description}*\n\n`;
         const skillPath = `📁 Skill location: \`${skill.skillPath}\`\n\n`;
         // Extract content after frontmatter
         const contentMatch = skill.content.match(/^---\n[\s\S]*?\n---\n([\s\S]*)$/);
-        const mainContent = contentMatch ? contentMatch[1].trim() : skill.content;
+        let mainContent = contentMatch ? contentMatch[1].trim() : skill.content;
+        // 1. Substitute parameters ($1, $ARGUMENTS, etc.)
+        mainContent = substituteCommandParameters(mainContent, argsString);
+        // 2. Parse and execute bash commands (!`command`)
+        const { commands } = parseBashCommands(mainContent);
+        if (commands.length > 0) {
+            const results = await executeBashCommands(commands, this.workdir);
+            mainContent = replaceBashCommandsWithOutput(mainContent, results);
+        }
         return header + description + skillPath + mainContent;
     }
     /**
@@ -250,6 +269,7 @@ export class SkillManager {
                 description: skill.description,
                 type: skill.type,
                 skillPath: skill.skillPath,
+                allowedTools: skill.allowedTools,
             });
             this.skillContent.set(skill.name, skill);
         }

package/dist/managers/slashCommandManager.d.ts

@@ -1,5 +1,6 @@
 import type { SlashCommand, CustomSlashCommand } from "../types/index.js";
 import { Container } from "../utils/container.js";
+import type { SkillMetadata } from "../types/skills.js";
 export interface SlashCommandManagerOptions {
     workdir: string;
 }
@@ -7,6 +8,7 @@ export declare class SlashCommandManager {
     private container;
     private commands;
     private customCommands;
+    private skillCommandIds;
     private workdir;
     constructor(container: Container, options: SlashCommandManagerOptions);
     initialize(): void;
@@ -14,11 +16,16 @@ export declare class SlashCommandManager {
     private get aiManager();
     private get backgroundTaskManager();
     private get taskManager();
+    private get skillManager();
     private initializeBuiltinCommands;
     /**
      * Load custom commands from filesystem
      */
     private loadCustomCommands;
+    /**
+     * Register skills as slash commands
+     */
+    registerSkillCommands(skills: SkillMetadata[]): void;
     /**
      * Register commands from a plugin with namespacing
      */

package/dist/managers/slashCommandManager.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"slashCommandManager.d.ts","sourceRoot":"","sources":["../../src/managers/slashCommandManager.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,YAAY,EAAE,kBAAkB,EAAE,MAAM,mBAAmB,CAAC;AAQ1E,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAclD,MAAM,WAAW,0BAA0B;IACzC,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,qBAAa,mBAAmB;IAM5B,OAAO,CAAC,SAAS;IALnB,OAAO,CAAC,QAAQ,CAAmC;IACnD,OAAO,CAAC,cAAc,CAAyC;IAC/D,OAAO,CAAC,OAAO,CAAS;gBAGd,SAAS,EAAE,SAAS,EAC5B,OAAO,EAAE,0BAA0B;IAK9B,UAAU,IAAI,IAAI;IAKzB,OAAO,KAAK,cAAc,GAEzB;IAED,OAAO,KAAK,SAAS,GAEpB;IAED,OAAO,KAAK,qBAAqB,GAEhC;IAED,OAAO,KAAK,WAAW,GAEtB;IAED,OAAO,CAAC,yBAAyB;IAsCjC;;OAEG;IACH,OAAO,CAAC,kBAAkB;IAyD1B;;OAEG;IACI,sBAAsB,CAC3B,UAAU,EAAE,MAAM,EAClB,QAAQ,EAAE,kBAAkB,EAAE,GAC7B,IAAI;IAwDP;;OAEG;IACI,oBAAoB,IAAI,IAAI;IAWnC;;OAEG;IACI,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,IAAI;IAKnD;;OAEG;IACH,OAAO,CAAC,iBAAiB;IAIzB;;OAEG;IACI,WAAW,IAAI,YAAY,EAAE;IAIpC;;OAEG;IACI,UAAU,CAAC,SAAS,EAAE,MAAM,GAAG,YAAY,GAAG,SAAS;IAI9D;;OAEG;IACU,cAAc,CACzB,SAAS,EAAE,MAAM,EACjB,IAAI,CAAC,EAAE,MAAM,GACZ,OAAO,CAAC,OAAO,CAAC;IAenB;;;OAGG;IACI,4BAA4B,CAAC,KAAK,EAAE,MAAM,GAAG;QAClD,OAAO,EAAE,OAAO,CAAC;QACjB,SAAS,CAAC,EAAE,MAAM,CAAC;QACnB,IAAI,CAAC,EAAE,MAAM,CAAC;KACf;IAeD;;OAEG;IACI,UAAU,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO;IAI7C;;OAEG;IACI,gBAAgB,CAAC,SAAS,EAAE,MAAM,GAAG,kBAAkB,GAAG,SAAS;IAI1E;;OAEG;IACI,iBAAiB,IAAI,kBAAkB,EAAE;IAIhD;;OAEG;YACW,+BAA+B;IA0E7C;;OAEG;IACI,mBAAmB,IAAI,IAAI;CAInC"}
+{"version":3,"file":"slashCommandManager.d.ts","sourceRoot":"","sources":["../../src/managers/slashCommandManager.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,YAAY,EAAE,kBAAkB,EAAE,MAAM,mBAAmB,CAAC;AAQ1E,OAAO,EAAE,SAAS,EAAE,MAAM,uBAAuB,CAAC;AAQlD,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAIxD,MAAM,WAAW,0BAA0B;IACzC,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,qBAAa,mBAAmB;IAO5B,OAAO,CAAC,SAAS;IANnB,OAAO,CAAC,QAAQ,CAAmC;IACnD,OAAO,CAAC,cAAc,CAAyC;IAC/D,OAAO,CAAC,eAAe,CAAqB;IAC5C,OAAO,CAAC,OAAO,CAAS;gBAGd,SAAS,EAAE,SAAS,EAC5B,OAAO,EAAE,0BAA0B;IAK9B,UAAU,IAAI,IAAI;IAKzB,OAAO,KAAK,cAAc,GAEzB;IAED,OAAO,KAAK,SAAS,GAEpB;IAED,OAAO,KAAK,qBAAqB,GAEhC;IAED,OAAO,KAAK,WAAW,GAEtB;IAED,OAAO,KAAK,YAAY,GAEvB;IAED,OAAO,CAAC,yBAAyB;IAoBjC;;OAEG;IACH,OAAO,CAAC,kBAAkB;IAyD1B;;OAEG;IACI,qBAAqB,CAAC,MAAM,EAAE,aAAa,EAAE,GAAG,IAAI;IAyD3D;;OAEG;IACI,sBAAsB,CAC3B,UAAU,EAAE,MAAM,EAClB,QAAQ,EAAE,kBAAkB,EAAE,GAC7B,IAAI;IAwDP;;OAEG;IACI,oBAAoB,IAAI,IAAI;IAWnC;;OAEG;IACI,eAAe,CAAC,OAAO,EAAE,YAAY,GAAG,IAAI;IAKnD;;OAEG;IACH,OAAO,CAAC,iBAAiB;IAIzB;;OAEG;IACI,WAAW,IAAI,YAAY,EAAE;IAIpC;;OAEG;IACI,UAAU,CAAC,SAAS,EAAE,MAAM,GAAG,YAAY,GAAG,SAAS;IAI9D;;OAEG;IACU,cAAc,CACzB,SAAS,EAAE,MAAM,EACjB,IAAI,CAAC,EAAE,MAAM,GACZ,OAAO,CAAC,OAAO,CAAC;IAenB;;;OAGG;IACI,4BAA4B,CAAC,KAAK,EAAE,MAAM,GAAG;QAClD,OAAO,EAAE,OAAO,CAAC;QACjB,SAAS,CAAC,EAAE,MAAM,CAAC;QACnB,IAAI,CAAC,EAAE,MAAM,CAAC;KACf;IAeD;;OAEG;IACI,UAAU,CAAC,SAAS,EAAE,MAAM,GAAG,OAAO;IAI7C;;OAEG;IACI,gBAAgB,CAAC,SAAS,EAAE,MAAM,GAAG,kBAAkB,GAAG,SAAS;IAI1E;;OAEG;IACI,iBAAiB,IAAI,kBAAkB,EAAE;IAIhD;;OAEG;YACW,+BAA+B;IAiD7C;;OAEG;IACI,mBAAmB,IAAI,IAAI;CAInC"}