tink-harness 1.2.1 → 1.2.2

package/bin/install.js

@@ -11,13 +11,20 @@ const __dirname = path.dirname(__filename);
 const root = path.resolve(__dirname, '..');
 const args = process.argv.slice(2);
 const command = args[0] || 'install';
-const isUpdate = command === 'update';
-const dryRun = args.includes('--dry-run');
-const force = args.includes('--force');
-const yes = args.includes('--yes') || args.includes('-y');
-const interactive = process.stdin.isTTY && process.stdout.isTTY && !yes && !dryRun;
-const source = 'https://github.com/dotoricode/tink-harness.git';
-const validSurfaces = new Set(['claude', 'codex']);
+const isUpdate = command === 'update';
+const dryRun = args.includes('--dry-run');
+const force = args.includes('--force');
+const yes = args.includes('--yes') || args.includes('-y');
+const interactive = process.stdin.isTTY && process.stdout.isTTY && !yes && !dryRun;
+const source = 'https://github.com/dotoricode/tink-harness.git';
+const validSurfaces = new Set(['claude', 'codex']);
+const operationLog = {
+  written: [],
+  updated: [],
+  preserved: [],
+  removedLegacy: [],
+  keptUnknown: []
+};
 
 const COPY = {
   en: {
@@ -67,124 +74,124 @@ const COPY = {
   }
 };
 
-const COMPONENTS = {
+const COMPONENTS = {
   en: [
-    { value: 'commands', label: 'Claude Code commands', hint: '/tink:setup, /tink:cast, /tink:verify, /tink:list, /tink:frog, /tink:weave, /tink:update' },
+    { value: 'commands', label: 'Claude Code commands', hint: '/tink:setup, /tink:cast, /tink:verify, /tink:list, /tink:frog, /tink:weave, /tink:update' },
     { value: 'skill', label: 'Tink skill', hint: 'Tink operating rules for Claude Code' },
     { value: 'harnesses', label: 'Built-in harnesses', hint: 'Reusable task templates' },
     { value: 'memory', label: 'Memory templates', hint: 'Approved mistakes/preferences/lessons files' },
     { value: 'hook', label: 'Hook recommendation (optional)', hint: 'Registers a safe UserPromptSubmit hook when selected. Off by default.' }
   ],
   ko: [
-    { value: 'commands', label: 'Claude Code 명령', hint: '/tink:setup, /tink:cast, /tink:verify, /tink:list, /tink:frog, /tink:weave, /tink:update' },
+    { value: 'commands', label: 'Claude Code 명령', hint: '/tink:setup, /tink:cast, /tink:verify, /tink:list, /tink:frog, /tink:weave, /tink:update' },
     { value: 'skill', label: 'Tink skill', hint: 'Claude Code가 읽는 Tink 작업 원칙' },
     { value: 'harnesses', label: '기본 harness', hint: '재사용 작업 템플릿' },
     { value: 'memory', label: 'Memory 템플릿', hint: '승인된 실수/선호/교훈 파일' },
     { value: 'hook', label: 'Hook 추천 (선택)', hint: '선택하면 안전한 UserPromptSubmit hook으로 등록합니다. 기본 off.' }
   ],
   zh: [
-    { value: 'commands', label: 'Claude Code 命令', hint: '/tink:setup, /tink:cast, /tink:verify, /tink:list, /tink:frog, /tink:weave, /tink:update' },
+    { value: 'commands', label: 'Claude Code 命令', hint: '/tink:setup, /tink:cast, /tink:verify, /tink:list, /tink:frog, /tink:weave, /tink:update' },
     { value: 'skill', label: 'Tink skill', hint: 'Claude Code 读取的 Tink 工作规则' },
     { value: 'harnesses', label: '内置 harness', hint: '可复用任务模板' },
     { value: 'memory', label: 'Memory 模板', hint: '经批准的错误/偏好/经验文件' },
     { value: 'hook', label: 'Hook 推荐（可选）', hint: '选择后注册安全的 UserPromptSubmit hook。默认关闭。' }
   ]
-};
-
-const SURFACE_OPTIONS = {
-  en: [
-    { value: 'claude', label: 'Claude Code', hint: 'Install /tink:* commands, Claude skill, and optional hook support' },
-    { value: 'codex', label: 'Codex', hint: 'Install $tink:* skills into CODEX_HOME or ~/.codex' }
-  ],
-  ko: [
-    { value: 'claude', label: 'Claude Code', hint: '/tink:* 명령, Claude skill, 선택 hook 지원 설치' },
-    { value: 'codex', label: 'Codex', hint: '$tink:* skills를 CODEX_HOME 또는 ~/.codex에 설치' }
-  ],
-  zh: [
-    { value: 'claude', label: 'Claude Code', hint: 'Install /tink:* commands, Claude skill, and optional hook support' },
-    { value: 'codex', label: 'Codex', hint: 'Install $tink:* skills into CODEX_HOME or ~/.codex' }
-  ]
-};
-
-function argValue(name) {
-  const prefix = `${name}=`;
-  const found = args.find((arg) => arg.startsWith(prefix));
-  return found ? found.slice(prefix.length) : undefined;
-}
-
-function usage() {
-  console.log(`Tink installer for Claude Code and Codex\n\nUsage:\n  npx tink-harness@latest [install] [--scope=repo|global] [--global] [--lang=en|ko|zh] [--yes] [--with-hook] [--dry-run] [--force]\n  npx tink-harness@latest update [--scope=repo|global] [--global] [--lang=en|ko|zh] [--yes] [--dry-run] [--force]\n\nCommands:\n  install  Install Tink.\n  update   Update Tink to the latest templates. Keeps user-modified files.\n\nDefault interactive flow:\n  1. Select language\n  2. Show TINK wizard\n  3. Select Claude Code, Codex, or both\n  4. Select components\n  5. Select repo/global installation scope\n  6. Select git tracking policy for project state\n\nScopes:\n  repo    Install shared .tink files into the current project.\n  global  Install shared .tink files into your home directory.\n`);
-}
-
-function normalizeSurfaces(surfaces) {
-  const values = [...new Set(surfaces)];
-  if (values.some((value) => !validSurfaces.has(value))) {
-    console.error(`Invalid install surface: ${values.find((value) => !validSurfaces.has(value))}`);
-    usage();
-    process.exit(1);
-  }
-  if (values.includes('claude') && values.includes('codex')) return 'all';
-  return values[0] || 'claude';
-}
-
-function resolveDefaultSurfaces() {
-  if (argValue('--agent')) {
-    console.error('--agent is no longer supported. Run the interactive installer and select Claude Code, Codex, or both during setup.');
-    usage();
-    process.exit(1);
-  }
-
-  const envValue = process.env.TINK_INSTALL_SURFACES;
-  if (!envValue) return 'claude';
-  if (envValue.trim().toLowerCase() === 'all') return 'all';
-  return normalizeSurfaces(envValue.split(',').map((value) => value.trim().toLowerCase()).filter(Boolean));
-}
-
-function includesClaude(agent) {
-  return agent === 'claude' || agent === 'all';
-}
-
-function includesCodex(agent) {
-  return agent === 'codex' || agent === 'all';
-}
-
-function codexHome() {
-  return process.env.CODEX_HOME || path.join(os.homedir(), '.codex');
-}
-
-function componentOptionsFor(agent, language) {
-  const options = COMPONENTS[language].filter((item) => {
-    if (item.value === 'commands') return includesClaude(agent);
-    if (item.value === 'hook') return includesClaude(agent);
-    return true;
-  });
-  return options.map((item) => {
-    if (item.value !== 'skill') return item;
-    if (agent === 'codex') {
-      return {
-        value: 'skill',
-        label: language === 'ko' ? 'Codex Tink skills' : language === 'zh' ? 'Codex Tink skills' : 'Codex Tink skills',
-        hint: language === 'ko'
-          ? 'Codex가 $tink로 읽는 Tink 작업 원칙'
-          : language === 'zh'
-            ? 'Codex 通过 $tink 读取的 Tink 工作规则'
-            : 'Tink operating rules for Codex through $tink:*'
-      };
-    }
-    if (agent === 'all') {
-      return {
-        value: 'skill',
-        label: language === 'ko' ? 'Tink skills' : language === 'zh' ? 'Tink skills' : 'Tink skills',
-        hint: language === 'ko'
-          ? 'Claude Code와 Codex가 읽는 Tink 작업 원칙'
-          : language === 'zh'
-            ? 'Claude Code 和 Codex 读取的 Tink 工作规则'
-            : 'Tink operating rules for Claude Code and Codex'
-      };
-    }
-    return item;
-  });
-}
+};
+
+const SURFACE_OPTIONS = {
+  en: [
+    { value: 'claude', label: 'Claude Code', hint: 'Install /tink:* commands, Claude skill, and optional hook support' },
+    { value: 'codex', label: 'Codex', hint: 'Install $tink:* skills into CODEX_HOME or ~/.codex' }
+  ],
+  ko: [
+    { value: 'claude', label: 'Claude Code', hint: '/tink:* 명령, Claude skill, 선택 hook 지원 설치' },
+    { value: 'codex', label: 'Codex', hint: '$tink:* skills를 CODEX_HOME 또는 ~/.codex에 설치' }
+  ],
+  zh: [
+    { value: 'claude', label: 'Claude Code', hint: 'Install /tink:* commands, Claude skill, and optional hook support' },
+    { value: 'codex', label: 'Codex', hint: 'Install $tink:* skills into CODEX_HOME or ~/.codex' }
+  ]
+};
+
+function argValue(name) {
+  const prefix = `${name}=`;
+  const found = args.find((arg) => arg.startsWith(prefix));
+  return found ? found.slice(prefix.length) : undefined;
+}
+
+function usage() {
+  console.log(`Tink installer for Claude Code and Codex\n\nUsage:\n  npx tink-harness@latest [install] [--scope=repo|global] [--global] [--lang=en|ko|zh] [--yes] [--with-hook] [--dry-run] [--force]\n  npx tink-harness@latest update [--scope=repo|global] [--global] [--lang=en|ko|zh] [--yes] [--dry-run] [--force]\n\nCommands:\n  install  Install Tink.\n  update   Update Tink to the latest templates. Keeps user-modified files.\n\nDefault interactive flow:\n  1. Select language\n  2. Show TINK wizard\n  3. Select Claude Code, Codex, or both\n  4. Select components\n  5. Select repo/global installation scope\n  6. Select git tracking policy for project state\n\nScopes:\n  repo    Install shared .tink files into the current project.\n  global  Install shared .tink files into your home directory.\n`);
+}
+
+function normalizeSurfaces(surfaces) {
+  const values = [...new Set(surfaces)];
+  if (values.some((value) => !validSurfaces.has(value))) {
+    console.error(`Invalid install surface: ${values.find((value) => !validSurfaces.has(value))}`);
+    usage();
+    process.exit(1);
+  }
+  if (values.includes('claude') && values.includes('codex')) return 'all';
+  return values[0] || 'claude';
+}
+
+function resolveDefaultSurfaces() {
+  if (argValue('--agent')) {
+    console.error('--agent is no longer supported. Run the interactive installer and select Claude Code, Codex, or both during setup.');
+    usage();
+    process.exit(1);
+  }
+
+  const envValue = process.env.TINK_INSTALL_SURFACES;
+  if (!envValue) return 'claude';
+  if (envValue.trim().toLowerCase() === 'all') return 'all';
+  return normalizeSurfaces(envValue.split(',').map((value) => value.trim().toLowerCase()).filter(Boolean));
+}
+
+function includesClaude(agent) {
+  return agent === 'claude' || agent === 'all';
+}
+
+function includesCodex(agent) {
+  return agent === 'codex' || agent === 'all';
+}
+
+function codexHome() {
+  return process.env.CODEX_HOME || path.join(os.homedir(), '.codex');
+}
+
+function componentOptionsFor(agent, language) {
+  const options = COMPONENTS[language].filter((item) => {
+    if (item.value === 'commands') return includesClaude(agent);
+    if (item.value === 'hook') return includesClaude(agent);
+    return true;
+  });
+  return options.map((item) => {
+    if (item.value !== 'skill') return item;
+    if (agent === 'codex') {
+      return {
+        value: 'skill',
+        label: language === 'ko' ? 'Codex Tink skills' : language === 'zh' ? 'Codex Tink skills' : 'Codex Tink skills',
+        hint: language === 'ko'
+          ? 'Codex가 $tink로 읽는 Tink 작업 원칙'
+          : language === 'zh'
+            ? 'Codex 通过 $tink 读取的 Tink 工作规则'
+            : 'Tink operating rules for Codex through $tink:*'
+      };
+    }
+    if (agent === 'all') {
+      return {
+        value: 'skill',
+        label: language === 'ko' ? 'Tink skills' : language === 'zh' ? 'Tink skills' : 'Tink skills',
+        hint: language === 'ko'
+          ? 'Claude Code와 Codex가 읽는 Tink 작업 원칙'
+          : language === 'zh'
+            ? 'Claude Code 和 Codex 读取的 Tink 工作规则'
+            : 'Tink operating rules for Claude Code and Codex'
+      };
+    }
+    return item;
+  });
+}
 
 function colorLine(line, color) {
   if (!process.stdout.isTTY && !interactive) return line;
@@ -235,13 +242,24 @@ function displayPath(base, filePath) {
   return path.relative(base, filePath) || '.';
 }
 
-function isAlwaysUpdatePath(src) {
-  const rel = path.relative(root, src).replace(/\\/g, '/');
-  return rel.startsWith('templates/claude/commands/') ||
-    rel.startsWith('templates/claude/skills/') ||
-    rel.startsWith('templates/codex/skills/') ||
-    rel.startsWith('templates/tink/maintenance/');
-}
+function recordOperation(kind, base, filePath) {
+  operationLog[kind].push(displayPath(base, filePath).replace(/\\/g, '/'));
+}
+
+function shortList(items, emptyText = '- none') {
+  if (!items.length) return emptyText;
+  const shown = items.slice(0, 8).map((item) => `- ${item}`);
+  if (items.length > shown.length) shown.push(`- ...and ${items.length - shown.length} more`);
+  return shown.join('\n');
+}
+
+function isAlwaysUpdatePath(src) {
+  const rel = path.relative(root, src).replace(/\\/g, '/');
+  return rel.startsWith('templates/claude/commands/') ||
+    rel.startsWith('templates/claude/skills/') ||
+    rel.startsWith('templates/codex/skills/') ||
+    rel.startsWith('templates/tink/maintenance/');
+}
 
 function writeFileFromTemplate(src, dest, base) {
   const exists = fs.existsSync(dest);
@@ -254,6 +272,7 @@ function writeFileFromTemplate(src, dest, base) {
       }
       if (!isAlwaysUpdatePath(src)) {
         log.message(`keep user-modified ${displayPath(base, dest)}`);
+        recordOperation('preserved', base, dest);
         return;
       }
       // commands/skills/maintenance: always update to new version
@@ -263,6 +282,7 @@ function writeFileFromTemplate(src, dest, base) {
     }
   }
   log.message(`${dryRun ? 'would write' : (exists ? 'update' : 'write')} ${displayPath(base, dest)}`);
+  recordOperation(exists ? 'updated' : 'written', base, dest);
   if (!dryRun) {
     fs.mkdirSync(path.dirname(dest), { recursive: true });
     fs.copyFileSync(src, dest);
@@ -282,7 +302,7 @@ function copyDir(src, dest, base) {
   }
 }
 
-function copyTinkCommands(templateRoot, target) {
+function copyTinkCommands(templateRoot, target) {
   const commandSrc = path.join(templateRoot, 'claude/commands/tink');
   const commandDest = path.join(target, '.claude/commands/tink');
   const flatCommandDest = path.join(target, '.claude/commands');
@@ -294,6 +314,7 @@ function copyTinkCommands(templateRoot, target) {
     const legacy = path.join(flatCommandDest, name);
     if (fs.existsSync(legacy)) {
       log.message(`${dryRun ? 'would remove legacy' : 'remove legacy'} ${displayPath(target, legacy)}`);
+      recordOperation('removedLegacy', target, legacy);
       if (!dryRun) fs.rmSync(legacy, { force: true });
     }
   }
@@ -301,6 +322,7 @@ function copyTinkCommands(templateRoot, target) {
     const legacy = path.join(commandDest, name);
     if (fs.existsSync(legacy)) {
       log.message(`${dryRun ? 'would remove legacy' : 'remove legacy'} ${displayPath(target, legacy)}`);
+      recordOperation('removedLegacy', target, legacy);
       if (!dryRun) fs.rmSync(legacy, { force: true });
     }
   }
@@ -308,12 +330,14 @@ function copyTinkCommands(templateRoot, target) {
     const legacy = path.join(flatCommandDest, name);
     if (fs.existsSync(legacy)) {
       log.message(`${dryRun ? 'would remove legacy Tiny' : 'remove legacy Tiny'} ${displayPath(target, legacy)}`);
+      recordOperation('removedLegacy', target, legacy);
       if (!dryRun) fs.rmSync(legacy, { force: true });
     }
   }
   for (const legacyDir of legacyDirs) {
     if (fs.existsSync(legacyDir)) {
       log.message(`${dryRun ? 'would remove legacy Tiny' : 'remove legacy Tiny'} ${displayPath(target, legacyDir)}`);
+      recordOperation('removedLegacy', target, legacyDir);
       if (!dryRun) fs.rmSync(legacyDir, { recursive: true, force: true });
     }
   }
@@ -321,29 +345,31 @@ function copyTinkCommands(templateRoot, target) {
     if (!entry.isFile() || !entry.name.endsWith('.md')) continue;
     writeFileFromTemplate(path.join(commandSrc, entry.name), path.join(commandDest, entry.name), target);
   }
-}
-
-function removeLegacyCodexSkill(codexTarget) {
-  const legacyDir = path.join(codexTarget, 'skills/tink');
-  const legacySkill = path.join(legacyDir, 'SKILL.md');
-  if (!fs.existsSync(legacyDir)) return;
-
-  let shouldRemove = false;
-  if (fs.existsSync(legacySkill)) {
-    const text = fs.readFileSync(legacySkill, 'utf8');
-    shouldRemove = text.includes('name: tink') && text.includes('Tink');
-  }
-
-  if (!shouldRemove) {
-    log.message(`keep unknown ${displayPath(codexTarget, legacyDir)}`);
-    return;
-  }
-
-  log.message(`${dryRun ? 'would remove legacy' : 'remove legacy'} ${displayPath(codexTarget, legacyDir)}`);
-  if (!dryRun) fs.rmSync(legacyDir, { recursive: true, force: true });
-}
-
-function readJsonFile(filePath, fallback) {
+}
+
+function removeLegacyCodexSkill(codexTarget) {
+  const legacyDir = path.join(codexTarget, 'skills/tink');
+  const legacySkill = path.join(legacyDir, 'SKILL.md');
+  if (!fs.existsSync(legacyDir)) return;
+
+  let shouldRemove = false;
+  if (fs.existsSync(legacySkill)) {
+    const text = fs.readFileSync(legacySkill, 'utf8');
+    shouldRemove = text.includes('name: tink') && text.includes('Tink');
+  }
+
+  if (!shouldRemove) {
+    log.message(`keep unknown ${displayPath(codexTarget, legacyDir)}`);
+    recordOperation('keptUnknown', codexTarget, legacyDir);
+    return;
+  }
+
+  log.message(`${dryRun ? 'would remove legacy' : 'remove legacy'} ${displayPath(codexTarget, legacyDir)}`);
+  recordOperation('removedLegacy', codexTarget, legacyDir);
+  if (!dryRun) fs.rmSync(legacyDir, { recursive: true, force: true });
+}
+
+function readJsonFile(filePath, fallback) {
   if (!fs.existsSync(filePath)) return fallback;
   try {
     return JSON.parse(fs.readFileSync(filePath, 'utf8'));
@@ -381,42 +407,42 @@ function registerClaudeHook(target, scope, base) {
   writeJsonFile(settingsPath, settings, base);
 }
 
-function copySelected(scope, components, agent) {
-  const repoTarget = process.cwd();
-  const globalTarget = os.homedir();
-  const codexTarget = codexHome();
-  const target = scope === 'global' ? globalTarget : repoTarget;
-  const templateRoot = path.join(root, 'templates');
-
-  if (includesClaude(agent) && components.includes('commands')) {
-    copyTinkCommands(templateRoot, target);
-  }
-  if (components.includes('skill')) {
-    if (includesClaude(agent)) {
-      copyDir(path.join(templateRoot, 'claude/skills'), path.join(target, '.claude/skills'), target);
-    }
-    if (includesCodex(agent)) {
-      removeLegacyCodexSkill(codexTarget);
-      copyDir(path.join(templateRoot, 'codex/skills'), path.join(codexTarget, 'skills'), codexTarget);
-    }
-  }
-  if (components.includes('harnesses')) {
-    copyDir(path.join(templateRoot, 'tink/harnesses'), path.join(target, '.tink/harnesses'), target);
-    copyDir(path.join(templateRoot, 'tink/rules'), path.join(target, '.tink/rules'), target);
-    copyDir(path.join(templateRoot, 'tink/schemas'), path.join(target, '.tink/schemas'), target);
-    copyDir(path.join(templateRoot, 'tink/maintenance'), path.join(target, '.tink/maintenance'), target);
-    writeFileFromTemplate(path.join(templateRoot, 'tink/config.json'), path.join(target, '.tink/config.json'), target);
-  }
+function copySelected(scope, components, agent) {
+  const repoTarget = process.cwd();
+  const globalTarget = os.homedir();
+  const codexTarget = codexHome();
+  const target = scope === 'global' ? globalTarget : repoTarget;
+  const templateRoot = path.join(root, 'templates');
+
+  if (includesClaude(agent) && components.includes('commands')) {
+    copyTinkCommands(templateRoot, target);
+  }
+  if (components.includes('skill')) {
+    if (includesClaude(agent)) {
+      copyDir(path.join(templateRoot, 'claude/skills'), path.join(target, '.claude/skills'), target);
+    }
+    if (includesCodex(agent)) {
+      removeLegacyCodexSkill(codexTarget);
+      copyDir(path.join(templateRoot, 'codex/skills'), path.join(codexTarget, 'skills'), codexTarget);
+    }
+  }
+  if (components.includes('harnesses')) {
+    copyDir(path.join(templateRoot, 'tink/harnesses'), path.join(target, '.tink/harnesses'), target);
+    copyDir(path.join(templateRoot, 'tink/rules'), path.join(target, '.tink/rules'), target);
+    copyDir(path.join(templateRoot, 'tink/schemas'), path.join(target, '.tink/schemas'), target);
+    copyDir(path.join(templateRoot, 'tink/maintenance'), path.join(target, '.tink/maintenance'), target);
+    writeFileFromTemplate(path.join(templateRoot, 'tink/config.json'), path.join(target, '.tink/config.json'), target);
+  }
   if (components.includes('memory')) {
     copyDir(path.join(templateRoot, 'tink/memory'), path.join(target, '.tink/memory'), target);
   }
-  if (includesClaude(agent) && components.includes('hook')) {
-    copyDir(path.join(templateRoot, 'tink/hooks'), path.join(target, '.tink/hooks'), target);
-    registerClaudeHook(target, scope, target);
-  }
-
-  return { repoTarget, globalTarget, codexTarget, installTarget: target };
-}
+  if (includesClaude(agent) && components.includes('hook')) {
+    copyDir(path.join(templateRoot, 'tink/hooks'), path.join(target, '.tink/hooks'), target);
+    registerClaudeHook(target, scope, target);
+  }
+
+  return { repoTarget, globalTarget, codexTarget, installTarget: target };
+}
 
 function updateGitignore(target, policy) {
   if (policy === 'all') {
@@ -441,31 +467,64 @@ function updateGitignore(target, policy) {
   }
 }
 
-function patchConfig(target, scope, hookScope, language) {
+function patchConfig(target, scope, hookScope, language) {
   const configPath = path.join(target, '.tink/config.json');
   if (!fs.existsSync(configPath) || dryRun) return;
+  const rel = displayPath(target, configPath).replace(/\\/g, '/');
+  if (isUpdate && !force && operationLog.preserved.includes(rel)) return;
   const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
   config.install_scope = scope;
   config.hook_scope = hookScope;
   config.language = language;
   fs.writeFileSync(configPath, `${JSON.stringify(config, null, 2)}\n`);
-}
-
-function nextStepFor(agent) {
-  if (agent === 'codex') {
-    return 'Next: open Codex and use $tink:cast <task> to start. Use $tink:setup only to review or change settings.';
-  }
-  if (agent === 'all') {
-    return 'Next: in Claude Code run /tink:cast <task>, or in Codex use $tink:cast <task>.';
-  }
-  return 'Next: open Claude Code and run /tink:cast <task> to start. Run /tink:setup only to review or change settings.';
-}
-
-function doneLineFor(agent) {
-  if (agent === 'codex') return '\nDone. Open Codex and use $tink:cast <task> to start.';
-  if (agent === 'all') return '\nDone. Use /tink:cast <task> in Claude Code or $tink:cast <task> in Codex to start.';
-  return '\nDone. Open Claude Code and run /tink:cast <task> to start.';
-}
+}
+
+function nextStepFor(agent) {
+  if (agent === 'codex') {
+    return 'Next: open Codex and use $tink:cast <task> to start. Use $tink:setup only to review or change settings.';
+  }
+  if (agent === 'all') {
+    return 'Next: in Claude Code run /tink:cast <task>, or in Codex use $tink:cast <task>.';
+  }
+  return 'Next: open Claude Code and run /tink:cast <task> to start. Run /tink:setup only to review or change settings.';
+}
+
+function doneLineFor(agent) {
+  if (agent === 'codex') return '\nDone. Open Codex and use $tink:cast <task> to start.';
+  if (agent === 'all') return '\nDone. Use /tink:cast <task> in Claude Code or $tink:cast <task> in Codex to start.';
+  return '\nDone. Open Claude Code and run /tink:cast <task> to start.';
+}
+
+function updateResultSummary(agent, targets) {
+  const locations = [
+    includesClaude(agent) ? `Claude Code commands: ${path.join(targets.installTarget, '.claude/commands/tink')}` : null,
+    includesClaude(agent) ? `Claude Code skill: ${path.join(targets.installTarget, '.claude/skills/tink')}` : null,
+    includesCodex(agent) ? `Codex skills: ${path.join(targets.codexTarget, 'skills')}` : null,
+    `Tink shared files: ${path.join(targets.installTarget, '.tink')}`
+  ].filter(Boolean);
+
+  return [
+    'Update Result Summary',
+    `Surfaces: ${agent}`,
+    `Install target: ${targets.installTarget}`,
+    includesCodex(agent) ? `Codex target: ${targets.codexTarget}` : null,
+    '',
+    'Updated or added:',
+    shortList([...operationLog.updated, ...operationLog.written]),
+    '',
+    'Preserved user-modified files:',
+    shortList(operationLog.preserved),
+    '',
+    'Removed legacy paths:',
+    shortList(operationLog.removedLegacy),
+    operationLog.keptUnknown.length ? ['', 'Kept unknown legacy-looking paths:', shortList(operationLog.keptUnknown)] : null,
+    '',
+    'Installed locations:',
+    locations.map((item) => `- ${item}`).join('\n'),
+    '',
+    `Next: ${nextStepFor(agent).replace(/^Next: /, '')}`
+  ].flat().filter((line) => line !== null).join('\n');
+}
 
 function detectLanguage() {
   const envLang = (process.env.LANG || process.env.LANGUAGE || process.env.LC_ALL || '').toLowerCase();
@@ -474,27 +533,27 @@ function detectLanguage() {
   return 'en';
 }
 
-async function resolveChoices() {
-  let agent = resolveDefaultSurfaces();
-  let scope = args.includes('--global') ? 'global' : (argValue('--scope') || undefined);
-  let language = argValue('--lang') || argValue('--language') || detectLanguage();
-  if (scope && !['repo', 'global'].includes(scope)) {
-    console.error(`Invalid scope: ${scope}`);
-    usage();
+async function resolveChoices() {
+  let agent = resolveDefaultSurfaces();
+  let scope = args.includes('--global') ? 'global' : (argValue('--scope') || undefined);
+  let language = argValue('--lang') || argValue('--language') || detectLanguage();
+  if (scope && !['repo', 'global'].includes(scope)) {
+    console.error(`Invalid scope: ${scope}`);
+    usage();
     process.exit(1);
-  }
-  if (!['en', 'ko', 'zh'].includes(language)) language = 'en';
-
-  let components = componentOptionsFor(agent, language).map((item) => item.value).filter((value) => value !== 'hook');
-  if (includesClaude(agent) && args.includes('--with-hook')) components.push('hook');
-  let gitPolicy = 'harnesses';
-  let hookScope = 'off';
-
-  if (!interactive) {
-    scope = scope || 'repo';
-    if (components.includes('hook')) hookScope = scope;
-    return { agent, scope, components, gitPolicy, hookScope, language };
-  }
+  }
+  if (!['en', 'ko', 'zh'].includes(language)) language = 'en';
+
+  let components = componentOptionsFor(agent, language).map((item) => item.value).filter((value) => value !== 'hook');
+  if (includesClaude(agent) && args.includes('--with-hook')) components.push('hook');
+  let gitPolicy = 'harnesses';
+  let hookScope = 'off';
+
+  if (!interactive) {
+    scope = scope || 'repo';
+    if (components.includes('hook')) hookScope = scope;
+    return { agent, scope, components, gitPolicy, hookScope, language };
+  }
 
   language = handleCancel(await select({
     message: COPY[language].language,
@@ -525,23 +584,23 @@ async function resolveChoices() {
     language === 'ko' ? '항목 설명' : language === 'zh' ? '项目说明' : 'Component notes'
   );
 
-  agent = normalizeSurfaces(handleCancel(await multiselect({
-    message: language === 'ko'
-      ? '설치할 agent surface를 선택하세요 (space로 토글)'
-      : 'Select agent surfaces to install (space to toggle)',
-    options: SURFACE_OPTIONS[language],
-    initialValues: agent === 'all' ? ['claude', 'codex'] : [agent],
-    required: true
-  })));
-  components = componentOptionsFor(agent, language).map((item) => item.value).filter((value) => value !== 'hook');
-  if (includesClaude(agent) && args.includes('--with-hook')) components.push('hook');
-
-  components = handleCancel(await multiselect({
-    message: copy.components,
-    options: componentOptionsFor(agent, language),
-    initialValues: components,
-    required: true
-  }));
+  agent = normalizeSurfaces(handleCancel(await multiselect({
+    message: language === 'ko'
+      ? '설치할 agent surface를 선택하세요 (space로 토글)'
+      : 'Select agent surfaces to install (space to toggle)',
+    options: SURFACE_OPTIONS[language],
+    initialValues: agent === 'all' ? ['claude', 'codex'] : [agent],
+    required: true
+  })));
+  components = componentOptionsFor(agent, language).map((item) => item.value).filter((value) => value !== 'hook');
+  if (includesClaude(agent) && args.includes('--with-hook')) components.push('hook');
+
+  components = handleCancel(await multiselect({
+    message: copy.components,
+    options: componentOptionsFor(agent, language),
+    initialValues: components,
+    required: true
+  }));
 
   scope = handleCancel(await select({
     message: copy.scope,
@@ -593,8 +652,8 @@ async function resolveChoices() {
     }));
   }
 
-  if (includesClaude(agent) && components.includes('hook')) {
-    hookScope = scope;
+  if (includesClaude(agent) && components.includes('hook')) {
+    hookScope = scope;
     note(
       language === 'ko'
         ? `Hook 추천을 ${scope} 범위의 Claude Code UserPromptSubmit에 등록합니다. 추가 범위 질문은 하지 않습니다. 작업 실행/저장 없이 일반 프롬프트에서만 Tink 사용을 추천합니다.`
@@ -605,8 +664,8 @@ async function resolveChoices() {
     );
   }
 
-  return { agent, scope, components, gitPolicy, hookScope, language };
-}
+  return { agent, scope, components, gitPolicy, hookScope, language };
+}
 
 async function main() {
   if (command === 'help' || args.includes('--help')) {
@@ -620,18 +679,18 @@ async function main() {
     process.exit(1);
   }
 
-  const { agent, scope, components, gitPolicy, hookScope, language } = await resolveChoices();
-
-  if (!interactive) {
-    console.log(`Installing Tink for ${agent === 'claude' ? 'Claude Code' : agent === 'codex' ? 'Codex' : 'Claude Code and Codex'}`);
-    console.log(`Source: ${source}`);
-    console.log(`surfaces ${agent}`);
-    console.log(`language ${language}`);
-    console.log(`scope ${scope}`);
-    console.log(`components ${components.join(', ')}`);
-  }
-
-  const targets = copySelected(scope, components, agent);
+  const { agent, scope, components, gitPolicy, hookScope, language } = await resolveChoices();
+
+  if (!interactive) {
+    console.log(`${isUpdate ? 'Updating' : 'Installing'} Tink for ${agent === 'claude' ? 'Claude Code' : agent === 'codex' ? 'Codex' : 'Claude Code and Codex'}`);
+    console.log(`Source: ${source}`);
+    console.log(`surfaces ${agent}`);
+    console.log(`language ${language}`);
+    console.log(`scope ${scope}`);
+    console.log(`components ${components.join(', ')}`);
+  }
+
+  const targets = copySelected(scope, components, agent);
 
   if (scope === 'repo' && components.some((item) => ['harnesses', 'memory', 'hook'].includes(item))) {
     updateGitignore(targets.repoTarget, gitPolicy);
@@ -640,25 +699,27 @@ async function main() {
   }
   patchConfig(targets.installTarget, scope, hookScope, language);
 
-  const summary = [
-    `Language: ${language}`,
-    `Surfaces: ${agent}`,
-    `Scope: ${scope}`,
-    `Install target: ${targets.installTarget}`,
-    includesCodex(agent) ? `Codex target: ${targets.codexTarget}` : null,
-    `Components: ${components.join(', ')}`,
-    `Hook scope: ${hookScope}`,
-    nextStepFor(agent)
-  ].filter(Boolean).join('\n');
+  const summary = [
+    `Language: ${language}`,
+    `Surfaces: ${agent}`,
+    `Scope: ${scope}`,
+    `Install target: ${targets.installTarget}`,
+    includesCodex(agent) ? `Codex target: ${targets.codexTarget}` : null,
+    `Components: ${components.join(', ')}`,
+    `Hook scope: ${hookScope}`,
+    nextStepFor(agent)
+  ].filter(Boolean).join('\n');
 
   if (interactive) {
     note(summary, COPY[language].installed);
+    if (isUpdate) note(updateResultSummary(agent, targets), 'Update Result Summary');
     outro(COPY[language].done);
-  } else {
-    console.log(`\n${summary}`);
-    console.log(doneLineFor(agent));
-  }
-}
+  } else {
+    console.log(`\n${summary}`);
+    if (isUpdate) console.log(`\n${updateResultSummary(agent, targets)}`);
+    console.log(doneLineFor(agent));
+  }
+}
 
 main().catch((error) => {
   console.error(error instanceof Error ? error.message : String(error));