Skip to content

Conversation

Copy link

Copilot AI commented Nov 30, 2025

transition.mp3 was playing twice—once at auto end and once after endauto.wav finished. The requested behavior is to play endauto.wav immediately when auto ends, then transition.mp3 exactly 3 seconds after auto ends (independent of endauto.wav duration).

Changes

  • AudioService: Added playTransitionAfterDelay() method that schedules transition sound after TRANSITION_TO_COUNTDOWN_DELAY_MS (3s)
  • MatchTimer: Decoupled transition audio from endauto callback—now calls playTransitionAfterDelay() directly when auto ends
// Before: transition played after endauto finished
audioService.playEndAuto(() -> {
    // state transition logic...
    audioService.playTransition();  // triggered by endauto completion
});

// After: both sounds scheduled from auto end, independently timed
audioService.playEndAuto(() -> {
    // state transition logic only
});
audioService.playTransitionAfterDelay();  // 3s delay from auto end
Original prompt

now it is playing the "transition.mp3" sound twice, once right at the end of auto and the other after the "endauto.wav" file. Can you just start the "endauto.wav" after the auto period ends, and then start the "transition.mp3" sound 3 seconds after the auto period ends?


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@vercel
Copy link

vercel bot commented Nov 30, 2025

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

Project Deployment Preview Comments Updated (UTC)
ftcscorer Ready Ready Preview Comment Nov 30, 2025 2:19am

Copilot AI changed the title [WIP] Update timing for transition and end audio Fix transition.mp3 playing 3 seconds after auto ends instead of after endauto.wav Nov 30, 2025
Copilot AI requested a review from acurra3398 November 30, 2025 02:20
@acurra3398 acurra3398 marked this pull request as ready for review November 30, 2025 02:21
@acurra3398 acurra3398 merged commit 934e092 into main Nov 30, 2025
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants