Skip to content

fix(tools): tools ms-teams sync reads messages again, since Teams renamed the replychains store to replychains-2 - #587

Merged
genesiscz merged 1 commit into
masterfrom
fix/2026-10-25-teams-sync
Oct 26, 2026
Merged

genesiscz merged 1 commit into
masterfrom
fix/2026-10-25-teams-sync

Conversation

@genesiscz

Copy link
Copy Markdown
Owner

Teams renamed the IndexedDB store replychains to replychains-2, so tools ms-teams sync read no messages. The ingest and the native IndexedDB dump now read either store.

  • 1396120 fix(tools): tools ms-teams sync reads messages again, since Teams renamed 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

Gates on this branch (clean worktree, 2026-10-25): bun scripts/git/check-commits.ts, root tsc --noEmit, biome on the changed files, the related tests, and scripts/ci/placeholder-check.sh. Tests: src/ms-teams/lib/decode.test.ts 7 pass.

Split out of the unified branch (#507); each commit is a cherry-pick of its unified original.

@eve-bot-lovinka

eve-bot-lovinka Bot commented Oct 25, 2026 •

Copy link
Copy Markdown

🐉 eve review — ✅ APPROVE · 0 findings

→ review · run

  • Queued 00:45:05Z
  • Reading diff — 5 files
  • Building repo map
  • Analyzing (find → verify) — 1 candidates → 0 survivors · reviewed by find codex/gpt-6.1-sol, verify codex/gpt-6-luna
  • Posting review
  • Review posted 00:51:06Z (6m 1s)

run wrun_01M5TTP70H0D17KXERSWETEBKN

Previous runs (1)
run head outcome findings took
1 1396120 ✅ APPROVE 0 7m 58s

@coderabbitai

coderabbitai Bot commented Oct 25, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Caution

Review failed

The pull request is closed.

Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@eve-bot-lovinka eve-bot-lovinka Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🐉 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).

@genesiscz

Copy link
Copy Markdown
Owner Author

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
@genesiscz
genesiscz force-pushed the fix/2026-10-25-teams-sync branch from 1396120 to a775742 Compare October 26, 2026 00:45
@genesiscz
genesiscz merged commit a775742 into master Oct 26, 2026

@eve-bot-lovinka eve-bot-lovinka Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🐉 eve review — ✅ Approved

a775742 · 0 actionable findings · reviewed by find codex/gpt-6.1-sol, verify codex/gpt-6-luna · view run ↗

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.

1 participant