@skillsmith/core 0.12.1 → 0.12.2

package/dist/src/services/SubagentGenerator.helpers.d.ts

@@ -0,0 +1,64 @@
+/**
+ * SMI-1788: Pure helper functions for SubagentGenerator — tool detection,
+ * trigger-phrase extraction, model selection, and Claude-tool-named
+ * guidance text. Split out under the 500-line standard (SMI-6276) following
+ * this directory's existing `.helpers.ts` convention (see
+ * SkillAnalyzer.helpers.ts / SkillDecomposer.helpers.ts).
+ */
+import type { ToolUsageAnalysis } from './SkillAnalyzer.js';
+import { type ClaudeModel } from './SubagentGenerator.types.js';
+/**
+ * Tool detection patterns for analyzing skill content
+ */
+export declare const TOOL_PATTERNS: Record<string, {
+    patterns: string[];
+    priority: number;
+}>;
+/**
+ * Minimum tools that most subagents need
+ */
+export declare const BASE_TOOLS: string[];
+/**
+ * SMI-1819: Maximum length for subagent names to prevent overly long identifiers
+ */
+export declare const MAX_SUBAGENT_NAME_LENGTH = 100;
+/**
+ * Generate tool usage guidelines for a subagent (Claude-named tools).
+ * Only used for clients whose `toolsPolicy` is `'claude-native'` — see
+ * `SubagentGenerator.client-profiles.ts` and `OMITTED_TOOLS_GUIDANCE`.
+ */
+export declare function generateToolGuidelines(tools: string[]): string;
+/**
+ * SMI-6276: fallback tool-usage guidance for any client whose companion
+ * subagent file does not carry Claude-named tools in its `tools:`
+ * frontmatter — either because the field is omitted entirely, or because it
+ * was translated into that client's OWN vocabulary (`mapped-array`, e.g.
+ * AntiGravity's `view_file`/`run_command`). Naming Claude-specific tool
+ * identifiers in this prose body would be misleading either way, so this
+ * stays generic. See `SubagentGenerator.client-profiles.ts`.
+ */
+export declare const OMITTED_TOOLS_GUIDANCE = "- Use the tools available on this platform minimally and efficiently";
+/**
+ * Generate CLAUDE.md integration snippet
+ *
+ * NOTE (SMI-6276): deliberately NOT client-parameterized — CLAUDE.md is
+ * inherently a Claude Code artifact regardless of which client the
+ * companion-agent FILE itself targets, so this always describes the
+ * Claude-side delegation recommendation. Out of this wave's scope; see
+ * `SubagentGenerator.client-profiles.ts`'s header for what IS in scope.
+ */
+export declare function generateClaudeMdSnippet(skillName: string, description: string, triggerPhrases: string[], tools: string[], model: ClaudeModel): string;
+/**
+ * Extract trigger phrases from skill description and content
+ */
+export declare function extractTriggerPhrases(description: string, content: string): string[];
+/**
+ * Determine optimal model for subagent
+ * SMI-1795: Uses CLAUDE_MODELS constants for type safety
+ */
+export declare function determineModel(toolUsage: ToolUsageAnalysis, lineCount: number): ClaudeModel;
+/**
+ * Detect tools needed from skill content
+ */
+export declare function detectTools(content: string): string[];
+//# sourceMappingURL=SubagentGenerator.helpers.d.ts.map

package/dist/src/services/SubagentGenerator.helpers.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"SubagentGenerator.helpers.d.ts","sourceRoot":"","sources":["../../../src/services/SubagentGenerator.helpers.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAEH,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,oBAAoB,CAAA;AAC3D,OAAO,EAAiB,KAAK,WAAW,EAAE,MAAM,8BAA8B,CAAA;AAE9E;;GAEG;AACH,eAAO,MAAM,aAAa,EAAE,MAAM,CAAC,MAAM,EAAE;IAAE,QAAQ,EAAE,MAAM,EAAE,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,CAoDlF,CAAA;AAED;;GAEG;AACH,eAAO,MAAM,UAAU,UAAW,CAAA;AAElC;;GAEG;AACH,eAAO,MAAM,wBAAwB,MAAM,CAAA;AAE3C;;;;GAIG;AACH,wBAAgB,sBAAsB,CAAC,KAAK,EAAE,MAAM,EAAE,GAAG,MAAM,CA6B9D;AAED;;;;;;;;GAQG;AACH,eAAO,MAAM,sBAAsB,yEACqC,CAAA;AAExE;;;;;;;;GAQG;AACH,wBAAgB,uBAAuB,CACrC,SAAS,EAAE,MAAM,EACjB,WAAW,EAAE,MAAM,EACnB,cAAc,EAAE,MAAM,EAAE,EACxB,KAAK,EAAE,MAAM,EAAE,EACf,KAAK,EAAE,WAAW,GACjB,MAAM,CA0BR;AAED;;GAEG;AACH,wBAAgB,qBAAqB,CAAC,WAAW,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,CAwDpF;AAED;;;GAGG;AACH,wBAAgB,cAAc,CAAC,SAAS,EAAE,iBAAiB,EAAE,SAAS,EAAE,MAAM,GAAG,WAAW,CAgB3F;AAED;;GAEG;AACH,wBAAgB,WAAW,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,EAAE,CAcrD"}

