Skip to content

feat(cli): expose read-only Codex account and instance data - #2359

Closed
agent-sije70cmyk wants to merge 6 commits into
jlcodes99:mainfrom
agent-sije70cmyk:agent/features/task-002-cockpit-tools-codex-read-only
Closed

agent-sije70cmyk wants to merge 6 commits into
jlcodes99:mainfrom
agent-sije70cmyk:agent/features/task-002-cockpit-tools-codex-read-only

Conversation

@agent-sije70cmyk

@agent-sije70cmyk agent-sije70cmyk commented Sep 9, 2026 •

Copy link
Copy Markdown

Summary

  • expose read-only Codex account, current-account, quota, and instance listing through the CLI;
  • add stable JSON output for the new read-only operations;
  • share AES-256-GCM account-detail storage between cockpit-core and Tauri;
  • decrypt encrypted Codex account details in CLI without creating keys, repairing files, migrating plaintext, or refreshing credentials;
  • add tests proving encrypted data and the secure-storage key are not rewritten by read-only commands.

Dependency

This PR is stacked on #2356 and includes its commits. Please merge #2356 first; this PR adds the Codex read-only operations on top of the shared CLI foundation.

CLI examples

cockpit-cli --json list codex
cockpit-cli --json current codex
cockpit-cli --json quota codex
cockpit-cli --json instances codex

Existing CLI commands remain compatible. Account mutations, quota refresh, process launching, and AGY support are intentionally out of scope.

Validation

  • cargo test -p cockpit-cli -p cockpit-core --all-targets — passed;
  • cargo clippy -p cockpit-cli -p cockpit-core --all-targets — passed with existing warnings;
  • cargo check -p cockpit-tools --lib — passed on Windows with Go 1.27.1;
  • git diff --check — passed;
  • manual Windows smoke test of the read-only Codex commands — passed;
  • cargo fmt --all -- --check — currently fails on pre-existing formatting differences in upstream main (including unrelated files); no unrelated formatting changes were included in this PR.

The rebase onto current upstream main preserved the upstream Codex module split and current Tauri dependency changes. The PR branch is now based on deacbe44 and has no merge conflicts.

@agent-sije70cmyk
agent-sije70cmyk force-pushed the agent/features/task-002-cockpit-tools-codex-read-only branch from 5bc5791 to b672a9a Compare September 9, 2026 21:18
@agent-sije70cmyk

Copy link
Copy Markdown
Author

Closing this PR because the CLI work is being restarted in a dedicated fork (cockpit-tools-cli) with a revised core boundary. The implementation will be re-evaluated and selectively carried forward rather than continued on this branch.

@agent-sije70cmyk
agent-sije70cmyk deleted the agent/features/task-002-cockpit-tools-codex-read-only branch September 14, 2026 22:42
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