flow-runtime: prune empty per-target sets from the liveWriters registry
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
- 78/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- backend
Línea de trabajo
La fuga está en packages/flow-runtime/src/provider.ts, en liveWriters/writersOf y SignalWriter.retire(). Lee cómo se llenan el Map por bridge y sus Sets, luego confirma que retire() deja el Set vacío. Haz que retire() elimine la entrada del mapa una vez que su Set esté vacío, y añade una prueba de que el registro de un bridge queda vacío después de cerrarlo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Raised by the Claude review of #80.
packages/flow-runtime/src/provider.ts (liveWriters / writersOf, SignalWriter.retire()): retire() deletes the writer from its target's Set but never removes the empty Set from the per-bridge Map. On a long-lived shared bridge whose documents keep changing signal.out targets (e.g. an editor session redeploying for hours), the map grows by one entry per distinct (connection, source) ever written. Bounded by distinct targets, so P3.
Fix: in retire(), after peers.delete(this), drop the map entry when the set is empty (keep the key string on the writer). Add a test that the registry for a bridge is empty after close.
🤖 Generated with Claude Code
- Lenguaje dominante
- TypeScript
- Estrellas
- 0
- Forks
- 0
- Merge medio
- 10 h 39 min
- PR fusionados (30 d)
- 38
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 hyperdrive-technology/holodeck
-
area:holodeck review-finding
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
hyperdrive-technology/holodeck#201 ·
Los mantenedores suelen responder en 1 día
-
area:holodeck review-finding
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
hyperdrive-technology/holodeck#196 ·
Los mantenedores suelen responder en 1 día
-
area:docs review-finding
Dificultad 1/5 Menos de una hora Aptitud para principiantes 70/100
hyperdrive-technology/holodeck#180 ·
Los mantenedores suelen responder en 1 día
-
Flaky E3 test: native cut with no keydown (holodeck#123) in editing-ux-render.test.tsPosiblemente ocupada @dezlitz la tomó hace 2 días. Abiertoarea:holodeck review-finding
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
hyperdrive-technology/holodeck#141 ·
Los mantenedores suelen responder en 1 día
-
assetUrlProblem: data:image check uses the un-normalised URL, so ' data:image/png...' is rejectedAbiertoarea:holodeck review-finding
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
hyperdrive-technology/holodeck#104 ·
Los mantenedores suelen responder en 1 día
Todos los issues de hyperdrive-technology/holodeck
Issues similares
-
Area/Cloud Needs Triage Type/Bug
Dificultad 1/5 1-3 horas Aptitud para principiantes 86/100
wso2/product-integrator#2847 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 67/100
bluesky-social/social-app#12118 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
heygen-com/hyperframes#6021 ·
Los mantenedores suelen responder en 1 día
-
Edit: Tele Posadas HDAbiertocheck:passed streams:edit
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día