fix(server): implement findSymbol endpoint via LSP workspaceSymbol#27773
fix(server): implement findSymbol endpoint via LSP workspaceSymbol#27773shafdev wants to merge 1 commit into
Conversation
The findSymbol handler was a stub returning [] unconditionally. Wire it to LSP.Service.workspaceSymbol so workspace symbol search actually queries active LSP servers. Closes anomalyco#27650
|
The following comment was made by an LLM, it may be inaccurate: I found a potential duplicate: PR #27643: "fix: implement /find/symbol endpoint to query LSP workspace symbols" This PR appears to be directly addressing the same issue - implementing the
I also found: PR #23612: "fix(opencode): add lsp sync range for roslyn and fix workspacesymbol query" This is less likely to be a duplicate but may be related to LSP workspaceSymbol handling. You should check the status of PR #27643 to determine if it's already been merged or if it's addressing the same issue. |
|
Thanks for updating your PR! It now meets our contributing guidelines. 👍 |
Issue for this PR
Closes #27650
Type of change
What does this PR do?
findSymbolinhandlers/file.tswas a stub returning[]unconditionally. Wires it toLSP.Service.workspaceSymbolso the endpoint actually queries active LSP servers. Also updates the test assertion which was asserting the stub behavior.Note: PR #27643 addresses the same issue but is submitted from the
devbranch directly. This PR uses a dedicated fix branch and includes a test update.How did you verify your code works?
bun typecheck— no new errorsbun test test/server/httpapi-file.test.ts --timeout 30000— endpoint returns 200 with array responseScreenshots / recordings
N/A
Checklist