agent-skill-installer 0.1.4 → 0.1.6

package/bin/cli.js

@@ -3,6 +3,7 @@
 const fs = require("node:fs/promises");
 const path = require("node:path");
 const os = require("node:os");
+const readline = require("node:readline/promises");
 
 const REPO_ROOT = path.resolve(__dirname, "..");
 const SKILLS_SOURCE_DIR = path.join(REPO_ROOT, "skills");
@@ -21,7 +22,7 @@ Commands:
 
 Options:
   --dest <path>          Custom destination directory.
-  --force                Overwrite destination if a skill already exists.
+  --force                Overwrite destination if a skill already exists (no prompt).
   -h, --help             Show this help.
 `);
 }
@@ -112,6 +113,33 @@ async function listSkills() {
   }
 }
 
+async function confirmOverwrite(name, targetPath) {
+  if (!process.stdin.isTTY || !process.stdout.isTTY) {
+    throw new Error(
+      `Destination already exists for "${name}": ${targetPath} (non-interactive mode, use --force to overwrite)`
+    );
+  }
+
+  const rl = readline.createInterface({
+    input: process.stdin,
+    output: process.stdout
+  });
+
+  try {
+    const answer = (
+      await rl.question(
+        `Skill "${name}" already exists at ${targetPath}. Overwrite it? [y/N] `
+      )
+    )
+      .trim()
+      .toLowerCase();
+
+    return answer === "y" || answer === "yes";
+  } finally {
+    rl.close();
+  }
+}
+
 async function installSkills(rawArgs) {
   const parsed = parseInstallArgs(rawArgs);
   if (parsed.help) {
@@ -137,13 +165,16 @@ async function installSkills(rawArgs) {
     const to = path.join(parsed.dest, name);
     const exists = await dirExists(to);
 
-    if (exists && !parsed.force) {
-      throw new Error(
-        `Destination already exists for "${name}": ${to} (use --force to overwrite)`
-      );
+    if (exists && parsed.force) {
+      await fs.rm(to, { recursive: true, force: true });
     }
 
-    if (exists && parsed.force) {
+    if (exists && !parsed.force) {
+      const shouldOverwrite = await confirmOverwrite(name, to);
+      if (!shouldOverwrite) {
+        console.log(`Skipped "${name}" (kept existing skill).`);
+        continue;
+      }
       await fs.rm(to, { recursive: true, force: true });
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-skill-installer",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "description": "Install one or more Codex skills from this package via npx.",
   "bin": {
     "agent-skill-installer": "bin/cli.js"

package/skills/git-commit-helper/SKILL.md

@@ -3,44 +3,46 @@ name: git-commit-helper
 description: "严格生成与校验中文 Git 提交信息，固定格式为 type(scope): summary，type 仅允许 feat/fix/refactor/perf/style/test/docs/chore，summary 必须用祈使句、不超过 50 字符、不得以句号结尾且不得包含 and/&/multiple changes。使用该技能于编写提交说明、检查提交文本是否合规，或执行使用 codex (codex-ice@gmail.com) 身份的提交时。"
 ---
 
-按固定规范生成中文提交信息，并在提交前进行格式校验。
-统一使用作者身份：`codex <codex-ice@gmail.com>`。
+生成中文提交信息，先校验再提交。统一使用作者身份：`codex <codex-ice@gmail.com>`。
 
 ## 工作流
 
-1. 先查看暂存变更，确认本次提交只表达一个意图：`git diff --cached --name-status`。
-2. 选择 `type`，必须是以下之一：`feat` `fix` `refactor` `perf` `style` `test` `docs` `chore`。
-3. 写首行：`<type>(<scope>): <summary>`。
-4. 首行后可选正文；若 `type=perf`，必须写正文说明优化原因。
-5. 使用脚本校验：`python3 scripts/validate_commit_message.py --file <msg-file>`。
-6. 通过校验后提交，并固定作者身份：
-   `git -c user.name='codex' -c user.email='codex-ice@gmail.com' commit -F <msg-file>`。
+1. 检查暂存区，只允许一个明确意图：
+`git diff --cached --name-status`
+2. 选择 `type`：`feat` `fix` `refactor` `perf` `style` `test` `docs` `chore`。
+3. 写提交消息文件，固定首行格式：
+`<type>(<scope>): <summary>`
+4. 若 `type=perf`，在空行后写正文并说明优化原因。
+5. 执行校验：
+`python3 skills/git-commit-helper/scripts/validate_commit_message.py --file <msg-file>`
+6. 校验通过后提交：
+`git -c user.name='codex' -c user.email='codex-ice@gmail.com' commit -F <msg-file>`
 
 ## 强制规则
 
-1. 格式必须严格为：`<type>(<scope>): <summary>`。
-2. `type` 只能从这 8 个值中选择：`feat` `fix` `refactor` `perf` `style` `test` `docs` `chore`。
-3. `summary` 必须满足：
-   - 使用祈使句。
-   - 不超过 50 字符。
-   - 不要句号（`。` 或 `.`）。
-   - 不要出现 `and`、`&`、`multiple changes`。
-   - 使用中文表达。
-4. 提交文本（`summary` 与可选正文）使用中文。
-5. `perf` 类型必须在正文说明优化原因。
+1. 首行必须是：`<type>(<scope>): <summary>`。
+2. `scope` 只允许小写字母、数字、连字符（例如 `album`、`build-cache`）。
+3. `summary` 必须使用中文祈使句，且不超过 50 字符。
+4. `summary` 不能以 `。` 或 `.` 结尾。
+5. `summary` 不能包含 `and`、`&`、`multiple changes`。
+6. `summary` 与正文必须使用中文表达。
+7. 若包含正文，第 2 行必须是空行。
+8. `perf` 必须包含正文说明优化原因。
 
-## 示例
+## 快速模板
 
-正确：
-- `fix(album): 避免滚动时重复渲染`
-- `docs(api): 补充鉴权参数说明`
+```text
+fix(scope): 优化某个单一行为
 
