@miphamai/cli 0.48.0 → 0.49.0

package/src/skills/skill-assets.ts

@@ -0,0 +1,37 @@
+import { existsSync, readFileSync, writeFileSync, mkdirSync } from 'node:fs'
+import { join, dirname } from 'node:path'
+import { homedir } from 'node:os'
+import { BUNDLED_SKILL_ASSETS, type BundledSkillAsset } from './bundled-skill-assets'
+
+export interface SkillAssetsOptions {
+  /** Base dir under which assets land as `<baseDir>/<skillName>/...`. Defaults to `~/.mipham/skills`. */
+  baseDir?: string
+  /** Asset map. Defaults to the compiled-in snapshot. Injectable for tests. */
+  assets?: Record<string, BundledSkillAsset[]>
+}
+
+/**
+ * Idempotently extract a skill's bundled executable assets to disk.
+ * Content-compare: only writes when a file is missing or its content drifted,
+ * so user-added files (e.g. site-patterns/*.md) are never overwritten.
+ * Assets are UTF-8 text only today — binary assets would need base64 (see
+ * generate-bundled-skills.ts). Returns the extraction root, or null if the
+ * skill bundles no assets.
+ */
+export function ensureSkillAssets(skillName: string, opts?: SkillAssetsOptions): string | null {
+  const map = opts?.assets ?? BUNDLED_SKILL_ASSETS
+  const base = opts?.baseDir ?? join(homedir(), '.mipham', 'skills')
+  const list = map[skillName]
+  if (!list) return null
+  const root = join(base, skillName)
+  for (const a of list) {
+    const dest = join(root, a.path)
+    const needsWrite = !existsSync(dest) || readFileSync(dest, 'utf-8') !== a.content
+    if (needsWrite) {
+      mkdirSync(dirname(dest), { recursive: true })
+      // Preserve the source exec bit (scripts/*.mjs), default 0o644 for docs.
+      writeFileSync(dest, a.content, { mode: a.mode ?? 0o644 })
+    }
+  }
+  return root
+}

package/src/tools/agent/skill.ts

@@ -1,6 +1,7 @@
-import type { ToolDefinition } from '../../shared/index.ts'
+import type { ToolDefinition } from '../../shared'
 import { executeForkedSkill } from '../../skills/fork-executor'
-import { sanitizeSkillBody } from '../../skills/sanitizer.js'
+import { sanitizeSkillBody } from '../../skills/sanitizer'
+import { ensureSkillAssets } from '../../skills/skill-assets'
 
 export const skillTool: ToolDefinition = {
   name: 'Skill',
@@ -36,6 +37,16 @@ export const skillTool: ToolDefinition = {
         }
       }
 
+      // Extract executable assets (scripts/references) if this skill bundles them.
+      // No-op for every skill that has no entry in BUNDLED_SKILL_ASSETS.
+      // A write failure (EACCES/ENOSPC) must not abort invocation: the skill body
+      // still delivers its own recovery guidance for the missing-script case.
+      try {
+        ensureSkillAssets(skillName)
+      } catch (err) {
+        console.warn(`Skill asset extraction failed for "${skillName}":`, err)
+      }
+
       // Check if skill has context: fork — execute in isolated subagent
       if (skill.context === 'fork') {
         const registry = ctx.registry