Skip to content

fix(memory): preserve valid traces with unresolved sources - #264

Merged
lipluscodex merged 1 commit into
mainfrom
issue-263-partial-memory
Oct 2, 2026
Merged

lipluscodex merged 1 commit into
mainfrom
issue-263-partial-memory

Conversation

@lipluscodex

Copy link
Copy Markdown
Member

Closes #263
未解決の資料が混ざった検索でも、正常資料の履歴と利用記録を保存する。部分記録の状態・除外件数・理由を返し、履歴のsettingsに残す。検索結果と順序は維持し、全件未解決と正常0件を区別する。
実経路の起点・中間・終点を検証して、不明なgraph pathを強化対象から除外する。旧D1行のID欠落原因を確認し、再現したissue_comment.deletedの部分削除でcanonicalを保持する修正を加えた。

検証: unit/Workers 410件、bridge 33件、TypeScript型検査、Worker/bridge schema一致が成功。

原因調査・証拠・限界: 調査記録。旧FTSのID未保存は確定。残存1件の本文は現在の公開コメントと異なるが、欠落の原因と削除経路との因果は未確定。削除済みとは断定しない。

本番DB変更・一括補修再実行・版番号更新・release作成は含まない。削除失敗は503で明示し、自動再試行は追加していない。

識別できない返却資料だけを除外し、正常資料を原子的に保存する。部分記録の状態・件数・理由を応答と履歴settingsに残し、全件未解決と正常0件を区別する。

graphの実経路nodeを検証し、不明経路を強化対象から除外する。issue_comment削除で索引片側の失敗後にcanonicalを消す欠陥を修正し、503と明示的な再配送を確認する。旧D1のID欠落原因と残存1件の未確定な起源を調査記録に分離した。

検証: unit/Workers 410件、bridge 33件、型検査、schema一致、diff whitespace成功。
Refs #263
@lipluscodex lipluscodex self-assigned this Oct 2, 2026
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
github-rag-mcp 0392389 Oct 02 2026, 12:19 PM

@lipluscodex lipluscodex left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lin: 自己レビュー PASS。対象head: 0392389。

Issue #263本文と18ファイルの変更を照合した。識別失敗だけを資料単位で除外し、正常sourceは従来の原子的saveへ渡す。results/graph_results/same_entity.othersを維持し、除外位置・理由と部分記録をsettingsへ保存する。全件未解決はtrace無し、正常0件は空trace。予期しない処理例外・retrieval不完全・save失敗は全体feedback不可のまま。

実際の経路node列から、返却枠外の祖先も含めて最大3node/pathの索引metadataを確認する。検証できないpathだけ保存snapshotで空にし、正常terminalの利用を維持する。正常pathの確認receiptと取消しが回帰で確認される。内部snapshotを返却・settingsに保存しない。

再現したissue_comment.deletedの部分削除について、Vectorize/FTSが片側失敗したらDO canonicalを保持して503を返す。DOの非2xx/例外も成功扱いにしない。署名付きsynthetic回帰で各失敗、正常削除、明示的再配送を検証済み。自動再試行の保証は加えていない。

調査記録は旧FTSのevent ID未保存をGit履歴から確定し、残存1件の本文・時刻の相違と因果未確定を区別する。IDの推測や本番データ削除は行っていない。狭い削除経路修正は更新済みIssueの要求内で、逸脱なし。README/要求仕様/英日memory仕様と生成JSONも整合している。

exact headのCI test/CI/Workers Buildsすべてsuccess(CI run 37006017149)。ローカル410件とbridge33件、型検査、schema drift成功。既存APIと保存形式を維持する小さな不具合修正としてpatch相当。USER_REPO2 autoのため人のPR確認ゲートなし。mergeable確認後にsquash mergeし、mainのWorkers Buildsと限定2件fetch/historyで本番反映を確認する。版番号更新・release作成はこの変更には含めない。

@lipluscodex
lipluscodex merged commit ff87b94 into main Oct 2, 2026
3 checks passed
@lipluscodex
lipluscodex deleted the issue-263-partial-memory branch October 2, 2026 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(memory): preserve valid sources when unresolved identity is present

1 participant