Fix Web UI session creation and slash command behavior#1322
Merged
Conversation
- Avoid opening the slash command picker for path-like input such as /users/alice - Close the picker once the leading slash token contains whitespace, such as /b - Preserve trailing spaces for slash-leading rich text input so picker dismissal can detect them - Keep input content when dismissing the slash picker with Escape - Add focused tests for slash picker query parsing and rich text trailing spaces
- add a Copy ID item to the session row overflow menu - copy session IDs with the shared clipboard fallback helper - add localized copy success/failure messages - document the new session menu test IDs
- register pending session creation before async model config loading - prevent remote workspace initialization from creating duplicate empty sessions - add regression coverage for concurrent create calls with the same creation key
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.
Summary
Improve Web UI session handling and chat input behavior.
Fixes #
Type and Areas
Type:
Feature / bug fix / test / docs
Areas:
Web UI, session management, chat input, navigation session menu, i18n resources, developer docs
Motivation / Impact
This improves reliability and usability in the Web UI:
Verification
pnpm run type-check:web- passedpnpm run i18n:audit- passed with 0 warningspnpm --dir src/web-ui run test:run src/infrastructure/services/flow-chat-manager/SessionModule.test.ts src/flow_chat/components/RichTextInput.test.tsx src/flow_chat/utils/slashCommand.test.ts src/flow_chat/utils/sessionOrdering.test.ts- passed, 3 test files / 34 testsReviewer Notes
This PR is scoped to Web UI behavior and supporting docs/i18n resources. The most important regression risk is session selection/creation behavior around remote workspaces, so review should pay particular attention to concurrent session creation and persisted remote workspace matching.
Checklist