@wooojin/forgen 0.5.7 → 0.5.9

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "$schema": "https://claude.ai/schemas/claude-plugin.json",
   "name": "forgen",
-  "version": "0.5.7",
+  "version": "0.5.9",
   "description": "Claude Code harness — the more you use Claude, the better it gets",
   "author": {
     "name": "jang-ujin",

package/CHANGELOG.md

@@ -7,6 +7,74 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.5.9] — 2026-10-02 — 남겨 둔 한계 정리: 사용자 스킬 보존 · AGENTS.md 전체 정리 · OpenCode uninstall · claude-mem 13.28
+
+### Fixed
+- **`forgen install` / npm postinstall 이 사용자가 만든 `forgen-*` 스킬을 지우던 것.** dev-guide 스킬의 stale 정리가 이름만 보고
+  디렉토리를 재귀 삭제했다 (`install claude` 는 `forgen-*` 전부, `install codex`/postinstall 은 `forgen-(react|vue|node|go)-*`).
+  이제 forgen 소유만 정리한다: 패키지가 현재 제공하는 이름, 또는 `SKILL.md` 가 `…/assets/dev-guide/…` 를 가리키는 심링크(이전
+  버전이 설치한 것). 삭제도 `SKILL.md` 한 파일과 빈 디렉토리만. 설치는 이미 있는 것을 덮어쓰지 않는다 — 심링크된 스킬
+  디렉토리(dotfiles 등)를 따라 들어가 쓰지 않고, 한 스킬 자리가 막혀 있어도 나머지 설치는 계속한다.
+- AGENTS.md 블록 처리: 마커가 한 줄을 통째로 차지할 때만 블록 경계로 본다. 본문에 마커 문자열을 인용한 문서에서 인용 줄부터
+  실제 블록 끝까지를 덮어쓰거나 지울 수 있었다.
+- **forgen-eval**: `workerStatus()` 가 "Worker is not running" 을 실행 중으로 판정하던 것. claude-mem recall 이 `sqlite3` CLI 에만
+  의존해 CLI 가 없는 머신에서는 조용히 비어 있던 것(forgen+mem arm 이 forgen-only 와 같아짐) — `node:sqlite` 우선 + CLI 폴백,
+  둘 다 없으면 run 경고. `CLAUDE_MEM_TESTED_VERSION` 이 12.4.8 에 머물러 매 실행 version-mismatch 경고가 나던 것.
+
+### Added
+- **`forgen uninstall` 이 다른 프로젝트의 AGENTS.md 블록도 정리한다.** `forgen install codex|opencode` 가 블록을 쓴 경로를 host
+  설정 디렉토리의 `forgen-agents-md.json` 에 기록하고, uninstall 이 기록된 경로 전부 + 실행한 cwd 를 정리한 뒤 기록을 지운다.
+  (0.5.9 이전 설치분은 기록이 없어 여전히 cwd 만 정리된다.)
+- **OpenCode uninstall.** forgen-managed plugin 제거(설치 때 백업한 사용자 plugin 이 있으면 되돌림), config(JSONC) 의
+  `mcp.forgen-compound` 를 forgen 시그니처일 때만 제거(다른 키·서버의 값은 보존, 인접 주석/포맷은 바뀔 수 있음; 파싱 불가면
+  건드리지 않음), AGENTS.md 블록. plugin 소유는 첫 줄의 `// forgen-managed` 로만 판정한다.
+- uninstall 이 쓰지 못한 AGENTS.md(읽기 전용 등)는 오류로 알리고 기록에 남겨 다음 실행에서 다시 시도한다.
+
+### Changed
+- **claude-mem 13.12.4 → 13.28.0** (forgen-eval 테스트베드 핀, Dependabot #136). 0.5.7 에서 "재측정 필요" 로 보류했으나 근거가
+  약했다 — 저장소의 측정 리포트는 claude-mem 버전에 묶여 있지 않고 forgen-eval 이 의존하는 것은 CLI 계약이다. 격리 HOME 에 두
+  버전을 설치해 계약이 동일함을 실측: `version`/`start`/`status`/`stop`, `search` 출력(JSON + `| #N |` 표), DB 컬럼, 비매칭
+  쿼리 동작.
+
+### Verified
+- fresh-context critic 리뷰: MAJOR 2 · MINOR 10 → 반영 또는 한계로 명시(ADR-016). 리뷰어의 재현 환경(심링크된 스킬 디렉토리)을
+  CLI `install both` 와 postinstall 에 다시 돌려 사용자 파일이 그대로임을 확인.
+- vitest 3242 통과 + forgen-eval 72 통과 (신규: 실 SQLite DB 로 recall 계약, worker 판정, 핀 일치 강제; 사용자 스킬 보존;
+  AGENTS.md 기록/정리; OpenCode uninstall).
+- 격리 HOME 실행: 실 claude-mem 13.28.0 워커에 대해 빌드된 recall 경로가 시드한 observation 본문을 돌려주고, 워커 on/off 판정이
+  맞음. CLI `install both` + `install opencode` 를 두 프로젝트에서 실행한 뒤 다른 디렉토리에서 `uninstall --force` —
+  두 AGENTS.md 블록·OpenCode plugin/MCP·Codex 등록분 제거, 사용자 스킬과 사용자 AGENTS.md 내용 보존.
+
+
+## [0.5.8] — 2026-10-02 — Codex 가 재배치한 마커에서도 설치·제거가 정확하게 (Codex 0.160 확인)
+
+### Fixed
+- **Codex 가 config.toml 을 다시 쓰면서 forgen 마커를 뒤집어 놓은 형태에서 `forgen uninstall` 이 MCP 서버 등록을 지우지
+  못하던 것 (0.5.7).** 실머신에서 `/hooks` 승인 후 관측: `# >>> forgen-managed-mcp` 는 forgen 테이블과 함께 파일 끝으로 가고
+  `# <<< forgen-managed-mcp` 는 앞쪽에 고아로 남는다. 이제 마커를 *범위* 가 아니라 *표식* 으로만 쓰고, 범위는 TOML 구조
+  (테이블 헤더 ~ 다음 헤더, notify 한 줄)로 정한다. 재설치는 고아 마커를 걷어 테이블 위아래로 정규화하고, 제거는 테이블·하위
+  테이블·모든 마커 줄을 지운다. notify 블록도 END 마커가 옮겨지거나 사라져도 동작한다.
+- 사용자가 forgen MCP 테이블만 지우고 마커가 남은 경우, 재설치가 고아 마커를 정리하고 블록을 한 번만 쓴다.
+- **`codex mcp add <다른 서버>` 뒤에는 forgen 테이블을 알아보지 못하던 것.** Codex 가 mcp_servers 를 다시 쓰며 시작 마커를
+  없앤다. 소유 판정을 "헤더 바로 위의 마커 또는 forgen 내용 시그니처(args 의 `dist/mcp/server.js` + `--host=codex`,
+  notify 의 `dist/host/codex-notify.js`)" 로 바꿨다 — 마커가 사라져도 경로 갱신과 제거가 된다.
+- CRLF config.toml 에서 블록 제거 후 파일이 `\r` 로만 끝나 Codex 가 로드하지 못할 수 있던 것 (0.5.8 작업 중 회귀로 발견, 미출시).
+- forgen 테이블 뒤·다음 테이블 헤더 위의 사용자 주석을 forgen 본문으로 취급하지 않는다.
+- Codex 가 forgen notify 블록 안의 값을 사용자의 notifier 로 바꿔 놓은 경우 그 값을 보존한다 (이전엔 재설치가 덮어썼다).
+- 같은 MCP 서버가 다른 TOML 표기(따옴표·공백 헤더, inline table, dotted key)로 정의돼 있으면 중복 정의를 추가하지 않는다
+  (추가하면 Codex 가 `duplicate key` 로 로드 실패).
+- 블록 제거가 "지웠다" 고 보고하는 것은 실제로 테이블/notify 줄을 지웠을 때만.
+
+### Verified
+- vitest 3222 통과 (신규: 실머신에서 채취한 재배치 형태, CRLF/BOM, 마커 소실, 사용자 주석, 다른 TOML 표기).
+- fresh-context critic 리뷰: CRITICAL 1 · MAJOR 2 · MINOR 6 → 전부 반영. 리뷰어의 케이스 68건 + 퍼징 14,000건에서 Codex 로드
+  불가 출력·사용자 데이터 변경·비멱등 0건, 실 Codex 0.160 재현 스크립트(`codex mcp add` 후 설치/제거, CRLF 왕복) 통과.
+- 이 머신의 실 `~/.codex/config.toml` 사본에 적용: 재설치 diff 는 고아 END 마커 한 줄 이동뿐(hooks.state 31개 보존),
+  제거 후 forgen 항목 0건, 두 결과 모두 Codex 0.160.0 이 정상 파싱.
+- **Codex 0.160.0 호환 확인**: 훅 출력 스키마 11종이 vendoring 한 0.153.4 사본과 동일, trust 해시 일치(실머신 22/22 trusted,
+  Codex `hooks/list` 30/30), 실세션에서 SessionStart(룰 블록 주입, 스필 없음)·UserPromptSubmit·Stop·SessionEnd 훅 발화, 오류 0건.
+
+
 ## [0.5.7] — 2026-10-02 — `forgen uninstall` 의 Codex 정리 · 의존성 메이저 업그레이드
 
 ### Added

