Make seek work consistently before and after playback starts

[EvandroLG]

The seek() method currently fails silently when called before audio is decoded or playback has started.

This PR implements a fix for this issue:

• Added _pendingSeekTime: number | null private property
• Refactored seek() to accept and store seeks at any lifecycle state
• Updated play() to apply pending seek before starting playback
• Improved clamping logic to handle unknown duration gracefully

---

Note

Improves seek() reliability across lifecycle states.

• Adds _pendingSeekTime to queue seeks made before decoding/initial playback
• Updates play() to apply any pending seek before starting
• Refactors seek() to clamp safely (handles unknown duration) and to queue when not decoded; recreates source when playing, updates pause time when paused
• Extends tests to cover seeking before play() and before decoding

^{Written by Cursor Bugbot for commit 26a30f6. This will update automatically on new commits. Configure here.}

[Copilot]

Pull request overview

This PR fixes the seek() method to work correctly when called before audio decoding or playback has started, eliminating silent failures in early lifecycle states.

Key Changes:

• Added _pendingSeekTime property to queue seek requests before audio is ready
• Modified seek() to store pending seeks instead of failing silently
• Updated play() to apply any pending seek time before starting playback

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
src/audio/Audio.ts	Implements pending seek storage, updates seek() to handle early calls, and modifies play() to apply pending seeks
src/audio/tests/Audio.test.ts	Adds test coverage for seeking before play() is called and before audio is decoded

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[cursor]

This PR is being reviewed by Cursor Bugbot

Details

You are on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

[cursor]

Pending seek ignored when called after play but before decoding

When seek() is called after play() but before audio decoding completes, the seek time is stored in _pendingSeekTime but never applied. The decoded event listener only checks _pauseTime, not _pendingSeekTime. This means seeks requested during the decoding phase are silently dropped. The pending seek logic at lines 199-203 only handles seeks made before play() is called, not seeks made during the async decoding window.

Additional Locations (1)

• src/audio/Audio.ts#L198-L203

 

[cursor]

Pending seek value not re-clamped when applied

When seek() is called before audio is loaded with a time exceeding the eventual duration, the value is stored in _pendingSeekTime without upper-bound clamping (since duration is unknown/zero). When this pending seek is applied in play(), _pauseTime is set directly without re-clamping to the now-known duration. The immediate seek path at line 361 re-clamps with Math.min(clampedTime, this.duration), but the pending seek application path doesn't. This causes source.start() to receive an offset beyond the buffer duration, resulting in immediate silent end of playback.

Additional Locations (1)

• src/audio/Audio.ts#L347-L354