fix(music): autoplay rejections are only logged and never retried

Ouverte
#499 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 1 jour

Personne n'a encore pris cette issue.

Évaluation

Difficulté
3/5
Temps estimé
1-2 jours
Accessibilité débutants
72/100
Type d'issue
Bug
Clarté
Clairement spécifiée
Activité
Active
Stack technique
typescript
Domaine
audio-video-rtc

Piste de recherche

Start with the rejection handlers at modules/music/src/music.library.ts:70-73 and modules/sound/src/sound.library.ts:62-64, then trace how music starts in main and how input events are registered. Check existing tests around playback and input handling, or add coverage for AbortError and NotAllowedError. Done means aborted switches are not logged as errors and blocked music retries after the first pointerdown or keydown.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

bug modules:music modules:sound
Which package is this bug report for?

music

Issue description

A NotAllowedError (autoplay blocked before a user gesture) is only logged as Got error: …, and _current points to a track that is not playing, so menu music started in main stays silent for the whole session. An expected AbortError (switching tracks quickly) is logged as an error too.

Where:

Evidence:

this._current.play().catch((error: unknown) => {
  console.error(`Got error: ${error}`);
});

Suggested fix: Ignore AbortError. On NotAllowedError, keep the pending track and retry on the first pointerdown / keydown.

Found during a code audit of main at be72076.

Code sample

No response

Versions
  • Engine main at be72076
  • pnpm 12.0.0, Node.js 26, Bun 1.3.14
Issue priority

Medium (should be fixed soon)

I have tested this issue on a development release

be72076

Langage dominant
TypeScript
Étoiles
7
Forks
2
Merge moyen
1 j 17 h
PR mergées (30 j)
17

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de NanoForge-dev/Engine

Toutes les issues de NanoForge-dev/Engine

Issues similaires

Plus d'issues TypeScript

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.