@fro.bot/systematic 2.33.0 → 2.33.1

package/dist/cli.js

@@ -6,7 +6,7 @@ import {
   findCommandsInDir,
   findSkillsInDir,
   getConfigPaths
-} from "./index-bwrrk4a8.js";
+} from "./index-vyzhzvap.js";
 
 // src/cli.ts
 import fs from "fs";

package/dist/{index-bwrrk4a8.js → index-vyzhzvap.js}

@@ -515,32 +515,35 @@ function parseDeprecated(data) {
   }
   return deprecated;
 }
+function extractFrontmatterFromContent(content) {
+  const { data, parseError } = parseFrontmatter(content);
+  if (parseError) {
+    return { name: "", description: "" };
+  }
+  const metadata = parseMetadata(data);
+  const deprecated = parseDeprecated(data);
+  const argumentHintRaw = extractNonEmptyString(data, "argument-hint");
+  const argumentHint = argumentHintRaw?.replace(/^["']|["']$/g, "") || undefined;
+  return {
+    name: extractString(data, "name"),
+    description: extractString(data, "description"),
+    license: extractNonEmptyString(data, "license"),
+    compatibility: extractNonEmptyString(data, "compatibility"),
+    metadata,
+    deprecated,
+    disableModelInvocation: extractBoolean(data, "disable-model-invocation"),
+    userInvocable: extractBoolean(data, "user-invocable"),
+    subtask: data.context === "fork" ? true : extractBoolean(data, "subtask") ?? undefined,
+    agent: extractNonEmptyString(data, "agent"),
+    model: extractNonEmptyString(data, "model"),
+    argumentHint: argumentHint !== "" ? argumentHint : undefined,
+    allowedTools: extractNonEmptyString(data, "allowed-tools")
+  };
+}
 function extractFrontmatter(filePath) {
   try {
     const content = fs3.readFileSync(filePath, "utf8");
-    const { data, parseError } = parseFrontmatter(content);
-    if (parseError) {
-      return { name: "", description: "" };
-    }
-    const metadata = parseMetadata(data);
-    const deprecated = parseDeprecated(data);
-    const argumentHintRaw = extractNonEmptyString(data, "argument-hint");
-    const argumentHint = argumentHintRaw?.replace(/^["']|["']$/g, "") || undefined;
-    return {
-      name: extractString(data, "name"),
-      description: extractString(data, "description"),
-      license: extractNonEmptyString(data, "license"),
-      compatibility: extractNonEmptyString(data, "compatibility"),
-      metadata,
-      deprecated,
-      disableModelInvocation: extractBoolean(data, "disable-model-invocation"),
-      userInvocable: extractBoolean(data, "user-invocable"),
-      subtask: data.context === "fork" ? true : extractBoolean(data, "subtask") ?? undefined,
-      agent: extractNonEmptyString(data, "agent"),
-      model: extractNonEmptyString(data, "model"),
-      argumentHint: argumentHint !== "" ? argumentHint : undefined,
-      allowedTools: extractNonEmptyString(data, "allowed-tools")
-    };
+    return extractFrontmatterFromContent(content);
   } catch {
     return { name: "", description: "" };
   }
@@ -16422,4 +16425,4 @@ function extractCommandFrontmatter(content) {
   };
 }
 
-export { parseFrontmatter, isRecord, convertContent, convertFileWithCache, walkDir, extractFrontmatter, findSkillsInDir, exports_external, AgentOverlaySchema, CategoryOverlaySchema, loadConfig, loadConfigWithSources, getConfigPaths, findAgentsInDir, extractAgentFrontmatter, findCommandsInDir, extractCommandFrontmatter };
+export { parseFrontmatter, isRecord, convertContent, convertFileWithCache, walkDir, extractFrontmatterFromContent, findSkillsInDir, exports_external, AgentOverlaySchema, CategoryOverlaySchema, loadConfig, loadConfigWithSources, getConfigPaths, findAgentsInDir, extractAgentFrontmatter, findCommandsInDir, extractCommandFrontmatter };

package/dist/index.js

@@ -6,7 +6,7 @@ import {
   exports_external,
   extractAgentFrontmatter,
   extractCommandFrontmatter,
-  extractFrontmatter,
+  extractFrontmatterFromContent,
   findAgentsInDir,
   findCommandsInDir,
   findSkillsInDir,
@@ -15,10 +15,10 @@ import {
   loadConfigWithSources,
   parseFrontmatter,
   walkDir
-} from "./index-bwrrk4a8.js";
+} from "./index-vyzhzvap.js";
 
 // src/index.ts
-import fs7 from "fs";
+import fs6 from "fs";
 import path9 from "path";
 import { fileURLToPath as fileURLToPath2 } from "url";
 
@@ -240,7 +240,6 @@ ${skillUsage}${catalogSection}
 }
 
 // src/lib/config-handler.ts
-import fs5 from "fs";
 import os3 from "os";
 import path7 from "path";
 
@@ -707,15 +706,13 @@ function globSkillFiles(rootDir, subdirNames) {
   return results;
 }
 function toDiscoveredSkill(skillPath, rootId) {
-  let stat;
+  let content;
   try {
-    stat = fs3.statSync(skillPath);
+    content = fs3.readFileSync(skillPath, "utf8");
   } catch {
     return;
   }
-  if (!stat.isFile())
-    return;
-  const frontmatter = extractFrontmatter(skillPath);
+  const frontmatter = extractFrontmatterFromContent(content);
   const name = frontmatter.name;
   if (!name || !isValidSkillName(name))
     return;
@@ -723,6 +720,7 @@ function toDiscoveredSkill(skillPath, rootId) {
     name,
     description: frontmatter.description,
     frontmatter,
+    body: parseFrontmatter(content).body,
     skillPath,
     root: rootId
   };
@@ -1226,10 +1224,8 @@ function collectSkillsAsCommands(dir, disabledSkills) {
 function loadDiscoveredSkillAsCommand(skill) {
   const description = formatSkillDescription(skill.description, skill.name);
   if (skill.frontmatter.disableModelInvocation === true) {
-    const content = fs5.readFileSync(skill.skillPath, "utf8");
-    const { body } = parseFrontmatter(content);
     return {
-      template: wrapSkillTemplate(skill.skillPath, body),
+      template: wrapSkillTemplate(skill.skillPath, skill.body),
       description
     };
   }
@@ -1245,7 +1241,7 @@ function buildDiscoveredSkillShimTemplate(skillName) {
 $ARGUMENTS
 </user-request>`;
 }
-function collectDiscoveredSkillsAsCommands(startDir, homeDir, configDir, opencodeConfigDirOverride) {
+function collectDiscoveredSkillsAsCommands(startDir, homeDir, configDir, opencodeConfigDirOverride, disabledCommands) {
   const commands = {};
   let discovered;
   try {
@@ -1261,6 +1257,8 @@ function collectDiscoveredSkillsAsCommands(startDir, homeDir, configDir, opencod
   for (const skill of discovered) {
     if (skill.frontmatter.userInvocable === false)
       continue;
+    if (disabledCommands.includes(skill.name))
+      continue;
     try {
       commands[skill.name] = loadDiscoveredSkillAsCommand(skill);
     } catch {}
@@ -1296,7 +1294,7 @@ function createConfigHandler(deps) {
     const resolvedOverlays = resolveAgentOverlaySet(validatedOverlays);
     const bundledAgents = collectAgents(bundledAgentsDir2, systematicConfig.disabled_agents, nativeAgents, resolvedOverlays, availabilitySet);
     const bundledCommands = collectCommands(bundledCommandsDir, systematicConfig.disabled_commands);
-    const discoveredSkillCommands = systematicConfig.skills_as_commands !== false ? collectDiscoveredSkillsAsCommands(directory, homeDir, opencodeConfigDir, opencodeConfigDirOverride) : {};
+    const discoveredSkillCommands = systematicConfig.skills_as_commands !== false ? collectDiscoveredSkillsAsCommands(directory, homeDir, opencodeConfigDir, opencodeConfigDirOverride, systematicConfig.disabled_commands) : {};
     const bundledAgentKeys = new Set(Object.keys(bundledAgents));
     config.agent = mergeSystematicEntries(existingAgents, bundledAgents, (key, agent) => bundledAgentKeys.has(key) && isSystematicAgentConfig(agent));
     const emittedCommands = {
@@ -1331,7 +1329,7 @@ function normalizePath(path8) {
 }
 
 // src/lib/skill-tool.ts
-import fs6 from "fs";
+import fs5 from "fs";
 import path8 from "path";
 import { pathToFileURL as pathToFileURL2 } from "url";
 import { tool } from "@opencode-ai/plugin/tool";
@@ -1356,7 +1354,7 @@ function discoverSkillFiles(dir, limit = 10) {
     if (files.length >= limit)
       return;
     try {
-      const entries = fs6.readdirSync(currentDir, { withFileTypes: true });
+      const entries = fs5.readdirSync(currentDir, { withFileTypes: true });
       for (const entry of entries) {
         if (files.length >= limit)
           break;
@@ -1487,9 +1485,9 @@ var bundledCommandsDir = path9.join(packageRoot2, "commands");
 var packageJsonPath = path9.join(packageRoot2, "package.json");
 var getPackageVersion = () => {
   try {
-    if (!fs7.existsSync(packageJsonPath))
+    if (!fs6.existsSync(packageJsonPath))
       return "unknown";
-    const content = fs7.readFileSync(packageJsonPath, "utf8");
+    const content = fs6.readFileSync(packageJsonPath, "utf8");
     const parsed = JSON.parse(content);
     return parsed.version ?? "unknown";
   } catch {

package/dist/lib/discovered-skills.d.ts

@@ -15,6 +15,9 @@ export interface DiscoveredSkill {
     name: string;
     description: string;
     frontmatter: SkillFrontmatter;
+    /** SKILL.md body (frontmatter stripped), read once at discovery so callers
+     * that inline the body don't re-read the file. */
+    body: string;
     skillPath: string;
     root: DiscoveryRootId;
 }

package/dist/lib/skills.d.ts

@@ -37,5 +37,11 @@ export interface SkillInfo {
     allowedTools?: string;
 }
 export declare const SKILL_FRONTMATTER_FIELDS: readonly ["name", "description", "argument-hint", "disable-model-invocation", "allowed-tools", "license", "compatibility", "metadata", "deprecated", "user-invocable", "agent", "model", "context", "subtask"];
+/**
+ * Parse skill frontmatter from already-read file content. Split out from
+ * `extractFrontmatter` so callers that also need the body (e.g. discovered-skill
+ * command emission) can read the file once and derive both. Never throws.
+ */
+export declare function extractFrontmatterFromContent(content: string): SkillFrontmatter;
 export declare function extractFrontmatter(filePath: string): SkillFrontmatter;
 export declare function findSkillsInDir(dir: string, maxDepth?: number): SkillInfo[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fro.bot/systematic",
-  "version": "2.33.0",
+  "version": "2.33.1",
   "description": "Structured engineering workflows for OpenCode",
   "type": "module",
   "homepage": "https://fro.bot/systematic",