Repository navigation
Share one chat session between the AI Mode overlay and the notebook panel - #1117
Merged
Merged
Conversation
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
Contributor
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
yattias
changed the base branch from
main
to
ai-mode-workspace/detail-blocks
September 22, 2026 09:07
This was referenced Sep 22, 2026
yattias
changed the title
This was referenced Sep 22, 2026
This was referenced Oct 5, 2026
yattias
added this pull request to stack #1146
October 6, 2026 20:45
…anel useAIModeChat and AgentChatPanel each carried a copy of the same machinery: create the chat on the first send, queue that first message until the chat is live, keep a draft per chat across switches and failed sends, wire model selection, tell the listing when a turn settles, and word the composer's failure notices. hooks/useChatSession now holds it once, keyed by `transport:chat`, and both surfaces sit on top of it; components/AgentChat/chatNotices.ts holds the copy, with a policy for what a surface calls a chat and who announces a spent budget. Along the way: the queued first message is bound to the chat it was queued for instead of firing on whichever chat is selected next; the "new" draft no longer collides across scopes; the notebook composer now clears on send and restores the text on failure, as the overlay always did; and the overlay's composer takes the panel's fuller busy/stop rules. Transports are handed out by getChatTransport so a scope always gets the same object, and createChat takes a ChatCreateInit ready for the fields a workspace conversation starts with. Verified with type-check, eslint and prettier only. Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
yattias
force-pushed
the
ai-mode-workspace/detail-blocks
branch
from
October 8, 2026 14:46
fcbb843 to
ddac2b8
Compare
yattias
force-pushed
the
ai-mode-workspace/chat-session
branch
from
October 8, 2026 14:46
d13aa1e to
39ac97b
Compare
|
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stack
Part of turning the AI Mode overlay into a funding workspace (plan and mock decisions in the first PR's thread). Stacked on #1118; merge that first, then this one.
🤖 Generated with Claude Code