@tienne/gestalt 0.75.1 → 0.76.1

package/dist/src/review-loop/signal.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"signal.d.ts","sourceRoot":"","sources":["../../../src/review-loop/signal.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AACH,MAAM,WAAW,SAAS;IACxB,oDAAoD;IACpD,IAAI,EAAE,OAAO,CAAC;IACd,kDAAkD;IAClD,QAAQ,EAAE,OAAO,CAAC;IAClB,0BAA0B;IAC1B,OAAO,EAAE,MAAM,CAAC;IAChB,iCAAiC;IACjC,OAAO,EAAE,OAAO,CAAC;IACjB,gCAAgC;IAChC,WAAW,EAAE,OAAO,CAAC;CACtB;AAED,MAAM,MAAM,UAAU,GAAG,QAAQ,GAAG,oBAAoB,GAAG,OAAO,GAAG,cAAc,GAAG,SAAS,CAAC;AAEhG;;;;;;;;;GASG;AACH,wBAAgB,YAAY,CAAC,KAAK,EAAE,SAAS,GAAG,UAAU,CAQzD;AAED;;;;;GAKG;AACH,eAAO,MAAM,aAAa,EAAE,MAAM,CAAC,UAAU,EAAE,MAAM,CAMpD,CAAC"}

package/dist/src/review-loop/signal.js

@@ -0,0 +1,37 @@
+/**
+ * 네 값에서 다음에 할 일을 정한다.
+ *
+ * 이 도출이 문서에 없던 자리다. 감시 모드를 미루며 폴링 스크립트를 지울 때 규칙이
+ * 그 안에 있다가 함께 사라졌다. 재리뷰 판정 표는 만들어지지 않는 이름으로 분기하고
+ * 있었다. 여기 두면 네 값의 곱집합이 전부 어느 한 쪽으로 간다.
+ *
+ * 순서가 규칙의 일부다. 닫힌 PR 은 나머지를 볼 것도 없다. 작성자가 명시적으로 다시
+ * 봐달라고 눌렀으면 미대응이 남아 있어도 그 뜻을 따른다.
+ */
+export function deriveSignal(state) {
+    if (!state.open)
+        return 'CLOSED';
+    if (state.rerequested)
+        return 'REREVIEW_REQUESTED';
+    if (state.pending > 0)
+        return 'WAITING';
+    // 아직 한 번도 안 본 PR 은 볼 때다. 이걸 안 가르면 첫 라운드가 "답만 오고 코드는
+    // 그대로" 로 읽혀 사람에게 넘어간다 — 실제 PR 에 돌려보고 나온 자리다
+    if (!state.reviewed)
+        return 'READY';
+    return state.changed ? 'READY' : 'REPLIES_ONLY';
+}
+/**
+ * 신호마다 루프가 다음에 하는 일.
+ *
+ * 문서의 재리뷰 판정 표가 이 문자열을 그대로 싣는다. 테스트가 표 행과 이 값을 대조하므로
+ * 한쪽만 고치면 게이트에서 걸린다.
+ */
+export const SIGNAL_ACTION = {
+    CLOSED: '루프를 끝낸다',
+    REREVIEW_REQUESTED: '재리뷰한다 — 미대응이 남아 있어도 간다',
+    READY: '재리뷰한다',
+    REPLIES_ONLY: '재리뷰하지 않고 사람에게 넘긴다',
+    WAITING: '지금 상태를 알리고 끝낸다. 다음에 부르면 이어서 돈다',
+};
+//# sourceMappingURL=signal.js.map

package/dist/src/review-loop/signal.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"signal.js","sourceRoot":"","sources":["../../../src/review-loop/signal.ts"],"names":[],"mappings":"AAoBA;;;;;;;;;GASG;AACH,MAAM,UAAU,YAAY,CAAC,KAAgB;IAC3C,IAAI,CAAC,KAAK,CAAC,IAAI;QAAE,OAAO,QAAQ,CAAC;IACjC,IAAI,KAAK,CAAC,WAAW;QAAE,OAAO,oBAAoB,CAAC;IACnD,IAAI,KAAK,CAAC,OAAO,GAAG,CAAC;QAAE,OAAO,SAAS,CAAC;IACxC,oDAAoD;IACpD,4CAA4C;IAC5C,IAAI,CAAC,KAAK,CAAC,QAAQ;QAAE,OAAO,OAAO,CAAC;IACpC,OAAO,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,cAAc,CAAC;AAClD,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,MAAM,aAAa,GAA+B;IACvD,MAAM,EAAE,SAAS;IACjB,kBAAkB,EAAE,wBAAwB;IAC5C,KAAK,EAAE,OAAO;IACd,YAAY,EAAE,mBAAmB;IACjC,OAAO,EAAE,gCAAgC;CAC1C,CAAC"}

package/dist/src/review-loop/state.d.ts

@@ -0,0 +1,20 @@
+export declare const LOGIN_FILE = "my-login";
+export declare const REVIEWED_HEAD_FILE = "reviewed-head";
+/**
+ * 라운드 상태를 두는 자리.
+ *
+ * git 디렉토리 아래다 — git 이 절대 추적하지 않는 자리다. 경로를 절대화해서 어느
+ * 디렉토리에서 불러도 같은 곳을 가리킨다 (`state.test.ts` 가 그걸 지킨다). 워킹트리 안에 두면 이 스킬이 플러그인으로 배포돼 도는 남의 레포에서
+ * 무시되지 않아 리뷰 대상 diff 에 상태 파일이 섞인다.
+ *
+ * `--git-common-dir` 이라 워크트리 여럿이 같은 자리를 공유하지만 PR 번호가 하위
+ * 디렉토리로 갈라 안 겹친다.
+ */
+export declare function stateDir(target: {
+    owner: string;
+    repo: string;
+    prNumber: number;
+}, cwd?: string): string;
+/** 상태 자리의 뿌리. PR 을 아직 못 가린 단계가 대상 문자열을 여기 둔다 */
+export declare function stateRoot(cwd?: string): string;
+//# sourceMappingURL=state.d.ts.map

package/dist/src/review-loop/state.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"state.d.ts","sourceRoot":"","sources":["../../../src/review-loop/state.ts"],"names":[],"mappings":"AAeA,eAAO,MAAM,UAAU,aAAa,CAAC;AACrC,eAAO,MAAM,kBAAkB,kBAAkB,CAAC;AAElD;;;;;;;;;GASG;AACH,wBAAgB,QAAQ,CACtB,MAAM,EAAE;IAAE,KAAK,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,QAAQ,EAAE,MAAM,CAAA;CAAE,EACzD,GAAG,SAAgB,GAClB,MAAM,CAoBR;AAED,gDAAgD;AAChD,wBAAgB,SAAS,CAAC,GAAG,SAAgB,GAAG,MAAM,CAMrD"}

package/dist/src/review-loop/state.js

