Repository navigation
fix(tools): tools ms-teams sync reads messages again, since Teams renamed the replychains store to replychains-2 - #587
Conversation
|
🐉 eve review — ✅ APPROVE · 0 findings
run Previous runs (1)
|
|
Caution Review failedThe pull request is closed. Finishing Touches
Comment |
There was a problem hiding this comment.
🐉 eve review — ✅ Approved
1396120· 0 actionable findings · reviewed by find codex/gpt-6.1-sol, verify codex/gpt-6-luna · view run ↗
ℹ️ Claims vs diff — 5 claims could not be checked from the diff (c1, c2, c3, c4, c5).
Review fixes 2026-10-26 01:44 from GPT-6.1 Sol judged by Claude Opus 5.5 (fable-judge)Round 1. 0 open threads (eve 0, CodeRabbit 0, humans 0). Review bodies and PR comments contain no actionable findings. CodeRabbit was rate limited; Eve approved with 0 findings. This round was the fixer's own review of the complete PR diff and relevant callers. No real defects found, so no changes or commit were needed. Verification passed: 17 focused tests, TypeScript, Biome, lint:rules, check-tool-commands, and a Python dumper fixture probe covering modern and legacy names, precedence, missing store and missing database. No GitHub posts were made. |
…amed the replychains store to replychains-2; the dump takes either name and warns about a store it cannot find instead of ingesting 0 in silence
1396120 to
a775742
Compare
There was a problem hiding this comment.
🐉 eve review — ✅ Approved
a775742· 0 actionable findings · reviewed by find codex/gpt-6.1-sol, verify codex/gpt-6-luna · view run ↗
Teams renamed the IndexedDB store
replychainstoreplychains-2, sotools ms-teams syncread no messages. The ingest and the native IndexedDB dump now read either store.Gates on this branch (clean worktree, 2026-10-25):
bun scripts/git/check-commits.ts, roottsc --noEmit, biome on the changed files, the related tests, andscripts/ci/placeholder-check.sh. Tests:src/ms-teams/lib/decode.test.ts7 pass.Split out of the unified branch (#507); each commit is a cherry-pick of its unified original.