From 7dccb233adf3a931e15ed843ad311ef6689ec9cc Mon Sep 17 00:00:00 2001 From: Steve Dudenhoeffer Date: Fri, 21 Aug 2026 23:52:25 -0400 Subject: [PATCH] docs(agent): drop the dangling isBackRef reference MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Round 2 (glm): the isSummaryCloser comment still explained itself in terms of "the whole isBackRef class" — a symbol deleted in f97c2b7. It named the right distinction with the wrong noun. Reworded to name backRefRe directly; no isBackRef occurrences remain in the package. --- agent/finalize.go | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/agent/finalize.go b/agent/finalize.go index ed08094..64c348d 100644 --- a/agent/finalize.go +++ b/agent/finalize.go @@ -304,10 +304,10 @@ func isCitationsOnly(s string) bool { // ratio in isSubstantiveAnswer keeps a matching closer in place when nothing // earlier clearly outweighs it. // -// Only pointsAbove is used here, not the whole isBackRef class: backRefRe's -// fixed phrases can appear anywhere in the text, and a 300-byte closer has -// room for a real answer that merely mentions "as I said" mid-sentence. -// pointsAbove is offset-bounded, so it stays anchored to the opening. +// pointsAbove is used here; its sibling backRefRe is NOT. Those fixed phrases +// are matched anywhere in the text, and a 300-byte closer has room for a real +// answer that merely mentions "as I said" mid-sentence. pointsAbove is +// offset-bounded, so it stays anchored to the opening. func isSummaryCloser(s string) bool { t := strings.TrimSpace(s) if t == "" || len(t) > summaryCloserMaxChars {