@@ -0,0 +1,55 @@
+import { execFileSync } from 'node:child_process';
+import { resolve } from 'node:path';
+/**
+ * 코드가 읽는 파일 이름.
+ *
+ * 나머지 상태 파일(`round`, `verdicts`, `target` 등)은 절차가 셸로만 다루므로 여기 안
+ * 둔다 — 쓰는 쪽이 없는 상수를 늘리면 고쳐도 아무것도 안 따라온다. 문서의 상태 자리
+ * 표가 전체 목록이고 `review-verdict-gate.test.ts` 가 그 표와 절차를 대조한다.
+ *
+ * 스레드 스냅샷은 여기 없다. 조회와 집계가 한 프로세스 안에서 끝나므로 중간 파일이
+ * 생기지 않는다.
+ */
+const ROOT_DIR = 'gestalt-review-loop';
+export const LOGIN_FILE = 'my-login';
+export const REVIEWED_HEAD_FILE = 'reviewed-head';
+/**
+ * 라운드 상태를 두는 자리.
+ *
+ * git 디렉토리 아래다 — git 이 절대 추적하지 않는 자리다. 경로를 절대화해서 어느
+ * 디렉토리에서 불러도 같은 곳을 가리킨다 (`state.test.ts` 가 그걸 지킨다). 워킹트리 안에 두면 이 스킬이 플러그인으로 배포돼 도는 남의 레포에서
+ * 무시되지 않아 리뷰 대상 diff 에 상태 파일이 섞인다.
+ *
+ * `--git-common-dir` 이라 워크트리 여럿이 같은 자리를 공유하지만 PR 번호가 하위
+ * 디렉토리로 갈라 안 겹친다.
+ */
+export function stateDir(target, cwd = process.cwd()) {
+    // 안전 정수 밖의 값은 자릿수가 뭉개져 이름이 지수 표기가 된다. 그러면 서로 다른 PR 이
+    // 같은 자리를 공유한다
+    if (!Number.isSafeInteger(target.prNumber) || target.prNumber <= 0) {
+        throw new Error(`PR 번호가 양의 정수가 아니다: ${target.prNumber}`);
+    }
+    // 레포까지 넣는다. 번호만으로 가르면 남의 레포 PR 을 볼 때 이쪽 같은 번호와 자리를
+    // 나눠 쓴다. 그 reviewed-head 로 새 커밋이 왔는지 보는 순간 재리뷰 판정이 뒤집힌다
+    for (const part of [target.owner, target.repo]) {
+        if (!/^[A-Za-z0-9][A-Za-z0-9._-]{0,99}$/.test(part)) {
+            throw new Error(`레포를 경로에 못 쓴다: ${JSON.stringify(part)}`);
+        }
+    }
+    // 레포 루트에서 부르면 `.git` 이라는 상대 경로가 온다. cwd 가 바뀐 단계에서 다른
+    // 자리를 가리키므로 여기서 절대 경로로 굳힌다 (--path-format=absolute 는 git 2.31+).
+    const common = execFileSync('git', ['rev-parse', '--git-common-dir'], {
+        cwd,
+        encoding: 'utf-8',
+    }).trim();
+    return resolve(cwd, common, ROOT_DIR, `${target.owner}--${target.repo}--${target.prNumber}`);
+}
+/** 상태 자리의 뿌리. PR 을 아직 못 가린 단계가 대상 문자열을 여기 둔다 */
+export function stateRoot(cwd = process.cwd()) {
+    const common = execFileSync('git', ['rev-parse', '--git-common-dir'], {
+        cwd,
+        encoding: 'utf-8',
+    }).trim();
+    return resolve(cwd, common, ROOT_DIR);
+}
+//# sourceMappingURL=state.js.map

package/dist/src/review-loop/state.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"state.js","sourceRoot":"","sources":["../../../src/review-loop/state.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAClD,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AAEpC;;;;;;;;;GASG;AACH,MAAM,QAAQ,GAAG,qBAAqB,CAAC;AAEvC,MAAM,CAAC,MAAM,UAAU,GAAG,UAAU,CAAC;AACrC,MAAM,CAAC,MAAM,kBAAkB,GAAG,eAAe,CAAC;AAElD;;;;;;;;;GASG;AACH,MAAM,UAAU,QAAQ,CACtB,MAAyD,EACzD,GAAG,GAAG,OAAO,CAAC,GAAG,EAAE;IAEnB,qDAAqD;IACrD,cAAc;IACd,IAAI,CAAC,MAAM,CAAC,aAAa,CAAC,MAAM,CAAC,QAAQ,CAAC,IAAI,MAAM,CAAC,QAAQ,IAAI,CAAC,EAAE,CAAC;QACnE,MAAM,IAAI,KAAK,CAAC,sBAAsB,MAAM,CAAC,QAAQ,EAAE,CAAC,CAAC;IAC3D,CAAC;IACD,mDAAmD;IACnD,wDAAwD;IACxD,KAAK,MAAM,IAAI,IAAI,CAAC,MAAM,CAAC,KAAK,EAAE,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC;QAC/C,IAAI,CAAC,mCAAmC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;YACpD,MAAM,IAAI,KAAK,CAAC,iBAAiB,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAC3D,CAAC;IACH,CAAC;IACD,qDAAqD;IACrD,iEAAiE;IACjE,MAAM,MAAM,GAAG,YAAY,CAAC,KAAK,EAAE,CAAC,WAAW,EAAE,kBAAkB,CAAC,EAAE;QACpE,GAAG;QACH,QAAQ,EAAE,OAAO;KAClB,CAAC,CAAC,IAAI,EAAE,CAAC;IACV,OAAO,OAAO,CAAC,GAAG,EAAE,MAAM,EAAE,QAAQ,EAAE,GAAG,MAAM,CAAC,KAAK,KAAK,MAAM,CAAC,IAAI,KAAK,MAAM,CAAC,QAAQ,EAAE,CAAC,CAAC;AAC/F,CAAC;AAED,gDAAgD;AAChD,MAAM,UAAU,SAAS,CAAC,GAAG,GAAG,OAAO,CAAC,GAAG,EAAE;IAC3C,MAAM,MAAM,GAAG,YAAY,CAAC,KAAK,EAAE,CAAC,WAAW,EAAE,kBAAkB,CAAC,EAAE;QACpE,GAAG;QACH,QAAQ,EAAE,OAAO;KAClB,CAAC,CAAC,IAAI,EAAE,CAAC;IACV,OAAO,OAAO,CAAC,GAAG,EAAE,MAAM,EAAE,QAAQ,CAAC,CAAC;AACxC,CAAC"}

package/dist/src/review-loop/target.d.ts

