Cursor rule

1---
2description: crit — go
3globs: crit/*.go,crit/**/*.go
4alwaysApply: false
5---
6 
7# Go Code Rules (semantic — linters handle syntax/style)
8 
9## Struct & State Integrity
10- When building structs for serialization or cross-boundary transfer, verify ALL source fields are mapped. Missing fields cause silent data loss.
11- For boolean config fields, always use presence tracking to distinguish "not set" from "set to false". Go's zero-value for bool is `false`, which is ambiguous.
12- When clearing/resetting state from a struct, enumerate ALL fields. Review comments, file comments, and metadata must all be cleared.
13- When storing error values in shared state (atomic.Value, channels), copy the value first: `e := err; store(&e)`.
14 
15## Code Organization
16- This is `package main`. Do NOT export functions/types unless required by tests in `_test.go` files.
17- Never create wrapper functions that just call another function with the same signature.
18- Route all review file writes through `saveCritJSON()`. Never write directly.
19- Before implementing logic inline, search for existing helper functions that do the same thing.
20- After replacing a function with a new implementation, delete the old function AND its tests in the same PR.
21 
22## Git Operations
23- When determining file status, always diff against baseRef consistently. Don't mix `git status --porcelain` (working tree) with `git diff --name-status` (branch diff).
24 
25## Daemon/Server
26- When checking if a service is alive, validate the response body, not just HTTP status.
27- Error paths in CLI subcommands must exit with non-zero status (`os.Exit(1)`).
28 
29## Testing
30- Use table-driven tests instead of separate test functions for parameter variations.
31- When adding a new API endpoint, add tests for: happy path, error cases, not-found, method-not-allowed.
32