Skip to content

chore(comments): delete comments that say what, not why, in src/ports and src/bus - #1030

Closed
V3RON wants to merge 1 commit into
mainfrom
chore/garden-comments-2026-10-26-ports-bus
Closed

V3RON wants to merge 1 commit into
mainfrom
chore/garden-comments-2026-10-26-ports-bus

Conversation

@V3RON

@V3RON V3RON commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Range: 1f42ae4..b09a52e6f
Slice: src/ports + src/bus

Listed: 127 blocks
Deleted: 80
Prefixed with why:: 47

Comments only: every removed line is a comment, and each added why: prefix sits on an existing comment. No code line changed. pnpm test:changed failed once in src/daemon/dispatcher.test.ts (a timing test outside this slice); it passed on rerun, and all ports and bus tests pass.

Review

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

Assessment: complete

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