Skip to content

Customisation for Codex CLI - Features from Claude Code and OpenCode#336

Draft
TGreen87 wants to merge 4 commits intoaffaan-m:mainfrom
TGreen87:main
Draft

Customisation for Codex CLI - Features from Claude Code and OpenCode#336
TGreen87 wants to merge 4 commits intoaffaan-m:mainfrom
TGreen87:main

Conversation

@TGreen87
Copy link

@TGreen87 TGreen87 commented Mar 5, 2026 •
edited by cubic-dev-ai bot
Loading

Description

Type of Change

  • fix: Bug fix
  • feat: New feature
  • refactor: Code refactoring
  • docs: Documentation
  • test: Tests
  • chore: Maintenance/tooling
  • ci: CI/CD changes

Checklist

  • Tests pass locally (node tests/run-all.js)
  • Validation scripts pass
  • Follows conventional commits format
  • Updated relevant documentation

Summary by cubic

Adds ECC tooling to Codex CLI: global git safety hooks, a sync script that backs up and updates ~/.codex (AGENTS + Codex supplement, skills, prompts, QA/rule packs), and MCP servers normalized to pnpm dlx. Adds a sanity checker to validate the global Codex state for consistent, safe setups.

  • New Features

    • Pre-commit secret scanner to block high-signal secrets.
    • Pre-push verifier for lint, typecheck, test, build; auto-detects package manager; runs Go tests and Python pytest when present; optional audit via ECC_PREPUSH_AUDIT=1.
    • Installer for global git hooks via core.hooksPath.
    • ECC sync script to backup and update ~/.codex (AGENTS + Codex supplement), sync skills, generate command prompts and tool prompts (run-tests, check-coverage, security-audit), add optional rule packs (common, TypeScript, Python, Go, Swift), and normalize MCP servers to pnpm dlx for @modelcontextprotocol/*, @upstash/context7-mcp, @playwright/mcp, with GitHub PAT bootstrap via gh.
    • Global sanity check to validate config (profiles, MCP sections, no duplicate Context7), hooksPath, executable hooks, skills count, and prompt manifests.

  • Migration

    • Ensure pnpm is installed; MCP servers now run via pnpm dlx.
    • Run scripts/sync-ecc-to-codex.sh (supports --dry-run), then restart Codex CLI.
    • To bypass hooks: set ECC_SKIP_PRECOMMIT=1 or ECC_SKIP_PREPUSH=1; to disable per repo, add .ecc-hooks-disable.

Written for commit d411887. Summary will update on new commits.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Mar 5, 2026 •
edited
Loading

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 735be779-04ee-4357-924d-e085e252fd5f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
📝 Coding Plan
  • Generate coding plan for human review comments

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@affaan-m
Copy link
Owner

affaan-m commented Mar 11, 2026

Hey @TGreen87 — this PR has merge conflicts with main. Could you rebase or merge main into your branch? Happy to take another look once that's sorted.

@affaan-m
Copy link
Owner

affaan-m commented Mar 13, 2026

Rebuilt this branch on top of current main and force-pushed the refreshed head.

Restack notes:

  • replayed the four original Codex-focused commits onto current main
  • kept the current .codex/config.toml from main instead of reintroducing the older pnpm-only MCP config from this draft branch
  • preserved the additive Codex scripts/hooks from the PR

Validation in the restack worktree:

  • git diff --check
  • bash -n on the added shell scripts and git hooks

@Yongle2025
Copy link

Yongle2025 commented Mar 13, 2026 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@TGreen87 @affaan-m @Yongle2025