Approving a Dot's revision creates a duplicate page; edits to existing pages skip review
Los mantenedores suelen responder en 2 días
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 64/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- backend
Línea de trabajo
Start in src/shared/page-review.ts and trace the review_space_page and edit_space_page entry points. Check how review cards carry page identity and how the existing revision check is applied. Done means approving a review for an existing page updates it in place without creating a duplicate, while new-page reviews still create pages.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
review_space_page only accepts {title, content, spaceId} (src/shared/page-review.ts), so approving a review card always creates a new page. Changes to existing pages go through edit_space_page, which has no human-review step.
How we hit it: in a page's chat, asked a second Dot to revise the page and "show the draft for review before saving over the page". After Approve & save, the Space contained two pages with the same title; the original was untouched and nothing links the two.
Expected: review cards can target an existing page (optional pageId) and update it in place with the usual revision check.
Env: Docker Compose (compose.yml + computers overlays), main @ c2569bb, OPENAI_MODEL=gpt-5.4-mini, Chromium.
- Lenguaje dominante
- TypeScript
- Estrellas
- 2.8k
- Forks
- 344
- Merge medio
- 1 d 8 h
- PR fusionados (30 d)
- 33
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 CopilotKit/OpenDots
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
CopilotKit/OpenDots#69 ·
Los mantenedores suelen responder en 2 días
-
Long page titles are clipped in the page editorPosiblemente ocupada @charan-rathore la tomó hace 2 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
CopilotKit/OpenDots#68 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
CopilotKit/OpenDots#100 ·
Los mantenedores suelen responder en 2 días
-
Plugins and wider controlAbierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
CopilotKit/OpenDots#88 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
CopilotKit/OpenDots#87 · 1 reacción ·
Los mantenedores suelen responder en 2 días
Todos los issues de CopilotKit/OpenDots
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
umbraco/Umbraco-CMS-MCP-Dev#512 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
wimpysworld/sidra#290 ·
Los mantenedores suelen responder en 1 día
-
defuFn invokes function values for inherited default propertiesPosiblemente ocupada @xiehuanyi la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
feature request good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
TabularisDB/tabularis#853 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 Menos de una hora Aptitud para principiantes 85/100
capricorn86/happy-dom#2474 ·
Los mantenedores suelen responder en 2 días