package/dist/src/services/SubagentGenerator.helpers.js

@@ -0,0 +1,228 @@
+/**
+ * SMI-1788: Pure helper functions for SubagentGenerator — tool detection,
+ * trigger-phrase extraction, model selection, and Claude-tool-named
+ * guidance text. Split out under the 500-line standard (SMI-6276) following
+ * this directory's existing `.helpers.ts` convention (see
+ * SkillAnalyzer.helpers.ts / SkillDecomposer.helpers.ts).
+ */
+import { CLAUDE_MODELS } from './SubagentGenerator.types.js';
+/**
+ * Tool detection patterns for analyzing skill content
+ */
+export const TOOL_PATTERNS = {
+    Read: {
+        patterns: ['read file', 'read the', 'examine', 'view file', 'Read tool', 'check file'],
+        priority: 1,
+    },
+    Write: {
+        patterns: ['write file', 'create file', 'save to', 'output to', 'Write tool', 'generate file'],
+        priority: 2,
+    },
+    Edit: {
+        patterns: [
+            'edit file',
+            'modify',
+            'update file',
+            'patch',
+            'Edit tool',
+            'change file',
+            'refactor',
+        ],
+        priority: 2,
+    },
+    Bash: {
+        patterns: [
+            'bash',
+            'npm',
+            'npx',
+            'git',
+            'docker',
+            'yarn',
+            'pnpm',
+            'terminal',
+            'shell',
+            'command',
+        ],
+        priority: 3,
+    },
+    Grep: {
+        patterns: ['grep', 'search for', 'find text', 'pattern match', 'Grep tool', 'search in'],
+        priority: 1,
+    },
+    Glob: {
+        patterns: ['glob', 'find file', 'file pattern', 'list files', 'Glob tool', 'locate'],
+        priority: 1,
+    },
+    WebFetch: {
+        patterns: ['fetch', 'http', 'api call', 'url', 'WebFetch', 'download', 'request'],
+        priority: 2,
+    },
+    WebSearch: {
+        patterns: ['web search', 'search online', 'lookup online', 'WebSearch', 'search the web'],
+        priority: 2,
+    },
+};
+/**
+ * Minimum tools that most subagents need
+ */
+export const BASE_TOOLS = ['Read'];
+/**
+ * SMI-1819: Maximum length for subagent names to prevent overly long identifiers
+ */
+export const MAX_SUBAGENT_NAME_LENGTH = 100;
+/**
+ * Generate tool usage guidelines for a subagent (Claude-named tools).
+ * Only used for clients whose `toolsPolicy` is `'claude-native'` — see
+ * `SubagentGenerator.client-profiles.ts` and `OMITTED_TOOLS_GUIDANCE`.
+ */
+export function generateToolGuidelines(tools) {
+    const guidelines = [];
+    if (tools.includes('Read')) {
+        guidelines.push('- **Read**: Use to examine files before modifications');
+    }
+    if (tools.includes('Write')) {
+        guidelines.push('- **Write**: Use for creating new files only');
+    }
+    if (tools.includes('Edit')) {
+        guidelines.push('- **Edit**: Use for modifying existing files');
+    }
+    if (tools.includes('Bash')) {
+        guidelines.push('- **Bash**: Use for command execution, prefer non-destructive commands');
+    }
+    if (tools.includes('Grep')) {
+        guidelines.push('- **Grep**: Use for searching file contents');
+    }
+    if (tools.includes('Glob')) {
+        guidelines.push('- **Glob**: Use for finding files by pattern');
+    }
+    if (tools.includes('WebFetch')) {
+        guidelines.push('- **WebFetch**: Use for fetching web content');
+    }
+    if (tools.includes('WebSearch')) {
+        guidelines.push('- **WebSearch**: Use for searching the web');
+    }
+    return guidelines.length > 0 ? guidelines.join('\n') : '- Use tools minimally and efficiently';
+}
+/**
+ * SMI-6276: fallback tool-usage guidance for any client whose companion
+ * subagent file does not carry Claude-named tools in its `tools:`
+ * frontmatter — either because the field is omitted entirely, or because it
+ * was translated into that client's OWN vocabulary (`mapped-array`, e.g.
+ * AntiGravity's `view_file`/`run_command`). Naming Claude-specific tool
+ * identifiers in this prose body would be misleading either way, so this
+ * stays generic. See `SubagentGenerator.client-profiles.ts`.
+ */
+export const OMITTED_TOOLS_GUIDANCE = '- Use the tools available on this platform minimally and efficiently';
+/**
+ * Generate CLAUDE.md integration snippet
+ *
+ * NOTE (SMI-6276): deliberately NOT client-parameterized — CLAUDE.md is
+ * inherently a Claude Code artifact regardless of which client the
+ * companion-agent FILE itself targets, so this always describes the
+ * Claude-side delegation recommendation. Out of this wave's scope; see
+ * `SubagentGenerator.client-profiles.ts`'s header for what IS in scope.
+ */
+export function generateClaudeMdSnippet(skillName, description, triggerPhrases, tools, model) {
+    const triggerPatterns = triggerPhrases.length > 0
+        ? triggerPhrases.map((p) => `- "${p}"`).join('\n')
+        : '- [add trigger patterns]';
+    const exampleTask = triggerPhrases.length > 0 ? triggerPhrases[0] : `execute ${skillName} task`;
+    return `
+### Subagent Delegation: ${skillName}
+
+When tasks match ${skillName} triggers, delegate to the ${skillName}-specialist
+subagent instead of executing directly. This provides context isolation and
+~37-97% token savings.
+
+**Trigger Patterns:**
+${triggerPatterns}
+
+**Delegation Example:**
+\`\`\`
+Task("${skillName}-specialist", "${exampleTask}", "${skillName}-specialist")
+\`\`\`
+
+**Model:** ${model}
+**Tools:** ${tools.join(', ')}
+`;
+}
+/**
+ * Extract trigger phrases from skill description and content
+ */
+export function extractTriggerPhrases(description, content) {
+    const phrases = [];
+    // Common trigger phrase patterns. CodeQL finding (SMI-6276, polynomial
+    // regex / ReDoS): the original patterns paired an optional word group
+    // (e.g. `(?:you\s+)?`) directly against a lazy wildcard capture
+    // (`(.+?)`) — since `.` also matches the letters/spaces the optional
+    // group matches, the engine has exponentially many ways to split
+    // ambiguous input (long runs of repeated words/spaces) between "the
+    // optional group matched" and "the wildcard consumed it instead".
+    // `textToSearch` includes up to 2000 chars of externally-authored
+    // SKILL.md content (a real untrusted-input path in a skill registry).
+    // Fix: bound each capture group to a small fixed max length instead of
+    // an unbounded lazy wildcard — extracted phrases are truncated to <=50
+    // chars a few lines below regardless, so this changes no real-world
+    // behavior, but caps worst-case backtracking work to a small constant
+    // instead of growing with input length.
+    const MAX_CANDIDATE_PHRASE_LENGTH = 60;
+    const patterns = [
+        new RegExp(`when\\s+(?:you\\s+)?(?:need\\s+to\\s+)?([^.,]{1,${MAX_CANDIDATE_PHRASE_LENGTH}})(?:[.,]|$)`, 'gi'),
+        new RegExp(`use\\s+(?:this\\s+)?(?:when|for)\\s+([^.,]{1,${MAX_CANDIDATE_PHRASE_LENGTH}})(?:[.,]|$)`, 'gi'),
+        new RegExp(`(?:helps?\\s+(?:you\\s+)?(?:to\\s+)?)?([^\\n]{1,${MAX_CANDIDATE_PHRASE_LENGTH}}?)(?:\\s+tasks?|\\s+operations?)`, 'gi'),
+    ];
+    const textToSearch = description + ' ' + content.slice(0, 2000); // Search first 2000 chars
+    for (const pattern of patterns) {
+        for (const match of textToSearch.matchAll(pattern)) {
+            const phrase = match[1].trim().toLowerCase();
+            if (phrase.length >= 5 && phrase.length <= 50 && !phrases.includes(phrase)) {
+                phrases.push(phrase);
+            }
+        }
+    }
+    // Extract from "trigger" or "invoke" mentions
+    const triggerMatch = content.match(/trigger[s]?\s*[:=]\s*["`']([^"`']+)["`']/gi);
+    if (triggerMatch) {
+        for (const match of triggerMatch) {
+            const phrase = match.replace(/trigger[s]?\s*[:=]\s*["`']/i, '').replace(/["`']$/, '');
+            if (!phrases.includes(phrase.toLowerCase())) {
+                phrases.push(phrase.toLowerCase());
+            }
+        }
+    }
+    return phrases.slice(0, 5); // Limit to 5 phrases
+}
+/**
+ * Determine optimal model for subagent
+ * SMI-1795: Uses CLAUDE_MODELS constants for type safety
+ */
+export function determineModel(toolUsage, lineCount) {
+    // Haiku for simple, fast operations
+    if (toolUsage.detectedTools.length <= 2 && lineCount < 200) {
+        return CLAUDE_MODELS.HAIKU;
+    }
+    // Opus for complex, reasoning-heavy tasks
+    if (toolUsage.bashCommandCount > 5 ||
+        (toolUsage.fileReadCount > 3 && toolUsage.fileWriteCount > 3)) {
+        return CLAUDE_MODELS.OPUS;
+    }
+    // Sonnet for balanced workloads (default)
+    return CLAUDE_MODELS.SONNET;
+}
+/**
+ * Detect tools needed from skill content
+ */
+export function detectTools(content) {
+    const lowerContent = content.toLowerCase();
+    const detectedTools = new Set(BASE_TOOLS);
+    for (const [tool, config] of Object.entries(TOOL_PATTERNS)) {
+        for (const pattern of config.patterns) {
+            if (lowerContent.includes(pattern.toLowerCase())) {
+                detectedTools.add(tool);
+                break;
+            }
+        }
+    }
+    return Array.from(detectedTools);
+}
+//# sourceMappingURL=SubagentGenerator.helpers.js.map

package/dist/src/services/SubagentGenerator.helpers.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"SubagentGenerator.helpers.js","sourceRoot":"","sources":["../../../src/services/SubagentGenerator.helpers.ts"],"names":[],"mappings":"AAAA;;;;;;GAMG;AAGH,OAAO,EAAE,aAAa,EAAoB,MAAM,8BAA8B,CAAA;AAE9E;;GAEG;AACH,MAAM,CAAC,MAAM,aAAa,GAA6D;IACrF,IAAI,EAAE;QACJ,QAAQ,EAAE,CAAC,WAAW,EAAE,UAAU,EAAE,SAAS,EAAE,WAAW,EAAE,WAAW,EAAE,YAAY,CAAC;QACtF,QAAQ,EAAE,CAAC;KACZ;IACD,KAAK,EAAE;QACL,QAAQ,EAAE,CAAC,YAAY,EAAE,aAAa,EAAE,SAAS,EAAE,WAAW,EAAE,YAAY,EAAE,eAAe,CAAC;QAC9F,QAAQ,EAAE,CAAC;KACZ;IACD,IAAI,EAAE;QACJ,QAAQ,EAAE;YACR,WAAW;YACX,QAAQ;YACR,aAAa;YACb,OAAO;YACP,WAAW;YACX,aAAa;YACb,UAAU;SACX;QACD,QAAQ,EAAE,CAAC;KACZ;IACD,IAAI,EAAE;QACJ,QAAQ,EAAE;YACR,MAAM;YACN,KAAK;YACL,KAAK;YACL,KAAK;YACL,QAAQ;YACR,MAAM;YACN,MAAM;YACN,UAAU;YACV,OAAO;YACP,SAAS;SACV;QACD,QAAQ,EAAE,CAAC;KACZ;IACD,IAAI,EAAE;QACJ,QAAQ,EAAE,CAAC,MAAM,EAAE,YAAY,EAAE,WAAW,EAAE,eAAe,EAAE,WAAW,EAAE,WAAW,CAAC;QACxF,QAAQ,EAAE,CAAC;KACZ;IACD,IAAI,EAAE;QACJ,QAAQ,EAAE,CAAC,MAAM,EAAE,WAAW,EAAE,cAAc,EAAE,YAAY,EAAE,WAAW,EAAE,QAAQ,CAAC;QACpF,QAAQ,EAAE,CAAC;KACZ;IACD,QAAQ,EAAE;QACR,QAAQ,EAAE,CAAC,OAAO,EAAE,MAAM,EAAE,UAAU,EAAE,KAAK,EAAE,UAAU,EAAE,UAAU,EAAE,SAAS,CAAC;QACjF,QAAQ,EAAE,CAAC;KACZ;IACD,SAAS,EAAE;QACT,QAAQ,EAAE,CAAC,YAAY,EAAE,eAAe,EAAE,eAAe,EAAE,WAAW,EAAE,gBAAgB,CAAC;QACzF,QAAQ,EAAE,CAAC;KACZ;CACF,CAAA;AAED;;GAEG;AACH,MAAM,CAAC,MAAM,UAAU,GAAG,CAAC,MAAM,CAAC,CAAA;AAElC;;GAEG;AACH,MAAM,CAAC,MAAM,wBAAwB,GAAG,GAAG,CAAA;AAE3C;;;;GAIG;AACH,MAAM,UAAU,sBAAsB,CAAC,KAAe;IACpD,MAAM,UAAU,GAAa,EAAE,CAAA;IAE/B,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;QAC3B,UAAU,CAAC,IAAI,CAAC,uDAAuD,CAAC,CAAA;IAC1E,CAAC;IACD,IAAI,KAAK,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,CAAC;QAC5B,UAAU,CAAC,IAAI,CAAC,8CAA8C,CAAC,CAAA;IACjE,CAAC;IACD,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;QAC3B,UAAU,CAAC,IAAI,CAAC,8CAA8C,CAAC,CAAA;IACjE,CAAC;IACD,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;QAC3B,UAAU,CAAC,IAAI,CAAC,wEAAwE,CAAC,CAAA;IAC3F,CAAC;IACD,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;QAC3B,UAAU,CAAC,IAAI,CAAC,6CAA6C,CAAC,CAAA;IAChE,CAAC;IACD,IAAI,KAAK,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;QAC3B,UAAU,CAAC,IAAI,CAAC,8CAA8C,CAAC,CAAA;IACjE,CAAC;IACD,IAAI,KAAK,CAAC,QAAQ,CAAC,UAAU,CAAC,EAAE,CAAC;QAC/B,UAAU,CAAC,IAAI,CAAC,8CAA8C,CAAC,CAAA;IACjE,CAAC;IACD,IAAI,KAAK,CAAC,QAAQ,CAAC,WAAW,CAAC,EAAE,CAAC;QAChC,UAAU,CAAC,IAAI,CAAC,4CAA4C,CAAC,CAAA;IAC/D,CAAC;IAED,OAAO,UAAU,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,uCAAuC,CAAA;AAChG,CAAC;AAED;;;;;;;;GAQG;AACH,MAAM,CAAC,MAAM,sBAAsB,GACjC,sEAAsE,CAAA;AAExE;;;;;;;;GAQG;AACH,MAAM,UAAU,uBAAuB,CACrC,SAAiB,EACjB,WAAmB,EACnB,cAAwB,EACxB,KAAe,EACf,KAAkB;IAElB,MAAM,eAAe,GACnB,cAAc,CAAC,MAAM,GAAG,CAAC;QACvB,CAAC,CAAC,cAAc,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC;QAClD,CAAC,CAAC,0BAA0B,CAAA;IAEhC,MAAM,WAAW,GAAG,cAAc,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,cAAc,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,WAAW,SAAS,OAAO,CAAA;IAE/F,OAAO;2BACkB,SAAS;;mBAEjB,SAAS,8BAA8B,SAAS;;;;;EAKjE,eAAe;;;;QAIT,SAAS,kBAAkB,WAAW,OAAO,SAAS;;;aAGjD,KAAK;aACL,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC;CAC5B,CAAA;AACD,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,qBAAqB,CAAC,WAAmB,EAAE,OAAe;IACxE,MAAM,OAAO,GAAa,EAAE,CAAA;IAE5B,uEAAuE;IACvE,sEAAsE;IACtE,gEAAgE;IAChE,qEAAqE;IACrE,iEAAiE;IACjE,oEAAoE;IACpE,kEAAkE;IAClE,kEAAkE;IAClE,sEAAsE;IACtE,uEAAuE;IACvE,uEAAuE;IACvE,oEAAoE;IACpE,sEAAsE;IACtE,wCAAwC;IACxC,MAAM,2BAA2B,GAAG,EAAE,CAAA;IACtC,MAAM,QAAQ,GAAG;QACf,IAAI,MAAM,CACR,mDAAmD,2BAA2B,cAAc,EAC5F,IAAI,CACL;QACD,IAAI,MAAM,CACR,gDAAgD,2BAA2B,cAAc,EACzF,IAAI,CACL;QACD,IAAI,MAAM,CACR,mDAAmD,2BAA2B,mCAAmC,EACjH,IAAI,CACL;KACF,CAAA;IAED,MAAM,YAAY,GAAG,WAAW,GAAG,GAAG,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,IAAI,CAAC,CAAA,CAAC,0BAA0B;IAE1F,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;QAC/B,KAAK,MAAM,KAAK,IAAI,YAAY,CAAC,QAAQ,CAAC,OAAO,CAAC,EAAE,CAAC;YACnD,MAAM,MAAM,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAA;YAC5C,IAAI,MAAM,CAAC,MAAM,IAAI,CAAC,IAAI,MAAM,CAAC,MAAM,IAAI,EAAE,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;gBAC3E,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,CAAA;YACtB,CAAC;QACH,CAAC;IACH,CAAC;IAED,8CAA8C;IAC9C,MAAM,YAAY,GAAG,OAAO,CAAC,KAAK,CAAC,4CAA4C,CAAC,CAAA;IAChF,IAAI,YAAY,EAAE,CAAC;QACjB,KAAK,MAAM,KAAK,IAAI,YAAY,EAAE,CAAC;YACjC,MAAM,MAAM,GAAG,KAAK,CAAC,OAAO,CAAC,6BAA6B,EAAE,EAAE,CAAC,CAAC,OAAO,CAAC,QAAQ,EAAE,EAAE,CAAC,CAAA;YACrF,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC,EAAE,CAAC;gBAC5C,OAAO,CAAC,IAAI,CAAC,MAAM,CAAC,WAAW,EAAE,CAAC,CAAA;YACpC,CAAC;QACH,CAAC;IACH,CAAC;IAED,OAAO,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAA,CAAC,qBAAqB;AAClD,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,cAAc,CAAC,SAA4B,EAAE,SAAiB;IAC5E,oCAAoC;IACpC,IAAI,SAAS,CAAC,aAAa,CAAC,MAAM,IAAI,CAAC,IAAI,SAAS,GAAG,GAAG,EAAE,CAAC;QAC3D,OAAO,aAAa,CAAC,KAAK,CAAA;IAC5B,CAAC;IAED,0CAA0C;IAC1C,IACE,SAAS,CAAC,gBAAgB,GAAG,CAAC;QAC9B,CAAC,SAAS,CAAC,aAAa,GAAG,CAAC,IAAI,SAAS,CAAC,cAAc,GAAG,CAAC,CAAC,EAC7D,CAAC;QACD,OAAO,aAAa,CAAC,IAAI,CAAA;IAC3B,CAAC;IAED,0CAA0C;IAC1C,OAAO,aAAa,CAAC,MAAM,CAAA;AAC7B,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,WAAW,CAAC,OAAe;IACzC,MAAM,YAAY,GAAG,OAAO,CAAC,WAAW,EAAE,CAAA;IAC1C,MAAM,aAAa,GAAG,IAAI,GAAG,CAAS,UAAU,CAAC,CAAA;IAEjD,KAAK,MAAM,CAAC,IAAI,EAAE,MAAM,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,aAAa,CAAC,EAAE,CAAC;QAC3D,KAAK,MAAM,OAAO,IAAI,MAAM,CAAC,QAAQ,EAAE,CAAC;YACtC,IAAI,YAAY,CAAC,QAAQ,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,EAAE,CAAC;gBACjD,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC,CAAA;gBACvB,MAAK;YACP,CAAC;QACH,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,aAAa,CAAC,CAAA;AAClC,CAAC"}

package/dist/src/services/SubagentGenerator.js

@@ -9,125 +9,69 @@
  *
  * Part of the Skillsmith Optimization Layer for transforming
  * community skills into more performant versions.
+ *
+ * SMI-6276 (Wave 6 Step 1): companion-agent frontmatter is now generated
+ * per TARGET CLIENT via `SubagentGenerator.client-profiles.ts` — this file
+ * no longer emits Claude-native tool names / a Claude `model:` value
+ * unconditionally for every client `COMPANION_AGENT_TARGETS` writes a
+ * companion file for. Types split to `.types.ts`, pure helpers split to
+ * `.helpers.ts` (500-line standard).
  */
-/**
- * SMI-1795: Claude model constants for type safety and consistency
- */
-export const CLAUDE_MODELS = {
-    HAIKU: 'haiku',
-    SONNET: 'sonnet',
-    OPUS: 'opus',
-};
-/**
- * Tool detection patterns for analyzing skill content
- */
-const TOOL_PATTERNS = {
-    Read: {
-        patterns: ['read file', 'read the', 'examine', 'view file', 'Read tool', 'check file'],
-        priority: 1,
-    },
-    Write: {
-        patterns: ['write file', 'create file', 'save to', 'output to', 'Write tool', 'generate file'],
-        priority: 2,
-    },
-    Edit: {
-        patterns: [
-            'edit file',
-            'modify',
-            'update file',
-            'patch',
-            'Edit tool',
-            'change file',
-            'refactor',
-        ],
-        priority: 2,
-    },
-    Bash: {
-        patterns: [
-            'bash',
-            'npm',
-            'npx',
-            'git',
-            'docker',
-            'yarn',
-            'pnpm',
-            'terminal',
-            'shell',
-            'command',
-        ],
-        priority: 3,
-    },
-    Grep: {
-        patterns: ['grep', 'search for', 'find text', 'pattern match', 'Grep tool', 'search in'],
-        priority: 1,
-    },
-    Glob: {
-        patterns: ['glob', 'find file', 'file pattern', 'list files', 'Glob tool', 'locate'],
-        priority: 1,
-    },
-    WebFetch: {
-        patterns: ['fetch', 'http', 'api call', 'url', 'WebFetch', 'download', 'request'],
-        priority: 2,
-    },
-    WebSearch: {
-        patterns: ['web search', 'search online', 'lookup online', 'WebSearch', 'search the web'],
-        priority: 2,
-    },
-};
-/**
- * Minimum tools that most subagents need
- */
-const BASE_TOOLS = ['Read'];
-/**
- * SMI-1819: Maximum length for subagent names to prevent overly long identifiers
- */
-const MAX_SUBAGENT_NAME_LENGTH = 100;
-/**
- * Generate tool usage guidelines for a subagent
- */
-function generateToolGuidelines(tools) {
-    const guidelines = [];
-    if (tools.includes('Read')) {
-        guidelines.push('- **Read**: Use to examine files before modifications');
-    }
-    if (tools.includes('Write')) {
-        guidelines.push('- **Write**: Use for creating new files only');
-    }
-    if (tools.includes('Edit')) {
-        guidelines.push('- **Edit**: Use for modifying existing files');
-    }
-    if (tools.includes('Bash')) {
-        guidelines.push('- **Bash**: Use for command execution, prefer non-destructive commands');
-    }
-    if (tools.includes('Grep')) {
-        guidelines.push('- **Grep**: Use for searching file contents');
-    }
-    if (tools.includes('Glob')) {
-        guidelines.push('- **Glob**: Use for finding files by pattern');
-    }
-    if (tools.includes('WebFetch')) {
-        guidelines.push('- **WebFetch**: Use for fetching web content');
-    }
-    if (tools.includes('WebSearch')) {
-        guidelines.push('- **WebSearch**: Use for searching the web');
-    }
-    return guidelines.length > 0 ? guidelines.join('\n') : '- Use tools minimally and efficiently';
-}
+export { CLAUDE_MODELS } from './SubagentGenerator.types.js';
+import { CANONICAL_CLIENT } from '../install/paths.js';
+import { getSubagentGenerationProfile, mapToolNames } from './SubagentGenerator.client-profiles.js';
+import { CLAUDE_MODELS, } from './SubagentGenerator.types.js';
+import { MAX_SUBAGENT_NAME_LENGTH, generateToolGuidelines, OMITTED_TOOLS_GUIDANCE, generateClaudeMdSnippet, extractTriggerPhrases, determineModel, detectTools, } from './SubagentGenerator.helpers.js';
 /**
  * Generate subagent markdown content
+ *
+ * SMI-6276 (Wave 6 Step 1): frontmatter shape is now driven by the target
+ * client's generation profile (`SubagentGenerator.client-profiles.ts`) —
+ * `Read, Write, Bash, WebFetch, Grep, Glob, WebSearch` plus a Claude
+ * `model:` value are no longer emitted unconditionally for every client.
  */
-function generateSubagentContent(name, description, triggerPhrases, tools, model) {
+function generateSubagentContent(name, description, triggerPhrases, tools, model, client) {
     const triggerString = triggerPhrases.length > 0
         ? triggerPhrases.map((p) => `"${p}"`).join(', ')
         : '[describe trigger conditions]';
-    const toolGuidelines = generateToolGuidelines(tools);
-    return `---
-name: ${name}
-description: ${description} Use when ${triggerString}.
-skills: ${name.replace('-specialist', '')}
-tools: ${tools.join(', ')}
-model: ${model}
----
+    const profile = getSubagentGenerationProfile(client);
+    // SMI-6276 pr-reviewer finding: `description` is free text sourced from
+    // the SKILL.md being transformed — an unquoted plain YAML scalar breaks
+    // if it contains `: ` (reinterpreted as a nested key) or ` #` (truncated
+    // as a comment). JSON.stringify() produces a valid YAML double-quoted
+    // scalar (YAML's double-quote escaping is a superset of JSON's), so this
+    // composes the full description+trigger text as a plain JS string FIRST,
+    // then quotes the whole thing once — not per-field — so the result is
+    // safe regardless of what the source content contains.
+    const frontmatterLines = [
+        '---',
+        `name: ${name}`,
+        `description: ${JSON.stringify(`${description} Use when ${triggerString}.`)}`,
+    ];
+    if (profile.includeSkillsField) {
+        frontmatterLines.push(`skills: ${name.replace('-specialist', '')}`);
+    }
+    if (profile.toolsPolicy === 'claude-native') {
+        frontmatterLines.push(`tools: ${tools.join(', ')}`);
+    }
+    else if (profile.toolsPolicy === 'mapped-array') {
+        // SMI-6276: this client expects a YAML array of ITS OWN tool names, not
+        // a Claude-shaped comma list — never emit the unmapped/mis-shaped value
+        // (the AntiGravity docs' own "Known Issue" warns an unmapped name can
+        // hang the subagent process on invocation). Omit the key entirely when
+        // nothing maps, rather than an empty `tools: []` that says nothing.
+        const mappedTools = mapToolNames(tools, profile);
+        if (mappedTools.length > 0) {
+            frontmatterLines.push('tools:', ...mappedTools.map((tool) => `  - ${tool}`));
+        }
+    }
+    frontmatterLines.push(...profile.extraFrontmatterLines);
+    if (profile.modelPolicy === 'claude-enum') {
+        frontmatterLines.push(`model: ${model}`);
+    }
+    frontmatterLines.push('---');
+    const toolGuidelines = profile.toolsPolicy === 'claude-native' ? generateToolGuidelines(tools) : OMITTED_TOOLS_GUIDANCE;
+    return `${frontmatterLines.join('\n')}
 
 ## Operating Protocol
 
@@ -160,98 +104,6 @@ If the task cannot be completed:
 *Generated by Skillsmith Optimization Layer*
 `;
 }
-/**
- * Generate CLAUDE.md integration snippet
- */
-function generateClaudeMdSnippet(skillName, description, triggerPhrases, tools, model) {
-    const triggerPatterns = triggerPhrases.length > 0
-        ? triggerPhrases.map((p) => `- "${p}"`).join('\n')
-        : '- [add trigger patterns]';
-    const exampleTask = triggerPhrases.length > 0 ? triggerPhrases[0] : `execute ${skillName} task`;
-    return `
-### Subagent Delegation: ${skillName}
-
-When tasks match ${skillName} triggers, delegate to the ${skillName}-specialist
-subagent instead of executing directly. This provides context isolation and
-~37-97% token savings.
-
-**Trigger Patterns:**
-${triggerPatterns}
-
-**Delegation Example:**
-\`\`\`
-Task("${skillName}-specialist", "${exampleTask}", "${skillName}-specialist")
-\`\`\`
-
-**Model:** ${model}
-**Tools:** ${tools.join(', ')}
-`;
-}
-/**
- * Extract trigger phrases from skill description and content
- */
-function extractTriggerPhrases(description, content) {
-    const phrases = [];
-    // Common trigger phrase patterns
-    const patterns = [
-        /when\s+(?:you\s+)?(?:need\s+to\s+)?(.+?)(?:[.,]|$)/gi,
-        /use\s+(?:this\s+)?(?:when|for)\s+(.+?)(?:[.,]|$)/gi,
-        /(?:helps?\s+(?:you\s+)?(?:to\s+)?)?(.+?)(?:\s+tasks?|\s+operations?)/gi,
-    ];
-    const textToSearch = description + ' ' + content.slice(0, 2000); // Search first 2000 chars
-    for (const pattern of patterns) {
-        for (const match of textToSearch.matchAll(pattern)) {
-            const phrase = match[1].trim().toLowerCase();
-            if (phrase.length >= 5 && phrase.length <= 50 && !phrases.includes(phrase)) {
-                phrases.push(phrase);
-            }
-        }
-    }
-    // Extract from "trigger" or "invoke" mentions
-    const triggerMatch = content.match(/trigger[s]?\s*[:=]\s*["`']([^"`']+)["`']/gi);
-    if (triggerMatch) {
-        for (const match of triggerMatch) {
-            const phrase = match.replace(/trigger[s]?\s*[:=]\s*["`']/i, '').replace(/["`']$/, '');
-            if (!phrases.includes(phrase.toLowerCase())) {
-                phrases.push(phrase.toLowerCase());
-            }
-        }
-    }
-    return phrases.slice(0, 5); // Limit to 5 phrases
-}
-/**
- * Determine optimal model for subagent
- * SMI-1795: Uses CLAUDE_MODELS constants for type safety
- */
-function determineModel(toolUsage, lineCount) {
-    // Haiku for simple, fast operations
-    if (toolUsage.detectedTools.length <= 2 && lineCount < 200) {
-        return CLAUDE_MODELS.HAIKU;
-    }
-    // Opus for complex, reasoning-heavy tasks
-    if (toolUsage.bashCommandCount > 5 ||
-        (toolUsage.fileReadCount > 3 && toolUsage.fileWriteCount > 3)) {
-        return CLAUDE_MODELS.OPUS;
-    }
-    // Sonnet for balanced workloads (default)
-    return CLAUDE_MODELS.SONNET;
-}
-/**
- * Detect tools needed from skill content
- */
-function detectTools(content) {
-    const lowerContent = content.toLowerCase();
-    const detectedTools = new Set(BASE_TOOLS);
-    for (const [tool, config] of Object.entries(TOOL_PATTERNS)) {
-        for (const pattern of config.patterns) {
-            if (lowerContent.includes(pattern.toLowerCase())) {
-                detectedTools.add(tool);
-                break;
-            }
-        }
-    }
-    return Array.from(detectedTools);
-}
 /**
  * Generate a companion subagent for a skill
  *
@@ -259,9 +111,12 @@ function detectTools(content) {
  * @param description - The skill's description
  * @param content - The full SKILL.md content
  * @param analysis - Analysis from SkillAnalyzer
+ * @param client - SMI-6276: target client for the generated companion-agent
+ *   frontmatter shape (default: canonical / `claude-code`, preserving prior
+ *   behavior for every existing caller). See `SubagentGenerator.client-profiles.ts`.
  * @returns Subagent generation result
  */
-export function generateSubagent(skillName, description, content, analysis) {
+export function generateSubagent(skillName, description, content, analysis, client = CANONICAL_CLIENT) {
     // SMI-1815: Validate skill name is non-empty
     if (!skillName || skillName.trim() === '') {
         return {
@@ -291,7 +146,7 @@ export function generateSubagent(skillName, description, content, analysis) {
     // Generate subagent description
     const subagentDescription = description || `Specialist agent for ${skillName} operations`;
     // Generate content
-    const subagentContent = generateSubagentContent(subagentName, subagentDescription, triggerPhrases, allTools, model);
+    const subagentContent = generateSubagentContent(subagentName, subagentDescription, triggerPhrases, allTools, model, client);
     // Generate CLAUDE.md snippet
     const claudeMdSnippet = generateClaudeMdSnippet(skillName, subagentDescription, triggerPhrases, allTools, model);
     return {
@@ -313,9 +168,12 @@ export function generateSubagent(skillName, description, content, analysis) {
  * @param skillName - The name of the skill
  * @param description - The skill's description
  * @param content - The full SKILL.md content
+ * @param client - SMI-6276: target client for the generated companion-agent
+ *   frontmatter shape (default: canonical / `claude-code`, preserving prior
+ *   behavior for every existing caller). See `SubagentGenerator.client-profiles.ts`.
  * @returns Subagent generation result
  */
-export function generateMinimalSubagent(skillName, description, content) {
+export function generateMinimalSubagent(skillName, description, content, client = CANONICAL_CLIENT) {
     // SMI-1815: Validate skill name is non-empty
     if (!skillName || skillName.trim() === '') {
         return {
@@ -333,7 +191,7 @@ export function generateMinimalSubagent(skillName, description, content) {
     const baseName = skillName.length > maxBaseLength ? skillName.slice(0, maxBaseLength) : skillName;
     const subagentName = `${baseName}${suffix}`;
     const subagentDescription = description || `Specialist agent for ${skillName} operations`;
-    const subagentContent = generateSubagentContent(subagentName, subagentDescription, triggerPhrases, detectedTools, model);
+    const subagentContent = generateSubagentContent(subagentName, subagentDescription, triggerPhrases, detectedTools, model, client);
     const claudeMdSnippet = generateClaudeMdSnippet(skillName, subagentDescription, triggerPhrases, detectedTools, model);
     return {
         generated: true,