DX audit and fix #3

Merged

Snider merged 2 commits from agent/dx-audit-and-fix--1--review-claude-md into dev

2026-03-24 11:43:14 +00:00

Virgil commented

2026-03-24 11:30:36 +00:00

Member

No description provided.

Virgil added 1 commit 2026-03-24 11:30:37 +00:00

fix(dx): use coreerr.E() and go-io, update CLAUDE.md, add tests

Security Scan / security (pull_request) Failing after 28s

Details

Test / test (pull_request) Failing after 1m59s

Details

b559562dd9

- Replace 90+ fmt.Errorf calls with coreerr.E() from go-log across
  display, window, systray, keybinding, contextmenu, and mcp packages
- Replace os.ReadFile/WriteFile/MkdirAll with coreio.Local in
  window/layout.go and window/state.go
- Update CLAUDE.md: fix key files table for new package structure,
  document error handling and file I/O conventions, add missing deps
- Add 37 tests for window package (task handlers, persistence,
  tiling modes, snap positions, workflow layouts)
- Window coverage: 47.1% → 69.8%

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

Virgil added 1 commit 2026-03-24 11:41:01 +00:00

Merge origin/dev into agent/dx-audit-and-fix--1--review-claude-md a36392ec08

Resolve go.mod conflict: keep go-io and go-log as direct dependencies
(used by PR code changes) with dev's newer dependency versions.

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