Fix stale test references in the task-execution and runtime READMEs
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
- 84/100
- Tipo de issue
- Documentación
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- documentation
Línea de trabajo
Lee packages/junior/src/chat/task-execution/README.md y packages/junior/src/chat/runtime/README.md; después, inspecciona durable-queue.test.ts y las ubicaciones actuales de las pruebas de integración. Actualiza o elimina los puntos obsoletos sobre resultados y la ruta de cobertura de runtime; comprueba #2001, porque el traslado de su prueba puede cambiar las referencias. La tarea está terminada cuando ambos READMEs solo apunten a pruebas existentes y relevantes, o cuando omitan las referencias obsoletas.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Two READMEs point to tests that no longer exist.
packages/junior/src/chat/task-execution/README.md
The section about durable-queue.test.ts lists the outcomes that the file covers. Two of them are not in the file:
- "Transient agent failure: failure before input commit retries without a Slack post; the retry limit stops with at most one fallback."
- "Mid-turn steer: a second
@mention folds into the active turn and still produces one final reply."
3a5be6f12 (#1571) removed both cases. durable-queue.test.ts now has six cases, and none of them covers these outcomes. #1398 still lists both as outcomes that durable-queue should own.
packages/junior/src/chat/runtime/README.md
Line 57 says: "Integration coverage lives under packages/junior/tests/integration/runtime/." That directory does not exist.
Change
- Point each bullet to the test that covers the outcome now, or remove the bullet.
- Fix or remove the runtime coverage path.
#2001 moves the tests that run the agent into packages/junior-evals. Check these references again when that work moves the tests.
Found in the test audit for #2001.
- Lenguaje dominante
- TypeScript
- Estrellas
- 362
- Forks
- 38
- Merge medio
- 6 h 39 min
- PR fusionados (30 d)
- 175
Preparar el entorno
- Incluye un Dockerfile o un 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 getsentry/junior
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
getsentry/junior#335 · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Rethink how automations executeAbiertotask
Dificultad 5/5 Más de una semana Aptitud para principiantes 28/100
getsentry/junior#2051 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de getsentry/junior
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
kind/chore priority/must
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
sidereal-io/sidereal#380 ·
Los mantenedores suelen responder en 1 día
-
Mend: dependency security vulnerability
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
opfab/operatorfabric-core#10653 ·
Los mantenedores suelen responder en 1 día
-
backend bug size:sm
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
chrisbenincasa/tunarr#2237 ·
Los mantenedores suelen responder en 1 día
-
documentation
Dificultad 2/5 Medio día Aptitud para principiantes 69/100
Lam30ne/regulate-app#39 ·