fix(ci): sync CLI version with v1.41.0 - #100
Conversation
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c9f3be5d46
ℹ️ 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".
| g.add_argument( | ||
| "--tui", | ||
| action="store_true", |
There was a problem hiding this comment.
Every CLI invocation and every import of zcoder.main now fails immediately because this g.add_argument( call is left unclosed and the remaining 4,102 lines—including main()—were deleted. I confirmed python -m py_compile src/zcoder/main.py raises SyntaxError: '(' was never closed, while pyproject.toml still maps the zcoder executable to zcoder.main:main; restore the deleted body while retaining the intended version update.
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d3f25a5fb9
ℹ️ 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".
| return | ||
| if args.spend_limit_requests_list: | ||
| from zcoder.claude.enterprise.admin_api import cmd_spend_limit_requests_list | ||
| from zcoder.claude.enterprise.admin_api import cmd_spend_limits_requests_list |
There was a problem hiding this comment.
When a user runs zcoder --spend-limit-requests-list, this import raises ImportError before making any API request because admin_api.py defines cmd_spend_limit_requests_list (singular limit), not cmd_spend_limits_requests_list. I reproduced the failure with an admin key argument; keep the existing helper name here so this CLI command remains usable.
Useful? React with 👍 / 👎.
1 participant
Superseded by current
main.The intended version fix is already present on
main(VERSION = "1.41.0"), and the later CLI wiring regression reported by this PR's stale CI is also already fixed on currentmain:cmd_spend_limit_requests_listis wired throughzcoder.main.This branch diverged from current
main, so merging it would reintroduce stale code/conflicts. Closing instead of bypassing CI or merging an outdated head.Current main baseline:
786579a048574dbf52d4807d3bc1c7923b08a27a.