Skip to content

chore(comments): delete comments that say what, not why, in src/daemon - #1029

Closed
V3RON wants to merge 3 commits into
mainfrom
chore/garden-comments-2026-10-26-daemon
Closed

V3RON wants to merge 3 commits into
mainfrom
chore/garden-comments-2026-10-26-daemon

Conversation

@V3RON

@V3RON V3RON commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Garden-comments pass for one slice.

  • Range: 1f42ae4..2300582a2
  • Slice: src/daemon
  • Listed: 167
  • Deleted: 104
  • Prefixed: 63 (why: added as the first word of the block)

Each listed block was judged as what-or-why. Comments that say what the code does, list callers or fields, cite an ADR, or give history were deleted. Comments that give a real reason (an ordering, an invariant, a platform quirk, or a non-obvious choice) keep their text with why: prefixed.

Review

Spec review: skipped. Code review: skipped. Claims review: 0 blocking, 0 fixed, 0 notes.
Rounds: 1.
Runs: 2. Cost USD: 0.3587. Agent seconds: 124.

Assessment: complete

Written by an agent.

@V3RON

V3RON commented Oct 26, 2026

Copy link
Copy Markdown
Contributor Author

Review findings (round 1, commit 9f3c43f)

Claims review: F0, Assessment: complete. Spec review skipped (no issue); code lens skipped (comments-only slice; coordinator verified every non-comment changed line is a formatter re-wrap with identical tokens: catch {} and one ToolsFakeDriver call). Note: review-inputs.sh printed kind full, not comments-only, though no code token changed.

Written by an agent.

@V3RON

V3RON commented Oct 26, 2026

Copy link
Copy Markdown
Contributor Author

Joined into #1033, which carries this branch's commits.

Written by an agent.

@V3RON V3RON closed this Oct 26, 2026
V3RON added a commit that referenced this pull request Oct 26, 2026
#1033)

Weekly `garden-comments` routine, run in six slices and joined into this
one PR.

Range: `1f42ae4..dc4160d`. Base for the lint: `1f42ae4` (the last
`chore(comments):` commit).

| Slice | Listed | Deleted | Prefixed | Was |
| --- | --- | --- | --- | --- |
| `src/core` | 152 | 91 | 61 | this PR |
| `src/daemon` | 167 | 104 | 63 | #1029 |
| `src/contract` | 122 | 95 | 27 | #1032 |
| `src/gateway` | 113 | 65 | 48 | #1031 |
| `src/ports` + `src/bus` | 127 | 80 | 47 | #1030 |
| e2e, scripts, test files | 115 | 93 | 22 | #1028 |

Left for a later run: 8 comments whose removal would change a code line
(six trailing comments on `clock.advance(...)` lines, and two whose
removal makes the formatter re-wrap code), so the diff stays
`comments-only`.



## Review

Spec review: skipped. Code review: skipped. Claims review: 0 blocking, 0
fixed, 3 notes.
Rounds: 1.
Runs: 4. Cost USD: 0.6555. Agent seconds: 309.

The five joined slices were claims-reviewed at their own heads (0
findings each); this round covered `src/core`.

Assessment: complete

*Written by an agent.*
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant