From 749076518d810148c32cc69ea1938db485285f22 Mon Sep 17 00:00:00 2001 From: Lukasz Raczylo Date: Sun, 21 Jun 2026 02:16:40 +0100 Subject: [PATCH] perf(lsp): default find_references to compact output References defaulted to one line per reference; compact (one line per file, columns collapsed) existed only as opt-in. Compact preserves every line:col pair with zero information loss while cutting roughly one line per reference for the LLM consumer. Default it to true; compact=false and the session compact_refs pref still select the verbose per-reference layout (explicit override wins). --- internal/server/handlers_lsp.go | 6 ++++-- internal/server/server.go | 2 +- 2 files changed, 5 insertions(+), 3 deletions(-) diff --git a/internal/server/handlers_lsp.go b/internal/server/handlers_lsp.go index 0ea596b..4ad211f 100644 --- a/internal/server/handlers_lsp.go +++ b/internal/server/handlers_lsp.go @@ -74,12 +74,14 @@ func (s *Server) handleLSPQuery(ctx context.Context, request mcp.CallToolRequest case "references": includeDecl := request.GetBool("include_declaration", true) - // compact: explicit call-time > session compact_refs pref > false + // compact: explicit call-time > session compact_refs pref > true. + // Compact (one line per file) is the default — it preserves every + // line:col pair while cutting ~one line per reference for the LLM consumer. var prefsCompact *bool if sp := s.sessionPrefs.Load(); sp != nil { prefsCompact = sp.CompactRefs } - compact := effectiveBool(request, "compact", prefsCompact, false) + compact := effectiveBool(request, "compact", prefsCompact, true) return s.lspReferences(ctx, file, line, col, includeDecl, compact, verbose) default: diff --git a/internal/server/server.go b/internal/server/server.go index fa9e479..dbbde9d 100644 --- a/internal/server/server.go +++ b/internal/server/server.go @@ -299,7 +299,7 @@ func (s *Server) registerTools() { mcp.Description("Include the declaration in results. Only valid for action=references (default: true)."), ), mcp.WithBoolean("compact", - mcp.Description("Compact output: one line per file with all refs in brackets. Only valid for action=references. Default: false."), + mcp.Description("Compact output: one line per file with all refs in brackets. Only valid for action=references. Default: true; set false for one line per reference."), ), mcp.WithBoolean("verbose", mcp.Description("Emit count/header preamble. Applies to all actions. Default: false."),