fix(audio): deduplicate getSourceTrackIdFromPath and fix companion track misclassification
Los mantenedores suelen responder en 2 días
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 74/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- audio-video-rtc
Línea de trabajo
Empieza comparando las dos implementaciones de getSourceTrackIdFromPath en src/lib/exporter/audioRoutingEngine.ts (líneas 32-37) y src/lib/exporter/audioProcessorShared.ts (líneas 69-97), centrándote en cómo maneja cada una -mic., _mic_, .wav y .mp4. Luego revisa los llamadores, como useClipAudioSettingsController.ts, que importan desde audioRoutingEngine. Se considera terminado cuando se elimina el duplicado, el parser canónico se vuelve a exportar y rutas como recording-mic.wav y system.mp4 se clasifican como mic y system en lugar de mixed.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
What is the issue?
There are currently two different implementations of getSourceTrackIdFromPath:
src/lib/exporter/audioRoutingEngine.ts(lines 32–37):
A naive parser that only checks.includes(".mic.")and.includes(".system.").src/lib/exporter/audioProcessorShared.ts(lines 69–97):
The canonical, robust parser checking-mic.,_mic_,/mic.,.wav, and.mp4suffixes.
Callers such as useClipAudioSettingsController.ts import from audioRoutingEngine. When companion audio paths use standard separators like recording-mic.wav or system.mp4, audioRoutingEngine misclassifies them as "mixed", causing track controls and export routing to misroute or drop companion audio.
Suggested Fix
Delete the hand-rolled duplicate parser in audioRoutingEngine.ts and re-export the canonical getSourceTrackIdFromPath from audioProcessorShared.ts.
Benefits
- Eliminates code duplication across audio pipeline modules.
- Fixes track classification bugs for hyphenated and extension-based companion audio paths.
- Lenguaje dominante
- TypeScript
- Estrellas
- 32.4k
- Forks
- 2.6k
- Merge medio
- 6 d 2 h
- PR fusionados (30 d)
- 18
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 webadderallorg/Recordly
-
System FFmpeg should be preferred over bundled ffmpeg-static on Linux for GPU hardware accelerationPosiblemente ocupada @LangerSword la tomó hace 19 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
webadderallorg/Recordly#757 · 1 reacción ·
Los mantenedores suelen responder en 2 días
-
Dificultad 4/5 3-5 días Aptitud para principiantes 43/100
webadderallorg/Recordly#1157 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
webadderallorg/Recordly#1156 ·
Los mantenedores suelen responder en 2 días
-
Camera frame duplicationAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 56/100
webadderallorg/Recordly#1152 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 4/5 1-2 días Aptitud para principiantes 48/100
webadderallorg/Recordly#1151 ·
Los mantenedores suelen responder en 2 días
Todos los issues de webadderallorg/Recordly
Issues similares
-
VS Code extension: clickable file links in chat drop the :line suffix, never jump to the lineAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
MoonshotAI/kimi-code#4398 ·
Los mantenedores suelen responder en 1 día
-
Xiaomi Mimo语音识别服务异常Abiertopending triage
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 1 día
-
package-lock.json : les binaires natifs macOS et Linux de unrs-resolver ont disparu depuis #1823Abiertobug dependencies
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
AxaFrance/design-system#2064 ·
Los mantenedores suelen responder en 3 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100