@heihei0299/matt-skills 1.1.1 → 1.1.2

package/.agents/skills/instance-test/references/instances.md

@@ -24,12 +24,15 @@ Prompt: verify idempotent init without --force
 Command: `init` twice without `--force`, second with local edit to `AGENTS.md`
 Expected: second stdout `模板已存在.*跳过`, `上游技能：已装 0、跳过 22`; local edit preserved; exit 0.
 
-### 3. Init --force with backup
-Prompt: verify forced init backs up
+### 3. Init --force with direct overwrite
+Prompt: verify forced init directly overwrites
 Command: `init --force --dest <tmp>` after local edit
-Expected: stdout `已备份` + `备份 22`; `AGENTS.md.bak` exists with local edit; `AGENTS.md` restored from template; `.agents/skills/tdd.bak` exists; exit 0.
+Expected: stdout `已覆盖` + `已装 22`; `AGENTS.md.bak` exists with local edit; `AGENTS.md` restored from template; no `.agents/skills/*.bak`; no `.opencode.bak`/`.pi.bak`; exit 0.
 
 ### 4. Sync on existing
+Prompt: verify sync directly updates existing project
+Command: `sync --dest <tmp>` after local edit
+Expected: stdout `同步` + `已同步`/`已更新`; `AGENTS.md.bak` exists; `.agents/skills/tdd` updated; exit 0.
 Prompt: verify sync backs up existing project
 Command: `sync --dest <tmp>` after local edit
 Expected: stdout `同步` + `已备份`; `AGENTS.md.bak` exists; exit 0.

package/README.md

@@ -31,16 +31,16 @@ npx @heihei0299/matt-skills init
 1. 复制 `template/` 快照（`AGENTS.md`、`.opencode/`、`.pi/`）到当前目录；
 2. 把上游技能（engineering 17 个 + productivity 5 个）复制到 `.agents/skills/`。
 
-选项：`--dest <path>` 指定目标目录（默认当前目录）；`--force` 覆盖已存在的文件并自动备份被覆盖文件到 `.bak`（默认跳过）。
+选项：`--dest <path>` 指定目标目录（默认当前目录）；`--force` 覆盖已存在的文件（直接覆盖，`AGENTS.md` 会备份到 `.bak`）（默认跳过）。
 
 **增量同步（已有项目）**：已有项目更新到最新模板与技能：
 
 ```sh
-npx @heihei0299/matt-skills sync            # 增量同步，备份被覆盖文件到 .bak
-npx @heihei0299/matt-skills sync --force    # 覆盖不备份
+npx @heihei0299/matt-skills sync            # 增量同步，直接更新
+npx @heihei0299/matt-skills sync --force    # 同上（直接覆盖）
 ```
 
-`sync` 专为已有项目设计：自动检测 `AGENTS.md` 是否存在，存在则增量更新模板与上游技能并备份，不存在则等同全新 `init`。与 `init --force` 的区别：`sync` 默认即备份、语义更明确，建议已有项目优先用 `sync`。
+`sync` 专为已有项目设计：自动检测 `AGENTS.md` 是否存在，存在则增量更新模板与上游技能（直接覆盖，`AGENTS.md` 会备份），不存在则等同全新 `init`。与 `init --force` 的区别：`sync` 语义更明确，建议已有项目优先用 `sync`。
 
 上游没有 `tdd-implement`、`grill-to-spec`、`diagnose-fix`、`commit-check`，复制天然不冲突。目标仓库会话即自动加载全部技能（上游在 `.agents/skills/`、独有在 `.opencode/skills/`；pi 侧独有在 `.pi/skills/`）与项目级全局配置（行为路由表、分文件约定）；`issue-audit` 以子代理 + 命令形式分发（`.opencode/agents/`、`.opencode/commands/`）；9 个显式触发技能注册为 opencode 命令（`.opencode/commands/`，`/命令名` 触发）。
 
