@bike4mind/cli 0.2.21-feat-cli-skill-frontmatter-spec.18205 → 0.2.21-feat-github-actions-tool.18208

package/dist/index.js

@@ -2500,8 +2500,10 @@ import { z } from "zod";
 var flexibleString = z.union([z.string(), z.array(z.string())]).transform((val) => Array.isArray(val) ? val.join(" ") : val).optional();
 function needsQuoting(value) {
   const firstChar = value.charAt(0);
-  const isProtected = firstChar === '"' || firstChar === "'" || firstChar === "[" || firstChar === "{";
-  return value.includes(":") && !isProtected;
+  const isAlreadyQuoted = firstChar === '"' || firstChar === "'";
+  const isStructured = firstChar === "[" || firstChar === "{";
+  const containsColon = value.includes(":");
+  return containsColon && !isAlreadyQuoted && !isStructured;
 }
 function preprocessFrontmatter(content) {
   const frontmatterMatch = content.match(/^---\r?\n([\s\S]*?)\r?\n---/);
@@ -2523,66 +2525,29 @@ function preprocessFrontmatter(content) {
 ${processedFrontmatter}
 ---`);
 }
-var HooksSchema = z.object({
-  /** Script to run before skill execution */
-  "pre-invoke": z.string().optional(),
-  /** Script to run after successful skill execution */
-  "post-invoke": z.string().optional(),
-  /** Script to run when skill execution fails */
-  "on-error": z.string().optional()
-}).optional();
-var AgentConfigSchema = z.union([
-  z.string(),
-  // Simple: "explore"
-  z.object({
-    // Complex config
-    type: z.string(),
-    // Agent type name
-    thoroughness: z.enum(["quick", "medium", "very_thorough"]).optional(),
-    config: z.record(z.any()).optional()
-    // Additional agent-specific config
-  })
-]);
-var VALID_MODELS = ["opus", "sonnet", "haiku"];
 var FrontmatterSchema = z.object({
-  // Display name for the skill (defaults to filename if not specified)
-  name: z.string().optional(),
   description: flexibleString,
   "argument-hint": flexibleString,
-  // Model override - validated against allowed values (opus, sonnet, haiku)
   model: z.string().optional(),
   // Agent integration fields
-  agent: AgentConfigSchema.optional(),
+  agent: z.string().optional(),
   thoroughness: z.enum(["quick", "medium", "very_thorough"]).optional(),
   variables: z.record(z.string()).optional(),
-  // Tool filtering - restrict which tools are available during skill execution
-  "allowed-tools": z.array(z.string()).optional(),
-  // Execution context: 'inline' (default) runs in main context, 'fork' runs in subagent
-  context: z.enum(["fork", "inline"]).default("inline"),
-  // Visibility controls
-  /** When true, skill is hidden from AI's auto-loading in system prompt */
-  "disable-model-invocation": z.boolean().default(false),
-  /** When false, skill is hidden from /commands menu but still callable */
-  "user-invocable": z.boolean().default(true),
-  // Lifecycle hooks
-  hooks: HooksSchema
-});
-var DESCRIPTION_MAX_LENGTH = 100;
-var DEFAULT_DESCRIPTION = "Custom command";
+  // Future fields (ignored for now):
+  "allowed-tools": z.any().optional(),
+  context: z.any().optional(),
+  "disable-model-invocation": z.any().optional(),
+  hooks: z.any().optional()
+});
 function extractDescriptionFromBody(body) {
   const lines = body.trim().split("\n");
-  const firstContentLine = lines.find((line) => {
-    const trimmed2 = line.trim();
-    return trimmed2 && !trimmed2.startsWith("#");
-  });
-  if (!firstContentLine) {
-    return DEFAULT_DESCRIPTION;
-  }
-  const trimmed = firstContentLine.trim();
-  if (trimmed.length <= DESCRIPTION_MAX_LENGTH) {
-    return trimmed;
+  for (const line of lines) {
+    const trimmed = line.trim();
+    if (trimmed && !trimmed.startsWith("#")) {
+      return trimmed.length > 100 ? trimmed.substring(0, 97) + "..." : trimmed;
+    }
   }
-  return trimmed.substring(0, DESCRIPTION_MAX_LENGTH - 3) + "...";
+  return "Custom command";
 }
 function parseCommandFile(fileContent, filePath, commandName, source) {
   try {
@@ -2596,20 +2561,9 @@ function parseCommandFile(fileContent, filePath, commandName, source) {
       );
     }
     const validFrontmatter = validationResult.success ? validationResult.data : {};
-    const description = validFrontmatter.description || extractDescriptionFromBody(body);
-    if (validFrontmatter.model && !VALID_MODELS.includes(validFrontmatter.model)) {
-      console.warn(
-        `Warning: Invalid model "${validFrontmatter.model}" in ${filePath}. Valid values are: ${VALID_MODELS.join(", ")}`
-      );
-    }
-    if (validFrontmatter.agent && validFrontmatter.context !== "fork") {
-      console.warn(
-        `Warning: Skill "${commandName}" has "agent" specified but "context" is not "fork". The agent field is only used when context is "fork". Consider adding "context: fork" to the frontmatter.`
-      );
-    }
+    const description = validFrontmatter.description || extractDescriptionFromBody(body) || "Custom command";
     return {
       name: commandName,
-      displayName: validFrontmatter.name,
       description,
       argumentHint: validFrontmatter["argument-hint"],
       model: validFrontmatter.model,
@@ -2618,14 +2572,7 @@ function parseCommandFile(fileContent, filePath, commandName, source) {
       filePath,
       agent: validFrontmatter.agent,
       thoroughness: validFrontmatter.thoroughness,
-      variables: validFrontmatter.variables,
-      // New Claude Code spec fields
-      allowedTools: validFrontmatter["allowed-tools"],
-      context: validFrontmatter.context || "inline",
-      disableModelInvocation: validFrontmatter["disable-model-invocation"] || false,
-      userInvocable: validFrontmatter["user-invocable"] !== false,
-      // Default true
-      hooks: validFrontmatter.hooks
+      variables: validFrontmatter.variables
     };
   } catch (error) {
     throw new Error(
@@ -2633,9 +2580,12 @@ function parseCommandFile(fileContent, filePath, commandName, source) {
     );
   }
 }
-var COMMAND_NAME_PATTERN = /^[a-z0-9-_:]+$/i;
 function isValidCommandName(name) {
-  return Boolean(name) && COMMAND_NAME_PATTERN.test(name);
+  if (!name || name.trim().length === 0) {
+    return false;
+  }
+  const validPattern = /^[a-z0-9-_:]+$/i;
+  return validPattern.test(name);
 }
 function extractCommandName(filename) {
   if (!filename.endsWith(".md")) {
@@ -12312,7 +12262,7 @@ import { isAxiosError as isAxiosError2 } from "axios";
 // package.json
 var package_default = {
   name: "@bike4mind/cli",
-  version: "0.2.21-feat-cli-skill-frontmatter-spec.18205+406977892",
+  version: "0.2.21-feat-github-actions-tool.18208+9161cdf04",
   type: "module",
   description: "Interactive CLI tool for Bike4Mind with ReAct agents",
   license: "UNLICENSED",
@@ -12419,10 +12369,10 @@ var package_default = {
   },
   devDependencies: {
     "@bike4mind/agents": "0.1.0",
-    "@bike4mind/common": "2.45.1-feat-cli-skill-frontmatter-spec.18205+406977892",
-    "@bike4mind/mcp": "1.25.1-feat-cli-skill-frontmatter-spec.18205+406977892",
-    "@bike4mind/services": "2.43.1-feat-cli-skill-frontmatter-spec.18205+406977892",
-    "@bike4mind/utils": "2.3.1-feat-cli-skill-frontmatter-spec.18205+406977892",
+    "@bike4mind/common": "2.45.1-feat-github-actions-tool.18208+9161cdf04",
+    "@bike4mind/mcp": "1.25.1-feat-github-actions-tool.18208+9161cdf04",
+    "@bike4mind/services": "2.43.1-feat-github-actions-tool.18208+9161cdf04",
+    "@bike4mind/utils": "2.3.1-feat-github-actions-tool.18208+9161cdf04",
     "@types/better-sqlite3": "^7.6.13",
     "@types/diff": "^5.0.9",
     "@types/jsonwebtoken": "^9.0.4",
@@ -12439,7 +12389,7 @@ var package_default = {
   optionalDependencies: {
     "@vscode/ripgrep": "^1.17.0"
   },
-  gitHead: "4069778926e7f1c81c2473a3ac5c25f1a44fbdc2"
+  gitHead: "9161cdf044fad2628e8edfb9fc74ad34465f47c2"
 };
 
 // src/config/constants.ts
@@ -13106,39 +13056,6 @@ function createTodoStore(onUpdate) {
 }
 
 // src/tools/skillTool.ts
-import { exec } from "child_process";
-import { promisify as promisify2 } from "util";
-var execAsync = promisify2(exec);
-async function executeHook(script, context) {
-  try {
-    const env = {
-      ...process.env,
-      SKILL_NAME: context.skillName,
-      SKILL_ARGS: context.args,
-      SKILL_RESULT: context.result || "",
-      SKILL_ERROR: context.error || ""
-    };
-    const { stdout, stderr } = await execAsync(script, {
-      env,
-      timeout: 3e4,
-      // 30 second timeout for hooks
-      cwd: process.cwd()
-    });
-    return { success: true, output: stdout || stderr };
-  } catch (error) {
-    const message = error instanceof Error ? error.message : String(error);
-    return { success: false, output: `Hook failed: ${message}` };
-  }
-}
-function parseAgentConfig(agent) {
-  if (!agent) {
-    return { name: "general-purpose", thoroughness: void 0 };
-  }
-  if (typeof agent === "string") {
-    return { name: agent, thoroughness: void 0 };
-  }
-  return { name: agent.type, thoroughness: agent.thoroughness };
-}
 function parseArguments(argsString) {
   const args = [];
   let current = "";
@@ -13166,8 +13083,7 @@ function parseArguments(argsString) {
   }
   return args;
 }
-function createSkillTool(deps) {
-  const { customCommandStore } = deps;
+function createSkillTool(customCommandStore) {
   return {
     toolFn: async (args) => {
       const params = args;
@@ -13175,84 +13091,27 @@ function createSkillTool(deps) {
         throw new Error("skill: skill parameter is required");
       }
       const skillName = params.skill.replace(/^\//, "");
-      const argsString = params.args || "";
       const command = customCommandStore.getCommand(skillName);
       if (!command) {
         const available = customCommandStore.getAllCommands().map((c) => c.name).join(", ");
         throw new Error(`skill: "${skillName}" not found. Available skills: ${available || "none"}`);
       }
-      if (command.hooks?.["pre-invoke"]) {
-        const hookResult = await executeHook(command.hooks["pre-invoke"], {
-          skillName,
-          args: argsString
-        });
-        if (!hookResult.success) {
-          throw new Error(`Pre-invoke hook failed: ${hookResult.output}`);
-        }
-      }
-      try {
-        const argsArray = params.args ? parseArguments(params.args) : [];
-        let expandedBody = substituteArguments(command.body, argsArray);
-        const processed = await processFileReferences(expandedBody);
-        expandedBody = processed.content;
-        if (processed.errors.length > 0) {
-          expandedBody += `
+      const argsArray = params.args ? parseArguments(params.args) : [];
+      let expandedBody = substituteArguments(command.body, argsArray);
+      const processed = await processFileReferences(expandedBody);
+      expandedBody = processed.content;
+      if (processed.errors.length > 0) {
+        expandedBody += `
 
 **File reference errors:**
 ${processed.errors.map((e) => `- ${e}`).join("\n")}`;
-        }
-        let result;
-        if (command.context === "fork") {
-          const { subagentOrchestrator, sessionId } = deps;
-          if (!subagentOrchestrator || !sessionId) {
-            const missing = !subagentOrchestrator ? "subagentOrchestrator" : "sessionId";
-            throw new Error(`Skill "${skillName}" requires context: fork but ${missing} is not available`);
-          }
-          const agentConfig = parseAgentConfig(command.agent);
-          const agentResult = await subagentOrchestrator.delegateToAgent({
-            task: expandedBody,
-            agentName: agentConfig.name,
-            thoroughness: agentConfig.thoroughness || command.thoroughness,
-            variables: command.variables,
-            parentSessionId: sessionId
-          });
-          const agentName = agentConfig.name;
-          result = `## Skill Executed: /${skillName} (via ${agentName} agent)
-
-${agentResult.summary}`;
-        } else {
-          result = `## Skill Loaded: /${skillName}
+      }
+      return `## Skill Loaded: /${skillName}
 
 ${expandedBody}
 
 ---
 *Follow the instructions above. This skill was invoked programmatically.*`;
-        }
-        if (command.hooks?.["post-invoke"]) {
-          const hookResult = await executeHook(command.hooks["post-invoke"], {
-            skillName,
-            args: argsString,
-            result
-          });
-          if (!hookResult.success) {
-            console.warn(`Post-invoke hook warning: ${hookResult.output}`);
-          }
-        }
-        return result;
-      } catch (error) {
-        if (command.hooks?.["on-error"]) {
-          const errorMessage = error instanceof Error ? error.message : String(error);
-          const hookResult = await executeHook(command.hooks["on-error"], {
-            skillName,
-            args: argsString,
-            error: errorMessage
-          });
-          if (hookResult.output) {
-            console.warn(`On-error hook output: ${hookResult.output}`);
-          }
-        }
-        throw error;
-      }
     },
     toolSchema: {
       name: "skill",
@@ -13297,14 +13156,9 @@ ${expandedBody}
 }
 
 // src/core/skillsPrompt.ts
-function getSkillDisplayName(cmd) {
-  return cmd.displayName || cmd.name;
-}
 function formatSkillEntry(cmd) {
-  const displayName = getSkillDisplayName(cmd);
   const argHint = cmd.argumentHint ? ` ${cmd.argumentHint}` : "";
-  const nameDisplay = cmd.displayName && cmd.displayName !== cmd.name ? `**${displayName}** (\`${cmd.name}\`)` : `**${cmd.name}**`;
-  return `- ${nameDisplay}${argHint}: ${cmd.description}
+  return `- **${cmd.name}**${argHint}: ${cmd.description}
 `;
 }
 function formatSkillGroup(heading, commands) {
@@ -13315,16 +13169,12 @@ function formatSkillGroup(heading, commands) {
 ### ${heading}
 ${commands.map(formatSkillEntry).join("")}`;
 }
-function filterAIVisibleSkills(commands) {
-  return commands.filter((cmd) => !cmd.disableModelInvocation);
-}
 function buildSkillsPromptSection(commands) {
-  const visibleCommands = filterAIVisibleSkills(commands);
-  if (visibleCommands.length === 0) {
+  if (commands.length === 0) {
     return "";
   }
-  const projectSkills = visibleCommands.filter((c) => c.source === "project");
-  const globalSkills = visibleCommands.filter((c) => c.source === "global");
+  const projectSkills = commands.filter((c) => c.source === "project");
+  const globalSkills = commands.filter((c) => c.source === "global");
   return `
 
 ## Available Skills
@@ -13619,11 +13469,7 @@ function CliApp() {
       const todoStore = createTodoStore();
       const writeTodosTool = createWriteTodosTool(todoStore);
       const enableSkillTool = config.preferences.enableSkillTool !== false;
-      const skillTool = enableSkillTool ? createSkillTool({
-        customCommandStore: state.customCommandStore,
-        subagentOrchestrator: orchestrator,
-        sessionId: newSession.id
-      }) : null;
+      const skillTool = enableSkillTool ? createSkillTool(state.customCommandStore) : null;
       const cliTools = [agentDelegateTool, writeTodosTool];
       if (skillTool) {
         cliTools.push(skillTool);
@@ -14188,18 +14034,17 @@ ${output}` : output.trim() || "(no output)",
         }
         const displayMessage = `/${command}${args.length > 0 ? " " + args.join(" ") : ""}`;
         if (customCommand.agent && state.orchestrator && state.agentStore) {
-          const agentName = typeof customCommand.agent === "string" ? customCommand.agent : customCommand.agent.type;
-          if (!state.agentStore.hasAgent(agentName)) {
+          if (!state.agentStore.hasAgent(customCommand.agent)) {
             const available = state.agentStore.getAgentNames().join(", ");
-            console.error(`\u274C Unknown agent "${agentName}" specified in command.`);
+            console.error(`\u274C Unknown agent "${customCommand.agent}" specified in command.`);
             console.error(`   Available agents: ${available}`);
             return;
           }
-          console.log(`\u{1F916} Delegating to ${agentName} agent...
+          console.log(`\u{1F916} Delegating to ${customCommand.agent} agent...
 `);
           const result = await state.orchestrator.delegateToAgent({
             task: substitutedBody,
-            agentName,
+            agentName: customCommand.agent,
             thoroughness: customCommand.thoroughness,
             variables: customCommand.variables,
             parentSessionId: state.session?.id || "unknown"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bike4mind/cli",
-  "version": "0.2.21-feat-cli-skill-frontmatter-spec.18205+406977892",
+  "version": "0.2.21-feat-github-actions-tool.18208+9161cdf04",
   "type": "module",
   "description": "Interactive CLI tool for Bike4Mind with ReAct agents",
   "license": "UNLICENSED",
@@ -107,10 +107,10 @@
   },
   "devDependencies": {
     "@bike4mind/agents": "0.1.0",
-    "@bike4mind/common": "2.45.1-feat-cli-skill-frontmatter-spec.18205+406977892",
-    "@bike4mind/mcp": "1.25.1-feat-cli-skill-frontmatter-spec.18205+406977892",
-    "@bike4mind/services": "2.43.1-feat-cli-skill-frontmatter-spec.18205+406977892",
-    "@bike4mind/utils": "2.3.1-feat-cli-skill-frontmatter-spec.18205+406977892",
+    "@bike4mind/common": "2.45.1-feat-github-actions-tool.18208+9161cdf04",
+    "@bike4mind/mcp": "1.25.1-feat-github-actions-tool.18208+9161cdf04",
+    "@bike4mind/services": "2.43.1-feat-github-actions-tool.18208+9161cdf04",
+    "@bike4mind/utils": "2.3.1-feat-github-actions-tool.18208+9161cdf04",
     "@types/better-sqlite3": "^7.6.13",
     "@types/diff": "^5.0.9",
     "@types/jsonwebtoken": "^9.0.4",
@@ -127,5 +127,5 @@
   "optionalDependencies": {
     "@vscode/ripgrep": "^1.17.0"
   },
-  "gitHead": "4069778926e7f1c81c2473a3ac5c25f1a44fbdc2"
+  "gitHead": "9161cdf044fad2628e8edfb9fc74ad34465f47c2"
 }