-
Notifications
You must be signed in to change notification settings - Fork 333
fix(codex): auto-restore the codex shim after external npm updates (#320) #356
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
5b11dad
11319c1
ae18374
ce65a75
916351f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -397,8 +397,12 @@ ocx service uninstall | |||||
| Оборачивает скриптовый лаунчер `codex` в PATH лёгким скриптом автозапуска. Настоящие цели | ||||||
| `codex.exe` не затрагиваются, чтобы не ломать вызовы точного исполняемого файла. | ||||||
|
|
||||||
| Если обновление Codex перезаписало обёртку, shim автоматически восстанавливается при следующем | ||||||
| вызове `install` — новый бинарный файл резервируется, и записывается свежая обёртка. | ||||||
| Если завершённое внешнее обновление Codex перезаписало установленный shim, следующая обычная команда | ||||||
| `ocx` до запуска сохранит стабильный новый лончер в резервную копию и восстановит shim. Лончер, | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win Исправьте опечатку в пользовательской документации. Используйте стандартную транслитерацию Предлагаемое исправление- до запуска сохранит стабильный новый лончер в резервную копию и восстановит shim. Лончер,
+ до запуска сохранит стабильный новый лаунчер в резервную копию и восстановит shim. Лаунчер,📝 Committable suggestion
Suggested change
🤖 Prompt for AI Agents |
||||||
| который ещё меняется, остаётся нетронутым до следующей попытки. Ошибка восстановления выдаёт | ||||||
| предупреждение, но не приводит к сбою запрошенной команды; ручной вариант — | ||||||
| `ocx codex-shim install`. Для отключения установите `codexShimAutoRestore` в `false` или задайте | ||||||
| процессу `OPENCODEX_CODEX_SHIM_AUTO_RESTORE=0`. | ||||||
|
|
||||||
| | Subcommand | Action | | ||||||
| | --- | --- | | ||||||
|
|
||||||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎯 Functional Correctness | 🟡 Minor | ⚡ Quick win
Clarify the
statusside effect consistently across locales.The CLI leaves
ocx codex-shim statuseligible for auto-restore, so it can rewrite launcher files before reporting state. Update each page to state this explicitly, or change the policy ifstatusmust remain read-only.docs-site/src/content/docs/reference/cli.md#L375-L379: document that Englishstatusinvocations may perform repair.docs-site/src/content/docs/ko/reference/cli.md#L379-L383: add the same side-effect warning to the Korean flow.docs-site/src/content/docs/ru/reference/cli.md#L400-L405: add the same side-effect warning to the Russian flow.As per path instructions, translated docs must remain synchronized with actual CLI behavior.
📍 Affects 3 files
docs-site/src/content/docs/reference/cli.md#L375-L379(this comment)docs-site/src/content/docs/ko/reference/cli.md#L379-L383docs-site/src/content/docs/ru/reference/cli.md#L400-L405🤖 Prompt for AI Agents
Source: Path instructions