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

[agent/codex] AX v0.8.0 polish pass. Fix ALL violations — banned imports... #17

Closed
Virgil wants to merge 5 commits from agent/review-for-ax-compliance--banned-imports into dev
pull from: agent/review-for-ax-compliance--banned-imports
merge into: core:dev
Conversation 3 Commits 5 Files changed 62 +2142 -413
Virgil commented 2026-03-26 18:00:46 +00:00
Member
Copy link

Task

AX v0.8.0 polish pass. Fix ALL violations — banned imports, test naming (TestFile_Function_{Good,Bad,Ugly}), usage-example comments on exports, error handling (core.E not fmt.Errorf).\n\nBanned: os, os/exec, encoding/json, fmt, errors, strings, path/filepath → use Core primitives.\n\nVerify: go build ./... && go vet ./... && go test ./... -count=1 && go mod tidy\nCommit. Co-Authored-By: Virgil virgil@lethean.io

Agent: codex
Commits: 24
Branch: agent/review-for-ax-compliance--banned-imports

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

## Task AX v0.8.0 polish pass. Fix ALL violations — banned imports, test naming (TestFile_Function_{Good,Bad,Ugly}), usage-example comments on exports, error handling (core.E not fmt.Errorf).\n\nBanned: os, os/exec, encoding/json, fmt, errors, strings, path/filepath → use Core primitives.\n\nVerify: go build ./... && go vet ./... && go test ./... -count=1 && go mod tidy\nCommit. Co-Authored-By: Virgil <virgil@lethean.io> **Agent:** codex **Commits:** 24 **Branch:** `agent/review-for-ax-compliance--banned-imports` --- Auto-created by core-agent dispatch system. Co-Authored-By: Virgil <virgil@lethean.io>
Virgil added 2 commits 2026-03-26 18:00:46 +00:00 
Co-Authored-By: Virgil <virgil@lethean.io>
Co-Authored-By: Virgil <virgil@lethean.io>
Virgil commented 2026-03-26 18:00:48 +00:00
Author
Member
Copy link

Tests Passed — Merge Failed

go test ./... passed but merge failed

## Tests Passed — Merge Failed `go test ./...` passed but merge failed
Virgil commented 2026-03-26 18:00:50 +00:00
Author
Member
Copy link

Tests Passed — Merge Failed

go test ./... passed but merge failed

## Tests Passed — Merge Failed `go test ./...` passed but merge failed
Virgil added the
needs-review
 label 2026-03-26 18:00:50 +00:00
Virgil commented 2026-03-26 18:00:50 +00:00
Author
Member
Copy link

Needs Review

Merge conflict persists after rebase. Auto-merge gave up after retry.

Labelled needs-review for human attention.

## Needs Review Merge conflict persists after rebase. Auto-merge gave up after retry. Labelled `needs-review` for human attention.
Virgil added 1 commit 2026-03-27 04:50:53 +00:00 
Co-Authored-By: Virgil <virgil@lethean.io>
Virgil added 1 commit 2026-03-27 19:38:36 +00:00 
Co-Authored-By: Virgil <virgil@lethean.io>
Virgil added 1 commit 2026-03-27 21:51:17 +00:00
Virgil closed this pull request 2026-03-30 08:54:00 +00:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
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
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-forge#17
No description provided.
Delete branch "agent/review-for-ax-compliance--banned-imports"

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?