[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 09:04:32 +00:00

Virgil commented

2026-03-17 09:04:25 +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 09:04:26 +00:00

fix(mcp): DX audit — update CLAUDE.md, replace os.* with go-io, add tests a7bc30f8ac

- CLAUDE.md: remove stale tools_ml.go reference (moved to go-ml),
  add agentic subsystem, document HTTP transport and MCP_AUTH_TOKEN
- Replace os.ReadDir with coreio.Local.List in agentic package
  (plan.go, queue.go, status.go)
- Replace os.Stat with coreio.Local.IsFile/List in agentic package
  (plan.go, pr.go, resume.go)
- Add DirectSubsystem tests (brain): apiCall, remember, recall, forget
  with httptest mock server — coverage 6.5% → 39.6%
- Add HTTP transport tests: health endpoint, Bearer auth, withAuth
  unit tests, Run() priority — coverage 51.2% → 56.0%

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