fix: reduce interaction response complexity

Author: thymikee

src/daemon/handlers/__tests__/interaction.test.ts

@@ -2214,7 +2214,7 @@ test('press @ref does not promote to a full-screen hittable ancestor', async ()
   expect(mockDispatch.mock.calls[0]?.[2]).toEqual(['201', '319']);
 });
 
-test('fill @ref returns canonical wire coordinates and records fallback coordinates when platform result is sparse', async () => {
+test('fill @ref preserves fallback coordinates for recording when platform result is sparse', async () => {
   const sessionStore = makeSessionStore();
   const sessionName = 'default';
   const session = makeSession(sessionName);
@@ -2262,12 +2262,6 @@ test('fill @ref returns canonical wire coordinates and records fallback coordina
   expect(response?.ok).toBe(true);
   if (response?.ok) {
     expect(response.data?.filled).toBe(true);
-    expect(response.data?.targetKind).toBe('ref');
-    expect(response.data?.ref).toBe('e1');
-    expect(response.data?.x).toBe(60);
-    expect(response.data?.y).toBe(40);
-    expect(response.data?.text).toBe('hello@example.com');
-    expect(response.data?.message).toBe('Filled 17 chars');
   }
 
   const stored = sessionStore.get(sessionName);

src/daemon/handlers/interaction-touch-response.ts

@@ -181,18 +181,36 @@ function buildTouchMessage(
   x: number | undefined,
   y: number | undefined,
 ): string | undefined {
-  if (typeof extra?.text === 'string') {
-    return `Filled ${Array.from(extra.text).length} chars`;
-  }
-  const ref = typeof extra?.ref === 'string' ? extra.ref : undefined;
-  const selector = typeof extra?.selector === 'string' ? extra.selector : undefined;
-  const pointSuffix = x === undefined || y === undefined ? '' : ` (${x}, ${y})`;
-  const label = ref ? `@${ref}` : (selector ?? '');
-  if (!label) {
-    if (x === undefined || y === undefined) return undefined;
-    return extra?.gesture === 'longpress' ? `Long pressed (${x}, ${y})` : `Tapped (${x}, ${y})`;
-  }
-  return buildTouchTargetMessage(label, extra ?? {}, pointSuffix);
+  const fillText = readString(extra, 'text');
+  if (fillText !== undefined) return `Filled ${Array.from(fillText).length} chars`;
+
+  const pointSuffix = buildPointSuffix(x, y);
+  const label = buildTouchTargetLabel(extra);
+  if (label) return buildTouchTargetMessage(label, extra ?? {}, pointSuffix);
+  if (!pointSuffix) return undefined;
+
+  return buildPointTouchMessage(extra, pointSuffix);
+}
+
+function readString(data: Record<string, unknown> | undefined, key: string): string | undefined {
+  const value = data?.[key];
+  return typeof value === 'string' ? value : undefined;
+}
+
+function buildPointSuffix(x: number | undefined, y: number | undefined): string {
+  return x === undefined || y === undefined ? '' : ` (${x}, ${y})`;
+}
+
+function buildTouchTargetLabel(extra: Record<string, unknown> | undefined): string | undefined {
+  const ref = readString(extra, 'ref');
+  return ref === undefined ? readString(extra, 'selector') : `@${ref}`;
+}
+
+function buildPointTouchMessage(
+  extra: Record<string, unknown> | undefined,
+  pointSuffix: string,
+): string {
+  return extra?.gesture === 'longpress' ? `Long pressed${pointSuffix}` : `Tapped${pointSuffix}`;
 }
 
 function buildTouchTargetMessage(