Audit PTT Live — 2026-05-26

Structure & Documentation

La structure réelle diverge de CLAUDE.md — l'implémentation a avancé au-delà du plan initial sans mise à jour de la doc.

Fichiers documentés mais absents : server/api/routes.js, client/src/utils/audio.js Fichiers présents mais non documentés : AudioBridgeManager.js, LiveKitServerBridge.js, AudioLevelsServer.js, + plusieurs composants React.

Problèmes critiques

1. Chaîne audio serveur incomplète Le bridge audio ne transmet pas l'audio capturé vers LiveKit. Les TODOs sont explicites :

server/bridge/AudioBridge.js:368 : // TODO: Envoyer opusData à LiveKit pour ce groupe spécifique
server/bridge/AudioBridge.js:439 : // TODO: Implémenter réception bas niveau Opus depuis LiveKit

Le flux carte son → LiveKit → clients n'est pas fonctionnel côté serveur.

2. LiveKitServerBridge.js jamais utilisé Fichier créé, jamais importé ni appelé. Contient lui-même un // TODO: Implémenter l'envoi réel vers LiveKit. Code mort qui confond les responsabilités avec LiveKitClient.js.

3. Pas d'authentification sur /admin N'importe qui sur le réseau peut modifier la configuration (groupes, routing, devices). Critique en production. Note du dev : c'est normal et non critique pour le moment.

Problèmes de sécurité

Sévérité	Problème
Haute	CORS `*` dans `server/index.js` — accès depuis n'importe quel domaine
Haute	API `/admin` sans authentification
Moyenne	Clés LiveKit hardcodées en fallback `'devkey'/'secret'`

Note du dev : l'app a pour vocation à être utilisée sur un réseau local.

Qualité du code

Points positifs

Architecture modulaire solide
EventEmitter bien utilisé pour la réactivité et le hot-reload
Gestion d'erreurs gracieuse (fallback sans crash si pas de carte son)
OpusCodec robuste avec presets configurables
JitterBuffer avec stats adaptatives

Points faibles

Logging DEBUG non retiré dans server/bridge/LiveKitClient.js:93
Device IDs hardcodés dans config.yaml (inputDeviceId: 4, outputDeviceId: 0) — non portable
Création de Float32Array à chaque frame audio → pression GC potentielle à 30+ clients

État des phases

Phase	Avancement	Bloquant
Phase 1 MVP	~80%	Bridge audio serveur incomplet
Phase 2 Fonctionnalités	~95%	Authentification manquante
Phase 3 Intégrations	~85%	Tests réels manquants

Recommandations par priorité

Priorité 1 — Bloquant

Implémenter la connexion AudioBridge → LiveKitClient (TODOs lignes 368/439)
Ajouter authentification sur /admin (token Bearer ou session)
Supprimer ou intégrer LiveKitServerBridge.js

Priorité 2 — Important

CORS : remplacer * par origin explicite du client
Retirer les console.log DEBUG de LiveKitClient.js
Device IDs : auto-détection plutôt que valeurs hardcodées

Priorité 3 — Amélioration

Pool de buffers audio pré-alloués pour tenir 30+ clients
Mettre à jour CLAUDE.md avec la structure réelle du code
Tests d'intégration E2E (latence mesurée, scénario multi-clients)

3.2 KiB Raw Permalink Blame History