makdoong2-team 2.3.1 → 3.0.0

package/dist/apply-patch-paths.js

@@ -0,0 +1,114 @@
+/**
+ * apply_patch 대상 경로 추출 — 순수 계약.
+ *
+ * ── 왜 필요한가 ──
+ * opencode 1.18 의 `ToolRegistry.tools` 는 모델 id 에 따라 **파일 쓰기 툴 자체를
+ * 바꿔 끼운다**. 1.18.23 바이너리에서 확인한 필터:
+ *
+ *   const isGpt = modelID.includes("gpt-") && !modelID.includes("oss") && !modelID.includes("gpt-4")
+ *   if (tool.id === apply_patch) return isGpt      // gpt-5 계열에만 노출
+ *   if (tool.id === edit || tool.id === write) return !isGpt   // 그 외 모델에만 노출
+ *
+ * 즉 `github-copilot/gpt-5.6-luna` 같은 세션에는 `write` · `edit` 가 **툴 스키마에
+ * 아예 없고** `apply_patch` 하나만 있다. "산출물은 write 툴로 만들어라" 는 stage
+ * 지침은 그 세션에서 물리적으로 수행 불가능한 지시가 된다 (GitHub #8 재발).
+ *
+ * ── 왜 경로 파싱인가 ──
+ * `apply_patch` 의 인자는 `{ patchText }` 한 덩어리라 `filePath` 인자가 없다.
+ * 그래서 산출물 제한 훅은 대상 파일을 몰라 **보수적으로 전부 차단**했고, 그
+ * 결과 planner 에게는 초안을 만들 합법적 수단이 하나도 남지 않아 워크플로가
+ * 구조적으로 정지했다. 패치 본문에서 대상 경로를 뽑아내면 write 와 동일한
+ * 정밀도로 허용/차단을 판정할 수 있다 — 우회가 아니라 같은 규칙의 확장이다.
+ *
+ * 파싱은 opencode 의 `Patch.parsePatch` 를 그대로 모사한다 (Begin/End 마커,
+ * `*** Add|Delete|Update File:`, `*** Move to:`). 훅과 실제 적용기가 서로 다른
+ * 파일을 본다고 믿게 되면 방어가 무너지므로 관대하게 파싱하지 않는다.
+ */
+/** opencode 가 apply_patch 로 노출하는 툴 이름들 (소문자 비교 기준). */
+export const APPLY_PATCH_TOOLS = new Set(["apply_patch", "applypatch", "patch"]);
+export const isApplyPatchTool = (toolName) => APPLY_PATCH_TOOLS.has(toolName.toLowerCase());
+/**
+ * opencode 의 정규화 단계: 패치 본문이 heredoc 으로 감싸여 오는 경우를 벗긴다.
+ * (`Patch` 모듈의 동일 정규식)
+ */
+const stripHeredoc = (text) => text.match(/^(?:cat\s+)?<<['"]?(\w+)['"]?\s*\n([\s\S]*?)\n\1\s*$/)?.[2] ?? text;
+const BEGIN = "*** Begin Patch";
+const END = "*** End Patch";
+/** 패치 본문에서 대상 경로를 모두 뽑는다. 형식 위반이면 실패로 보고한다. */
+export const parsePatchTargets = (patchText) => {
+    if (typeof patchText !== "string" || patchText.trim() === "") {
+        return { ok: false, reason: "patch_text_empty" };
+    }
+    const lines = stripHeredoc(patchText.trim()).split("\n");
+    const begin = lines.findIndex((l) => l.trim() === BEGIN);
+    const end = lines.findIndex((l) => l.trim() === END);
+    if (begin === -1 || end === -1 || begin >= end) {
+        return { ok: false, reason: "missing_begin_end_markers" };
+    }
+    const paths = [];
+    for (let i = begin + 1; i < end; i++) {
+        const line = lines[i];
+        if (line.startsWith("*** Add File:")) {
+            const p = line.slice("*** Add File:".length).trim();
+            if (!p)
+                return { ok: false, reason: "empty_path_in_add" };
+            paths.push(p);
+        }
+        else if (line.startsWith("*** Delete File:")) {
+            const p = line.slice("*** Delete File:".length).trim();
+            if (!p)
+                return { ok: false, reason: "empty_path_in_delete" };
+            paths.push(p);
+        }
+        else if (line.startsWith("*** Update File:")) {
+            const p = line.slice("*** Update File:".length).trim();
+            if (!p)
+                return { ok: false, reason: "empty_path_in_update" };
+            paths.push(p);
+            // Update 바로 다음 줄에만 올 수 있는 이동 대상. 원본과 목적지 둘 다
+            // 쓰기 대상이므로 둘 다 검사 대상에 넣는다.
+            const next = lines[i + 1];
+            if (next && next.startsWith("*** Move to:")) {
+                const mv = next.slice("*** Move to:".length).trim();
+                if (!mv)
+                    return { ok: false, reason: "empty_path_in_move" };
+                paths.push(mv);
+                i++;
+            }
+        }
+    }
+    if (paths.length === 0)
+        return { ok: false, reason: "no_target_path" };
+    return { ok: true, paths };
+};
+/**
+ * apply_patch 툴 인자에서 대상 경로를 뽑는다.
+ *
+ * 두 가지 인자 모양을 모두 받는다:
+ *  - opencode 내장 툴: `{ patchText }` (별칭 `patch`)
+ *  - OpenAI provider tool 경유: `{ callId, operation: { type, path, move_path } }`
+ */
+export const extractApplyPatchPaths = (args) => {
+    if (!args || typeof args !== "object")
+        return { ok: false, reason: "args_not_object" };
+    const a = args;
+    const op = a.operation;
+    if (op && typeof op === "object") {
+        const o = op;
+        const paths = [];
+        for (const key of ["path", "move_path", "movePath"]) {
+            const v = o[key];
+            if (typeof v === "string" && v.trim() !== "")
+                paths.push(v.trim());
+        }
+        if (paths.length > 0)
+            return { ok: true, paths };
+        return { ok: false, reason: "operation_without_path" };
+    }
+    for (const key of ["patchText", "patch_text", "patch", "input"]) {
+        const v = a[key];
+        if (typeof v === "string" && v.trim() !== "")
+            return parsePatchTargets(v);
+    }
+    return { ok: false, reason: "no_patch_text" };
+};

package/dist/config.d.ts

@@ -33,13 +33,6 @@ export interface TimeoutConfig {
     stall_escalate_threshold?: number;
 }
 export declare const DEFAULT_STALL_ESCALATE_THRESHOLD = 5;
-export interface ResearchConfig {
-    /** Max research sub-sessions spawned simultaneously. Clamped to [1, 6]. */
-    max_parallel?: number;
-    /** Per-source wall-clock budget. Shorter than a substage on purpose — a source
-     *  that cannot answer in this window is reported as failed, not waited on. */
-    timeout_minutes?: number;
-}
 export type LogLevel = "silent" | "error" | "warn" | "info" | "debug" | "trace";
 export type LogMode = "stdin" | "file";
 export interface LoggingConfig {
@@ -56,7 +49,6 @@ export interface Makdoong2Config {
         threshold?: number;
     };
     timeout?: TimeoutConfig;
-    research?: ResearchConfig;
     tmux?: TmuxConfigJson;
     worktree?: {
         extra_exclude?: string;
@@ -107,4 +99,25 @@ export declare function loadOpencodeExternalDirAllows(): string[];
  * because scripts/install-lib.mjs deploys skill directories there, not into the
  * package root. All other paths default to package-root subdirs.
  */
+/**
+ * 플러그인이 **자기 자신을 실행하기 위해** 반드시 접근해야 하는 디렉토리들.
+ *
+ * 모든 substage 가 `bash <SCRIPTS_DIR>/state.sh` 를 호출하고, 게이트·stage spec·
+ * 리서치 스킬도 전부 opencode 설정 디렉토리(또는 패키지 루트) 아래에 있다.
+ * 이들은 **워크스페이스 밖**이므로 opencode 가 `external_directory` 승인을 묻는다
+ * (1.18 `ShellTool.ask` 는 bash 명령이 참조하는 디렉토리에도 발화한다).
+ *
+ * 종전에는 그 승인 근거가 설치 시 opencode.json 에 심어둔 시드 하나뿐이었다.
+ * 시드가 남지 않은 부분 설치에서는 서브에이전트가 **자기 상태 파일조차 읽지 못한 채**
+ * PERMISSION_STALL 로 죽었다 (GitHub #8). 파일 배포만 성공한 설치는 겉보기에 정상이라
+ * 원인 규명도 어려웠다.
+ *
+ * 그래서 플러그인이 자기 경로를 **설정과 무관하게** 스스로 허용한다. opencode.json
+ * 시드는 1차 방어로 그대로 두고(사용자가 직접 승인할 때의 UX), 이것이 2차 방어다 —
+ * 다른 하드룰들과 같은 이중 방어 구조다.
+ *
+ * 허용 범위를 넓히는 변경이 아니다: 여기 담기는 경로는 플러그인이 이미 자기 손으로
+ * 실행하고 있는 것들뿐이고, 워크스페이스·`/tmp`·홈 디렉토리는 포함되지 않는다.
+ */
+export declare function pluginOwnAllowPatterns(paths?: Required<PathsConfig>): string[];
 export declare function resolvePaths(): Required<PathsConfig>;

package/dist/config.js

@@ -196,6 +196,40 @@ export function loadOpencodeExternalDirAllows() {
  * because scripts/install-lib.mjs deploys skill directories there, not into the
  * package root. All other paths default to package-root subdirs.
  */
+/**
+ * 플러그인이 **자기 자신을 실행하기 위해** 반드시 접근해야 하는 디렉토리들.
+ *
+ * 모든 substage 가 `bash <SCRIPTS_DIR>/state.sh` 를 호출하고, 게이트·stage spec·
+ * 리서치 스킬도 전부 opencode 설정 디렉토리(또는 패키지 루트) 아래에 있다.
+ * 이들은 **워크스페이스 밖**이므로 opencode 가 `external_directory` 승인을 묻는다
+ * (1.18 `ShellTool.ask` 는 bash 명령이 참조하는 디렉토리에도 발화한다).
+ *
+ * 종전에는 그 승인 근거가 설치 시 opencode.json 에 심어둔 시드 하나뿐이었다.
+ * 시드가 남지 않은 부분 설치에서는 서브에이전트가 **자기 상태 파일조차 읽지 못한 채**
+ * PERMISSION_STALL 로 죽었다 (GitHub #8). 파일 배포만 성공한 설치는 겉보기에 정상이라
+ * 원인 규명도 어려웠다.
+ *
+ * 그래서 플러그인이 자기 경로를 **설정과 무관하게** 스스로 허용한다. opencode.json
+ * 시드는 1차 방어로 그대로 두고(사용자가 직접 승인할 때의 UX), 이것이 2차 방어다 —
+ * 다른 하드룰들과 같은 이중 방어 구조다.
+ *
+ * 허용 범위를 넓히는 변경이 아니다: 여기 담기는 경로는 플러그인이 이미 자기 손으로
+ * 실행하고 있는 것들뿐이고, 워크스페이스·`/tmp`·홈 디렉토리는 포함되지 않는다.
+ */
+export function pluginOwnAllowPatterns(paths = resolvePaths()) {
+    const seen = new Set();
+    const out = [];
+    for (const dir of [paths.hooks, paths.gates, paths.scripts, paths.stages, paths.skills]) {
+        if (typeof dir !== "string" || dir.trim() === "")
+            continue;
+        const norm = dir.replace(/\\/g, "/").replace(/\/+$/, "");
+        if (norm === "" || norm === "/" || seen.has(norm))
+            continue;
+        seen.add(norm);
+        out.push(`${norm}/**`);
+    }
+    return out;
+}
 export function resolvePaths() {
     const p = loadConfig().paths ?? {};
     const root = packageRoot();

package/dist/model-fallback-policy.js

@@ -74,10 +74,6 @@ export const POLICIES = {
         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" }],