makdoong2-team 1.3.0 → 1.3.1

package/README.md

@@ -176,10 +176,46 @@ npm run release:major   # 0.2.3 → 1.0.0  (breaking)
 ## Testing
 
 ```bash
-npm test               # build + smoke + gate policy + install-lib
-npm run test:install   # 설치 라이브러리 테스트
+npm test               # build + 43개 단계 전체 (scripts/run-tests.mjs)
+npm run test:ubuntu    # 위와 동일한 스위트를 Ubuntu 24.04 컨테이너에서 실행
+npm run test:install   # 설치 라이브러리 테스트만
 ```
 
+`npm test` 는 `scripts/run-tests.mjs` 로 각 단계를 **순차 실행하되 실패해도 멈추지 않고**
+끝까지 돌린 뒤 실패 목록을 모아 보고한다. 이전의 `&&` 체인은 앞 단계가 하나만 깨져도
+뒤 단계가 전부 미실행 상태가 되어, 실제로 여러 실패가 서로를 가려왔다.
+
+### Ubuntu 에서 실행 — `npm run test:ubuntu`
+
+플러그인은 Ubuntu 에서 개발·운영되지만 테스트는 macOS 에서도 자주 돌린다. 두 환경은
+다음 지점에서 갈리며, 셋 다 실제로 테스트 실패로 나타난 적이 있다.
+
+| 갈리는 지점 | macOS | Linux |
+|---|---|---|
+| `isalnum()` (UTF-8 로케일, 상위 바이트) | `0xEA` 를 alnum 으로 보고 → `$VAR한글` 이 깨짐 | 상위 바이트는 non-alnum → 정상 |
+| `os.tmpdir()` | `/var/folders/…` (`/var` 는 심볼릭 링크) | `/tmp` (실제 디렉토리) |
+| 기본 bash | 3.2 | 5.x |
+
+`scripts/test-ubuntu.sh` 가 `Dockerfile.test` (ubuntu:24.04 + Node 22) 를 빌드해 같은
+스위트를 컨테이너에서 돌린다. Docker 또는 OrbStack 이 필요하다. 호스트 uid 로 실행하므로
+root 소유 파일이 생기지 않고, Linux 용 `node_modules` 는 `.docker-test/` (gitignore) 에
+따로 두어 호스트 것을 덮어쓰지 않는다.
+
+```bash
+npm run test:ubuntu                       # 전체 스위트
+bash scripts/test-ubuntu.sh npm run build # 임의 명령
+bash scripts/test-ubuntu.sh bash          # 컨테이너 셸 (디버깅)
+```
+
+**자동 정리** — 컨테이너는 `docker run --rm` 으로 종료 즉시 제거된다. docker 데몬
+(OrbStack/colima VM) 이 꺼져 있어서 스크립트가 직접 기동한 경우에는 테스트가 끝날 때
+자동으로 다시 끈다. 실행 전부터 떠 있었다면 다른 작업을 끊지 않도록 그대로 둔다.
+`MAKDOONG2_KEEP_DOCKER=1` 로 자동 종료를 끌 수 있다.
+
+**주의**: Ubuntu 러너는 macOS 전용 버그를 잡지 못한다 — 위 표의 첫 줄이 정확히 그런
+경우다. 그 부류는 `test/shell-portability.test.mjs` 가 정적으로 검사한다 (`$VAR` 뒤에
+non-ASCII 가 붙으면 실패, `${VAR}` 로 감쌀 것).
+
 `.husky/pre-push` 가 unit test 실행 + version 변경 감지 시 자동 배포 훅을 트리거한다.
 
 ## 문서

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "makdoong2-team",
-  "version": "1.3.0",
+  "version": "1.3.1",
   "publishConfig": {
     "access": "public"
   },
