Skip to content

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

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

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

Conversation

@V3RON

@V3RON V3RON commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

Comment garden for src/gateway, range 1f42ae4..a2da6860c.

  • Slice: src/gateway (dispatcher.test.ts, dispatcher.ts, service.ts, worker-registry.ts)
  • Listed: 113
  • Deleted: 65
  • Prefixed: 48 (why: added to blocks that explain an invariant, hazard, ordering, platform quirk or unexpected behaviour; original comment markers kept)

Review

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

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