[agent/claude:opus] DX audit and fix. 1) Review CLAUDE.md — update any outdate... #1

Merged

Virgil merged 1 commit from agent/dx-audit-and-fix--1--review-claude-md into main

2026-03-17 08:03:32 +00:00

Virgil commented

2026-03-17 07:19:40 +00:00

Member

Task

DX audit and fix. 1) Review CLAUDE.md — update any outdated commands, ensure it explains the package well for new developers. 2) Run go test -cover ./... — report coverage, add tests for any critical untested paths. 3) Check error handling: all errors must use coreerr.E() from go-log, not fmt.Errorf or errors.New. Fix any violations. 4) Check that no code uses os.ReadFile/os.WriteFile directly (this IS the I/O package so it's expected internally, but public API should be clean). 5) Run tests to confirm. Commit fixes.

Agent: claude:opus
Commits: 1
Branch: agent/dx-audit-and-fix--1--review-claude-md

Auto-created by core-agent dispatch system.
Co-Authored-By: Virgil virgil@lethean.io

## Task DX audit and fix. 1) Review CLAUDE.md — update any outdated commands, ensure it explains the package well for new developers. 2) Run `go test -cover ./...` — report coverage, add tests for any critical untested paths. 3) Check error handling: all errors must use coreerr.E() from go-log, not fmt.Errorf or errors.New. Fix any violations. 4) Check that no code uses os.ReadFile/os.WriteFile directly (this IS the I/O package so it's expected internally, but public API should be clean). 5) Run tests to confirm. Commit fixes. **Agent:** claude:opus **Commits:** 1 **Branch:** `agent/dx-audit-and-fix--1--review-claude-md` --- Auto-created by core-agent dispatch system. Co-Authored-By: Virgil <virgil@lethean.io>

Virgil added 1 commit 2026-03-17 07:19:40 +00:00

fix(dx): audit errors, update CLAUDE.md, clean up node import guard dbe5086a0a

- Replace errors.New() with coreerr.E() in sigil/sigils.go (HashSigil.In, NewSigil)
- Update CLAUDE.md: add missing deps (go-crypt, x/crypto, testify), fix go/pkg/core path,
  add GOWORK=off note, document sentinel error convention, add qa command
- Remove redundant unused import guard in node/node.go

Co-Authored-By: Virgil <virgil@lethean.io>