core/go-rag
8
0
Fork 0
Code Issues 4 Pull requests Projects Releases Packages Wiki Activity Actions

[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:43:42 +00:00
Conversation 1 Commits 1 Files changed 8 +139 -16
Virgil commented 2026-03-17 08:43:40 +00:00
Member
Copy link

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>
fix(dx): audit and fix error handling, CLAUDE.md, and test coverage
All checks were successful
Security Scan / security (pull_request) Successful in 8s
Details
Test / test (pull_request) Successful in 1m1s
Details
d2366daad9 
Replace all fmt.Errorf with log.E() in production code (helpers.go,
cmd/rag/) to use the project's structured error convention. Update
CLAUDE.md to reflect cobra→cli migration. Add tests for
ListCollectionsSeq, KeywordFilterSeq, and NewOllamaClient to cover
previously untested critical paths (68.3% → 70.8%).

Co-Authored-By: Virgil <virgil@lethean.io>
Virgil merged commit 0944184a01 into main 2026-03-17 08:43:42 +00:00
Virgil deleted branch agent/dx-audit-and-fix--1--review-claude-md 2026-03-17 08:43:43 +00:00
Virgil commented 2026-03-17 08:43:43 +00:00
Author
Member
Copy link

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.
Reviewers
No reviewers
Labels
Clear labels   
needs-review
  
athena
Assign to Athena AI agent (M3 Ultra)

  
athena-gemini
Dispatch to Gemini CLI on M3 Ultra

  
audit
Code audit task

  
clotho
Assign to Clotho AI agent for processing

  
clotho-gemini
Dispatch to Gemini CLI on darbs (AU)

  
codex
Dispatch to Codex CLI on gateway for analysis

  
darbs-claude
Dispatch to Claude on darbs (AU)

  
security
Security review task

  
wiki
Documentation/wiki update

No labels
needs-review
athena
athena-gemini
audit
clotho
clotho-gemini
codex
darbs-claude
security
wiki
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
core/go-rag!1
No description provided.