@@ -69,9 +69,10 @@
     "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 && bash scripts/lint-agent-prompts.sh && node scripts/smoke-test.mjs && bash scripts/gate-policy-test.sh && node test/install-lib.test.mjs && node test/skill-mcp-registry.test.mjs && node --test test/state-write-guard.test.mjs && node --test test/state-sh-schema.test.mjs && node --test test/state-sh-init-review-shape.test.mjs && node --test test/doctor-phantom-scan.test.mjs && node --test test/mcp-secret-injector.test.mjs && node --test test/poll-sub-session.test.mjs && node --test test/tmux-monitor.test.mjs && node --test test/gate-already-done-block.test.mjs && node --test test/gate-hybrid-first-entry.test.mjs && node --test test/gate-post-pr-verify.test.mjs && node --test test/gate-post-review-verify.test.mjs && node --test test/worktree-sync-gate.test.mjs && node --test test/planner-prompt-early-exit.test.mjs && node --test test/plugin-bug-fixes.test.mjs && node --test test/omo-tmux-detection.test.mjs && node --test test/empty-output-done-override.test.mjs && node --test test/tmux-monitor-orphan.test.mjs && node --test test/poll-permission-scope.test.mjs && node --test test/dispatch-review-fixes.test.mjs && node --test test/dispatch-stage-redispatch.test.mjs && node --test test/dispatch-stage-dev-resume-prompt.test.mjs && node --test test/logger.test.mjs && node --test test/redact-secrets.test.mjs && node --test test/commit-atomicity-verify.test.mjs && node --test test/verdict-reason-injection.test.mjs && node --test test/verdict-hash-normalize.test.mjs && node --test test/session-index-fallback.test.mjs && node --test test/plugin-exports-shape.test.mjs && node --test test/stale-worktree-recovery.test.mjs && node --test test/rollback-commits.test.mjs && node --test test/config-dir-home-fallback.test.mjs && node --test test/state-path-migration.test.mjs && node --test test/state-path-relative.test.mjs && node --test test/with-fallback-no-bun.test.mjs && node --test test/doctor-exit-code.test.mjs && node --test test/example-config-portability.test.mjs",
+    "test:all": "npm run build && node scripts/run-tests.mjs",
     "test:install": "node test/install-lib.test.mjs",
     "test:postinstall": "node scripts/test-postinstall.mjs",
+    "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",

package/scripts/rollback-commits.sh

