feat(webchat): add new chat session button#52042
feat(webchat): add new chat session button#52042bobashopcashier wants to merge 28 commits intoopenclaw:mainfrom
Conversation
Greptile SummaryThis PR adds a Key observations:
Confidence Score: 4/5
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e7c255e997
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: a96c8938d7
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@greptile review |
|
@codex review |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
6b6337d to
fee4c6f
Compare
|
@greptile review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 0e022dac2c
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@greptile review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: b09094f4db
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@greptile review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: f6cdb38b9f
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@greptile review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9d9034f233
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
@greptile review |
|
@greptile review |
9d65eca to
73a067f
Compare
|
@BunsDev Hey tysm for looking at this PR! Can you look over it again. I have fixed the things addressed. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 73a067fb6b
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 62a04f3861
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| canCreateSession: | ||
| state.connected && !state.chatRunId && !state.newChatSessionPending, | ||
| onCreateSession: () => void createNewChatSession(state), |
There was a problem hiding this comment.
Hide new-session action during onboarding
When state.onboarding is true, ui/src/ui/app-render.ts:306 forces chat focus mode and the normal header controls are hidden, so these props are what make the floating + action appear on the onboarding chat. Other chat controls are explicitly locked down in onboarding (ui/src/ui/app-render.helpers.ts:231-360), but this still allows createNewChatSession(), which can switch the user into a blank dashboard session mid-onboarding and pull them away from the guided conversation they were supposed to continue.
Useful? React with 👍 / 👎.
|
Closing due to high volume of changes and breaking changes upstream. |

Summary
Describe the problem and fix in 2–5 bullets:
/new, which only resets the current one.+affordance beside the session selector that creates a new optional labeled dashboard session and auto-switches only when the originating session and composer state are still safe to move.Change Type (select all)
Scope (select all touched areas)
Linked Issue/PR
User-visible / Behavior Changes
+button beside the chat session selector.+prompts for an optional label and creates a fresh dashboard session.Security Impact (required)
Yes/No)NoYes/No)NoYes/No)NoYes/No)NoYes/No)NoYes, explain risk + mitigation:Repro + Verification
Environment
DarwinNode v25.8.1,pnpm 10.23.0Steps
pnpm test -- ui/src/ui/controllers/chat.test.ts,pnpm test -- ui/src/ui/controllers/sessions.test.ts, andpnpm test -- ui/src/ui/views/chat.test.ts.pnpm build, plus the targeted Telegram/gateway/infra/helper expectation-alignment checks touched after mergingmain.Expected
+are preserved by staying in the current session when switching would be unsafe.mainmerge and do not expand the user-visible feature scope.Actual
Evidence
Attach at least one:
Human Verification (required)
What you personally verified (not just CI), and how:
chat.historyresponses, successful session creation, missing-key create responses, thrownsessions.createerrors, delayed create responses after a user session switch, delayed create responses after draft or attachment edits, delayed create responses with queued prompts already present, and refresh retries whilesessions.listis already in flight.Review Conversations
If a bot review conversation is addressed by this PR, resolve that conversation yourself. Do not leave bot review conversation cleanup for maintainers.
Compatibility / Migration
Yes/No)YesYes/No)NoYes/No)NoFailure Recovery (if this breaks)
ui/src/ui/app-render.helpers.ts,ui/src/ui/controllers/chat.ts, and the related webchat tests.Risks and Mitigations
List only real risks for this PR. Add/remove entries as needed. If none, write
None.sessions.createresponses could still leave the UI briefly out of sync with backend session state.mainmerge and are not treated as separate user-facing behavior in this PR.