makdoong2-team 2.3.2 → 3.0.0

package/dist/poll-sub-session.js

@@ -314,9 +314,15 @@ export async function pollSubSession(client, sessionId, options = {}) {
                         .catch(() => undefined);
                 }
                 else {
+                    // 두 실패를 뭉뚱그리지 않는다. "허용 경로 밖" 은 대안 경로를 알려주면
+                    // 회복 가능하고, "external_directory 가 아닌 권한" 은 경로와 무관한
+                    // 에이전트 설정 문제라 조치가 정반대다. 종전에는 한 문장이었다.
+                    const reason = p.permission === "external_directory"
+                        ? "outside_allowed_roots"
+                        : "non_external_permission";
                     err(`[pollSubSession] PERMISSION_STALL session=${sessionId} polls=${pollCount}` +
                         ` permissionID=${p.id} type=${p.permission} patterns=${JSON.stringify(p.patterns)}` +
-                        ` — auto-rejecting (outside worktree scope or non-external_directory)`);
+                        ` reason=${reason} — auto-rejecting`);
                     await client.permission
                         .reply({ path: { requestID: p.id }, body: { reply: "reject" } })
                         .catch(() => undefined);
@@ -329,6 +335,7 @@ export async function pollSubSession(client, sessionId, options = {}) {
                         permissionID: p.id,
                         permissionType: p.permission,
                         permissionPatterns: p.patterns,
+                        permissionReason: reason,
                     };
                 }
             }
@@ -379,6 +386,7 @@ export async function pollSubSession(client, sessionId, options = {}) {
                 permissionID: stalledPerm?.id,
                 permissionType: stalledPerm?.permission,
                 permissionPatterns: stalledPerm?.patterns,
+                permissionReason: "tool_call_stall",
             };
         }
         // Message stall: sub-agent registered (sessionEverAppeared) and appears
@@ -601,17 +609,37 @@ export function pollOutcomeToLegacy(outcome) {
             };
         case "aborted":
             return { text: `(aborted: ${outcome.reason})`, success: false };
