@miphamai/cli 0.81.6 → 0.81.7

package/src/tools/exec/enter-worktree.ts

@@ -1,10 +1,11 @@
 import type { ToolDefinition } from '../../shared/index.ts'
+import { worktreeRoot } from '../../core/paths.ts'
 
 export const enterWorktreeTool: ToolDefinition = {
   name: 'EnterWorktree',
   description:
     'Create an isolated git worktree for parallel development. ' +
-    'Creates a new worktree at .claude/worktrees/<name> on its own branch. ' +
+    'Creates a new worktree at .mipham/worktrees/<name> on its own branch. ' +
     'Use this when you need to work on a separate task without affecting the main workspace. ' +
     'Pair with ExitWorktree to clean up when done.',
   category: 'exec',
@@ -56,10 +57,10 @@ export const enterWorktreeTool: ToolDefinition = {
 
     const cwd = ctx.cwd
     const { resolve } = await import('node:path')
-    const worktreePath = resolve(`${cwd}/.claude/worktrees/${name}`)
-    const allowedPrefix = resolve(`${cwd}/.claude/worktrees/`)
+    const worktreePath = resolve(worktreeRoot(cwd), name)
+    const allowedPrefix = resolve(worktreeRoot(cwd))
 
-    // Defense-in-depth: verify resolved path is within .claude/worktrees/
+    // Defense-in-depth: verify resolved path stays within the worktree root
     if (
       !worktreePath.startsWith(allowedPrefix + '/') &&
       worktreePath !== allowedPrefix.slice(0, -1)
@@ -67,7 +68,7 @@ export const enterWorktreeTool: ToolDefinition = {
       return {
         success: false,
         content: '',
-        error: 'Worktree path must be within .claude/worktrees/.',
+        error: 'Worktree path must be within .mipham/worktrees/.',
       }
     }
 

package/src/tools/exec/exit-worktree.ts

@@ -1,4 +1,5 @@
 import type { ToolDefinition } from '../../shared/index.ts'
+import { worktreeRoots } from '../../core/paths.ts'
 
 export const exitWorktreeTool: ToolDefinition = {
   name: 'ExitWorktree',
@@ -13,7 +14,8 @@ export const exitWorktreeTool: ToolDefinition = {
     properties: {
       path: {
         type: 'string',
-        description: 'Absolute path to the worktree to exit. Must be under .claude/worktrees/.',
+        description:
+          'Absolute path to the worktree to exit. Must be under .mipham/worktrees/ (or the legacy .claude/worktrees/).',
       },
       action: {
         type: 'string',
@@ -34,18 +36,21 @@ export const exitWorktreeTool: ToolDefinition = {
     const action = params.action as string
     const discardChanges = params.discard_changes === true
 
-    // Validate the path is under .claude/worktrees
+    // Validate the path is under a worktree root（新目录优先，兼容历史 .claude/）
     const cwd = ctx.cwd
     const { resolve } = await import('node:path')
     const resolvedPath = resolve(worktreePath)
-    const allowedPrefix = resolve(`${cwd}/.claude/worktrees/`)
+    const roots = worktreeRoots(cwd).map((root) => resolve(root))
+    const inWorktree = roots.some(
+      (root) => resolvedPath === root || resolvedPath.startsWith(root + '/'),
+    )
 
-    if (!resolvedPath.startsWith(allowedPrefix)) {
+    if (!inWorktree) {
       return {
         success: false,
         content: '',
         error:
-          `Path "${worktreePath}" is not under .claude/worktrees/. ` +
+          `Path "${worktreePath}" is not under .mipham/worktrees/ or .claude/worktrees/. ` +
           `Only worktrees created by EnterWorktree can be managed here.`,
       }
     }

package/src/tools/exec/git.ts

@@ -1,4 +1,5 @@
 import type { ToolDefinition } from '../../shared/index.ts'
+import { findWorktreeMarker } from '../../core/paths.ts'
 
 // P0-4 (v2.1.222 alignment): Regex-based word-boundary patterns replace
 // fragile substring matching. Each pattern describes what it blocks.
@@ -52,14 +53,23 @@ export const DANGEROUS_GIT_PATTERNS: Array<{ pattern: RegExp; description: strin
  * when operating in a worktree context.
  */
 function isOutsideWorktree(command: string, cwd: string): string | null {
-  const WORKTREE_MARKER = '.claude/worktrees/'
-  if (!cwd.includes(WORKTREE_MARKER)) return null
-
-  // Extract the project root (everything before .claude/worktrees/)
-  const worktreeRoot = cwd.substring(0, cwd.indexOf(WORKTREE_MARKER))
-
-  // Detect git commands that reference the main checkout path
-  const mainCheckoutPaths = [/\b--work-tree=([^\s]+)/g, /\b--git-dir=([^\s]+)/g, /\b-C\s+([^\s]+)/g]
+  // 标记取自 core/paths.ts：新目录与历史 .claude/worktrees/ 都认，隔离度只增不减。
+  const marker = findWorktreeMarker(cwd)
+  if (!marker) return null
+
+  // Extract the project root (everything before the worktree marker)
+  const worktreeRoot = marker.root
+
+  // Detect git commands that reference the main checkout path.
+  // NOTE: no `\b` before the leading `-` — `\b` needs a word/non-word
+  // transition and `-` is itself a non-word char, so `\b--work-tree=` never
+  // matched anything and this whole check was silently dead. `-C` needs an
+  // explicit boundary because without one it would match inside a path.
+  const mainCheckoutPaths = [
+    /--work-tree=([^\s]+)/g,
+    /--git-dir=([^\s]+)/g,
+    /(?:^|\s)-C\s+([^\s]+)/g,
+  ]
 
   for (const pathPattern of mainCheckoutPaths) {
     let match: RegExpExecArray | null

package/src/tools/system/config.ts

@@ -4,8 +4,8 @@ import { homedir } from 'node:os'
 import { parse as parseYaml, stringify } from 'yaml'
 import type { ToolDefinition } from '../../shared/index.ts'
 
-const MIPHAM_DIR = join(homedir(), '.mipham')
-const USER_CONFIG = join(MIPHAM_DIR, 'config.yml')
+const MIPHAM_HOME = join(homedir(), '.mipham')
+const USER_CONFIG = join(MIPHAM_HOME, 'config.yml')
 
 export const configTool: ToolDefinition = {
   name: 'Config',
@@ -26,7 +26,7 @@ export const configTool: ToolDefinition = {
     required: ['action'],
   },
   async execute(params, _ctx) {
-    mkdirSync(MIPHAM_DIR, { recursive: true })
+    mkdirSync(MIPHAM_HOME, { recursive: true })
     const action = params.action as string
 
     let config: Record<string, unknown> = {}

package/src/ui/app.tsx

@@ -14,6 +14,8 @@ import type { SkillsLoader } from '../skills/loader'
 import type { PluginManager } from '../plugin/plugin-manager'
 import { setPreference } from '../config/preferences'
 import { saveProviderApiKey } from '../config/loader'
+import { recordCommand } from '../telemetry/index'
+import { recordCrash } from '../telemetry/crash'
 import { AgentRegistry } from '../agent/agent-registry'
 import { getBackgroundAgentRegistry } from '../agent/background-registry'
 import { getMessageRouter, parseMention, resolveRecipientSession } from '../agent/message-router'
@@ -46,6 +48,7 @@ import type { AgentViewManager } from '../agent-view/agent-view-manager'
 import { WorkflowProgress } from './workflow-progress.js'
 import { GoalProgress } from './goal-progress.js'
 import {
+  commandLabelFor,
   getCommand,
   looksLikeSlashCommand,
   parseSlashCommand,
@@ -206,9 +209,13 @@ export function App({
   useEffect(() => {
     if (!gitBranch) return
     let cancelled = false
-    resolveGitPr(gitBranch).then((pr) => {
-      if (!cancelled) setGitPr(pr)
-    })
+    resolveGitPr(gitBranch)
+      .then((pr) => {
+        if (!cancelled) setGitPr(pr)
+      })
+      .catch(() => {
+        /* best effort — the PR badge is decoration, never worth surfacing */
+      })
     return () => {
       cancelled = true
     }
@@ -217,11 +224,15 @@ export function App({
   // 启动后台查新版（非阻塞；离线静默失败）
   useEffect(() => {
     let cancelled = false
-    checkForUpdatesAsync().then((update) => {
-      if (!cancelled && update.available) {
-        setUpdateStatus({ state: 'available', latest: update.latest })
-      }
-    })
+    checkForUpdatesAsync()
+      .then((update) => {
+        if (!cancelled && update.available) {
+          setUpdateStatus({ state: 'available', latest: update.latest })
+        }
+      })
+      .catch(() => {
+        /* offline — silent, per the comment above */
+      })
     return () => {
       cancelled = true
     }
@@ -750,7 +761,7 @@ export function App({
       }
 
       // Turn finished — drain any /loop wakeup queued while we were running.
-      drainLoopQueueRef.current?.(turnId)
+      void drainLoopQueueRef.current?.(turnId)
     },
     [engine, syncBgAgents, config, t],
   )
@@ -794,7 +805,7 @@ export function App({
   useEffect(() => {
     if (wakeupTick === 0) return
     if (isLoading) return
-    drainLoopQueueRef.current?.(turnIdRef.current)
+    void drainLoopQueueRef.current?.(turnIdRef.current)
   }, [wakeupTick, isLoading])
 
   // ── cron poller ──
@@ -850,6 +861,15 @@ export function App({
       if (looksLikeSlashCommand(input)) {
         const { command, args } = parseSlashCommand(input)
 
+        // Counted here rather than at the registry lookup below: /switch, /pick,
+        // /model-picker, /exit, /quit and /focus are special-cased and return
+        // before ever reaching it, so counting there would silently under-report
+        // six of the most-used commands.
+        //
+        // `commandLabelFor` collapses unrecognised names into `/unknown` — see it
+        // for why an unbounded `command_name` is not a cosmetic problem.
+        recordCommand(commandLabelFor(command))
+
         // /switch takes args, handled separately
         if (command === '/switch') {
           const result = await handleSwitch(mkCtx(), args)
@@ -1099,7 +1119,16 @@ export function App({
   }
 
   return (
-    <ErrorBoundary>
+    <ErrorBoundary
+      onError={(error) => {
+        // The boundary's own job is *surviving* a render error — it renders a
+        // fallback and the session continues. But this is the exact failure the
+        // boundary was written for (a frozen layout with a live process, i.e. a
+        // silent hang), so record it as a crash signal rather than letting it
+        // vanish once the fallback paints over the evidence.
+        recordCrash(error, 'render')
+      }}
+    >
       <Box flexDirection="column" padding={1} height="100%">
         {/* Workflow progress — auto-detects active workflows, renders nothing when idle */}
         <WorkflowProgress />

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"]
-      }
-    }
-  ]
-}