Memory erase swallows cleanup errors with no retry, so derived data can be left behind
Maintainer thường phản hồi trong vòng 1 ngày
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 45/100
Hướng nghiên cứu
Start with erase() in sentient/memory/facts.py around lines 953–955, then find the hourly memory upkeep entry point and its tests. Trace the listed cleanup steps and how upkeep is scheduled before choosing where failed work should be recorded. Done means a test makes one cleanup step fail once and confirms the next upkeep run removes the derived data.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Raised by CodeRabbit on #453, but it predates that PR. erase() in sentient/memory/facts.py (~953-955) catches and drops errors from its cleanup steps (vectors, links, summaries, MEMORY.md/USER.md, insight evidence) with no retry. If one step fails (a locked file, a transient DB error), the memory row is gone but some derived data stays, and nothing ever tries again. This affects manual deletes and #441's expiry.
Fix: record failed cleanup steps (for example a small pending_erase table, or a retry list that the hourly memory upkeep drains) and retry them until they succeed. Log them plainly.
Done when: a test that makes one cleanup step fail once shows the derived data is removed on the next upkeep run.
- Ngôn ngữ chính
- Python
- Star
- 710
- Fork
- 125
- Merge trung bình
- 8 giờ 24 phút
- Pull request đã merge (30 ngày)
- 183
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của existence-master/Sentient
-
Flaky test: test_interrupt_stops_audio_but_text_completes fails under loadCó thể đã có người làm @ShryukGrandhi đã nhận hôm nay. Đang mởarea: voice bug flaky priority: P2 size: S
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
existence-master/Sentient#490 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Telegram/Discord: tapping "Local only" in the /model list should ask too, and "/model Local only no" should cancelCó thể đã có người làm @bernalalexis-try đã nhận 3 ngày trước. Đang mởarea: channels enhancement good first issue priority: P2 size: S
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
existence-master/Sentient#457 · 4 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Settings label reads "Claude code max concurrent" (generated from the config key)Có thể đã có người làm @sharmis-tp đã nhận 1 ngày trước. Đang mởarea: desktop bug good first issue priority: P2 size: S
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
existence-master/Sentient#454 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
A test or demo data folder offers the real Documents folder as its default backup locationCó thể đã có người làm @bercedev đã nhận 1 ngày trước. Đang mởarea: desktop bug priority: P2 size: S
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
existence-master/Sentient#440 · 2 bình luận · 1 người được giao ·
Maintainer thường phản hồi trong vòng 1 ngày
-
area: desktop bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
existence-master/Sentient#426 ·
Maintainer thường phản hồi trong vòng 1 ngày
Tất cả issue của existence-master/Sentient
Issue tương tự
-
Python Volcengine provider returns failures instead of raising themCó thể đã có người làm @Ricardo-M-L đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 79/100
agent-infra/sandbox#248 ·
-
Textractor leaves empty temporary files when remote retrieval failsCó thể đã có người làm @HuaTNA đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
Maintainer thường phản hồi trong vòng 1 ngày
-
test(bm_generation): pin the learning parameters and CPU sync shortcutCó thể đã có người làm @iloveeyjafjalla đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
qboson/kaiwu-pytorch-plugin#1341 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
elder-plinius/V3R1T4S#625 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
foxglove/mcap#1907 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày