Skip to content

Fix IDE tool response handling#274

Merged
ThomasK33 merged 1 commit into
mainfrom
bridge-5twy
Jun 12, 2026
Merged

Fix IDE tool response handling#274
ThomasK33 merged 1 commit into
mainfrom
bridge-5twy

Conversation

@ThomasK33

Copy link
Copy Markdown
Member

Summary

  • Update IDE diagnostic results to return grouped URI-based diagnostic payloads with editor-native ranges and severity names.
  • Stop advertising an unsupported resources capability during initialization.
  • Harden background file opens by ignoring empty text anchors, preserving focus, returning metadata from the target buffer, and serializing tool JSON compactly.

Validation

  • mise exec -- stylua --check lua/claudecode/server/init.lua lua/claudecode/tools/check_document_dirty.lua lua/claudecode/tools/get_current_selection.lua lua/claudecode/tools/get_diagnostics.lua lua/claudecode/tools/get_latest_selection.lua lua/claudecode/tools/get_open_editors.lua lua/claudecode/tools/get_workspace_folders.lua lua/claudecode/tools/open_file.lua lua/claudecode/tools/save_document.lua tests/unit/server_spec.lua tests/unit/tools/get_diagnostics_spec.lua tests/unit/tools/open_file_spec.lua
  • find lua -name '*.lua' -type f -print0 | xargs -0 -I{} mise exec -- luajit -e "assert(loadfile('{}'))"
  • mise exec -- luacheck lua/ tests/ --no-unused-args --no-max-line-length
  • mise exec -- busted -v <all test files> — 643 successes / 0 failures / 0 errors / 0 pending
  • Dogfooded in a real Neovim terminal session with an IDE protocol scenario; captured screenshot and recording artifacts for review.

Generated with mux • Model: openai:gpt-5.5 • Thinking: xhigh

Align diagnostics payloads, omit unsupported resources capability, harden background open-file metadata, and serialize tool JSON responses compactly.
@ThomasK33 ThomasK33 merged commit 80752b2 into main Jun 12, 2026
2 checks passed
@ThomasK33 ThomasK33 deleted the bridge-5twy branch June 12, 2026 08:03
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.

1 participant