Skip to content

feat: add forge import and drift review to TUI - #110

Merged
aksOps merged 6 commits into
mainfrom
codex/issue-93-forge-import
Aug 18, 2026
Merged

feat: add forge import and drift review to TUI#110
aksOps merged 6 commits into
mainfrom
codex/issue-93-forge-import

Conversation

@aksOps

@aksOps aksOps commented Aug 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • extract forge parsing, guarded fetch, AI transformation, provenance, duplicate classification, drift, baseline CAS, and probe behavior into internal/forge
  • add cancellable TUI issue import with safe duplicate review, atomic task/provenance writes, and retry guards
  • add card-detail drift review with explicit no-sync behavior, revision conflicts, baseline updates, and root input ownership
  • preserve TUI ship mechanics and delete flows #89 ship/delete action ownership and block destructive input while forge workflows are active
  • harden request budgets, terminal sanitization, qualified identities, durable provenance, and concurrent baseline writes

Validation

  • go test -count=1 ./...
  • go test -race -count=1 ./...
  • scripts/check-go-coverage.sh (96.4% total; all package floors pass)
  • scripts/check-go-format.sh
  • scripts/check-go-checkers.test.sh
  • go vet ./...
  • CGO_ENABLED=0 go build ./...
  • independent spec and standards reviews: no findings

Closes #93

Comment thread internal/forge/forge.go
// paths are url.PathEscape'd), but both are stripped anyway so no future caller
// can turn this line into a log-forging primitive.
func forgeRequestError(ref forgeRef, path string) error {
log.Printf("forge: request failed source=%s path=%s", logSafe(ref.Source.Name), logSafe(path))
@sonarqubecloud

Copy link
Copy Markdown

@aksOps
aksOps marked this pull request as ready for review August 18, 2026 05:00
@aksOps
aksOps merged commit 62f26e8 into main Aug 18, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

TUI import and drift

2 participants