• Will Chen's avatar
    Fix summarize to new chat in local-agent mode (#2294) · a34709c3
    Will Chen 提交于
    ## Summary
    - Fixed chat summarization failing in local-agent mode with "no
    technical discussion" error
    - Added `messageOverride` parameter to `handleLocalAgentStream` to pass
    transformed messages
    - When summarize intent is detected, the formatted chat content is now
    correctly passed to the local agent handler instead of relying on
    database messages
    
    Fixes #2292
    
    ## Test plan
    - [ ] In local-agent mode, trigger "Summarize to new chat" from a chat
    with technical content
    - [ ] Verify the summarization completes successfully instead of showing
    "no technical discussion" error
    - [ ] Verify normal chat functionality in local-agent mode is not
    affected
    
    🤖 Generated with [Claude Code](https://claude.com/claude-code)
    
    <!-- This is an auto-generated description by cubic. -->
    ---
    ## Summary by cubic
    Fixes "Summarize to new chat" failing in local-agent mode by passing the
    formatted chat content to the local agent instead of reading DB history,
    addressing #2292. Prevents the "no technical discussion" error while
    keeping normal chat behavior unchanged.
    
    - **Bug Fixes**
    - Added messageOverride to handleLocalAgentStream to use transformed
    messages.
    - chat_stream_handlers passes chatMessages on summarize intent;
    otherwise uses DB-derived history.
    - Added Playwright E2E test to verify summarize-to-new-chat works in
    local-agent mode.
    
    <sup>Written for commit c88a646b22127a655b8e96f77956e3cbd01f4a45.
    Summary will update on new commits.</sup>
    
    <!-- End of auto-generated description by cubic. -->
    
    <!-- CURSOR_SUMMARY -->
    ---
    
    > [!NOTE]
    > **Fix local-agent summarization**
    > 
    > - Detects summarize intent and passes formatted `chatMessages` via
    `messageOverride` to `handleLocalAgentStream`; handler now prefers
    overrides over DB-derived history
    > - Uses read-only system prompt in ask-mode local-agent path and wires
    both ask/local-agent paths to support overrides
    > - Adds E2E test `local_agent_summarize.spec.ts` with snapshot to
    verify summarize-to-new-chat works
    > 
    > <sup>Written by [Cursor
    Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
    c88a646b22127a655b8e96f77956e3cbd01f4a45. This will update automatically
    on new commits. Configure
    [here](https://cursor.com/dashboard?tab=bugbot).</sup>
    <!-- /CURSOR_SUMMARY -->
    
    ---------
    Co-authored-by: 's avatarClaude Opus 4.5 <noreply@anthropic.com>
    a34709c3
local_agent_summarize.spec.ts 1.6 KB