Skip to content

feat: add explicit-source agent skills#72

Merged
lorenjphillips merged 1 commit into
mainfrom
feat/aci-4-skills
May 19, 2026
Merged

feat: add explicit-source agent skills#72
lorenjphillips merged 1 commit into
mainfrom
feat/aci-4-skills

Conversation

@lorenjphillips
Copy link
Copy Markdown
Contributor

Summary

  • Adds coval --agent agent skills list and install.
  • Requires explicit local --source/COVAL_SKILLS_SOURCE; no repo URL or mutable remote catalog is hardcoded.
  • Rejects remote sources with a structured error and tells agents to clone/pin locally.
  • Installs by copying a selected local skill directory into explicit --dest/COVAL_SKILLS_DEST.

Testing

  • cargo fmt
  • cargo clippy --all-targets -- -D warnings
  • cargo test

Stacked PR 4 of 5. Base is feat/aci-3-input-json. No version bump or release changes.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 19, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 099626a2-14c8-4af4-a8e8-a27d58275e2d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/aci-4-skills

Comment @coderabbitai help to get the list of available commands and usage tips.

@lorenjphillips lorenjphillips force-pushed the feat/aci-3-input-json branch from e705f7c to 685a175 Compare May 19, 2026 03:06
@lorenjphillips lorenjphillips force-pushed the feat/aci-3-input-json branch from 685a175 to 66d17fa Compare May 19, 2026 03:11
@lorenjphillips lorenjphillips force-pushed the feat/aci-4-skills branch 3 times, most recently from 194749b to 8f05010 Compare May 19, 2026 03:23
@lorenjphillips lorenjphillips force-pushed the feat/aci-3-input-json branch 2 times, most recently from a336c59 to f0a28a2 Compare May 19, 2026 15:40
@lorenjphillips lorenjphillips force-pushed the feat/aci-4-skills branch 2 times, most recently from 6972b43 to 7988932 Compare May 19, 2026 15:51
@lorenjphillips lorenjphillips force-pushed the feat/aci-3-input-json branch from f0a28a2 to b4802a7 Compare May 19, 2026 15:51
Copy link
Copy Markdown

@callumreid callumreid left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like rust to me!

@lorenjphillips lorenjphillips force-pushed the feat/aci-3-input-json branch from c375320 to 2844fca Compare May 19, 2026 20:20
@lorenjphillips lorenjphillips changed the base branch from feat/aci-3-input-json to main May 19, 2026 20:32
@lorenjphillips lorenjphillips merged commit f178f2f into main May 19, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants