Skip to content

fix(serve): query LSP data from daemon via UDS#91

Merged
jamestexas merged 2 commits intomainfrom
fix/lsp-uds-query
Mar 13, 2026
Merged

fix(serve): query LSP data from daemon via UDS#91
jamestexas merged 2 commits intomainfrom
fix/lsp-uds-query

Conversation

@jamestexas
Copy link
Copy Markdown
Contributor

Summary

  • LSP enrichment writes to the ley-line arena, not mache's in-memory graph. Handlers now use a single UDS connection: enrich via tool op, then query _lsp_hover/_lsp directly from the arena
  • Removes triggerLSPEnrichment (inlined into enrichAndQuery* helpers)
  • Fixes "table still missing" error after successful enrichment

Test plan

  • E2E: mache serve --stdio . β†’ get_type_info with file param returns hover data
  • task lint β€” clean
  • task test β€” all pass

πŸ€– Generated with Claude Code

jamestexas and others added 2 commits March 13, 2026 10:00
LSP enrichment writes to the ley-line arena, not mache's in-memory
graph. The handlers now use a single UDS connection: enrich via
tool op, then query _lsp_hover/_lsp directly from the arena.

Removes triggerLSPEnrichment (inlined into enrichAndQuery* helpers).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
The schemaPath condition was inverted β€” when a schema file was
provided, it ignored it and ran FCA inference from nil instead.
Now uses resolveSchema() to load the file properly.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jamestexas jamestexas merged commit 042912e into main Mar 13, 2026
12 checks passed
@jamestexas jamestexas deleted the fix/lsp-uds-query branch March 13, 2026 16:13
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