Go Review

Go Review

This is the native Codex review skill for Go work in this repository.

Use it when:

• the user asks for a review
• the task is to validate a Go change
• the task is to assess risks before or after a refactor

Review Priorities

Report findings first. Prioritize:

• correctness bugs
• behavioral regressions
• API and JSON compatibility changes
• missing or weak tests
• concurrency and shutdown risks
• dependency policy violations
• facade leakage from root package into domain logic

Repository-Specific Checks

• Root package wrappers must stay zero-logic.
• New behavior belongs in internal/app<domain>/.
• Do not assume third-party helpers are acceptable in this zero-dependency repo.
• Treat json:",omitempty" and json:",omitzero" changes as API behavior changes.
• Check that goroutines have shutdown and ownership semantics.

Review Output

• Lead with concrete findings.
• Use file references.
• Keep summaries brief.
• If there are no findings, say that explicitly and mention residual risk or testing gaps.