makdoong2-team 2.1.0 → 2.3.0

package/dist/verifier-verdict.js

@@ -0,0 +1,110 @@
+// verifier-verdict.ts — dispatch_verifier 응답의 순수 분류기.
+//
+// 왜 별도 파일인가: opencode 의 플러그인 로더는 `src/opencode-plugin.ts` 의
+// **모든 named export 를 plugin factory 로 호출**한다. 헬퍼를 거기에 export 하면
+// 로더가 그것을 팩토리로 부르고 downstream `for (S of hooks) { S.auth }` 에서
+// 터진다 (`test/plugin-exports-shape.test.ts` 가 export 집합을 고정한다).
+// 그래서 순수 로직은 항상 별도 모듈에 두고 import 한다.
+//
+// ── 무엇을 고치는가 ────────────────────────────────────────────────────────
+// 종전 dispatch_verifier 는 다섯 갈래의 결과를 `verdictSource` 로 **구분해 로그에
+// 남기면서도** 툴 반환값의 `verdict` 는 전부 `REJECTED` 하나로 눌러 내보냈다.
+// 그래서 호출자(team-leader)는 다음 둘을 구별할 수 없었다:
+//
+//   (가) 콘텐츠 반려 — verifier 가 실제로 검사하고 산출물을 물렸다.
+//        → 올바른 조치: `.done=false` 로 되돌리고 stage 를 재실행한다.
+//   (나) 인프라 실패 — verifier 세션이 판정을 내리지 못하고 죽었다.
+//        → 올바른 조치: **verifier 만 재실행한다.** stage 산출물은 멀쩡하다.
+//
+// 실제로 (나)가 (가)로 보고되어, 마커가 전부 정상인 `1_planning.jira` 를
+// team-leader 가 통째로 재실행했다 (planner 200초 재가동). 재검증 결과는
+// VERIFIED 였다 — 원 작업에는 처음부터 결함이 없었다 (GitHub issue #7).
+//
+// 구분에 필요한 정보는 이미 전부 있었다. 계약에 반영하지 않았을 뿐이다.
+const VERDICT_TAG_RE = /<verifier-verdict>\s*(VERIFIED|REJECTED)\s*<\/verifier-verdict>/i;
+const VERDICT_JSON_RE = /"verdict"\s*:\s*"(VERIFIED|REJECTED)"/i;
+const NEXT_ACTION_VERIFIED = "다음 substage 로 진행한다.";
+const NEXT_ACTION_REJECTED = "검증이 산출물을 반려했다. 해당 substage 의 `.done` 을 false 로 되돌리고 " +
+    "dispatch_stage 를 재호출한다 (이전 실패 사유는 프롬프트에 자동 주입된다).";
+const NEXT_ACTION_ERROR = "검증이 **수행되지 않았다** (verifier 세션 인프라 실패). 같은 인자로 " +
+    "dispatch_verifier 만 재호출한다. stage 를 재실행하지 말고 `.done` 을 false 로 " +
+    "되돌리지도 말 것 — 판정을 못 얻었을 뿐 stage 산출물은 그대로다. " +
+    "verifier 는 idempotent 하므로 재호출이 안전하다.";
+/**
+ * verifier 세션 결과 → 판정 분류.
+ *
+ * 판정 태그는 세션 실패보다 우선한다: 태그를 이미 뱉은 뒤 세션 꼬리가 죽었다면
+ * 그 판정은 실제로 산출된 것이므로 존중한다.
+ *
+ * `malformed_output_default`(세션은 살아서 끝났는데 태그가 없음)는 `ERROR` 로
+ * 올리지 않고 종전대로 `REJECTED` 를 유지한다 — 형식 위반은 verifier 가 관측
+ * 가능한 콘텐츠 결함이고, `same_reason_streak` 이 이미 무한루프를 막는다.
+ * 여기서 완화 대상은 판정이 물리적으로 존재하지 않는 두 경로뿐이다.
+ */
+export function classifyVerifierOutcome(input) {
+    const tag = input.raw.match(VERDICT_TAG_RE);
+    if (tag) {
+        const verdict = tag[1].toUpperCase();
+        return {
+            verdict,
+            source: "verdict_tag",
+            retryable: false,
+            countsAsRejection: verdict === "REJECTED",
+            parsed: "verdict tag found",
+            nextAction: verdict === "VERIFIED" ? NEXT_ACTION_VERIFIED : NEXT_ACTION_REJECTED,
+        };
+    }
+    const json = input.raw.match(VERDICT_JSON_RE);
+    if (json) {
+        const verdict = json[1].toUpperCase();
+        return {
+            verdict,
+            source: "json_fallback",
+            retryable: false,
+            countsAsRejection: verdict === "REJECTED",
+            parsed: `verdict tag missing — extracted from JSON body (${verdict})`,
+            nextAction: verdict === "VERIFIED" ? NEXT_ACTION_VERIFIED : NEXT_ACTION_REJECTED,
+        };
+    }
+    if (input.sessionGone || !input.success) {
+        const source = input.sessionGone
+            ? "session_gone_default"
+            : "session_failed_default";
+        return {
+            verdict: "ERROR",
+            source,
+            retryable: true,
+            countsAsRejection: false,
+            parsed: `verifier session failed — 판정 없음 (${source}): ${input.raw.slice(0, 120)}`,
+            nextAction: NEXT_ACTION_ERROR,
+        };
+    }
+    return {
+        verdict: "REJECTED",
+        source: "malformed_output_default",
+        retryable: false,
+        countsAsRejection: true,
+        parsed: "verdict tag missing — defaulted to REJECTED (verifier output malformed)",
+        nextAction: NEXT_ACTION_REJECTED,
+    };
+}
+/**
+ * 인프라 실패로 판정을 못 얻은 횟수의 연속 상한.
+ *
+ * `same_reason_streak`(REJECTED 무한루프 차단)의 ERROR 판 대응물이다. ERROR 는
+ * 집계에서 빠지므로 그쪽 상한이 걸리지 않는다 — 별도 상한이 없으면 verifier 만
+ * 재호출하는 루프가 영원히 돈다. 3회로 잡은 것은 재호출이 저렴하고(단일 read-only
+ * 세션) 3회 연속 실패면 모델·인프라 문제라 자동 재시도로 풀리지 않기 때문이다.
+ */
+export const VERIFIER_ERROR_STREAK_LIMIT = 3;
+/** 이번 결과를 반영한 ERROR 연속 횟수. ERROR 가 아니면 0 으로 리셋된다. */
+export function nextVerifierErrorStreak(previousStreak, classification) {
+    if (classification.verdict !== "ERROR")
+        return 0;
+    const prev = Number.isFinite(previousStreak) && previousStreak > 0 ? previousStreak : 0;
+    return prev + 1;
+}
+/** 상한 도달 여부. 도달하면 team-leader 는 재호출을 멈추고 사용자에게 보고한다. */
+export function verifierErrorStreakExceeded(streak, limit = VERIFIER_ERROR_STREAK_LIMIT) {
+    return streak >= limit;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "makdoong2-team",
-  "version": "2.1.0",
+  "version": "2.3.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/references/pr-template.md

@@ -83,7 +83,8 @@
 ## 4. PR 생성 규칙
 
 1. 반드시 **Draft 로 생성**한다 → `bitbucket_createPullRequest(..., draft=true)`.
-2. **리뷰어를 추가하지 않는다** → `reviewers` 파라미터 자체를 생략한다 (`[]` 도 아니라 미전달).
+2. **토큰 소유자를 리뷰어로 추가한다** → §4-1 로 식별한 username 을 `reviewers` 에 전달한다. 자기 자신이 PR 작성자여서 Bitbucket DC 가 거부하면 `reviewer_self_skipped=true` 로 기록한다 (`stages/08-pr.md` §7-5).
+   > 종전 이 항목은 "리뷰어를 추가하지 않는다 (`reviewers` 파라미터 자체를 생략)" 였다. 같은 문서의 §4-1·§4-2 와도, `stages/08-pr.md` 와도, `gates/stage7-post-pr-verify.sh` 의 reviewer 마커 검사(`reviewer_added` XOR `reviewer_self_skipped`, 미기록 시 fail)와도 정면으로 어긋나 있었다. 게이트가 마커를 요구하는 이상 리뷰어는 추가하는 것이 맞다.
 3. 생성 후 각 커밋별 diff 를 분석하여 **인라인 코멘트** 를 **개별 live 코멘트** 로 작성한다 (아래 §5–6).
 
 ### 4-1. 토큰 소유자 식별 (리뷰어 추가용)

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",
@@ -51,6 +50,7 @@ const STEPS = [
   "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/verifier-verdict.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",
@@ -83,7 +83,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 +95,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/stages/08-pr.md

@@ -175,3 +175,30 @@ post-verify 실패 시:
 
 
 
+
+## 7-7. publisher 세션이 반복 실패할 때 (대체 경로)
+
+`dispatch_stage(3_delivery.pr)` 가 **연속 3회 이상** 실패 반환하면 (empty / session_gone / timeout — 즉 산출물 문제가 아니라 세션 자체가 결과를 못 낸 경우), 부장님(team-leader)은 아래 조건을 **모두** 확인한 뒤 남은 작업을 직접 완결할 수 있다.
+
+> 근거: 실제로 이 단계가 4회 연속 empty 반환했고, 그 사이 `git push` 는 실제로 수행됐는데 PR 생성과 마커 기록만 미완으로 남았다. 재-dispatch 를 반복해도 같은 지점에서 멈춘다 — 남은 일에 git 명령이 없기 때문이다 (GitHub issue #7).
+
+**진입 조건 (하나라도 false 면 진입 금지 — 사용자에게 보고하고 지시를 기다린다)**
+
+| 항목 | 확인 방법 |
+|---|---|
+| 1 | `dispatch_stage(3_delivery.pr)` 가 3회 이상 `ok:false` 로 반환됐다 |
+| 2 | `origin/<브랜치>` 가 이미 존재한다 (**push 가 끝나 있다**) — `git ls-remote` 는 부장님 권한 밖이므로 직전 publisher 세션 로그 또는 `hang_history` 로 확인 |
+| 3 | 남은 작업에 **git 명령이 하나도 없다** — PR 생성(MCP)·마커 기록(`state.sh`)뿐이다 |
+| 4 | `escalate: true` 를 받지 않았다 (받았으면 재디스패치·대체 실행 모두 금지, 사용자 에스컬레이션만) |
+
+**허용 범위 (이것만 한다)**
+
+- `bitbucket_createPullRequest` / `bitbucket_getPullRequest` / `bitbucket_updatePullRequest` — MCP 호출이며 git 명령이 아니다.
+- `state.sh set` 으로 §완료 기록의 마커 기록.
+- `gates/stage7-post-pr-verify.sh` 실행.
+
+**금지 (조건 2 가 거짓이면 그냥 막힌다)**
+
+- `git push` 를 비롯한 모든 git 명령. 부장님은 permission 이 deny 이고, 우회하지 않는다. push 가 남아 있으면 이 대체 경로를 쓸 수 없다 — 사용자에게 보고한다.
+
+**보고**: `[3_delivery.pr PUBLISHER FALLBACK] publisher <N>회 실패 → 남은 작업(PR 생성·마커)에 git 없음을 확인하고 부장님이 직접 완결. push 는 <세션 ID> 에서 이미 완료됨`

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);