Skip to content

Conversation

@pakrym-oai
Copy link
Collaborator

@pakrym-oai pakrym-oai commented Oct 20, 2025

Summary

  • make the plan tool available by default by removing the feature flag and always registering the handler
  • drop plan-tool CLI and API toggles across the exec, TUI, MCP server, and app server code paths
  • update tests and configs to reflect the always-on plan tool and guard workspace restriction tests against env leakage

Testing

Manually tested the extension.

https://chatgpt.com/codex/tasks/task_i_68f67a3ff2d083209562a773f814c1f9

Copy link
Collaborator

@jif-oai jif-oai left a comment

Choose a reason for hiding this comment

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

Easy to review :D

@@ -0,0 +1,10 @@
#!/bin/bash
Copy link
Collaborator

Choose a reason for hiding this comment

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

huh?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Makes it easy to wire codex CLI to vscode for testing. otherwise you have to point to /Users/pakrym/code/code/codex-rs/target/debug/codex and keep rebuilding it.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I'll ping the channel to let people know.

@pakrym-oai pakrym-oai enabled auto-merge (squash) October 21, 2025 16:22
@pakrym-oai pakrym-oai merged commit 1b10a3a into main Oct 21, 2025
20 checks passed
@pakrym-oai pakrym-oai deleted the codex/make-plan-tool-enabled-by-default branch October 21, 2025 16:25
@github-actions github-actions bot locked and limited conversation to collaborators Oct 21, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants