Skip to content

Guide TUI users through Anthropic secret setup - #131

Merged
nishu-builder merged 1 commit into
codex/conversation-secretsfrom
codex/tui-secret-setup-guidance
Aug 20, 2026
Merged

Guide TUI users through Anthropic secret setup#131
nishu-builder merged 1 commit into
codex/conversation-secretsfrom
codex/tui-secret-setup-guidance

Conversation

@nishu-builder

Copy link
Copy Markdown
Contributor

Stacked on #113.

Before entering raw terminal mode, the TUI now verifies that the conversation secret is configured. Missing credentials produce a complete setup recipe: the local file, both llm-step and llm-call readers, and the entropy command. The same checked loader is used by turn preparation, title generation, dispatch, and recovery so their errors stay consistent.

Tests:

  • cargo fmt --all -- --check
  • cargo test --workspace
  • nix build
  • CAOS_SALT=$(date -u +%Y-%m-%dT%H:%M:%SZ) result/bin/caos-cli run-tool test (44/44)
  • git diff --check
  • pseudo-terminal launch without .caos-secrets (confirmed instructions render before raw mode)

@nishu-builder
nishu-builder force-pushed the codex/tui-secret-setup-guidance branch from f55f994 to 30eb708 Compare August 20, 2026 21:08
@nishu-builder
nishu-builder force-pushed the codex/tui-secret-setup-guidance branch from 30eb708 to 25483fd Compare August 20, 2026 21:32
@nishu-builder
nishu-builder merged commit da2c82e into main Aug 20, 2026
1 check passed
@nishu-builder
nishu-builder deleted the codex/tui-secret-setup-guidance branch August 20, 2026 21:37
malcolmredheron added a commit that referenced this pull request Aug 20, 2026
Drops nothing of ours. The only conflict was an import list in
`std/llm-step/src/main.rs` — our side added `eval_then_catching`, main's added
`secret` — resolved by taking both.

`Fix TUI clippy regression (#133)` supersedes the `#[allow]` reverted in
a1ddfbd: it takes `SubmitMessagePolicy` where the function was taking
`require_absent`/`proposal` loose and packing them into that struct two lines
later. Seven arguments, no suppression. A suppression that outlives its cause
is worse than none — the next reader takes it to mean "these arguments travel
together" when one of them was two arguments wearing a trenchcoat.

Also arriving: `Use secrets in conversations (#113)` and `Guide TUI users
through secret setup (#131)`, which move the secret store this branch threads
through `ClientEvalHost::mark_curry` and the eval memo key — worth the full
gate rather than an assumption.

Verified on the merged tree: nix build, lint-flake-src.sh, cargo
build/clippy/fmt/test, `cargo doc -D warnings`, and
`CAOS_SALT=$(date --iso=s) run-tool test` — 45/45.
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.

1 participant