fix(mcp,cli): reduce redundancy on memory tree/fetch output formats (#9)

## Summary

Reduce redundancy on `MemoryFetch`, `MemoryTree` and `remindb inspect
--tree` output formats.

## Self-review

### Verified

- [x] Tested manually via CLI or local MCP plugin install
- [x] Ran the test suite with fuzzing (`make test-all`, `make fuzz`)

### Touched

- [x] **MCP tools or public skills** — code and skill stay in sync
(`skills/remind/`, `skills/memoize/`)
- [ ] **Temperature config** — both public skills reflect new values
- [ ] **Parser** — fuzz target covers the change
- [ ] **Schema or migrations** — FTS5 triggers in sync
- [ ] **Write path** — each write tool call still produces exactly one
snapshot
- [ ] **Client output format** — unchanged, or breaking change noted in
summary
- [x] **CLI** — user-facing changes documented in summary

Author: radimsem

README.md

@@ -42,15 +42,15 @@ A real slice (from `remindb inspect --tree`):
 ```
 [preamble] Preamble: framework, language, project (id=3kGXxidmWBp file=CLAUDE.md temp=0.50 tok=14)
 [heading] Project Instructions (id=6EuIVj5zt5j file=CLAUDE.md temp=0.75 tok=5)
-  [heading] Architecture (id=603qfsg4qd2 file=CLAUDE.md temp=0.88 tok=3)
-    [text] Next.js 15 conventions with a clear separation of data… (id=3GGuLAq3yNP file=CLAUDE.md temp=0.82 tok=111)
-    [list] 7-item list: app/, components/, lib/, db/, hooks/, types… (id=ITAKw5NVNPt file=CLAUDE.md temp=0.71 tok=228)
-  [heading] Data Model (id=FQwpXL4bm6Y file=CLAUDE.md temp=0.62 tok=3)
-    [list] 7-item list: products, variants, orders, carts, users, s… (id=Il8jcgTJOGt file=CLAUDE.md temp=0.55 tok=155)
-  [heading] Payment Integration (id=LTQZLSkPsDW file=CLAUDE.md temp=0.30 tok=5)
-    [text] Stripe Payment Intents; not legacy Checkout Sessions… (id=GLbXrUYs32G file=CLAUDE.md temp=0.24 tok=35)
-  [heading] Observability (id=2wkOdf47OjR file=CLAUDE.md temp=0.08 tok=4)
-    [list] 4-item list: Sentry · Vercel logs · OTel tracing · Prom… (id=C1HCYSAOkpu file=CLAUDE.md temp=0.08 tok=90)
+  [heading] Architecture (id=603qfsg4qd2 temp=0.88 tok=3)
+    [text] Next.js 15 conventions with a clear separation of data… (id=3GGuLAq3yNP temp=0.82 tok=111)
+    [list] 7-item list: app/, components/, lib/, db/, hooks/, types… (id=ITAKw5NVNPt temp=0.71 tok=228)
+  [heading] Data Model (id=FQwpXL4bm6Y temp=0.62 tok=3)
+    [list] 7-item list: products, variants, orders, carts, users, s… (id=Il8jcgTJOGt temp=0.55 tok=155)
+  [heading] Payment Integration (id=LTQZLSkPsDW temp=0.30 tok=5)
+    [text] Stripe Payment Intents; not legacy Checkout Sessions… (id=GLbXrUYs32G temp=0.24 tok=35)
+  [heading] Observability (id=2wkOdf47OjR temp=0.08 tok=4)
+    [list] 4-item list: Sentry · Vercel logs · OTel tracing · Prom… (id=C1HCYSAOkpu temp=0.08 tok=90)
 ```
 
 A fresh compile starts every node at `temp=0.50`. The spread above is what an agent sees after a few sessions of reading. *Architecture* is hot because the agent keeps coming back to it. *Observability* has gone cold and will get flagged for summarization on the next nudge.

cmd/remindb/inspect.go

@@ -5,6 +5,7 @@ import (
 	"fmt"
 	"io"
 	"os"
+	"path/filepath"
 	"strings"
 	"unicode/utf8"
 
@@ -89,8 +90,10 @@ func runInspect(cmd *cobra.Command, _ []string) error {
 	_, _ = fmt.Fprintln(w, paint(ansiBold+ansiCyan, "=== Node Tree ==="))
 	roots, childMap := store.BuildTree(all)
 
+	compileRoot, _ := st.GetLatestCompileRoot(ctx)
+
 	for _, root := range roots {
-		printTree(w, childMap, root, 0, inspectTreeDepth)
+		printTree(w, childMap, root, "", compileRoot, 0, inspectTreeDepth)
 	}
 	return nil
 }
@@ -121,15 +124,21 @@ func printStats(w io.Writer, s *store.Stats) {
 	_, _ = fmt.Fprintln(w)
 }
 
-func printTree(w io.Writer, children map[string][]*store.Node, n *store.Node, depth, maxDepth int) {
+func printTree(w io.Writer, children map[string][]*store.Node, n *store.Node, parentSource, compileRoot string, depth, maxDepth int) {
 	indent := strings.Repeat("  ", depth)
 
-	_, _ = fmt.Fprintf(w, "%s%s %s (%s %s %s %s)\n",
+	_, _ = fmt.Fprintf(w, "%s%s %s (%s",
 		indent,
 		paint(ansiYellow, "["+n.NodeType+"]"),
 		paint(ansiBrightW, n.Label),
 		paint(ansiDim, "id="+n.ID),
-		paint(ansiDim, "file="+n.SourceFile),
+	)
+
+	if n.SourceFile != parentSource {
+		_, _ = fmt.Fprintf(w, " %s", paint(ansiDim, "file="+relSourcePath(n.SourceFile, compileRoot)))
+	}
+
+	_, _ = fmt.Fprintf(w, " %s %s)\n",
 		"temp="+tempPaint(n.Temperature),
 		paint(ansiDim, fmt.Sprintf("tok=%d", n.TokenCount)),
 	)
@@ -139,10 +148,23 @@ func printTree(w io.Writer, children map[string][]*store.Node, n *store.Node, de
 	}
 
 	for _, child := range children[n.ID] {
-		printTree(w, children, child, depth+1, maxDepth)
+		printTree(w, children, child, n.SourceFile, compileRoot, depth+1, maxDepth)
 	}
 }
 
+func relSourcePath(source, compileRoot string) string {
+	if compileRoot == "" || !filepath.IsAbs(source) {
+		return source
+	}
+
+	rel, err := filepath.Rel(compileRoot, source)
+	if err != nil || strings.HasPrefix(rel, "..") {
+		return source
+	}
+
+	return rel
+}
+
 func paint(code, s string) string {
 	if !inspectColorOn {
 		return s

pkg/mcp/tools/tools_test.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"os"
 	"path/filepath"
+	"strings"
 	"testing"
 	"time"
 
@@ -369,6 +370,49 @@ func TestHandleTree(t *testing.T) {
 	}
 }
 
+func TestHandleTree_FileFieldRelativeAndOnRootOnly(t *testing.T) {
+	d, st := setup(t)
+	ctx := context.Background()
+	dir := t.TempDir()
+
+	aPath := filepath.Join(dir, "a.md")
+	bPath := filepath.Join(dir, "b.md")
+	if err := os.WriteFile(aPath, []byte("# A heading\n\nA paragraph.\n"), 0o644); err != nil {
+		t.Fatal(err)
+	}
+	if err := os.WriteFile(bPath, []byte("# B heading\n\nB paragraph.\n"), 0o644); err != nil {
+		t.Fatal(err)
+	}
+
+	if _, err := compiler.CompileDir(ctx, st, dir, "initial"); err != nil {
+		t.Fatalf("CompileDir: %v", err)
+	}
+
+	result, _, err := d.HandleTree(ctx, &gomcp.CallToolRequest{}, TreeInput{})
+	if err != nil {
+		t.Fatalf("HandleTree: %v", err)
+	}
+	if len(result.Content) == 0 {
+		t.Fatal("empty content")
+	}
+
+	text := result.Content[0].(*gomcp.TextContent).Text
+
+	if !strings.Contains(text, "file=a.md") {
+		t.Errorf("expected file=a.md (relative), got:\n%s", text)
+	}
+	if !strings.Contains(text, "file=b.md") {
+		t.Errorf("expected file=b.md (relative), got:\n%s", text)
+	}
+	if strings.Contains(text, "file="+dir) {
+		t.Errorf("absolute path leaked into output:\n%s", text)
+	}
+
+	if got := strings.Count(text, "file="); got != 2 {
+		t.Errorf("expected 2 file= entries (one per file root, none on descendants), got %d. Tree:\n%s", got, text)
+	}
+}
+
 func TestHandleWrite_BlocksOnOpMu(t *testing.T) {
 	d, st := setup(t)
 	ctx := context.Background()

pkg/mcp/tools/tree.go

@@ -3,6 +3,7 @@ package tools
 import (
 	"context"
 	"fmt"
+	"path/filepath"
 	"strings"
 	"time"
 
@@ -44,9 +45,11 @@ func (d *Deps) HandleTree(ctx context.Context, _ *gomcp.CallToolRequest, input T
 		roots = []*store.Node{root}
 	}
 
+	compileRoot, _ := d.Store.GetLatestCompileRoot(ctx)
+
 	var b strings.Builder
 	for _, root := range roots {
-		writeTreeNode(&b, childMap, root, 0, maxDepth)
+		writeTreeNode(&b, childMap, root, "", compileRoot, 0, maxDepth)
 	}
 
 	if b.Len() == 0 {
@@ -59,15 +62,33 @@ func (d *Deps) HandleTree(ctx context.Context, _ *gomcp.CallToolRequest, input T
 	}, nil, nil
 }
 
-func writeTreeNode(b *strings.Builder, children map[string][]*store.Node, n *store.Node, depth, maxDepth int) {
+func writeTreeNode(b *strings.Builder, children map[string][]*store.Node, n *store.Node, parentSource, compileRoot string, depth, maxDepth int) {
 	indent := strings.Repeat("  ", depth)
-	fmt.Fprintf(b, "%s[%s] %s (id=%s file=%s temp=%.2f tok=%d)\n", indent, n.NodeType, n.Label, n.ID, n.SourceFile, n.Temperature, n.TokenCount)
+	fmt.Fprintf(b, "%s[%s] %s (id=%s", indent, n.NodeType, n.Label, n.ID)
+
+	if n.SourceFile != parentSource {
+		fmt.Fprintf(b, " file=%s", relSourcePath(n.SourceFile, compileRoot))
+	}
+	fmt.Fprintf(b, " temp=%.2f tok=%d)\n", n.Temperature, n.TokenCount)
 
 	if depth >= maxDepth {
 		return
 	}
 
 	for _, child := range children[n.ID] {
-		writeTreeNode(b, children, child, depth+1, maxDepth)
+		writeTreeNode(b, children, child, n.SourceFile, compileRoot, depth+1, maxDepth)
 	}
 }
+
+func relSourcePath(source, compileRoot string) string {
+	if compileRoot == "" || !filepath.IsAbs(source) {
+		return source
+	}
+
+	rel, err := filepath.Rel(compileRoot, source)
+	if err != nil || strings.HasPrefix(rel, "..") {
+		return source
+	}
+
+	return rel
+}

pkg/query/engine_test.go

@@ -293,7 +293,7 @@ func TestFormat(t *testing.T) {
 		t.Fatal("empty output")
 	}
 
-	expected := "[heading] Title (score=1.25)\nhello world\n\n---\n\n[code] Example (score=2.50)\nx := 1\n"
+	expected := "[heading] (score=1.25)\nhello world\n\n---\n\n[code] (score=2.50)\nx := 1\n"
 	if out != expected {
 		t.Errorf("Format =\n%q\nwant\n%q", out, expected)
 	}

pkg/query/format.go

@@ -13,7 +13,7 @@ func Format(result *Result) string {
 			b.WriteString("\n---\n\n")
 		}
 		n := sn.Node
-		fmt.Fprintf(&b, "[%s] %s (score=%.2f)\n", n.NodeType, n.Label, sn.Score)
+		fmt.Fprintf(&b, "[%s] (score=%.2f)\n", n.NodeType, sn.Score)
 
 		b.WriteString(n.Content)
 		b.WriteByte('\n')