🤖 fix: emit metadata update after setting task state

Address Codex review comment: when a task workspace is created,
workspaceService.create() emits metadata before taskState fields are
set. Now we call workspaceService.emitMetadataUpdate() after
setWorkspaceTaskState() so the renderer learns about parentWorkspaceId
and agentType immediately, enabling nested sidebar styling without
restart.

Added emitMetadataUpdate() method to workspaceService.

Change-Id: I7679051bf422554f0d0a3fa543d14587c163f233
Signed-off-by: Thomas Kosiewski <tk@coder.com>

Author: ThomasK33

src/node/services/sessionTimingService.test.ts

@@ -79,6 +79,7 @@ describe("SessionTimingService", () => {
       messageId,
       toolCallId: "t1",
       toolName: "bash",
+      args: { cmd: "echo hi" },
       result: { ok: true },
       timestamp: startTime + 3000,
     });
@@ -166,6 +167,7 @@ describe("SessionTimingService", () => {
       messageId,
       toolCallId: "t1",
       toolName: "bash",
+      args: { cmd: "sleep" },
       result: { ok: true },
       timestamp: startTime + 10_100,
     });

src/node/services/taskService.test.ts

@@ -276,6 +276,11 @@ class FakeWorkspaceService {
     this.appendedMessages.push({ workspaceId, message: muxMessage });
     return Promise.resolve(Ok(undefined));
   }
+
+  emitMetadataUpdate(_workspaceId: string): Promise<void> {
+    // No-op in tests - we don't need to verify metadata emission
+    return Promise.resolve();
+  }
 }
 
 function createWorkspaceMetadata(

src/node/services/taskService.ts

@@ -227,6 +227,11 @@ export class TaskService extends EventEmitter {
 
     await this.config.setWorkspaceTaskState(taskId, taskState);
 
+    // Emit metadata update so the renderer knows about parentWorkspaceId/agentType.
+    // The initial create() call emits metadata before these fields are set,
+    // so we need to re-emit after persisting the task state.
+    await this.workspaceService.emitMetadataUpdate(taskId);
+
     log.debug(`Created task workspace ${taskId} for parent ${parentWorkspaceId}`, {
       agentType,
       status: taskState.taskStatus,

src/node/services/workspaceService.ts

@@ -747,6 +747,23 @@ export class WorkspaceService extends EventEmitter {
     return allMetadata.find((m) => m.id === workspaceId) ?? null;
   }
 
+  /**
+   * Emit a metadata update event for a workspace.
+   * Used when workspace metadata changes outside of normal workspaceService flows
+   * (e.g., when taskService sets parentWorkspaceId/agentType after workspace creation).
+   */
+  async emitMetadataUpdate(workspaceId: string): Promise<void> {
+    const allMetadata = await this.config.getAllWorkspaceMetadata();
+    const metadata = allMetadata.find((m) => m.id === workspaceId) ?? null;
+
+    const session = this.sessions.get(workspaceId);
+    if (session) {
+      session.emitMetadata(metadata);
+    } else {
+      this.emit("metadata", { workspaceId, metadata });
+    }
+  }
+
   async rename(workspaceId: string, newName: string): Promise<Result<{ newWorkspaceId: string }>> {
     try {
       if (this.aiService.isStreaming(workspaceId)) {