-错误：
-- `update code`
-- `fix bug and refactor logic`
-- `optimize`
+仅在需要时写正文，说明原因或背景
+```
 
-## 脚本资源
+## 失败处理
 
-- `scripts/validate_commit_message.py`
-  校验提交文本是否符合本技能的强制规则。
+1. 阅读脚本返回的逐条错误。
+2. 仅修改被指出的问题，避免一次改多个意图。
+3. 重新执行校验，直到输出“校验通过”。
+
+## 资源
+
+- `skills/git-commit-helper/scripts/validate_commit_message.py`：提交信息规则校验器

package/skills/git-commit-helper/agents/openai.yaml

@@ -1,4 +1,4 @@
 interface:
   display_name: "git-commit-helper"
-  short_description: "Generate Chinese Git commit messages with strict checks"
-  default_prompt: "Use $git-commit-helper to draft a Chinese commit message and validate it before committing as codex <codex-ice@gmail.com>."
+  short_description: "严格生成并校验中文 Git 提交信息"
+  default_prompt: "Use $git-commit-helper to draft a Chinese commit message in `type(scope): summary`, validate it with the bundled script, then commit as codex <codex-ice@gmail.com>."

package/skills/git-commit-helper/scripts/validate_commit_message.py

@@ -19,6 +19,18 @@ FORBIDDEN_SUMMARY_PATTERNS = (
     re.compile(r"&"),
     re.compile(r"multiple changes", re.IGNORECASE),
 )
+PAST_TENSE_PREFIXES = (
+    "已",
+    "已经",
+    "完成",
+    "修复了",
+    "新增了",
+    "优化了",
+    "更新了",
+    "重构了",
+    "添加了",
+    "改了",
+)
 FORBIDDEN_ENDINGS = (".", "。")
 MAX_SUMMARY_LEN = 50
 
@@ -49,36 +61,50 @@ def validate_commit_message(message: str) -> list[str]:
 
     match = HEADER_RE.match(header)
     commit_type = None
+    raw_summary = ""
     summary = ""
     if not match:
         errors.append("首行格式必须为 <type>(<scope>): <summary>")
     else:
         commit_type = match.group("type")
-        summary = match.group("summary").strip()
+        raw_summary = match.group("summary")
+        summary = raw_summary.strip()
 
     if len(lines) > 1 and lines[1] != "":
         errors.append("若包含正文，第 2 行必须为空行")
 
-    body = "\n".join(lines[2:]).strip() if len(lines) > 2 else ""
+    body_lines = lines[2:] if len(lines) > 2 else []
+    body = "\n".join(body_lines).strip()
 
     if commit_type and commit_type not in ALLOWED_TYPES:
         errors.append("type 必须是 feat/fix/refactor/perf/style/test/docs/chore 之一")
 
+    if not summary and match:
+        errors.append("summary 不能为空")
+
     if summary:
+        if raw_summary != summary:
+            errors.append("summary 前后不能有空格")
         if len(summary) > MAX_SUMMARY_LEN:
             errors.append("summary 长度不能超过 50 字符")
         if summary.endswith(FORBIDDEN_ENDINGS):
             errors.append("summary 结尾不能使用句号")
         if not CJK_RE.search(summary):
             errors.append("summary 必须使用中文")
+        if summary.startswith(PAST_TENSE_PREFIXES):
+            errors.append("summary 必须使用祈使句，避免使用“已/完成/xxx了”开头")
 
         for pattern in FORBIDDEN_SUMMARY_PATTERNS:
             if pattern.search(summary):
                 errors.append("summary 不能包含 and / & / multiple changes")
                 break
 
-    if body and not CJK_RE.search(body):
-        errors.append("正文需使用中文")
+    for line_no, line in enumerate(body_lines, start=3):
+        stripped = line.strip()
+        if not stripped:
+            continue
+        if not CJK_RE.search(stripped):
+            errors.append(f"正文第 {line_no} 行需使用中文")
 
     if commit_type == "perf" and not body:
         errors.append("perf 类型必须提供正文说明优化原因")