@@ -116,13 +116,13 @@ pi 下对应能力以内置工具或已装扩展为准（`AGENTS.md`「能力边
 
 ```sh
 node bin/cli.js init [--dest <dir>] [--force]   # 初始化项目：template + 上游技能
-node bin/cli.js sync [--dest <dir>] [--force]   # 同步已有项目到最新（备份到 .bak）
+node bin/cli.js sync [--dest <dir>] [--force]   # 同步已有项目到最新（直接覆盖）
 node bin/cli.js list [--json]                   # 列出 .agents/skills/ 下全部技能及描述
 node bin/cli.js install [选项]                  # 把技能复制到目标工具目录（交互式选择）
 ```
 
-`init` 选项：`--dest <dir>` 指定目标目录（默认当前目录）；`--force` 覆盖已存在的文件并备份到 `.bak`（默认跳过），见「初始化」。
-`sync` 选项：`--dest <dir>` 指定目标目录；`--force` 覆盖不备份（默认备份），见「初始化」增量同步。
+`init` 选项：`--dest <dir>` 指定目标目录（默认当前目录）；`--force` 覆盖已存在的文件（直接覆盖，`AGENTS.md` 会备份到 `.bak`）（默认跳过），见「初始化」。
+`sync` 选项：`--dest <dir>` 指定目标目录；`--force` 直接覆盖（与默认一致），见「初始化」增量同步。
 
 `install` 选项：
 

package/bin/cli.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { readdir, readFile, cp, stat, writeFile, mkdir } from 'node:fs/promises';
+import { readdir, readFile, cp, stat } from 'node:fs/promises';
 import os from 'node:os';
 import path from 'node:path';
 import { fileURLToPath } from 'node:url';