@@ -0,0 +1,23 @@
+/** 리뷰 대상에서 읽어낸 것. URL 로 주면 레포까지 함께 온다 */
+export interface PrTarget {
+    prNumber: number;
+    /** URL 로 준 경우에만 있다. 숫자나 #숫자 로 주면 현재 레포를 쓴다 */
+    owner?: string;
+    repo?: string;
+}
+/**
+ * 사용자가 준 리뷰 대상을 읽는다.
+ *
+ * 이 값은 뒤에서 상태 디렉토리 이름과 `gh` 인자가 된다. 문서가 이 파싱을 셸로 적던 때는
+ * 대상 문자열을 작은따옴표 안에 합성했다. 그래서 따옴표가 섞인 값이 정수 검증에 닿기
+ * 전에 명령으로 실행됐다. 문자열을 셸에 넘기지 않으면 그 자리가 없다.
+ *
+ * **URL 이면 레포도 함께 읽는다.** 번호만 뽑고 버리면 남의 레포 PR 을 가리켜도 조회는
+ * 현재 레포의 같은 번호를 본다. 그 수로 승인이 나간다.
+ *
+ * 받는 꼴은 셋이다 — `18`, `#18`, `https://github.com/owner/repo/pull/18`.
+ */
+export declare function parseTarget(target: string): PrTarget;
+/** 번호만 필요한 자리. 레포를 버리므로 대조가 필요한 곳에서는 parseTarget 을 쓴다 */
+export declare function parsePrNumber(target: string): number;
+//# sourceMappingURL=target.d.ts.map

package/dist/src/review-loop/target.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"target.d.ts","sourceRoot":"","sources":["../../../src/review-loop/target.ts"],"names":[],"mappings":"AAAA,yCAAyC;AACzC,MAAM,WAAW,QAAQ;IACvB,QAAQ,EAAE,MAAM,CAAC;IACjB,8CAA8C;IAC9C,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAMD;;;;;;;;;;;GAWG;AACH,wBAAgB,WAAW,CAAC,MAAM,EAAE,MAAM,GAAG,QAAQ,CAyBpD;AAED,yDAAyD;AACzD,wBAAgB,aAAa,CAAC,MAAM,EAAE,MAAM,GAAG,MAAM,CAEpD"}

package/dist/src/review-loop/target.js

