TUI: messagesBeforeRevert/messagesFromRevert duplicate revert boundary logic
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
- 75/100
- Tipo de issue
- Refactorización
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- frontend
Línea de trabajo
Lee routes/session/index.tsx alrededor de las líneas 134–174 y sigue cómo messagesBeforeRevert y messagesFromRevert usan el límite de revert. Extrae su búsqueda compartida del límite en una función auxiliar y luego comprueba el comportamiento de la vista de sesión en los casos con y sin un mensaje de revert coincidente; el trabajo estará terminado cuando ambas funciones usen la búsqueda compartida y conserven sus resultados actuales.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Summary
routes/session/index.tsx:134-174 — messagesBeforeRevert and messagesFromRevert both search for the revert boundary — duplicate logic.
Root cause
const messagesBeforeRevert = () => {
const messageID = session()?.revert?.messageID
const index = messages().findIndex((message) => message.id === messageID)
return index === -1 ? messages() : messages().slice(0, index)
}
const messagesFromRevert = () => {
const messageID = session()?.revert?.messageID
const index = messages().findIndex((message) => message.id === messageID)
return index === -1 ? [] : messages().slice(index)
}
Both functions find the revert boundary index independently.
Suggested fix
Extract the revert boundary index lookup into a shared helper.
- Lenguaje dominante
- TypeScript
- Estrellas
- 215k
- Forks
- 28.9k
- Merge medio
- 11 h 19 min
- PR fusionados (30 d)
- 320
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 anomalyco/opencode
-
pending close triaging
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
anomalyco/opencode#57209 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
reproduced
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
anomalyco/opencode#57127 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
pending close triaging
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
anomalyco/opencode#57070 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Plan mode leave reminder becomes the final user turn after a tool-driven agent switch, models refusePosiblemente ocupada @Veld101 la tomó hace 1 día. Abiertoreproduced
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
anomalyco/opencode#57013 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
anomalyco/opencode#56891 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de anomalyco/opencode
Issues similares
-
DQ assisgnment closing at CRAbiertoenhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
equinor/prisma-decision-web#919 ·
Los mantenedores suelen responder en 1 día
-
[DC] OneLogin sync imports a partial user list when an API request failsPosiblemente ocupada @RaphaelFakhri la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
bitwarden/directory-connector#1276 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
home-assistant/installer#324 ·
Los mantenedores suelen responder en 1 día
-
Mondriaan
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
knaw-huc/textannoviz#733 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día