What
go vet reports two append with no values warnings in cmdWatch:
Both are pre-existing — they predate the MCP serve (#18) and contradiction-flagging (#19) work and were left untouched there to keep those PRs focused.
Why it matters
append(x) with no values is a no-op that almost always signals a dropped argument or a copy-paste slip. Either it's dead code that should go, or it's a real bug where the appended value got lost. Worth a look either way, and clearing it keeps go vet clean so future real warnings don't hide in the noise.
Fix
Inspect both sites in cmdWatch, decide whether the append was meant to carry a value or should be removed, and clean up. Confirm go vet ./... is quiet afterward.
What
go vetreports twoappend with no valueswarnings incmdWatch:kb.go:3341kb.go:3382Both are pre-existing — they predate the MCP serve (#18) and contradiction-flagging (#19) work and were left untouched there to keep those PRs focused.
Why it matters
append(x)with no values is a no-op that almost always signals a dropped argument or a copy-paste slip. Either it's dead code that should go, or it's a real bug where the appended value got lost. Worth a look either way, and clearing it keepsgo vetclean so future real warnings don't hide in the noise.Fix
Inspect both sites in
cmdWatch, decide whether the append was meant to carry a value or should be removed, and clean up. Confirmgo vet ./...is quiet afterward.