Skip to content

feat: add desktop plans and native remote MCP - #73

Merged
killswitch-GUI merged 4 commits into
mainfrom
codex/desktop-embedded-terminal
Jul 31, 2026
Merged

feat: add desktop plans and native remote MCP#73
killswitch-GUI merged 4 commits into
mainfrom
codex/desktop-embedded-terminal

Conversation

@killswitch-GUI

Copy link
Copy Markdown
Collaborator

Summary

  • complete the Desktop Plan workflow and embedded terminal experience, including durable plan lifecycle/cancellation projections, API and SDK surfaces, tests, design QA, and architecture diagrams
  • add native stateful MCP Streamable HTTP support for Splunk with bounded JSON/SSE responses, session handling, fresh schema discovery, and allowedTools: ["*"]
  • add OAuth 2.1 PKCE bootstrap and refresh support with environment-backed client secrets, keyring/encrypted-state persistence, and CLI/TUI/worker authentication operations
  • harden the remote MCP network and policy boundary with exact-origin authorization, DNS pinning, bounded bodies/timeouts, credential redaction, and process-vs-network effect enforcement
  • regenerate public SDK bindings/examples and update event-listener to the RustSec-patched 5.4.2 release in root and Desktop lockfiles

Impact

Colossus can connect directly to current Splunk MCP endpoints without mcp-remote. Operators can choose explicit tool allowlists or opt into dynamic wildcard discovery while retaining schema validation, policy authorization, approvals, quarantine, and audit. Desktop users also receive the completed Plan lifecycle and terminal workflow.

Validation

  • cargo xtask check rust
  • cargo xtask pr --base origin/main
  • generated SDK compatibility plus TypeScript, Python, and Go checks
  • Desktop typecheck, 137 unit tests, security contracts, formatting, production build, and bundle-size check
  • root, fuzz, and Desktop dependency policy/RustSec audits

Credential- or environment-gated live tests (Splunk, OPA, Chroma, OCI, and WORM) remain opt-in and were not run without configured endpoints.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 178bba4ecd

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread crates/colossus-api-runtime/src/service.rs Outdated
Comment thread crates/colossus-api-runtime/src/service.rs Outdated
Comment thread crates/colossus-mcp/src/executor.rs Outdated
Preserve unknown outcome certainty across plan terminals and mark remote MCP calls unknown only after dispatch.

Consume publicly approved plans across cancellation and isolate goal execution on its own Tokio task to prevent worker stack overflow.
@killswitch-GUI killswitch-GUI added the ci:full Run cost-bounded full pre-merge acceptance on the current PR head label Jul 31, 2026 — with GitHub Codespaces
@killswitch-GUI
killswitch-GUI merged commit 954efcf into main Jul 31, 2026
27 of 28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:full Run cost-bounded full pre-merge acceptance on the current PR head

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant