comments that explain the code, not how it was written

About 340 comments cited the development process: task and plan
numbers, fix waves, rulings, reviewers, and the author in the third
person with a date. None of that exists outside the work itself, so to
a reader it pointed at nothing. Each one now states the engineering
reason it was standing in front of; where a comment was provenance and
nothing else, it is gone.

References to docs/design.md and docs/gui-design.md by section stay:
both ship with the repository. The design documents lose their
amendment diaries - CHANGELOG.md is that record - and the GUI's says
plainly that the window has gone further than the document.

Only comments changed. Every .go file was parsed and its code printed
with comments stripped, before and after: the two hashes are identical
across all 175 files.

1 files changed, 9 insertions, 9 deletions

diff --git a/internal/engine/facts.go b/internal/engine/facts.go
index ce05df4..d664b0e 100644
--- a/internal/engine/facts.go
+++ b/internal/engine/facts.go
@@ -66,7 +66,7 @@ func (run *matchRun) warnings() []string {
 }
 
 // drainDupErrors appends every duplicate index's candidate-hashing errors
-// (A1: a candidate other than the file being looked up that could not be
+// (a candidate other than the file being looked up that could not be
 // hashed) to run.warn, once matching is done and every index has seen every
 // Lookup it is going to see. Candidate paths are abbreviated with
 // xdg.Abbrev, as every other user-visible path is.
@@ -80,8 +80,8 @@ func (run *matchRun) drainDupErrors() {
 	}
 }
 
-// dupWarning is the one form of a duplicate warning (triage 6): "duplicate:
-// PATH: cause", the path abbreviated like every other one shown, and an OS
+// dupWarning is the one form of a duplicate warning: "duplicate: PATH:
+// cause", the path abbreviated like every other one shown, and an OS
 // error's cause without the raw path it would repeat.
 func dupWarning(err error) string {
 	return "duplicate: " + dupCause(err).Error()
@@ -89,8 +89,8 @@ func dupWarning(err error) string {
 
 // dupCause is a duplicate check's error as "PATH: cause", the path
 // shortened with xdg.Abbrev and not repeated inside the cause - a candidate
-// that could not be hashed, or an OS error on the file itself (plan 11
-// review L7). It still unwraps to err.
+// that could not be hashed, or an OS error on the file itself. It still
+// unwraps to err.
 func dupCause(err error) error {
 	path, cause := "", err
 	var ce dup.CandidateError
@@ -229,12 +229,12 @@ func (f *facts) extract(opt cond.Options, keywords []string) {
 	switch {
 	case errors.Is(err, extract.ErrPartial):
 		// Partly read: the keywords found in it are answered; one not found
-		// is unknown (ContentContains), and nothing is cached (plan 11).
+		// is unknown (ContentContains), and nothing is cached.
 		f.partialErr = err
 	case errors.Is(err, extract.ErrUnsupported):
 		// A format with no text cannot contain a keyword: every answer is
-		// no, with no warning, and the answers are cached like any other
-		// (re-review N2). Only a real read failure is "unreadable".
+		// no, with no warning, and the answers are cached like any other.
+		// Only a real read failure is "unreadable".
 		text = ""
 	case err != nil:
 		f.contentErr = err
@@ -296,7 +296,7 @@ func (f *facts) Duplicate(dirs []string) (string, bool, error) {
 	}
 	// The absolute path is kept for a front end that has to act on the
 	// other copy - the window offers to keep this one instead - while the
-	// reason text stays as it reads best (plan 21).
+	// reason text stays as it reads best.
 	f.dupOriginal = orig
 	return displayOriginal(orig, root), true, nil
 }