coracle/flotilla
1
6
Fork 23
Code Issues 42 Pull Requests 2 Actions Packages Projects 5 Releases Wiki Activity

Fix voice call cold-start timeout and preserve custom timeout message (#174) #203

Merged
hodlbod merged 1 commits from Khushvendra/flotilla:fix/voice-calls-timeout-174 into dev 2026-04-15 20:37:03 +00:00
Conversation 0 Commits 1 Files Changed 3
+6 -3
Khushvendra commented 2026-04-15 08:24:04 +00:00
Contributor
Copy Link
Copy Source

Summary

Fixes #174 where voice join could fail on first attempt due to an aggressive timeout, and timeout errors always showed the generic "Timed out" message.

Root Cause

  • Voice connect raced against a 5s timeout; cold starts can exceed 5s.
  • The timeout helper ignored the provided message option and always created TimeoutError with the default message.

Changes

  • Increased voice connect timeout from 5s to 15s.
  • Forwarded the timeout message option into TimeoutError.
  • Kept the packageManager field added in package.json during local tooling execution.

Validation

  • pnpm run check passes.
  • pnpm run lint passes.
  • Reproduction simulation confirms expected behavior:
    • 8s connect + 5s timeout: first attempt times out.
    • 1s connect + 5s timeout: retry succeeds.
    • 8s connect + 15s timeout: first attempt succeeds.
    • Timeout now surfaces the custom "Connection timed out..." message.

Notes

A 15s timeout aligns with LiveKit defaults and better accommodates cold-start latency while still failing on genuine connectivity issues.

Closes #174.

## Summary Fixes #174 where voice join could fail on first attempt due to an aggressive timeout, and timeout errors always showed the generic "Timed out" message. ## Root Cause - Voice connect raced against a 5s timeout; cold starts can exceed 5s. - The timeout helper ignored the provided message option and always created `TimeoutError` with the default message. ## Changes - Increased voice connect timeout from 5s to 15s. - Forwarded the timeout `message` option into `TimeoutError`. - Kept the `packageManager` field added in `package.json` during local tooling execution. ## Validation - `pnpm run check` passes. - `pnpm run lint` passes. - Reproduction simulation confirms expected behavior: - 8s connect + 5s timeout: first attempt times out. - 1s connect + 5s timeout: retry succeeds. - 8s connect + 15s timeout: first attempt succeeds. - Timeout now surfaces the custom "Connection timed out..." message. ## Notes A 15s timeout aligns with LiveKit defaults and better accommodates cold-start latency while still failing on genuine connectivity issues. Closes #174.
Khushvendra added 1 commit 2026-04-15 08:24:04 +00:00
Fix voice call timeout handling on cold starts 8642a52b48
hodlbod merged commit 4cc1cc95ca into dev 2026-04-15 20:37:03 +00:00
hodlbod deleted branch fix/voice-calls-timeout-174 2026-04-15 20:37:04 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
design
For issues related to design work. Development should be done in another issue and refer back to this one.
 dev
Issues related to development work
 easy
Issues that can be executed on by new contributors
 idea
For issues that aren't yet actionable
 priority
For issues related to high priority features and bugs
 usability
Issues that have to be validated with real live human beings.
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
hodlbod (hodlbod) mplorentz (Matt Lorentz) sakshamjain (Saksham Jain) sondreb triesap userAdityaa (Aditya Chaudhary)
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: coracle/flotilla#203
Delete Branch "Khushvendra/flotilla:fix/voice-calls-timeout-174"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?