Skip to content

fix: preserve Telegram topic inheritance history#91

Merged
artemgetmann merged 1 commit into
mainfrom
codex/chat-worktree-20260323-1651
Mar 23, 2026
Merged

fix: preserve Telegram topic inheritance history#91
artemgetmann merged 1 commit into
mainfrom
codex/chat-worktree-20260323-1651

Conversation

@artemgetmann
Copy link
Copy Markdown
Owner

Summary

  • preserve Telegram future-thread model/thinking history by topic/thread boundary
  • use that history when seeding untouched forum topics whose topic-create snapshot was missed
  • tighten Telegram live E2E docs and forum probe sender resolution

Why

Telegram forum topic creation is not reliable enough to be the only snapshot path. Without a fallback, an older untouched topic can inherit the newest parent default the first time it is used, which is wrong.

Verification

  • pnpm test -- src/sessions/future-thread-defaults.test.ts
  • pnpm test -- src/auto-reply/reply/session.test.ts -t "historical parent defaults|native target session routing|forum topic sessions"
  • pnpm test -- extensions/telegram/src/thread-session-seeding.test.ts
  • pnpm build
  • live Telegram E2E in a forum-enabled test group:
    • topic X set to anthropic/claude-sonnet-4-6 + adaptive
    • new topic Y inherited Sonnet + adaptive
    • older untouched topic Z stayed on Codex + medium

- what: store historical future-thread defaults by topic/thread boundary, wire the fallback into Telegram session seeding, and tighten the Telegram E2E probe/docs
- why: older untouched forum topics were inheriting the newest parent model/thinking when Telegram missed topic-create seeding, which made live behavior inconsistent
- risk: future-thread defaults now carry extra session metadata, so Telegram thread inheritance and probe assumptions are the main regression surface
@artemgetmann artemgetmann force-pushed the codex/chat-worktree-20260323-1651 branch from 37ae7c7 to 30c746d Compare March 23, 2026 05:17
@artemgetmann artemgetmann merged commit ab2766c into main Mar 23, 2026
3 of 10 checks passed
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