Skip to content

Conversation

@breaking-brake
Copy link
Owner

Problem

When running a workflow with Skills via Codex CLI, the project trust setting was automatically added to ~/.codex/config.toml without user confirmation.

Current Behavior

  1. User clicks "Run for Codex CLI"
  2. ❌ Project trust is silently added to config.toml without any notification

Expected Behavior

  1. User clicks "Run for Codex CLI"
  2. ✅ Confirmation dialog appears explaining why the setting is needed
  3. User can choose to proceed or cancel

Solution

Added a confirmation dialog that explains the purpose of the project trust setting before adding it to the config file.

Changes

File: src/extension/services/codex-mcp-sync-service.ts

  • Added needsProjectTrustForCodexCli() function to check if project trust needs to be added

File: src/extension/commands/codex-handlers.ts

  • Modified Step 0 to show confirmation dialog before adding project trust
  • Dialog message: "To allow Codex CLI to recognize skills in this project, a project trust setting needs to be added to ~/.codex/config.toml."
  • User can cancel the workflow execution by selecting "No"

Impact

  • Improved UX: Users are now informed about config changes before they happen
  • No breaking changes
  • Workflow execution can be cancelled if user declines

Testing

  • Manual E2E testing completed
  • Code quality checks passed (npm run build)

🤖 Generated with Claude Code

- Added needsProjectTrustForCodexCli function to check if trust is needed
- Show explanation dialog before adding project trust to ~/.codex/config.toml
- Allow user to cancel workflow execution if they decline

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@breaking-brake breaking-brake merged commit e5523d5 into main Jan 24, 2026
3 checks passed
@breaking-brake breaking-brake deleted the improvement/codex-project-trust-confirmation branch January 24, 2026 08:44
@breaking-brake breaking-brake mentioned this pull request Jan 24, 2026
braking-brake-semantic-release bot pushed a commit that referenced this pull request Jan 24, 2026
## [3.18.0](v3.17.0...v3.18.0) (2026-01-24)

### Features

* add OpenAI Codex CLI integration support ([#496](#496)) ([60d5c96](60d5c96)), closes [openai/codex#9752](openai/codex#9752)

### Improvements

* add Codex CLI feature announcement banner ([#497](#497)) ([bf8128a](bf8128a))
* add confirmation dialog for Codex project trust setting ([#499](#499)) ([e5523d5](e5523d5))
* detect skills under .github directory ([#494](#494)) ([eb38d8a](eb38d8a)), closes [#493](#493) [#493](#493) [#6366f1](https://github.com/breaking-brake/cc-wf-studio/issues/6366f1) [#c2410c](https://github.com/breaking-brake/cc-wf-studio/issues/c2410c)
* show "Slash Command / Skill" label when Copilot/Codex enabled ([#498](#498)) ([2be7ac6](2be7ac6))
@braking-brake-semantic-release

🎉 This PR is included in version 3.18.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants