fix(hitl): stop duplicating input fields across listed pause points

Author: waleedlatif1

apps/sim/lib/workflows/executor/human-in-the-loop-manager.ts

@@ -3624,12 +3624,13 @@ export class PauseResumeManager {
         entry.contextId === contextId && (entry.status === 'claimed' || entry.status === 'pending')
     )
 
-    // The selected pause point's full `response.data` and `display` are already
-    // returned via `pausePoint` below; strip both from `execution.pausePoints` so a
-    // large HITL display payload isn't duplicated in full within the same response.
+    // The selected pause point's full `response.data`, `display`, and `inputFields`
+    // are already returned via `pausePoint` below; strip them from
+    // `execution.pausePoints` so a large HITL payload isn't duplicated in full
+    // within the same response.
     const execution: PausedExecutionDetail = {
       ...detail,
-      pausePoints: detail.pausePoints.map(({ display, ...point }) =>
+      pausePoints: detail.pausePoints.map(({ display, inputFields, ...point }) =>
         point.response ? { ...point, response: { ...point.response, data: undefined } } : point
       ),
     }

apps/sim/lib/workflows/executor/pause-persistence.integration.ts

@@ -445,7 +445,7 @@ describe('readPausedRunPoints', () => {
     }
   })
 
-  it('returns the selected point display once, not again in every listed point', async () => {
+  it('returns the selected point display and input fields once, not again in every listed point', async () => {
     const registry = registryWithResolvedSecret()
     const executionId = await publishPausedRun([
       humanBranchPoint(0, registry),
@@ -461,9 +461,11 @@ describe('readPausedRunPoints', () => {
     expect(detail?.pausePoint.display).toEqual({
       responseStructure: [{ name: 'summary', type: 'string', value: 'Token is {{API_KEY}}' }],
     })
+    expect(detail?.pausePoint.inputFields).toHaveLength(1)
     expect(detail?.execution.pausePoints).toHaveLength(2)
     for (const point of detail?.execution.pausePoints ?? []) {
       expect(point).not.toHaveProperty('display')
+      expect(point).not.toHaveProperty('inputFields')
     }
   })