-        case "permission_stall":
+        case "permission_stall": {
+            // 사유별로 **처방이 다르다.** 종전에는 셋을 한 문장으로 보고해서, 워크스페이스
+            // 밖 접근(대안 경로를 알려주면 끝날 일)과 부분 설치(재설치가 필요한 일)와
+            // 에이전트 오설정이 전부 같은 메시지로 나왔다.
+            const detail = outcome.permissionType
+                ? `${outcome.permissionType} permission (id=${outcome.permissionID}` +
+                    `${outcome.permissionPatterns ? `, patterns=${JSON.stringify(outcome.permissionPatterns)}` : ""})`
+                : "an unidentified permission request";
+            const remedy = (() => {
+                switch (outcome.permissionReason) {
+                    case "outside_allowed_roots":
+                        return "워크스페이스 밖 경로 접근이라 차단됐다. " +
+                            "임시 파일이 필요하면 `/tmp` 이 아니라 worktree 안의 " +
+                            "`.makdoong2-team/<이슈키>/tmp/` 에 만들어라 — 그 경로는 cwd 안이라 승인이 필요 없고, " +
+                            "git exclude 와 worktree 동기화 대상이다. `/tmp` 에 쓴 것은 동기화도 커밋도 되지 않아 조용히 사라진다.";
+                    case "non_external_permission":
+                        return "external_directory 가 아닌 권한 요청이 서브에이전트에 도달했다 — 경로 문제가 아니라 " +
+                            "에이전트 permission 설정 문제다. 해당 에이전트의 frontmatter `permission:` 블록을 점검하라 " +
+                            "(정식 키는 `edit`; `write` 키는 존재하지 않아 조용히 무시된다).";
+                    default:
+                        return "대기 대상을 특정하지 못했다. 점검: opencode.json 의 permission.external_directory 시드 존재 여부 " +
+                            "(`npx makdoong2-team doctor`).";
+                }
+            })();
             return {
-                text: outcome.permissionType
-                    ? `(permission_stall: sub-agent blocked on ${outcome.permissionType} permission ` +
-                        `(id=${outcome.permissionID}${outcome.permissionPatterns ? `, patterns=${JSON.stringify(outcome.permissionPatterns)}` : ""}) ` +
-                        `— cannot be answered in subagent context; aborted after ${outcome.stalledMs}ms)`
-                    : `(permission_stall: sub-agent tool call stalled for ${outcome.stalledMs}ms — likely waiting for a permission approval ` +
-                        `that cannot be answered in subagent context; pending permission could not be identified (permission.list unavailable or empty). ` +
-                        `점검: opencode.json 의 permission.external_directory 시드 존재 여부 (npx makdoong2-team doctor))`,
+                text: `(permission_stall: sub-agent blocked on ${detail} — cannot be answered in subagent context; ` +
+                    `aborted after ${outcome.stalledMs}ms. reason=${outcome.permissionReason ?? "unknown"})\n` +
+                    `→ ${remedy}`,
                 success: false,
             };
+        }
         case "session_gone":
             return {
                 text: outcome.reason === "message_stall"

package/gates/stage-analysis-verify.sh

@@ -5,7 +5,8 @@
 #
 # 진입 조건:
 #   1. state.json 존재
-#   2. 1_planning.scope substage done=true
+#   2. 1_planning.requirements substage done=true + 승인 + 품질 게이트
+#      (scope 흡수 전 stage3-scope-verify.sh 가 하던 검사를 그대로 이어받는다)
 #
 # SKIP 조건:
 #   프로젝트 루트에 build tool 마커 파일이 하나도 없음
@@ -41,11 +42,68 @@ q() {
 ANALYSIS_DONE="$(q '.stages."2_implementation".substages."analysis".done')"
 [ "$ANALYSIS_DONE" != "true" ] || fail "이미 done=true 완료됨 — auto_advance_stage 로 다음 단계 진행"
 
-# 3. scope substage 완료 확인
-SCOPE_DONE="$(q '.stages."1_planning".substages."scope".done')"
-[ "$SCOPE_DONE" = "true" ] || fail "1_planning.scope 미완료 (done=$SCOPE_DONE)"
+# 3. requirements substage 완료 + 승인 + 품질 확인
+#
+# 종전에는 `1_planning.scope` substage 의 done 만 봤다. scope 가 requirements 로
+# 흡수되면서(범위 확정은 요구사항 확정과 같은 판단의 연속이다) 그 자리를 지키던
+# stage3-scope-verify.sh 의 검사가 통째로 이 게이트로 넘어왔다. 검사를 옮기지 않고
+# 게이트만 지우면 승인·모호성·spec drift 검증이 파이프라인에서 사라진다.
+REQ_DONE="$(q '.stages."1_planning".substages."requirements".done')"
+[ "$REQ_DONE" = "true" ] || fail "1_planning.requirements 미완료 (done=$REQ_DONE)"
+
+if [ "$(q '.policy.auto_approve."1_planning.requirements"')" != "true" ]; then
+  [ "$(q '.stages."1_planning".substages."requirements".approved_by_user')" = "true" ] \
+    || fail "requirements substage 사용자 승인 없음 (또는 .policy.auto_approve.\"1_planning.requirements\" 미설정)"
+  if [ "$(q '.stages."1_planning".substages."requirements".verification_pending')" = "true" ]; then
+    fail "requirements substage 검증 대기 중 (verification_pending) — 사용자 승인 후 approved_by_user를 기록하라"
+  fi
+fi
+
+if [ "$(q '.stages."1_planning".substages."requirements".interview_required')" = "true" ]; then
+  [ "$(q '.stages."1_planning".substages."requirements".interview_completed')" = "true" ] \
+    || fail "requirements substage 인터뷰 미완료 — 모든 미결 항목 해소 후 interview_completed=true 기록 필요"
+fi
+
+# --- 요구사항 품질 게이트 (조건부 — 마커 존재 시에만 검사, 구형 state 호환) ---
+# 1) Ambiguity Score 수렴: 기록된 값이 0.2 초과면 요구사항 미수렴 (stages/02-requirements.md)
+AMB="$(q '.stages."1_planning".substages."requirements".ambiguity_score')"
+if [ "$AMB" != "__MISSING__" ] && [ "$AMB" != "null" ] && [ -n "$AMB" ]; then
+  awk -v a="$AMB" 'BEGIN { exit (a + 0 <= 0.2) ? 0 : 1 }' \
+    || fail "ambiguity_score=$AMB > 0.2 — 요구사항 미수렴. 인터뷰로 미결 항목 해소 후 재산정하라"
+fi
+
+# 2) 명세 동결(spec drift) 검증: spec_hash 기록 시 draft 파일 해시 재계산 일치 필요
+SPEC_HASH="$(q '.stages."1_planning".substages."requirements".spec_hash')"
+if [ "$SPEC_HASH" != "__MISSING__" ] && [ "$SPEC_HASH" != "null" ] && [ -n "$SPEC_HASH" ]; then
+  DRAFT="$(q '.stages."1_planning".substages."requirements".draft_path')"
+  ROOT="$("$SCRIPTS/state.sh" root)"
+  # 마커 누락과 파일 부재를 구분해서 알린다. 둘 다 "확정 명세 파일 없음" 으로
+  # 뭉뚱그리면 파일은 멀쩡하고 마커만 빠진 흔한 경우(issue #6-①)에 무엇을 해야
+  # 하는지 알 수 없다.
+  DEFAULT_DRAFT=".makdoong2-team/${ISSUE}/requirements-draft.md"
+  if [ "$DRAFT" = "__MISSING__" ] || [ "$DRAFT" = "null" ] || [ -z "$DRAFT" ]; then
+    if [ -f "$ROOT/$DEFAULT_DRAFT" ]; then
+      fail "spec_hash 는 기록됐는데 draft_path 마커가 없다 — 파일은 ${DEFAULT_DRAFT} 에 있다.
+  복구: (1) sha256sum \"${ROOT}/${DEFAULT_DRAFT}\" 가 spec_hash(${SPEC_HASH}) 와 같은지 대조하고,
+        (2) 같으면 state.sh set 으로 requirements.draft_path 에 \"${DEFAULT_DRAFT}\" 를 기록한다,
+        (3) 다르면 명세가 동결 후 변경된 것이므로 requirements substage 를 재작업한다."
+    fi
+    fail "spec_hash 는 기록됐는데 draft_path 마커도 확정 명세 파일도 없다 — requirements substage 를 재작업하라 (stages/02-requirements.md)"
+  fi
+  # "생성된 적 없음" 을 안내에서 빼면 복구 방향을 잘못 잡는다 — 실제로 가장 흔한
+  # 경우인데 종전 메시지는 동기화 누락·삭제만 언급해 리더가 동기화 문제부터
+  # 의심했다 (issue #8).
+  [ -f "$ROOT/$DRAFT" ] \
+    || fail "draft_path=${DRAFT} 마커는 있으나 파일이 없다 (기준 경로 ${ROOT}). 가능한 원인 순서대로:
+  (1) 애초에 생성된 적 없음 — planner 가 마커만 기록하고 파일 생성에 실패한 경우. requirements substage 를 재작업해 쓰기 툴로 초안부터 생성하라 (stages/02-requirements.md),
+  (2) worktree 동기화 누락 — 다른 cwd(main repo/worktree)의 같은 상대경로에 파일이 있는지 확인,
+  (3) 파일이 삭제됨 — 삭제 경위 확인 후 requirements 재작업"
+  ACTUAL="$(sha256sum "$ROOT/$DRAFT" | cut -d' ' -f1)"
+  [ "$ACTUAL" = "$SPEC_HASH" ] \
+    || fail "확정 명세 무단 변경 감지 (spec drift) — 동결 후 변경은 사용자 재승인 + spec_hash 재기록 절차만 허용 (stages/02-requirements.md)"
+fi
 
-# 3. SKIP 판정: worktree 루트에서 build tool 마커 파일 존재 여부
+# 4. SKIP 판정: worktree 루트에서 build tool 마커 파일 존재 여부
 WT="$(q '.worktree' | tr -d '"')"
 [ -d "$WT" ] || WT="$(pwd)"
 

package/gates/stage4-dev-verify.sh

@@ -23,7 +23,7 @@ assert_worktree_sibling(){
   [ "$pwt" = "$pmain" ] || fail "worktree가 메인 repo의 형제 디렉토리가 아님: $wt (메인: $main)"
 }
 [ "$(q '.stages."2_implementation".substages."dev".done')" != "true" ] || fail "이미 done=true 완료됨 — auto_advance_stage 로 다음 단계 진행"
-[ "$(q '.stages."1_planning".substages."scope".done')" = "true" ] || fail "scope substage 미완료"
+[ "$(q '.stages."1_planning".substages."requirements".done')" = "true" ] || fail "requirements substage 미완료"
 # analysis substage 는 게이트가 SKIP 시 done=true 로 마킹하므로 skipped 여부와 무관하게 done=true 검사만 하면 된다.
 [ "$(q '.stages."2_implementation".substages."analysis".done')" = "true" ] || fail "analysis substage 미완료 (skipped 도 done=true 로 처리됨)"
 
@@ -31,11 +31,11 @@ WT="$(q '.worktree')"
 [ -n "$WT" ] && [ "$WT" != "__MISSING__" ] || fail "worktree 경로 미설정 — state.json의 .worktree 필드 확인"
 assert_worktree_sibling "$WT"
 
-if [ "$(q '.policy.auto_approve."1_planning.scope"')" != "true" ]; then
-  [ "$(q '.stages."1_planning".substages."scope".approved_by_user')" = "true" ] \
-    || fail "scope substage 사용자 승인 없음 (또는 .policy.auto_approve.\"1_planning.scope\" 미설정)"
-  if [ "$(q '.stages."1_planning".substages."scope".verification_pending')" = "true" ]; then
-    fail "scope substage 검증 대기 중 (verification_pending) — 사용자 승인 후 approved_by_user를 기록하라"
+if [ "$(q '.policy.auto_approve."1_planning.requirements"')" != "true" ]; then
+  [ "$(q '.stages."1_planning".substages."requirements".approved_by_user')" = "true" ] \
+    || fail "requirements substage 사용자 승인 없음 (또는 .policy.auto_approve.\"1_planning.requirements\" 미설정)"
+  if [ "$(q '.stages."1_planning".substages."requirements".verification_pending')" = "true" ]; then
+    fail "requirements substage 검증 대기 중 (verification_pending) — 사용자 승인 후 approved_by_user를 기록하라"
   fi
 fi
 echo "MAKDOONG2-GATE OK: 2_implementation.dev"

package/gates/verify.sh

@@ -23,8 +23,6 @@ case "$TARGET" in
     echo "MAKDOONG2-GATE OK: $TARGET (jira validation handled by extension gate)" ;;
   1_planning.requirements)
     "$HERE/stage2-requirements-verify.sh" "$ISSUE" ;;
-  1_planning.scope)
-    "$HERE/stage3-scope-verify.sh" "$ISSUE" ;;
   2_implementation.analysis)
     "$HERE/stage-analysis-verify.sh" "$ISSUE" ;;
   2_implementation.dev)

