steve/majordomo
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs(agent): drop the dangling isBackRef reference
CI / Tidy (pull_request) Successful in 9m26s
Details
CI / Build & Test (pull_request) Successful in 9m49s
Details

Browse Source 
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.
This commit is contained in:
steve
2026-08-21 23:52:25 -04:00
parent 9534e442fb
commit 7dccb233ad
1 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files
agent/finalize.go
+4 -4
View File
@@ -304,10 +304,10 @@ func isCitationsOnly(s string) bool {
// ratio in isSubstantiveAnswer keeps a matching closer in place when nothing // ratio in isSubstantiveAnswer keeps a matching closer in place when nothing
// earlier clearly outweighs it. // earlier clearly outweighs it.
// //
// Only pointsAbove is used here, not the whole isBackRef class: backRefRe's // pointsAbove is used here; its sibling backRefRe is NOT. Those fixed phrases
// fixed phrases can appear anywhere in the text, and a 300-byte closer has // are matched anywhere in the text, and a 300-byte closer has room for a real
// room for a real answer that merely mentions "as I said" mid-sentence. // answer that merely mentions "as I said" mid-sentence. pointsAbove is
// pointsAbove is offset-bounded, so it stays anchored to the opening. // offset-bounded, so it stays anchored to the opening.
func isSummaryCloser(s string) bool { func isSummaryCloser(s string) bool {
t := strings.TrimSpace(s) t := strings.TrimSpace(s)
if t == "" || len(t) > summaryCloserMaxChars { if t == "" || len(t) > summaryCloserMaxChars {