[colbymchenry]

Follow-up to #2678 and #2679. A constant declared inside a const ( … ) group lost the doc comment written above it:

const (
	// ErrorTypeBind is used when Context.Bind() fails.
	ErrorTypeBind ErrorType = 1 << 63
	// ErrorTypeRender is used when Context.Render() fails.
	ErrorTypeRender ErrorType = 1 << 62

gin's ErrorTypeBind had no docstring. Both extractors read every member's docstring from the comment above const (, the group's, and never from the member's own comment. That is the comment go doc prints beside the member, and the one gopls shows on hover.

Change

• In the Go branch of variable extraction (extractVariable in TS, extract_variable in the kernel), each const_spec's docstring is now its own doc comment, read with the Go rules from fix(go): a type declared on its own keeps its doc comment #2438/fix(go): a comment a blank line separates from a declaration is not its doc #2678/fix(go): a comment directive like //go:embed is not part of a declaration's doc #2679 (trailing comments, blank lines and directives).
• Decision: a member with no comment of its own keeps the group's comment, as before. go/doc treats a value group's comment as the whole group's, so dropping it from undocumented members would lose documentation they have today. fix(go): a type declared on its own keeps its doc comment #2438 deliberately keeps a type ( … ) group's comment off its members; that stays as it is.
• An ungrouped const X = 1 has no comment inside its declaration, so it reads the declaration's doc as before.

Verification

• New __tests__/go-grouped-const-docs.test.ts, kernel and wasm, each on LF and CRLF:
  • fails 4/4 on main f43432c and passes 4/4 here;
  • covers a member's own doc in a documented group, in an undocumented group and in a group of one; a member with no doc, after a trailing comment, or under a detached comment (all keep the group's comment); an own doc with a directive under it; and ungrouped const/var.
• The torture.go parity fixture gains a const group with one documented member; kernel-tsjs-parity passes 99/99. The Go doc tests from fix(go): a type declared on its own keeps its doc comment #2438, fix(go): a comment a blank line separates from a declaration is not its doc #2678 and fix(go): a comment directive like //go:embed is not part of a declaration's doc #2679 pass unchanged.
• Full suite with the kernel built and CODEGRAPH_KERNEL_EXPECT=1 (--maxWorkers=4 --minWorkers=1): 7,513 passed, 0 failed, 80 skipped (7,593 tests, 0 failed files).

A/B on real repos

main f43432c vs this branch; etcd 6782f522, prometheus 0b57789d, gin 0f09c3a9.

Nodes, edges, unresolved refs and files are byte-identical on all three repos, on both the kernel and wasm paths. Only docstrings change, and the two paths change the same ones:

repo	gained their own doc (had none)	own doc replaces the group's
etcd	113	0
prometheus	202	13
gin	10	3

• Every change equals the member's own doc comment as go doc reads it, taken from the file: the comment-only lines right above the member, with no blank line and with directives left out.
• Full check of every grouped constant against "own doc comment, else the group's" (580 / 1,475 / 82 members): main disagrees on 113 / 215 / 13, this branch on 0 / 0 / 0. The members with only a group doc (8 / 114 / 39) and with none (459 / 1,146 / 30) are unchanged.
• Examples: prometheus FloatChunkEncodingXOR ("selects standard XOR encoding for float chunks"); prometheus lintErrExitCode; etcd's deprecated authpb.READ / WRITE aliases; and the generated const _ = protoimpl.EnforceVersion(…) guards ("Verify that this generated code is sufficiently up-to-date").
• Kernel parity (kernel-parity.mjs --lang go): 0 diffs on all three repos. etcd 1065/1105, the same 40 erroring files deferred as on main; prometheus 746/746; gin 99/99.
• Cost: one extra preceding-comment walk per constant; no measurable change.

Notes

• Next, and last, follow-up as its own PR: variables declared in a var ( … ) block are not indexed at all. tree-sitter-go wraps a var group's specs in a var_spec_list node, which neither extractor reads, so etcd, prometheus and gin index no grouped variable. That PR gives them their own docs the same way.

🤖 Generated with Claude Code