[waleedlatif1]

Summary

• The room coalescing from perf(db): cut hot-path database work and close copilot cleanup gaps #9419 awaits each delivery's refetches before the next delivery, but a fetch has no timeout of its own: one that hung held the room's deliveries forever, so the tab stopped picking up remote file changes
• Each delivery now waits at most DELIVERY_SETTLE_CAP_MS (60s, above the slowest list read) via the existing withinDeadline helper, which clears its timer; past it, the next delivery's invalidateQueries (cancelRefetch: true) cancels and restarts the hung fetch, the recovery every broadcast gave before coalescing
• Normal deliveries are unchanged: a refetch that settles within the cap is still waited for, never overlapped; a hung first load (no cached data) is joined rather than restarted, as before

Type of Change

• Bug fix

Testing

• New test: a refetch that never settles no longer blocks the next delivery; fails on the pre-fix hook ("expected 2 times, got 1"), passes after; the existing coalescing tests still pass (6/6)
• bun run lint, bun run check:audits (61), bun run type-check

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing (new tests pass the test-audit authoring gate)
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated
docs	Skipped		Oct 29, 2026 12:42am UTC

[greptile-apps]

Confidence Score: 5/5

[Medium impact] The PR appears safe to merge.

Summary

Caps how long each room delivery waits for its refetches.

• Workspace room refreshes resume after a callback waits longer than 60 seconds.

Diagram

%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Room change arrives] --> B[Run callbacks]
  B --> C[Wait for refetches or 60-second cap]
  C --> D[Wait one second]
  D --> E{More changes arrived?}
  E -->|Yes| B
  E -->|No| F[Finish delivery]

Loading

_{Reviews (3) · Last reviewed commit: "fix(files): cap a room delivery's wait a..." · Reviewed by Greptile}

[cubic-dev-ai]

All reported issues were addressed across 2 files

<sub>Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic</sub>

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

All reported issues were addressed across 2 files

<sub>Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic</sub>

[waleedlatif1]

@greptile

[waleedlatif1]

@cubic-dev-ai review this PR

[cubic-dev-ai]

@cubic-dev-ai review this PR

@waleedlatif1 I have started the AI code review. It will take a few minutes to complete.

[cubic-dev-ai]

No issues found across 2 files

Confidence score: 5/5

• Automated review surfaced no issues in the provided summaries.
• No files require special attention.

_{Turn on auto-fix | Re-trigger cubic}