chore(code): Biome fixes and inbox formatting

Author: Twixes

apps/code/src/main/services/agent/service.test.ts

@@ -356,7 +356,12 @@ describe("AgentService", () => {
       service.recordActivity("run-1");
       const secondDeadline = getIdleTimeouts(service).get("run-1")?.deadline;
 
-      expect(secondDeadline).toBeGreaterThan(firstDeadline!);
+      expect(firstDeadline).toBeDefined();
+      expect(secondDeadline).toBeDefined();
+      if (firstDeadline === undefined || secondDeadline === undefined) {
+        throw new Error("expected idle deadlines");
+      }
+      expect(secondDeadline).toBeGreaterThan(firstDeadline);
     });
 
     it("kills idle session after timeout expires", () => {

apps/code/src/main/services/suspension/service.test.ts

@@ -136,11 +136,13 @@ function seedWorktreeWorkspace(
     repositoryId: overrides.repositoryId ?? "repo-1",
     mode: overrides.mode ?? "worktree",
   });
-  const stored = mocks.workspaceRepo._workspaces.get(ws.id)!;
+  const stored = mocks.workspaceRepo._workspaces.get(ws.id);
+  if (!stored) throw new Error("expected workspace in test map");
   if (overrides.lastActivityAt !== undefined)
     stored.lastActivityAt = overrides.lastActivityAt;
   if (overrides.createdAt !== undefined) stored.createdAt = overrides.createdAt;
-  const resolved = mocks.workspaceRepo.findById(ws.id)!;
+  const resolved = mocks.workspaceRepo.findById(ws.id);
+  if (!resolved) throw new Error("expected workspace after create");
   mocks.worktreeRepo.create({
     workspaceId: resolved.id,
     name: `wt-${resolved.taskId}`,

apps/code/src/main/services/suspension/service.ts

@@ -321,6 +321,12 @@ export class SuspensionService extends TypedEventEmitter<SuspensionServiceEvents
 
     if (isWorktreeMode) {
       const worktreePath = worktree.path;
+      const checkpointId = suspendedTask.checkpointId;
+      if (!checkpointId) {
+        throw new Error(
+          "Internal error: worktree suspend requires checkpoint id",
+        );
+      }
 
       const branch = await this.getCurrentBranchName(worktreePath);
       if (branch && branch !== "HEAD") suspendedTask.branchName = branch;
@@ -330,12 +336,12 @@ export class SuspensionService extends TypedEventEmitter<SuspensionServiceEvents
           await this.captureWorktreeCheckpoint(
             folderPath,
             worktreePath,
-            suspendedTask.checkpointId!,
+            checkpointId,
           );
         },
         async () => {
           const git = createGitClient(folderPath);
-          await deleteCheckpoint(git, suspendedTask.checkpointId!);
+          await deleteCheckpoint(git, checkpointId);
         },
       );
 
@@ -380,13 +386,14 @@ export class SuspensionService extends TypedEventEmitter<SuspensionServiceEvents
       workspace.mode === "worktree" &&
       suspension.checkpointId
     ) {
+      const restoreCheckpointId = suspension.checkpointId;
       await step(
         async () => {
           restoredWorktreeName = await this.restoreWorktreeFromCheckpoint(
             folderPath,
             workspace,
             suspension.branchName,
-            suspension.checkpointId!,
+            restoreCheckpointId,
             recreateBranch,
           );
         },

apps/code/src/renderer/features/inbox/components/InboxSignalsTab.tsx

@@ -449,7 +449,9 @@ export function InboxSignalsTab() {
                   variant="detail"
                 />
                 <Flex align="center" gap="2" wrap="wrap">
-                  <SignalReportPriorityBadge priority={selectedReport.priority} />
+                  <SignalReportPriorityBadge
+                    priority={selectedReport.priority}
+                  />
                   <Badge variant="soft" color="gray" size="1">
                     {selectedReport.signal_count} occurrences
                   </Badge>

apps/code/src/renderer/features/inbox/components/ReportCard.tsx

@@ -101,25 +101,26 @@ export function ReportCard({
               {report.title ?? "Untitled signal"}
             </Text>
             <Flex align="center" gapX="2" wrap="wrap">
-            <span
-              className="shrink-0 rounded-sm px-1 py-px font-mono text-[9px] uppercase tracking-wider"
-              style={{
-                color: accent,
-                backgroundColor: isReady ? "var(--green-3)" : "var(--gray-3)",
-                border: `1px solid ${isReady ? "var(--green-6)" : "var(--gray-6)"}`,
-              }}
-            >
-              {statusLabel}
-            </span>
-            <SignalReportPriorityBadge priority={report.priority} />
+              <span
+                className="shrink-0 rounded-sm px-1 py-px font-mono text-[9px] uppercase tracking-wider"
+                style={{
+                  color: accent,
+                  backgroundColor: isReady ? "var(--green-3)" : "var(--gray-3)",
+                  border: `1px solid ${isReady ? "var(--green-6)" : "var(--gray-6)"}`,
+                }}
+              >
+                {statusLabel}
+              </span>
+              <SignalReportPriorityBadge priority={report.priority} />
+            </Flex>
+            <div style={{ opacity: isReady ? 1 : 0.82 }}>
+              <SignalReportSummaryMarkdown
+                content={report.summary}
+                fallback="No summary yet — still collecting context."
+                variant="list"
+              />
+            </div>
           </Flex>
-          <div style={{ opacity: isReady ? 1 : 0.82 }}>
-            <SignalReportSummaryMarkdown
-              content={report.summary}
-              fallback="No summary yet — still collecting context."
-              variant="list"
-            />
-          </div></Flex>
         </Flex>
         <Flex direction="column" align="end" gap="1" className="shrink-0">
           <Text size="1" color="gray" className="font-mono text-[11px]">

apps/code/src/renderer/features/message-editor/components/AttachmentMenu.tsx

@@ -145,11 +145,11 @@ export function AttachmentMenu({
                 issueButton
               )}
             </div>
-          ) : (
+          ) : repoPath ? (
             <div className="issue-picker">
-              <IssuePicker repoPath={repoPath!} onSelect={handleIssueSelect} />
+              <IssuePicker repoPath={repoPath} onSelect={handleIssueSelect} />
             </div>
-          )}
+          ) : null}
         </Popover.Content>
       </Popover.Root>
     </>

apps/code/src/renderer/utils/analytics.ts

@@ -136,7 +136,7 @@ export function buildPermissionToolMetadata(
   permission?: PermissionRequest,
   selectedOptionId?: string,
   customInput?: string,
-): Record<string, any> {
+): Record<string, unknown> {
   const selectedOption = permission?.options?.find(
     (o) => o.optionId === selectedOptionId,
   );