@cat-factory/app 0.228.0 → 0.228.1

package/app/components/board/AddTaskModal.vue

@@ -59,12 +59,7 @@ const { t } = useI18n()
 
 const { linkPending, presentLinkFailures } = useContextLinking()
 
-const open = computed({
-  get: () => ui.addTaskContainerId !== null,
-  set: (v: boolean) => {
-    if (!v) void requestClose()
-  },
-})
+const open = computed(() => ui.addTaskContainerId !== null)
 
 const container = computed(() =>
   ui.addTaskContainerId ? board.getBlock(ui.addTaskContainerId) : undefined,
@@ -591,6 +586,15 @@ const { requestClose } = useUnsavedGuard({
   }),
 })
 
+// The template's v-model binding: dismissal (Escape / backdrop) routes through the guard.
+// Declared after the guard so the setter's `requestClose` reference is never in its TDZ.
+const modalOpen = computed({
+  get: () => open.value,
+  set: (v: boolean) => {
+    if (!v) void requestClose()
+  },
+})
+
 // A recurring task only needs a target frame (its details are filled in the schedule
 // modal); every other type needs a title. A review task additionally needs a target PR.
 // The Ralph loop's completion criterion (its `ralph.validationCommand` agent-config id). The
@@ -757,7 +761,7 @@ function openReviewFrictionDialog(conflict: NonNullable<ReturnType<typeof parseC
 </script>
 
 <template>
-  <UModal v-model:open="open" :title="t('board.addTask.title')">
+  <UModal v-model:open="modalOpen" :title="t('board.addTask.title')">
     <template #body>
       <div class="space-y-4" data-testid="add-task-modal">
         <p v-if="container" class="text-xs text-slate-400">

package/app/components/board/RecurringPipelineModal.vue

@@ -22,12 +22,7 @@ const toast = useToast()
 const access = useWorkspaceAccess()
 const { t, te } = useI18n()
 
-const open = computed({
-  get: () => ui.addRecurringFrameId !== null,
-  set: (v: boolean) => {
-    if (!v) void requestClose()
-  },
-})
+const open = computed(() => ui.addRecurringFrameId !== null)
 
 const frame = computed(() =>
   ui.addRecurringFrameId ? board.getBlock(ui.addRecurringFrameId) : undefined,
@@ -240,6 +235,15 @@ const { requestClose } = useUnsavedGuard({
   }),
 })
 
