Deleting a memory with a broken stored field fails in the REST route

オープン
#539 コメント 2 件 リアクション 0 件 担当者 1 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

@Ankit874396 がすでに取り組んでいます。

2026年10月31日 から。

評価

この issue はまだ評価されていません。

説明

area: memory bug good first issue priority: P2 size: S

Noted by @hskl18 in #537. Since #535, DELETE /api/memories/{id} checks that the memory exists by reading the whole row first. If one of the row's stored fields can't be read, for example broken JSON, the check fails, and a memory that should be easy to remove can't be deleted from the app.

#537 fixed the same thing for the memory_forget tool, with a check that only asks whether the id exists. Make the route use that same existence-only check.

Done when: a test stores a memory with a broken field, and DELETE /api/memories/{id} removes it; a missing id still answers 404.

主要言語
Python
スター
716
フォーク
132
平均マージ
16時間 34分
マージ済み PR(30日)
215

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

existence-master/Sentient のほかの issue

existence-master/Sentient の issue をすべて見る

似ている issue

Python の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。