@@ -0,0 +1,57 @@
+const PR_PATH = /^\/([^/]+)\/([^/]+)\/pull\/(\d+)(?:\/.*)?$/;
+const GITHUB_HOSTS = new Set(['github.com', 'www.github.com']);
+const OWNER_REPO = /^[A-Za-z0-9][A-Za-z0-9._-]{0,99}$/;
+/**
+ * 사용자가 준 리뷰 대상을 읽는다.
+ *
+ * 이 값은 뒤에서 상태 디렉토리 이름과 `gh` 인자가 된다. 문서가 이 파싱을 셸로 적던 때는
+ * 대상 문자열을 작은따옴표 안에 합성했다. 그래서 따옴표가 섞인 값이 정수 검증에 닿기
+ * 전에 명령으로 실행됐다. 문자열을 셸에 넘기지 않으면 그 자리가 없다.
+ *
+ * **URL 이면 레포도 함께 읽는다.** 번호만 뽑고 버리면 남의 레포 PR 을 가리켜도 조회는
+ * 현재 레포의 같은 번호를 본다. 그 수로 승인이 나간다.
+ *
+ * 받는 꼴은 셋이다 — `18`, `#18`, `https://github.com/owner/repo/pull/18`.
+ */
+export function parseTarget(target) {
+    const trimmed = target.trim();
+    if (/^https?:\/\//i.test(trimmed)) {
+        let url;
+        try {
+            url = new URL(trimmed);
+        }
+        catch {
+            throw new Error(`PR URL 로 못 읽었습니다: ${target}`);
+        }
+        // 호스트를 먼저 본다. 안 보면 아무 도메인의 /pull/ 경로가 PR 번호로 읽혀
+        // 조회 대상이 사용자가 지목한 것과 조용히 갈린다
+        if (!GITHUB_HOSTS.has(url.hostname.toLowerCase())) {
+            throw new Error(`github.com 주소가 아닙니다: ${target}`);
+        }
+        const m = PR_PATH.exec(url.pathname);
+        if (!m)
+            throw new Error(`PR URL 로 못 읽었습니다: ${target}`);
+        const [, owner, repo, num] = m;
+        if (!OWNER_REPO.test(owner) || !OWNER_REPO.test(repo)) {
+            throw new Error(`레포를 못 읽었습니다: ${target}`);
+        }
+        return { prNumber: toPrNumber(num, target), owner: owner, repo: repo };
+    }
+    return { prNumber: toPrNumber(trimmed.replace(/^#/, ''), target) };
+}
+/** 번호만 필요한 자리. 레포를 버리므로 대조가 필요한 곳에서는 parseTarget 을 쓴다 */
+export function parsePrNumber(target) {
+    return parseTarget(target).prNumber;
+}
+function toPrNumber(raw, target) {
+    if (!/^\d+$/.test(raw)) {
+        throw new Error(`PR 번호로 못 읽었습니다: ${target}`);
+    }
+    const n = Number(raw);
+    // 자릿수가 많으면 Number 가 지수 표기로 뭉개 정수 검사를 통과한다
+    if (!Number.isSafeInteger(n) || n <= 0) {
+        throw new Error(`PR 번호로 못 읽었습니다: ${target}`);
+    }
+    return n;
+}
+//# sourceMappingURL=target.js.map

package/dist/src/review-loop/target.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"target.js","sourceRoot":"","sources":["../../../src/review-loop/target.ts"],"names":[],"mappings":"AAQA,MAAM,OAAO,GAAG,4CAA4C,CAAC;AAC7D,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,CAAC,YAAY,EAAE,gBAAgB,CAAC,CAAC,CAAC;AAC/D,MAAM,UAAU,GAAG,mCAAmC,CAAC;AAEvD;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,WAAW,CAAC,MAAc;IACxC,MAAM,OAAO,GAAG,MAAM,CAAC,IAAI,EAAE,CAAC;IAE9B,IAAI,eAAe,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC;QAClC,IAAI,GAAQ,CAAC;QACb,IAAI,CAAC;YACH,GAAG,GAAG,IAAI,GAAG,CAAC,OAAO,CAAC,CAAC;QACzB,CAAC;QAAC,MAAM,CAAC;YACP,MAAM,IAAI,KAAK,CAAC,qBAAqB,MAAM,EAAE,CAAC,CAAC;QACjD,CAAC;QACD,gDAAgD;QAChD,6BAA6B;QAC7B,IAAI,CAAC,YAAY,CAAC,GAAG,CAAC,GAAG,CAAC,QAAQ,CAAC,WAAW,EAAE,CAAC,EAAE,CAAC;YAClD,MAAM,IAAI,KAAK,CAAC,wBAAwB,MAAM,EAAE,CAAC,CAAC;QACpD,CAAC;QACD,MAAM,CAAC,GAAG,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC,QAAQ,CAAC,CAAC;QACrC,IAAI,CAAC,CAAC;YAAE,MAAM,IAAI,KAAK,CAAC,qBAAqB,MAAM,EAAE,CAAC,CAAC;QACvD,MAAM,CAAC,EAAE,KAAK,EAAE,IAAI,EAAE,GAAG,CAAC,GAAG,CAAC,CAAC;QAC/B,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,KAAM,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,IAAK,CAAC,EAAE,CAAC;YACxD,MAAM,IAAI,KAAK,CAAC,gBAAgB,MAAM,EAAE,CAAC,CAAC;QAC5C,CAAC;QACD,OAAO,EAAE,QAAQ,EAAE,UAAU,CAAC,GAAI,EAAE,MAAM,CAAC,EAAE,KAAK,EAAE,KAAM,EAAE,IAAI,EAAE,IAAK,EAAE,CAAC;IAC5E,CAAC;IAED,OAAO,EAAE,QAAQ,EAAE,UAAU,CAAC,OAAO,CAAC,OAAO,CAAC,IAAI,EAAE,EAAE,CAAC,EAAE,MAAM,CAAC,EAAE,CAAC;AACrE,CAAC;AAED,yDAAyD;AACzD,MAAM,UAAU,aAAa,CAAC,MAAc;IAC1C,OAAO,WAAW,CAAC,MAAM,CAAC,CAAC,QAAQ,CAAC;AACtC,CAAC;AAED,SAAS,UAAU,CAAC,GAAW,EAAE,MAAc;IAC7C,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC;QACvB,MAAM,IAAI,KAAK,CAAC,mBAAmB,MAAM,EAAE,CAAC,CAAC;IAC/C,CAAC;IACD,MAAM,CAAC,GAAG,MAAM,CAAC,GAAG,CAAC,CAAC;IACtB,0CAA0C;IAC1C,IAAI,CAAC,MAAM,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;QACvC,MAAM,IAAI,KAAK,CAAC,mBAAmB,MAAM,EAAE,CAAC,CAAC;IAC/C,CAAC;IACD,OAAO,CAAC,CAAC;AACX,CAAC"}

package/dist/src/review-loop/threads.d.ts

@@ -0,0 +1,43 @@
+/** 코멘트 하나. 삭제된 계정이면 author 가 null 로 온다 */
+interface Comment {
+    author: {
+        login: string;
+    } | null;
+}
+/**
+ * GraphQL `reviewThreads` 가 돌려주는 스레드에서 판정에 쓰는 부분만.
+ *
+ * 개설자와 마지막 코멘트를 별칭으로 따로 받는다. 코멘트를 한 덩어리로 받으면 긴
+ * 스레드에서 잘려, 마지막이라고 읽은 게 실제 마지막이 아니게 된다 — 그 값이
+ * "작성자가 답했는지"를 정하므로 판정이 양방향으로 뒤집힌다.
+ */
+export interface ReviewThread {
+    isResolved: boolean;
+    isOutdated: boolean;
+    opener: {
+        nodes: Comment[];
+    };
+    latest: {
+        nodes: Comment[];
+    };
+}
+/**
+ * 내가 열어둔 스레드 중 작성자가 아직 손대지 않은 수.
+ *
+ * 이 수가 0이어야 승인이 나간다. 그래서 셀 수 없는 상태는 0이 아니라 예외로 답한다 —
+ * 셸 스크립트로 적던 때는 로그인이 빈 문자열이면 어떤 스레드와도 안 맞아 0과 종료
+ * 코드 0이 함께 나왔다. 그 값이 승인 게이트를 그대로 통과했다.
+ *
+ * 스레드가 닫혔거나, 코드가 바뀌어 outdated 가 됐거나, 마지막 코멘트가 내 것이 아니면
+ * 대응된 것으로 센다 — 마지막은 작성자가 답을 달았다는 뜻이다.
+ */
+export declare function countPending(threads: readonly ReviewThread[], me: string): number;
+/**
+ * 내가 연 스레드인지.
+ *
+ * 남이 연 스레드는 이 루프가 판정할 자리가 아니다 — 그건 그 사람이 닫는다. 사람에게
+ * 보이는 스레드 수도 이 필터를 거친 것이라야 한다.
+ */
+export declare function isMine(thread: ReviewThread, me: string): boolean;
+export {};
+//# sourceMappingURL=threads.d.ts.map

package/dist/src/review-loop/threads.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"threads.d.ts","sourceRoot":"","sources":["../../../src/review-loop/threads.ts"],"names":[],"mappings":"AAAA,0CAA0C;AAC1C,UAAU,OAAO;IACf,MAAM,EAAE;QAAE,KAAK,EAAE,MAAM,CAAA;KAAE,GAAG,IAAI,CAAC;CAClC;AAED;;;;;;GAMG;AACH,MAAM,WAAW,YAAY;IAC3B,UAAU,EAAE,OAAO,CAAC;IACpB,UAAU,EAAE,OAAO,CAAC;IACpB,MAAM,EAAE;QAAE,KAAK,EAAE,OAAO,EAAE,CAAA;KAAE,CAAC;IAC7B,MAAM,EAAE;QAAE,KAAK,EAAE,OAAO,EAAE,CAAA;KAAE,CAAC;CAC9B;AAED;;;;;;;;;GASG;AACH,wBAAgB,YAAY,CAAC,OAAO,EAAE,SAAS,YAAY,EAAE,EAAE,EAAE,EAAE,MAAM,GAAG,MAAM,CAQjF;AAED;;;;;GAKG;AACH,wBAAgB,MAAM,CAAC,MAAM,EAAE,YAAY,EAAE,EAAE,EAAE,MAAM,GAAG,OAAO,CAEhE"}

package/dist/src/review-loop/threads.js

@@ -0,0 +1,40 @@
+/**
+ * 내가 열어둔 스레드 중 작성자가 아직 손대지 않은 수.
+ *
+ * 이 수가 0이어야 승인이 나간다. 그래서 셀 수 없는 상태는 0이 아니라 예외로 답한다 —
+ * 셸 스크립트로 적던 때는 로그인이 빈 문자열이면 어떤 스레드와도 안 맞아 0과 종료
+ * 코드 0이 함께 나왔다. 그 값이 승인 게이트를 그대로 통과했다.
+ *
+ * 스레드가 닫혔거나, 코드가 바뀌어 outdated 가 됐거나, 마지막 코멘트가 내 것이 아니면
+ * 대응된 것으로 센다 — 마지막은 작성자가 답을 달았다는 뜻이다.
+ */
+export function countPending(threads, me) {
+    const login = me.trim().toLowerCase();
+    if (!login) {
+        throw new Error('내 로그인이 비었다 — 집계하지 않는다');
+    }
+    // GitHub 로그인은 대소문자를 안 가린다. 그대로 비교하면 캐시에 적힌 철자가 API 응답과
+    // 한 글자만 달라도 어떤 스레드와도 안 맞아 미대응이 0이 되고 승인이 열린다
+    return threads.filter((t) => isPending(t, login)).length;
+}
+/**
+ * 내가 연 스레드인지.
+ *
+ * 남이 연 스레드는 이 루프가 판정할 자리가 아니다 — 그건 그 사람이 닫는다. 사람에게
+ * 보이는 스레드 수도 이 필터를 거친 것이라야 한다.
+ */
+export function isMine(thread, me) {
+    return thread.opener.nodes[0]?.author?.login?.toLowerCase() === me.trim().toLowerCase();
+}
+/** me 는 소문자로 정규화된 로그인이다 */
+function isPending(thread, me) {
+    if (thread.isResolved)
+        return false;
+    if (thread.isOutdated)
+        return false;
+    if (!isMine(thread, me))
+        return false;
+    const last = thread.latest.nodes[0]?.author?.login?.toLowerCase();
+    return last === me;
+}
+//# sourceMappingURL=threads.js.map

package/dist/src/review-loop/threads.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"threads.js","sourceRoot":"","sources":["../../../src/review-loop/threads.ts"],"names":[],"mappings":"AAmBA;;;;;;;;;GASG;AACH,MAAM,UAAU,YAAY,CAAC,OAAgC,EAAE,EAAU;IACvE,MAAM,KAAK,GAAG,EAAE,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;IACtC,IAAI,CAAC,KAAK,EAAE,CAAC;QACX,MAAM,IAAI,KAAK,CAAC,uBAAuB,CAAC,CAAC;IAC3C,CAAC;IACD,uDAAuD;IACvD,6CAA6C;IAC7C,OAAO,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,SAAS,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC,CAAC,MAAM,CAAC;AAC3D,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,MAAM,CAAC,MAAoB,EAAE,EAAU;IACrD,OAAO,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,MAAM,EAAE,KAAK,EAAE,WAAW,EAAE,KAAK,EAAE,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC;AAC1F,CAAC;AAED,2BAA2B;AAC3B,SAAS,SAAS,CAAC,MAAoB,EAAE,EAAU;IACjD,IAAI,MAAM,CAAC,UAAU;QAAE,OAAO,KAAK,CAAC;IACpC,IAAI,MAAM,CAAC,UAAU;QAAE,OAAO,KAAK,CAAC;IACpC,IAAI,CAAC,MAAM,CAAC,MAAM,EAAE,EAAE,CAAC;QAAE,OAAO,KAAK,CAAC;IAEtC,MAAM,IAAI,GAAG,MAAM,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,MAAM,EAAE,KAAK,EAAE,WAAW,EAAE,CAAC;IAClE,OAAO,IAAI,KAAK,EAAE,CAAC;AACrB,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tienne/gestalt",
-  "version": "0.75.1",
+  "version": "0.76.1",
   "description": "TypeScript AI Development Harness - Gestalt psychology-driven requirement clarification",
   "repository": {
     "type": "git",

package/plugin/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "gestalt",
-  "version": "0.75.1",
+  "version": "0.76.1",
   "description": "Gestalt psychology-driven AI development harness. Transforms scattered requirements into structured, validated specifications through interactive interviews.",
   "author": {
     "name": "tienne"
@@ -22,7 +22,7 @@
   "interface": {
     "displayName": "Gestalt",
     "shortDescription": "Requirements-to-spec pipeline with review and PR workflows",
-    "longDescription": "Gestalt turns vague ideas into validated specifications through Gestalt-principle interviews, then drives execution, code review, and PR authoring. Bundles 19 workflow skills and 27 role agents over 12 MCP tools.",
+    "longDescription": "Gestalt turns vague ideas into validated specifications through Gestalt-principle interviews, then drives execution, code review, and PR authoring. Bundles 21 workflow skills and 27 role agents over 12 MCP tools.",
     "developerName": "tienne",
     "category": "Developer Tools",
     "capabilities": [

package/plugin/.mcp.json

@@ -2,7 +2,7 @@
   "mcpServers": {
     "gestalt": {
       "command": "npx",
-      "args": ["-y", "@tienne/gestalt@0.75.1", "serve"],
+      "args": ["-y", "@tienne/gestalt@0.76.1", "serve"],
       "startup_timeout_sec": 180,
       "tool_timeout_sec": 900
     }

package/plugin/mcp.json

@@ -2,7 +2,7 @@
   "mcpServers": {
     "gestalt": {
       "command": "npx",
-      "args": ["-y", "@tienne/gestalt@0.75.1", "serve"],
+      "args": ["-y", "@tienne/gestalt@0.76.1", "serve"],
       "startup_timeout_sec": 180,
       "tool_timeout_sec": 900
     }

package/plugin/skills/_shared/proactive-routing.md

@@ -41,6 +41,7 @@
 | PR·브랜치·커밋 코드 리뷰 요청 | `/review` 스킬 사용 |
 | 받은 리뷰 반영·답글 게시 요청 ("리뷰 반영해줘", "리뷰 코멘트에 답해줘", "받은 리뷰 처리해줘") | `review-reply` 스킬 사용 (스레드 수집 → 유형 분류 승인 → 수정·커밋 → 답글 승인 → 게시) |
 | 리뷰가 수렴할 때까지 돌려 GitHub까지 내보내는 요청 ("출하해줘", "리뷰 통과할 때까지", "코파일럿 리뷰까지 받아줘") | `ship` 스킬 사용 (로컬 PR → 리뷰 수렴 루프 → 승인 → draft PR → Copilot 수렴 루프 → ready). 리뷰 한 번만이면 `review` |
+| 남의 PR을 리뷰하고 대응까지 따라가는 요청 ("리뷰하고 대응 확인해줘", "대응했으면 재리뷰", "approve 날 때까지") | `review-loop` 스킬 사용 (리뷰 → 인라인 코멘트 → 판정 게시 → 대응 확인 → 재리뷰). **한 번 부르면 한 번 본다** — 이어 보려면 사람이 다시 부른다. 리뷰 한 번만이면 `review`, 내 PR을 밀어 올리는 쪽은 `ship` |
 | PR 작성·생성 요청 ("PR 만들어줘", "PR 작성해줘", "PR 올려줘") | `gestalt:pr` 스킬 사용 |
 | 실행 태스크를 외부 런타임 워커로 뿌리는 요청 ("orca로 실행", "codex로 실행", "워커 띄워서 실행") | `dispatch` 스킬 사용 (런타임 감지 → 같은 워크트리에 터미널 → worker_done 대기 → ready 재계산). 런타임 없으면 execute의 기본 병렬 경로 |
 

package/plugin/skills/local-pr/SKILL.md

@@ -48,6 +48,7 @@ outputs:
 | 이미 있는 변경을 검토받기 | `review` |
 | 받은 리뷰에 답하기 | `review-reply` |
 | 리뷰가 수렴할 때까지 돌려 GitHub까지 내보내기 | `ship` |
+| 남의 GitHub PR을 리뷰하고 대응을 지켜보다 재리뷰하기 | `review-loop` |
 
 원격 PR은 사람이 읽고 판단하라고 올린다. 에이전트끼리 주고받는 데는 `gh`도 인증도 원격 왕복도 군더더기다. 워크트리 여럿이 `.gestalt/reviews.db` 하나를 공유하므로 어느 워크트리에서 쳐도 같은 목록을 본다.
 

package/plugin/skills/review/SKILL.md

@@ -34,6 +34,10 @@ inputs:
     type: string
     required: false
     description: "인라인 코멘트를 누가 읽는지. peer | junior. 사용자가 붙인 `--audience junior`나 `--junior` 플래그가 이 값으로 들어온다 (`--junior` 축약은 이 스킬 전용이다 — 받는 값이 둘뿐이라 축약이 성립한다). 기본값 peer — 지금까지의 코멘트가 그대로 나온다"
+  postVerdict:
+    type: boolean
+    required: false
+    description: "GitHub PR에 리뷰 이벤트(APPROVE, REQUEST_CHANGES, COMMENT)까지 남길지. 기본값 true. false면 인라인 코멘트만 올리고 이벤트는 `COMMENT`로 고정한다 — PR의 리뷰 상태를 안 건드린다. 라운드를 도는 스킬이 판정을 자기가 내려고 이 스킬을 부를 때 쓴다. `prTarget`이 `local`이면 이 값을 안 본다"
 outputs:
   - reviewIntent
   - changeContext
@@ -41,6 +45,7 @@ outputs:
   - verdict
   - continuityVerdict
   - postedReview
+  - reviewSummary
 ---
 
 # Review Skill
@@ -70,7 +75,7 @@ execute 세션 없이 PR, 브랜치, 커밋의 변경사항을 직접 리뷰 파
 /review --junior               # 인라인 코멘트를 주니어 눈높이로 (= --audience junior)
 ```
 
-리뷰 한 번이 이 스킬의 범위입니다. 이슈가 없어질 때까지 리뷰와 대응을 반복하고 GitHub PR까지 내보내려면 `ship` 스킬을 씁니다 — 그쪽이 라운드마다 이 스킬을 부릅니다.
+리뷰 한 번이 이 스킬의 범위입니다. 반복은 부르는 쪽이 갈립니다. **내 PR**을 리뷰 통과 상태까지 밀어 GitHub에 내보내려면 `ship`입니다. **남의 PR**을 리뷰하고 대응을 지켜보다 재리뷰하려면 `review-loop`이고요. 둘 다 라운드마다 이 스킬을 부릅니다.
 
 ## 전제 조건
 
@@ -579,9 +584,11 @@ Agent {
 
 #### 진입 경로 두 가지
 
-이 단계는 `/review`를 처음부터 돌린 흐름뿐 아니라, **대화 도중 "이제 PR에 코멘트 남겨줘"처럼 게시만 따로 요청**받았을 때도 진입점이 됩니다 (위 triggers의 "PR에 코멘트 남겨줘" 등). 두 경우 모두 아래 **신선도 가드를 먼저 통과해야** 게시할 수 있습니다.
+이 단계는 `/review`를 처음부터 돌린 흐름뿐 아니라, **대화 도중 "이제 PR에 코멘트 남겨줘"처럼 게시만 따로 요청**받았을 때도 진입점이 됩니다 (위 triggers의 "PR에 코멘트 남겨줘" 등). 두 경우 모두 아래 **consensus 일치 검사를 먼저 통과해야** 게시할 수 있습니다.
+
+**`prTarget`이 `none`이면 여기서 끝냅니다.** 브랜치나 커밋 범위를 리뷰한 경우가 그런데, GitHub에도 로컬에도 대응하는 PR이 없는 자리입니다. 4.7단계를 통째로 건너뛰고 결과 표시로 갑니다 — 게시할 자리가 없는데 consensus를 다시 맞춰볼 이유가 없습니다.
 
-#### 신선도 가드 (stale consensus 게시 금지)
+#### consensus 일치 검사 (stale이면 게시 금지)
 
 게시 직전에, 게시하려는 consensus가 **현재 diff와 일치하는지** 반드시 확인합니다. 리뷰를 끝낸 뒤 코드가 바뀌었거나(커밋 추가·로컬 수정), 애초에 활성 리뷰 세션이 없으면 그 consensus는 stale이므로 **그대로 올리지 않습니다.**
 
@@ -598,12 +605,14 @@ pnpm tsx bin/gestalt.ts pr --json show <id>   # headSha 필드로 비교
 
 판단 기준:
 
-- **이번 세션에 방금 리뷰를 끝냈고 그 뒤 diff 변화가 없다** → consensus가 신선함. 곧장 게시 진행.
-- **리뷰 후 코드가 바뀌었다 / 활성 리뷰 세션이 없다 / 다른 세션의 오래된 결과다** → consensus가 stale. **게시하지 말고**, 1단계(git diff)부터 현재 diff로 리뷰 파이프라인(1~4단계)을 다시 돌린 뒤, 새로 나온 consensus로 4.7을 진행합니다. 사용자에게 "변경이 있어 현재 코드로 다시 리뷰한 뒤 게시할게요"라고 한 줄 알립니다.
+- **이번 세션에 방금 리뷰를 끝냈고 그 뒤 diff 변화가 없다** → 그 consensus는 현재 diff와 일치합니다. 곧장 게시 진행.
+- **리뷰 후 코드가 바뀌었다 / 활성 리뷰 세션이 없다 / 다른 세션의 오래된 결과다** → 그 consensus는 stale입니다. **게시하지 말고**, 1단계(git diff)부터 현재 diff로 리뷰 파이프라인(1~4단계)을 다시 돌린 뒤, 새로 나온 consensus로 4.7을 진행합니다. 사용자에게 "변경이 있어 현재 코드로 다시 리뷰한 뒤 게시할게요"라고 한 줄 알립니다.
 
 인라인 코멘트는 **언제 요청받든 항상 "현재 diff 기준 consensus + code-review-writer voice"** 로만 게시됩니다. 옛 리뷰 메모리를 그대로 옮겨 적거나 Claude가 손으로 코멘트를 짜는 경로는 없습니다.
 
-**PR 식별.** 대상 판별은 1단계 직후에 이미 끝났습니다. 여기서는 그때 보관한 `prTarget`과 PR 식별자를 그대로 씁니다. **같은 조회를 다시 하지 않습니다.** `prTarget`이 `none`이면(GitHub에도 로컬에도 대응하는 PR이 없는 브랜치나 커밋 범위 리뷰) 이 단계를 통째로 건너뛰고 결과 표시로 갑니다.
+#### 게시 준비 (PR 식별, audience 확인, 게시 확인)
+
+**PR 식별.** 대상 판별은 1단계 직후에 이미 끝났습니다. 여기서는 그때 보관한 `prTarget`과 PR 식별자를 그대로 씁니다. **같은 조회를 다시 하지 않습니다.**
 
 게시 직전에 그 PR이 아직 살아 있는지만 한 번 확인합니다.
 
@@ -621,7 +630,9 @@ pnpm tsx bin/gestalt.ts pr --json show <id> 2>/dev/null
 
 **게시 확인.** PR이 식별되면 사용자에게 한 번 확인합니다: **"발견된 이슈 N건을 PR #<number 또는 로컬 PR id>에 인라인 코멘트로 게시할까요?"** 동의하지 않으면 리포트만 보여주고 종료합니다.
 
-**코멘트 본문 작성 (code-review-writer).** **서브에이전트에 위임합니다.** 이 에이전트는 본문 18.8KB에 `author-voice.md` 19KB를 딸고 오는, 이 스킬에서 제일 무거운 자리입니다.
+#### 코멘트 본문 작성 (code-review-writer)
+
+**서브에이전트에 위임합니다.** 이 에이전트는 본문 18.8KB에 `author-voice.md` 19KB를 딸고 오는, 이 스킬에서 제일 무거운 자리입니다.
 
 ```
 Agent {
@@ -654,7 +665,22 @@ Agent {
 }
 ```
 
-**어투 검사 (필수).** 작성된 코멘트를 게시 전에 스캔합니다. 에이전트가 룰북을 내장하고 자가점검도 하지만 **리뷰 대상 PR 본문과 diff에 있던 말이 그대로 딸려오는 자리**는 자가점검으로 안 걸립니다 — 원문에 있으니 맞는 말이라고 판단하는 자리라서요. 그 자리를 잡는 게 이 검사입니다.
+**`path`·`line`·`side`·`severity`는 메인 세션이 채웁니다.** 서브에이전트는 `id`와 본문만 돌려주고 메인이 `id`로 `mergedIssues`를 되짚어 나머지를 붙입니다. 전부 코멘트 문체와 무관한 기계적 매핑이라 위임할 이유가 없고 서브에이전트가 라인이나 등급을 바꿔 적을 여지도 없앱니다. **원본을 이미 들고 있는 값을 되돌려 받아 쓰지 않습니다.**
+
+- `side`는 diff의 신규 라인이면 `RIGHT`, 삭제된 라인을 짚으면 `LEFT`입니다.
+- 라인 매핑이 불확실한 이슈(파일 전반이거나 구조적인 것)는 `comments`에 넣지 않고 리뷰 `body` 요약에 한 줄로 돌립니다. 임의 라인에 억지로 붙이지 않습니다.
+
+아래 규칙은 `code-review-writer` AGENT.md에 있어서 서브에이전트가 읽습니다. 여기 적어두는 건 사람이 읽을 계약이고 두 곳이 갈라지면 AGENT.md가 기준입니다. (바로 위 `path`·`line`·`side` 규칙은 반대로 **스킬 쪽에만** 있습니다 — 메인 세션이 하는 일이라 AGENT.md에 없습니다.)
+
+- code-review-writer는 `author-voice.md`(제안형·온기·물결·이모지)와 `ai-tell-quick-rules.md`(음차 교정)를 이미 내장하므로 **리포트처럼 humanize-monolith로 통째로 윤문하지는 않습니다.** 대신 위의 `humanize-scan` 검사를 거칩니다 — 내장이 자가점검을 시키는 것이고 검사는 그게 실제로 됐는지 보는 것이라 층이 다릅니다.
+- 에이전트 룰에 따라 `[출처]` 태깅, "…권장." 체언 종지는 쓰지 않습니다. 이건 Claude artifact이지 실제 리뷰어 어투가 아닙니다.
+- **출처를 밝히는 태그는 형태를 가리지 않고 쓰지 않습니다.** `[게슈탈트 리뷰]`, `[Gestalt]`, `[AI 리뷰]`, 🤖 처럼 도구가 썼다는 표시를 붙이지 않습니다. 리뷰는 계정 주인이 남기는 것입니다. **내부 리뷰 에이전트 이름(QA, Architect, security-reviewer 등)도 본문에 드러내지 않습니다** — 관점이 여럿이어도 코멘트는 리뷰어 한 사람이 남긴 것처럼 씁니다.
+- **강제성은 `r:`/`c:`/`a:` 접두어로 표기합니다** (레포에 자체 리뷰 컨벤션이 없을 때의 기본값). 코멘트 본문 맨 앞에 severity에 따라 붙입니다 — `r:` 꼭 반영(critical/high), `c:` 웬만하면 반영(warning), `a:` 사소한 의견(suggestion). 접두어는 강제성 라벨이고 본문 어투는 그대로 제안형입니다. **접두어 앞에는 아무것도 오지 않습니다** — 출처 태그나 굵은 제목 줄이 접두어를 밀어내면 리뷰이가 강제성을 한눈에 못 봅니다. (리뷰 이벤트 판정은 접두어가 아니라 `severity`로 하므로 그쪽은 영향받지 않습니다.)
+- **개행은 GitHub 렌더링 기준으로 조립합니다.** GitHub GFM은 한 줄 개행(`\n`)을 무시하고 같은 문단으로 이어 붙이므로, 줄을 실제로 나누려면 **빈 줄(`\n\n`)로 블록을 분리**해야 합니다. 접두어 → 문제 설명 → 제안 → 코드 스니펫을 각각 빈 줄로 띄우고 여러 줄 코드는 fenced code block(` ```lang ``` `)으로 감쌉니다. 한 줄 개행으로 이어 붙이면 PR에서 한 덩어리로 뭉쳐 읽기 어렵습니다 (code-review-writer의 Output Format 개행 규칙과 동일).
+
+#### 어투 검사 (필수)
+
+작성된 코멘트를 게시하기 직전에 스캔합니다. 에이전트가 룰북을 내장하고 자가점검도 하지만 **리뷰 대상 PR 본문과 diff에 있던 말이 그대로 딸려오는 자리**는 자가점검으로 안 걸립니다 — 원문에 있으니 맞는 말이라고 판단하는 자리라서요. 그 자리를 잡는 게 이 검사입니다.
 
 **코멘트마다 파일 하나로 떨굽니다. 한 파일에 모으지 않습니다.** 모아서 한 번에 스캔하면 검사가 배치 전체를 한 덩어리로 봅니다. 그러면 코멘트 하나를 통째로 `>` 인용으로 감싸도 다른 코멘트의 산문에 묻혀 안 걸립니다. 코멘트별로 갈라야 그 판정이 코멘트 단위로 섭니다. 어느 코멘트가 걸렸는지도 파일 이름으로 바로 읽히고요.
 
@@ -707,20 +733,9 @@ done
 
 **큰따옴표로 감싼 평문은 형태로 못 가릅니다.** 한국어에서 큰따옴표는 인용만이 아니라 강조로도 쓰여서요. 룰북 Do-NOT 목록이 "큰따옴표 안 직접 인용"을 예외로 두지만 그건 사람과 모델이 판단하는 자리의 기준입니다. 자동 스캔은 위 셋만 뺍니다. 원문을 가리켜야 하면 백틱이나 블록인용을 씁니다.
 
-**`path`·`line`·`side`·`severity`는 메인 세션이 채웁니다.** 서브에이전트는 `id`와 본문만 돌려주고 메인이 `id`로 `mergedIssues`를 되짚어 나머지를 붙입니다. 전부 코멘트 문체와 무관한 기계적 매핑이라 위임할 이유가 없고 서브에이전트가 라인이나 등급을 바꿔 적을 여지도 없앱니다. **원본을 이미 들고 있는 값을 되돌려 받아 쓰지 않습니다.**
+#### 리뷰 이벤트 결정 (postVerdict와 본인 PR 예외)
 
-- `side`는 diff의 신규 라인이면 `RIGHT`, 삭제된 라인을 짚으면 `LEFT`입니다.
-- 라인 매핑이 불확실한 이슈(파일 전반이거나 구조적인 것)는 `comments`에 넣지 않고 리뷰 `body` 요약에 한 줄로 돌립니다. 임의 라인에 억지로 붙이지 않습니다.
-
-아래 규칙은 `code-review-writer` AGENT.md에 있어서 서브에이전트가 읽습니다. 여기 적어두는 건 사람이 읽을 계약이고 두 곳이 갈라지면 AGENT.md가 기준입니다. (바로 위 `path`·`line`·`side` 규칙은 반대로 **스킬 쪽에만** 있습니다 — 메인 세션이 하는 일이라 AGENT.md에 없습니다.)
-
-- code-review-writer는 `author-voice.md`(제안형·온기·물결·이모지)와 `ai-tell-quick-rules.md`(음차 교정)를 이미 내장하므로 **리포트처럼 humanize-monolith로 통째로 윤문하지는 않습니다.** 대신 위의 `humanize-scan` 검사를 거칩니다 — 내장이 자가점검을 시키는 것이고 검사는 그게 실제로 됐는지 보는 것이라 층이 다릅니다.
-- 에이전트 룰에 따라 `[출처]` 태깅, "…권장." 체언 종지는 쓰지 않습니다. 이건 Claude artifact이지 실제 리뷰어 어투가 아닙니다.
-- **출처를 밝히는 태그는 형태를 가리지 않고 쓰지 않습니다.** `[게슈탈트 리뷰]`, `[Gestalt]`, `[AI 리뷰]`, 🤖 처럼 도구가 썼다는 표시를 붙이지 않습니다. 리뷰는 계정 주인이 남기는 것입니다. **내부 리뷰 에이전트 이름(QA, Architect, security-reviewer 등)도 본문에 드러내지 않습니다** — 관점이 여럿이어도 코멘트는 리뷰어 한 사람이 남긴 것처럼 씁니다.
-- **강제성은 `r:`/`c:`/`a:` 접두어로 표기합니다** (레포에 자체 리뷰 컨벤션이 없을 때의 기본값). 코멘트 본문 맨 앞에 severity에 따라 붙입니다 — `r:` 꼭 반영(critical/high), `c:` 웬만하면 반영(warning), `a:` 사소한 의견(suggestion). 접두어는 강제성 라벨이고 본문 어투는 그대로 제안형입니다. **접두어 앞에는 아무것도 오지 않습니다** — 출처 태그나 굵은 제목 줄이 접두어를 밀어내면 리뷰이가 강제성을 한눈에 못 봅니다. (리뷰 이벤트 판정은 접두어가 아니라 `severity`로 하므로 그쪽은 영향받지 않습니다.)
-- **개행은 GitHub 렌더링 기준으로 조립합니다.** GitHub GFM은 한 줄 개행(`\n`)을 무시하고 같은 문단으로 이어 붙이므로, 줄을 실제로 나누려면 **빈 줄(`\n\n`)로 블록을 분리**해야 합니다. 접두어 → 문제 설명 → 제안 → 코드 스니펫을 각각 빈 줄로 띄우고 여러 줄 코드는 fenced code block(` ```lang ``` `)으로 감쌉니다. 한 줄 개행으로 이어 붙이면 PR에서 한 덩어리로 뭉쳐 읽기 어렵습니다 (code-review-writer의 Output Format 개행 규칙과 동일).
-
-**리뷰 이벤트 결정.** `mergedIssues`의 `severity`로 리뷰 전체의 `event`를 정합니다. 본문 첫 글자를 파싱하지 않습니다 — 접두어는 사람이 읽는 라벨이지 판정 입력이 아닙니다.
+`mergedIssues`의 `severity`로 리뷰 전체의 `event`를 정합니다. 본문 첫 글자를 파싱하지 않습니다 — 접두어는 사람이 읽는 라벨이지 판정 입력이 아닙니다.
 
 - `critical`이나 `high`가 하나라도 있으면 → `REQUEST_CHANGES` (본문 접두어 `r:`)
 - 없고 `warning`만 있으면 → `COMMENT` (접두어 `c:`)
@@ -728,6 +743,14 @@ done
 
 4단계 `overallApproved`(결함 심급 blocking 여부)와도 일치합니다 — blocking 이슈가 있으면 critical이나 high가 존재하므로 `REQUEST_CHANGES`가 됩니다. 단 `APPROVE`/`REQUEST_CHANGES`는 리뷰 상태를 바꾸는 행위이므로, 위 **"게시 확인"**에서 사용자 동의를 받은 뒤에만 게시합니다.
 
+**`postVerdict`가 `false`면 위 계산을 하지 않고 `event=COMMENT`로 고정합니다.** 인라인 코멘트는 그대로 올라가고 PR의 리뷰 상태만 안 건드립니다. 부르는 쪽이 판정을 자기가 내겠다는 뜻이라, 여기서 `APPROVE`나 `REQUEST_CHANGES`를 먼저 내보내면 그쪽 판정이 도착하기 전에 리뷰 상태가 정해집니다. `COMMENT`는 기존 상태를 안 바꾸므로 뒤이어 오는 판정이 그대로 섭니다.
+
+이 값은 `prTarget`이 `github`일 때만 걸립니다. 로컬 PR은 `review_publish`가 파이프라인과 같은 경계로 판정을 정하므로 부르는 쪽이 그걸 억제할 이유가 없습니다.
+
+**부르는 쪽이 게시 결과를 확인할 수 있게 합니다.** `postVerdict: false`로 불렀는데 이 스킬이 그 값을 못 읽고 `APPROVE`나 `REQUEST_CHANGES`를 내보냈다면, 그쪽 판정이 도착하기 전에 리뷰 상태가 이미 정해집니다. 게시 직후 `gh pr view <번호> --json reviewDecision`으로 상태를 확인해 `postedReview`에 담아 돌려줍니다 — 부르는 쪽이 그 값으로 어긋남을 알아챕니다.
+
+**`reviewSummary`를 출력으로 돌려줍니다.** 위 서브에이전트가 돌려준 `{ comments, summary }`의 `summary`를 그대로 담습니다 — 따로 짓지 않습니다. `postVerdict: false`로 부른 쪽이 자기 판정 본문을 지을 때 이 값을 뼈대로 씁니다 — 같은 에이전트를 판정 본문만으로 한 번 더 부르면 룰북을 라운드마다 두 번 싣게 됩니다.
+
 > **본인 PR 예외 (github)**: GitHub는 PR 작성자 본인이 자기 PR을 `APPROVE`/`REQUEST_CHANGES`하는 걸 막습니다(422). `gh pr view --json author`와 `gh api user`로 작성자가 현재 사용자와 같은지 확인하고 같으면 `event=COMMENT`로 폴백해 게시합니다 (접두어 r/c/a는 본문에 그대로 유지). 이때 사용자에게 "본인 PR이라 승인/변경요청 상태는 못 걸어서 코멘트로 남겼어요"라고 한 줄 알립니다. **local**은 `gestalt pr review`가 이 제약을 두지 않습니다 — author가 본인과 같아도 verdict 그대로 게시하되, 사용자에게 그 사실만 한 줄 알립니다.
 
 **게시.** `prTarget`에 따라 갈립니다.