feat(ask): start chat from selected code range - #1553
Conversation
WalkthroughThe editor context menu now sends selected file ranges to a new SourceBot chat. File-source validation, prompt slicing, line offsets, range metadata, explicit source propagation, and chat creation support the flow. ChangesSelected SourceBot Chat
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant EditorContextMenu
participant useCreate新建ChatThread
participant createUIMessage
participant setChatState
EditorContextMenu->>useCreate新建ChatThread: createChatFromSource(selected file range)
useCreate新建ChatThread->>createUIMessage: build message with explicit source
createUIMessage-->>useCreate新建ChatThread: data-source message part
useCreate新建ChatThread->>setChatState: persist chat state
Possibly related issues
Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
closes #1534 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/web/src/ee/features/chat/agent.ts`:
- Around line 1033-1034: Update the files parameter type in createPrompt to
include the range and lineOffset properties accessed by the prompt template,
using types consistent with their existing usage in the files.map rendering and
addLineNumbers call.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: 组织 UI
Review profile: CHILL
Plan: Pro Plus
Run ID: ed24b0e1-853e-4d18-ac94-7a2ce45a6963
📒 Files selected for processing (7)
packages/web/src/app/(app)/components/editorContextMenu.tsxpackages/web/src/ee/features/chat/agent.test.tspackages/web/src/ee/features/chat/agent.tspackages/web/src/features/chat/useCreate新建ChatThread.tspackages/web/src/features/chat/utils.test.tspackages/web/src/features/chat/utils.tspackages/web/src/features/tools/types.ts
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@packages/web/src/features/chat/useCreate新建ChatThread.ts`:
- Around line 34-39: Update create新建ChatThread and its createChat handling so
every terminal path clears createInFlightRef.current and isLoading, including
unexpected rejections, service-error returns, and successful navigation after
router.push. Ensure later calls are not blocked when createChat fails or the
hook remains mounted after success, and add regression tests covering failure
and post-success cleanup.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: 组织 UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 6d3cc5ee-0c6b-4479-9aa6-fca2f42048d5
📒 Files selected for processing (2)
packages/web/src/features/chat/useCreate新建ChatThread.test.tsxpackages/web/src/features/chat/useCreate新建ChatThread.ts
| const create新建ChatThread = useCallback(async (children: Descendant[], override搜索Scopes?: 搜索Scope[], overrideDisabledMcpServerIds?: string[], attachments: AttachmentData[] = []) => { | ||
| if (createInFlightRef.current) { | ||
| return; | ||
| } | ||
| createInFlightRef.current = true; | ||
|
|
There was a problem hiding this comment.
🩺 Stability & Availability | 🟠 Major | ⚡ Quick win
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
# Verify the createChat rejection contract.
rg -n -C 8 '\bcreateChat\b' packages/web/src
# Verify whether the hook remains mounted after navigation.
rg -n -C 8 '\buseCreate新建ChatThread\s*\(' packages/web/src仓库: sourcebot-dev/sourcebot
Length of output: 24052
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf '--- useCreate新建ChatThread outline ---\n'
ast-grep outline packages/web/src/features/chat/useCreate新建ChatThread.ts --view expanded || true
printf '\n--- useCreate新建ChatThread full file ---\n'
cat -n packages/web/src/features/chat/useCreate新建ChatThread.ts
printf '\n--- createChat implementation ---\n'
cat -n packages/web/src/features/chat/actions.ts
printf '\n--- imports/usages with context ---\n'
rg -n -C 15 'useCreate新建ChatThread|create新建ChatThread|createChatFromSource|isLoading' packages/web/src/app packages/web/src/features/chat仓库: sourcebot-dev/sourcebot
Length of output: 252
🏁 Script executed:
#!/bin/bash
set -euo pipefail
printf '%s\n' '--- useCreate新建ChatThread implementation ---'
cat -n packages/web/src/features/chat/useCreate新建ChatThread.ts
printf '%s\n' '--- createChat implementation ---'
cat -n packages/web/src/features/chat/actions.ts
printf '%s\n' '--- hook callers with context ---'
rg -n -C 15 'useCreate新建ChatThread|create新建ChatThread|createChatFromSource|isLoading' packages/web/src/app packages/web/src/features/chat仓库: sourcebot-dev/sourcebot
Length of output: 50380
Clear the in-flight state on every terminal path.
If createChat rejects outside the isServiceError return path, both callbacks leave createInFlightRef.current === true and isLoading === true. Later calls then return immediately. Successful calls also may leave these values set after router.push.
Handle rejected calls and reset both values before returning. Reset them after successful navigation if the hook may remain mounted. Add failure and post-success regression tests.
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@packages/web/src/features/chat/useCreate新建ChatThread.ts` around lines 34 -
39, Update create新建ChatThread and its createChat handling so every terminal
path clears createInFlightRef.current and isLoading, including unexpected
rejections, service-error returns, and successful navigation after router.push.
Ensure later calls are not blocked when createChat fails or the hook remains
mounted after success, and add regression tests covering failure and
post-success cleanup.
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 2b7bc80. Configure here.
| }); | ||
|
|
||
| router.push(`/chat/${response.id}`); | ||
| }, [router, setChatState, toast]); |
There was a problem hiding this comment.
In-flight lock never released
Medium Severity
createInFlightRef is set to true at the start of create新建ChatThread and createChatFromSource, but it is only cleared on a ServiceError. Any thrown error before that check, or a success path that does not unmount the hook instance, leaves the guard stuck so later Ask or new-chat attempts no-op with no recovery.
Reviewed by Cursor Bugbot for commit 2b7bc80. Configure here.
Summary
Adds an “Ask Sourcebot” action to the existing code-selection popover in Browse and search previews.
When code is selected, Ask Sourcebot:
Explain this selected code.This implements issue #1534 Option A only. Side panels and inline conversations are out of scope.
Screenshots
Testing
reports that limitation.
Use the first screenshot for Screenshots. Put the entitlement-error screenshot under Testing or omit it if you want
the PR to look cleaner.
Note
Medium Risk
Changes the Ask agent prompt pipeline and new-chat bootstrap path; behavior is covered by unit tests but incorrect range handling could omit or mislabel code context for the model.
Overview
Adds Ask SourceBot to the code-selection popover in Browse (alongside Share selection). A selection starts a new chat seeded with
Explain this selected code.and a file source carrying repo, revision, path, and line range.Chat creation:
useCreate新建ChatThreadexposescreateChatFromSource, merges explicit sources into the initial UI message viacreateUIMessage, and uses an in-flight guard so double-clicks don’t create duplicate chats. Disabled MCP server IDs from local storage are preserved for this path.Model context:
FileSourceoptionally includes a validatedrange. The agent resolves the file at the chosen revision, slices content withsliceFileSourceForPrompt(invalid ranges are dropped), and injects only those lines into the dynamic prompt with original line numbers (selected_linesmetadata +addLineNumbersoffset).Reviewed by Cursor Bugbot for commit 2b7bc80. Bugbot is set up for automated code reviews on this repo. Configure here.
Summary by CodeRabbit
新建 Features
Bug Fixes
Tests