feat: add gladia-using-cli skill#13
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThis PR adds a new Gladia CLI skill page documenting terminal-based transcription workflows, authentication, commands, flags, and agent guidance. It also adds a CLI-vs-SDK routing reference, and updates the plugin README and related SDK/pre-recorded transcription skill docs with cross-links to the new CLI skill. ChangesGladia CLI documentation updates
Estimated code review effort: 2 (Simple) | ~10 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (2)
plugins/gladia/skills/gladia-using-cli/SKILL.md (2)
42-42: 🔒 Security & Privacy | 🔵 Trivial | 💤 Low valueConsider adding checksum verification to the install script.
The
curl | shpattern is convenient but bypasses package managers and code review. For a trusted official repository this is acceptable, but consider adding a note about verifying the install script checksum or using a package manager when available. Not blocking given the official source.🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@plugins/gladia/skills/gladia-using-cli/SKILL.md` at line 42, The install command in the Gladia CLI skill uses a direct curl-to-shell pattern without any verification guidance. Update the SKILL.md instructions around the install step to mention verifying the install script checksum or preferring a package manager when available, and keep the guidance tied to the current `curl -fsSL ... | sh` install flow so users have a safer alternative.
27-36: 📐 Maintainability & Code Quality | 🔵 Trivial | ⚡ Quick winAdd
gladia-sdk-integrationto the References section.The References section is missing a link to
../gladia-sdk-integration/SKILL.md. Per the guideline, skills that generate code should cross-referencesdk-integrationfor setup, and the References section is the standard location for skill-to-skill links. This is already done in the callout and "When NOT to Use," but adding it to References ensures agents discover it at load time.- ./references/cli-vs-sdk.md -- Full feature matrix: CLI flag vs SDK skill routing - ../gladia-pre-recorded-transcription/SKILL.md -- SDK pre-recorded workflow and options - ../gladia-audio-intelligence/SKILL.md -- Addons beyond CLI flags +- ../gladia-sdk-integration/SKILL.md -- SDK setup and initialization - ../gladia-troubleshooting/SKILL.md -- API key, upload, and polling errors - [gladia-cli repository](https://github.com/gladiaio/gladia-cli/)🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@plugins/gladia/skills/gladia-using-cli/SKILL.md` around lines 27 - 36, The References section in gladia-using-cli is missing the cross-link to the SDK setup skill, so add `../gladia-sdk-integration/SKILL.md` alongside the existing skill references. Update the References list in the `SKILL.md` for `gladia-using-cli` so agents can discover `gladia-sdk-integration` at load time, matching the existing pattern used by the callout and “When NOT to Use” guidance.Source: Coding guidelines
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@plugins/gladia/skills/gladia-using-cli/SKILL.md`:
- Line 73: The `--model` flag description in the Gladia CLI skill has a
duplicated model name; update the table entry in `SKILL.md` so the second listed
option in the `--model` row uses the correct value from the CLI docs,
`solaria-3`, instead of repeating `solaria-1`.
---
Nitpick comments:
In `@plugins/gladia/skills/gladia-using-cli/SKILL.md`:
- Line 42: The install command in the Gladia CLI skill uses a direct
curl-to-shell pattern without any verification guidance. Update the SKILL.md
instructions around the install step to mention verifying the install script
checksum or preferring a package manager when available, and keep the guidance
tied to the current `curl -fsSL ... | sh` install flow so users have a safer
alternative.
- Around line 27-36: The References section in gladia-using-cli is missing the
cross-link to the SDK setup skill, so add `../gladia-sdk-integration/SKILL.md`
alongside the existing skill references. Update the References list in the
`SKILL.md` for `gladia-using-cli` so agents can discover
`gladia-sdk-integration` at load time, matching the existing pattern used by the
callout and “When NOT to Use” guidance.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 7db01cc5-d49f-4fdf-a375-16e2b16743fd
📒 Files selected for processing (5)
plugins/gladia/README.mdplugins/gladia/skills/gladia-pre-recorded-transcription/SKILL.mdplugins/gladia/skills/gladia-sdk-integration/SKILL.mdplugins/gladia/skills/gladia-using-cli/SKILL.mdplugins/gladia/skills/gladia-using-cli/references/cli-vs-sdk.md
karamouche
force-pushed
the
feat/gladia-cli-skill
branch
from
cec438a to
9047c1b
Compare
There was a problem hiding this comment.
🧹 Nitpick comments (1)
plugins/gladia/skills/gladia-using-cli/SKILL.md (1)
39-43: 🔒 Security & Privacy | 🔵 Trivial | ⚡ Quick winAdd checksum verification or safer install alternative.
The
curl … | shpattern bypasses code review of the executed script. Consider adding a note to verify the release checksum or prefer a package-manager install when available.```bash - curl -fsSL https://github.com/gladiaio/gladia-cli/releases/latest/download/install.sh | sh + # Verify checksum before executing (see releases page for SHA256) + curl -fsSL https://github.com/gladiaio/gladia-cli/releases/latest/download/install.sh -o install.sh + sh install.shOr document `pip install gladia-cli` / `npm install -g gladia-cli` if those are official options. <details> <summary>🤖 Prompt for AI Agents</summary>Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.In
@plugins/gladia/skills/gladia-using-cli/SKILL.mdaround lines 39 - 43, The
install instructions currently use a direct curl-to-shell pattern, so update the
Gladia CLI setup guidance to avoid executing an unchecked script. In the
SKILL.md install section, revise the shell install example to download the
script first and add a checksum verification note, or replace it with an
official package-manager install path if one exists. Keep the change localized
to the install snippet and surrounding guidance so users can safely install
gladia-cli.</details> <!-- cr-comment:v1:92925ecd3068a0e43dc43dec --> </blockquote></details> </blockquote></details> <details> <summary>🤖 Prompt for all review comments with AI agents</summary>Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.Nitpick comments:
In@plugins/gladia/skills/gladia-using-cli/SKILL.md:
- Around line 39-43: The install instructions currently use a direct
curl-to-shell pattern, so update the Gladia CLI setup guidance to avoid
executing an unchecked script. In the SKILL.md install section, revise the shell
install example to download the script first and add a checksum verification
note, or replace it with an official package-manager install path if one exists.
Keep the change localized to the install snippet and surrounding guidance so
users can safely install gladia-cli.</details> --- <details> <summary>ℹ️ Review info</summary> <details> <summary>⚙️ Run configuration</summary> **Configuration used**: defaults **Review profile**: CHILL **Plan**: Pro **Run ID**: `d571dda9-5160-4082-a6da-050c0d2dedf7` </details> <details> <summary>📥 Commits</summary> Reviewing files that changed from the base of the PR and between cec438ad5eb6efccfe42fbb57eea8a1fa8786ea1 and 9047c1b6e89a067967c39102b7faddaec0184637. </details> <details> <summary>📒 Files selected for processing (5)</summary> * `plugins/gladia/README.md` * `plugins/gladia/skills/gladia-pre-recorded-transcription/SKILL.md` * `plugins/gladia/skills/gladia-sdk-integration/SKILL.md` * `plugins/gladia/skills/gladia-using-cli/SKILL.md` * `plugins/gladia/skills/gladia-using-cli/references/cli-vs-sdk.md` </details> <details> <summary>✅ Files skipped from review due to trivial changes (4)</summary> * plugins/gladia/README.md * plugins/gladia/skills/gladia-pre-recorded-transcription/SKILL.md * plugins/gladia/skills/gladia-sdk-integration/SKILL.md * plugins/gladia/skills/gladia-using-cli/references/cli-vs-sdk.md </details> </details> <!-- This is an auto-generated comment by CodeRabbit for review status -->
…ty and consistency
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@plugins/gladia/skills/gladia-using-cli/SKILL.md`:
- Around line 96-103: The summary table in SKILL.md is out of sync with the CLI
model options because the Model choice row in the table only mentions solaria-1
even though the earlier flag table also includes solaria-3. Update the Model
choice entry in the summary table so it mirrors the available model options
exactly, using the same wording and identifiers as the CLI flag section to keep
both references aligned.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: c5c450c9-f931-4c4a-9668-e999508953d5
📒 Files selected for processing (2)
plugins/gladia/skills/gladia-using-cli/SKILL.mdplugins/gladia/skills/gladia-using-cli/references/cli-vs-sdk.md
✅ Files skipped from review due to trivial changes (1)
- plugins/gladia/skills/gladia-using-cli/references/cli-vs-sdk.md
2 participants
Summary by CodeRabbit