sneak/prompts
1
0
Fork 1
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

style(go): add rule against type-only packages (per upaas #126 review) #2

Merged
sneak merged 2 commits from clawbot/prompts:add-no-type-only-packages-rule into main 2026-02-23 22:14:02 +01:00
Conversation 1 Commits 2 Files Changed 1 +23
clawbot commented 2026-02-23 20:48:14 +01:00
Contributor
Copy Link

Adds a rule to CODE_STYLEGUIDE_GO.md: define types alongside their implementations, not in separate types, domain, or models packages.

Type-only packages cause alias imports and indicate poor package design.

Prompted by sneak's review comment on upaas PR #126.

Adds a rule to CODE_STYLEGUIDE_GO.md: define types alongside their implementations, not in separate `types`, `domain`, or `models` packages. Type-only packages cause alias imports and indicate poor package design. Prompted by sneak's review comment on upaas PR #126.
sneak was assigned by clawbot 2026-02-23 20:48:14 +01:00
clawbot added 1 commit 2026-02-23 20:48:14 +01:00 
Types should live alongside their implementations, not in separate
'types', 'domain', or 'models' packages. Type-only packages cause
alias imports and indicate poor package design.

Prompted by review feedback on upaas PR #126.
clawbot added 1 commit 2026-02-23 20:56:18 +01:00
clawbot commented 2026-02-23 20:56:23 +01:00
Author
Contributor
Copy Link

Added the Stringer rule: custom string-based types must implement fmt.Stringer, and .String() should be used at SDK/library boundaries instead of string(v).

Added the Stringer rule: custom string-based types must implement `fmt.Stringer`, and `.String()` should be used at SDK/library boundaries instead of `string(v)`.
sneak merged commit dda0d01faa into main 2026-02-23 22:14:02 +01:00
sneak deleted branch add-no-type-only-packages-rule 2026-02-23 22:14:02 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
sneak
1 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: sneak/prompts#2
No description provided.
Delete Branch "clawbot/prompts:add-no-type-only-packages-rule"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?