e2e: register cursor-cli concurrency gate (PR #1577 review)

13a04b9→main·

toothbrush·2w ago·1 file·+4 added/-0 removed

E2E_CONCURRENT_TEST_LIMIT=2 for cursor-cli was a no-op: AcquireSlot only throttles agents that called RegisterGate, and cursor-cli never did, so its tests ran at full go-test parallelism and kept hitting "Increase limits for faster responses". Register the gate (default 2) so the throttle actually applies.

Note: factoryai-droid has the same latent no-op (sets the env limit but registers no gate) — left as-is pending a decision on whether to actually serialize it.

Co-Authored-By: Claude Opus 4.8 (1M context) noreply@anthropic.com

Sessions

164b295c482aView transcript

Changes

1

• e2e/agents
  • Mcursor_cli.go+4

15 unmodified lines

16
17
18
19
20
21
22
23
24
25

15 unmodified lines

return
    }
    Register(&CursorCLI{})
    // Cursor is rate-limited ("Increase limits for faster responses"), so gate
    // its parallelism. Without a registered gate, AcquireSlot is a no-op and the
    // E2E_CONCURRENT_TEST_LIMIT override the workflow sets has no effect.
    RegisterGate("cursor-cli", 2)
}

// CursorCLI implements the E2E Agent interface for the Cursor Agent CLI binary.

Me2e/agents/cursor_cli.go+4