Skip to content

fix(desktop): isolate QQ turn routing by tab#1672

Merged
esengine merged 1 commit into
esengine:mainfrom
wade19990814-hue:codex/desktop-qq-turn-routing-fix
May 24, 2026
Merged

fix(desktop): isolate QQ turn routing by tab#1672
esengine merged 1 commit into
esengine:mainfrom
wade19990814-hue:codex/desktop-qq-turn-routing-fix

Conversation

@wade19990814-hue

Copy link
Copy Markdown
Contributor

Summary

  • isolate desktop QQ reply routing and pending follow-up prompts by tab
  • keep active-tab binding semantics for future QQ ingress, while preserving final replies for older QQ-started turns already in flight
  • add a regression test for the per-tab QQ routing state

Testing

  • npm test -- --run tests/desktop-qq-turn-routing.test.ts tests/desktop-user-message.test.ts
  • npm run typecheck
  • npm run lint
  • npm run verify

Context

Fixes the actionable part of #1619. Switching the active tab is still expected to retarget later QQ messages, but an older QQ-started turn should not lose its final reply after a local turn starts in another tab, and local-only pause prompts should not leak into QQ.

@esengine esengine merged commit a6c2fb6 into esengine:main May 24, 2026
4 checks passed
@esengine

Copy link
Copy Markdown
Owner

Thanks — merged. Cleanly factored: pure routing module + per-tab Set/Map replaces the global flat fields, exactly what #1619 needed.

esengine pushed a commit that referenced this pull request May 24, 2026
…moved, persisted usage stats, plan dispatch gate

Headline themes:
- Desktop: bundle the CLI-hosted React dashboard, retire Tauri+Preact duplicate (#1418)
- Config: drop preset abstraction; flash/pro are direct model selections (#1657, #1630)
- Stats: persist cumulative usage to session meta + auto-restore on startup (#1667, #1680, #1643, #1628)
- Plans: editMode="plan" enforced at the ToolRegistry dispatch gate (#1681); step advance fix (#1629)
- Context: fold once at turn start, drop pre-flight + byte-ceiling (#1642, #1646); collapsible compacted card (#1649)
- Subagents: per-skill flash/pro override + Settings UI (#1632)
- Desktop polish: sidebar drag-resize (#1688), responsive collapse (#1585), copy/edit overlay + msg-history nav (#1645), Esc closes modal not turn (#1685), QQ tab isolation (#1672), DiffCard for edits (#1662), theme-aware highlighting (#1655), system events toggle (#1654/#1650), macOS TCC inheritance (#1614), dashboard.enabled (#1612)
- Dashboard polish: persistent session URL (#1586, #1589, #1599), theme-aware highlighting (#1664), IME confirm-enter guard (#1689), code-fence lang fix (#1677), vendor chunk split (#1587), markdown table h-scroll (#1562)
- TUI: Alt+S input stash/recall; static history isolated from input rerenders (#1635); legacy mouse drop (#1637, #1648); multi-edit gated in review (#1647)
- Diff: SplitDiff column border holds under CJK (#1686)
- MCP: workspace roots passed to servers (#1625); codeCommand honors mcpServers (#1603)
- Config plumbing: (baseUrl, apiKey) resolved as a tuple (#1658); stale model id self-heal (#1663)

See CHANGELOG for the full list.
@wade19990814-hue wade19990814-hue deleted the codex/desktop-qq-turn-routing-fix branch May 24, 2026 15:20
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.

2 participants