makdoong2-team 2.1.0 → 2.2.0

package/README.md

@@ -250,7 +250,52 @@ Resolved chain (primary → fallbacks):
 
 ---
 
-## 5. 테스트
+## 5. 개발 환경과 빌드
+
+이 저장소에는 **소스 파일만 커밋한다.** 손으로 쓴 JavaScript 도, 커밋된 빌드
+산출물도 없다 — 구현은 TypeScript(`.ts`/`.mts`)와 셸(`.sh`)이고, `.js`/`.mjs`
+와 `dist/` 는 소스에서 생성되며 `.gitignore` 대상이다.
+
+### 요구 도구
+
+| 도구 | 버전 | 용도 |
+|---|---|---|
+| **Node.js** | **≥ 22.18** | 진입점·테스트 실행 (네이티브 type-stripping). 클라이언트 환경은 node 24. |
+| npm | Node 동봉 | 스크립트·의존성 |
+| TypeScript | `devDependencies` 고정 (5.9.x) | 플러그인·진입점 빌드, 타입 검사 |
+| jq, git | 시스템 | 런타임(state.sh·게이트) |
+| tmux | ≥ 3.0 | 서브에이전트 pane (선택) |
+| docker | 선택 | 비-Linux 호스트의 Ubuntu 교차 검증 |
+
+> **왜 node ≥ 22.18 인가.** 개발 중에는 CLI(`bin/cli.ts`)·`postinstall.mts`·
+> 테스트(`test/*.test.ts`)를 컴파일하지 않고 `node`가 확장자를 보고 타입을 지워
+> 소스를 바로 실행한다(22.18/23.6부터 기본 활성). 배포된 패키지는 `node_modules`
+> 안에서 실행되는데 node 는 거기서는 type-stripping 을 하지 않으므로, `bin`·
+> `postinstall`은 배포 시 빌드된 `.js`가 실행된다. 플러그인 런타임은 opencode(Bun)가
+> `dist/`를 로드하므로 이 하한과 무관하다.
+
+### 빌드 — 개발엔 불필요, 배포 시 자동
+
+```bash
+npm run build        # src/**  →  dist/**   (opencode 플러그인. tsc)
+npm run build:entry  # 진입점 4개  →  bin/cli.js · postinstall.mjs · scripts/{install-lib,model-policy}.mjs
+npm run typecheck    # 타입 검사만 (noEmit)
+npm run clean        # dist/ 제거
+```
+
+**개발 중에는 빌드가 필요 없다** — 소스를 직접 실행한다:
+
+```bash
+node bin/cli.ts doctor
+node scripts/run-tests.mts
+```
+
+빌드 산출물은 커밋하지 않는다. 배포(`npm publish`) 시 `prepack`이 `build` +
+`build:entry`를 돌려 tarball에만 담는다. 근거·경계는 ARCHITECTURE.md §12.6.
+
+---
+
+## 6. 테스트
 
 ```bash
 npm test               # 전체
@@ -267,7 +312,7 @@ MAKDOONG2_SKIP_LINUX_CHECK=1 git push   # 그 push 는 Linux 검증 없이 나
 
 ---
 
-## 6. 배포 (Maintainer)
+## 7. 배포 (Maintainer)
 
 두 경로 모두 **승인 게이트 2회**를 통과해야 배포된다.
 
@@ -289,7 +334,7 @@ npm run release:major   # 1.3.1 → 2.0.0  (breaking)
 
 ---
 
-## 7. 문서
+## 8. 문서
 
 | 문서 | 내용 |
 |---|---|

package/bin/cli.js

@@ -1,7 +1,6 @@
 #!/usr/bin/env node
 // bin/cli.ts — makdoong2-team installer & doctor.
 //
-// 산출물은 같은 자리의 bin/cli.js 다 (`npm run build:entry`). 그 .js 를 직접
 // 편집하지 말 것 — 다음 빌드가 덮어쓴다.
 //
 // **dist/ 를 import 하지 않는다.** doctor/validate 는 설치가 깨졌을 때 실행하는

package/bin/cli.ts

@@ -1,7 +1,6 @@
 #!/usr/bin/env node
 // bin/cli.ts — makdoong2-team installer & doctor.
 //
-// 산출물은 같은 자리의 bin/cli.js 다 (`npm run build:entry`). 그 .js 를 직접
 // 편집하지 말 것 — 다음 빌드가 덮어쓴다.
 //
 // **dist/ 를 import 하지 않는다.** doctor/validate 는 설치가 깨졌을 때 실행하는

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "makdoong2-team",
-  "version": "2.1.0",
+  "version": "2.2.0",
   "publishConfig": {
     "access": "public"
   },
@@ -65,21 +65,21 @@
   },
   "scripts": {
     "build": "tsc -p tsconfig.build.json",
-    "build:entry": "tsc -p tsconfig.entry.json && node -e \"import('node:fs').then(({chmodSync})=>{for(const f of ['bin/cli.js','postinstall.mjs','scripts/run-tests.mjs','scripts/test-postinstall.mjs']) chmodSync(f, 0o755)})\"",
     "typecheck": "tsc -p tsconfig.json",
     "clean": "node -e \"import('node:fs').then(({rmSync})=>rmSync('dist',{recursive:true,force:true}))\"",
-    "prepack": "npm run clean && npm run build && node --test test/entry-artifacts.test.ts",
+    "prepack": "npm run clean && npm run build && npm run build:entry",
     "postinstall": "node postinstall.mjs",
     "prepare": "husky install",
     "test": "XDG_CONFIG_HOME=\"$(mktemp -d)\" XDG_CACHE_HOME=\"$(mktemp -d)\" npm run test:all",
-    "test:all": "npm run build && node scripts/run-tests.mjs",
+    "test:all": "npm run build && node scripts/run-tests.mts",
     "test:install": "node test/install-lib.test.ts",
-    "test:postinstall": "node scripts/test-postinstall.mjs",
+    "test:postinstall": "node scripts/test-postinstall.mts",
     "test:ubuntu": "bash scripts/test-ubuntu.sh",
     "release": "bash scripts/release.sh",
     "release:patch": "bash scripts/release.sh patch",
     "release:minor": "bash scripts/release.sh minor",
-    "release:major": "bash scripts/release.sh major"
+    "release:major": "bash scripts/release.sh major",
+    "build:entry": "tsc -p tsconfig.entry.json && node -e \"import('node:fs').then(({chmodSync})=>{chmodSync('bin/cli.js',0o755)})\""
   },
   "homepage": "https://github.com/y00njinuk/makdoong2-team#readme",
   "bugs": {

package/postinstall.mjs

@@ -1,8 +1,6 @@
 #!/usr/bin/env node
 // postinstall.mts — Auto-install on npm install -g
 //
-// 산출물은 같은 자리의 postinstall.mjs 다 (`npm run build:entry`). 그 .mjs 를
-// 직접 편집하지 말 것 — 다음 빌드가 덮어쓴다.
 //
 // **이 파일은 반드시 패키지 루트에 있어야 한다.** pkgRoot = 자기 디렉토리
 // (`..` 없음)가 계약이라, 한 단계라도 깊어지면 agents/skills 복사가 ENOENT 로

package/scripts/install-lib.mjs

@@ -1,7 +1,5 @@
 // scripts/install-lib.mts — Reusable install logic for CLI and postinstall.
 //
-// 산출물은 같은 자리의 scripts/install-lib.mjs 다 (`npm run build:entry`).
-// 그 .mjs 를 직접 편집하지 말 것 — 다음 빌드가 덮어쓴다.
 //
 // Extracted from bin/cli doInstall() to enable both:
 //   1) Manual install via `makdoong2-team install` (CLI)

package/scripts/install-lib.mts

@@ -1,7 +1,5 @@
 // scripts/install-lib.mts — Reusable install logic for CLI and postinstall.
 //
-// 산출물은 같은 자리의 scripts/install-lib.mjs 다 (`npm run build:entry`).
-// 그 .mjs 를 직접 편집하지 말 것 — 다음 빌드가 덮어쓴다.
 //
 // Extracted from bin/cli doInstall() to enable both:
 //   1) Manual install via `makdoong2-team install` (CLI)

package/scripts/model-policy.mjs

@@ -1,7 +1,5 @@
 // model-policy.mts — pure mirror of src/model-fallback-policy.ts.
 //
-// 산출물은 같은 자리의 scripts/model-policy.mjs 다 (`npm run build:entry`).
-// 그 .mjs 를 직접 편집하지 말 것 — 다음 빌드가 덮어쓴다.
 //
 // 왜 미러인가: bin/cli.js 는 dist/ 없이 돌아야 한다. doctor/validate 는 설치가
 // 깨졌을 때 실행하는 진단 도구인데, dist/ 를 import 하면 (a) .gitignore 대상이라

package/scripts/model-policy.mts

@@ -1,7 +1,5 @@
 // model-policy.mts — pure mirror of src/model-fallback-policy.ts.
 //
-// 산출물은 같은 자리의 scripts/model-policy.mjs 다 (`npm run build:entry`).
-// 그 .mjs 를 직접 편집하지 말 것 — 다음 빌드가 덮어쓴다.
 //
 // 왜 미러인가: bin/cli.js 는 dist/ 없이 돌아야 한다. doctor/validate 는 설치가
 // 깨졌을 때 실행하는 진단 도구인데, dist/ 를 import 하면 (a) .gitignore 대상이라

package/scripts/run-tests.mts

@@ -1,7 +1,6 @@
 #!/usr/bin/env node
 // scripts/run-tests.mts — 전체 테스트를 순차 실행하고 실패를 모아서 보고한다.
 //
-// 산출물은 같은 자리의 scripts/run-tests.mjs 다 (`npm run build:entry`).
 // **새 테스트를 등록할 때는 이 .mts 의 STEPS 를 고친다** — .mjs 는 산출물이라
 // 다음 빌드에 날아간다.
 //
@@ -37,7 +36,7 @@ const REPO_ROOT = join(dirname(fileURLToPath(import.meta.url)), "..");
 const STEPS = [
   "bash scripts/lint-agent-prompts.sh",
   "node --test test/shell-portability.test.ts",
-  "node scripts/smoke-test.mjs",
+  "node scripts/smoke-test.mts",
   "bash scripts/gate-policy-test.sh",
   "node test/install-lib.test.ts",
   "node test/skill-mcp-registry.test.ts",
@@ -83,7 +82,6 @@ const STEPS = [
   "node --test test/doctor-exit-code.test.ts",
   "node --test test/example-config-portability.test.ts",
   "node --test test/research-fanout.test.ts",
-  "node --test test/entry-artifacts.test.ts",
   "node --test test/state-sh-write-atomicity.test.ts",
   "node --test test/gate-locale-and-path.test.ts",
   "node --test test/model-policy-parity.test.ts",
@@ -96,7 +94,7 @@ const STEPS = [
   "node --test test/sync-state-commit-detection.test.ts",
   "node --test test/skill-tls-config.test.ts",
   "node --test test/opencode-json-read.test.ts",
-  "node scripts/test-postinstall.mjs"
+  "node scripts/test-postinstall.mts"
 ];
 
 const CYAN = "\u001b[36m", GREEN = "\u001b[32m", RED = "\u001b[31m", YELLOW = "\u001b[33m", OFF = "\u001b[0m";

package/scripts/smoke-test.mts

@@ -1,6 +1,5 @@
 // smoke-test.mts — smoke test for the model-fallback-policy chain logic.
 //
-// 산출물은 같은 자리의 scripts/smoke-test.mjs 다 (`npm run build:entry`).
 //
 // 주의: 이 파일은 **미러(scripts/model-policy.mts)만** 검증한다. 정본
 // src/model-fallback-policy.ts 와의 동치는 test/model-policy-parity.test.ts 가

package/scripts/test-postinstall.mts

@@ -1,7 +1,6 @@
 #!/usr/bin/env node
 // scripts/test-postinstall.mts — integration test for npm install -g postinstall hook
 //
-// 산출물은 같은 자리의 scripts/test-postinstall.mjs 다 (`npm run build:entry`).
 //
 // Simulates global install in isolated HOME directory:
 //   1) npm pack → produces tarball

package/scripts/run-tests.mjs

@@ -1,155 +0,0 @@
-#!/usr/bin/env node
-// scripts/run-tests.mts — 전체 테스트를 순차 실행하고 실패를 모아서 보고한다.
-//
-// 산출물은 같은 자리의 scripts/run-tests.mjs 다 (`npm run build:entry`).
-// **새 테스트를 등록할 때는 이 .mts 의 STEPS 를 고친다** — .mjs 는 산출물이라
-// 다음 빌드에 날아간다.
-//
-// 왜 `&&` 체인을 대체하는가:
-//   기존 test:all 은 42개 단계를 `&&` 로 이어 붙였다. 앞 단계가 하나라도 실패하면
-//   뒤 단계는 아예 실행되지 않는다. 실제로 tmux-monitor 실패가 그 뒤의
-//   rollback-commits / worktree-sync-gate 실패를 오래 가려왔고, gate-policy-test 가
-//   깨져 있던 동안에는 사실상 전 구간이 미실행 상태였다.
-//
-//   순차 실행은 유지한다 (병렬화하지 않는다). npm test 가 XDG_CONFIG_HOME 을 단일
-//   임시 디렉토리로 고정하므로, 설치 계열 테스트를 동시에 돌리면 서로의 config 를
-//   덮어써 경합이 난다.
-//
-// 2단계 구성:
-//   1) 호스트 단계 — STEPS 를 순차 실행한다.
-//   2) Linux 교차 검증 — 호스트가 Linux 가 아니면 같은 스위트를 Ubuntu 컨테이너에서
-//      한 번 더 돌린다. 별도 명령을 기억할 필요 없이 npm test 만으로 두 플랫폼이
-//      함께 검증된다.
-//
-// 호스트 단계를 컨테이너 실행으로 "대체" 하지 않는 이유: 실제로 잡힌 결함 중
-// 하나(rollback-commits.sh 의 unbraced 변수)는 Darwin libc 에서만 재현된다.
-// 컨테이너로 갈아타면 그 부류가 영구히 은폐된다. 두 번 돌려야 둘 다 잡힌다.
-//
-// 종료 코드: 실패 단계가 하나라도 있으면 1.
-import { spawnSync } from "node:child_process";
-import { existsSync } from "node:fs";
-import { dirname, join } from "node:path";
-import { fileURLToPath } from "node:url";
-const REPO_ROOT = join(dirname(fileURLToPath(import.meta.url)), "..");
-const STEPS = [
-    "bash scripts/lint-agent-prompts.sh",
-    "node --test test/shell-portability.test.ts",
-    "node scripts/smoke-test.mjs",
-    "bash scripts/gate-policy-test.sh",
-    "node test/install-lib.test.ts",
-    "node test/skill-mcp-registry.test.ts",
-    "node --test test/state-write-guard.test.ts",
-    "node --test test/state-access-guard.test.ts",
-    "node --test test/git-exclude-registration.test.ts",
-    "node --test test/issue-reporter-guard.test.ts",
-    "node --test test/release-confirm.test.ts",
-    "node --test test/state-sh-schema.test.ts",
-    "node --test test/state-sh-init-review-shape.test.ts",
-    "node --test test/doctor-phantom-scan.test.ts",
-    "node --test test/mcp-secret-injector.test.ts",
-    "node --test test/poll-sub-session.test.ts",
-    "node --test test/tmux-monitor.test.ts",
-    "node --test test/gate-already-done-block.test.ts",
-    "node --test test/gate-hybrid-first-entry.test.ts",
-    "node --test test/gate-post-pr-verify.test.ts",
-    "node --test test/gate-post-review-verify.test.ts",
-    "node --test test/gate-requirements-quality.test.ts",
-    "node --test test/worktree-sync-gate.test.ts",
-    "node --test test/planner-prompt-early-exit.test.ts",
-    "node --test test/plugin-bug-fixes.test.ts",
-    "node --test test/omo-tmux-detection.test.ts",
-    "node --test test/empty-output-done-override.test.ts",
-    "node --test test/tmux-monitor-orphan.test.ts",
-    "node --test test/poll-permission-scope.test.ts",
-    "node --test test/dispatch-review-fixes.test.ts",
-    "node --test test/dispatch-stage-redispatch.test.ts",
-    "node --test test/dispatch-stage-dev-resume-prompt.test.ts",
-    "node --test test/logger.test.ts",
-    "node --test test/redact-secrets.test.ts",
-    "node --test test/commit-atomicity-verify.test.ts",
-    "node --test test/verdict-reason-injection.test.ts",
-    "node --test test/verdict-hash-normalize.test.ts",
-    "node --test test/session-index-fallback.test.ts",
-    "node --test test/plugin-exports-shape.test.ts",
-    "node --test test/stale-worktree-recovery.test.ts",
-    "node --test test/rollback-commits.test.ts",
-    "node --test test/config-dir-home-fallback.test.ts",
-    "node --test test/state-path-migration.test.ts",
-    "node --test test/state-path-relative.test.ts",
-    "node --test test/with-fallback-no-bun.test.ts",
-    "node --test test/doctor-exit-code.test.ts",
-    "node --test test/example-config-portability.test.ts",
-    "node --test test/research-fanout.test.ts",
-    "node --test test/entry-artifacts.test.ts",
-    "node --test test/state-sh-write-atomicity.test.ts",
-    "node --test test/gate-locale-and-path.test.ts",
-    "node --test test/model-policy-parity.test.ts",
-    // npm install -g 전 구간(pack → 전역 설치 → postinstall → doctor)을 검증하는
-    // 유일한 테스트다. 종전에는 STEPS 에 없어서 `npm test` 가 한 번도 실행하지
-    // 않았고, 그 사이 내용이 현행 구현과 반대로 썩어 있었다 (plugin ref 형식,
-    // secrets.env 처리). 등록하지 않으면 같은 일이 반복된다.
-    "node --test test/agent-permission-keys.test.ts",
-    "node --test test/gate-missing-state-sentinel.test.ts",
-    "node --test test/sync-state-commit-detection.test.ts",
-    "node --test test/skill-tls-config.test.ts",
-    "node --test test/opencode-json-read.test.ts",
-    "node scripts/test-postinstall.mjs"
-];
-const CYAN = "\u001b[36m", GREEN = "\u001b[32m", RED = "\u001b[31m", YELLOW = "\u001b[33m", OFF = "\u001b[0m";
-const say = (s) => { process.stdout.write(s); };
-// ── 1단계: 호스트 ──
-const results = [];
-for (const [i, step] of STEPS.entries()) {
-    const [cmd, ...args] = step.split(" ");
-    say(`\n${CYAN}[${String(i + 1).padStart(2)}/${STEPS.length}]${OFF} ${step}\n`);
-    const r = spawnSync(cmd, args, { stdio: "inherit" });
-    results.push({ step, code: r.status === null ? 1 : r.status });
-}
-// ── 2단계: Linux 교차 검증 ──
-/**
- * 컨테이너 교차 검증을 돌려야 하는지 판단한다.
- * @returns {{run: true} | {run: false, reason: string, loud: boolean}}
- */
-function linuxCheckPlan() {
-    // 컨테이너 안에서 다시 컨테이너를 띄우지 않는다 (무한 재귀 방지).
-    if (process.env.MAKDOONG2_IN_TEST_CONTAINER === "1") {
-        return { run: false, reason: "이미 테스트 컨테이너 안", loud: false };
-    }
-    // 호스트가 Linux 면 방금 돌린 것이 곧 Linux 결과다.
-    if (process.platform === "linux") {
-        return { run: false, reason: "호스트가 이미 Linux", loud: false };
-    }
-    if (process.env.MAKDOONG2_SKIP_LINUX_CHECK === "1") {
-        return { run: false, reason: "MAKDOONG2_SKIP_LINUX_CHECK=1", loud: true };
-    }
-    if (!existsSync(join(REPO_ROOT, "Dockerfile.test"))) {
-        return { run: false, reason: "Dockerfile.test 없음 (배포 tarball 등)", loud: false };
-    }
-    if (spawnSync("docker", ["--version"], { stdio: "ignore" }).status !== 0) {
-        return { run: false, reason: "docker 없음 — Linux 전용 회귀를 놓칠 수 있다", loud: true };
-    }
-    return { run: true };
-}
-const plan = linuxCheckPlan();
-let linuxResult = null;
-if (plan.run) {
-    say(`\n${CYAN}[linux]${OFF} Ubuntu 컨테이너에서 교차 검증 (${process.platform} 호스트)\n`);
-    const r = spawnSync("bash", [join(REPO_ROOT, "scripts", "test-ubuntu.sh")], { stdio: "inherit" });
-    linuxResult = { step: "linux 교차 검증 (scripts/test-ubuntu.sh)", code: r.status === null ? 1 : r.status };
-    results.push(linuxResult);
-}
-else if (plan.loud) {
-    say(`\n${YELLOW}[linux] 교차 검증 건너뜀 — ${plan.reason}${OFF}\n`);
-}
-// ── 요약 ──
-const failed = results.filter((r) => r.code !== 0);
-say(`\n${"=".repeat(70)}\n`);
-if (failed.length === 0) {
-    const suffix = linuxResult ? " (호스트 + Ubuntu 교차 검증)" : "";
-    say(`${GREEN}[run-tests] ${results.length}개 단계 전부 통과${suffix}${OFF}\n`);
-    process.exit(0);
-}
-say(`${RED}[run-tests] ${failed.length}/${results.length} 단계 실패:${OFF}\n`);
-for (const f of failed)
-    say(`  ${RED}\u2716${OFF} ${f.step}  (exit ${f.code})\n`);
-process.exit(1);

package/scripts/smoke-test.mjs

@@ -1,178 +0,0 @@
-// smoke-test.mts — smoke test for the model-fallback-policy chain logic.
-//
-// 산출물은 같은 자리의 scripts/smoke-test.mjs 다 (`npm run build:entry`).
-//
-// 주의: 이 파일은 **미러(scripts/model-policy.mts)만** 검증한다. 정본
-// src/model-fallback-policy.ts 와의 동치는 test/model-policy-parity.test.ts 가
-// 따로 강제한다 — 예전 주석은 이 파일이 정본까지 검증한다고 적었으나 거짓이었다.
-//
-// Run: node scripts/smoke-test.mjs
-// Exits 0 on all-pass, 1 on any failure.
-import { DEFAULT_ALLOWED_PRIMARIES, DEFAULT_POLICIES, nextModel, validatePolicies, buildPoliciesFromConfig, } from "./model-policy.mjs";
-let passed = 0, failed = 0;
-const t = (name, fn) => {
-    try {
-        fn();
-        console.log(`  ✓ ${name}`);
-        passed++;
-    }
-    catch (e) {
-        console.error(`  ✗ ${name}: ${e instanceof Error ? e.message : String(e)}`);
-        failed++;
-    }
-};
-const eq = (a, b, msg) => {
-    if (JSON.stringify(a) !== JSON.stringify(b)) {
-        throw new Error(`${msg}\n    expected: ${JSON.stringify(b)}\n    got:      ${JSON.stringify(a)}`);
-    }
-};
-const assertThrows = (fn, matcher, msg) => {
-    let thrown = null;
-    try {
-        fn();
-    }
-    catch (e) {
-        thrown = e;
-    }
-    if (!thrown)
-        throw new Error(`${msg}: expected throw, got none`);
-    const message = thrown instanceof Error ? thrown.message : String(thrown);
-    if (matcher && !matcher.test(message)) {
-        throw new Error(`${msg}: message did not match ${matcher}\n    got: ${message}`);
-    }
-};
-/** `next` 가 null 이 아님을 단언하고 좁혀서 돌려준다. */
-const expectNext = (r) => {
-    if (!r.next)
-        throw new Error("expected a next model, got null (chain exhausted)");
-    return r.next;
-};
-console.log("\n[smoke-test] model-fallback-policy chain logic\n");
-// ── Default chain semantics ──
-t("makdoong2-team-leader: gpt-5.6-luna → haiku on first failure", () => {
-    const r = nextModel({ agent: "makdoong2-team-leader", current: "github-copilot/gpt-5.6-luna", reason: "rate_limit" });
-    eq(r.exhausted, false, "should not be exhausted");
-    eq(expectNext(r).id, "github-copilot/claude-haiku-4.5", "next should be haiku-4.5");
-});
-t("makdoong2-team-leader: haiku → exhausted after second failure", () => {
-    const r = nextModel({ agent: "makdoong2-team-leader", current: "github-copilot/claude-haiku-4.5" });
-    eq(r.exhausted, true, "should be exhausted");
-    eq(r.next, null, "next should be null");
-});
-t("makdoong2-planner: gpt-5.6-luna → haiku", () => {
-    const r = nextModel({ agent: "makdoong2-planner", current: "github-copilot/gpt-5.6-luna", reason: "5xx" });
-    eq(expectNext(r).id, "github-copilot/claude-haiku-4.5", "planner fallback should be haiku");
-});
-t("makdoong2-engineer: gpt-5.6-luna → haiku (lower-tier degrade)", () => {
-    const r = nextModel({ agent: "makdoong2-engineer", current: "github-copilot/gpt-5.6-luna", reason: "context_exceeded" });
-    eq(expectNext(r).id, "github-copilot/claude-haiku-4.5", "engineer fallback should be haiku");
-});
-t("makdoong2-publisher: gpt-5.6-luna → haiku", () => {
-    const r = nextModel({ agent: "makdoong2-publisher", current: "github-copilot/gpt-5.6-luna" });
-    eq(expectNext(r).id, "github-copilot/claude-haiku-4.5", "publisher fallback should be haiku");
-});
-t("makdoong2-verifier: gpt-5.6-luna → haiku", () => {
-    const r = nextModel({ agent: "makdoong2-verifier", current: "github-copilot/gpt-5.6-luna" });
-    eq(expectNext(r).id, "github-copilot/claude-haiku-4.5", "verifier fallback should be haiku");
-});
-t("unknown agent returns exhausted", () => {
-    const r = nextModel({ agent: "stage99-fictional", current: "any" });
-    eq(r.exhausted, true, "unknown agent should be exhausted");
-});
-t("unknown current model starts from primary", () => {
-    const r = nextModel({ agent: "makdoong2-team-leader", current: "anthropic/claude-fictional" });
-    eq(expectNext(r).id, "github-copilot/gpt-5.6-luna", "unknown current should start at primary");
-});
-t("all default primaries are in DEFAULT_ALLOWED_PRIMARIES", () => {
-    for (const [agent, policy] of Object.entries(DEFAULT_POLICIES)) {
-        if (!DEFAULT_ALLOWED_PRIMARIES.has(policy.primary.id)) {
-            throw new Error(`'${agent}' primary '${policy.primary.id}' is not allowed`);
-        }
-    }
-});
-t("all fallback tiers are strictly lower than primary tier", () => {
-    validatePolicies(DEFAULT_POLICIES, DEFAULT_ALLOWED_PRIMARIES);
-});
-// ── Config-driven overrides ──
-t("agents block: primary + fallback override (string form, backward-compat)", () => {
-    const { policies } = buildPoliciesFromConfig({
-        agents: {
-            "makdoong2-engineer": {
-                model: "github-copilot/claude-sonnet-4.6",
-                fallback_models: ["github-copilot/claude-haiku-4.5"],
-            },
-        },
-    });
-    eq(policies["makdoong2-engineer"].primary.id, "github-copilot/claude-sonnet-4.6", "primary updated");
-    eq(policies["makdoong2-engineer"].fallbacks.map(f => f.id), ["github-copilot/claude-haiku-4.5"], "fallbacks set");
-    eq(policies["makdoong2-engineer"].fallbacks[0].tier, "low", "string-form fallback defaults to tier=low");
-});
-t("agents block: fallback object form {id, tier} accepted", () => {
-    const { policies } = buildPoliciesFromConfig({
-        agents: {
-            "makdoong2-planner": {
-                model: "github-copilot/claude-sonnet-4.6",
-                fallback_models: [{ id: "github-copilot/claude-haiku-4.5", tier: "low" }],
-            },
-        },
-    });
-    eq(policies["makdoong2-planner"].fallbacks[0], { id: "github-copilot/claude-haiku-4.5", tier: "low" }, "object-form preserved");
-});
-t("model_policy.allowed_primaries: extends default allow-list", () => {
-    const { policies, allowed } = buildPoliciesFromConfig({
-        model_policy: { allowed_primaries: ["custom-provider/custom-model"] },
-        agents: {
-            "makdoong2-engineer": {
-                model: "custom-provider/custom-model",
-                fallback_models: ["github-copilot/claude-haiku-4.5"],
-            },
-        },
-    });
-    eq(allowed.has("custom-provider/custom-model"), true, "extra primary allowed");
-    eq(allowed.has("local/qwen3.6-27b"), true, "built-in defaults preserved");
-    eq(policies["makdoong2-engineer"].primary.id, "custom-provider/custom-model", "custom primary set");
-});
-t("validation rejects primary outside allow-list (no model_policy extension)", () => {
-    assertThrows(() => buildPoliciesFromConfig({
-        agents: { "makdoong2-engineer": { model: "rogue-provider/rogue-model" } },
-    }), /violates policy/, "should reject unlisted primary");
-});
-t("validation rejects fallback tier >= primary tier", () => {
-    assertThrows(() => buildPoliciesFromConfig({
-        agents: {
-            "makdoong2-engineer": {
-                model: "local/qwen3.6-27b",
-                fallback_models: [{ id: "github-copilot/claude-haiku-4.5", tier: "medium" }],
-            },
-        },
-    }), /not strictly lower/, "fallback at primary tier must be rejected");
-});
-t("validation rejects invalid tier label", () => {
-    assertThrows(() => buildPoliciesFromConfig({
-        agents: {
-            "makdoong2-engineer": {
-                model: "local/qwen3.6-27b",
-                // 의도적으로 잘못된 tier 라벨 — 런타임 검증이 잡아야 한다.
-                fallback_models: [{ id: "github-copilot/claude-haiku-4.5", tier: "ultra" }],
-            },
-        },
-    }), /not a valid tier/, "unknown tier must be rejected");
-});
-t("build is pure: invalid config throws but does not mutate DEFAULT_POLICIES", () => {
-    const before = JSON.stringify(DEFAULT_POLICIES);
-    try {
-        buildPoliciesFromConfig({
-            agents: { "makdoong2-engineer": { model: "rogue-provider/rogue-model" } },
-        });
-    }
-    catch { /* expected */ }
-    const after = JSON.stringify(DEFAULT_POLICIES);
-    eq(after, before, "DEFAULT_POLICIES must remain untouched after a failed override");
-});
-t("empty config returns defaults verbatim", () => {
-    const { policies, allowed } = buildPoliciesFromConfig({});
-    eq(Object.keys(policies).sort(), Object.keys(DEFAULT_POLICIES).sort(), "same agent set");
-    eq(allowed.size, DEFAULT_ALLOWED_PRIMARIES.size, "no extra primaries added");
-});
-console.log(`\n[smoke-test] ${passed} passed, ${failed} failed\n`);
-process.exit(failed === 0 ? 0 : 1);

package/scripts/test-postinstall.mjs

@@ -1,144 +0,0 @@
-#!/usr/bin/env node
-// scripts/test-postinstall.mts — integration test for npm install -g postinstall hook
-//
-// 산출물은 같은 자리의 scripts/test-postinstall.mjs 다 (`npm run build:entry`).
-//
-// Simulates global install in isolated HOME directory:
-//   1) npm pack → produces tarball
-//   2) npm install -g <tarball> with isolated HOME
-//   3) Verify deployment artifacts
-//   4) Verify idempotency (re-install doesn't create duplicate backups)
-//   5) Verify secrets.env preservation
-import { mkdtempSync, rmSync, existsSync, writeFileSync, readFileSync, readdirSync } from "node:fs";
-import { tmpdir, platform } from "node:os";
-import { join, dirname } from "node:path";
-import { fileURLToPath } from "node:url";
-import { spawnSync } from "node:child_process";
-const HERE = dirname(fileURLToPath(import.meta.url));
-const PKG_ROOT = join(HERE, "..");
-function run(cmd, args, opts = {}) {
-    const result = spawnSync(cmd, args, { ...opts, stdio: "pipe", encoding: "utf8" });
-    if (result.error)
-        throw result.error;
-    return result;
-}
-function assert(cond, msg) {
-    if (!cond) {
-        console.error(`✗ FAIL: ${msg}`);
-        process.exit(1);
-    }
-    console.log(`✓ ${msg}`);
-}
-console.log("[test-postinstall] Starting npm install -g smoke test");
-// 1) npm pack
-console.log("\n[1/6] Running npm pack...");
-const packResult = run("npm", ["pack"], { cwd: PKG_ROOT });
-if (packResult.status !== 0) {
-    console.error("npm pack failed:", packResult.stderr);
-    process.exit(1);
-}
-const tarballLine = packResult.stdout.trim().split("\n").pop();
-if (!tarballLine) {
-    console.error("npm pack produced no tarball name on stdout");
-    process.exit(1);
-}
-const tarball = join(PKG_ROOT, tarballLine);
-assert(existsSync(tarball), `tarball created: ${tarball}`);
-try {
-    // 2) Create isolated HOME
-    console.log("\n[2/6] Setting up isolated HOME...");
-    const tmpHome = mkdtempSync(join(tmpdir(), "mkd2-postinstall-"));
-    const npmPrefix = join(tmpHome, "npm");
-    const configDir = join(tmpHome, ".config", "opencode");
-    console.log(`  tmpHome: ${tmpHome}`);
-    console.log(`  npmPrefix: ${npmPrefix}`);
-    console.log(`  configDir: ${configDir}`);
-    // 3) npm install -g with isolated env
-    console.log("\n[3/6] Running npm install -g (first install)...");
-    // 격리는 HOME 만으로 부족하다. resolveConfigDir() 은 XDG_CONFIG_HOME 을 HOME
-    // 보다 **먼저** 보고, seedOpencodeCache() 는 XDG_CACHE_HOME 을 본다. `npm test`
-    // 가 두 변수를 임시 디렉토리로 고정하므로, 상속하면 배포가 tmpHome 이 아니라
-    // 그 임시 디렉토리로 가서 "npm install -g 는 성공했는데 configDir 이 비어 있다"
-    // 가 된다 — 단독 실행(`npm run test:postinstall`)에서는 두 변수가 없어 통과하고
-    // 스위트 안에서만 실패하는 형태라 원인을 찾기 어렵다.
-    const installEnv = {
-        ...process.env,
-        HOME: tmpHome,
-        XDG_CONFIG_HOME: join(tmpHome, ".config"),
-        XDG_CACHE_HOME: join(tmpHome, ".cache"),
-        npm_config_prefix: npmPrefix,
-        npm_config_global: "true",
-        CI: undefined,
-        CONTINUOUS_INTEGRATION: undefined,
-    };
-    const installResult = run("npm", ["install", "-g", tarball], { env: installEnv });
-    if (installResult.status !== 0) {
-        console.error("npm install -g failed:", installResult.stderr);
-        process.exit(1);
-    }
-    console.log("  npm install -g completed");
-    // 4) Verify deployment artifacts
-    console.log("\n[4/6] Verifying deployment artifacts...");
-    // Binary shim
-    const isWin = platform() === "win32";
-    const binName = isWin ? "makdoong2-team.cmd" : "makdoong2-team";
-    const binPath = join(npmPrefix, "bin", binName);
-    assert(existsSync(binPath), `binary shim exists: ${binPath}`);
-    // Agents in configDir
-    const agentPath = join(configDir, "agents", "makdoong2-team-leader.md");
-    assert(existsSync(agentPath), `agent deployed to configDir: ${agentPath}`);
-    // Skills in configDir
-    const skillPath = join(configDir, "skills", "jira-research", "SKILL.md");
-    assert(existsSync(skillPath), `skill deployed to configDir: ${skillPath}`);
-    // opencode.json plugin ref (relative path to npm module)
-    const ocPath = join(configDir, "opencode.json");
-    assert(existsSync(ocPath), `opencode.json created: ${ocPath}`);
-    const oc = JSON.parse(readFileSync(ocPath, "utf8"));
-    // 플러그인 ref 는 **npm 패키지 이름**이다. 예전에는 src/opencode-plugin.ts 를
-    // 가리키는 절대 경로였고 이 단언도 그 시절 형태로 남아 있었다 — 즉 이 테스트는
-    // 지금 돌리면 실패한다. STEPS 에 등록돼 있지 않아 아무도 몰랐다.
-    // (install-lib 의 pluginRef() / isLegacyPluginEntry() 가 계약의 출처다.)
-    const PKG_NAME = "makdoong2-team";
-    const matchesRef = (v) => typeof v === "string" && (v === PKG_NAME || v.startsWith(`${PKG_NAME}@`));
-    assert(Array.isArray(oc.plugin) && oc.plugin.some((p) => matchesRef(p) || (Array.isArray(p) && matchesRef(p[0]))), `opencode.json has plugin ref (${PKG_NAME})`);
-    assert(!JSON.stringify(oc.plugin).includes("opencode-plugin.ts"), "opencode.json has no legacy path-based plugin ref");
-    assert(oc.tools?.verify_stage === true, "opencode.json has tools");
-    // makdoong2-team doctor
-    console.log("\n[5/6] Running makdoong2-team doctor...");
-    const doctorResult = run(binPath, ["doctor"], { env: installEnv });
-    assert(doctorResult.status === 0, "makdoong2-team doctor exits 0");
-    // 5) Re-install (idempotency test)
-    console.log("\n[6/6] Re-installing (idempotency check)...");
-    const beforeOcBackups = readdirSync(configDir).filter((f) => f.startsWith("opencode.json.bak."));
-    const reinstallResult = run("npm", ["install", "-g", tarball], { env: installEnv });
-    assert(reinstallResult.status === 0, "re-install succeeds");
-    const afterOcBackups = readdirSync(configDir).filter((f) => f.startsWith("opencode.json.bak."));
-    assert(afterOcBackups.length === beforeOcBackups.length, `no new opencode.json backups on re-install (idempotent): ${beforeOcBackups.length} → ${afterOcBackups.length}`);
-    // 6) legacy secrets.env cleanup
-    //
-    // 종전 이 블록은 "secrets.env 가 보존된다" 를 단언했다. 현행 install() 은 정반대로
-    // 동작한다 — 자격증명은 makdoong2-team.json .secrets.* 하나로 옮겨졌고, 남아 있는
-    // 레거시 파일은 백업 후 **삭제**하는 것이 계약이다 (install-lib 의
-    // cleanupLegacySecretsEnv). 규약이 뒤집혔는데 테스트가 따라오지 않았다.
-    console.log("\n[Bonus] Testing legacy secrets.env cleanup...");
-    const secretsPath = join(configDir, "skills", "jira-research", "secrets.env");
-    writeFileSync(secretsPath, "JIRA_TOKEN=test123");
-    const reinstall2Result = run("npm", ["install", "-g", tarball], { env: installEnv });
-    assert(reinstall2Result.status === 0, "third install succeeds");
-    assert(!existsSync(secretsPath), "legacy secrets.env removed on re-install");
-    const backups = readdirSync(join(configDir, "skills", "jira-research"))
-        .filter((f) => f.startsWith("secrets.env.bak."));
-    assert(backups.length >= 1, "legacy secrets.env backed up before removal");
-    // 자격증명의 새 자리가 실제로 만들어져 있어야 한다.
-    const cfg = JSON.parse(readFileSync(join(configDir, "makdoong2-team.json"), "utf8"));
-    assert(cfg.secrets && typeof cfg.secrets === "object", "makdoong2-team.json has .secrets scaffolding");
-    assert("JIRA_API_TOKEN" in cfg.secrets, "makdoong2-team.json .secrets has JIRA_API_TOKEN key");
-    console.log("\n✅ [test-postinstall] All checks passed!");
-    // Cleanup
-    rmSync(tmpHome, { recursive: true, force: true });
-}
-finally {
-    // Cleanup tarball
-    if (existsSync(tarball))
-        rmSync(tarball);
-}