package/dist/core/uninstall.js

@@ -364,7 +364,8 @@ export async function handleUninstall(cwd, options) {
     console.log('  4. Remove forgen block from CLAUDE.md');
     console.log('  5. Remove slash commands (~/.claude/commands/forgen/) and the forgen-managed verify skill (~/.claude/skills/verify/)');
     console.log('  6. Remove plugin artifacts (cache, installed_plugins.json, plugin directory), dev-guide skills, and the forgen-compound MCP entry in ~/.claude.json');
-    console.log('  7. Codex (if $CODEX_HOME or ~/.codex exists): forgen hooks in hooks.json, the MCP/notify blocks in config.toml, forgen skills, ch-*.toml agents, and the AGENTS.md block — hooks and files from other tools are kept');
+    console.log('  7. Codex (if $CODEX_HOME or ~/.codex exists): forgen hooks in hooks.json, the MCP/notify blocks in config.toml, forgen skills, ch-*.toml agents, and the AGENTS.md blocks (every project recorded at install time) — hooks and files from other tools are kept');
+    console.log('     OpenCode (if its config dir exists): the forgen plugin, the forgen-compound MCP entry, AGENTS.md blocks');
     if (options.purge) {
         console.log('  8. --purge: Delete ~/.forgen/ entirely (rules, me/, state/, solutions/, behavior/)');
         console.log('     WARNING: this erases all accumulated corrections, rules, drift, and lifecycle history.');
@@ -409,6 +410,14 @@ export async function handleUninstall(cwd, options) {
     catch (e) {
         console.error('  ✗ Codex cleanup failed:', e instanceof Error ? e.message : String(e));
     }
+    try {
+        const { planOpencodeUninstall, renderOpencodeUninstall } = await import('../host/uninstall-opencode.js');
+        for (const line of renderOpencodeUninstall(planOpencodeUninstall({ pkgRoot })))
+            console.log(line);
+    }
+    catch (e) {
+        console.error('  ✗ OpenCode cleanup failed:', e instanceof Error ? e.message : String(e));
+    }
     cleanPluginArtifacts();
     if (options.purge) {
         try {

package/dist/host/install-claude.js

@@ -18,7 +18,7 @@ import * as fs from 'node:fs';
 import * as os from 'node:os';
 import * as path from 'node:path';
 import { generateHooksJson } from '../hooks/hooks-generator.js';
-import { hasManagedSkillMarker } from './managed-marker.js';
+import { hasManagedSkillMarker, installSkillFile, removeOwnedDevGuideSkills } from './managed-marker.js';
 const PLUGIN_KEY = 'forgen@forgen-local';
 const FORGEN_MANAGED_MARKER = '<!-- forgen-managed -->';
 function readPkgVersion(pkgRoot) {
@@ -266,40 +266,14 @@ function installDevGuideSkills(opts) {
         return { skillsPath: skillsDir, skillsInstalled: entries.length, skillsRemoved: 0 };
     }
     fs.mkdirSync(skillsDir, { recursive: true });
-    // Remove stale forgen-* skill dirs (idempotent re-install, do not touch user's own skills)
-    let removed = 0;
-    for (const entry of fs.readdirSync(skillsDir)) {
-        if (!entry.startsWith(FORGEN_SKILL_PREFIX))
-            continue;
-        const fullPath = path.join(skillsDir, entry);
-        if (fs.statSync(fullPath).isDirectory()) {
-            fs.rmSync(fullPath, { recursive: true, force: true });
-            removed += 1;
-        }
-    }
-    // Install each skill via symlink → cpSync fallback (mirrors plugin cache pattern)
+    // stale 정리 — forgen 이 설치한 dev-guide 스킬만 (패키지가 제공하는 이름 / dev-guide 를 가리키는 심링크).
+    // 이전엔 `forgen-*` 디렉토리를 이름만 보고 재귀 삭제해 사용자가 만든 스킬까지 지웠다 (ADR-016 0.5.9).
+    const removed = removeOwnedDevGuideSkills(skillsDir, pkgRoot);
+    // 설치 — 이미 있는 것(사용자 심링크 디렉토리, 남아 있는 SKILL.md)은 덮어쓰지 않는다.
     let installed = 0;
     for (const { name, src } of entries) {
-        const targetDir = path.join(skillsDir, name);
-        fs.mkdirSync(targetDir, { recursive: true });
-        const targetFile = path.join(targetDir, 'SKILL.md');
-        let linked = false;
-        let symlinkErr = null;
-        try {
-            fs.symlinkSync(src, targetFile, 'file');
-            linked = true;
-        }
-        catch (e) {
-            symlinkErr = e;
-        }
-        if (!linked && symlinkErr) {
-            const code = symlinkErr.code ?? 'UNKNOWN';
-            process.stderr.write(`[forgen] symlink ${src} → ${targetFile} failed (${code}); falling back to copyFile.\n`);
-        }
-        if (!linked) {
-            fs.copyFileSync(src, targetFile);
-        }
-        installed += 1;
+        if (installSkillFile(src, skillsDir, name))
+            installed += 1;
     }
     return { skillsPath: skillsDir, skillsInstalled: installed, skillsRemoved: removed };
 }

package/dist/host/install-codex.d.ts

@@ -95,9 +95,9 @@ export declare const FORGEN_SKILL_MARKER = "<!-- forgen-managed -->";
 export declare function isForgenHookCommand(command: unknown, pkgRoot: string): boolean;
 export declare function readJsonFile<T>(p: string): T | null;
 /**
- * forgen MCP 블록 제거 (uninstall, ADR-016 D4). forgen 테이블(본문 + `[mcp_servers.forgen-compound.*]` 하위
- * 테이블)과 마커만 걷어내고, 블록 사이에 Codex 가 끼워 넣은 다른 내용은 그 자리에 보존한다.
- * 마커 없는 사용자 관리 테이블은 건드리지 않는다.
+ * forgen MCP 블록 제거 (uninstall, ADR-016 D4). forgen 테이블(본문 + 하위 테이블)과 마커 줄만 걷어낸다.
+ * 마커도 시그니처도 없는 같은 이름의 테이블(사용자 관리)은 건드리지 않는다.
+ * `removed` 는 테이블을 실제로 지웠을 때만 true — 고아 마커만 치운 경우는 false.
  */
 export declare function removeMcpBlock(currentToml: string): {
     content: string;
@@ -107,22 +107,24 @@ export declare function removeMcpBlock(currentToml: string): {
  * config.toml 에 forgen notify 블록을 upsert.
  *
  * - Codex 의 `notify` 는 top-level 단일 argv 다. 사용자가 이미 정의했으면 **건드리지 않는다** — 그리고
- *   forgen 블록이 남아 있으면 제거한다 (중복 키 = config.toml 파싱 실패 → Codex 기동 불가).
+ *   forgen 줄이 남아 있으면 제거한다 (중복 키 = config.toml 파싱 실패 → Codex 기동 불가).
  * - top-level 키는 첫 테이블 헤더 앞에 와야 하므로 블록은 항상 파일 최상단(BOM 뒤)에 둔다.
- * - 사용자가 forgen 블록의 argv 뒤에 `"--", "<prog>", …` 로 자기 notifier 를 체인해 뒀으면 그 꼬리를 보존.
+ * - 사용자가 forgen argv 뒤에 `"--", "<prog>", …` 로 자기 notifier 를 체인해 뒀으면 그 꼬리를 보존.
  *   블록의 notify 줄을 한 줄 JSON 으로 읽을 수 없으면(여러 줄 배열 등 손편집) 아무것도 바꾸지 않는다.
- * - 블록 사이에 Codex 가 끼워 넣은 줄(root 키)은 블록 바로 뒤로 옮겨 보존한다.
+ * - forgen 이 쓴 줄 외에는 원래 순서 그대로 둔다 (Codex 가 사이에 끼워 넣은 root 키 포함).
  */
 export declare function upsertNotifyBlock(currentToml: string, pkgRoot: string): {
     content: string;
     status: CodexNotifyStatus;
 };
 /**
- * forgen notify 블록 제거 (`--no-notify`, uninstall). 블록 사이에 끼어든 다른 줄은 보존.
+ * forgen notify 블록 제거 (`--no-notify`, uninstall).
  *
  * - 사용자가 블록의 notify 줄을 여러 줄 배열 등으로 손편집했으면(`custom`) **건드리지 않는다** — 첫 줄만
  *   지우면 남은 줄이 깨진 TOML 이 되어 Codex 가 기동하지 못한다 (critic 2026-10-02).
  * - `"--"` 뒤에 사용자가 체인해 둔 자기 notifier 가 있으면 그 argv 만으로 `notify` 를 되돌려 놓는다.
+ * - `removed` 는 forgen notify 줄을 실제로 지웠을 때만 true. 고아 마커/주석만 치운 경우는 false
+ *   (내용은 정리된 것을 돌려준다).
  */
 export declare function removeNotifyBlock(currentToml: string): {
     content: string;
@@ -197,7 +199,29 @@ export declare function upsertForgenRulesInAgentsMd(opts: {
 }): {
     injected: boolean;
 };
-/** AGENTS.md 의 forgen 블록 제거 (uninstall). 블록뿐이던 파일은 삭제한다. */
+export declare function readAgentsMdInstalls(hostDir: string): string[];
+export declare function recordAgentsMdInstall(hostDir: string, agentsMdPath: string): void;
+export declare function clearAgentsMdInstalls(hostDir: string): void;
+/**
+ * 기록된 경로 + 지금의 cwd 경로에서 forgen 블록을 걷어낸다.
+ * 쓰지 못한 경로(읽기 전용 등)는 `failed` 로 돌려주고 **기록에 남긴다** — 다음 uninstall 이 다시 시도할 수 있게.
+ */
+export declare function removeForgenRulesEverywhere(opts: {
+    hostDir: string;
+    cwdAgentsMdPath: string;
+    dryRun: boolean;
+}): {
+    cleaned: string[];
+    failed: Array<{
+        path: string;
+        error: string;
+    }>;
+};
+/**
+ * AGENTS.md 의 forgen 블록 제거 (uninstall). 블록뿐이던 파일은 삭제한다.
+ * BEGIN/END 마커가 각각 한 줄로, 순서대로 있을 때만 동작한다 (블록이 여러 개면 전부 제거).
+ * 읽을 수 없으면 no-op, 쓸 수 없으면 throw — 호출부가 보고한다.
+ */
 export declare function removeForgenRulesFromAgentsMd(opts: {
     agentsMdPath: string;
     dryRun: boolean;