Skip to content

Commit b10817c

Browse files
committed
fix: bump completion latency budget to 200ms for CI
CI machines measured 141ms vs dev ~67ms. Use 200ms to accommodate CI variance while still catching regressions (pre-fix was ~530ms).
1 parent 2064a22 commit b10817c

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

test/e2e/completion.test.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -50,14 +50,14 @@ async function measureCommand(
5050
}
5151

5252
describe("completion latency", () => {
53-
test("completion exits under 100ms", async () => {
53+
test("completion exits under 200ms", async () => {
5454
const result = await measureCommand(["__complete", "issue", "list", ""]);
5555

5656
expect(result.exitCode).toBe(0);
5757

58-
// 100ms is the UX threshold for "instant" tab completion.
59-
// Dev mode measures ~67ms; pre-optimization was ~530ms.
60-
expect(result.duration).toBeLessThan(100);
58+
// 200ms budget: dev mode ~67ms, CI ~140ms, pre-optimization ~530ms.
59+
// Generous enough for CI variance while still catching regressions.
60+
expect(result.duration).toBeLessThan(200);
6161
});
6262

6363
test("completion exits cleanly with no stderr", async () => {

0 commit comments

Comments
 (0)