hanoman 0.9.34 → 0.9.36

package/dist/server.js

@@ -4159,7 +4159,7 @@ var init_enums = __esm({
     zSeverity = external_exports.enum(["critical", "major", "minor"]);
     zTicketCategory = external_exports.enum(["bug", "fitur", "pertanyaan", "lainnya"]);
     zTicketStatus = external_exports.enum(["new", "accepted", "rejected"]);
-    VERIFY_SCOPES = ["changed", "full"];
+    VERIFY_SCOPES = ["changed", "full", "none"];
     zVerifyScope = external_exports.enum(VERIFY_SCOPES);
   }
 });
@@ -4299,6 +4299,8 @@ var init_method_catalog = __esm({
             "superpowers:test-driven-development",
             VERIFICATION_GATE
           ],
+          // ADR-0230 · fase Review resmi: audit spec & plan vs diff + gerbang verifikasi.
+          Review: ["superpowers:requesting-code-review", VERIFICATION_GATE],
           // SPEC-407 · fase `Goal` sengaja TANPA skill: seluruh inti flow itu membebaskan sesi dari
           // proses kaku. Yang tetap dijaga cuma pintu keluarnya.
           Verifikasi: [VERIFICATION_GATE]
@@ -4342,6 +4344,8 @@ var init_method_catalog = __esm({
             "mattpocock-skills:tdd",
             "mattpocock-skills:code-review"
           ],
+          // ADR-0230 · fase Review resmi: audit spec & plan vs diff + gerbang verifikasi.
+          Review: ["mattpocock-skills:code-review", VERIFICATION_GATE],
           Verifikasi: [VERIFICATION_GATE]
         },
         exitSkills: [VERIFICATION_GATE],
@@ -4382,6 +4386,8 @@ var init_method_catalog = __esm({
             "agent-skills:test-driven-development",
             "agent-skills:code-review-and-quality"
           ],
+          // ADR-0230 · fase Review resmi: audit spec & plan vs diff + gerbang verifikasi.
+          Review: ["agent-skills:code-review-and-quality", VERIFICATION_GATE],
           Verifikasi: [VERIFICATION_GATE]
         },
         exitSkills: [VERIFICATION_GATE],
@@ -5168,7 +5174,7 @@ var init_logs = __esm({
 function phaseAgentName(phase) {
   return PHASE_AGENT_PREFIX + phase.toLowerCase().replace(/[^a-z0-9]+/g, "-").replace(/^-+|-+$/g, "");
 }
-var ORCHESTRATION_FLOWS, FLOW_PHASES, PHASE_AGENT_PREFIX, isPhaseAgentName, PHASE_EVIDENCE_GRACE_MS, EXECUTE_MODE_FLOWS, REVIEWER_CELL, REVIEWER_FLOWS, REVIEWER_DEFAULTS;
+var ORCHESTRATION_FLOWS, FLOW_PHASES, PHASE_AGENT_PREFIX, isPhaseAgentName, PHASE_EVIDENCE_GRACE_MS, EXECUTE_MODE_FLOWS, REVIEWER_CELL, REVIEWER_FLOWS, REVIEWER_DEFAULTS, UNSKIPPABLE_PHASES;
 var init_orchestration = __esm({
   "../shared/src/orchestration.ts"() {
     "use strict";
@@ -5184,8 +5190,9 @@ var init_orchestration = __esm({
       "no_effort"
     ];
     FLOW_PHASES = {
-      feature: ["Brainstorm", "Objective", "Spec", "Plan", "Execute"],
-      qa: ["Audit", "Spec", "Plan", "Execute"],
+      // ADR-0230 · `Review` = fase resmi sesudah Execute (dulu langkah R di dalam Execute, ADR-0170 #4).
+      feature: ["Brainstorm", "Objective", "Spec", "Plan", "Execute", "Review"],
+      qa: ["Audit", "Spec", "Plan", "Execute", "Review"],
       scaffold: ["Brainstorm", "Objective", "Doc index"],
       reverse: ["Scan", "Docs teknis", "Wawancara", "Konvensi & index", "Serah terima"],
       prd: ["Brainstorm", "PRD"],
@@ -5208,12 +5215,15 @@ var init_orchestration = __esm({
       ORCHESTRATION_FLOWS.filter((f) => FLOW_PHASES[f].some((p3) => p3 === "Execute" || p3 === "Goal"))
     );
     REVIEWER_CELL = "Review";
-    REVIEWER_FLOWS = /* @__PURE__ */ new Set(["feature", "qa"]);
+    REVIEWER_FLOWS = new Set(
+      ORCHESTRATION_FLOWS.filter((f) => FLOW_PHASES[f].includes(REVIEWER_CELL))
+    );
     REVIEWER_DEFAULTS = {
       claude: { model: "opus", effort: "high" },
       codex: { model: "gpt-5.6-sol", effort: "high" },
       antigravity: { model: "inherit", effort: null }
     };
+    UNSKIPPABLE_PHASES = /* @__PURE__ */ new Set([REVIEWER_CELL]);
   }
 });
 
@@ -5224,7 +5234,7 @@ var init_runtime_defaults = __esm({
     "use strict";
     init_orchestration();
     init_zod();
-    RUNTIME_DEFAULTS_VERSION = "2026-09-23-v2";
+    RUNTIME_DEFAULTS_VERSION = "2026-10-25-v1";
     zRuntimeDefaultState = external_exports.enum(["seeded", "user"]);
     zBuiltinRuntimeDefaults = external_exports.object({
       version: external_exports.string().default(RUNTIME_DEFAULTS_VERSION),
@@ -5281,14 +5291,16 @@ var init_runtime_defaults = __esm({
           Objective: cell("sonnet", "medium"),
           Spec: cell("opus", "high"),
           Plan: cell("opus", "high"),
-          Execute: cell("sonnet", "high")
+          Execute: cell("sonnet", "high"),
+          Review: cell("opus", "high")
         },
         {
           Brainstorm: cell("gpt-5.6-terra", "medium"),
           Objective: cell("gpt-5.6-terra", "medium"),
           Spec: cell("gpt-5.6-sol", "high"),
           Plan: cell("gpt-5.6-sol", "high"),
-          Execute: cell("gpt-5.6-terra", "high")
+          Execute: cell("gpt-5.6-terra", "high"),
+          Review: cell("gpt-5.6-sol", "high")
         }
       ),
       qa: flow(
@@ -5296,13 +5308,15 @@ var init_runtime_defaults = __esm({
           Audit: cell("opus", "high"),
           Spec: cell("opus", "high"),
           Plan: cell("sonnet", "high"),
-          Execute: cell("sonnet", "high")
+          Execute: cell("sonnet", "high"),
+          Review: cell("opus", "high")
         },
         {
           Audit: cell("gpt-5.6-sol", "high"),
           Spec: cell("gpt-5.6-sol", "high"),
           Plan: cell("gpt-5.6-terra", "high"),
-          Execute: cell("gpt-5.6-terra", "high")
+          Execute: cell("gpt-5.6-terra", "high"),
+          Review: cell("gpt-5.6-sol", "high")
         }
       ),
       scaffold: flow(
@@ -5856,20 +5870,16 @@ function resolvePhasePlan(input) {
     const model = input.runtime === "claude" && picked === CLAUDE_DEFAULT_ALIAS ? CLAUDE_SUBAGENT_INHERIT : picked;
     return { model, effort: coerce3(model, override?.effort ?? cell4?.effort ?? fallback3.effort ?? "medium") };
   };
-  const reviewer = REVIEWER_FLOWS.has(input.flow) ? {
-    agentName: phaseAgentName(REVIEWER_CELL),
-    phase: "Execute",
-    ...resolve25(REVIEWER_CELL, REVIEWER_DEFAULTS[input.runtime])
-  } : void 0;
   return {
     flow: input.flow,
     runtime: input.runtime,
+    // ADR-0170 P2 · ADR-0230 · fase Review TIDAK mewarisi orchestrator: default-nya konstanta peran,
+    // sel/override `Review` menimpanya.
     phases: FLOW_PHASES[input.flow].map((phase) => ({
       phase,
       agentName: phaseAgentName(phase),
-      ...resolve25(phase, input.orchestrator)
+      ...resolve25(phase, phase === REVIEWER_CELL ? REVIEWER_DEFAULTS[input.runtime] : input.orchestrator)
     })),
-    ...reviewer ? { reviewer } : {},
     // Amandemen ADR-0170 P2 · cara Execute mengerjakan plan; hanya dipakai sesi claude.
     executeMode: cfg?.executeMode ?? "inline"
   };
@@ -13215,7 +13225,7 @@ var init_sessions = __esm({
             effort: str("Effort. Kosong = default global."),
             goal: bool("Mode goal per-sesi. Kosong = ikut setelan global."),
             goalCondition: str("Kondisi selesai mode goal (maks 4000 karakter)."),
-            verifyScope: enumStr(["changed", "full"], "Scope verifikasi. Kosong = ikut setelan global."),
+            verifyScope: enumStr(["changed", "full", "none"], "Scope verifikasi: changed = hanya yang berubah, full = seluruh project, none = jangan jalankan test. Kosong = ikut setelan global."),
             method: str("Metode workflow. Id tak dikenal jatuh ke default, bukan ditolak."),
             phaseOverrides: {
               type: "object",
@@ -16766,15 +16776,25 @@ function parseHookEvent(body) {
   }
   const turn = zTurnEnd.safeParse(body);
   if (turn.success && !turn.data.stop_hook_active) {
+    if (turn.data.background_tasks?.some((t) => t.status === "running")) return null;
+    const message = clip3(turn.data.last_assistant_message, ASK_MESSAGE_MAX);
     return {
       source: "turn-end",
-      askId: turn.data.turn_id,
+      askId: turn.data.turn_id ?? `stop:${turn.data.prompt_id ?? ""}:${fnv1a(message)}`,
       questions: [],
-      message: clip3(turn.data.last_assistant_message, ASK_MESSAGE_MAX)
+      message
     };
   }
   return null;
 }
+function fnv1a(s2) {
+  let h2 = 2166136261;
+  for (let i = 0; i < s2.length; i++) {
+    h2 ^= s2.charCodeAt(i);
+    h2 = Math.imul(h2, 16777619);
+  }
+  return (h2 >>> 0).toString(16);
+}
 var ASK_QUESTION_MAX, ASK_MESSAGE_MAX, clip3, ASK_MAX_QUESTIONS, ASK_MAX_OPTIONS, zOption, zQuestion, zAskTool, zTurnEnd;
 var init_session_ask = __esm({
   "../shared/src/session-ask.ts"() {
@@ -16803,7 +16823,11 @@ var init_session_ask = __esm({
     });
     zTurnEnd = external_exports.object({
       hook_event_name: external_exports.literal("Stop"),
-      turn_id: external_exports.string().min(1),
+      // ADR-0231 · codex mengirim `turn_id`; Stop claude (sesi bertanya lewat teks) tidak — ia membawa
+      // `prompt_id` dan `background_tasks`.
+      turn_id: external_exports.string().min(1).optional(),
+      prompt_id: external_exports.string().optional(),
+      background_tasks: external_exports.array(external_exports.object({ status: external_exports.string().optional() }).passthrough()).optional(),
       last_assistant_message: external_exports.string().optional(),
       // Codex menyalakan bendera ini saat Stop dipicu oleh hook Stop lain. Melayani giliran yang
       // dibangkitkan gate mode goal (ADR-0074) berarti lead menjawab dirinya sendiri.
@@ -17243,7 +17267,7 @@ dengan folder implementasi nyata repo ini (hasil fase Scan):
 });
 
 // ../runner/src/verify-scope.ts
-var CHANGED, verifyScopeClause;
+var CHANGED, NONE, verifyScopeClause;
 var init_verify_scope = __esm({
   "../runner/src/verify-scope.ts"() {
     "use strict";
@@ -17284,7 +17308,26 @@ var init_verify_scope = __esm({
       "tipe/kontrak bersama, skema, atau berkas yang diimpor banyak modul), perluas scope seperlunya",
       "dan katakan alasannya. Ini panduan biaya, bukan larangan."
     ].join("\n");
-    verifyScopeClause = (scope) => scope === "changed" ? CHANGED : "";
+    NONE = [
+      "Scope verifikasi: TANPA menjalankan test. Operator memilih mode ini karena perangkatnya tak",
+      "kuat menjalankan test \u2014 run sebelumnya membuat mesin tersendat.",
+      "",
+      "- JANGAN menjalankan test apa pun: `pnpm test`, `vitest` (termasuk `--changed`/`related`),",
+      "  `jest`, `pytest`, `go test`, Playwright/e2e, maupun skrip yang memanggilnya. Aturan ini",
+      '  MENIMPA instruksi skill/metode (mis. TDD "jalankan test, lihat merah lalu hijau"), instruksi',
+      "  fase Verifikasi/review, dan Definition of Done repo target.",
+      "- Menulis atau memperbarui berkas test tetap boleh \u2014 hanya jangan menjalankannya.",
+      "- JANGAN build, lint seluruh repo, boot server + curl, atau smoke end-to-end.",
+      "- Typecheck: paling banyak SEKALI di akhir, hanya untuk paket yang tersentuh",
+      "  (mis. `pnpm --filter ./server typecheck`). JANGAN `pnpm -r typecheck`.",
+      '- Verifikasi dengan MEMBACA: telusuri diff-mu (`git diff "$HANOMAN_BASE_SHA"...HEAD`,',
+      "  `git status --porcelain`), periksa pemanggil, tipe, dan cabang error dengan grep/baca berkas.",
+      "- Laporan: nyatakan terang bahwa test TIDAK dijalankan (scope verifikasi `none`) dan sebut berkas",
+      "  test yang perlu dijalankan manusia. Jangan pernah mengklaim test hijau.",
+      "",
+      "Menjalankan test adalah tugas MANUSIA sebelum merge, bukan tugas sesi."
+    ].join("\n");
+    verifyScopeClause = (scope) => scope === "changed" ? CHANGED : scope === "none" ? NONE : "";
   }
 });
 
@@ -17340,29 +17383,25 @@ function orchestratorClause(plan, o = {}) {
   const resume = antigravity2 ? "send_message ke conversation ID child yang sama" : codex2 ? "send_input ke agent id yang sama" : "SendMessage ke agent ID yang kamu catat";
   const describeRule = antigravity2 ? "Cantumkan label `Fase <Nama Fase>` dalam tugas. Pilih nama definisi native fase, JANGAN membuat agen umum pengganti. Catat conversation ID hasil invoke_subagent; ID itu dipakai untuk SEMUA relay keputusan dan rework lewat send_message, bukan invoke_subagent baru. Gunakan tool native langsung, bukan shell atau CLI hanoman untuk menyimulasikan delegasi. Bila tool tidak tersedia, laporkan galat langkah 3; jangan membuat bridge atau mengeksekusi isi fase sendiri." : codex2 ? "Deskripsi/label pemanggilan diisi persis `Fase <Nama Fase>` \u2014 tanpa nomor urut, dan BUKAN baris pertama blok serah-terima di atas." : "Deskripsi pemanggilan (parameter `description`) diisi persis `Fase <Nama Fase>` \u2014 tanpa nomor urut, dan BUKAN baris pertama blok serah-terima di atas.\nJANGAN pernah memanggil tool Agent tanpa `subagent_type` atau dengan `general-purpose`: fase itu jatuh ke agen umum ber-model orchestrator dan model/effort fasenya hilang. Sebelum memanggil, periksa ulang bahwa `subagent_type` = nama agen fase itu; bila tool menolaknya, itu galat langkah 3 \u2014 bukan alasan beralih ke agen umum. Catat agent ID yang dikembalikan hasil pemanggilan: itulah satu-satunya pegangan untuk melanjutkan agen yang SAMA. Melanjutkan agen itu (relay jawaban langkah 4) HANYA lewat SendMessage \u2014 tool Agent tak punya parameter `to`, dan memanggilnya lagi melahirkan agen BARU tanpa konteks (duplikat).";
   const retry = antigravity2 ? " (invoke_subagent BARU dengan nama definisi fase yang sama)" : codex2 ? "" : " (pemanggilan tool Agent BARU dengan `subagent_type` yang sama)";
-  const askEscalation = codex2 || antigravity2 ? "tanyakan di terminal ini lalu tunggu jawaban" : "tanyakan lewat AskUserQuestion";
-  const askDecisions = antigravity2 ? "di terminal ini sekaligus dalam satu pesan, tiap pertanyaan bernomor dan diakhiri `?` beserta opsi, rekomendasi dan alasannya, lalu tunggu jawaban" : codex2 ? "di terminal ini sekaligus dalam satu pesan, tiap pertanyaan bernomor dan diakhiri `?` beserta opsinya, lalu tunggu jawaban" : "lewat AskUserQuestion \u2014 tool itu memuat paling banyak 4 pertanyaan per panggilan, jadi pecah sisanya ke panggilan berikutnya sampai semuanya terjawab";
+  const askEscalation = codex2 || antigravity2 ? "tanyakan di terminal ini lalu tunggu jawaban" : ASK_ESCALATION_TOOL;
+  const askDecisions = antigravity2 ? "di terminal ini sekaligus dalam satu pesan, tiap pertanyaan bernomor dan diakhiri `?` beserta opsi, rekomendasi dan alasannya, lalu tunggu jawaban" : codex2 ? "di terminal ini sekaligus dalam satu pesan, tiap pertanyaan bernomor dan diakhiri `?` beserta opsinya, lalu tunggu jawaban" : ASK_DECISIONS_TOOL;
   const turnTaking = plan.flow === "reverse" || plan.flow === "scaffold" || plan.flow === "prd" ? " Fase yang bergiliran dengan manusia (Wawancara, Brainstorm prd/scaffold) melapor satu topik per putaran \u2014 relay tiap putaran dengan cara yang sama, bawa konteks & opsi dari laporannya, dan biarkan jawaban bebas." : "";
   const handoff = plan.phases.some((p3) => p3.phase === "Serah terima") ? "Sesudah fase Serah terima tercatat, tampilkan bagian `Ringkasan serah terima:` dari laporannya APA ADANYA ke manusia di terminal ini (teks biasa \u2014 informasi, bukan pertanyaan) sebelum menutup pekerjaan." : "";
   const runtimeLabel = (entry2) => antigravity2 ? `tier native ${entry2.model} \xB7 effort native unknown` : `${entry2.model} \xB7 ${entry2.effort}`;
   const phaseList = plan.phases.map((p3, i) => `${i + 1}. ${p3.phase} \u2192 \`${p3.agentName}\` \xB7 ${runtimeLabel(p3)}`).join("\n");
-  const reviewer = plan.reviewer && plan.phases.some((p3) => p3.phase === plan.reviewer.phase) ? plan.reviewer : void 0;
-  const list2 = reviewer ? `${phaseList}
-Reviewer Execute \u2192 \`${reviewer.agentName}\` \xB7 ${runtimeLabel(reviewer)} (bukan fase \u2014 hanya dipanggil di langkah R)` : phaseList;
+  const review = plan.phases.find((p3) => p3.phase === REVIEWER_CELL);
   const hasExecute = plan.phases.some((p3) => p3.phase === "Execute");
-  const artefactRule = "Baris `Artefak fase sebelumnya:` diisi path PERSIS yang dilaporkan fase sebelumnya di `Artefak:` laporannya" + (hasExecute ? " \u2014 untuk Execute: dokumen spec DAN berkas plan yang dilaporkan fase Plan" : "") + ", bukan pola glob.";
-  const reviewStep = reviewer ? "R. Review Execute: sesudah agen Execute melapor `Status: selesai` dengan bukti dan tanpa keputusan terbuka, SEBELUM menulis `Execute done`, panggil " + (antigravity2 ? `\`${reviewer.agentName}\` lewat invoke_subagent` : codex2 ? `\`${reviewer.agentName}\` lewat spawn_agent` : `tool Agent dengan \`subagent_type\` \`${reviewer.agentName}\``) + ` (deskripsi \`Review Execute\`) dengan blok serah-terima:
-Tujuan: <objective backlog>
-Base SHA: $HANOMAN_BASE_SHA
+  const artefactRule = "Baris `Artefak fase sebelumnya:` diisi path PERSIS yang dilaporkan fase sebelumnya di `Artefak:` laporannya" + (hasExecute ? " \u2014 untuk Execute: dokumen spec DAN berkas plan yang dilaporkan fase Plan" : "") + (review ? "; untuk Review: dokumen spec & berkas plan yang SAMA dengan Execute" : "") + ", bukan pola glob.";
+  const reviewStep = review ? `R. Fase Review: sesudah \`Execute done\` tercatat, panggil agen fasenya seperti langkah 1 dengan baris serah-terima tambahan:
 Artefak: <path PERSIS dokumen spec & berkas plan> (qa yang melewati Spec & Plan: dokumen audit)
 Laporan Execute: <tabel AC \u2192 bukti dari laporannya>
 Putaran rework: <r>/2
 
-\`Verdict: lulus\` \u2192 langkah 2 (tulis \`Execute done\`). \`Verdict: rework\` \u2192 lanjutkan agen Execute yang SAMA lewat ${resume} dengan \`Temuan wajib diperbaiki:\` reviewer apa adanya; sesudah ia melapor \`selesai\` lagi, lanjutkan reviewer yang SAMA lewat ${resume} dengan laporan barunya. Batasnya maks 2 putaran rework: masih \`rework\` sesudah putaran ke-2 \u2192 perlakukan temuan tersisa sebagai \`Keputusan terbuka\` langkah 4 (tanyakan, teruskan jawabannya ke agen Execute, lalu review lagi). Putaran rework bukan percobaan ulang langkah 3; reviewer galat atau tanpa baris \`Verdict:\` \u2192 langkah 3 berlaku padanya, \`menunggu-keputusan\` \u2192 langkah 4. JANGAN memperbaiki temuan sendiri dan JANGAN menulis \`Execute done\` tanpa \`Verdict: lulus\`.` : "";
+\`Verdict: lulus\` \u2192 langkah 2 (tulis \`Review done\`). \`Verdict: rework\` \u2192 lanjutkan agen Execute yang SAMA lewat ${resume} dengan \`Temuan wajib diperbaiki:\` reviewer apa adanya; sesudah ia melapor \`selesai\` lagi, lanjutkan reviewer yang SAMA lewat ${resume} dengan laporan barunya. Batasnya maks 2 putaran rework: masih \`rework\` sesudah putaran ke-2 \u2192 perlakukan temuan tersisa sebagai \`Keputusan terbuka\` langkah 4 (tanyakan, teruskan jawabannya ke agen Execute, lalu review lagi). Putaran rework bukan percobaan ulang langkah 3 dan tak menulis ulang \`Execute done\`; reviewer galat atau tanpa baris \`Verdict:\` \u2192 langkah 3 berlaku padanya, \`menunggu-keputusan\` \u2192 langkah 4. JANGAN memperbaiki temuan sendiri, JANGAN menulis \`Review done\` tanpa \`Verdict: lulus\`, dan Review tak pernah \`skipped\`.` : "";
   return [
     antigravity2 ? "Sesi ini ORCHESTRATOR. Setiap fase dikerjakan child native persisten miliknya dengan tier inherit|flash|pro, bukan slug model parent atau effort independen. Runtime asynchronous: protocol wait mewajibkan menunggu laporan child sebelum fase berikutnya, relay atau marker; ini bukan jaminan foreground runtime, model exact atau effort terkunci. Jangan gunakan CLI one-shot/bridge untuk sesi kerja child. Kamu TIDAK mengerjakan isi fase sendiri." : "Sesi ini ORCHESTRATOR. Setiap fase dikerjakan subagent fase miliknya dengan model & effort yang sudah terkunci di definisinya \u2014 kamu TIDAK mengerjakan isi fase sendiri.",
     `Fase berurutan:
-${list2}`,
+${phaseList}`,
     "Untuk SETIAP fase, berurutan:",
     `1. Panggil subagent fasenya lewat ${call}, tugasnya berupa blok serah-terima berbentuk tetap:
 Urutan: <n>/<total> \xB7 <Nama Fase>
@@ -17374,14 +17413,14 @@ Lampiran: <path INDEX.md lampiran, atau ->
 Percobaan: <k>/2
 
 ${artefactRule} ${describeRule}`,
-    '2. Baca laporannya. `Status: selesai` DENGAN bukti \u2192 SEBELUM hal lain (memanggil fase berikutnya, commit, atau push): jalankan persis `echo "<Nama Fase> done" >> "$HANOMAN_PHASE_FILE"`, lalu verifikasi dengan `tail -1 "$HANOMAN_PHASE_FILE"` bahwa barisnya benar tertulis. Kamu satu-satunya penulis berkas itu.' + (reviewer ? " Khusus Execute: langkah R dulu; marker ditulis sesudah `Verdict: lulus`." : ""),
+    '2. Baca laporannya. `Status: selesai` DENGAN bukti \u2192 SEBELUM hal lain (memanggil fase berikutnya, commit, atau push): jalankan persis `echo "<Nama Fase> done" >> "$HANOMAN_PHASE_FILE"`, lalu verifikasi dengan `tail -1 "$HANOMAN_PHASE_FILE"` bahwa barisnya benar tertulis. Kamu satu-satunya penulis berkas itu.' + (review ? " Khusus Review: marker hanya sesudah `Verdict: lulus` (langkah R)." : ""),
     `3. \`Status: sebagian\`/\`terhalang\`, galat, atau laporan tanpa bukti \u2192 delegasikan ULANG SEKALI ke subagent fase yang sama${retry} dengan laporan gagalnya disertakan (\`Percobaan: 2/2\`). Gagal lagi \u2192 BERHENTI dan ${askEscalation} apa yang harus dilakukan. Aturan ini berlaku walau klausa otonomi di prompt ini menyuruhmu tak bertanya.`,
     `4. \`Status: menunggu-keputusan\` atau \`Keputusan terbuka:\` yang bukan \`-\` \u2192 SEBELUM menulis marker fase atau memanggil fase berikutnya, ajukan SEMUA pertanyaannya ${askDecisions}, lalu LANJUTKAN subagent yang SAMA lewat ${resume} dengan seluruh jawabannya. Ulangi selama subagent masih melaporkan keputusan terbuka \u2014 tak ada batas jumlah pertanyaan maupun putaran, dan giliran relay ini bukan percobaan ulang. JANGAN pernah menjawab sendiri, termasuk di sesi scheduler: yang menjawab ${DECIDER} \u2014 hanoman yang merutekannya, kamu cukup bertanya.${turnTaking}`,
     o.fastPath ? '5. `Rekomendasi fase: jalur-cepat` sesudah Audit \u2192 SEBELUM lanjut ke Execute: jalankan `echo "Spec skipped" >> "$HANOMAN_PHASE_FILE"` lalu `echo "Plan skipped" >> "$HANOMAN_PHASE_FILE"` (gerbang yang sama seperti langkah 2), lalu lanjut ke Execute. `penuh` \u2192 Spec \u2192 Plan \u2192 Execute.' : "",
     reviewStep,
     handoff,
     "DILARANG mengerjakan isi fase sendiri \u2014 termasuk saat subagent gagal. Menulis `skipped` untuk fase yang dilewati bukan mengerjakannya.",
-    'Pekerjaan ini BELUM selesai sampai SEMUA fase di daftar di atas punya baris `done` atau `skipped` di $HANOMAN_PHASE_FILE \u2014 JANGAN commit/push final atau menyatakan tuntas sebelum itu. Periksa dengan `cat "$HANOMAN_PHASE_FILE"` sebelum commit terakhir.'
+    "Pekerjaan ini BELUM selesai sampai SEMUA fase di daftar di atas punya baris `done` atau `skipped` di $HANOMAN_PHASE_FILE" + (review ? " (Review wajib `done`)" : "") + ' \u2014 JANGAN commit/push final atau menyatakan tuntas sebelum itu. Periksa dengan `cat "$HANOMAN_PHASE_FILE"` sebelum commit terakhir.'
   ].filter(Boolean).join("\n\n");
 }
 function startPrompt(flow2, spec, branchTo, autonomy, verifyScope, method, attachments, plan) {
@@ -17419,13 +17458,15 @@ function continuePrompt(flow2, spec, branchTo, autonomy, verifyScope, method, at
   const m = resolveMethod(method);
   const head = `hanoman ${flow2} \u2014 MELANJUTKAN backlog item yang sebelumnya ditandai selesai padahal pekerjaannya belum tuntas. Ikuti internal/docs sebagai Source of Truth; perbarui docs yang tersentuh dan link-nya di index, dalam commit yang sama.`;
   const push = `Setelah selesai: commit, lalu \`git push origin HEAD:refs/heads/${branchTo}\`. Worktree ini detached HEAD \u2014 itu memang disengaja.`;
+  const hasReview = plan ? plan.phases.some((p3) => p3.phase === REVIEWER_CELL) : PIPELINES[flow2].includes(REVIEWER_CELL);
   if (plan) {
     return [
       head,
-      `JANGAN mengulang fase awal \u2014 spec & plan sudah ada di ${m.planDir}/**. Lanjutkan HANYA fase Execute lewat subagent fasenya.`,
+      `JANGAN mengulang fase awal \u2014 spec & plan sudah ada di ${m.planDir}/**. Lanjutkan HANYA fase ` + (hasReview ? "Execute lalu Review lewat subagent fasenya." : "Execute lewat subagent fasenya."),
       // S3 · berkas fase tak pernah dikosongkan: baris `Execute done` run lama masih ada dan akan
       // memenuhi gerbang penutup sebelum agen Execute sesi ini bekerja sama sekali.
-      "$HANOMAN_PHASE_FILE masih memuat baris `Execute done` LAMA dari sesi sebelumnya \u2014 baris itu TIDAK berlaku untuk sesi ini dan tidak memenuhi gerbang penutup. Sesudah agen fase Execute SESI INI melapor `Status: selesai` (dan lolos langkah R bila ada), tulis baris `Execute done` BARU (verifikasi `tail -1`); commit/push final baru sah sesudah baris baru itu.",
+      // ADR-0230 · sama untuk `Review done` lama.
+      hasReview ? "$HANOMAN_PHASE_FILE bisa masih memuat baris `Execute done` dan `Review done` LAMA dari sesi sebelumnya \u2014 keduanya TIDAK berlaku untuk sesi ini dan tidak memenuhi gerbang penutup. Sesudah agen fase Execute SESI INI melapor `Status: selesai`, tulis baris `Execute done` BARU; sesudah agen fase Review SESI INI melapor `Verdict: lulus`, tulis baris `Review done` BARU (verifikasi `tail -1` tiap kali); commit/push final baru sah sesudah kedua baris baru itu." : "$HANOMAN_PHASE_FILE masih memuat baris `Execute done` LAMA dari sesi sebelumnya \u2014 baris itu TIDAK berlaku untuk sesi ini dan tidak memenuhi gerbang penutup. Sesudah agen fase Execute SESI INI melapor `Status: selesai`, tulis baris `Execute done` BARU (verifikasi `tail -1`); commit/push final baru sah sesudah baris baru itu.",
       orchestratorClause(plan),
       autonomyClause(autonomy),
       orchestratorAttachmentClause(attachments),
@@ -17435,13 +17476,13 @@ function continuePrompt(flow2, spec, branchTo, autonomy, verifyScope, method, at
   }
   return [
     head,
-    `JANGAN mengulang fase awal \u2014 spec & plan sudah ada. Lanjut di fase Execute: baca plan di ${m.planDir}/** untuk backlog item ini, periksa task yang sudah \`[x]\` dan selesaikan yang masih \`[ ]\`. Verifikasi nyata sebelum klaim selesai.`,
+    `JANGAN mengulang fase awal \u2014 spec & plan sudah ada. Lanjut di fase Execute: baca plan di ${m.planDir}/** untuk backlog item ini, periksa task yang sudah \`[x]\` dan selesaikan yang masih \`[ ]\`. Verifikasi nyata sebelum klaim selesai.` + (hasReview ? ' Sesudah itu jalankan fase Review, lalu tulis `echo "Review done" >> "$HANOMAN_PHASE_FILE"` BARU (baris `Review done` lama di berkas itu tak berlaku).\n' + SELF_REVIEW_CLAUSE : ""),
     autonomyClause(autonomy),
     scopeClause(flow2, verifyScope),
     codeStyleClause(flow2),
     methodClause(m),
     attachmentClause(attachments),
-    skillInstruction(flow2, ["Execute"], m),
+    skillInstruction(flow2, hasReview ? ["Execute", REVIEWER_CELL] : ["Execute"], m),
     push,
     specContext(spec)
   ].filter(Boolean).join("\n\n");
@@ -17614,7 +17655,7 @@ Stack: ${project.stack || "\u2014"}`,
 ${cron.prompt}`
   ].join("\n\n");
 }
-var PIPELINES, WORK_PHASES, DECIDER, ASK_ROUTE, AMBIGUOUS, AUTONOMY_CLAUSE, AUTONOMY_CLAUSE_FULL, autonomyClause, PLAN_SHAPE, phaseInstruction, skillInstruction, auditDecisionInstruction, ESCALATION_CONTRACT, auditOnlyInstruction, auditContinuationInstruction, auditContinuationForOrchestrator, writesCode, scopeClause, codeStyleClause, methodClause, attachmentClause, orchestratorAttachmentClause, PROJECT_METHOD, specContext, goalDetail, goalBlock, goalContext, projectContext, scaffoldContext, prdBriefBlock, prdAuditBlock, prdContext, breakdownContext, prdPhaseLines, breakdownPhaseLines, resumeClause, RESUMED_WORKTREE_NOTE, REVERSE_PHASE_GUIDE, SCAFFOLD_PHASE_GUIDE;
+var PIPELINES, WORK_PHASES, DECIDER, ASK_ROUTE, ASK_ESCALATION_TOOL, ASK_DECISIONS_TOOL, AMBIGUOUS, AUTONOMY_CLAUSE, AUTONOMY_CLAUSE_FULL, autonomyClause, PLAN_SHAPE, phaseInstruction, SELF_REVIEW_CLAUSE, skillInstruction, auditDecisionInstruction, ESCALATION_CONTRACT, auditOnlyInstruction, auditContinuationInstruction, auditContinuationForOrchestrator, writesCode, scopeClause, codeStyleClause, methodClause, attachmentClause, orchestratorAttachmentClause, PROJECT_METHOD, specContext, goalDetail, goalBlock, goalContext, projectContext, scaffoldContext, prdBriefBlock, prdAuditBlock, prdContext, breakdownContext, prdPhaseLines, breakdownPhaseLines, resumeClause, RESUMED_WORKTREE_NOTE, REVERSE_PHASE_GUIDE, SCAFFOLD_PHASE_GUIDE;
 var init_prompt = __esm({
   "../runner/src/prompt.ts"() {
     "use strict";
@@ -17627,6 +17668,8 @@ var init_prompt = __esm({
     WORK_PHASES = ["Execute", "Goal", "Kerjakan"];
     DECIDER = "hanoman-lead bila aktif untuk project ini, selain itu manusia";
     ASK_ROUTE = `ajukan lewat AskUserQuestion bila agenmu punya tool itu (paling banyak 4 pertanyaan per panggilan \u2014 pecah sisanya ke panggilan berikutnya); bila tidak, tanyakan di terminal ini sekaligus, tiap pertanyaan bernomor dan diakhiri \`?\` beserta opsinya, lalu tunggu jawabannya. Tak ada batas jumlah pertanyaan: selama masih ambigu dan akan mempengaruhi hasil, tanyakan. Yang menjawab ditentukan hanoman \u2014 ${DECIDER}; jangan menjawabnya sendiri.`;
+    ASK_ESCALATION_TOOL = "tanyakan lewat AskUserQuestion";
+    ASK_DECISIONS_TOOL = "lewat AskUserQuestion \u2014 tool itu memuat paling banyak 4 pertanyaan per panggilan, jadi pecah sisanya ke panggilan berikutnya sampai semuanya terjawab";
     AMBIGUOUS = "hal yang masih ambigu dan akan mempengaruhi hasil (data model, kontrak API, scope, perilaku yang terlihat pengguna, atau asumsi yang terpaksa kamu ambil agar bisa lanjut)";
     AUTONOMY_CLAUSE = `Jalankan seluruh pipeline sampai tuntas tanpa berhenti di batas antar-fase. Checkpoint "review"/"approval"/"need review" milik skill superpowers BUKAN titik berhenti di sini \u2014 lanjut saja ke fase berikutnya. Berhenti HANYA untuk ${AMBIGUOUS}: JANGAN memutuskannya sendiri, ${ASK_ROUTE} Selain itu, terus lanjut.`;
     AUTONOMY_CLAUSE_FULL = `Kamu diluncurkan scheduler \u2014 mungkin tak ada manusia yang sedang menonton terminal ini. Tembus seluruh pipeline sampai stage \`done\`, lalu commit & push: checkpoint "review"/"approval" milik skill BUKAN titik berhenti dan jangan menunggu persetujuan siapa pun. Tetapi JANGAN memutuskan sendiri ${AMBIGUOUS}: ${ASK_ROUTE} Catat keputusan penting di pesan commit. Merge ke branch utama tetap dilakukan manusia, bukan kamu.`;
@@ -17638,8 +17681,10 @@ Setiap kali sebuah fase selesai (atau kamu putuskan dilewati), append satu baris
       if (!phases2.includes("Plan") || !phases2.includes("Execute")) return base3;
       return base3 + `
 Execute BELUM selesai selama plan (\`${method.planDir}/**\`) masih punya task \`- [ ]\`: kerjakan SEMUA PR/task sampai tiap kotak jadi \`- [x]\` sebelum menulis \`Execute done\`. hanoman menahan backlog di \`executing\`, bukan \`done\`, selama masih ada \`- [ ]\`.
-${PLAN_SHAPE}`;
+${PLAN_SHAPE}` + (phases2.includes(REVIEWER_CELL) ? `
+${SELF_REVIEW_CLAUSE}` : "");
     };
+    SELF_REVIEW_CLAUSE = "Fase Review (sesudah `Execute done`): periksa ulang hasil Execute terhadap dokumen spec & plan backlog ini \u2014 baca seluruh perubahan sesi ini (diff dari commit awal worktree) dan jalankan test sesuai scope verifikasi. Tulis tabel `AC \u2192 putusan \u2192 bukti` (setiap acceptance criteria \u2192 perintah & hasil yang kamu baca, atau path:baris) lalu satu baris `Verdict: lulus` atau `Verdict: rework`. `rework` \u2192 perbaiki temuannya, lalu review ulang; maks 2 putaran rework, sisanya jadi keputusan terbuka. Tulis `Review done` HANYA sesudah `Verdict: lulus`; Review TIDAK PERNAH `skipped`.";
     skillInstruction = (flow2, phases2, method) => {
       const lines2 = phases2.map((p3) => {
         const skills2 = phaseSkillsFor(flow2, p3, method);
@@ -17950,7 +17995,7 @@ var init_settings2 = __esm({
       'process.stdin.on("data",c=>d+=c).on("end",()=>{let n=0;try{n=(JSON.parse(d).background_tasks||[]).filter(t=>t&&t.type==="subagent"&&t.status==="running").length}catch{}require("fs").writeFileSync(process.argv[1],"1\\n".repeat(n))})'
     ].join(";");
     busySnapshotCommand = (quotedSubFile) => `node -e '${BUSY_SNAPSHOT_SCRIPT}' ${quotedSubFile} >/dev/null 2>&1`;
-    guardSettings = (decisionFile, goal, eventHook, subagentStatusLine) => {
+    guardSettings = (decisionFile, goal, eventHook, subagentStatusLine, askText) => {
       const hooks2 = {};
       if (decisionFile) {
         const f = `'${decisionFile.split("'").join("'\\''")}'`;
@@ -17978,6 +18023,8 @@ var init_settings2 = __esm({
           hooks2.UserPromptSubmit = [{ hooks: [{ type: "command", command: `: > ${f}; : > ${sub}` }] }];
         }
       }
+      if (eventHook && askText)
+        (hooks2.Stop ??= []).push({ hooks: [{ type: "command", command: EVENT_HOOK_COMMAND }] });
       if (goal) (hooks2.Stop ??= []).push({ hooks: [{ type: "prompt", prompt: goal }] });
       return {
         hooks: hooks2,
@@ -18006,7 +18053,7 @@ function defaultGoalCondition({ flow: flow2, specId, branchTo, spec }) {
   const phases2 = PIPELINES[flow2];
   const planGate = phases2.includes("Plan") && phases2.includes("Execute");
   const clauses = [
-    `1. output \`cat "$HANOMAN_PHASE_FILE"\` yang memuat satu baris untuk SETIAP fase ${phases2.join(" \u2192 ")}, masing-masing berakhiran \`done\` atau \`skipped\`;`
+    `1. output \`cat "$HANOMAN_PHASE_FILE"\` yang memuat satu baris untuk SETIAP fase ${phases2.join(" \u2192 ")}, masing-masing berakhiran \`done\` atau \`skipped\`` + phases2.filter((p3) => UNSKIPPABLE_PHASES.has(p3)).map((p3) => ` (\`${p3}\` wajib \`done\`)`).join("") + ";"
   ];
   if (planGate) {
     const dirs = PLAN_DIRS.map((d) => `${d}/`).join(" ");
@@ -18079,7 +18126,8 @@ function codexGoalScript(o) {
   ];
   for (const p3 of phases2) {
     lines2.push(
-      `grep -qE ${shq(`^${p3} (done|skipped)[[:space:]]*$`)} "$PF" 2>/dev/null || missing="$missing\\n- fase ${p3} belum tercatat di \\$HANOMAN_PHASE_FILE"`
+      // ADR-0230 · fase `UNSKIPPABLE_PHASES` (Review) hanya lolos dengan `done`.
+      `grep -qE ${shq(`^${p3} ${UNSKIPPABLE_PHASES.has(p3) ? "done" : "(done|skipped)"}[[:space:]]*$`)} "$PF" 2>/dev/null || missing="$missing\\n- fase ${p3} belum tercatat di \\$HANOMAN_PHASE_FILE"`
     );
   }
   if (planGate) {
@@ -18156,7 +18204,7 @@ function agentFlags(o) {
     ...o.effort ? ["--effort", o.effort] : [],
     "--dangerously-skip-permissions",
     "--settings",
-    JSON.stringify(guardSettings(o.decisionFile, o.goal, o.eventHook, o.subagentStatusLine))
+    JSON.stringify(guardSettings(o.decisionFile, o.goal, o.eventHook, o.subagentStatusLine, o.askText))
   ];
 }
 var init_agent_cli = __esm({
@@ -18430,7 +18478,7 @@ function agentDelegationClause(defs, runtime = "claude") {
   if (defs.every((def2) => def2.kind === "phase")) return "";
   const codex2 = runtime === "codex";
   const antigravity2 = runtime === "antigravity";
-  const relay = antigravity2 ? "ajukan semuanya di terminal ini (bernomor, tiap pertanyaan diakhiri `?`), lalu teruskan jawabannya lewat send_message ke conversation ID child yang sama" : codex2 ? "ajukan semuanya di terminal ini (bernomor, tiap pertanyaan diakhiri `?`), lalu teruskan jawabannya lewat send_input ke agent id yang sama" : "ajukan semuanya lewat AskUserQuestion (pecah per 4 pertanyaan), lalu teruskan jawabannya lewat SendMessage ke agent ID yang sama";
+  const relay = antigravity2 ? "ajukan semuanya di terminal ini (bernomor, tiap pertanyaan diakhiri `?`), lalu teruskan jawabannya lewat send_message ke conversation ID child yang sama" : codex2 ? "ajukan semuanya di terminal ini (bernomor, tiap pertanyaan diakhiri `?`), lalu teruskan jawabannya lewat send_input ke agent id yang sama" : ASK_RELAY_TOOL;
   return [
     "",
     "",
@@ -18439,7 +18487,7 @@ function agentDelegationClause(defs, runtime = "claude") {
     ""
   ].join("\n");
 }
-var PHASE_CONTEXT_HEADER, liveMentions, policyOf, READ_ONLY_DENIED_OPERATORS, READ_ONLY_DENIED_COMMANDS, deniedCommandsProse, readOnlyAllowlistLines, NO_FAKE_EXPERIMENT, policyLines, ownPolicyLines, workLimitClause, STATUS_LINE, handoffClause, contractReference, READ_ONLY_TOOLS;
+var ASK_RELAY_TOOL, PHASE_CONTEXT_HEADER, liveMentions, policyOf, READ_ONLY_DENIED_OPERATORS, READ_ONLY_DENIED_COMMANDS, deniedCommandsProse, readOnlyAllowlistLines, NO_FAKE_EXPERIMENT, policyLines, ownPolicyLines, workLimitClause, STATUS_LINE, handoffClause, contractReference, READ_ONLY_TOOLS;
 var init_custom_agents = __esm({
   "../runner/src/custom-agents.ts"() {
     "use strict";
@@ -18447,6 +18495,7 @@ var init_custom_agents = __esm({
     init_code_style();
     init_prompt();
     init_agent_readonly();
+    ASK_RELAY_TOOL = "ajukan semuanya lewat AskUserQuestion (pecah per 4 pertanyaan), lalu teruskan jawabannya lewat SendMessage ke agent ID yang sama";
     PHASE_CONTEXT_HEADER = "=== KONTEKS ===";
     liveMentions = (def2, roster) => {
       if (def2.workspacePolicy === "read-only") return [];
@@ -18575,6 +18624,35 @@ var init_custom_agents = __esm({
   }
 });
 
+// ../runner/src/ask-via.ts
+function textAsk(s2) {
+  let out4 = s2;
+  for (const [tool, text2] of SWAPS) out4 = out4.split(tool).join(text2);
+  return out4;
+}
+var NO_TOOL, TEXT_ASK_ROUTE, SWAPS;
+var init_ask_via = __esm({
+  "../runner/src/ask-via.ts"() {
+    "use strict";
+    init_prompt();
+    init_custom_agents();
+    NO_TOOL = "JANGAN memakai AskUserQuestion (juga bila sebuah skill menyuruhnya)";
+    TEXT_ASK_ROUTE = `${NO_TOOL}: tanyakan di terminal ini sekaligus dalam satu pesan, tiap pertanyaan bernomor dan diakhiri \`?\` beserta opsi, rekomendasi dan alasannya, lalu berhenti dan tunggu jawabannya. Tak ada batas jumlah pertanyaan: selama masih ambigu dan akan mempengaruhi hasil, tanyakan. Yang menjawab ditentukan hanoman \u2014 ${DECIDER}; jangan menjawabnya sendiri.`;
+    SWAPS = [
+      [ASK_ROUTE, TEXT_ASK_ROUTE],
+      [
+        ASK_DECISIONS_TOOL,
+        `di terminal ini sekaligus dalam satu pesan (${NO_TOOL}), tiap pertanyaan bernomor dan diakhiri \`?\` beserta opsi, rekomendasi dan alasannya, lalu berhenti dan tunggu jawaban`
+      ],
+      [
+        ASK_RELAY_TOOL,
+        `ajukan semuanya di terminal ini (${NO_TOOL}; bernomor, tiap pertanyaan diakhiri \`?\`), lalu teruskan jawabannya lewat SendMessage ke agent ID yang sama`
+      ],
+      [ASK_ESCALATION_TOOL, `tanyakan di terminal ini (${NO_TOOL}) lalu berhenti dan tunggu jawaban`]
+    ];
+  }
+});
+
 // ../runner/src/runtime-profile.ts
 function resolveHardening(env) {
   if (env.HANOMAN_HARDENING === "1") return true;
@@ -18834,7 +18912,7 @@ function phaseAgentInstructions(entry2, index, plan, ctx) {
   const guide = PROJECT_FLOWS.has(flow2) ? projectGuide(flow2, phase, ctx) : backlogGuide(flow2, phase, ctx);
   const executeMode = plan.executeMode ?? "inline";
   const skills2 = phaseSkillsFor(flow2, phase, method, plan.runtime, executeMode);
-  const skipSkillFinalReview = phase === "Execute" && plan.runtime === "claude" && executeMode === "subagent" && !!plan.reviewer && skills2.includes("superpowers:subagent-driven-development");
+  const skipSkillFinalReview = phase === "Execute" && plan.runtime === "claude" && executeMode === "subagent" && plan.phases.some((p3) => p3.phase === REVIEWER_CELL) && skills2.includes("superpowers:subagent-driven-development");
   const work = WORK_PHASES.includes(phase);
   const verifies = work || phase === "Verifikasi";
   return [
@@ -18843,7 +18921,7 @@ function phaseAgentInstructions(entry2, index, plan, ctx) {
     DOC_WRITING_PHASES.has(phase) ? `=== STANDAR DOCS ===
 ${REVERSE_STANDARD}` : "",
     skills2.length ? `Skills ${method.label} WAJIB untuk fase ini \u2014 muat & ikuti dengan mekanisme yang tersedia di agenmu: ` + skills2.join(", ") : "",
-    skipSkillFinalReview ? "LEWATI final whole-branch review milik skill subagent-driven-development: review akhir dipegang reviewer `hanoman-fase-review` yang dipanggil orchestrator sesudah fase ini. Review per task tetap jalan." : "",
+    skipSkillFinalReview ? "LEWATI final whole-branch review milik skill subagent-driven-development: review akhir dipegang fase Review (`hanoman-fase-review`) yang dipanggil orchestrator sesudah fase ini. Review per task tetap jalan." : "",
     verifies ? scopeClause(flow2, ctx.verifyScope) : "",
     work ? codeStyleClause(flow2) : "",
     methodClause(method),
@@ -18858,40 +18936,29 @@ ${REVERSE_STANDARD}` : "",
   ].filter(Boolean).join("\n\n");
 }
 function buildPhaseAgents(plan, ctx) {
-  const phases2 = plan.phases.map((entry2, index) => ({
-    kind: "phase",
-    phase: entry2.phase,
-    name: entry2.agentName,
-    description: `Fase ${entry2.phase} flow ${plan.flow} hanoman \u2014 hanya dipanggil orchestrator sesi ini.`,
-    instructions: phaseAgentInstructions(entry2, index, plan, ctx),
-    // T2 · konteks bersama dibawa TERPISAH (lihat `phasePromptOf`): claude merujuknya lewat satu
-    // berkas di temp dir sesi, codex tetap inline.
-    context: ctx.context,
-    tools: null,
-    model: entry2.model,
-    effort: entry2.effort,
-    mentions: [],
-    executeMode: plan.executeMode ?? "inline"
-  }));
-  const r = plan.reviewer;
-  if (!r || !plan.phases.some((p3) => p3.phase === r.phase)) return phases2;
-  return [...phases2, {
-    kind: "phase",
-    phase: r.phase,
-    name: r.agentName,
-    description: `Reviewer independen hasil fase Execute flow ${plan.flow} hanoman \u2014 hanya dipanggil orchestrator sesi ini, sesudah Execute melapor selesai.`,
-    instructions: reviewerInstructions(plan, ctx),
-    context: ctx.context,
-    tools: null,
-    model: r.model,
-    effort: r.effort,
-    mentions: []
-  }];
+  return plan.phases.map((entry2, index) => {
+    const review = entry2.phase === REVIEWER_CELL;
+    return {
+      kind: "phase",
+      phase: entry2.phase,
+      name: entry2.agentName,
+      description: review ? `Reviewer independen hasil fase Execute flow ${plan.flow} hanoman (fase Review) \u2014 hanya dipanggil orchestrator sesi ini, sesudah Execute ditutup.` : `Fase ${entry2.phase} flow ${plan.flow} hanoman \u2014 hanya dipanggil orchestrator sesi ini.`,
+      instructions: review ? reviewerInstructions(plan, ctx) : phaseAgentInstructions(entry2, index, plan, ctx),
+      // T2 · konteks bersama dibawa TERPISAH (lihat `phasePromptOf`): claude merujuknya lewat satu
+      // berkas di temp dir sesi, codex tetap inline.
+      context: ctx.context,
+      tools: null,
+      model: entry2.model,
+      effort: entry2.effort,
+      mentions: [],
+      ...review ? {} : { executeMode: plan.executeMode ?? "inline" }
+    };
+  });
 }
 function reviewerInstructions(plan, ctx) {
   const m = ctx.method;
   return [
-    `Kamu reviewer independen hasil fase Execute flow ${plan.flow} hanoman. Kamu dipanggil orchestrator sesudah agen Execute melapor \`Status: selesai\` dan SEBELUM fase itu ditutup. Kamu pengadu janji: yang kamu nilai bukan bagus-tidaknya kode, melainkan apakah yang DIMINTA benar-benar ada dan TERBUKTI pada keadaan akhir worktree ini.`,
+    `Kamu agen fase Review \u2014 reviewer independen hasil fase Execute flow ${plan.flow} hanoman. Kamu dipanggil orchestrator sesudah fase Execute ditutup, dan fase Review baru ditutup bila kamu memberi \`Verdict: lulus\`. Kamu pengadu janji: yang kamu nilai bukan bagus-tidaknya kode, melainkan apakah yang DIMINTA benar-benar ada dan TERBUKTI pada keadaan akhir worktree ini.`,
     "Prosedur:",
     [
       `1. Baca dokumen spec & plan dari baris \`Artefak:\` serah-terima (qa jalur-cepat: dokumen audit adalah doc-of-record; bila path tak ada, cari berkas ber-spec-id di \`${m.specDir}/\` & \`${m.planDir}/\`). Plan yang menyimpang dari spec adalah temuan.`,
@@ -20880,6 +20947,10 @@ var init_antigravity_agent_config = __esm({
 var src_exports = {};
 __export(src_exports, {
   ANTIGRAVITY_ENV_KEYS: () => ANTIGRAVITY_ENV_KEYS,
+  ASK_DECISIONS_TOOL: () => ASK_DECISIONS_TOOL,
+  ASK_ESCALATION_TOOL: () => ASK_ESCALATION_TOOL,
+  ASK_RELAY_TOOL: () => ASK_RELAY_TOOL,
+  ASK_ROUTE: () => ASK_ROUTE,
   AUTONOMY_CLAUSE: () => AUTONOMY_CLAUSE,
   CODEX_NATIVE_AGENTS_MIN_CLIENT: () => CODEX_NATIVE_AGENTS_MIN_CLIENT,
   CODE_STYLE_CLAUSE: () => CODE_STYLE_CLAUSE,
@@ -20902,8 +20973,10 @@ __export(src_exports, {
   REVERSE_PHASE_GUIDE: () => REVERSE_PHASE_GUIDE,
   REVERSE_STANDARD: () => REVERSE_STANDARD,
   SCAFFOLD_PHASE_GUIDE: () => SCAFFOLD_PHASE_GUIDE,
+  SELF_REVIEW_CLAUSE: () => SELF_REVIEW_CLAUSE,
   SUBAGENT_STATUSLINE_SCRIPT: () => SUBAGENT_STATUSLINE_SCRIPT,
   SkillError: () => SkillError,
+  TEXT_ASK_ROUTE: () => TEXT_ASK_ROUTE,
   WORK_PHASES: () => WORK_PHASES,
   agentContractOf: () => agentContractOf,
   agentDefinitionHash: () => agentDefinitionHash,
@@ -21004,6 +21077,7 @@ __export(src_exports, {
   startProjectPrompt: () => startProjectPrompt,
   startPrompt: () => startPrompt,
   startScaffoldPrompt: () => startScaffoldPrompt,
+  textAsk: () => textAsk,
   tomlBasicString: () => tomlBasicString,
   validateAgentEvalManifest: () => validateAgentEvalManifest,
   verifyScopeClause: () => verifyScopeClause,
@@ -21027,6 +21101,7 @@ var init_src2 = __esm({
     init_codex_settings();
     init_agent_cli();
     init_custom_agents();
+    init_ask_via();
     init_agent_definition();
     init_phase_agents();
     init_agent_readonly();
@@ -21325,6 +21400,7 @@ function stageFor(phases2) {
     if (WORK_PHASES.includes(p3.name) && p3.state === "active")
       best = Math.max(best, STAGES.indexOf("executing"));
     if (p3.state !== "done" && p3.state !== "skipped") continue;
+    if (p3.state === "skipped" && UNSKIPPABLE_PHASES.has(p3.name)) continue;
     const s2 = REACHED[p3.name];
     if (s2) best = Math.max(best, STAGES.indexOf(s2));
   }
@@ -21439,8 +21515,11 @@ var init_session_phases = __esm({
       Audit: "objective",
       Spec: "spec-ready",
       Plan: "planned",
+      // ADR-0230 · flow feature & qa ditutup fase Review resmi: Execute selesai baru `executing`, Review
+      // yang mencapai `done`. `Review` unik lintas PIPELINES dan bukan fase kerja (`WORK_PHASES`).
       Laporan: "done",
-      Execute: "done",
+      Execute: "executing",
+      Review: "done",
       // SPEC-407 · ADR-0089 · flow goal (Goal → Verifikasi): fase kerja mencapai `executing`, fase
       // verifikasi yang mencapai `done`. Kedua nama unik lintas PIPELINES — peta ini berkunci nama.
       Goal: "executing",
@@ -21470,7 +21549,7 @@ var init_session_phases = __esm({
       if (PLAN_DIRS.some((d) => rel.startsWith(`${d}/`) && !rel.slice(d.length + 1).includes("/"))) return false;
       return re.test(rel.slice(rel.lastIndexOf("/") + 1).toLowerCase());
     });
-    phasesComplete = (phases2) => phases2.length > 0 && phases2.every((p3) => p3.state === "done" || p3.state === "skipped");
+    phasesComplete = (phases2) => phases2.length > 0 && phases2.every((p3) => p3.state === "done" || p3.state === "skipped" && !UNSKIPPABLE_PHASES.has(p3.name));
   }
 });
 
@@ -48954,8 +49033,13 @@ function createSession(projectId, cwd, opts2 = {}) {
   }
   const agentForDefs = opts2.agent ?? "claude";
   const sessionEffort = agentForDefs === "codex" && opts2.model && opts2.effort ? coerceCodexEffort(opts2.model, opts2.effort) : opts2.effort;
-  const requestedPhaseDefs = opts2.command ? [] : opts2.phaseAgents ?? [];
-  const bornPrompt = (orch) => orch ? opts2.prompt : opts2.legacyPrompt ?? opts2.prompt;
+  const askText = !opts2.command && agentForDefs === "claude" && opts2.askVia === "text";
+  const asked = (s2) => askText ? textAsk(s2) : s2;
+  const requestedPhaseDefs = opts2.command ? [] : (opts2.phaseAgents ?? []).map((d) => askText ? { ...d, instructions: textAsk(d.instructions) } : d);
+  const bornPrompt = (orch) => {
+    const p3 = orch ? opts2.prompt : opts2.legacyPrompt ?? opts2.prompt;
+    return p3 === void 0 ? p3 : asked(p3);
+  };
   const selectionContext = {
     projectId,
     runtime: agentForDefs,
@@ -49041,7 +49125,7 @@ function createSession(projectId, cwd, opts2 = {}) {
           return [error instanceof Error ? error.message : String(error)];
         }
         agentsFile = file;
-        rosterBlock = agentDelegationClause(customDefs, "claude");
+        rosterBlock = asked(agentDelegationClause(customDefs, "claude"));
         liveAgentDefs = defs;
         renderedDefs = defs;
         return [];
@@ -49147,7 +49231,8 @@ function createSession(projectId, cwd, opts2 = {}) {
       // SPEC-909 · ADR-0146 · sesi ber-`opts.command` tak pernah sampai ke sini (cabang shell
       // mentah di atas), jadi hook event hanya pernah terpasang di sesi agen.
       eventHook: true,
-      subagentStatusLine: statusLineCommand
+      subagentStatusLine: statusLineCommand,
+      askText
     }).map(sq).join(" ");
     const agentsArg = agentsFile ? `--agents "$(cat ${sq(agentsFile)})"` : "";
     const nativeAgentArgs = agent === "codex" ? codexAgentArgs.map(sq).join(" ") : "";
@@ -58187,6 +58272,22 @@ function statesFor(raw) {
         if (key.startsWith(prefix)) orchestration[key] = state;
       }
     }
+    const rawOrchestration = isRecord(object2.orchestration) ? object2.orchestration : {};
+    for (const flow2 of ORCHESTRATION_FLOWS) {
+      const rawFlow = isRecord(rawOrchestration[flow2]) ? rawOrchestration[flow2] : {};
+      for (const runtime of ["claude", "codex"]) {
+        const rawRuntime = isRecord(rawFlow[runtime]) ? rawFlow[runtime] : {};
+        for (const phase of FLOW_PHASES[flow2]) {
+          const rawCell = rawRuntime[phase];
+          if (!isRecord(rawCell)) continue;
+          for (const field of ["model", "effort"]) {
+            const key = `${flow2}.${runtime}.${phase}.${field}`;
+            if (!(key in marker.data.orchestration) && rawCell[field] !== void 0 && rawCell[field] !== null)
+              orchestration[key] = "user";
+          }
+        }
+      }
+    }
     const rawMarker = isRecord(object2.builtinRuntimeDefaults) ? object2.builtinRuntimeDefaults : {};
     return {
       claude: marker.data.claude,
@@ -59409,7 +59510,7 @@ var init_flow = __esm({
 });
 
 // src/services/lead/pane.ts
-function readCodexTurn(message) {
+function readTurn(message) {
   const body = message.trim();
   if (!body) return { asking: false, reason: "giliran codex berakhir tanpa pesan" };
   if (CODEX_FINISHED.some((re) => re.test(body)))
@@ -60144,7 +60245,7 @@ async function runChain(ask, s2, deps) {
   const stop = (reason, failed = false) => ({ acted, done: false, failed, reason, at, flowId, step });
   const asked = ask.source === "ask-tool";
   if (ask.source === "turn-end") {
-    const read2 = readCodexTurn(ask.message);
+    const read2 = readTurn(ask.message);
     if (!read2.asking) return stop(read2.reason);
   }
   const steps = ask.source === "ask-tool" ? ask.questions : [null];
@@ -66215,6 +66316,28 @@ var init_project_checkout = __esm({
   }
 });
 
+// src/services/lead/ask-via.ts
+function leadDecides(cfg, optIn, projectId) {
+  return optIn.includes(projectId) && leadActive(cfg, projectId);
+}
+async function askViaFor(projectId, agent, deps = prodDeps2) {
+  if ((agent ?? "claude") !== "claude") return void 0;
+  try {
+    const [cfg, optIn] = await Promise.all([deps.cfg(), deps.optIn()]);
+    return leadDecides(cfg, optIn, projectId) ? "text" : "tool";
+  } catch {
+    return "tool";
+  }
+}
+var prodDeps2;
+var init_ask_via2 = __esm({
+  "src/services/lead/ask-via.ts"() {
+    "use strict";
+    init_config5();
+    prodDeps2 = { cfg: getLead, optIn: leadProjects };
+  }
+});
+
 // src/services/session-launch.ts
 function resumeState(repoDir, worktree, branchTo, headSha) {
   if (realGit.worktreeAlive(worktree)) return { worktreeKept: true };
@@ -66332,7 +66455,7 @@ async function startSpecSession(spec, opts2) {
     const resumeCtx = resume ? buildResumeCtx(repoDir, id2, opts2.flow, resume.worktreeKept) : void 0;
     let plan = fullPlan;
     if (fullPlan && isContinue && !isGoalFlow) {
-      const phases2 = fullPlan.phases.filter((p3) => p3.phase === "Execute");
+      const phases2 = fullPlan.phases.filter((p3) => p3.phase === "Execute" || p3.phase === REVIEWER_CELL);
       plan = phases2.length ? { ...fullPlan, phases: phases2 } : null;
     }
     const buildPrompt = (p3) => {
@@ -66362,6 +66485,7 @@ async function startSpecSession(spec, opts2) {
       ...resumeCtx ? { resume: resumeCtx } : {}
     }) : [];
     const scopeEnv = { HANOMAN_BASE_SHA: baseSha, HANOMAN_VERIFY_SCOPE: verifyScope };
+    const askVia = await askViaFor(spec.projectId, agent);
     const s2 = createSession(spec.projectId, worktree, {
       specId: spec.id,
       flow: opts2.flow,
@@ -66369,13 +66493,14 @@ async function startSpecSession(spec, opts2) {
       effort: effort2,
       goal,
       agent,
+      askVia,
       phaseFile: phaseFilePath(repoDir, id2),
       decisionFile: decisionFilePath(repoDir, id2),
       attachmentsDir: attachments.items.length ? attachments.dir : void 0,
       prompt,
       legacyPrompt,
       phaseAgents,
-      // S3 · SPEC-172 · berkas fase continue masih memuat `Execute done` run lama; fase yang DIKERJAKAN
+      // S3 · SPEC-172 · berkas fase continue masih memuat `Execute done` (ADR-0230 · dan `Review done`) run lama; fase yang DIKERJAKAN
       // ULANG sesi ini tak boleh dihitung `doneAtBirth`, supaya ⚠ bukti tetap bisa menyala.
       ...isContinue && plan ? { rerunPhases: plan.phases.map((p3) => p3.phase) } : {},
       env: scopeEnv
@@ -66406,6 +66531,7 @@ var init_session_launch = __esm({
     init_remote_session();
     init_registry();
     init_event_log();
+    init_ask_via2();
     init_pty();
     LaunchError = class extends Error {
       // SPEC-447 · `blockers` hanya terisi untuk kind "blocked"; route memetakannya ke body 409.
@@ -79238,6 +79364,7 @@ async function refuseSelfDecision(input) {
 // src/routes/terminal.ts
 init_tui_dialog();
 init_runtime();
+init_ask_via2();
 var ATTACHMENT_MIME = /* @__PURE__ */ new Set(["image/png", "image/jpeg", "image/webp"]);
 var DIALOG_ANSWER_ERROR = {
   stale: "layar dialog sudah berubah \u2014 muat ulang pertanyaannya",
@@ -79359,12 +79486,14 @@ async function terminal_default(app3, opts2) {
         const brief = { id: project.id, name: project.name, desc: project.desc, stack: project.stack };
         const plan = sessionPhasePlan(await getSetting(), "reverse", agent2, { model: model2, effort: effort3 });
         const legacyPrompt = startProjectPrompt("reverse", brief, "reverse-docs") + resumeNote(reused);
+        const askVia = await askViaFor(project.id, agent2);
         const s3 = createSession(project.id, wt, {
           id: id2,
           flow: "reverse",
           model: model2,
           effort: effort3,
           agent: agent2,
+          askVia,
           phaseFile: phaseFilePath(repoDir, id2),
           decisionFile: decisionFilePath(repoDir, id2),
           prompt: plan ? startProjectPrompt("reverse", brief, "reverse-docs", plan) + resumeNote(reused) : legacyPrompt,
@@ -79397,12 +79526,14 @@ async function terminal_default(app3, opts2) {
         const brief = { id: project.id, name: project.name, desc: project.desc, stack: project.stack };
         const plan = sessionPhasePlan(await getSetting(), "scaffold", agent2, { model: model2, effort: effort3 });
         const legacyPrompt = startScaffoldPrompt(brief, "scaffold-docs") + resumeNote(reused);
+        const askVia = await askViaFor(project.id, agent2);
         const s3 = createSession(project.id, wt, {
           id: id2,
           flow: "scaffold",
           model: model2,
           effort: effort3,
           agent: agent2,
+          askVia,
           phaseFile: phaseFilePath(repoDir, id2),
           decisionFile: decisionFilePath(repoDir, id2),
           prompt: plan ? startScaffoldPrompt(brief, "scaffold-docs", plan) + resumeNote(reused) : legacyPrompt,
@@ -79439,6 +79570,7 @@ async function terminal_default(app3, opts2) {
         const audit = auditDoc ? { id: fromAudit, path: auditDoc.path, content: auditDoc.content } : void 0;
         const plan = sessionPhasePlan(await getSetting(), "prd", agent2, { model: model2, effort: effort3 });
         const legacyPrompt = startPrdPrompt(project_, brief, `prd/${slug2}`, audit) + resumeNote(reused);
+        const askVia = await askViaFor(project.id, agent2);
         const s3 = createSession(project.id, wt, {
           id: id2,
           flow: "prd",
@@ -79446,6 +79578,7 @@ async function terminal_default(app3, opts2) {
           model: model2,
           effort: effort3,
           agent: agent2,
+          askVia,
           phaseFile: phaseFilePath(repoDir, id2),
           decisionFile: decisionFilePath(repoDir, id2),
           prompt: plan ? startPrdPrompt(project_, brief, `prd/${slug2}`, audit, plan) + resumeNote(reused) : legacyPrompt,
@@ -79487,6 +79620,7 @@ async function terminal_default(app3, opts2) {
         const prd = { title, path: prdPath, content };
         const plan = sessionPhasePlan(await getSetting(), "breakdown", agent2, { model: model2, effort: effort3 });
         const legacyPrompt = startBreakdownPrompt(project_, prd, `breakdown/${slug2}`) + resumeNote(reused);
+        const askVia = await askViaFor(project.id, agent2);
         const s3 = createSession(project.id, wt, {
           id: id2,
           flow: "breakdown",
@@ -79494,6 +79628,7 @@ async function terminal_default(app3, opts2) {
           model: model2,
           effort: effort3,
           agent: agent2,
+          askVia,
           phaseFile: phaseFilePath(repoDir, id2),
           decisionFile: decisionFilePath(repoDir, id2),
           prompt: plan ? startBreakdownPrompt(project_, prd, `breakdown/${slug2}`, plan) + resumeNote(reused) : legacyPrompt,
@@ -86981,6 +87116,7 @@ init_settings3();
 init_codex_trust();
 init_qa_attachment_transfer();
 init_uploads();
+init_ask_via2();
 import { createHash as createHash21 } from "node:crypto";
 import { execFileSync as execFileSync10 } from "node:child_process";
 import { join as join50 } from "node:path";
@@ -87053,6 +87189,7 @@ ${files2.join("\n")}` : "",
       CODE_STYLE_CLAUSE,
       reused ? RESUMED_WORKTREE_NOTE : ""
     ].filter(Boolean).join("\n\n");
+    const askVia = await askViaFor(d.projectId, agent);
     const s2 = createSession(d.projectId, wt, {
       id: id2,
       branch,
@@ -87060,6 +87197,7 @@ ${files2.join("\n")}` : "",
       model,
       effort: effort2,
       prompt,
+      askVia,
       env: { HANOMAN_BASE_SHA: baseSha, HANOMAN_VERIFY_SCOPE: verifyScope },
       attachmentsDir: files2.length ? sessionUploadDir(id2) : void 0
     });
@@ -90562,6 +90700,7 @@ init_settings3();
 init_codex_trust();
 init_pty();
 init_session_launch_gate();
+init_ask_via2();
 var cronSessionId = (cronId) => `cron-${cronId.toLowerCase().replace(/[^a-z0-9_-]/g, "_")}`;
 async function liveCronSession(cronId) {
   const s2 = await getSessionAsync(cronSessionId(cronId));
@@ -90582,11 +90721,13 @@ async function startCronSession(cron) {
     if (agent === "codex") ensureCodexTrust(repoDir);
     const wt = `${repoDir}/.worktrees/${id2}`;
     if (!realGit.worktreeAlive(wt)) realGit.addWorktree(repoDir, wt, "HEAD");
+    const askVia = await askViaFor(cron.projectId, agent);
     const s2 = createSession(cron.projectId, wt, {
       id: id2,
       agent,
       model,
       effort: effort2,
+      askVia,
       prompt: cronPrompt(
         { id: project.id, name: project.name, desc: project.desc, stack: project.stack },
         { name: cron.name, prompt: cron.prompt }
@@ -90707,7 +90848,7 @@ async function tick4(now, deps, end2 = prodEnd) {
     console.error("scheduler drain:", e);
   }
 }
-var prodDeps2 = {
+var prodDeps3 = {
   liveCount: async () => (await listSessionsAsync()).filter((s2) => !s2.exited).length,
   isLive: async (specId) => {
     const s2 = await getSessionAsync(sessionIdForSpec(specId));
@@ -90741,7 +90882,7 @@ var prodDeps2 = {
 };
 var TICK_MS3 = 1e4;
 var timer7;
-function startScheduler(deps = prodDeps2) {
+function startScheduler(deps = prodDeps3) {
   if (timer7) return;
   timer7 = setInterval(() => void tick4(Date.now(), deps), TICK_MS3);
   timer7.unref();
@@ -90778,7 +90919,7 @@ init_db();
 init_session_launch();
 init_session_admission2();
 init_notifications2();
-var prodDeps3 = {
+var prodDeps4 = {
   // ADR-0170 · mengamandemen ADR-0169 keputusan #4: `bypassCapacity` DICABUT. Auto-resume kini
   // tunduk cap/beban host seperti "Lanjutkan" manual — mesin 8 GB operator sudah dua kali kernel
   // panic akibat batch sesi paralel tepat SAAT boot (memori mac-mini-8gb-panic-agen-paralel),
@@ -90787,7 +90928,7 @@ var prodDeps3 = {
   recordFail: recordFailure,
   recordDeferred: recordFailure
 };
-async function resumeReconciledSessions(cutoff, deps = prodDeps3) {
+async function resumeReconciledSessions(cutoff, deps = prodDeps4) {
   const report = { resumed: [], failed: [], deferred: [] };
   const specIds = await reconciledSpecIdsSince(cutoff);
   if (!specIds.length) return report;