@miphamai/cli 0.81.5 → 0.81.7

package/src/ui/commands.ts

@@ -16,6 +16,7 @@ import { McpClient } from '../mcp/client'
 import { buildCapabilityReport } from '../core/capability-inventory'
 import { InstructionsLoader } from '../core/instructions'
 import { findDerivableSections, DERIVABLE_HINTS } from '../core/claude-md-audit'
+import { worktreeRoot, workflowScriptDir, workflowScriptDirs } from '../core/paths.ts'
 import { fixDoctor, fixConfig, fixCache, selectRepoClaudeFiles } from '../core/fix'
 import { fixCodeTarget } from '../core/fix-code'
 import { homedir } from 'node:os'
@@ -56,6 +57,15 @@ import { NPM_UPDATE_COMMAND, PACKAGE_VERSION, COAUTHOR_TRAILER } from '../shared
 import { getPreference } from '../config/preferences'
 import { loadCrossSessionConfig, tryRestoreFromBackup } from '../config/loader'
 import { getMemoryManager } from '../core/memory/memory-loader'
+import {
+  resolveTelemetry,
+  readTelemetrySettings,
+  setTelemetryEnabled,
+  setTelemetryEndpoint,
+  resetInstallId,
+} from '../telemetry/consent'
+import { getTelemetryConsent, enableTelemetryNow } from '../telemetry/index'
+import { NO_ENDPOINT } from '../telemetry/endpoint'
 import { stripIndent } from './strip-indent.js'
 import { createT } from '../i18n-core/t'
 import type { TranslationMap } from '../i18n-core/types'
@@ -251,7 +261,7 @@ const helpCmd: CommandHandler = (ctx) => {
       /init          Initialize .mipham config
       /setup         Guided project setup wizard
       /recommend     Analyze project + recommend setup
-      /permissions   Show permission settings
+      /permissions   Show or persist permission rules
       /add-dir <dir> Add workspace directory
       /security      Security review checklist
       /audit         Same as /security
@@ -2076,8 +2086,8 @@ const todosCmd: CommandHandler = (_ctx, args) => {
       return { content: t('commands.todos.usage_create') }
     }
     return {
-      content: `${t('commands.todos.create_title')}\n\nCreating task: "${title.trim()}"\n\nPassing to AI for structured task creation with TaskCreate...`,
-      forwardToAI: `Create a new task using TaskCreate with subject "${title.trim()}". Set a clear description and activeForm.`,
+      content: `${t('commands.todos.create_title')}\n\nCreating task: "${title.trim()}"\n\nPassing to AI for structured task creation with the Task tool...`,
+      forwardToAI: `Create a new task using the Task tool with action "create" and subject "${title.trim()}". Set a clear description and activeForm.`,
     }
   }
 
@@ -2092,7 +2102,7 @@ const todosCmd: CommandHandler = (_ctx, args) => {
           ${t('commands.todos.item_create')}
       `,
       forwardToAI:
-        'Use TaskList to show all current tasks. Present them in a clear summary grouped by status (pending/in_progress/completed). If there are no tasks, suggest creating one.',
+        'Use the Task tool with action "list" to show all current tasks. Present them in a clear summary grouped by status (pending/in_progress/completed). If there are no tasks, suggest creating one.',
     }
   }
 
@@ -2104,7 +2114,8 @@ const todosCmd: CommandHandler = (_ctx, args) => {
 
       ${t('commands.todos.default_body')}
     `,
-    forwardToAI: 'Use TaskList to show all current tasks, then present them clearly.',
+    forwardToAI:
+      'Use the Task tool with action "list" to show all current tasks, then present them clearly.',
   }
 }
 
