chore(cli): migrate from scripts/cookiy.sh to npm cookiy-cli - #39
Open
yupeng-dev wants to merge 5 commits into
Open
chore(cli): migrate from scripts/cookiy.sh to npm cookiy-cli#39yupeng-dev wants to merge 5 commits into
yupeng-dev wants to merge 5 commits into
Conversation
The bundled shell CLI is superseded by the standalone `cookiy-cli`
TypeScript package published to npm:
https://www.npmjs.com/package/cookiy-cli
https://github.com/cookiy-ai/cookiy-cli
Changes:
- Remove references/cookiy/scripts/cookiy.sh
- Replace every `scripts/cookiy.sh <cmd>` example with `npx cookiy-cli <cmd>`
across all module reference docs
- Add Install / Upgrade section to references/cookiy/cookiy.md with
npx / npm install -g / npm update -g instructions
Supersedes #38 (the Node.js rewrite intermediate step).
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
7 tasks
All module examples now invoke `cookiy <subcommand>` directly — the
`npx cookiy-cli <subcommand>` wording was a transition crutch. The
expected flow is:
npm install -g cookiy-cli # first-time only
cookiy <subcommand>
Rewrote Install / Upgrade section in references/cookiy/cookiy.md to
tell agents to verify `command -v cookiy` and install globally before
running commands. All 8 module reference docs now use the `cookiy`
binary form exclusively.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Matches cookiy-cli@1.21.1 which removed all server-side wait/polling flags (the shell CLI had already dropped them in commit 3885545). - Remove `study guide wait` subcommand section - Remove `study report wait` subcommand section - Rewrite qual.md's "Waiting on Async Operations" to point at `cookiy study status` as the sole progress-check path Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Previous commit (0ea460b) used `git add -A` which slurped up untracked audit artifacts that are tracked separately. Revert them back to untracked; only the CLI migration + wait-removal docs belong in this PR. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The prior removal was too aggressive — cookiy-cli still supports both `study guide wait` and `study report wait` (reintroduced in 1.21.7). Restore the corresponding reference sections and update cookiy-qual.md to point back at them. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The bundled shell CLI
references/cookiy/scripts/cookiy.shis superseded by the standalonecookiy-cliTypeScript package distributed via npm.1.21.0(tracks the shell CLI's last version)What changed in this repo
references/cookiy/scripts/cookiy.sh(977 lines)scripts/cookiy.sh <cmd>example withnpx cookiy-cli <cmd>across all module docs (8 files)references/cookiy/cookiy.mdwithnpx/npm install -g/npm update -gguidanceHow agents should use it now
Requires Node.js 18+.
Supersedes
Closes #38 (the intermediate Node.js rewrite). That PR was part of the same migration path and is replaced by this one + the new standalone repo.
Test plan
cookiy-cli@1.21.0published to npm (verifiednpx cookiy-cli --version)cookiy-clilive-tested locally:billing balance,study list,quant list,quant get,billing transactionsreturn identical JSON structure to the removed shell CLIclaude plugin validateafter merge (SKILL.md + plugin manifests unaffected, expected pass)🤖 Generated with Claude Code