instrlint 0.1.2 → 0.1.3

package/dist/cli.js

@@ -5,6 +5,7 @@ import { Command } from "commander";
 
 // src/commands/run-command.ts
 import { execSync } from "child_process";
+import { createInterface } from "readline";
 import { basename as basename2 } from "path";
 import chalk4 from "chalk";
 
@@ -1565,9 +1566,13 @@ var en_default = {
   "ci.writtenTo": "\u2192 written to {{file}}",
   "initCi.created": "\u2713 Created {{path}}",
   "initCi.alreadyExists": "{{path}} already exists. Use --force to overwrite.",
-  "install.installed": "\u2713 Installed to {{path}}",
+  "install.installed": "\u2713 Installed to {{path}}\n  \u2192 Restart Claude Code to activate /instrlint",
   "install.alreadyExists": "{{path}} already exists. Use --force to overwrite.",
   "install.unknownTarget": "Specify --claude-code or --codex",
+  "install.outdatedTitle": "instrlint skill is outdated",
+  "install.outdatedVersions": "installed: {{installed}} \u2192 current: {{current}}",
+  "install.updateCmd": "npx instrlint install {{flag}} --force",
+  "install.updatePrompt": "Update skill now?",
   "fix.manualActions": "MANUAL ACTIONS NEEDED",
   "fix.hookCreate": "Add to .claude/settings.json:",
   "fix.hookWarning": "\u26A0 Hook executes shell commands \u2014 review carefully before adding",
@@ -1666,9 +1671,13 @@ var zh_TW_default = {
   "ci.writtenTo": "\u2192 \u5DF2\u5BEB\u5165 {{file}}",
   "initCi.created": "\u2713 \u5DF2\u5EFA\u7ACB {{path}}",
   "initCi.alreadyExists": "{{path}} \u5DF2\u5B58\u5728\u3002\u4F7F\u7528 --force \u8986\u84CB\u3002",
-  "install.installed": "\u2713 \u5DF2\u5B89\u88DD\u81F3 {{path}}",
+  "install.installed": "\u2713 \u5DF2\u5B89\u88DD\u81F3 {{path}}\n  \u2192 \u91CD\u65B0\u555F\u52D5 Claude Code \u5F8C\u5373\u53EF\u4F7F\u7528 /instrlint",
   "install.alreadyExists": "{{path}} \u5DF2\u5B58\u5728\u3002\u4F7F\u7528 --force \u8986\u84CB\u3002",
   "install.unknownTarget": "\u8ACB\u6307\u5B9A --claude-code \u6216 --codex",
+  "install.outdatedTitle": "instrlint skill \u7248\u672C\u904E\u820A",
+  "install.outdatedVersions": "\u5DF2\u5B89\u88DD\uFF1A{{installed}} \u2192 \u6700\u65B0\uFF1A{{current}}",
+  "install.updateCmd": "npx instrlint install {{flag}} --force",
+  "install.updatePrompt": "\u662F\u5426\u7ACB\u5373\u66F4\u65B0\uFF1F",
   "fix.manualActions": "\u9700\u8981\u624B\u52D5\u64CD\u4F5C",
   "fix.hookCreate": "\u52A0\u5165 .claude/settings.json\uFF1A",
   "fix.hookWarning": "\u26A0 Hook \u6703\u57F7\u884C shell command\uFF0C\u8ACB\u4ED4\u7D30\u78BA\u8A8D\u5F8C\u518D\u52A0\u5165",
@@ -2263,6 +2272,136 @@ function markdownStructureSuggestions(suggestions, projectRoot) {
   return lines;
 }
 
+// src/utils/skill-version.ts
+import { existsSync as existsSync7, readFileSync as readFileSync8 } from "fs";
+import { join as join7 } from "path";
+import { homedir } from "os";
+var CURRENT_VERSION = "0.1.3";
+var VERSION_RE = /^instrlint-version:\s*(.+)$/m;
+function extractVersion(content) {
+  const m = VERSION_RE.exec(content);
+  return m ? m[1].trim() : null;
+}
+function readInstalledVersion(path) {
+  if (!existsSync7(path)) return null;
+  try {
+    return extractVersion(readFileSync8(path, "utf8"));
+  } catch {
+    return null;
+  }
+}
+function checkSkillUpdate(projectRoot) {
+  const candidates = [
+    {
+      path: join7(projectRoot, ".claude", "commands", "instrlint.md"),
+      isProject: true
+    },
+    {
+      path: join7(homedir(), ".claude", "commands", "instrlint.md"),
+      isProject: false
+    }
+  ];
+  for (const { path, isProject } of candidates) {
+    const installed = readInstalledVersion(path);
+    if (installed !== null && installed !== CURRENT_VERSION) {
+      return {
+        installedVersion: installed,
+        currentVersion: CURRENT_VERSION,
+        installPath: path,
+        isProject
+      };
+    }
+  }
+  return null;
+}
+function injectVersion(content, version) {
+  return content.replace(/^(---\n[\s\S]*?)(---)$/m, `$1instrlint-version: ${version}
+$2`);
+}
+
+// src/commands/install-command.ts
+import { existsSync as existsSync8, mkdirSync, readFileSync as readFileSync9, writeFileSync as writeFileSync2 } from "fs";
+import { join as join8 } from "path";
+import { homedir as homedir2 } from "os";
+import { fileURLToPath } from "url";
+function resolveSkillFile(target) {
+  const thisFile = fileURLToPath(import.meta.url);
+  const subDir = target === "claude-code" ? "claude-code" : "codex";
+  for (const levels of [2, 3]) {
+    const parts = Array(levels).fill("..");
+    const candidate = join8(thisFile, ...parts, "skills", subDir, "SKILL.md");
+    if (existsSync8(candidate)) return candidate;
+  }
+  return join8(thisFile, "..", "..", "skills", subDir, "SKILL.md");
+}
+function readSkillContent(target) {
+  const skillPath = resolveSkillFile(target);
+  try {
+    const raw = readFileSync9(skillPath, "utf8");
+    return injectVersion(raw, CURRENT_VERSION);
+  } catch {
+    throw new Error(
+      `Could not read skill file at ${skillPath}. Make sure the package is properly installed.`
+    );
+  }
+}
+function installClaudeCode(content, projectRoot, isProject, force, output) {
+  const targetDir = isProject ? join8(projectRoot, ".claude", "commands") : join8(homedir2(), ".claude", "commands");
+  const targetPath = join8(targetDir, "instrlint.md");
+  if (existsSync8(targetPath) && !force) {
+    output.error(t("install.alreadyExists", { path: targetPath }));
+    return { exitCode: 1, errorMessage: "file already exists" };
+  }
+  mkdirSync(targetDir, { recursive: true });
+  writeFileSync2(targetPath, content, "utf8");
+  output.log(t("install.installed", { path: targetPath }));
+  return { exitCode: 0 };
+}
+function installCodex(content, projectRoot, force, output) {
+  const targetDir = join8(projectRoot, ".agents", "skills", "instrlint");
+  const targetPath = join8(targetDir, "SKILL.md");
+  if (existsSync8(targetPath) && !force) {
+    output.error(t("install.alreadyExists", { path: targetPath }));
+    return { exitCode: 1, errorMessage: "file already exists" };
+  }
+  mkdirSync(targetDir, { recursive: true });
+  writeFileSync2(targetPath, content, "utf8");
+  output.log(t("install.installed", { path: targetPath }));
+  return { exitCode: 0 };
+}
+function runInstall(opts, output = console) {
+  const projectRoot = opts.projectRoot ?? process.cwd();
+  const force = opts.force ?? false;
+  if (opts.claudeCode) {
+    let content;
+    try {
+      content = readSkillContent("claude-code");
+    } catch (err) {
+      output.error(String(err));
+      return { exitCode: 1, errorMessage: String(err) };
+    }
+    return installClaudeCode(
+      content,
+      projectRoot,
+      opts.project ?? false,
+      force,
+      output
+    );
+  }
+  if (opts.codex) {
+    let content;
+    try {
+      content = readSkillContent("codex");
+    } catch (err) {
+      output.error(String(err));
+      return { exitCode: 1, errorMessage: String(err) };
+    }
+    return installCodex(content, projectRoot, force, output);
+  }
+  output.error(t("install.unknownTarget"));
+  return { exitCode: 1, errorMessage: "no target specified" };
+}
+
 // src/commands/run-command.ts
 function isGitClean(cwd) {
   try {
@@ -2352,6 +2491,7 @@ async function runAll(opts, output = console) {
     printStructureSuggestions(suggestions, projectRoot, output);
     return { exitCode: 0 };
   }
+  const skillUpdate = checkSkillUpdate(projectRoot);
   if (opts.format === "json") {
     output.log(reportJson(report));
     return { exitCode: 0 };
@@ -2359,12 +2499,52 @@ async function runAll(opts, output = console) {
   if (opts.format === "markdown") {
     const mdSuggestions = buildStructureSuggestions(allFindings);
     const mdExtra = markdownStructureSuggestions(mdSuggestions, projectRoot);
-    output.log(reportMarkdown(report, mdExtra));
+    const updateSection = skillUpdate ? [
+      "",
+      `> \u26A0\uFE0F **${t("install.outdatedTitle")}** (${t("install.outdatedVersions", { installed: skillUpdate.installedVersion, current: skillUpdate.currentVersion })})`,
+      `> \`${t("install.updateCmd", { flag: skillUpdate.isProject ? "--claude-code --project" : "--claude-code" })}\``
+    ] : [];
+    output.log(reportMarkdown(report, [...mdExtra, ...updateSection]));
     return { exitCode: 0 };
   }
   printCombinedTerminal(report, output);
+  if (skillUpdate && process.stdout.isTTY) {
+    output.log("");
+    output.log(
+      `  ${chalk4.yellow("\u26A0")}  ${chalk4.bold(t("install.outdatedTitle"))} (${t("install.outdatedVersions", { installed: skillUpdate.installedVersion, current: skillUpdate.currentVersion })})`
+    );
+    const confirmed = await promptYesNo(
+      `  ${t("install.updatePrompt")}`,
+      output
+    );
+    if (confirmed) {
+      runInstall(
+        {
+          claudeCode: true,
+          project: skillUpdate.isProject,
+          force: true,
+          projectRoot
+        },
+        output
+      );
+    }
+    output.log("");
+  }
   return { exitCode: 0 };
 }
+function promptYesNo(question, output) {
+  return new Promise((resolve) => {
+    const rl = createInterface({
+      input: process.stdin,
+      output: process.stdout
+    });
+    rl.question(`${question} ${chalk4.gray("[Y/n]")} `, (answer) => {
+      rl.close();
+      const trimmed = answer.trim().toLowerCase();
+      resolve(trimmed === "" || trimmed === "y");
+    });
+  });
+}
 
 // src/commands/deadrules-command.ts
 import chalk5 from "chalk";
@@ -2525,7 +2705,7 @@ async function runStructure(opts, output = console) {
 }
 
 // src/commands/ci-command.ts
-import { writeFileSync as writeFileSync2 } from "fs";
+import { writeFileSync as writeFileSync3 } from "fs";
 import { basename as basename3 } from "path";
 
 // src/reporters/sarif.ts
@@ -2653,7 +2833,7 @@ async function runCi(opts, output = console) {
     formatted = reportJson(report);
   }
   if (opts.output != null) {
-    writeFileSync2(opts.output, formatted, "utf8");
+    writeFileSync3(opts.output, formatted, "utf8");
     const pass = !shouldFail(allFindings, failOn);
     const statusKey = pass ? "ci.passed" : "ci.failed";
     output.error(
@@ -2667,8 +2847,8 @@ async function runCi(opts, output = console) {
 }
 
 // src/commands/init-ci-command.ts
-import { existsSync as existsSync7, mkdirSync, writeFileSync as writeFileSync3 } from "fs";
-import { join as join7 } from "path";
+import { existsSync as existsSync9, mkdirSync as mkdirSync2, writeFileSync as writeFileSync4 } from "fs";
+import { join as join9 } from "path";
 function githubWorkflow() {
   return `name: instrlint
 
@@ -2737,14 +2917,14 @@ instrlint:
 function runInitCi(opts, output = console) {
   const projectRoot = opts.projectRoot ?? process.cwd();
   if (opts.github) {
-    const workflowDir = join7(projectRoot, ".github", "workflows");
-    const workflowPath = join7(workflowDir, "instrlint.yml");
-    if (existsSync7(workflowPath) && !opts.force) {
+    const workflowDir = join9(projectRoot, ".github", "workflows");
+    const workflowPath = join9(workflowDir, "instrlint.yml");
+    if (existsSync9(workflowPath) && !opts.force) {
       output.error(t("initCi.alreadyExists", { path: workflowPath }));
       return { exitCode: 1, errorMessage: "file already exists" };
     }
-    mkdirSync(workflowDir, { recursive: true });
-    writeFileSync3(workflowPath, githubWorkflow(), "utf8");
+    mkdirSync2(workflowDir, { recursive: true });
+    writeFileSync4(workflowPath, githubWorkflow(), "utf8");
     output.log(t("initCi.created", { path: workflowPath }));
     return { exitCode: 0 };
   }
@@ -2756,88 +2936,6 @@ function runInitCi(opts, output = console) {
   return { exitCode: 1, errorMessage: "no target specified" };
 }
 
-// src/commands/install-command.ts
-import { existsSync as existsSync8, mkdirSync as mkdirSync2, readFileSync as readFileSync8, writeFileSync as writeFileSync4 } from "fs";
-import { join as join8 } from "path";
-import { homedir } from "os";
-import { fileURLToPath } from "url";
-function resolveSkillFile(target) {
-  const thisFile = fileURLToPath(import.meta.url);
-  const subDir = target === "claude-code" ? "claude-code" : "codex";
-  for (const levels of [2, 3]) {
-    const parts = Array(levels).fill("..");
-    const candidate = join8(thisFile, ...parts, "skills", subDir, "SKILL.md");
-    if (existsSync8(candidate)) return candidate;
-  }
-  return join8(thisFile, "..", "..", "skills", subDir, "SKILL.md");
-}
-function readSkillContent(target) {
-  const skillPath = resolveSkillFile(target);
-  try {
-    return readFileSync8(skillPath, "utf8");
-  } catch {
-    throw new Error(
-      `Could not read skill file at ${skillPath}. Make sure the package is properly installed.`
-    );
-  }
-}
-function installClaudeCode(content, projectRoot, isProject, force, output) {
-  const targetDir = isProject ? join8(projectRoot, ".claude", "commands") : join8(homedir(), ".claude", "commands");
-  const targetPath = join8(targetDir, "instrlint.md");
-  if (existsSync8(targetPath) && !force) {
-    output.error(t("install.alreadyExists", { path: targetPath }));
-    return { exitCode: 1, errorMessage: "file already exists" };
-  }
-  mkdirSync2(targetDir, { recursive: true });
-  writeFileSync4(targetPath, content, "utf8");
-  output.log(t("install.installed", { path: targetPath }));
-  return { exitCode: 0 };
-}
-function installCodex(content, projectRoot, force, output) {
-  const targetDir = join8(projectRoot, ".agents", "skills", "instrlint");
-  const targetPath = join8(targetDir, "SKILL.md");
-  if (existsSync8(targetPath) && !force) {
-    output.error(t("install.alreadyExists", { path: targetPath }));
-    return { exitCode: 1, errorMessage: "file already exists" };
-  }
-  mkdirSync2(targetDir, { recursive: true });
-  writeFileSync4(targetPath, content, "utf8");
-  output.log(t("install.installed", { path: targetPath }));
-  return { exitCode: 0 };
-}
-function runInstall(opts, output = console) {
-  const projectRoot = opts.projectRoot ?? process.cwd();
-  const force = opts.force ?? false;
-  if (opts.claudeCode) {
-    let content;
-    try {
-      content = readSkillContent("claude-code");
-    } catch (err) {
-      output.error(String(err));
-      return { exitCode: 1, errorMessage: String(err) };
-    }
-    return installClaudeCode(
-      content,
-      projectRoot,
-      opts.project ?? false,
-      force,
-      output
-    );
-  }
-  if (opts.codex) {
-    let content;
-    try {
-      content = readSkillContent("codex");
-    } catch (err) {
-      output.error(String(err));
-      return { exitCode: 1, errorMessage: String(err) };
-    }
-    return installCodex(content, projectRoot, force, output);
-  }
-  output.error(t("install.unknownTarget"));
-  return { exitCode: 1, errorMessage: "no target specified" };
-}
-
 // src/cli.ts
 var program = new Command();
 program.enablePositionalOptions().name("instrlint").description(
@@ -2855,27 +2953,27 @@ program.command("budget").description("Token budget analysis only").option(
   "--format <type>",
   "output format (terminal|json|markdown)",
   "terminal"
-).option("--tool <name>", "force tool detection (claude-code|codex|cursor)").action(async function() {
+).option("--lang <locale>", "output language (en|zh-TW)").option("--tool <name>", "force tool detection (claude-code|codex|cursor)").action(async function() {
   const opts = this.opts();
-  const lang = this.parent?.opts()?.lang;
+  const lang = opts.lang ?? this.parent?.opts()?.lang;
   const result = await runBudget({
     ...opts,
     ...lang !== void 0 && { lang }
   });
   if (result.exitCode !== 0) process.exit(result.exitCode);
 });
-program.command("deadrules").description("Dead rule detection only").option("--format <type>", "output format (terminal|json)", "terminal").option("--tool <name>", "force tool detection (claude-code|codex|cursor)").action(async function() {
+program.command("deadrules").description("Dead rule detection only").option("--format <type>", "output format (terminal|json)", "terminal").option("--lang <locale>", "output language (en|zh-TW)").option("--tool <name>", "force tool detection (claude-code|codex|cursor)").action(async function() {
   const opts = this.opts();
-  const lang = this.parent?.opts()?.lang;
+  const lang = opts.lang ?? this.parent?.opts()?.lang;
   const result = await runDeadRules({
     ...opts,
     ...lang !== void 0 && { lang }
   });
   if (result.exitCode !== 0) process.exit(result.exitCode);
 });
-program.command("structure").description("Structural analysis only").option("--format <type>", "output format (terminal|json)", "terminal").option("--tool <name>", "force tool detection (claude-code|codex|cursor)").action(async function() {
+program.command("structure").description("Structural analysis only").option("--format <type>", "output format (terminal|json)", "terminal").option("--lang <locale>", "output language (en|zh-TW)").option("--tool <name>", "force tool detection (claude-code|codex|cursor)").action(async function() {
   const opts = this.opts();
-  const lang = this.parent?.opts()?.lang;
+  const lang = opts.lang ?? this.parent?.opts()?.lang;
   const result = await runStructure({
     ...opts,
     ...lang !== void 0 && { lang }