@@ -2192,7 +2203,7 @@ const goalCmd: CommandHandler = (ctx, args) => {
   if (decompose) {
     lines.push(t('commands.goal.decompose_enabled'))
     // Decompose by creating initial subtasks
-    const decomposeMsg = `Break down this goal into 3-5 subtasks: "${goal}". For each subtask, use TaskCreate with the subject and description. Mark each as blocked by the previous one to create a dependency chain.`
+    const decomposeMsg = `Break down this goal into 3-5 subtasks: "${goal}". For each subtask, use the Task tool with action "create", giving the subject and description. Mark each as blocked by the previous one to create a dependency chain.`
     return {
       content: lines.join('\n'),
       forwardToAI: decomposeMsg,
@@ -2813,12 +2824,12 @@ const tasksCmd: CommandHandler = (ctx) => {
   const c = ctx.engine.getContext()
   const msgs = c.getMessages()
 
-  // Scan for task-related tool uses in message history
+  // Scan for task-related tool uses in message history.
+  // 任务工具只有一个 `Task`，动作走 action 参数 —— 过滤条件必须按真实工具名匹配，
+  // 否则计数恒为 0，「已检测到 N 次任务操作」这条分支永远不可达。
   const toolUses = msgs.flatMap((m) => {
     if (Array.isArray(m.content)) {
-      return m.content.filter(
-        (b) => b.type === 'tool_use' && ['TaskCreate', 'TaskUpdate', 'TaskList'].includes(b.name),
-      )
+      return m.content.filter((b) => b.type === 'tool_use' && b.name === 'Task')
     }
     return []
   })
@@ -2830,12 +2841,13 @@ const tasksCmd: CommandHandler = (ctx) => {
       ${toolUses.length > 0 ? t('commands.task_list.detected', { count: String(toolUses.length) }) : t('commands.task_list.no_tasks')}
 
       ${t('commands.task_list.reference')}
-        TaskCreate  — create a new task
-        TaskList    — list all tasks
-        TaskUpdate  — update task status
-        TaskGet     — get task details
-        TaskOutput  — get background task output
-        TaskStop    — stop a running task
+        Task(action: "create")  — create a new task
+        Task(action: "list")    — list all tasks
+        Task(action: "update")  — update task status
+        Task(action: "get")     — get task details
+        Task(action: "delete")  — delete a task
+        Task(action: "output")  — get background task output
+        Task(action: "stop")    — stop a running task
 
       ${t('commands.task_list.legacy_hint')}
     `,
@@ -3807,6 +3819,72 @@ const hooksCmd: CommandHandler = async (ctx) => {
   return { content: lines.join('\n') }
 }
 
+const telemetryCmd: CommandHandler = async (ctx, args) => {
+  const sub = (args[0] ?? 'status').toLowerCase()
+
+  if (sub === 'on' || sub === 'off') {
+    const enable = sub === 'on'
+    setTelemetryEnabled(enable)
+    if (enable) enableTelemetryNow()
+    return {
+      content: enable
+        ? '✓ Telemetry enabled. Anonymous usage counts will be sent to the configured endpoint.'
+        : '✓ Telemetry disabled. Nothing is collected and no queue file is written.',
+    }
+  }
+
+  if (sub === 'reset-id') {
+    const id = resetInstallId()
+    return { content: `✓ New anonymous install id: ${id}` }
+  }
+
+  if (sub === 'endpoint') {
+    const url = args[1]
+    if (!url) return { content: 'Usage: /telemetry endpoint <url|none>' }
+    setTelemetryEndpoint(url)
+    return {
+      content:
+        url === NO_ENDPOINT
+          ? '✓ Destination cleared. Telemetry stays on, but nothing is ever sent.'
+          : `✓ Endpoint set to ${url}`,
+    }
+  }
+
+  if (sub !== 'status') {
+    return {
+      content: 'Usage: /telemetry [status|on|off|reset-id|endpoint <url|none>]',
+    }
+  }
+
+  // ── status ──
+  const consent = getTelemetryConsent() ?? resolveTelemetry()
+  const settings = readTelemetrySettings('user')
+  // Two ways for `endpoint` to be empty, and they mean different things: never
+  // resolved (the kill switch fired first) or deliberately cleared (the `none`
+  // sentinel). A third — nothing configured — is unreachable now that the
+  // default is a real URL, which is why this used to be a one-line fallback.
+  const destination = consent.endpoint
+    ? consent.endpoint
+    : consent.endpointSource === 'off'
+      ? '_(not resolved — telemetry is off)_'
+      : '_(nothing — the `none` sentinel cleared it)_'
+  const lines = [
+    '## 📡 Telemetry',
+    '',
+    `| Field | Value |`,
+    `| --- | --- |`,
+    `| State | ${consent.enabled ? '🟢 enabled' : '⚪ disabled'} |`,
+    `| Decided by | \`${consent.source}\` |`,
+    `| Endpoint | ${destination} |`,
+    `| Source | \`${consent.endpointSource}\` |`,
+    `| Install id | ${settings.installId ?? '_(not yet generated)_'} |`,
+    `| Prompted | ${settings.promptedAt ?? '_(never)_'} |`,
+    '',
+    'Full data dictionary: `docs/telemetry.md`',
+  ]
+  return { content: lines.join('\n') }
+}
+
 const hooksHealthCmd: CommandHandler = (ctx) => {
   const hookEngine = ctx.engine.getHookEngine?.()
   if (!hookEngine) {
@@ -4319,10 +4397,7 @@ const workflowsCmd: CommandHandler = async () => {
   const { existsSync, readdirSync, readFileSync } = await import('node:fs')
   const { join } = await import('node:path')
 
-  const locations = [
-    join(process.cwd(), '.claude', 'workflows'),
-    join(homedir(), '.claude', 'workflows'),
-  ]
+  const locations = workflowScriptDirs(process.cwd())
 
   const lines: string[] = ['─ Workflows ─', '']
   let found = 0
@@ -4360,10 +4435,11 @@ const workflowsCmd: CommandHandler = async () => {
     lines.push('No workflow scripts found.')
     lines.push('')
     lines.push('Workflows are multi-agent orchestration scripts stored in:')
-    lines.push('  .claude/workflows/   (project-level)')
-    lines.push('  ~/.claude/workflows/ (user-level)')
+    lines.push('  .mipham/workflows/   (project-level — new scripts go here)')
+    lines.push('  .claude/workflows/   (project-level, legacy — still read)')
+    lines.push('  ~/.claude/workflows/ (user-level, legacy — still read)')
     lines.push('')
-    lines.push('Create a .js file in either location to add a workflow.')
+    lines.push('Create a .js file in either project location to add a workflow.')
   } else {
     lines.push('')
     lines.push(`${found} workflow(s) found.`)
@@ -4381,14 +4457,20 @@ const workflowSaveCmd = async (name: string): Promise<CommandResult> => {
   const { existsSync, mkdirSync, writeFileSync, readFileSync } = await import('node:fs')
   const { join } = await import('node:path')
 
-  const targetDir = join(process.cwd(), '.claude', 'workflows')
+  const targetDir = workflowScriptDir(process.cwd())
   if (!existsSync(targetDir)) {
     mkdirSync(targetDir, { recursive: true })
   }
 
-  // Read the last-run state persisted by the Workflow tool
-  const stateFile = join(targetDir, '.last-run.json')
-  if (!existsSync(stateFile)) {
+  // Read the last-run state persisted by the Workflow tool. Every readable
+  // location is checked, not just the writable one: a run that predates the
+  // move to `.mipham/` left its state in `.claude/workflows/`, so looking in
+  // the new directory alone would report "no recent run" on the first save
+  // after upgrading.
+  const stateFile = workflowScriptDirs(process.cwd())
+    .map((dir) => join(dir, '.last-run.json'))
+    .find((file) => existsSync(file))
+  if (!stateFile) {
     return { content: 'No recent workflow run found. Run a workflow first with /workflow <task>.' }
   }
 
@@ -4417,10 +4499,7 @@ const workflowRunCmd = async (name: string): Promise<CommandResult> => {
 
   const safeName = name.replace(/[^a-zA-Z0-9_-]/g, '-')
 
-  const locations = [
-    join(process.cwd(), '.claude', 'workflows'),
-    join(homedir(), '.claude', 'workflows'),
-  ]
+  const locations = workflowScriptDirs(process.cwd())
 
   for (const loc of locations) {
     const scriptPath = join(loc, `${safeName}.js`)
@@ -4435,7 +4514,9 @@ const workflowRunCmd = async (name: string): Promise<CommandResult> => {
   }
 
   return {
-    content: `Workflow "${safeName}" not found in .claude/workflows/ or ~/.claude/workflows/`,
+    content:
+      `Workflow "${safeName}" not found in .mipham/workflows/, ` +
+      `.claude/workflows/ or ~/.claude/workflows/`,
   }
 }
 
@@ -5003,7 +5084,7 @@ const forkCmd: CommandHandler = async (ctx, args) => {
     .slice(0, 40)
   const name = `${slug}-${Date.now().toString(36)}`
   const branch = `worktree/${name}`
-  const wtPath = join(process.cwd(), '.claude', 'worktrees', name)
+  const wtPath = join(worktreeRoot(process.cwd()), name)
 
   try {
     execSync(`git worktree add -b ${branch} ${wtPath} HEAD`, { stdio: 'ignore', timeout: 30_000 })
@@ -5304,6 +5385,7 @@ const commandsListCmd: CommandHandler = () => {
     '/keys audit': 'Account',
     '/keys view': 'Account',
     '/feedback': 'Account',
+    '/telemetry': 'Account',
     '/agents': 'Agents',
     '/bg': 'Agents',
     '/fork': 'Agents',
@@ -5525,6 +5607,7 @@ registry.set('/cd', cdCmd)
 registry.set('/hooks', hooksCmd)
 registry.set('/hooks health', hooksHealthCmd)
 registry.set('/hooks enable', hooksEnableCmd)
+registry.set('/telemetry', telemetryCmd)
 registry.set('/batch', batchCmd)
 
 // ═══════════════════════════════════════════════════════════════
@@ -5539,6 +5622,47 @@ export function getCommandNames(): string[] {
   return Array.from(registry.keys()).sort()
 }
 
+/**
+ * The commands `app.tsx` intercepts **before** the registry lookup.
+ *
+ * They return early and never reach `getCommand`, so the registry is not
+ * authoritative for them. `/model-picker` is the one that is not a registry key
+ * at all — the other five are, so listing them here is belt-and-braces rather
+ * than a claim that they are missing.
+ */
+const PRE_REGISTRY_COMMANDS = ['/switch', '/pick', '/model-picker', '/exit', '/quit', '/focus']
+
+/**
+ * The bucket an unrecognised command name is recorded under.
+ *
+ * **Why this exists.** `parseSlashCommand` returns `parts[0]` — whatever the user
+ * typed. Without a convergence point, `/foobar` mints a `command_calls./foobar`
+ * series on the spot, and nothing bounds the number of series:
+ * `MAX_LABEL_LENGTH` in `payload.ts` truncates the **value**, not the key count.
+ * `tool_name` really is closed by construction (the registry declares the tool
+ * set); `command_name` is not.
+ */
+export const UNKNOWN_COMMAND = '/unknown'
+
+/** The label a command name is recorded under: itself if we ship it, else the bucket. */
+export function commandLabelFor(command: string): string {
+  if (registry.has(command) || PRE_REGISTRY_COMMANDS.includes(command)) return command
+  return UNKNOWN_COMMAND
+}
+
+/**
+ * Every label `command_calls` can carry — the source the collector's allowlist is
+ * generated from.
+ *
+ * Two more than `getCommandNames()`: `/model-picker` (user-typable, not a registry
+ * key) and `UNKNOWN_COMMAND`. Miss either and the allowlist does not contain it,
+ * so the collector folds those events into `__other__` — and `__other__` is
+ * exactly what T4 must not be reading when it votes.
+ */
+export function getCommandLabelNames(): string[] {
+  return Array.from(new Set([...registry.keys(), ...PRE_REGISTRY_COMMANDS, UNKNOWN_COMMAND])).sort()
+}
+
 export interface CommandEntry {
   name: string
   description: string
@@ -5640,7 +5764,7 @@ const COMMAND_DESCRIPTIONS: Record<string, string> = {
   '/loop': 'Run prompt on interval',
   '/init': 'Initialize .mipham config',
   '/setup': 'Guided project setup wizard',
-  '/permissions': 'Show permission settings',
+  '/permissions': 'Show or persist permission rules',
   '/add-dir': 'Add workspace directory',
   '/recommend': 'Analyze project + recommend skills & setup',
   '/security': 'Security review checklist',
@@ -5681,6 +5805,7 @@ const COMMAND_DESCRIPTIONS: Record<string, string> = {
   '/hooks health': 'Check hook health — see failures, disabled hooks, recovery status',
   '/hooks enable': 'Manually re-enable a hook that was auto-disabled after repeated failures',
   '/batch': 'Apply changes across multiple files',
+  '/telemetry': 'Show or change anonymous usage-reporting settings',
 }
 
 export function getCommandList(): CommandEntry[] {

package/src/workflow/primitives/agent.ts

@@ -1,8 +1,10 @@
+import { join } from 'node:path'
 import { SubAgent } from '../../agent/sub-agent'
 import type { ProviderRegistry } from '../../providers/registry'
 import type { Llm } from '../../providers/llm'
 import type { ToolDefinition } from '../../shared/index.ts'
 import type { PermissionSystem } from '../../core/permission'
+import { worktreeRoot } from '../../core/paths.ts'
 import { validateJSONSchema, formatValidationErrors } from '../schema-validator'
 
 export interface WorkflowAgentOpts {
@@ -32,7 +34,7 @@ export interface WorkflowAgentOpts {
  *   4. Returns the validated object, or { raw, validationErrors } on final failure.
  *
  * When `isolation: 'worktree'` is set:
- *   1. A git worktree is created at .claude/worktrees/wf-<slug>
+ *   1. A git worktree is created at .mipham/worktrees/wf-<slug>
  *   2. The sub-agent runs with its cwd set to the worktree path
  *   3. Changes are auto-committed (best-effort)
  *   4. The worktree is cleaned up after execution
@@ -60,7 +62,7 @@ export async function workflowAgent(
   if (opts.isolation === 'worktree') {
     const slug = `wf-${Date.now().toString(36)}-${Math.random().toString(36).slice(2, 6)}`
     worktreeBranch = `worktree/${slug}`
-    worktreePath = `.claude/worktrees/${slug}`
+    worktreePath = join(worktreeRoot(process.cwd()), slug)
 
     const proc = Bun.spawn(['git', 'worktree', 'add', '-b', worktreeBranch, worktreePath, 'HEAD'], {
       stdout: 'pipe',

package/src/core/task-runner-tasks.json

@@ -1,14 +0,0 @@
-{
-  "version": 1,
-  "tasks": [
-    {
-      "id": "task-answer-fn",
-      "instruction": "用 Write 工具在 <taskDir>/solution.ts 写入一个 TypeScript 文件，导出 `export function answer(): number { return 42 }`。",
-      "groundTruth": {
-        "kind": "file-contains",
-        "file": "solution.ts",
-        "contains": ["export function answer", "42"]
-      }
-    }
-  ]
-}

package/src/core/task-runner.ts

@@ -1,163 +0,0 @@
-// apps/cli/src/core/task-runner.ts
-// CRSI 端到端任务运行器（C-MVP）——行为效果度量基建。
-import { existsSync, readFileSync, mkdirSync, rmSync } from 'node:fs'
-import { join } from 'node:path'
-import tasksFile from './task-runner-tasks.json' with { type: 'json' }
-import { QueryEngine } from './engine'
-import { ContextManager } from './context'
-import { PermissionSystem } from './permission'
-import { ProviderRegistry } from '../providers/registry'
-import type { Llm } from '../providers/llm'
-import { createToolRegistry } from '../tools'
-import type { PermissionLevel } from '../shared'
-
-export type RunnerGroundTruth = { kind: 'file-contains'; file: string; contains: string[] }
-
-export interface RunnerTask {
-  id: string
-  instruction: string
-  groundTruth: RunnerGroundTruth
-}
-
-export function loadRunnerTasks(): RunnerTask[] {
-  return tasksFile.tasks as unknown as RunnerTask[]
-}
-
-export function judgeTask(task: RunnerTask, taskDir: string): { passed: boolean; detail?: string } {
-  if (task.groundTruth.kind !== 'file-contains') {
-    return { passed: false, detail: `unsupported groundTruth kind: ${task.groundTruth.kind}` }
-  }
-  const filePath = join(taskDir, task.groundTruth.file)
-  if (!existsSync(filePath)) {
-    return { passed: false, detail: `file not found: ${task.groundTruth.file}` }
-  }
-  const content = readFileSync(filePath, 'utf-8')
-  for (const needle of task.groundTruth.contains) {
-    if (!content.includes(needle)) {
-      return { passed: false, detail: `missing substring: ${needle}` }
-    }
-  }
-  return { passed: true }
-}
-
-export interface TaskRunResult {
-  taskId: string
-  passed: boolean
-  detail?: string
-}
-
-const TASK_DIR_PLACEHOLDER = '<taskDir>'
-
-function buildEngine(llm: Llm, permission: PermissionLevel, systemPrompt?: string): QueryEngine {
-  const registry = new ProviderRegistry([], 'test', 'test-model')
-  // 注册一个永不 chat 的占位 provider——llm 被 setLlm 覆盖，但 process() 内部
-  // 多处调用 registry.getActive().config.id 记录 provider id，必须能取到。
-  registry.register('test', {
-    config: { id: 'test', name: 'Test', protocol: 'openai-compatible', apiKey: 'key', models: [] },
-    chat: async function* () {
-      yield { type: 'stop' }
-    },
-    listModels: async () => [],
-    healthCheck: async () => true,
-  })
-  const context = new ContextManager({ maxTokens: 100_000, compactionThreshold: 0.9 })
-  if (systemPrompt !== undefined) context.setSystemPrompt(systemPrompt)
-  const tools = createToolRegistry()
-  const engine = new QueryEngine(registry, context, tools, new PermissionSystem(permission))
-  engine.setLlm(llm)
-  return engine
-}
-
-export async function runTask(
-  task: RunnerTask,
-  llm: Llm,
-  opts: { taskDir?: string; permission?: PermissionLevel; systemPrompt?: string } = {},
-): Promise<TaskRunResult> {
-  const taskDir = opts.taskDir ?? join(process.cwd(), '.mipham', 'task-runner')
-  const permission = opts.permission ?? 'bypassPermissions'
-
-  rmSync(taskDir, { recursive: true, force: true })
-  mkdirSync(taskDir, { recursive: true })
-
-  const instruction = task.instruction.replaceAll(TASK_DIR_PLACEHOLDER, taskDir)
-  const engine = buildEngine(llm, permission, opts.systemPrompt)
-
-  for await (const _ of engine.process(instruction)) {
-    /* drain agentic loop */
-  }
-
-  const verdict = judgeTask(task, taskDir)
-  return { taskId: task.id, passed: verdict.passed, detail: verdict.detail }
-}
-
-export interface TaskRunStats {
-  taskId: string
-  samples: number
-  passed: number
-  /** 0-1 */
-  passRate: number
-}
-
-export async function runTaskN(
-  task: RunnerTask,
-  llm: Llm,
-  n: number,
-  opts: { taskDir?: string; permission?: PermissionLevel; systemPrompt?: string } = {},
-): Promise<TaskRunStats> {
-  let passed = 0
-  for (let i = 0; i < n; i++) {
-    const result = await runTask(task, llm, opts)
-    if (result.passed) passed++
-  }
-  return { taskId: task.id, samples: n, passed, passRate: n > 0 ? passed / n : 0 }
-}
-
-export interface RunComparison {
-  baseline: TaskRunStats
-  candidate: TaskRunStats
-  /** 弱判：candidate 不退化（不低于 baseline 且至少 1 次成功） */
-  notDegraded: boolean
-  /** candidate 严格更好（通过率更高） */
-  improved: boolean
-}
-
-export function isNotDegraded(baseline: TaskRunStats, candidate: TaskRunStats): boolean {
-  return candidate.passRate >= baseline.passRate && candidate.passed >= 1
-}
-
-export function isImproved(baseline: TaskRunStats, candidate: TaskRunStats): boolean {
-  return candidate.passRate > baseline.passRate
-}
-
-export function compareRuns(baseline: TaskRunStats, candidate: TaskRunStats): RunComparison {
-  return {
-    baseline,
-    candidate,
-    notDegraded: isNotDegraded(baseline, candidate),
-    improved: isImproved(baseline, candidate),
-  }
-}
-
-export async function runBeforeAfter(
-  task: RunnerTask,
-  llm: Llm,
-  n: number,
-  opts: {
-    beforePrompt?: string
-    afterPrompt?: string
-    taskDir?: string
-    permission?: PermissionLevel
-  } = {},
-): Promise<RunComparison> {
-  const baseline = await runTaskN(task, llm, n, {
-    taskDir: opts.taskDir,
-    permission: opts.permission,
-    systemPrompt: opts.beforePrompt,
-  })
-  const candidate = await runTaskN(task, llm, n, {
-    taskDir: opts.taskDir,
-    permission: opts.permission,
-    systemPrompt: opts.afterPrompt,
-  })
-  return compareRuns(baseline, candidate)
-}

package/src/skills/mipham/runtime.ts

@@ -1,66 +0,0 @@
-import type { SkillDefinition } from '../../shared/index.ts'
-
-export interface MiphamRuntimeContext {
-  skill: SkillDefinition
-  cwd: string
-  sessionId: string
-  modelId: string
-  providerId: string
-}
-
-/**
- * Mipham exclusive skill runtime.
- * Mipham skills use `.mipham-skill.md` extension and have access to
- * Mipham-specific features: model optimization, security features, etc.
- */
-export class MiphamRuntime {
-  private context: MiphamRuntimeContext
-
-  constructor(context: MiphamRuntimeContext) {
-    this.context = context
-  }
-
-  getSkill(): SkillDefinition {
-    return this.context.skill
-  }
-
-  getPrompts(): Record<string, string> {
-    return this.context.skill.prompts || {}
-  }
-
-  getTools(): SkillDefinition['tools'] {
-    return this.context.skill.tools || []
-  }
-
-  getHooks(): SkillDefinition['hooks'] {
-    return this.context.skill.hooks || []
-  }
-
-  /**
-   * Execute a named prompt with Mipham-specific context.
-   */
-  async executePrompt(name: string, variables?: Record<string, string>): Promise<string> {
-    const prompt = this.context.skill.prompts?.[name]
-    if (!prompt) {
-      throw new Error(`Prompt "${name}" not found in skill "${this.context.skill.name}"`)
-    }
-
-    let result = prompt
-    const allVars: Record<string, string> = {
-      provider: this.context.providerId,
-      model: this.context.modelId,
-      session: this.context.sessionId,
-      cwd: this.context.cwd,
-      ...variables,
-    }
-
-    // Single-pass substitution: replacement values are NOT re-scanned for more
-    // template markers, so a variable containing `${key}` can't be re-expanded
-    // into another variable's value.
-    result = result.replace(/\$\{([^}]+)\}/g, (match, key: string) =>
-      Object.prototype.hasOwnProperty.call(allVars, key) ? allVars[key]! : match,
-    )
-
-    return result
-  }
-}

package/src/skills/standard/runtime.ts

@@ -1,62 +0,0 @@
-import type { SkillDefinition } from '../../shared/index.ts'
-
-export interface StandardRuntimeContext {
-  skill: SkillDefinition
-  cwd: string
-  sessionId: string
-}
-
-/**
- * Standard skill runtime — executes SKILL.md definitions.
- * Standard skills follow the open-source SKILL.md specification.
- */
-export class StandardRuntime {
-  private context: StandardRuntimeContext
-
-  constructor(context: StandardRuntimeContext) {
-    this.context = context
-  }
-
-  getSkill(): SkillDefinition {
-    return this.context.skill
-  }
-
-  getPrompts(): Record<string, string> {
-    return this.context.skill.prompts || {}
-  }
-
-  getPrompt(name: string): string | undefined {
-    return this.context.skill.prompts?.[name]
-  }
-
-  getTools(): SkillDefinition['tools'] {
-    return this.context.skill.tools || []
-  }
-
-  getHooks(): SkillDefinition['hooks'] {
-    return this.context.skill.hooks || []
-  }
-
-  /**
-   * Execute a named prompt from the skill.
-   * Returns the prompt text with any variable substitution applied.
-   */
-  async executePrompt(name: string, variables?: Record<string, string>): Promise<string> {
-    const prompt = this.getPrompt(name)
-    if (!prompt) {
-      throw new Error(`Prompt "${name}" not found in skill "${this.context.skill.name}"`)
-    }
-
-    let result = prompt
-    if (variables) {
-      // Single-pass substitution: replacement values are NOT re-scanned for
-      // more template markers, so an argument containing `${key}` can't be
-      // re-expanded into another variable's value.
-      result = result.replace(/\$\{([^}]+)\}/g, (match, key: string) =>
-        Object.prototype.hasOwnProperty.call(variables, key) ? variables[key]! : match,
-      )
-    }
-
-    return result
-  }
-}