Skip to content

Add top-level skills install command#35

Merged
sdairs merged 22 commits intobump-v0.1.12from
codex/skills-command
Mar 16, 2026
Merged

Add top-level skills install command#35
sdairs merged 22 commits intobump-v0.1.12from
codex/skills-command

Conversation

@sdairs
Copy link
Copy Markdown
Collaborator

@sdairs sdairs commented Mar 14, 2026

Summary

  • add a new top-level \ command for installing bundled ClickHouse agent skills
  • support interactive multi-select installs plus non-interactive \ and \ flows
  • detect supported agent folders, write the bundled skill package, and document the new command

Testing

  • cargo test

@sdairs sdairs force-pushed the codex/skills-command branch from 4ca285c to e708850 Compare March 14, 2026 10:36
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new top-level clickhousectl skills command that installs bundled ClickHouse agent skills into supported agent configuration directories, with interactive selection and non-interactive flag-driven flows.

Changes:

  • Introduce skills subcommand + CLI args, wiring it into main command dispatch and error handling.
  • Implement download + tarball extraction + installation logic for agent skills, including interactive TTY pickers and unit tests.
  • Document the new command and ignore common agent config directories in this repo.

Reviewed changes

Copilot reviewed 13 out of 15 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
src/skills.rs Implements skills download/extract/install logic, interactive selection UI, and tests.
src/cli.rs Adds skills subcommand and SkillsArgs parsing/tests; updates help text.
src/main.rs Wires the new skills command into top-level dispatch.
src/error.rs Adds Error::Skills for skills-related user-facing errors.
README.md Documents clickhousectl skills usage, supported agents, and scope behavior.
Cargo.toml / Cargo.lock Adds flate2 + tar dependencies for archive handling.
.gitignore Ignores agent config directories in this repo.
src/version_manager/*, src/server.rs, src/cloud/* Formatting-only changes (no functional deltas apparent in diff).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Validate empty cloud tag keys before request build
@sdairs sdairs merged commit 427475b into bump-v0.1.12 Mar 16, 2026
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