core/go-crypt

Fork 0

[agent/codex] A specs/ folder has been injected into this workspace with R... #13

Open

Virgil wants to merge 9 commits from agent/ax-review--banned-imports--test-naming into dev

Virgil commented

2026-03-27 19:30:54 +00:00

Member

Task

A specs/ folder has been injected into this workspace with RFC stub files for each package. Populate them from the actual code. For each specs file, read the MATCHING source directory, then update with full documentation of what EXISTS. Document every exported struct, interface, type alias, function, and method. Format: # {package}, Import: {path}, Files: {n}, then ## Types and ## Functions sections. Write ONLY what exists. Create missing specs files if needed. Commit.

Agent: codex
Commits: 2
Branch: agent/ax-review--banned-imports--test-naming

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

## Task A specs/ folder has been injected into this workspace with RFC stub files for each package. Populate them from the actual code. For each specs file, read the MATCHING source directory, then update with full documentation of what EXISTS. Document every exported struct, interface, type alias, function, and method. Format: # {package}, **Import:** `{path}`, **Files:** {n}, then ## Types and ## Functions sections. Write ONLY what exists. Create missing specs files if needed. Commit. **Agent:** codex **Commits:** 2 **Branch:** `agent/ax-review--banned-imports--test-naming` --- Auto-created by core-agent dispatch system. Co-Authored-By: Virgil <virgil@lethean.io>

Virgil added 10 commits 2026-03-27 19:30:55 +00:00

refactor: migrate imports to dappco.re/go/core/* paths

Security Scan / security (pull_request) Failing after 9s

Details

Test / test (pull_request) Failing after 8m29s

Details

62482c7dc9

Update module path from forge.lthn.ai/core/go-crypt to
dappco.re/go/core/crypt. Migrate go-log, go-io, go-i18n imports to
their new dappco.re/go/core/* paths with updated versions (core v0.5.0,
log v0.1.0, io v0.2.0, i18n v0.2.0). Un-migrated modules (cli,
go-store, go-inference) remain at forge.lthn.ai paths.

Also fixes merge conflict marker and duplicate imports in
crypt/openpgp/service.go, and updates CLAUDE.md to reflect new paths.

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

Merge pull request '[agent/claude] Update go.mod require lines from forge.lthn.ai to dappco.re ...' (#6 ) from agent/update-go-mod-require-lines-from-forge-l into main

Security Scan / security (push) Failing after 10s

Details

Test / test (push) Failing after 10m27s

Details

12281f9e76

fix(crypt): address AX import and test review 814ef96624

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

docs: finish AX review follow-ups f15fb2b475

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

docs(crypt): add ax v0.8.0 upgrade plan 87f87bfc0a

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

refactor: upgrade core v0.8.0-alpha.1 and replace banned imports f46cd04e2f

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

refactor(crypt): complete AX v0.8.0 polish pass 53d7d59a9d

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

chore: verification pass a63b08156e

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

test: enforce AX exact test naming 693964145e

Renames the remaining AX v0.8.0 test cases to the exact TestFile_Function_{Good,Bad,Ugly} pattern.

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

docs: populate package specs 60b713114b

Generate specs for every Go package by reading the current source tree and documenting exported types, functions, and methods in a mirrored specs/ layout.

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

Virgil commented

2026-03-27 19:31:04 +00:00

Author

Member

Tests Passed — Merge Failed

go test ./... passed but merge failed

## Tests Passed — Merge Failed `go test ./...` passed but merge failed

Virgil commented

2026-03-27 19:31:14 +00:00

Author

Member

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-27 19:31:15 +00:00

Virgil commented

2026-03-27 19:31:15 +00:00

Author

Member

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 21:46:01 +00:00

Add RFC specs for auth crypt and trust packages 82fd4e5267

This pull request has changes conflicting with the target branch.

auth/auth.go
cmd/testcmd/cmd_runner.go
crypt/chachapoly/chachapoly.go
crypt/chachapoly/chachapoly_test.go
crypt/rsa/rsa_test.go
trust/config.go
trust/policy_test.go
trust/scope_test.go

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin agent/ax-review--banned-imports--test-naming:agent/ax-review--banned-imports--test-naming

git checkout agent/ax-review--banned-imports--test-naming

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git checkout dev

git merge --no-ff agent/ax-review--banned-imports--test-naming

git checkout agent/ax-review--banned-imports--test-naming

git rebase dev

git checkout dev

git merge --ff-only agent/ax-review--banned-imports--test-naming

git checkout agent/ax-review--banned-imports--test-naming

git rebase dev

git checkout dev

git merge --no-ff agent/ax-review--banned-imports--test-naming

git checkout dev

git merge --squash agent/ax-review--banned-imports--test-naming