+// The template's v-model binding: dismissal (Escape / backdrop) routes through the guard.
+// Declared after the guard so the setter's `requestClose` reference is never in its TDZ.
+const modalOpen = computed({
+  get: () => open.value,
+  set: (v: boolean) => {
+    if (!v) void requestClose()
+  },
+})
+
 // The board field required for the picked source must be filled before a bug-intake schedule saves.
 const intakeReady = computed(() => {
   if (!showIntake.value) return true
@@ -339,7 +343,7 @@ async function add() {
 </script>
 
 <template>
-  <UModal v-model:open="open" :title="t('board.recurring.title')">
+  <UModal v-model:open="modalOpen" :title="t('board.recurring.title')">
     <template #body>
       <div class="space-y-4">
         <p v-if="frame" class="text-xs text-slate-400">

package/app/components/bootstrap/BootstrapModal.vue

@@ -18,12 +18,7 @@ const { freeFramePosition, focusFrame } = useFramePlacement()
 const { t } = useI18n()
 const { confirmAction, toastDone } = useConfirmAction()
 
-const open = computed({
-  get: () => ui.bootstrapOpen,
-  set: (v: boolean) => {
-    if (!v) void requestClose()
-  },
-})
+const open = computed(() => ui.bootstrapOpen)
 
 // Load the workspace's reference architectures + recent jobs, plus (best-effort)
 // the GitHub repos the user can access so the base form can pick from them.
@@ -100,6 +95,15 @@ const { requestClose } = useUnsavedGuard({
   }),
 })
 
+// The template's v-model binding: dismissal (Escape / backdrop) routes through the guard.
+// Declared after the guard so the setter's `requestClose` reference is never in its TDZ.
+const modalOpen = computed({
+  get: () => open.value,
+  set: (v: boolean) => {
+    if (!v) void requestClose()
+  },
+})
+
 // Mirror of the backend `slugField` rule (@cat-factory/contracts bootstrap
 // schema): the new repo name is a SINGLE GitHub name segment — no "owner/"
 // prefix — so reject a bad value inline before we hit the API. Kept in sync with
@@ -420,7 +424,7 @@ const statusLabel = computed<Record<BootstrapStatus, string>>(() => ({
 </script>
 
 <template>
-  <UModal v-model:open="open" :title="t('bootstrap.title')" :ui="{ content: 'max-w-2xl' }">
+  <UModal v-model:open="modalOpen" :title="t('bootstrap.title')" :ui="{ content: 'max-w-2xl' }">
     <template #body>
       <div class="space-y-6">
         <!-- Three states, because each promises the user something different about the repo.

package/app/components/documents/RepoContextDocPicker.vue

@@ -75,6 +75,10 @@ watch(selectedRepoId, (id) => {
   if (found) selectedRepo.value = found
 })
 
+// The file-search query (its matches live in the file-selection section below). Declared
+// above `clearRepo` and the repo-switch watcher, which both reset it.
+const fileQuery = ref('')
+
 function clearRepo() {
   selectedRepoId.value = undefined
   selectedRepo.value = undefined
@@ -127,7 +131,6 @@ watch(selectedRepoId, (id) => {
   if (id !== undefined) void ensureFilesLoaded()
 })
 
-const fileQuery = ref('')
 // Matches are computed client-side from the cached tree (no per-keystroke server call).
 // A query is required so a large repo never renders thousands of rows at once; results
 // are capped for the same reason.

package/app/components/fragments/FragmentLibraryManager.vue

@@ -171,6 +171,19 @@ const editDraft = ref<{
   brief: string
   tags: string
 } | null>(null)
+// The linked SHORT VERSION is an OVERRIDE of what the platform does by default (condense a
+// long standard automatically, fold a short one in full), so it follows the override rule:
+// hidden in basic mode while unset, revealed as soon as the fragment carries one — a
+// basic-mode curator is never left unable to see or clear a brief a teammate linked.
+//
+// Both flags are LATCHED at the moment the form opens rather than tracking the live draft.
+// Recomputing per keystroke makes the control delete itself the instant a basic-mode curator
+// empties it — mid-edit, under the cursor, on the one interaction (clearing, to hand the
+// standard back to auto-generation) the rule exists to keep reachable.
+const uiMode = useUiModeStore()
+const showEditBrief = ref(false)
+const showDraftBrief = computed(() => showOverrideField(uiMode.isAdvanced, null))
+
 function startEdit(f: (typeof library.fragments)[number]) {
   editDraft.value = {
     id: f.id,
@@ -192,19 +205,6 @@ const editValid = computed(
     !!editDraft.value.summary.trim() &&
     !!editDraft.value.body.trim(),
 )
-// The linked SHORT VERSION is an OVERRIDE of what the platform does by default (condense a
-// long standard automatically, fold a short one in full), so it follows the override rule:
-// hidden in basic mode while unset, revealed as soon as the fragment carries one — a
-// basic-mode curator is never left unable to see or clear a brief a teammate linked.
-//
-// Both flags are LATCHED at the moment the form opens rather than tracking the live draft.
-// Recomputing per keystroke makes the control delete itself the instant a basic-mode curator
-// empties it — mid-edit, under the cursor, on the one interaction (clearing, to hand the
-// standard back to auto-generation) the rule exists to keep reachable.
-const uiMode = useUiModeStore()
-const showEditBrief = ref(false)
-const showDraftBrief = computed(() => showOverrideField(uiMode.isAdvanced, null))
-
 async function saveEdit() {
   const d = editDraft.value
   if (!d || !editValid.value) return
@@ -277,11 +277,6 @@ async function removeFragment(id: string) {
 // Link a Confluence/Notion page or GitHub file as a fragment that is re-resolved
 // from the source at run time (a living source of truth, not a frozen snapshot).
 const docDraft = ref({ source: '' as DocumentSourceKind | '', ref: '', tags: '' })
-const docDraftValid = computed(() => {
-  if (docLinkDisabled.value || !docDraft.value.source) return false
-  // The GitHub picker validates on staged files; every other path on the free-text ref.
-  return usingDocPicker.value ? docFilePaths.value.length > 0 : !!docDraft.value.ref.trim()
-})
 
 // ---- GitHub file picker (documents tab) -----------------------------------
 // For a GitHub source, let the user search a repo + browse to one or MORE files
@@ -364,6 +359,12 @@ const stagedDocRefs = computed(() =>
 /** When the rich picker drives the ref(s); otherwise the free-text field does. */
 const usingDocPicker = computed(() => showGithubDocPicker.value)
 
+const docDraftValid = computed(() => {
+  if (docLinkDisabled.value || !docDraft.value.source) return false
+  // The GitHub picker validates on staged files; every other path on the free-text ref.
+  return usingDocPicker.value ? docFilePaths.value.length > 0 : !!docDraft.value.ref.trim()
+})
+
 /**
  * A pasted GitHub file/directory URL resolved to a repo + location: select the repo
  * (through the same refs the search select drives), then stage the file or jump the

package/app/components/gates/GateResultView.vue

@@ -25,9 +25,6 @@ const access = useWorkspaceAccess()
 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(
-  () => `${meta.value.label}${block.value ? ` — ${block.value.title}` : ''}`,
-)
 
 const instance = computed(() =>
   instanceId.value === null ? null : (execution.getInstance(instanceId.value) ?? null),
@@ -42,6 +39,9 @@ const isCi = computed(() => step.value?.agentKind === 'ci')
 const isHumanReview = computed(() => step.value?.agentKind === 'human-review')
 const isDocQuality = computed(() => step.value?.agentKind === 'doc-quality')
 const meta = computed(() => agentKindMeta(step.value?.agentKind ?? 'ci'))
+const headerTitle = computed(
+  () => `${meta.value.label}${block.value ? ` — ${block.value.title}` : ''}`,
+)
 const helperKind = computed(() =>
   isHumanReview.value
     ? 'fixer'

package/app/components/github/AddServiceFromRepoModal.vue

@@ -189,6 +189,16 @@ function removeSelected(path: string) {
   if (i >= 0) selectedDirectories.value.splice(i, 1)
 }
 
+// The just-added whole-repo service, kept on the board store so the user can configure it
+// (test infra + fragments) right here — the same controls as the inspector. Only the
+// whole-repo flow surfaces this inline configure step; a monorepo adds several services at
+// once and they're configured later in the inspector. Declared above the watcher and
+// `resetSelection` below, both of which clear it.
+const configuredBlockId = ref<string | undefined>(undefined)
+const configuredBlock = computed(() =>
+  configuredBlockId.value ? board.getBlock(configuredBlockId.value) : undefined,
+)
+
 // On repo change, capture the picked repo (from the volatile loaded list, before a later
 // search replaces it), seed the monorepo toggle from its persisted flag, and clear the rest.
 watch(selectedRepoId, (id) => {
@@ -227,15 +237,6 @@ function openManageInstall() {
   if (manageInstallUrl.value) window.open(manageInstallUrl.value, '_blank', 'noopener')
 }
 
-// The just-added whole-repo service, kept on the board store so the user can configure it
-// (test infra + fragments) right here — the same controls as the inspector. Only the
-// whole-repo flow surfaces this inline configure step; a monorepo adds several services at
-// once and they're configured later in the inspector.
-const configuredBlockId = ref<string | undefined>(undefined)
-const configuredBlock = computed(() =>
-  configuredBlockId.value ? board.getBlock(configuredBlockId.value) : undefined,
-)
-
 // On open: ensure we know the connection + which repos the App can access, and
 // the workspace's already-tracked repos (to flag ones already on the board).
 // Declared after every ref resetSelection() touches so the `immediate` run

package/app/components/panels/AgentStepDetail.vue

@@ -200,15 +200,10 @@ function openDedicatedWindow() {
   else if (park === 'fork-decision') ui.openForkDecision(c.instanceId, c.stepIndex)
 }
 
-function close() {
-  // Reset the approval-mode sub-states so reopening the same step is clean
-  // (the step-change watch only fires when the step key actually changes).
-  approval.resetForClose()
-  ui.closeStepDetail()
-}
-
 // The GitHub-style approval/review state machine for a pending gate step. A park a
 // dedicated window owns is NOT reviewable here, so it doesn't count as pending.
+// (`close` is passed by hoisted function reference; it's declared below `approval`,
+// which it resets.)
 const approval = useStepApproval({
   step: () => step.value,
   scrollEl: () => scrollEl.value,
@@ -246,6 +241,13 @@ const {
   reject,
 } = approval
 
+function close() {
+  // Reset the approval-mode sub-states so reopening the same step is clean
+  // (the step-change watch only fires when the step key actually changes).
+  approval.resetForClose()
+  ui.closeStepDetail()
+}
+
 /**
  * Why the gate refuses this viewer, worded for them. An exhaustive Record over the shared refusal
  * vocabulary with LITERAL keys, so the typed-message-key check sees them and a new refusal reason

package/app/components/panels/InspectorPanel.vue

@@ -20,10 +20,6 @@ const requirements = useRequirementsStore()
 const access = useWorkspaceAccess()
 const { t } = useI18n()
 
-// When the selected task block backs a recurring pipeline, the inspector shows the
-// schedule controls + history, and "Delete" removes the schedule (block + history).
-const schedule = computed(() => (block.value ? recurring.byBlock(block.value.id) : undefined))
-
 onMounted(() => {
   fragments.ensureLoaded()
   github.ensureLoaded()
@@ -32,6 +28,10 @@ onMounted(() => {
 const block = computed<Block | undefined>(() =>
   ui.selectedBlockId ? board.getBlock(ui.selectedBlockId) : undefined,
 )
+
+// When the selected task block backs a recurring pipeline, the inspector shows the
+// schedule controls + history, and "Delete" removes the schedule (block + history).
+const schedule = computed(() => (block.value ? recurring.byBlock(block.value.id) : undefined))
 const level = computed(() => block.value?.level ?? 'frame')
 const isFrame = computed(() => level.value === 'frame')
 

package/app/components/panels/inspector/ServiceTestConfig.vue

@@ -190,6 +190,18 @@ function rootUnderDirectory(directory: string | null | undefined, path: string):
   return segs.join('/')
 }
 
+// The repo + service subdirectory backing this frame (the manifest prefills, the fixer
+// run, and the compose-file browser all root paths under it). A monorepo service isn't
+// on the `github_repos` blockId link (that stays null), so fall back to the service
+// catalog mapping, which carries the repo + directory.
+const repoContext = computed<{ githubId: number; directory?: string | null } | undefined>(() => {
+  if (props.repo) return props.repo
+  const svc = services.serviceByFrameBlock[props.block.id]
+  if (svc?.repoGithubId != null) return { githubId: svc.repoGithubId, directory: svc.directory }
+  const r = github.repoForBlock(props.block.id)
+  return r ? { githubId: r.githubId } : undefined
+})
+
 function setCustomManifestId(value: string) {
   // Prefill the manifest path with the selected type's default, rooted under the service subtree
   // (repo-root-relative, editable afterwards) so a monorepo service targets the right location
@@ -366,17 +378,6 @@ async function stopEnvTest() {
 // ephemeral-environment provisioner, not commonly tuned — keep them collapsed by default.
 const showProvisioning = ref(false)
 
-// The repo + service subdirectory backing this frame, for the compose-file browser.
-// A monorepo service isn't on the `github_repos` blockId link (that stays null), so
-// fall back to the service catalog mapping, which carries the repo + directory.
-const repoContext = computed<{ githubId: number; directory?: string | null } | undefined>(() => {
-  if (props.repo) return props.repo
-  const svc = services.serviceByFrameBlock[props.block.id]
-  if (svc?.repoGithubId != null) return { githubId: svc.repoGithubId, directory: svc.directory }
-  const r = github.repoForBlock(props.block.id)
-  return r ? { githubId: r.githubId } : undefined
-})
-
 // Repo-path picker, shared by the compose file (`docker compose -f <path>`) and the
 // kubernetes colocated manifests path. The stored path is relative to the repo root (the
 // browser starts inside the service's subdirectory for convenience).

package/app/components/panels/inspector/TaskExecution.vue

@@ -56,6 +56,13 @@ const instance = computed(() => execution.getInstance(props.block.executionId))
 // is that nothing will merge, and WHY is answered by the run's own notes and the merge decision.
 const sandboxed = computed(() => isDryRun(instance.value?.mode))
 
+// A failed pipeline run surfaces the shared failure banner + retry — the
+// execution failure surface that the old `pr_ready` flip used to hide.
+const failedRun = computed(() => {
+  const run = agentRuns.byBlock[props.block.id]
+  return run && run.status === 'failed' ? run : null
+})
+
 // Nothing to show yet: no run, no failed run, no PR, and not awaiting a merge — render an
 // empty state instead of a blank gap so the section reads as "no runs yet" rather than broken.
 const isEmpty = computed(
@@ -77,13 +84,6 @@ const runFailed = computed(() => instance.value?.status === 'failed')
  */
 const inputGateNotice = computed(() => inputGateNoticeFor(instance.value))
 
-// A failed pipeline run surfaces the shared failure banner + retry — the
-// execution failure surface that the old `pr_ready` flip used to hide.
-const failedRun = computed(() => {
-  const run = agentRuns.byBlock[props.block.id]
-  return run && run.status === 'failed' ? run : null
-})
-
 // Failures from prior attempts, preserved across retries — shown regardless of the run's
 // CURRENT status, so the error trail stays viewable after a restart clears the top banner.
 const failureHistory = computed(() => agentRuns.byBlock[props.block.id]?.failureHistory ?? [])

package/app/components/pipeline/PipelineProgress.vue

@@ -160,6 +160,11 @@ const STATUS_META = computed<Record<ExecutionInstance['status'], { label: string
 const steps = computed(() => props.instance.steps)
 const total = computed(() => steps.value.length)
 
+// A failed run is no longer executing: a step left mid-flight (state still `working`,
+// its container caught mid cold-boot) must stop looking live — no spinner, no pulse,
+// no "spinning up container" phase.
+const runFailed = computed(() => props.instance.status === 'failed')
+
 // A shared 1s tick drives every step's live elapsed clock, so a step that hasn't yet
 // emitted subtask counts still shows it is progressing rather than reading as hung.
 const nowTick = useNowTick()
@@ -172,10 +177,6 @@ function stepElapsed(s: PipelineStep): string | null {
 // human can see at a glance whether the fixer ran or was skipped.
 const companionByStep = computed(() => steps.value.map((s) => gateCompanionFor(s, runFailed.value)))
 
-// A failed run is no longer executing: a step left mid-flight (state still `working`,
-// its container caught mid cold-boot) must stop looking live — no spinner, no pulse,
-// no "spinning up container" phase.
-const runFailed = computed(() => props.instance.status === 'failed')
 /**
  * A reviewer gate (requirements-review / clarity-review) folding the answers or
  * re-reviewing in the durable driver: the step parks in `waiting_decision` but is actively

package/app/components/providers/ApiKeysSection.vue

@@ -33,6 +33,9 @@ const { confirmAction, toastDone } = useConfirmAction()
 /** Account-wide mode (single account scope) vs the default workspace/user toggle. */
 const isAccount = computed(() => !!props.accountId)
 
+/** Which store the form writes to: the shared workspace keys or the user's own. */
+const scope = ref<'workspace' | 'user'>('workspace')
+
 // "My keys" (user scope) are stored per-user, so they need a signed-in user. Block just
 // that scope when there's none (a deployment without sign-in); workspace/account keys are
 // unaffected. The scope toggle stays enabled so the user can switch back to a shared scope.
@@ -136,7 +139,6 @@ const PROVIDERS = computed(() =>
 )
 const ALL_PROVIDERS = computed(() => [...DIRECT_PROVIDERS.value, ...PROXY_PROVIDERS.value])
 
-const scope = ref<'workspace' | 'user'>('workspace')
 const provider = ref<ApiKeyProvider>(props.category === 'proxy' ? 'openrouter' : 'openai')
 const label = ref('')
 const key = ref('')

package/app/components/ralph/RalphLoopResultView.vue

@@ -19,9 +19,6 @@ const { t, d } = useI18n()
 // overlay behaviour.
 const { open, blockId, instanceId, stepIndex, close } = useResultView('ralph-loop')
 const block = computed(() => (blockId.value ? board.getBlock(blockId.value) : undefined))
-const headerTitle = computed(
-  () => `${meta.value.label}${block.value ? ` — ${block.value.title}` : ''}`,
-)
 const prUrl = computed(() => block.value?.pullRequest?.url ?? null)
 
 const instance = computed(() =>
@@ -33,6 +30,9 @@ const step = computed(() => {
 })
 const ralph = computed<RalphStepState | null>(() => step.value?.ralph ?? null)
 const meta = computed(() => agentKindMeta('ralph'))
+const headerTitle = computed(
+  () => `${meta.value.label}${block.value ? ` — ${block.value.title}` : ''}`,
+)
 
 // Iterations, newest-first for the timeline.
 const attempts = computed(() => [...(ralph.value?.attemptLog ?? [])].reverse())

package/app/components/visualConfirm/VisualConfirmationWindow.vue

@@ -131,6 +131,20 @@ function buildFindings(): { text: string; structured: { view?: string; note: str
   return { text: blocks.join('\n\n'), structured }
 }
 
+// Degraded-basis approval guard (no capture / a fix landed after these shots): require an
+// explicit "I reviewed this another way" acknowledgement before the one-click approve.
+const ackDegraded = ref(false)
+watch(
+  () => vc.value?.degradedReason ?? null,
+  () => {
+    ackDegraded.value = false
+  },
+)
+const needsAck = computed(() => !!vc.value?.degradedReason)
+const canApprove = computed(
+  () => awaitingHuman.value && !busy.value && (!needsAck.value || ackDegraded.value),
+)
+
 async function approve() {
   if (!blockId.value || !canApprove.value) return
   await visualConfirm.approve(blockId.value)
@@ -169,20 +183,6 @@ async function onFilePicked(e: Event) {
   uploadView.value = ''
   if (fileInput.value) fileInput.value.value = ''
 }
-
-// Degraded-basis approval guard (no capture / a fix landed after these shots): require an
-// explicit "I reviewed this another way" acknowledgement before the one-click approve.
-const ackDegraded = ref(false)
-watch(
-  () => vc.value?.degradedReason ?? null,
-  () => {
-    ackDegraded.value = false
-  },
-)
-const needsAck = computed(() => !!vc.value?.degradedReason)
-const canApprove = computed(
-  () => awaitingHuman.value && !busy.value && (!needsAck.value || ackDegraded.value),
-)
 </script>
 
 <template>

package/app/pages/index.vue

@@ -285,6 +285,27 @@ watch(
   { immediate: true },
 )
 
+// Probe the GitHub integration as soon as a board is active (re-probe per board —
+// connections are per workspace). The result drives the onboarding gate in the template
+// before the board mounts, so an unconnected user can't slip past it. `ensureProbed`
+// single-flights per board (app-startup initiative, item 12), so this and the SideBar's
+// probe collapse to one request on a cold open instead of two.
+watch(
+  () => workspace.workspaceId,
+  (id) => {
+    if (id) void github.ensureProbed()
+  },
+  { immediate: true },
+)
+
+// Hard gate: the App is enabled on the backend but this workspace has no
+// installation yet. `available === null` means the probe is still in flight.
+// Both are declared here, ahead of the tutorial offer below, because that offer reads them
+// from a watcher that runs synchronously during setup (`immediate: true`). Declared after
+// it, they'd still be in their TDZ and the first run would throw.
+const needsGitHubInstall = computed(() => github.available === true && !github.connected)
+const githubProbePending = computed(() => github.available === null)
+
 // Offer the tutorial on launch, once the board is up. Yields to every other startup
 // surface — the GitHub onboarding gate and the advisory/onboarding modals above — so a
 // first launch never stacks the tour prompt on top of a dialog that needs answering
@@ -347,24 +368,6 @@ useTutorialNudge()
 // simply has nothing to talk to, and the browser-persisted store carries on alone.
 useTutorialSync()
 
-// Probe the GitHub integration as soon as a board is active (re-probe per board —
-// connections are per workspace). The result drives the onboarding gate below
-// before the board mounts, so an unconnected user can't slip past it. `ensureProbed`
-// single-flights per board (app-startup initiative, item 12), so this and the SideBar's
-// probe collapse to one request on a cold open instead of two.
-watch(
-  () => workspace.workspaceId,
-  (id) => {
-    if (id) void github.ensureProbed()
-  },
-  { immediate: true },
-)
-
-// Hard gate: the App is enabled on the backend but this workspace has no
-// installation yet. `available === null` means the probe is still in flight.
-const needsGitHubInstall = computed(() => github.available === true && !github.connected)
-const githubProbePending = computed(() => github.available === null)
-
 // Subscribe to the backend's real-time event stream and (re)connect whenever the
 // active workspace changes. Runs advance durably server-side; progress arrives as
 // pushed events rather than by polling.

package/app/stores/execution.ts

@@ -131,6 +131,12 @@ export const useExecutionStore = defineStore('execution', () => {
     } else instances.value.push(instance)
   }
 
+  const byId = computed(() => {
+    const map = new Map<string, ExecutionInstance>()
+    for (const e of instances.value) map.set(e.id, e)
+    return map
+  })
+
   /**
    * Run an action that returns a run's authoritative sub-state and apply that state to the cached
    * run as an OPTIMISTIC ECHO — but only when the event stream has not delivered a newer revision
@@ -169,12 +175,6 @@ export const useExecutionStore = defineStore('execution', () => {
     return state
   }
 
-  const byId = computed(() => {
-    const map = new Map<string, ExecutionInstance>()
-    for (const e of instances.value) map.set(e.id, e)
-    return map
-  })
-
   function getInstance(id: string | null | undefined) {
     return id ? byId.value.get(id) : undefined
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cat-factory/app",
-  "version": "0.228.0",
+  "version": "0.228.1",
   "description": "Reusable Nuxt layer for the Agent Architecture Board SPA (components, stores, composables, pages). Consume it from a thin deployment app via `extends: ['@cat-factory/app']` and point it at your backend with NUXT_PUBLIC_API_BASE. See deploy/frontend for an example.",
   "repository": {
     "type": "git",