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

Fix all lint issues, add AGENTS.md with workflow rules #3

Merged
clawbot merged 2 commits from fix/agents-md-workflow into main 2026-02-09 21:33:34 +01:00
Conversation 0 Commits 2 Files Changed 14 +268 -126
clawbot commented 2026-02-09 21:33:19 +01:00
Collaborator
Copy Link
  • Adds .golangci.yml (copied from upaas) and comprehensive Makefile
  • Fixes all 141 lint issues:
    • Doc comments on all exported symbols
    • Package comments on all packages
    • camelCase JSON tags
    • Named constants for magic numbers
    • Whitespace/formatting per nlreturn, wsl_v5
    • errors.Is() usage
    • Stuttering type name fixes (e.g. config.ConfigParams → config.Params)
  • Adds AGENTS.md with coding rules and git workflow
  • Zero lint issues, builds clean, tests pass (no test files yet)

Workflow rule: all changes via feature branches + PR, no direct commits to main

- Adds `.golangci.yml` (copied from upaas) and comprehensive `Makefile` - Fixes all 141 lint issues: - Doc comments on all exported symbols - Package comments on all packages - camelCase JSON tags - Named constants for magic numbers - Whitespace/formatting per nlreturn, wsl_v5 - errors.Is() usage - Stuttering type name fixes (e.g. config.ConfigParams → config.Params) - Adds `AGENTS.md` with coding rules and git workflow - Zero lint issues, builds clean, tests pass (no test files yet) Workflow rule: **all changes via feature branches + PR, no direct commits to main**
sneak was assigned by clawbot 2026-02-09 21:33:19 +01:00
clawbot added 2 commits 2026-02-09 21:33:20 +01:00 
- Fix stuttering type names (e.g. config.ConfigParams → config.Params)
- Add doc comments to all exported types/functions/methods
- Add package doc comments to all packages
- Fix JSON tags to camelCase
- Extract magic numbers to constants
- Add blank lines per nlreturn/wsl_v5 rules
- Use errors.Is() for error comparison
- Unexport NewLoggingResponseWriter (not used externally)
- Replace for-range on ctx.Done() with channel receive
- Rename unused parameters to _
- AGENTS.md: all changes via feature branches, no direct main commits
clawbot merged commit e6621ef7c6 into main 2026-02-09 21:33:34 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bot

Agent's turn to work

  
merge-ready

All checks pass, reviewed, rebased, ready for merge

  
needs-checks

make check does not pass cleanly

  
needs-rebase

PR has merge conflicts or is behind main

  
needs-review

Code review not yet done

  
needs-rework

Code review found issues to fix

No Label
bot
merge-ready
needs-checks
needs-rebase
needs-review
needs-rework
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/chat#3
No description provided.
Delete Branch "fix/agents-md-workflow"

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?