@cat-factory/app 0.138.1 → 0.139.0

package/app/components/consensus/ConsensusSessionWindow.vue

@@ -10,6 +10,7 @@ import { computed } from 'vue'
 import type { ConsensusContribution, ConsensusSession } from '~/types/consensus'
 import CopyButton from '~/components/common/CopyButton.vue'
 import MarkdownProse from '~/components/common/MarkdownProse.vue'
+import ResultWindowShell from '~/components/panels/ResultWindowShell.vue'
 import { agentKindMeta } from '~/utils/catalog'
 
 const { t, n } = useI18n()
@@ -30,6 +31,24 @@ const session = computed<ConsensusSession | null>(() =>
 )
 const loading = computed(() => (blockId.value ? consensus.isLoading(blockId.value) : false))
 
+// Composed header: "Consensus · <strategy>" as the dialog title (with the block appended when
+// present), and the agent kind + participant count as the subtitle line.
+const headerTitle = computed(() => {
+  const base = session.value
+    ? `${t('consensus.titlePrefix')} · ${strategyLabel(session.value.strategy)}`
+    : t('consensus.titlePrefix')
+  return block.value ? `${base} — ${block.value.title}` : base
+})
+const headerSubtitle = computed(() =>
+  session.value
+    ? `${agentKindMeta(session.value.agentKind).label} · ${t(
+        'consensus.participantCount',
+        { count: session.value.participants.length },
+        session.value.participants.length,
+      )}`
+    : undefined,
+)
+
 // Exhaustive enum→key maps (literal key strings keep the typed-key drift guard live,
 // vs a runtime-built `consensus.strategy.${value}`).
 const STRATEGY_LABEL_KEYS: Record<string, string> = {
@@ -94,168 +113,137 @@ function topScore(c: ConsensusContribution): { label: string; value: number } |
 </script>
 
 <template>
-  <Teleport to="body">
-    <div
-      v-if="open"
-      class="fixed inset-0 z-50 flex max-h-[100dvh] items-center justify-center bg-slate-950/70 p-4 backdrop-blur-sm"
-      @click.self="close"
-    >
-      <div
-        class="flex max-h-[90dvh] w-full max-w-5xl flex-col overflow-hidden rounded-2xl border border-slate-800 bg-slate-900 shadow-2xl"
-        role="dialog"
-        aria-modal="true"
+  <ResultWindowShell
+    :open="open"
+    icon="i-lucide-users-round"
+    icon-class="bg-amber-500/15 text-amber-300"
+    :title="headerTitle"
+    :subtitle="headerSubtitle"
+    variant="centered"
+    width="5xl"
+    @close="close"
+  >
+    <template v-if="session" #header-extras>
+      <span
+        class="rounded-full px-2.5 py-1 text-xs font-medium"
+        :class="STATUS_CLASS[session.status] ?? 'bg-slate-700 text-slate-300'"
       >
-        <!-- header -->
-        <header class="flex items-center gap-3 border-b border-slate-800 px-6 py-4">
-          <div class="flex h-9 w-9 shrink-0 items-center justify-center rounded-lg bg-amber-500/15">
-            <UIcon name="i-lucide-users-round" class="h-5 w-5 text-amber-300" />
-          </div>
-          <div class="min-w-0 flex-1">
-            <h2 class="truncate text-sm font-semibold text-slate-100">
-              {{ t('consensus.titlePrefix') }} ·
-              {{ session ? strategyLabel(session.strategy) : '' }}
-              <span v-if="block" class="font-normal text-slate-400">— {{ block.title }}</span>
-            </h2>
-            <p v-if="session" class="text-xs text-slate-500">
-              {{ agentKindMeta(session.agentKind).label }} ·
-              {{
-                t(
-                  'consensus.participantCount',
-                  { count: session.participants.length },
-                  session.participants.length,
-                )
-              }}
-            </p>
-          </div>
-          <span
-            v-if="session"
-            class="rounded-full px-2.5 py-1 text-xs font-medium"
-            :class="STATUS_CLASS[session.status] ?? 'bg-slate-700 text-slate-300'"
-          >
-            {{ statusLabel(session.status) }}
-          </span>
-          <button
-            class="rounded-lg p-1.5 text-slate-400 hover:bg-slate-800 hover:text-slate-200"
-            @click="close"
-          >
-            <UIcon name="i-lucide-x" class="h-5 w-5" />
-          </button>
-        </header>
+        {{ statusLabel(session.status) }}
+      </span>
+    </template>
 
-        <div class="flex-1 overflow-y-auto px-6 py-5">
-          <div v-if="loading && !session" class="py-16 text-center text-sm text-slate-500">
-            {{ t('consensus.loading') }}
-          </div>
-          <div v-else-if="!session" class="py-16 text-center text-sm text-slate-500">
-            {{ t('consensus.empty') }}
+    <div class="flex-1 overflow-y-auto px-6 py-5">
+      <div v-if="loading && !session" class="py-16 text-center text-sm text-slate-500">
+        {{ t('consensus.loading') }}
+      </div>
+      <div v-else-if="!session" class="py-16 text-center text-sm text-slate-500">
+        {{ t('consensus.empty') }}
+      </div>
+      <template v-else>
+        <!-- failure -->
+        <div
+          v-if="session.status === 'failed'"
+          class="mb-5 flex items-start gap-2 rounded-lg border border-rose-800/60 bg-rose-950/40 px-4 py-3 text-sm text-rose-200"
+        >
+          <span class="min-w-0 flex-1">{{
+            t('consensus.failed', { error: session.error ?? t('consensus.unknownError') })
+          }}</span>
+          <CopyButton v-if="session.error" :text="session.error" class="-me-1 shrink-0" />
+        </div>
+
+        <!-- synthesized result -->
+        <section v-if="session.synthesis" class="mb-6">
+          <div class="mb-2 flex items-center gap-2">
+            <h3 class="text-xs font-semibold uppercase tracking-wide text-slate-400">
+              {{ t('consensus.synthesizedResult') }}
+            </h3>
+            <span
+              v-if="session.confidence != null"
+              class="rounded bg-emerald-500/15 px-1.5 py-0.5 text-xs text-emerald-300"
+              >{{ t('consensus.confidence', { pct: pct(session.confidence) }) }}</span
+            >
+            <CopyButton :text="session.synthesis" class="ms-auto -my-1" />
           </div>
-          <template v-else>
-            <!-- failure -->
+          <MarkdownProse
+            :text="session.synthesis"
+            class="rounded-lg border border-slate-800 bg-slate-950/60 px-4 py-3 text-sm text-slate-200"
+          />
+          <ul v-if="session.dissent?.length" class="mt-2 space-y-1">
+            <li
+              v-for="(d, i) in session.dissent"
+              :key="i"
+              class="flex items-start gap-2 text-xs text-amber-300/90"
+            >
+              <UIcon name="i-lucide-triangle-alert" class="mt-0.5 h-3.5 w-3.5 shrink-0" />
+              <span>{{ d }}</span>
+            </li>
+          </ul>
+        </section>
+
+        <!-- participants -->
+        <section class="mb-6">
+          <h3 class="mb-2 text-xs font-semibold uppercase tracking-wide text-slate-400">
+            {{ t('consensus.panel') }}
+          </h3>
+          <div class="flex flex-wrap gap-2">
             <div
-              v-if="session.status === 'failed'"
-              class="mb-5 flex items-start gap-2 rounded-lg border border-rose-800/60 bg-rose-950/40 px-4 py-3 text-sm text-rose-200"
+              v-for="(p, i) in session.participants"
+              :key="p.id"
+              class="rounded-lg border border-slate-800 bg-slate-950/40 px-3 py-1.5 text-xs"
             >
-              <span class="min-w-0 flex-1">{{
-                t('consensus.failed', { error: session.error ?? t('consensus.unknownError') })
+              <span class="font-medium text-slate-200">{{
+                t('consensus.expert', { letter: String.fromCharCode(65 + i) })
               }}</span>
-              <CopyButton v-if="session.error" :text="session.error" class="-me-1 shrink-0" />
+              <span class="text-slate-400"> · {{ p.role }}</span>
+              <span v-if="p.modelId" class="ms-1 text-slate-500"
+                >({{ models.labelForRef(p.modelId) ?? p.modelId }})</span
+              >
             </div>
+          </div>
+        </section>
 
-            <!-- synthesized result -->
-            <section v-if="session.synthesis" class="mb-6">
-              <div class="mb-2 flex items-center gap-2">
-                <h3 class="text-xs font-semibold uppercase tracking-wide text-slate-400">
-                  {{ t('consensus.synthesizedResult') }}
-                </h3>
+        <!-- rounds -->
+        <section v-for="round in session.rounds" :key="round.index" class="mb-5">
+          <h3 class="mb-2 text-xs font-semibold uppercase tracking-wide text-slate-400">
+            {{ t('consensus.round.heading', { n: round.index + 1 }) }} ·
+            {{ roundLabel(round.kind) }}
+          </h3>
+          <div class="space-y-3">
+            <div
+              v-for="c in round.contributions"
+              :key="c.participantId"
+              class="rounded-lg border border-slate-800 bg-slate-950/40 px-4 py-3"
+            >
+              <div class="mb-1 flex items-center gap-2">
+                <span class="text-xs font-semibold text-slate-200">{{
+                  anonLabel(c.participantId)
+                }}</span>
+                <span class="text-xs text-slate-500">{{ roleFor(c.participantId) }}</span>
                 <span
-                  v-if="session.confidence != null"
-                  class="rounded bg-emerald-500/15 px-1.5 py-0.5 text-xs text-emerald-300"
-                  >{{ t('consensus.confidence', { pct: pct(session.confidence) }) }}</span
-                >
-                <CopyButton :text="session.synthesis" class="ms-auto -my-1" />
-              </div>
-              <MarkdownProse
-                :text="session.synthesis"
-                class="rounded-lg border border-slate-800 bg-slate-950/60 px-4 py-3 text-sm text-slate-200"
-              />
-              <ul v-if="session.dissent?.length" class="mt-2 space-y-1">
-                <li
-                  v-for="(d, i) in session.dissent"
-                  :key="i"
-                  class="flex items-start gap-2 text-xs text-amber-300/90"
-                >
-                  <UIcon name="i-lucide-triangle-alert" class="mt-0.5 h-3.5 w-3.5 shrink-0" />
-                  <span>{{ d }}</span>
-                </li>
-              </ul>
-            </section>
-
-            <!-- participants -->
-            <section class="mb-6">
-              <h3 class="mb-2 text-xs font-semibold uppercase tracking-wide text-slate-400">
-                {{ t('consensus.panel') }}
-              </h3>
-              <div class="flex flex-wrap gap-2">
-                <div
-                  v-for="(p, i) in session.participants"
-                  :key="p.id"
-                  class="rounded-lg border border-slate-800 bg-slate-950/40 px-3 py-1.5 text-xs"
+                  v-if="topScore(c)"
+                  class="ms-auto rounded bg-slate-800 px-1.5 py-0.5 text-xs text-slate-300"
+                  >{{
+                    t('consensus.topScore', {
+                      label: topScore(c)!.label,
+                      pct: pct(topScore(c)!.value),
+                    })
+                  }}</span
                 >
-                  <span class="font-medium text-slate-200">{{
-                    t('consensus.expert', { letter: String.fromCharCode(65 + i) })
-                  }}</span>
-                  <span class="text-slate-400"> · {{ p.role }}</span>
-                  <span v-if="p.modelId" class="ms-1 text-slate-500"
-                    >({{ models.labelForRef(p.modelId) ?? p.modelId }})</span
-                  >
-                </div>
+                <CopyButton :text="c.text" :class="topScore(c) ? '-my-1' : 'ms-auto -my-1'" />
               </div>
-            </section>
-
-            <!-- rounds -->
-            <section v-for="round in session.rounds" :key="round.index" class="mb-5">
-              <h3 class="mb-2 text-xs font-semibold uppercase tracking-wide text-slate-400">
-                {{ t('consensus.round.heading', { n: round.index + 1 }) }} ·
-                {{ roundLabel(round.kind) }}
-              </h3>
-              <div class="space-y-3">
-                <div
-                  v-for="c in round.contributions"
-                  :key="c.participantId"
-                  class="rounded-lg border border-slate-800 bg-slate-950/40 px-4 py-3"
+              <MarkdownProse :text="c.text" class="text-sm text-slate-300" />
+              <div v-if="c.scores?.length" class="mt-2 flex flex-wrap gap-1.5">
+                <span
+                  v-for="s in c.scores"
+                  :key="s.dimension"
+                  class="rounded bg-slate-800/80 px-1.5 py-0.5 text-xs text-slate-400"
+                  >{{ s.dimension }}: {{ pct(s.value) }}</span
                 >
-                  <div class="mb-1 flex items-center gap-2">
-                    <span class="text-xs font-semibold text-slate-200">{{
-                      anonLabel(c.participantId)
-                    }}</span>
-                    <span class="text-xs text-slate-500">{{ roleFor(c.participantId) }}</span>
-                    <span
-                      v-if="topScore(c)"
-                      class="ms-auto rounded bg-slate-800 px-1.5 py-0.5 text-xs text-slate-300"
-                      >{{
-                        t('consensus.topScore', {
-                          label: topScore(c)!.label,
-                          pct: pct(topScore(c)!.value),
-                        })
-                      }}</span
-                    >
-                    <CopyButton :text="c.text" :class="topScore(c) ? '-my-1' : 'ms-auto -my-1'" />
-                  </div>
-                  <MarkdownProse :text="c.text" class="text-sm text-slate-300" />
-                  <div v-if="c.scores?.length" class="mt-2 flex flex-wrap gap-1.5">
-                    <span
-                      v-for="s in c.scores"
-                      :key="s.dimension"
-                      class="rounded bg-slate-800/80 px-1.5 py-0.5 text-xs text-slate-400"
-                      >{{ s.dimension }}: {{ pct(s.value) }}</span
-                    >
-                  </div>
-                </div>
               </div>
-            </section>
-          </template>
-        </div>
-      </div>
+            </div>
+          </div>
+        </section>
+      </template>
     </div>
-  </Teleport>
+  </ResultWindowShell>
 </template>

package/app/components/docs/DocInterviewWindow.vue

@@ -8,6 +8,7 @@
 // `doc-interviewer` step's result view. Live `docInterview` stream events patch the store, so an
 // open window follows the interview as it progresses. Mirrors InitiativePlanningWindow.vue.
 import { computed, reactive, watch } from 'vue'
+import ResultWindowShell from '~/components/panels/ResultWindowShell.vue'
 
 const board = useBoardStore()
 const docInterview = useDocInterviewStore()
@@ -67,147 +68,118 @@ const onProceed = () => flushThen((id) => docInterview.proceedInterview(id))
 </script>
 
 <template>
-  <Teleport to="body">
-    <div
-      v-if="open"
-      class="fixed inset-0 z-50 flex max-h-[100dvh] items-stretch justify-center bg-slate-950/70 backdrop-blur-sm"
-      @click.self="close"
-    >
+  <ResultWindowShell
+    :open="open"
+    icon="i-lucide-messages-square"
+    icon-class="bg-indigo-500/15 text-indigo-300"
+    :title="block?.title ?? t('docInterview.title')"
+    :subtitle="t('docInterview.subtitle')"
+    width="3xl"
+    testid="doc-interview-window"
+    @close="close"
+  >
+    <template v-if="session" #header-extras>
+      <UBadge :color="converged ? 'success' : 'primary'" variant="subtle" size="sm">
+        {{ t(converged ? 'docInterview.status.done' : 'docInterview.status.awaiting') }}
+      </UBadge>
+    </template>
+
+    <div class="min-h-0 flex-1 overflow-y-auto px-5 py-4">
+      <!-- No session yet -->
       <div
-        class="m-4 flex w-full max-w-3xl flex-col overflow-hidden rounded-2xl border border-slate-800 bg-slate-900 shadow-2xl"
-        role="dialog"
-        aria-modal="true"
-        data-testid="doc-interview-window"
+        v-if="!session"
+        class="flex h-full flex-col items-center justify-center gap-2 text-center text-slate-400"
       >
-        <!-- Header -->
-        <header class="flex items-center gap-3 border-b border-slate-800 px-5 py-3">
-          <span
-            class="flex h-8 w-8 items-center justify-center rounded-lg bg-indigo-500/15 text-indigo-300"
-          >
-            <UIcon name="i-lucide-messages-square" class="h-4 w-4" />
-          </span>
-          <div class="min-w-0 flex-1">
-            <h2 class="truncate text-sm font-semibold text-slate-100">
-              {{ block?.title ?? t('docInterview.title') }}
-            </h2>
-            <p class="truncate text-[11px] text-slate-400">
-              {{ t('docInterview.subtitle') }}
-            </p>
-          </div>
-          <UBadge
-            v-if="session"
-            :color="converged ? 'success' : 'primary'"
-            variant="subtle"
-            size="sm"
-          >
-            {{ t(converged ? 'docInterview.status.done' : 'docInterview.status.awaiting') }}
-          </UBadge>
-          <button
-            class="rounded-md p-1.5 text-slate-400 hover:bg-slate-800 hover:text-slate-200"
-            @click="close"
-          >
-            <UIcon name="i-lucide-x" class="h-4 w-4" />
-          </button>
-        </header>
-
-        <div class="min-h-0 flex-1 overflow-y-auto px-5 py-4">
-          <!-- No session yet -->
-          <div
-            v-if="!session"
-            class="flex h-full flex-col items-center justify-center gap-2 text-center text-slate-400"
+        <UIcon name="i-lucide-messages-square" class="h-8 w-8 opacity-40" />
+        <p class="text-sm">{{ t('docInterview.empty') }}</p>
+      </div>
+
+      <template v-else>
+        <p class="mb-4 text-[13px] leading-relaxed text-slate-300">
+          {{ t('docInterview.intro') }}
+        </p>
+
+        <!-- Converged: show the synthesized authoring brief -->
+        <div
+          v-if="converged"
+          class="rounded-lg border border-slate-800 bg-slate-950/40 p-4"
+          data-testid="doc-interview-converged"
+        >
+          <p class="mb-2 text-[11px] font-medium uppercase tracking-wide text-slate-500">
+            {{ t('docInterview.brief') }}
+          </p>
+          <pre
+            v-if="session.brief"
+            class="whitespace-pre-wrap break-words text-[13px] leading-relaxed text-slate-300"
+            >{{ session.brief }}</pre
           >
-            <UIcon name="i-lucide-messages-square" class="h-8 w-8 opacity-40" />
-            <p class="text-sm">{{ t('docInterview.empty') }}</p>
-          </div>
-
-          <template v-else>
-            <p class="mb-4 text-[13px] leading-relaxed text-slate-300">
-              {{ t('docInterview.intro') }}
-            </p>
-
-            <!-- Converged: show the synthesized authoring brief -->
-            <div
-              v-if="converged"
-              class="rounded-lg border border-slate-800 bg-slate-950/40 p-4"
-              data-testid="doc-interview-converged"
-            >
-              <p class="mb-2 text-[11px] font-medium uppercase tracking-wide text-slate-500">
-                {{ t('docInterview.brief') }}
-              </p>
-              <pre
-                v-if="session.brief"
-                class="whitespace-pre-wrap break-words text-[13px] leading-relaxed text-slate-300"
-                >{{ session.brief }}</pre
-              >
-              <p v-else class="text-[13px] text-slate-400">{{ t('docInterview.converged') }}</p>
-            </div>
-
-            <!-- No pending questions but not yet converged -->
-            <div
-              v-else-if="questions.length === 0"
-              class="rounded-lg border border-slate-800 bg-slate-950/40 p-4 text-center text-[13px] text-slate-400"
-            >
-              {{ t('docInterview.converged') }}
-            </div>
-
-            <!-- Interview questions -->
-            <ul v-else class="space-y-4">
-              <li
-                v-for="q in questions"
-                :key="q.key"
-                class="rounded-lg border border-slate-800 bg-slate-950/40 p-3"
-                data-testid="doc-interview-question"
-              >
-                <p class="mb-2 text-[13px] font-medium text-slate-200">{{ q.question }}</p>
-                <UTextarea
-                  v-model="drafts[q.key]"
-                  :rows="2"
-                  autoresize
-                  :placeholder="t('docInterview.answerPlaceholder')"
-                  class="w-full"
-                  data-testid="doc-interview-answer"
-                  @blur="persist(q)"
-                />
-              </li>
-            </ul>
-          </template>
+          <p v-else class="text-[13px] text-slate-400">{{ t('docInterview.converged') }}</p>
         </div>
 
-        <!-- Action rail -->
-        <footer
-          v-if="session && !converged && questions.length > 0"
-          class="flex items-center justify-between gap-3 border-t border-slate-800 px-5 py-3"
+        <!-- No pending questions but not yet converged -->
+        <div
+          v-else-if="questions.length === 0"
+          class="rounded-lg border border-slate-800 bg-slate-950/40 p-4 text-center text-[13px] text-slate-400"
         >
-          <p class="text-[11px] text-slate-500">
-            {{ t('docInterview.hint') }}
-          </p>
-          <div class="flex items-center gap-2">
-            <UButton
-              color="neutral"
-              variant="ghost"
-              size="sm"
-              :loading="resuming"
-              :disabled="!access.canExecuteRuns.value"
-              :title="access.canExecuteRuns.value ? undefined : t('access.noRunExecute')"
-              data-testid="doc-interview-proceed"
-              @click="onProceed"
-            >
-              {{ t('docInterview.proceed') }}
-            </UButton>
-            <UButton
-              color="primary"
-              size="sm"
-              :loading="resuming"
-              :disabled="!allAnswered || !access.canExecuteRuns.value"
-              :title="access.canExecuteRuns.value ? undefined : t('access.noRunExecute')"
-              data-testid="doc-interview-continue"
-              @click="onContinue"
-            >
-              {{ t('docInterview.continue') }}
-            </UButton>
-          </div>
-        </footer>
-      </div>
+          {{ t('docInterview.converged') }}
+        </div>
+
+        <!-- Interview questions -->
+        <ul v-else class="space-y-4">
+          <li
+            v-for="q in questions"
+            :key="q.key"
+            class="rounded-lg border border-slate-800 bg-slate-950/40 p-3"
+            data-testid="doc-interview-question"
+          >
+            <p class="mb-2 text-[13px] font-medium text-slate-200">{{ q.question }}</p>
+            <UTextarea
+              v-model="drafts[q.key]"
+              :rows="2"
+              autoresize
+              :placeholder="t('docInterview.answerPlaceholder')"
+              class="w-full"
+              data-testid="doc-interview-answer"
+              @blur="persist(q)"
+            />
+          </li>
+        </ul>
+      </template>
     </div>
-  </Teleport>
+
+    <!-- Action rail -->
+    <footer
+      v-if="session && !converged && questions.length > 0"
+      class="flex items-center justify-between gap-3 border-t border-slate-800 px-5 py-3"
+    >
+      <p class="text-[11px] text-slate-500">
+        {{ t('docInterview.hint') }}
+      </p>
+      <div class="flex items-center gap-2">
+        <UButton
+          color="neutral"
+          variant="ghost"
+          size="sm"
+          :loading="resuming"
+          :disabled="!access.canExecuteRuns.value"
+          :title="access.canExecuteRuns.value ? undefined : t('access.noRunExecute')"
+          data-testid="doc-interview-proceed"
+          @click="onProceed"
+        >
+          {{ t('docInterview.proceed') }}
+        </UButton>
+        <UButton
+          color="primary"
+          size="sm"
+          :loading="resuming"
+          :disabled="!allAnswered || !access.canExecuteRuns.value"
+          :title="access.canExecuteRuns.value ? undefined : t('access.noRunExecute')"
+          data-testid="doc-interview-continue"
+          @click="onContinue"
+        >
+          {{ t('docInterview.continue') }}
+        </UButton>
+      </div>
+    </footer>
+  </ResultWindowShell>
 </template>

package/app/components/followUp/FollowUpWindow.vue

@@ -21,11 +21,9 @@ const access = useWorkspaceAccess()
 
 const { t } = useI18n()
 
-// `manageEscape: false` — `ResultWindowShell` owns Escape (and focus trap + scroll lock +
-// stacking) via the shared overlay behaviour.
-const { open, blockId, instanceId, stepIndex, close } = useResultView('follow-ups', {
-  manageEscape: false,
-})
+// `ResultWindowShell` owns Escape (and focus trap + scroll lock + stacking) via the shared
+// overlay behaviour.
+const { open, blockId, instanceId, stepIndex, close } = useResultView('follow-ups')
 
 const block = computed(() => (blockId.value ? board.getBlock(blockId.value) : undefined))
 const headerTitle = computed(() =>

package/app/components/forkDecision/ForkDecisionWindow.vue

@@ -24,11 +24,9 @@ const access = useWorkspaceAccess()
 const { t } = useI18n()
 
 // Hybrid: state rides the coder step (like follow-ups), but warm it from the GET on open too.
-// `manageEscape: false` — `ResultWindowShell` owns Escape (and focus trap + scroll lock +
-// stacking). No `stepRef`: this is a pre-run decision, so there's no "restart from here".
+// No `stepRef`: this is a pre-run decision, so there's no "restart from here".
 const { open, blockId, instanceId, stepIndex, close } = useResultView('fork-decision', {
   onOpen: (id) => void forkDecision.load(id),
-  manageEscape: false,
 })
 
 const block = computed(() => (blockId.value ? board.getBlock(blockId.value) : undefined))

package/app/components/gates/GateResultView.vue

@@ -20,11 +20,9 @@ const { t } = useI18n()
 const access = useWorkspaceAccess()
 
 // Synchronous window: it reads its state straight off the execution step, so there's
-// nothing to fetch on open (no `onOpen` loader). `manageEscape: false` — `ResultWindowShell`
-// owns Escape (and focus trap + scroll lock + stacking).
-const { open, blockId, instanceId, stepIndex, close } = useResultView('gate', {
-  manageEscape: false,
-})
+// nothing to fetch on open (no `onOpen` loader). `ResultWindowShell` owns Escape (and focus
+// trap + scroll lock + stacking).
+const { open, blockId, instanceId, stepIndex, close } = useResultView('gate')
 const block = computed(() => (blockId.value ? board.getBlock(blockId.value) : undefined))
 const prUrl = computed(() => block.value?.pullRequest?.url ?? null)
 const headerTitle = computed(

package/app/components/humanTest/HumanTestWindow.vue

@@ -25,11 +25,9 @@ const access = useWorkspaceAccess()
 const { confirmAction, toastDone } = useConfirmAction()
 
 // Shared seam contract (open/blockId/close). No `onOpen` loader: the gate state rides on the
-// execution step, pushed over the stream. `manageEscape: false` — `ResultWindowShell` owns
-// Escape (and focus trap + scroll lock + stacking).
-const { open, blockId, instanceId, stepIndex, close } = useResultView('human-test', {
-  manageEscape: false,
-})
+// execution step, pushed over the stream. `ResultWindowShell` owns Escape (and focus trap +
+// scroll lock + stacking).
+const { open, blockId, instanceId, stepIndex, close } = useResultView('human-test')
 const block = computed(() => (blockId.value ? board.getBlock(blockId.value) : undefined))
 const headerTitle = computed(() =>
   block.value ? t('humanTest.titleWithBlock', { title: block.value.title }) : t('humanTest.title'),