feat: schema-driven LSP materialization with file_sets and include#81
Merged
jamestexas merged 2 commits intomainfrom Mar 12, 2026
Merged
feat: schema-driven LSP materialization with file_sets and include#81jamestexas merged 2 commits intomainfrom
jamestexas merged 2 commits intomainfrom
Conversation
Replace hardcoded materializeLSP() with schema-driven materializeContentSources(). LSP files (hover, diagnostics, defs, refs) are now declared in the topology schema via content_source on Leaf, not hardcoded in Go. Add file_sets/include to reduce duplication: define reusable groups of file definitions at the schema level, reference them from any node. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…ames instead of IDs Three bugs found during E2E dogfooding (mache build → leyline lsp → mache serve → MCP): 1. buildServeGraph skipped materializeVirtuals for .db files, so content_source files (hover, diagnostics) were never created in the serve path 2. SQLiteGraph.ListChildren nodes-table path returned name instead of id, breaking GetNode lookups downstream (all children silently invisible) 3. list_directory duplicated callers/callees when both materialized in db and injected as virtual entries Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
materializeLSP()with schema-drivenmaterializeContentSources()— LSP files are now declared viacontent_sourceonLeaf, not hardcoded in Gofile_sets/includetoTopology/Nodefor reusable file set groups — eliminates duplication across construct typeslsp_hover,lsp_diagnostics,lsp_defs,lsp_refsTest plan
task test)task install)leyline lsp→ enriched .db →mache servewith updated schema🤖 Generated with Claude Code