Replace parallel skill with official parallel-cli wrapper#31
Conversation
Rewrites the parallel skill from a 262-line Python HTTP client to a thin bash wrapper around the official parallel-cli binary. The wrapper auto-installs the CLI on first use and guards against missing API keys to prevent interactive prompts in automated contexts. New capabilities via parallel-cli: deep research with processor tiers, CSV/JSON data enrichment, entity discovery (findall), and web change monitoring — on top of the existing search and extract. Fleet-wide changes: - Added parallel-cli to Brewfile (macOS) and install instructions (Linux) - Added verification to both machine setup compliance checks - Cross-platform installer (parallel.ai/install.sh) as canonical method - SKILL.md rewritten with comprehensive command reference and decision table Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
![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: 0ccf3389f0
ℹ️ 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 [[ -x "$HOME/.local/bin/parallel-cli" ]]; then | ||
| export PATH="$HOME/.local/bin:$PATH" | ||
| else | ||
| install_parallel_cli | ||
| fi |
There was a problem hiding this comment.
The wrapper attempts install_parallel_cli before it validates PARALLEL_API_KEY, so on fresh machines without parallel-cli a call like parallel search ... with no key takes the network install path first instead of failing fast with the missing-key error. In cron/gateway contexts this defeats the stated non-interactive guard and can block/fail for installation reasons unrelated to auth, which is a regression from the previous implementation that errored on missing key immediately.
Useful? React with 👍 / 👎.
| case "${1:-}" in | ||
| --version|--help|-h|help) ;; | ||
| *) | ||
| if [[ -z "${PARALLEL_API_KEY:-}" ]]; then |
There was a problem hiding this comment.
The key gate only exempts explicit help/version tokens, so an empty invocation (parallel) falls into the default branch and requires PARALLEL_API_KEY. This regresses the prior behavior where no args showed help, and now users without a key cannot discover usage unless they already know to run help or --help.
Useful? React with 👍 / 👎.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Move API key check before parallel-cli install check so missing key fails fast without triggering network install on fresh machines - Add empty invocation to help bypass (no args shows CLI help) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
1 participant
Summary
parallel-clibinaryparallel.ai/install.sh(cross-platform: macOS + Linux)PARALLEL_API_KEYto prevent interactive login prompts in cron/gateway contextsparallel-clito fleet machine setup specs (Brewfile, apt-packages, both compliance checks)New capabilities (via parallel-cli)
researchenrichfindallmonitorFiles changed
skills/parallel/parallelskills/parallel/SKILL.mdtests/test_parallel.pyBrewfileparallel-web/tap/parallel-clidevops/apt-packages.txtdevops/machine-setup.mddevops/machine-setup-linux.mdREADME.mdVERSIONTest plan
--version/--helpwork without API key/fleetand verifyparallel-cli --versionon all machines🤖 Generated with Claude Code