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

Closed
Virgil wants to merge 1 commit from agent/dx-audit-and-fix--1--review-claude-md into main
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:28:19 +00:00
fix(dx): update CLAUDE.md and improve test coverage
All checks were successful
Security Scan / security (pull_request) Successful in 8s
Test / test (pull_request) Successful in 48s
023c8db79d
- Fix WASM size gate: 3 MB → 3.5 MB to match size_test.go
- Fix dependencies: remove stale replace directive, add go-io and go-log
- Add error handling and I/O conventions to coding standards
- Add tests: reader error (cmd/codegen), duplicate tag dedup, bundle
  error propagation (codegen) — coverage: codegen 89→96%, cmd/codegen
  71→78%

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

Verification Passed — Merge Failed

Tests passed (go test ./...) but merge failed: forgeMergePR: HTTP 405: Please try again later

## Verification Passed — Merge Failed Tests passed (`go test ./...`) but merge failed: forgeMergePR: HTTP 405: Please try again later
Virgil closed this pull request 2026-03-17 13:11:41 +00:00
All checks were successful
Security Scan / security (pull_request) Successful in 8s
Test / test (pull_request) Successful in 48s

Pull request closed

Sign in to join this conversation.
No description provided.