fix(sound): playing a sound that is already playing does nothing
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 82/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- audio-video-rtc
Línea de trabajo
Lee modules/sound/src/sound.library.ts en las líneas 57–66 y la promesa de superposición en modules/sound/README.md. Revisa las pruebas o los scripts existentes del paquete de sonido para averiguar cómo se activa la reproducción. Reproduce dos llamadas rápidas a play("hit") y luego confirma que las llamadas repetidas activan la reproducción tal como se describe en el README.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Which package is this bug report for?
sound
Issue description
Each key has one HTMLAudioElement. Calling play() on an element that is already playing does not restart it, so two play("hit") calls within the clip length give a single sound. Repeated effects (shots, hits) are dropped, although the README says sounds may overlap.
Where:
Evidence:
const element = this._sounds.get(key);
if (!element) throw new NfNotFound(key);
element.play().catch(...)
Suggested fix: Set element.currentTime = 0 before playing, or play a cloneNode() per call if same-key overlap is intended.
Found during a code audit of main at be72076.
Code sample
No response
Versions
- Engine
mainat 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
- Lenguaje dominante
- TypeScript
- Estrellas
- 7
- Forks
- 2
- Merge medio
- 1 d 17 h
- PR fusionados (30 d)
- 17
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de NanoForge-dev/Engine
-
dependencies enhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
NanoForge-dev/Engine#513 ·
Los mantenedores suelen responder en 1 día
-
bug chore tests
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
NanoForge-dev/Engine#511 ·
Los mantenedores suelen responder en 1 día
-
docs(network): TSDoc says ICE_SERVERS defaults to an empty list, the code uses public STUN serversAbiertodocumentation good first issue modules:network
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
NanoForge-dev/Engine#509 ·
Los mantenedores suelen responder en 1 día
-
documentation modules:ecs packages:asset packages:core
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
NanoForge-dev/Engine#508 ·
Los mantenedores suelen responder en 1 día
-
bug good first issue modules:music modules:sound
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
NanoForge-dev/Engine#505 ·
Los mantenedores suelen responder en 1 día
Todos los issues de NanoForge-dev/Engine
Issues similares
-
Mend: dependency security vulnerability
Dificultad 1/5 1-3 horas Aptitud para principiantes 65/100
fdm-monster/fdm-monster#5563 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
be-BOP-io-SA/be-BOP#2849 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
Los mantenedores suelen responder en 1 día
-
1️⃣ phase/under-investigation flaky/confirmed flaky/needs-decision type/bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
growilabs/growi#12246 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
🐛 The Rulekeeper - An index run that gives up on the lock ends SucceededPosiblemente ocupada @github-actions la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
SSWConsulting/SSW.Rules#2809 · 1 comentario ·
Los mantenedores suelen responder en 1 día