Skip to content

Add Codex platform support#219

Closed
clcc2019 wants to merge 3 commits intotirth8205:mainfrom
clcc2019:main
Closed

Add Codex platform support#219
clcc2019 wants to merge 3 commits intotirth8205:mainfrom
clcc2019:main

Conversation

@clcc2019
Copy link
Copy Markdown

code-review-graph install: error: argument --platform: invalid choice: 'codex' (choose from claude, claude-code, cursor, windsurf, zed, continue, opencode, antigravity, all)

@tirth8205
Copy link
Copy Markdown
Owner

Skipping auto-merge: after updating against current main, CI shows this conflicts with existing Codex support that already landed via #177.

Specific failures:

  • code_review_graph/skills.py:126F601 duplicate "codex" key in PLATFORMS dict (both this branch and main add it)
  • tests/test_skills.py::TestInstallPlatformConfigs::test_install_codex_preserves_existing_tomlKeyError: 'code-review-graph'
  • tests/test_skills.py::TestInstallPlatformConfigs::test_install_all_detectedAssertionError: assert 'Codex' in ['Claude Code', 'OpenCode']

Since #177 already delivers Codex MCP install support, this PR is effectively duplicate work. Could you rebase and either (a) drop the overlap and keep only the extras here, or (b) close this if #177 covers everything you needed?

@clcc2019 clcc2019 closed this Apr 12, 2026
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.

2 participants