refactor(tui): clarify subscribeInput locals and drop test bangs

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: armancharan

packages/tui/src/context/event-interest.ts

@@ -21,8 +21,10 @@ export function interestEqual(left?: EventInterest, right?: EventInterest) {
 /** Map TUI interest into the generated `event.subscribe` query shape. */
 export function subscribeInput(interest?: EventInterest) {
   if (!interest) return undefined
+  const location = interest.location
+  const session = interest.sessions && interest.sessions.length > 0 ? [...interest.sessions] : undefined
   return {
-    ...(interest.location ? { location: interest.location } : {}),
-    ...(interest.sessions && interest.sessions.length > 0 ? { session: [...interest.sessions] } : {}),
+    ...(location ? { location } : {}),
+    ...(session ? { session } : {}),
   }
 }

packages/tui/test/cli/tui/client-scope.test.tsx

@@ -51,8 +51,8 @@ describe("ClientProvider event.scope", () => {
     const { app, client, eventUrls } = await mount({ location: { directory } })
     try {
       await wait(() => client.connection.status() === "connected" && eventUrls.length >= 1)
-      expect(eventUrls[0]!.searchParams.get("location[directory]")).toBe(directory)
-      expect(eventUrls[0]!.searchParams.getAll("session")).toEqual([])
+      expect(eventUrls[0]?.searchParams.get("location[directory]")).toBe(directory)
+      expect(eventUrls[0]?.searchParams.getAll("session")).toEqual([])
     } finally {
       app.renderer.destroy()
     }
@@ -65,8 +65,8 @@ describe("ClientProvider event.scope", () => {
 
       client.event.scope({ location: { directory }, sessions: ["ses_a"] })
       await wait(() => eventUrls.length >= 2)
-      expect(eventUrls[1]!.searchParams.get("location[directory]")).toBe(directory)
-      expect(eventUrls[1]!.searchParams.getAll("session")).toEqual(["ses_a"])
+      expect(eventUrls[1]?.searchParams.get("location[directory]")).toBe(directory)
+      expect(eventUrls[1]?.searchParams.getAll("session")).toEqual(["ses_a"])
     } finally {
       app.renderer.destroy()
     }