@@ -26,7 +26,13 @@ git -C "$WT" merge-base --is-ancestor "$BASE" HEAD 2>/dev/null || {
 # 커밋만 취소, 변경은 보존 (soft reset)
 N=$(git -C "$WT" rev-list --count "$BASE..HEAD" 2>/dev/null || echo 0)
 git -C "$WT" reset --soft "$BASE"
-echo "[rollback] HEAD를 $BASE 로 soft-reset 완료 (취소된 커밋 $N개, working tree/index 변경은 보존됨)"
+# ${N} and ${BASE} must stay braced. Unbraced, bash reads the variable name with
+# legal_variable_char() == isalnum() one byte at a time, and Darwin libc reports
+# 0xEA — the lead byte of the Hangul counter suffix that follows — as alnum in
+# UTF-8 locales. The name absorbs that byte, becomes unbound, and set -u aborts.
+# glibc keeps high bytes non-alnum, so this only bites on macOS.
+# Guarded by test/shell-portability.test.mjs.
+echo "[rollback] HEAD를 ${BASE} 로 soft-reset 완료 (취소된 커밋 ${N}개, working tree/index 변경은 보존됨)"
 
 # 3_delivery.commit 관련 state 초기화 (base_sha는 보존 — 동일 기준점으로 재커밋)
 "$HERE/state.sh" set "$ISSUE" '.stages."3_delivery".substages."commit".done' 'false' >/dev/null

package/scripts/run-tests.mjs

@@ -0,0 +1,81 @@
+#!/usr/bin/env node
+// scripts/run-tests.mjs — 전체 테스트를 순차 실행하고 실패를 모아서 보고한다.
+//
+// 왜 `&&` 체인을 대체하는가:
+//   기존 test:all 은 42개 단계를 `&&` 로 이어 붙였다. 앞 단계가 하나라도 실패하면
+//   뒤 단계는 아예 실행되지 않는다. 실제로 tmux-monitor 실패가 그 뒤의
+//   rollback-commits / worktree-sync-gate 실패를 오래 가려왔고, gate-policy-test 가
+//   깨져 있던 동안에는 사실상 전 구간이 미실행 상태였다.
+//
+//   순차 실행은 유지한다 (병렬화하지 않는다). npm test 가 XDG_CONFIG_HOME 을 단일
+//   임시 디렉토리로 고정하므로, 설치 계열 테스트를 동시에 돌리면 서로의 config 를
+//   덮어써 경합이 난다.
+//
+// 종료 코드: 실패 단계가 하나라도 있으면 1.
+
+import { spawnSync } from "node:child_process";
+
+const STEPS = [
+  "bash scripts/lint-agent-prompts.sh",
+  "node --test test/shell-portability.test.mjs",
+  "node scripts/smoke-test.mjs",
+  "bash scripts/gate-policy-test.sh",
+  "node test/install-lib.test.mjs",
+  "node test/skill-mcp-registry.test.mjs",
+  "node --test test/state-write-guard.test.mjs",
+  "node --test test/state-sh-schema.test.mjs",
+  "node --test test/state-sh-init-review-shape.test.mjs",
+  "node --test test/doctor-phantom-scan.test.mjs",
+  "node --test test/mcp-secret-injector.test.mjs",
+  "node --test test/poll-sub-session.test.mjs",
+  "node --test test/tmux-monitor.test.mjs",
+  "node --test test/gate-already-done-block.test.mjs",
+  "node --test test/gate-hybrid-first-entry.test.mjs",
+  "node --test test/gate-post-pr-verify.test.mjs",
+  "node --test test/gate-post-review-verify.test.mjs",
+  "node --test test/gate-requirements-quality.test.mjs",
+  "node --test test/worktree-sync-gate.test.mjs",
+  "node --test test/planner-prompt-early-exit.test.mjs",
+  "node --test test/plugin-bug-fixes.test.mjs",
+  "node --test test/omo-tmux-detection.test.mjs",
+  "node --test test/empty-output-done-override.test.mjs",
+  "node --test test/tmux-monitor-orphan.test.mjs",
+  "node --test test/poll-permission-scope.test.mjs",
+  "node --test test/dispatch-review-fixes.test.mjs",
+  "node --test test/dispatch-stage-redispatch.test.mjs",
+  "node --test test/dispatch-stage-dev-resume-prompt.test.mjs",
+  "node --test test/logger.test.mjs",
+  "node --test test/redact-secrets.test.mjs",
+  "node --test test/commit-atomicity-verify.test.mjs",
+  "node --test test/verdict-reason-injection.test.mjs",
+  "node --test test/verdict-hash-normalize.test.mjs",
+  "node --test test/session-index-fallback.test.mjs",
+  "node --test test/plugin-exports-shape.test.mjs",
+  "node --test test/stale-worktree-recovery.test.mjs",
+  "node --test test/rollback-commits.test.mjs",
+  "node --test test/config-dir-home-fallback.test.mjs",
+  "node --test test/state-path-migration.test.mjs",
+  "node --test test/state-path-relative.test.mjs",
+  "node --test test/with-fallback-no-bun.test.mjs",
+  "node --test test/doctor-exit-code.test.mjs",
+  "node --test test/example-config-portability.test.mjs"
+];
+
+const results = [];
+for (const [i, step] of STEPS.entries()) {
+  const [cmd, ...args] = step.split(" ");
+  process.stdout.write(`\n\u001b[36m[${String(i + 1).padStart(2)}/${STEPS.length}]\u001b[0m ${step}\n`);
+  const r = spawnSync(cmd, args, { stdio: "inherit" });
+  const code = r.status === null ? 1 : r.status;
+  results.push({ step, code });
+}
+
+const failed = results.filter((r) => r.code !== 0);
+process.stdout.write(`\n${"=".repeat(70)}\n`);
+if (failed.length === 0) {
+  process.stdout.write(`\u001b[32m[run-tests] ${results.length}개 단계 전부 통과\u001b[0m\n`);
+  process.exit(0);
+}
+process.stdout.write(`\u001b[31m[run-tests] ${failed.length}/${results.length} 단계 실패:\u001b[0m\n`);
+for (const f of failed) process.stdout.write(`  \u001b[31m\u2716\u001b[0m ${f.step}  (exit ${f.code})\n`);
+process.exit(1);

package/scripts/test-ubuntu.sh

@@ -0,0 +1,155 @@
+#!/usr/bin/env bash
+# test-ubuntu.sh — run the test suite inside the Ubuntu image the plugin targets.
+#
+# 사용법:
+#   scripts/test-ubuntu.sh                 # npm test 전체 실행
+#   scripts/test-ubuntu.sh npm run build   # 임의 명령 실행
+#   scripts/test-ubuntu.sh bash            # 컨테이너 셸 진입 (디버깅)
+#
+# 왜 필요한가:
+#   플러그인은 Ubuntu 에서 개발·운영되지만 테스트는 macOS 에서도 자주 돌린다.
+#   두 환경은 libc ctype 테이블, /var 심볼릭 링크, bash 3.2 vs 5.x 에서 갈리며
+#   실제로 이 세 가지가 모두 테스트 실패로 나타난 적이 있다. 이 스크립트는
+#   Linux 쪽 결과를 어느 호스트에서든 재현할 수 있게 한다.
+#
+# 동작:
+#   - 저장소를 /w 에 bind-mount 한다.
+#   - node_modules 는 호스트의 .docker-test/node_modules 를 겹쳐 mount 한다.
+#     (macOS 용 node_modules 를 덮어쓰지 않기 위함 — 네이티브 바이너리가 다르다)
+#   - 호스트 uid 를 담은 /etc/passwd 를 만들어 넣는다. 이게 없으면 os.homedir()
+#     가 HOME 미설정 시 getpwuid 폴백에서 ENOENT 로 죽어 실제 Ubuntu 장비에서는
+#     나지 않는 실패가 발생한다 (test/config-dir-home-fallback.test.mjs).
+#   - package-lock.json 이 바뀐 경우에만 npm ci 를 다시 돌린다.
+#   - 호스트 uid/gid 로 실행해 root 소유 파일이 생기지 않게 한다.
+#
+# 정리(cleanup):
+#   - 컨테이너는 `docker run --rm` 으로 종료 즉시 제거된다.
+#   - docker 데몬(OrbStack/colima VM)이 꺼져 있어서 이 스크립트가 직접 기동한
+#     경우에만 끝날 때 다시 끈다. 이미 떠 있었다면 사용자의 다른 작업을 끊지
+#     않도록 그대로 둔다. MAKDOONG2_KEEP_DOCKER=1 로 자동 종료를 끌 수 있다.
+#
+# 주의: dist/ 는 mount 된 저장소 안에 다시 빌드된다. tsc 산출물은 플랫폼
+#       독립적이므로 macOS 빌드와 내용이 같다.
+set -euo pipefail
+
+IMAGE="${MAKDOONG2_TEST_IMAGE:-makdoong2-team-test:ubuntu-24.04}"
+ROOT="$(cd "$(dirname "$0")/.." && pwd)"
+WORKDIR="$ROOT/.docker-test"
+MODULES="$WORKDIR/node_modules"
+PASSWD="$WORKDIR/passwd"
+
+RED=$'\033[0;31m'; GREEN=$'\033[0;32m'; YELLOW=$'\033[0;33m'; CYAN=$'\033[0;36m'; RESET=$'\033[0m'
+log()  { printf "%s[test-ubuntu]%s %s\n" "$CYAN" "$RESET" "$*"; }
+warn() { printf "%s[warn]%s        %s\n" "$YELLOW" "$RESET" "$*"; }
+err()  { printf "%s[error]%s       %s\n" "$RED" "$RESET" "$*" >&2; }
+
+# ---------- docker 데몬 준비 ----------
+STARTED_DAEMON=0
+DAEMON_KIND=""
+
+cleanup() {
+  local status=$?
+  # 컨테이너는 --rm 이 이미 지웠다. 여기서는 "우리가 켠 것"만 되돌린다.
+  if [ "$STARTED_DAEMON" = "1" ]; then
+    if [ "${MAKDOONG2_KEEP_DOCKER:-0}" = "1" ]; then
+      log "MAKDOONG2_KEEP_DOCKER=1 — $DAEMON_KIND 를 켠 채로 둔다"
+    else
+      log "$DAEMON_KIND 종료 (이 스크립트가 기동했으므로 원상복구)"
+      case "$DAEMON_KIND" in
+        orbstack) orb stop    >/dev/null 2>&1 || warn "orb stop 실패 — 수동으로 'orb stop'" ;;
+        colima)   colima stop >/dev/null 2>&1 || warn "colima stop 실패 — 수동으로 'colima stop'" ;;
+      esac
+    fi
+  fi
+  return "$status"
+}
+trap cleanup EXIT
+
+if ! command -v docker >/dev/null 2>&1; then
+  err "docker 를 찾을 수 없다. Docker / OrbStack / colima 중 하나가 필요하다."
+  exit 1
+fi
+
+if docker info >/dev/null 2>&1; then
+  log "docker 데몬 이미 실행 중 — 종료하지 않고 그대로 둔다"
+else
+  if command -v orb >/dev/null 2>&1; then
+    DAEMON_KIND="orbstack"; log "docker 데몬이 꺼져 있다 — OrbStack 기동"; orb start >/dev/null 2>&1 || true
+  elif command -v colima >/dev/null 2>&1; then
+    DAEMON_KIND="colima";   log "docker 데몬이 꺼져 있다 — colima 기동";   colima start >/dev/null 2>&1 || true
+  else
+    err "docker 데몬에 연결할 수 없고 자동 기동 수단(orb/colima)도 없다. Docker Desktop 을 먼저 실행하라."
+    exit 1
+  fi
+
+  for _ in $(seq 1 40); do
+    docker info >/dev/null 2>&1 && break
+    sleep 2
+  done
+  if ! docker info >/dev/null 2>&1; then
+    err "$DAEMON_KIND 기동 후에도 docker 데몬에 연결할 수 없다."
+    exit 1
+  fi
+  STARTED_DAEMON=1        # 여기서부터 cleanup 이 되돌릴 책임을 진다
+  log "$DAEMON_KIND 기동 완료 — 테스트 종료 시 자동으로 다시 끈다"
+fi
+
+# ---------- 이미지 + 마운트 준비 ----------
+log "이미지 빌드: $IMAGE"
+docker build --quiet -f "$ROOT/Dockerfile.test" -t "$IMAGE" "$ROOT" >/dev/null
+
+mkdir -p "$MODULES"
+
+# 컨테이너 안에서 호스트 uid 가 실재하는 사용자로 보이도록 최소 passwd 를 만든다.
+{
+  echo "root:x:0:0:root:/root:/bin/bash"
+  echo "tester:x:$(id -u):$(id -g):tester:/tmp:/bin/bash"
+} > "$PASSWD"
+
+# 컨테이너 안에서 실행할 명령. 인자가 없으면 전체 테스트.
+if [ "$#" -eq 0 ]; then
+  INNER_CMD="npm test"
+else
+  INNER_CMD="$(printf '%q ' "$@")"
+fi
+
+# ---------- 실행 ----------
+log "실행: $INNER_CMD"
+STATUS=0
+docker run --rm -t \
+  -v "$ROOT:/w" \
+  -v "$MODULES:/w/node_modules" \
+  -v "$PASSWD:/etc/passwd:ro" \
+  -w /w \
+  --user "$(id -u):$(id -g)" \
+  -e HOME=/tmp \
+  -e npm_config_cache=/tmp/.npm \
+  -e npm_config_update_notifier=false \
+  -e CI=true \
+  "$IMAGE" \
+  bash -euo pipefail -c '
+    git config --global --add safe.directory /w
+    git config --global user.email "test@makdoong2.local"
+    git config --global user.name  "makdoong2 test"
+
+    # package-lock.json 이 바뀌었을 때만 npm ci. 해시를 node_modules 안에 남긴다.
+    LOCK_HASH="$(sha256sum package-lock.json | cut -d" " -f1)"
+    STAMP="node_modules/.lock-hash"
+    if [ ! -f "$STAMP" ] || [ "$(cat "$STAMP")" != "$LOCK_HASH" ]; then
+      echo "[test-ubuntu] npm ci (lockfile 변경 감지)"
+      npm ci --no-audit --no-fund
+      printf "%s" "$LOCK_HASH" > "$STAMP"
+    else
+      echo "[test-ubuntu] node_modules 재사용 (lockfile 변경 없음)"
+    fi
+
+    echo "[test-ubuntu] node $(node -v) | bash $(bash --version | head -1 | sed "s/.*version //;s/ .*//") | tmux $(tmux -V | cut -d" " -f2) | LANG=$LANG"
+    '"$INNER_CMD"'
+  ' || STATUS=$?
+
+if [ "$STATUS" -eq 0 ]; then
+  printf "%s[test-ubuntu]%s %sUbuntu 실행 성공%s\n" "$CYAN" "$RESET" "$GREEN" "$RESET"
+else
+  err "Ubuntu 실행 실패 (exit $STATUS)"
+fi
+exit "$STATUS"