package/opencode.json.example

@@ -34,7 +34,6 @@
     "verify_stage": true,
     "dispatch_stage": true,
     "dispatch_verifier": true,
-    "dispatch_research": true,
     "auto_advance_stage": true,
     "get_fallback_model": true
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "makdoong2-team",
-  "version": "2.3.2",
+  "version": "3.0.0",
   "publishConfig": {
     "access": "public"
   },

package/scripts/gate-policy-test.sh

@@ -51,44 +51,45 @@ expect(){
 echo ""
 echo "[gate-policy-test] 조건부 auto-approve 게이트 분기"
 echo ""
-echo "── stage3-scope-verify (2→3) ──"
+echo "── stage-analysis-verify (requirements→analysis) ──"
+# scope substage 흡수로 stage-analysis-verify.sh 의 승인·정합성 검사가 이 게이트로 이관됐다.
 rm -f "$REPORT"
 
 # minor (기본): auto_approve true → 승인/pending 무시하고 통과
 mk '.stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."requirements".verification_pending=true | .policy={category:"minor",auto_approve:{"1_planning.requirements":true}}'
-expect "minor 기본 auto-approve (승인 없이 통과)" pass stage3-scope-verify.sh
+expect "minor 기본 auto-approve (승인 없이 통과)" pass stage-analysis-verify.sh
 
 # major (기본): auto_approve true → 통과 (minor 와 동일 흐름 — category 는 라벨용)
 mk '.stages."1_planning".substages."requirements".done=true | .policy={category:"major",auto_approve:{"1_planning.requirements":true}}'
-expect "major 기본도 요구사항 게이트는 무인 통과" pass stage3-scope-verify.sh
+expect "major 기본도 요구사항 게이트는 무인 통과" pass stage-analysis-verify.sh
 
 # legacy(no policy) + 미승인 → 차단
 mk '.stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."requirements".approved_by_user=false'
-expect "구형 state 미승인 → 차단" block stage3-scope-verify.sh
+expect "구형 state 미승인 → 차단" block stage-analysis-verify.sh
 
 # legacy(no policy) + 승인 → 통과
 mk '.stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."requirements".approved_by_user=true | .stages."1_planning".substages."requirements".verification_pending=false'
-expect "구형 state 사용자 승인 → 통과" pass stage3-scope-verify.sh
+expect "구형 state 사용자 승인 → 통과" pass stage-analysis-verify.sh
 
 # auto-approve여도 인터뷰 미완료면 차단 (정합성 게이트는 독립)
 mk '.stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."requirements".interview_required=true | .stages."1_planning".substages."requirements".interview_completed=false | .policy={category:"minor",auto_approve:{"1_planning.requirements":true}}'
-expect "인터뷰 미완료 → 차단 (auto와 무관)" block stage3-scope-verify.sh
+expect "인터뷰 미완료 → 차단 (auto와 무관)" block stage-analysis-verify.sh
 
 echo ""
 echo "── stage4-dev-verify (3→4) ──"
 # worktree 경로 검증을 위해 실제 git worktree 생성 (메인 repo의 형제)
 WT="$(dirname "$TMP")/$(basename "$TMP")-TEST-1"
 (cd "$ROOT" && git worktree add "$WT" -b test-branch 2>/dev/null || true)
-mk '.stages."1_planning".substages."scope".done=true | .stages."1_planning".substages."scope".verification_pending=true | .stages."2_implementation".substages."analysis".done=true | .policy={category:"minor",auto_approve:{"1_planning.scope":true}} | .worktree="'"$WT"'"'
+mk '.stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."requirements".verification_pending=true | .stages."2_implementation".substages."analysis".done=true | .policy={category:"minor",auto_approve:{"1_planning.requirements":true}} | .worktree="'"$WT"'"'
 expect "minor auto-approve → 통과" pass stage4-dev-verify.sh
-mk '.stages."1_planning".substages."scope".done=true | .stages."1_planning".substages."scope".approved_by_user=false | .stages."2_implementation".substages."analysis".done=true | .worktree="'"$WT"'"'
+mk '.stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."requirements".approved_by_user=false | .stages."2_implementation".substages."analysis".done=true | .worktree="'"$WT"'"'
 expect "구형 state 미승인 → 차단" block stage4-dev-verify.sh
-mk '.stages."1_planning".substages."scope".done=true | .stages."1_planning".substages."scope".approved_by_user=true | .stages."2_implementation".substages."analysis".done=true | .worktree="'"$WT"'"'
+mk '.stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."requirements".approved_by_user=true | .stages."2_implementation".substages."analysis".done=true | .worktree="'"$WT"'"'
 expect "구형 state 승인 → 통과" pass stage4-dev-verify.sh
 
 echo ""
 echo "── stage6-commit-verify (5→6) ──"
-TESTS='.stages."1_planning".substages."jira".done=true | .stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."scope".done=true | .stages."2_implementation".substages."dev".done=true | .stages."2_implementation".substages."test".unit="pass" | .stages."2_implementation".substages."test".integration="skip" | .stages."2_implementation".substages."test".coverage="exempt"'
+TESTS='.stages."1_planning".substages."jira".done=true | .stages."1_planning".substages."requirements".done=true | .stages."2_implementation".substages."dev".done=true | .stages."2_implementation".substages."test".unit="pass" | .stages."2_implementation".substages."test".integration="skip" | .stages."2_implementation".substages."test".coverage="exempt"'
 
 # minor (기본): commit auto_approve true → 보고서/승인 없이 통과
 rm -f "$REPORT"
@@ -130,20 +131,20 @@ expect "구형 state → 휴먼 게이트 없이 통과" pass stage6-commit-veri
 
 echo ""
 echo "── stage8-review-verify (7→8) ──"
-BV='.stages."1_planning".substages."jira".done=true | .stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."scope".done=true | .stages."2_implementation".substages."dev".done=true | .stages."2_implementation".substages."test".unit="pass" | .stages."3_delivery".substages."commit".done=true | .stages."3_delivery".substages."pr".draft_url="http://pr/1" | .stages."3_delivery".substages."pr".body_validation={no_orphan_scenarios:true,template_match:true,section_content_match:true} | .stages."3_delivery".substages."pr".reviewer_added=true | .stages."3_delivery".substages."review".comments=1 | .stages."3_delivery".substages."review".all_comments_inline=true'
+BV='.stages."1_planning".substages."jira".done=true | .stages."1_planning".substages."requirements".done=true | .stages."2_implementation".substages."dev".done=true | .stages."2_implementation".substages."test".unit="pass" | .stages."3_delivery".substages."commit".done=true | .stages."3_delivery".substages."pr".draft_url="http://pr/1" | .stages."3_delivery".substages."pr".body_validation={no_orphan_scenarios:true,template_match:true,section_content_match:true} | .stages."3_delivery".substages."pr".reviewer_added=true | .stages."3_delivery".substages."review".comments=1 | .stages."3_delivery".substages."review".all_comments_inline=true'
 mk "$BV | .policy={category:\"minor\",auto_approve:{\"3_delivery.pr\":true}}"
 expect "minor auto-approve → 통과" pass stage8-review-verify.sh
 mk "$BV | .stages.\"3_delivery\".substages.\"pr\".approved_by_user=false"
 expect "구형 state 미승인 → 차단" block stage8-review-verify.sh
 mk "$BV | .stages.\"3_delivery\".substages.\"pr\".approved_by_user=true | .stages.\"3_delivery\".substages.\"pr\".verification_pending=false"
 expect "구형 state 승인 → 통과" pass stage8-review-verify.sh
-mk '.stages."1_planning".substages."jira".done=true | .stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."scope".done=true | .stages."2_implementation".substages."dev".done=true | .stages."2_implementation".substages."test".unit="pass" | .stages."3_delivery".substages."commit".done=true | .stages."3_delivery".substages."pr".draft_url="http://pr/1" | .stages."3_delivery".substages."pr".body_validation={no_orphan_scenarios:false,template_match:true,section_content_match:true} | .stages."3_delivery".substages."pr".reviewer_added=true | .stages."3_delivery".substages."review".comments=1 | .stages."3_delivery".substages."review".all_comments_inline=true | .policy={category:"minor",auto_approve:{"3_delivery.pr":true}}'
+mk '.stages."1_planning".substages."jira".done=true | .stages."1_planning".substages."requirements".done=true | .stages."2_implementation".substages."dev".done=true | .stages."2_implementation".substages."test".unit="pass" | .stages."3_delivery".substages."commit".done=true | .stages."3_delivery".substages."pr".draft_url="http://pr/1" | .stages."3_delivery".substages."pr".body_validation={no_orphan_scenarios:false,template_match:true,section_content_match:true} | .stages."3_delivery".substages."pr".reviewer_added=true | .stages."3_delivery".substages."review".comments=1 | .stages."3_delivery".substages."review".all_comments_inline=true | .policy={category:"minor",auto_approve:{"3_delivery.pr":true}}'
 expect "body_validation false → 차단 (auto와 무관)" block stage8-review-verify.sh
 
-BSS='.stages."1_planning".substages."jira".done=true | .stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."scope".done=true | .stages."2_implementation".substages."dev".done=true | .stages."2_implementation".substages."test".unit="pass" | .stages."3_delivery".substages."commit".done=true | .stages."3_delivery".substages."pr".draft_url="http://pr/1" | .stages."3_delivery".substages."pr".body_validation={no_orphan_scenarios:true,template_match:true,section_content_match:true} | .stages."3_delivery".substages."pr".reviewer_self_skipped=true | .stages."3_delivery".substages."review".comments=1 | .stages."3_delivery".substages."review".all_comments_inline=true'
+BSS='.stages."1_planning".substages."jira".done=true | .stages."1_planning".substages."requirements".done=true | .stages."2_implementation".substages."dev".done=true | .stages."2_implementation".substages."test".unit="pass" | .stages."3_delivery".substages."commit".done=true | .stages."3_delivery".substages."pr".draft_url="http://pr/1" | .stages."3_delivery".substages."pr".body_validation={no_orphan_scenarios:true,template_match:true,section_content_match:true} | .stages."3_delivery".substages."pr".reviewer_self_skipped=true | .stages."3_delivery".substages."review".comments=1 | .stages."3_delivery".substages."review".all_comments_inline=true'
 mk "$BSS | .policy={category:\"minor\",auto_approve:{\"3_delivery.pr\":true}}"
 expect "reviewer_self_skipped=true → reviewer 없어도 통과" pass stage8-review-verify.sh
-mk '.stages."1_planning".substages."jira".done=true | .stages."1_planning".substages."requirements".done=true | .stages."1_planning".substages."scope".done=true | .stages."2_implementation".substages."dev".done=true | .stages."2_implementation".substages."test".unit="pass" | .stages."3_delivery".substages."commit".done=true | .stages."3_delivery".substages."pr".draft_url="http://pr/1" | .stages."3_delivery".substages."pr".body_validation={no_orphan_scenarios:true,template_match:true,section_content_match:true} | .stages."3_delivery".substages."review".comments=1 | .stages."3_delivery".substages."review".all_comments_inline=true | .policy={category:"minor",auto_approve:{"3_delivery.pr":true}}'
+mk '.stages."1_planning".substages."jira".done=true | .stages."1_planning".substages."requirements".done=true | .stages."2_implementation".substages."dev".done=true | .stages."2_implementation".substages."test".unit="pass" | .stages."3_delivery".substages."commit".done=true | .stages."3_delivery".substages."pr".draft_url="http://pr/1" | .stages."3_delivery".substages."pr".body_validation={no_orphan_scenarios:true,template_match:true,section_content_match:true} | .stages."3_delivery".substages."review".comments=1 | .stages."3_delivery".substages."review".all_comments_inline=true | .policy={category:"minor",auto_approve:{"3_delivery.pr":true}}'
 expect "reviewer_added·reviewer_self_skipped 모두 없음 → 차단" block stage8-review-verify.sh
 
 echo ""

package/scripts/install-lib.mjs

@@ -134,7 +134,7 @@ const LEGACY_RELATIVE_PLUGIN_REF = "./plugins/makdoong2-team/src/opencode-plugin
 const LEGACY_ABSOLUTE_TS_SUFFIX = "/src/opencode-plugin.ts";
 const LEGACY_SCOPED_NAME = "@local/makdoong2-team";
 const PLUGIN_PACKAGE_NAME = "makdoong2-team";
-const TOOLS = ["verify_stage", "dispatch_stage", "dispatch_verifier", "dispatch_research", "auto_advance_stage", "get_fallback_model"];
+const TOOLS = ["verify_stage", "dispatch_stage", "dispatch_verifier", "auto_advance_stage", "get_fallback_model"];
 const TOOL_SEARCH_PLUGIN_PREFIX = "opencode-tool-search";
 // UTIL_SCRIPTS is no longer deployed (scripts live inside the npm module).
 // Retained for cleanup: older installs copied these into ~/.config/opencode/scripts/

package/scripts/install-lib.mts

@@ -164,7 +164,7 @@ const LEGACY_RELATIVE_PLUGIN_REF = "./plugins/makdoong2-team/src/opencode-plugin
 const LEGACY_ABSOLUTE_TS_SUFFIX = "/src/opencode-plugin.ts";
 const LEGACY_SCOPED_NAME = "@local/makdoong2-team";
 const PLUGIN_PACKAGE_NAME = "makdoong2-team";
-const TOOLS = ["verify_stage", "dispatch_stage", "dispatch_verifier", "dispatch_research", "auto_advance_stage", "get_fallback_model"];
+const TOOLS = ["verify_stage", "dispatch_stage", "dispatch_verifier", "auto_advance_stage", "get_fallback_model"];
 const TOOL_SEARCH_PLUGIN_PREFIX = "opencode-tool-search";
 // UTIL_SCRIPTS is no longer deployed (scripts live inside the npm module).
 // Retained for cleanup: older installs copied these into ~/.config/opencode/scripts/

package/scripts/model-policy.mjs

@@ -69,10 +69,6 @@ export const DEFAULT_POLICIES = Object.freeze({
         fallbacks: [{ id: "github-copilot/claude-haiku-4.5", tier: "low" }],
     },
     // Research fan-out worker — one session per source, spawned in parallel.
-    "makdoong2-researcher": {
-        primary: { id: "github-copilot/gpt-5.6-luna", variant: "xhigh", tier: "medium" },
-        fallbacks: [{ id: "github-copilot/claude-haiku-4.5", tier: "low" }],
-    },
     "makdoong2-planner": {
         primary: { id: "github-copilot/gpt-5.6-luna", variant: "xhigh", tier: "medium" },
         fallbacks: [{ id: "github-copilot/claude-haiku-4.5", tier: "low" }],

package/scripts/model-policy.mts

@@ -130,10 +130,6 @@ export const DEFAULT_POLICIES: Readonly<Record<string, AgentModelPolicy>> = Obje
     fallbacks: [{ id: "github-copilot/claude-haiku-4.5",                         tier: "low" }],
   },
   // Research fan-out worker — one session per source, spawned in parallel.
-  "makdoong2-researcher": {
-    primary:   { id: "github-copilot/gpt-5.6-luna",          variant: "xhigh",  tier: "medium" },
-    fallbacks: [{ id: "github-copilot/claude-haiku-4.5",                         tier: "low" }],
-  },
   "makdoong2-planner": {
     primary:   { id: "github-copilot/gpt-5.6-luna",          variant: "xhigh",  tier: "medium" },
     fallbacks: [{ id: "github-copilot/claude-haiku-4.5",                         tier: "low" }],

package/scripts/release.sh

@@ -160,7 +160,19 @@ info "다음 버전: $NEXT_VERSION"
 log "이번 릴리스에 포함될 커밋:"
 LAST_TAG="$(git describe --tags --abbrev=0 2>/dev/null || echo '')"
 if [ -n "$LAST_TAG" ]; then
-  git log --oneline "${LAST_TAG}..HEAD" | head -20
+  # `git log … | head -N` 을 쓰지 않는다. set -o pipefail 아래서 head 가 N 줄을
+  # 읽고 먼저 종료하면 git 이 SIGPIPE(141)로 죽고, 그 코드가 파이프라인 전체의
+  # 상태가 되어 errexit 가 **릴리스를 통째로 중단**시킨다. 커밋이 20개를 넘은
+  # 적이 없어 지금까지 잠복해 있었고, 48개 릴리스에서 실제로 발화했다.
+  # "1 파일 = 1 commit" 규약을 지키면 커밋 수는 앞으로도 쉽게 20을 넘는다.
+  # 개수 제한은 git 에게 맡긴다 — 파이프가 없어지면 발화 조건 자체가 사라진다.
+  COMMIT_TOTAL="$(git rev-list --count "${LAST_TAG}..HEAD")"
+  git log --oneline -20 "${LAST_TAG}..HEAD"
+  # `[ … ] && info …` 형태로 쓰지 않는다. 조건이 거짓이면 AND 리스트 전체가 1을
+  # 반환하고 errexit 가 그 자리에서 스크립트를 끝낸다 (CLAUDE.md 게이트 사례와 동형).
+  if [ "$COMMIT_TOTAL" -gt 20 ]; then
+    info "... 외 $((COMMIT_TOTAL - 20))개 (총 ${COMMIT_TOTAL}개)"
+  fi
 else
   git log --oneline -10
 fi

package/scripts/run-tests.mts

@@ -41,6 +41,9 @@ const STEPS = [
   "node test/install-lib.test.ts",
   "node test/skill-mcp-registry.test.ts",
   "node --test test/state-write-guard.test.ts",
+  "node --test test/apply-patch-paths.test.ts",
+  "node --test test/pipeline-shape.test.ts",
+  "node --test test/permission-scope-hardening.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",
@@ -82,7 +85,6 @@ const STEPS = [
   "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/stage-completion.test.ts",
   "node --test test/state-sh-write-atomicity.test.ts",
   "node --test test/gate-locale-and-path.test.ts",

package/scripts/state.sh

@@ -185,8 +185,7 @@ case "$cmd" in
       "done": false,
       "substages": {
         "jira":         {"done": false},
-        "requirements": {"done": false, "approved_by_user": false},
-        "scope":        {"done": false, "approved_by_user": false}
+        "requirements": {"done": false, "approved_by_user": false}
       }
     },
     "2_implementation": {
@@ -333,6 +332,29 @@ JSON
       | move("3_delivery.commit";       "3_delivery"; "commit")
       | move("3_delivery.pr";           "3_delivery"; "pr")
       | move("3_delivery.review";       "3_delivery"; "review")
+
+      # scope substage 흡수 (범위 확정 → requirements).
+      # 진행 중이던 워크플로우가 남긴 scope 마커를 버리지 않고 requirements 로
+      # 접는다. done / approved_by_user 는 **AND** 로 접는다 — scope 가 아직
+      # 끝나지 않았는데 requirements 만 보고 통과시키면, 흡수 전이라면 막혔을
+      # 지점을 마이그레이션이 열어주는 셈이 된다. 접은 뒤 scope 키는 지운다.
+      | (if (.stages."1_planning".substages? // {} | has("scope")) then
+             .stages."1_planning".substages.requirements =
+               ((.stages."1_planning".substages.requirements // {})
+                 + {"done": ((.stages."1_planning".substages.requirements.done // false)
+                             and (.stages."1_planning".substages.scope.done // false)),
+                    "approved_by_user":
+                      ((.stages."1_planning".substages.requirements.approved_by_user // false)
+                       and (.stages."1_planning".substages.scope.approved_by_user // false))})
+           | del(.stages."1_planning".substages.scope)
+         else . end)
+      # auto_approve 는 flat 표기를 유지하는 유일한 예외라 여기서 같이 옮긴다.
+      | (if (.policy.auto_approve? // {} | has("1_planning.scope")) then
+             .policy.auto_approve = (.policy.auto_approve
+               | ."1_planning.requirements" =
+                   ((."1_planning.requirements" // false) and ."1_planning.scope")
+               | del(."1_planning.scope"))
+         else . end)
     '
     echo "migrated[$ISSUE] $P" ;;
   *) echo "usage: state.sh {root|issue|init|status|get|set|append|migrate} ..." >&2; exit 64 ;;

package/skills/_lib/load-secret.sh

@@ -42,16 +42,26 @@ EOF
 
   # `jq -e` fails when the file cannot be parsed; capture stderr for diagnostics.
   local token jq_err
-  jq_err=$(mktemp 2>/dev/null || echo "/tmp/load-secret.$$.err")
-  token=$(jq -r --arg k "$var_name" '.secrets[$k] // empty' "$cfg_file" 2>"$jq_err")
+  # 임시 파일은 **설정 파일과 같은 디렉토리**에 만든다. 종전 폴백은 `/tmp` 을
+  # 명령문에 노출시켰는데, opencode 1.18 의 ShellTool 은 bash 명령이 참조하는
+  # 디렉토리마다 external_directory 승인을 묻고, 서브에이전트 세션에서 그 요청은
+  # 자동 거부 + 세션 abort 로 이어진다 (워크스페이스 밖 경로이므로).
+  # mktemp 실패 시에도 워크스페이스 밖으로 새지 않게 한다.
+  jq_err=""
+  jq_err=$(mktemp "$(dirname "$cfg_file")/.load-secret.XXXXXX" 2>/dev/null) || jq_err=""
+  # mktemp 실패 시 /dev/null 로 흘린다. 이때 `jq_err` 는 빈 문자열로 남으므로
+  # 아래 정리 단계가 **실제로 만든 파일만** 지운다 — `rm -f /dev/null` 은
+  # root 로 도는 환경(이슈 #8 로그의 /root/.nvm)에서 진짜로 장치 노드를 지운다.
+  local jq_err_sink="${jq_err:-/dev/null}"
+  token=$(jq -r --arg k "$var_name" '.secrets[$k] // empty' "$cfg_file" 2>"$jq_err_sink")
   local jq_status=$?
   if [ "$jq_status" -ne 0 ]; then
     printf '[%s] makdoong2-team.json is not valid JSON: %s\n' "$tag" "$cfg_file" >&2
-    if [ -s "$jq_err" ]; then sed -e "s/^/[$tag] jq: /" "$jq_err" >&2; fi
-    rm -f "$jq_err"
+    if [ -s "$jq_err_sink" ]; then sed -e "s/^/[$tag] jq: /" "$jq_err_sink" >&2; fi
+    if [ -n "$jq_err" ]; then rm -f -- "$jq_err"; fi
     return 67
   fi
-  rm -f "$jq_err"
+  if [ -n "$jq_err" ]; then rm -f -- "$jq_err"; fi
 
   if [ -z "$token" ] || [ "$token" = "null" ]; then
     cat >&2 <<EOF

package/src/hooks/session-start.sh

@@ -55,7 +55,6 @@ jq -r '
   # Planning substages
   ($p1.substages.jira // {} | "  1_planning.jira: done=" + ((.done // false) | tostring) + (if .approved_by_user then " (approved)" else "" end)),
   ($p1.substages.requirements // {} | "  1_planning.requirements: done=" + ((.done // false) | tostring) + (if .approved_by_user then " (approved)" else "" end)),
-  ($p1.substages.scope // {} | "  1_planning.scope: done=" + ((.done // false) | tostring) + (if .approved_by_user then " (approved)" else "" end)),
   
   # Implementation substages
   ($p2.substages.analysis // {} | "  2_implementation.analysis: done=" + ((.done // false) | tostring) + (if .skipped then " (skipped)" else "" end)),