[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:26:20 +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:26:19 +00:00
- CLAUDE.md: fix module count (174→41+3), KnownModules count (68→80),
  update error handling convention to reflect coreerr.E() migration
- cmd/ansible: replace fmt.Errorf with coreerr.E(), os.Stat with
  coreio.Local.Exists()/IsDir(), os.Getwd with filepath.Abs()
- Add 38 tests for untested critical paths: non-SSH module handlers
  (debug, fail, assert, set_fact, include_vars, meta), handleLookup,
  SetInventory, iterator functions, resolveExpr with registered vars/
  facts/task vars/host vars/filters

Coverage: 23.4% → 27.7%

Co-Authored-By: Virgil <virgil@lethean.io>
Virgil merged commit da6c3cae21 into main 2026-03-17 08:26:20 +00:00
Virgil deleted branch agent/dx-audit-and-fix--1--review-claude-md 2026-03-17 08:26:20 +00:00
Author
Member

Auto-Verified & Merged

Tests: go test ./... — PASS

Auto-merged by core-agent dispatch system.

## Auto-Verified & Merged **Tests:** `go test ./...` — PASS Auto-merged by core-agent dispatch system.
Sign in to join this conversation.
No description provided.