test(server): give the Daybreak compact case the server budget

[lidge-jun]

Summary

A full-suite parallel run on the Linux test host reported one failure:

(fail) server local API auth > #2097: Daybreak compact uses the stable wire model without retention [5003.01ms]

5003ms is Bun's default 5s test timeout, not an assertion failure. The case passes 3/3 in isolation on both macOS and the Linux host.

It starts a real server through startPoolRetryHarness, and its neighbours in this file already declare SERVER_BUDGET_MS for that reason — this one was left on the default intended for pure unit tests. Under full-suite parallel load, server startup plus the request exceeded 5s.

Verification

bun test ./tests/server-auth.test.ts    86 pass / 0 fail

Isolation runs before the change: 3/3 pass on macOS (2.78s, 504ms, 737ms) and 3/3 on Linux (301ms, 325ms, 315ms) — which is what identifies this as load-dependent rather than a real defect.

Checklist

• Targets dev
• Test-only; no production behavior changed
• Budget matches the existing convention in the same file

Summary by CodeRabbit

• Tests
  • Updated the pool retry test to use the server time budget for more reliable timeout handling.

[github-actions]

✅ Deterministic PR hygiene checks passed.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 68a27bc7-aa29-42d3-9d1a-8c44891d7824

📥 Commits

Reviewing files that changed from the base of the PR and between 6ccb13c and 1dadec5.

📒 Files selected for processing (1)

• tests/server-auth.test.ts

---

📝 Walkthrough

Walkthrough

The pool retry test in tests/server-auth.test.ts now uses SERVER_BUDGET_MS for its timeout instead of Bun’s default test timeout.

Changes

Server authentication test timeout

Layer / File(s)	Summary
Pool retry timeout update tests/server-auth.test.ts	The pool retry test now uses SERVER_BUDGET_MS as its timeout at lines 2246–2250.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: ingwannu, wibias, luvs01

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch codex/server-auth-daybreak-budget

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}