Replace Composio Connect with project sessions - #165
Merged
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (25)
📝 WalkthroughWalkthroughComposio integration now uses project API keys and reusable Sessions. The server persists Session metadata, supports external secret storage, and exposes Session-based MCP and toolkit operations. Electron stores keys securely, while the UI and documentation use Composio Sessions terminology. ChangesComposio Sessions integration
Estimated code review effort: 4 (Complex) | ~60 minutes Sequence Diagram(s)sequenceDiagram
participant SettingsUI
participant ElectronIPC
participant ServerConfig
participant ComposioSessions
participant ClaudeDriver
SettingsUI->>ElectronIPC: setCredential("composioApiKey", value)
ElectronIPC->>ServerConfig: update Composio configuration
ServerConfig->>ComposioSessions: validate or create Session
ComposioSessions-->>ServerConfig: Session metadata
ServerConfig-->>ElectronIPC: configured status
ClaudeDriver->>ServerConfig: request Composio integration
ServerConfig->>ComposioSessions: retrieve Session MCP details
ComposioSessions-->>ClaudeDriver: MCP URL and x-api-key headers
Possibly related PRs
Suggested reviewers: ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
milind-soni
marked this pull request as ready for review
August 17, 2026 06:12
kargnas
added a commit
to kargnas/OpenMausBot
that referenced
this pull request
Aug 17, 2026
upstream 17 커밋(milind-soni#137~milind-soni#165: webhook 자동모드 게이트, reasoning-effort per-bot, Composio project sessions, Local VM 수명주기 강화, CUA 강화 등)을 머지했다. fork 고유 기능(단일 인스턴스 잠금, 서버 복구, CLI 경로 오버라이드, 동적 모델 catalog, dev:desktop 통합 실행)은 그대로 보존했다. 충돌 해소 요점: - effort 타입을 upstream EffortLevel 유니언으로 좁히고 fork의 serviceTier/ modelProvider 필드를 병합保留 - ACP catalog probe 실패 시 throw 대신 error-catalog로 degradation — probe가 turn 시작을 거부하지 않게 (CLI가 모델 id의 최종 판정자) - PATCH/startTurn의 모델 존재 검증을 완화: catalog는 계정 구동이라 존재하지 않는 모델 id는 CLI로 넘기고, effort/tier 조합 검증만 catalog 대상 유지 - grok/claude/codex에 upstream effortLevels 선언 반영 Tested: pnpm typecheck; pnpm test (63 files, 517 passed) Constraint: fork 동적 catalog와 upstream 정적 models 인터페이스가 충돌 Rejected: upstream 정적 models로 회귀 | fork catalog 기능 상실 Confidence: high Scope-risk: moderate Reversibility: moderate
This was referenced Aug 17, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
Why
Fresh installs could not use connected apps because the UI still asked for the old Connect key, while current Composio projects use project API keys and Sessions. The old split-key setup also made onboarding needlessly confusing.
User impact
A self-hosting user now creates or copies one
ak_…project key, pastes it once, and connects providers from the Connected apps screen. Provider OAuth tokens remain in Composio; OpenMausBot stores only its project key and non-secret Session identifiers.The packaged desktop app encrypts the project key with the operating system credential store. Source/headless installs can use
COMPOSIO_API_KEY.Validation
pnpm test: 471 passed, 8 skippedpnpm typecheckpnpm check:electronpnpm buildpnpm package:preparegit diff --checkRemaining live check
One real provider authorization should be completed with a real Composio project key before marking this ready for merge.
Summary by CodeRabbit
New Features
Bug Fixes
Documentation