Use Codex app-server install and drop global rescue agent#15
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 60d5b031aa
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| if (changed) { | ||
| writeConfigFile(nextContent); | ||
| const pluginRemoval = removeTomlSections(existing, new Set([PLUGIN_CONFIG_HEADER])); |
There was a problem hiding this comment.
Remove legacy managed cc-rescue registration during uninstall
This uninstall cleanup now removes only [plugins."cc@local-plugins"], so users upgrading from earlier releases keep the previously managed cc-rescue global-agent artifacts ([agents."cc-rescue"] in config.toml and ~/.codex/agents/cc-rescue.toml). After uninstall deletes the plugin directory, that leftover agent registration becomes stale and can still surface a broken rescue agent path; uninstall should also prune the managed legacy agent block/file (while leaving user-customized agent entries intact).
Useful? React with 👍 / 👎.
Summary
cc-rescueagent and keep only managed Codex hooks outside the plugin directoryTesting