The memory_forget tool fails when the memory is deleted but its cleanup is queued
Los mantenedores suelen responder en 1 día
Evaluación
Este issue todavía no se ha evaluado.
Descripción
Follow-up to #535 (#534). Since #535, DELETE /api/memories/{id} answers success with cleanup_pending: true when the memory is gone but a cleanup step failed and was queued for the hourly upkeep. The memory_forget tool the model uses calls the same delete, but it may still raise in that case. The chat then says the forget failed, although the memory is already gone.
Fix: make memory_forget match the route. Report success when the memory row is gone, and say the cleanup will finish later if you like. Keep a plain error for a failure before anything was deleted. Look at how #535 handles it in sentient/gateway/routes/memory.py, and reuse its helper in sentient/memory/facts.py.
Done when: a test makes the cleanup fail after the row is gone, and the tool returns success; the memory isn't listed; and the queued cleanup still runs.
- Lenguaje dominante
- Python
- Estrellas
- 718
- Forks
- 135
- Merge medio
- 16 h 34 min
- PR fusionados (30 d)
- 215
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 existence-master/Sentient
-
Telegram/Discord: tapping "Local only" in the /model list should ask too, and "/model Local only no" should cancelPosiblemente ocupada @bernalalexis-try la tomó hace 15 días. Abiertoarea: channels enhancement good first issue priority: P2 size: S
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
existence-master/Sentient#457 · 4 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
area: devices good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
existence-master/Sentient#96 ·
Los mantenedores suelen responder en 1 día
-
Memory-off /v1 turns: memory_remember's answer can echo the user's name and stored memoriesPosiblemente ocupada @hskl18 la tomó hace 2 días. Abiertoarea: memory bug priority: P2 size: S
Dificultad 3/5 1-3 horas Aptitud para principiantes 42/100
existence-master/Sentient#547 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Flaky: test_voice_note_is_transcribed_and_answered misses the sendVoice reply under loadPosiblemente ocupada @PruthviRajSolase la tomó hace 2 días. Abiertoflaky
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
existence-master/Sentient#546 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Deleting a memory with a broken stored field fails in the REST routePosiblemente ocupada @Ankit874396 la tomó hace 4 días. Abiertoarea: memory bug good first issue priority: P2 size: S
existence-master/Sentient#539 · 2 comentarios · 1 asignado ·
Los mantenedores suelen responder en 1 día
Todos los issues de existence-master/Sentient
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
microsoft/onnxscript#3155 ·
Los mantenedores suelen responder en 4 días
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
bug docs severity:low
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
-
docs good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100