sneak/chat
1
0
Fork 0
Code Issues 5 Pull Requests 2 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
AGENTS.md: no direct commits to main, all changes via feature branches 7b0ff178d4
Fix all lint issues and update AGENTS.md workflow rules 6a108749a1 
- 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
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-review
needs-rework
Code review found issues to fix
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
clawbot sneak
No Assignees sneak
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sneak/chat#3
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?