Skip to content

feat: add TUI settings and integrations - #103

Merged
aksOps merged 3 commits into
mainfrom
codex/issue-94-settings
Aug 17, 2026
Merged

feat: add TUI settings and integrations#103
aksOps merged 3 commits into
mainfrom
codex/issue-94-settings

Conversation

@aksOps

@aksOps aksOps commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a keyboard-driven TUI settings pane backed directly by the existing scoped store
  • test unsaved AI and forge values through the shared guarded probes, with cancellation, timeouts, and credential-origin enforcement
  • add/save/test/remove forge integrations while locking saved identity and requiring a second removal confirmation
  • keep persisted credentials write-only and cover state, storage, redaction, polling, and rendering with deterministic tests and a golden

Verification

  • GOFLAGS=-buildvcs=false npm run coverage:go — pass, 96.5% total; TUI 94.0%; server 98.1%
  • npm test — pass, 695 tests; 98.26% statement coverage
  • npm run build — pass
  • GOFLAGS=-buildvcs=false go build ./... — pass
  • GOFLAGS=-buildvcs=false go vet ./... — pass
  • GOFLAGS=-buildvcs=false go test -race ./... -count=1 — pass
  • sh scripts/check-go-format.sh — pass
  • git diff --check — pass

Security note

The activated clipboard transitive is the latest tagged github.com/atotto/clipboard v0.1.4, BSD-3-Clause, inherited from the already-pinned Bubbles text input. Local govulncheck found no reachable third-party dependency vulnerability, but did report nine reachable advisories in the installed Go 1.26.2 standard library, all fixed by Go 1.26.6. This PR does not change the project toolchain.

Closes #94

@aksOps
aksOps force-pushed the codex/issue-94-settings branch 2 times, most recently from f22c644 to c5816e2 Compare August 17, 2026 15:08
@aksOps
aksOps force-pushed the codex/issue-94-settings branch from c5816e2 to 35a357d Compare August 17, 2026 15:20
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
94.9% Coverage on New Code (required ≥ 95%)

See analysis details on SonarQube Cloud

@aksOps
aksOps marked this pull request as ready for review August 17, 2026 15:27
@aksOps
aksOps merged commit 18d3aff into main Aug 17, 2026
8 of 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 settings and integrations

1 participant