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

Merged

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

2026-03-17 08:15:01 +00:00

Virgil commented

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

Member

Task

DX audit and fix. 1) Review CLAUDE.md — update any outdated commands. 2) Run go test -cover ./... — report coverage, add tests for untested critical paths. 3) Check error handling: all errors must use coreerr.E() from go-log, not fmt.Errorf. Fix violations. 4) Check file I/O uses go-io not os.ReadFile/os.WriteFile. Fix violations. 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. 2) Run `go test -cover ./...` — report coverage, add tests for untested critical paths. 3) Check error handling: all errors must use coreerr.E() from go-log, not fmt.Errorf. Fix violations. 4) Check file I/O uses go-io not os.ReadFile/os.WriteFile. Fix violations. 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 08:01:32 +00:00

fix(ai): DX audit — update CLAUDE.md, add tests, fix io consistency

Security Scan / security (pull_request) Successful in 9s

Details

Test / test (pull_request) Failing after 1m33s

Details

569f553b1a

- CLAUDE.md: remove deleted daemon/ entry, add embed-bench/ and lab/,
  fix build commands for go workspace (./... → module path),
  update error handling guidance to coreerr.E(), remove stale
  cli.AddDaemonCommand reference
- Replace os.MkdirAll with coreio.Local.EnsureDir in bench/test code
- Add 14 unit tests for metrics (Record, ReadEvents, Summary, sortedMap,
  readMetricsFile edge cases) — ai/ coverage 65.3% → 67.3%
- Add parseDuration tests for cmd/metrics — coverage 0% → 24.6%
- No fmt.Errorf or errors.New violations found
- No os.ReadFile/os.WriteFile violations found

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

Virgil commented

2026-03-17 08:01:33 +00:00

Author

Member

Verification Failed

# ./...
pattern ./...: directory prefix . does not contain modules listed in go.work or their selected dependencies
FAIL	./... [setup failed]
FAIL

Exit code: 1

PR will not be auto-merged.

## Verification Failed ``` # ./... pattern ./...: directory prefix . does not contain modules listed in go.work or their selected dependencies FAIL ./... [setup failed] FAIL ``` **Exit code:** 1 PR will not be auto-merged.