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

[agent/codex] Fix ALL findings from issue #2. Read CLAUDE.md first. os.Get... #3

Merged
Virgil merged 6 commits from agent/fix-all-findings-from-issue--2--read-cla into dev 2026-03-23 07:38:53 +00:00
Conversation 1 Commits 6 Files changed 7 +195 -47
Virgil commented 2026-03-23 07:38:52 +00:00
Member
Copy link

Task

Fix ALL findings from issue #2. Read CLAUDE.md first. os.Getenv(USER/USERNAME) → core.Env (log.go:289-292), fmt.Sprintf → core.Sprintf (errors.go:32-39, log.go:228-237), strings.* → core.* (18 hits in tests + prod). Fix all.

Agent: codex
Commits: 6
Branch: agent/fix-all-findings-from-issue--2--read-cla

Auto-created by core-agent dispatch system.
Co-Authored-By: Virgil virgil@lethean.io

## Task Fix ALL findings from issue #2. Read CLAUDE.md first. os.Getenv(USER/USERNAME) → core.Env (log.go:289-292), fmt.Sprintf → core.Sprintf (errors.go:32-39, log.go:228-237), strings.* → core.* (18 hits in tests + prod). Fix all. **Agent:** codex **Commits:** 6 **Branch:** `agent/fix-all-findings-from-issue--2--read-cla` --- Auto-created by core-agent dispatch system. Co-Authored-By: Virgil <virgil@lethean.io>
Virgil added 6 commits 2026-03-23 07:38:52 +00:00 
- CLAUDE.md: add missing exported symbols (NewError, Message, AllOps,
  FormatStackTrace, Security, Username) to architecture description
- Add tests for SetOutput, SetRedactKeys, odd keyval handling,
  op/stack deduplication, RotationWriterFactory, default output,
  Username, and all package-level proxy functions
- Add ErrCode tests for plain error and nil inputs
- Coverage: 86.1% → 97.7%

Co-Authored-By: Virgil <virgil@lethean.io>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Merge remote-tracking branch 'origin/main'
Some checks failed
CI / auto-merge (push) Failing after 0s
Details
CI / test (push) Failing after 2s
Details
CI / auto-fix (push) Failing after 0s
Details
08b2a70fc5
Virgil merged commit 6008ca5c3a into dev 2026-03-23 07:38:53 +00:00
Virgil deleted branch agent/fix-all-findings-from-issue--2--read-cla 2026-03-23 07:38:53 +00:00
Virgil commented 2026-03-23 07:38:53 +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

   
needs-review

   
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
needs-review
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".

-

Dependencies

No dependencies set.

Reference: core/go-log#3
No description provided.
Delete branch "agent/fix-all-findings-from-issue--2--read-cla"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?