Skip to content

fix: reload chat messages when resuming an older session - #1175

Merged
catatafishen merged 1 commit into
masterfrom
fix/resume-session-shows-wrong-messages
Oct 12, 2026
Merged

catatafishen merged 1 commit into
masterfrom
fix/resume-session-shows-wrong-messages

Conversation

@agentbridge-fixer

Copy link
Copy Markdown

Selecting an older session on reconnect left the previous session's messages in the chat panel. Now the chat is cleared and restored from the newly selected session (.current-session-id).

  • AcpConnectPanel: SessionChoice.Older returns true (reset chat)
  • ChatToolWindowContent: on reconnect with that flag, clear and call restoreConversation

Not built or tested locally.

Selecting an older session on reconnect left the previous session's messages
in the chat panel. Treat it like a fresh session: clear the chat and restore
from the newly selected session.
@codecov

codecov Bot commented Oct 12, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown

IDE Compatibility Matrix

✅ pass ❌ fail 🚫 unavailable ❓ no test yet

Generated from the real-IDE integration bench (:ide-integration-tests).

Tool CL IU RD
PSI / Navigation      
search_symbols ✅ ✅ 🚫
get_file_outline ✅ ✅ ❓
find_references ✅ ✅ ❓
go_to_declaration ✅ ✅ ❓
get_documentation ✅ ✅ ❓
get_symbol_info ✅ ✅ ❓
get_class_outline ❓ ❓ ❓
Code Quality      
get_highlights ✅ ✅ ✅
get_problems ✅ ✅ ✅
get_compilation_errors ✅ ✅ ✅
get_available_actions ❓ ❓ ❓
apply_action ❓ ❓ ❓
apply_quickfix ❓ ❓ ❓
format_code ❓ ❓ ❓
optimize_imports ❓ ❓ ❓
suppress_inspection ❓ ❓ ❓
Refactoring      
get_type_hierarchy 🚫 ✅ ❓
get_call_hierarchy 🚫 ✅ ❓
find_implementations 🚫 ✅ ❓
find_super_methods ❓ ❓ ❓
refactor ❓ ❓ ❓
convert_java_to_kotlin ❓ ❓ ❓
Testing      
run_tests ❓ ❓ ❓
list_tests ❓ ❓ ❓
run_configuration ❓ ❓ ❓
get_coverage ❓ ❓ ❓
Debug      
debug_session_start ❓ ❓ ❓
breakpoint_manage ❓ ❓ ❓
debug_evaluate ❓ ❓ ❓
debug_snapshot ❓ ❓ ❓
debug_inspect_frame ❓ ❓ ❓
debug_variable_detail ❓ ❓ ❓
debug_read_console ❓ ❓ ❓
debug_step ❓ ❓ ❓
debug_run_to_line ❓ ❓ ❓
Build      
read_build_output ❓ ❓ ❓
reload_project_model ❓ ❓ ❓

Logged IDE errors (non-gating)

Scraped from the IDE log during the run. These do not fail a matrix cell (the cell reflects the tool's MCP response) — they are listed for visibility while the bench matures.

⚠️ CL — (java.lang.InterruptedException: null)
Test: com.github.catatafishen.agentbridge.integration.GetHighlightsIntegrationTest.get_highlights returns a non-error response ⏎ You can find logs and other useful info in CI artifacts under the path getHighlights ⏎ java.lang.InterruptedException ⏎ at kotlinx.coroutines.BlockingCoroutine.joinBlocking(Builders.kt:124) ⏎ at kotlinx.coroutines.BuildersKt__BuildersKt.runBlocking$BuildersKt__BuildersKt(Builders.kt:87) ⏎ at kotlinx.coroutines.BuildersKt__BuildersKt.runBlockingWithParallelismCompensation(Builders.kt:63) ⏎ at kotlinx.coroutines.BuildersKt.runBlockingWithParallelismCompensation(Unknown Source) ⏎ at kotlinx.coroutines.internal.intellij.IntellijCoroutines.runBlockingWithParallelismCompensation(intellij.kt:48)
⚠️ CL — (java.lang.IllegalStateException: There is no ProgressIndicator or Job in this thread, the current job is not cancellable.)
Test: com.github.catatafishen.agentbridge.integration.GetHighlightsIntegrationTest.get_highlights returns a non-error response ⏎ You can find logs and other useful info in CI artifacts under the path getHighlights ⏎ java.lang.IllegalStateException: There is no ProgressIndicator or Job in this thread, the current job is not cancellable. ⏎ at com.intellij.openapi.progress.CoroutinesKt.runBlockingCancellable$lambda$0(coroutines.kt:153) ⏎ at com.intellij.openapi.progress.ContextKt.prepareThreadContext$lambda$0(context.kt:93) ⏎ at com.intellij.concurrency.ThreadContext.resetThreadContext(threadContext.kt:294) ⏎ at com.intellij.openapi.progress.ContextKt.prepareThreadContext(context.kt:92) ⏎ at com.intellij.openapi.progress.CoroutinesKt.runBlockingCancellable(coroutines.kt:151)

@catatafishen
catatafishen merged commit bf89767 into master Oct 12, 2026
23 checks passed
@catatafishen
catatafishen deleted the fix/resume-session-shows-wrong-messages branch October 12, 2026 17:21

This branch was successfully deployed

1 active deployment
fuzzing — 792bbb32 Deployed Oct 12, 2026 by agentbridge-fixer[bot] via ClusterFuzzLite PR (address) #1631
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants