From 973f644ea8ef190f74bfb8b30fc40106849723e8 Mon Sep 17 00:00:00 2001 From: tae2089 Date: Tue, 11 Aug 2026 16:45:28 +0900 Subject: [PATCH] docs(graph): count Identity's fields as five, not four The doc comment said "The four fields are the columns of the node table's uniqueness index ... plus the kind", which reads as four in total. The struct has five, and four is the count this change existed to correct: the old tie-break compared four keys and called two declarations equal when one file declares the same qualified name at two lines. Co-Authored-By: Claude Opus 5 --- internal/domain/graph/identity.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/domain/graph/identity.go b/internal/domain/graph/identity.go index 80c0a87..e0e1c42 100644 --- a/internal/domain/graph/identity.go +++ b/internal/domain/graph/identity.go @@ -11,7 +11,7 @@ import "cmp" // stable sort, a comparison across two databases — belongs on this and not on // the id. // -// The four fields are the columns of the node table's uniqueness index +// The five fields are the four columns of the node table's uniqueness index // (namespace, qualified_name, file_path, start_line) plus the kind, so two // different nodes cannot share one Identity. // @intent give ranking a key that survives re-indexing, which the node id does not.