@@ -17,17 +17,17 @@ const HELP = `matt-skills — install and manage this skill collection
 
 Usage:
   matt-skills init [options]                   Initialize a project: template + upstream skills
-  matt-skills sync [options]                   Sync existing project to latest template + skills (backs up overwritten files to .bak)
+  matt-skills sync [options]                   Sync existing project to latest template + skills
   matt-skills list [--json]                    List available skills and their descriptions
   matt-skills install [options]                Install skills (interactive by default)
   matt-skills --help                          Show this help
 
 Init options:
   --dest <path>   Target directory (default: current directory)
-  --force         Overwrite existing files (backs up to .bak)
+  --force         Overwrite existing files
 Sync options:
   --dest <path>   Target directory (default: current directory)
-  --force         Overwrite without backup (default: backup to .bak)
+  --force         Overwrite existing files
 
 Install options:
   --tools <a,b>   Install for the given tools (codex, pi, opencode, claude); skips tool selection
@@ -88,21 +88,6 @@ async function backupIfExists(p) {
   return bak;
 }
 
-async function ensureSkillsGitignore(skillsDir) {
-  try {
-    await mkdir(skillsDir, { recursive: true });
-    const gi = path.join(skillsDir, '.gitignore');
-    const content = '# matt-skills backup — pi skips .bak via explicit filter + this fallback\n*.bak\n*.bak/\n';
-    if (await pathExists(gi)) {
-      const cur = await readFile(gi, 'utf8');
-      if (cur.includes('*.bak')) return;
-      await writeFile(gi, cur.endsWith('\n') ? `${cur}${content}` : `${cur}\n${content}`);
-    } else {
-      await writeFile(gi, content);
-    }
-  } catch {}
-}
-
 const TOOLS = ['codex', 'pi', 'opencode', 'claude'];
 
 const PROJECT_DIRS = {
@@ -188,15 +173,13 @@ async function initCommand({ dest, force }) {
   const target = dest ? path.resolve(process.cwd(), dest) : process.cwd();
   const marker = path.join(target, 'AGENTS.md');
   if (!force && (await pathExists(marker))) {
-    process.stdout.write('模板已存在（AGENTS.md），跳过；用 --force 覆盖（自动备份到 .bak）\n');
+    process.stdout.write('模板已存在（AGENTS.md），跳过；用 --force 覆盖\n');
   } else {
     if (force && (await pathExists(marker))) {
-      for (const name of ['AGENTS.md', '.opencode', '.pi']) {
-        const cur = path.join(target, name);
-        if (await pathExists(cur)) await backupIfExists(cur);
-      }
+      const cur = path.join(target, 'AGENTS.md');
+      if (await pathExists(cur)) await backupIfExists(cur);
       await cp(TEMPLATE_DIR, target, { recursive: true, force: true });
-      process.stdout.write('模板：已备份到 .bak 并覆盖（AGENTS.md、.opencode/、.pi/）\n');
+      process.stdout.write('模板：已覆盖（AGENTS.md、.opencode/、.pi/）\n');
     } else {
       await cp(TEMPLATE_DIR, target, { recursive: true, force: true });
       process.stdout.write('模板：已复制（AGENTS.md、.opencode/、.pi/）\n');
@@ -210,7 +193,6 @@ async function initCommand({ dest, force }) {
   const skillsDir = path.join(target, '.agents', 'skills');
   let installed = 0;
   let skipped = 0;
-  let backedUp = 0;
   for (const name of upstream) {
     const src = path.join(SKILLS_DIR, name);
     const dst = path.join(skillsDir, name);
@@ -222,19 +204,10 @@ async function initCommand({ dest, force }) {
       skipped++;
       continue;
     }
-    if (force && (await pathExists(dst))) {
-      await backupIfExists(dst);
-      backedUp++;
-    }
     await cp(src, dst, { recursive: true, force: true });
     installed++;
   }
-  if (force && backedUp > 0) {
-    process.stdout.write(`上游技能：已装 ${installed}、跳过 ${skipped}、备份 ${backedUp} 到 .bak\n`);
-  } else {
-    process.stdout.write(`上游技能：已装 ${installed}、跳过 ${skipped}\n`);
-  }
-  await ensureSkillsGitignore(skillsDir);
+  process.stdout.write(`上游技能：已装 ${installed}、跳过 ${skipped}\n`);
   process.stdout.write(`目标路径：${target}\n`);
 }
 
@@ -247,15 +220,13 @@ async function syncCommand({ dest, force }) {
     await cp(TEMPLATE_DIR, target, { recursive: true, force: true });
     process.stdout.write('模板：已复制（AGENTS.md、.opencode/、.pi/）\n');
   } else {
-    process.stdout.write('同步：检测到现有项目，将增量更新并备份被覆盖文件到 .bak\n');
+    process.stdout.write('同步：检测到现有项目，将增量更新\n');
     if (backup) {
-      for (const name of ['AGENTS.md', '.opencode', '.pi']) {
-        const cur = path.join(target, name);
-        if (await pathExists(cur)) await backupIfExists(cur);
-      }
+      const cur = path.join(target, 'AGENTS.md');
+      if (await pathExists(cur)) await backupIfExists(cur);
     }
     await cp(TEMPLATE_DIR, target, { recursive: true, force: true });
-    process.stdout.write(backup ? '模板：已备份并同步（AGENTS.md、.opencode/、.pi/）\n' : '模板：已覆盖（AGENTS.md、.opencode/、.pi/）\n');
+    process.stdout.write(backup ? '模板：已同步（AGENTS.md、.opencode/、.pi/）\n' : '模板：已覆盖（AGENTS.md、.opencode/、.pi/）\n');
   }
   const entries = await readdir(SKILLS_DIR, { withFileTypes: true });
   const upstream = entries
@@ -265,7 +236,6 @@ async function syncCommand({ dest, force }) {
   const skillsDir = path.join(target, '.agents', 'skills');
   let installed = 0;
   let updated = 0;
-  let backedUp = 0;
   for (const name of upstream) {
     const src = path.join(SKILLS_DIR, name);
     const dst = path.join(skillsDir, name);
@@ -275,10 +245,6 @@ async function syncCommand({ dest, force }) {
     }
     const exists = await pathExists(dst);
     if (exists) {
-      if (backup) {
-        await backupIfExists(dst);
-        backedUp++;
-      }
       await cp(src, dst, { recursive: true, force: true });
       updated++;
     } else {
@@ -286,12 +252,7 @@ async function syncCommand({ dest, force }) {
       installed++;
     }
   }
-  if (backup) {
-    process.stdout.write(`上游技能：新增 ${installed}、更新 ${updated}（已备份 ${backedUp} 到 .bak）\n`);
-  } else {
-    process.stdout.write(`上游技能：新增 ${installed}、更新 ${updated}\n`);
-  }
-  await ensureSkillsGitignore(skillsDir);
+  process.stdout.write(`上游技能：新增 ${installed}、更新 ${updated}\n`);
   process.stdout.write(`目标路径：${target}\n`);
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@heihei0299/matt-skills",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "Agent skills + 项目配置模板：一条命令初始化 opencode / pi-agent 项目（含 mattpocock/skills 上游技能）",
   "type": "module",
   "bin": {

package/template/.opencode/skills/instance-test/references/instances.md

@@ -24,12 +24,15 @@ Prompt: verify idempotent init without --force
 Command: `init` twice without `--force`, second with local edit to `AGENTS.md`
 Expected: second stdout `模板已存在.*跳过`, `上游技能：已装 0、跳过 22`; local edit preserved; exit 0.
 
-### 3. Init --force with backup
-Prompt: verify forced init backs up
+### 3. Init --force with direct overwrite
+Prompt: verify forced init directly overwrites
 Command: `init --force --dest <tmp>` after local edit
-Expected: stdout `已备份` + `备份 22`; `AGENTS.md.bak` exists with local edit; `AGENTS.md` restored from template; `.agents/skills/tdd.bak` exists; exit 0.
+Expected: stdout `已覆盖` + `已装 22`; `AGENTS.md.bak` exists with local edit; `AGENTS.md` restored from template; no `.agents/skills/*.bak`; no `.opencode.bak`/`.pi.bak`; exit 0.
 
 ### 4. Sync on existing
+Prompt: verify sync directly updates existing project
+Command: `sync --dest <tmp>` after local edit
+Expected: stdout `同步` + `已同步`/`已更新`; `AGENTS.md.bak` exists; `.agents/skills/tdd` updated; exit 0.
 Prompt: verify sync backs up existing project
 Command: `sync --dest <tmp>` after local edit
 Expected: stdout `同步` + `已备份`; `AGENTS.md.bak` exists; exit 0.

package/template/.pi/skills/instance-test/references/instances.md

@@ -24,12 +24,15 @@ Prompt: verify idempotent init without --force
 Command: `init` twice without `--force`, second with local edit to `AGENTS.md`
 Expected: second stdout `模板已存在.*跳过`, `上游技能：已装 0、跳过 22`; local edit preserved; exit 0.
 
-### 3. Init --force with backup
-Prompt: verify forced init backs up
+### 3. Init --force with direct overwrite
+Prompt: verify forced init directly overwrites
 Command: `init --force --dest <tmp>` after local edit
-Expected: stdout `已备份` + `备份 22`; `AGENTS.md.bak` exists with local edit; `AGENTS.md` restored from template; `.agents/skills/tdd.bak` exists; exit 0.
+Expected: stdout `已覆盖` + `已装 22`; `AGENTS.md.bak` exists with local edit; `AGENTS.md` restored from template; no `.agents/skills/*.bak`; no `.opencode.bak`/`.pi.bak`; exit 0.
 
 ### 4. Sync on existing
+Prompt: verify sync directly updates existing project
+Command: `sync --dest <tmp>` after local edit
+Expected: stdout `同步` + `已同步`/`已更新`; `AGENTS.md.bak` exists; `.agents/skills/tdd` updated; exit 0.
 Prompt: verify sync backs up existing project
 Command: `sync --dest <tmp>` after local edit
 Expected: stdout `同步` + `已备份`; `AGENTS.md.bak` exists; exit 0.