FEA-1550: Fix pack install UI contract

- Align pack install IPC results and streamed output payloads across main, preload, and renderer types
- Handle failed catalog mutation starts without opening a stale install modal
- Pass selected recent project cwd through project-scoped install and uninstall actions
- Keep local-only agent session user and org identity explicitly null until server-owned identity is available

Testing: Desktop typecheck, lint, and renderer build passed

Risks: Low; scoped to pack catalog install UI flow and typed IPC contracts

Author: mikeangstadt

apps/desktop/src/main/agent-dashboard-design-system-runtime.ts

@@ -400,12 +400,13 @@ function registerDesignSystemDbIpcHandlers(
     });
   });
 
-  ipcMain.handle("desktop:db:catalog-uninstall", async (_event, packId: unknown, harness: unknown) => {
+  ipcMain.handle("desktop:db:catalog-uninstall", async (_event, packId: unknown, harness: unknown, cwd?: unknown) => {
     if (typeof packId !== "string" || typeof harness !== "string") return { started: false };
     return streamRun(dbForStores, {
       pack_id: packId,
       harness,
       action: "uninstall",
+      cwd: typeof cwd === "string" ? cwd : undefined,
       getWindow: options.getWindow,
       onComplete: () => void runPackScanner(dbForStores).catch(() => {}),
     });

apps/desktop/src/main/app.ts

@@ -1447,6 +1447,8 @@ export class DesktopApplication {
           await createAgentDashboardDesignSystemRuntime({
             userDataPath: app.getPath("userData"),
             getWindow: () => this.desktopWindow.getWindow(),
+            // User/org IDs are server-owned. Local-only sessions keep these columns null.
+            getUserIdentity: () => null,
             onTerminalFailure: (reason) => {
               const notification = new Notification({
                 title: "ClosedLoop Agent Monitor",

apps/desktop/src/main/preload-design-system.ts

@@ -4,6 +4,7 @@ import type {
   AgentRow,
   AnalyticsData,
   CatalogEntry,
+  CatalogMutationResult,
   DashboardCoreFeatures,
   DashboardPackSummary,
   DashboardPlanSummary,
@@ -15,6 +16,7 @@ import type {
   EventCountByType,
   EventRow,
   EventWithSession,
+  InstallOutputChunk,
   InstallRunRecord,
   InstalledPack,
   InstalledPackDetail,
@@ -67,8 +69,8 @@ const designSystemDashboardApi = {
     getCatalogReadme: (packId: string) => ipcRenderer.invoke("desktop:db:get-catalog-readme", packId) as Promise<string | null>,
     getCatalogContents: (packId: string) => ipcRenderer.invoke("desktop:db:get-catalog-contents", packId) as Promise<unknown[] | null>,
     getCatalogHistory: (packId: string) => ipcRenderer.invoke("desktop:db:get-catalog-history", packId) as Promise<Array<{ fetchedAt: string; stars: number; forks: number }>>,
-    catalogInstall: (packId: string, harness: string, cwd?: string) => ipcRenderer.invoke("desktop:db:catalog-install", packId, harness, cwd) as Promise<{ runId: number }>,
-    catalogUninstall: (packId: string, harness: string) => ipcRenderer.invoke("desktop:db:catalog-uninstall", packId, harness) as Promise<{ runId: number }>,
+    catalogInstall: (packId: string, harness: string, cwd?: string) => ipcRenderer.invoke("desktop:db:catalog-install", packId, harness, cwd) as Promise<CatalogMutationResult>,
+    catalogUninstall: (packId: string, harness: string, cwd?: string) => ipcRenderer.invoke("desktop:db:catalog-uninstall", packId, harness, cwd) as Promise<CatalogMutationResult>,
     catalogRefresh: () => ipcRenderer.invoke("desktop:db:catalog-refresh") as Promise<void>,
     getInstallRuns: (packId?: string) => ipcRenderer.invoke("desktop:db:get-install-runs", packId) as Promise<InstallRunRecord[]>,
 
@@ -107,8 +109,8 @@ const designSystemDashboardApi = {
     return () => ipcRenderer.removeListener("desktop:db:changed", handler);
   },
   /** Subscribe to streamed pack install/uninstall output (FEA-1314). */
-  onInstallOutput: (callback: (payload: { runId: number; type: string; data: string }) => void) => {
-    const handler = (_event: unknown, payload: { runId: number; type: string; data: string }) =>
+  onInstallOutput: (callback: (payload: InstallOutputChunk) => void) => {
+    const handler = (_event: unknown, payload: InstallOutputChunk) =>
       callback(payload);
     ipcRenderer.on("desktop:pack:install-output", handler);
     return () => ipcRenderer.removeListener("desktop:pack:install-output", handler);

apps/desktop/src/renderer/components/features/InstallModal.tsx

@@ -53,14 +53,18 @@ export function InstallModal({
     const unsubscribe = window.desktopApi.onInstallOutput?.((payload) => {
       if (payload.runId !== runId) return;
 
-      if (payload.type === "exit") {
-        const code = parseInt(payload.data, 10);
-        setExitCode(Number.isNaN(code) ? null : code);
+      if (payload.type === "complete") {
+        const code = extractExitCode(payload.data);
         setDone(true);
+        setExitCode(code);
         return;
       }
 
-      setLines((prev) => [...prev, { type: payload.type, data: payload.data }]);
+      const rendered = formatOutputPayload(payload.type, payload.data);
+      if (!rendered) {
+        return;
+      }
+      setLines((prev) => [...prev, { type: payload.type, data: rendered }]);
     });
 
     return () => {
@@ -166,3 +170,41 @@ export function InstallModal({
     </Dialog>
   );
 }
+
+function extractExitCode(data: unknown): number | null {
+  if (typeof data === "number" && Number.isFinite(data)) {
+    return data;
+  }
+  if (typeof data === "string") {
+    const parsed = Number.parseInt(data, 10);
+    return Number.isNaN(parsed) ? null : parsed;
+  }
+  if (data && typeof data === "object" && "exit_code" in data) {
+    const value = (data as { exit_code?: unknown }).exit_code;
+    return typeof value === "number" && Number.isFinite(value) ? value : null;
+  }
+  return null;
+}
+
+function formatOutputPayload(type: string, data: unknown): string | null {
+  if (type === "start") {
+    return null;
+  }
+  if (typeof data === "string") {
+    return data;
+  }
+  if (data && typeof data === "object" && "message" in data) {
+    const message = (data as { message?: unknown }).message;
+    if (typeof message === "string") {
+      return message;
+    }
+  }
+  if (type === "post_install" || type === "copy_command" || type === "error") {
+    try {
+      return JSON.stringify(data, null, 2);
+    } catch {
+      return String(data);
+    }
+  }
+  return null;
+}