Skip to content

Tidy source comments - #1923

Merged
santoshkumarradha merged 5 commits into
devfrom
chore/tidy-d
Oct 24, 2026
Merged

santoshkumarradha merged 5 commits into
devfrom
chore/tidy-d

Conversation

@santoshkumarradha

Copy link
Copy Markdown
Member

Tidies source comments so they explain the code in product terms, and adds a guard test that keeps them that way.

  • Comments keep the why; history-only remarks are removed.
  • internal/commentlaw checks comments in non-test source files against a small set of patterns, with an explicit allowlist.
  • No behaviour change: executable code is untouched.

Tests: go build ./..., go vet ./..., gofmt, make test-quick, and the touched package suites.

🤖 Generated with Claude Code

santoshkumarradha and others added 5 commits October 24, 2026 13:05
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
# Conflicts:
#	internal/session/asklane.go
#	internal/session/prefixbudget_test.go
#	internal/splitgate/modes.go
#	internal/tui3/lanewalk_test.go
#	internal/tui3/questionkeys.go
#	internal/tui3/steerelbow.go
#	internal/tui3/wiring_test.go
#	scripts/ledger.py
# Conflicts:
#	internal/cellstore/guard_test.go
#	internal/e2e/questions_e2e_test.go
#	internal/e2e/tui_e2e_test.go
#	internal/session/asklane_test.go
#	internal/session/taskdelta_test.go
#	internal/syncsetup/history_test.go
#	internal/syncsetup/secrets_test.go
#	internal/syncsetup/twohomes_test.go
#	internal/telemetry/usage_c3_contract_test.go
#	internal/tui3/home_test.go
#	internal/tui3/lanes_test.go
#	internal/tui3/placehost_test.go
#	internal/tui3/scrollback_test.go
#	internal/tui3/walldock_test.go
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@santoshkumarradha
santoshkumarradha merged commit 233e231 into dev Oct 24, 2026
7 checks passed
@santoshkumarradha
santoshkumarradha deleted the chore/tidy-d branch October 24, 2026 17:12
santoshkumarradha added a commit that referenced this pull request Oct 24, 2026
* Tidy source comments

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Number the change entry

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

* Keep the comment check free of private terms

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
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