[agentbridge-fixer]

Copilot returns HTTP 400 tool call must have a tool call ID and function name when any tool call in the resent history lacks an ID or name. KoogConversation.assemble() copied both straight from the stream frame, so a single malformed call mid-run poisoned every later request in the conversation.

• Blank/null call IDs get a generated koog-<uuid>; blank names become unknown_tool.
• executeCall now uses the same fallback ID for the tool result as for the call.

Not reproduced locally; this hardens the history against the suspected cause. If it recurs, the provider's raw frame should be logged.

[codecov]

Codecov Report

❌ Patch coverage is 81.25000% with 3 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...fishen/agentbridge/client/koog/KoogConversation.kt	81.25%	0 Missing and 3 partials ⚠️

📢 Thoughts on this report? Let us know!

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
81.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

[github-actions]

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)

[Copilot]

🟢 Approval recommended

The normalization is narrow and preserves call/result pairing; additional regression coverage is a nonblocking improvement.

1 open issue

• Add regression tests for tool-call ID and name fallbacks · New

What changed in this PR

Hardens Koog conversation history against malformed tool-call metadata that can cause Copilot HTTP 400 responses.

Changes:

• Generates IDs for null or blank call IDs and substitutes unknown_tool for blank names.
• Uses the execution ID consistently in tool results.

File	Description
plugin-core/​src/​main/​java/​com/​github/​catatafishen/​agentbridge/​client/​koog/​KoogConversation.kt	Normalizes streamed tool-call metadata and aligns result IDs.

_{Review effort: Balanced}