Skip to content

Sync hf CLI skills#69

Merged
hanouticelina merged 5 commits intohuggingface:mainfrom
hanouticelina:sync-hf-cli-skills
Mar 6, 2026
Merged

Sync hf CLI skills#69
hanouticelina merged 5 commits intohuggingface:mainfrom
hanouticelina:sync-hf-cli-skills

Conversation

@hanouticelina
Copy link
Collaborator

@hanouticelina hanouticelina commented Mar 5, 2026

Following some users feedback, we made the skill much more token-efficient (~12k tokens -> ~1.3k tokens) so it actually works with open models that can't handle that many tokens at once. The skill is now auto-generated by introspecting the CLI's command tree.

We're also setting up a GitHub Action in huggingface_hub huggingface/huggingface_hub#3885 that will automatically open a PR here whenever we cut a new release, so the skill will always stay in sync with the CLI.

@@ -1,186 +1,130 @@
---
name: hugging-face-cli
description: Execute Hugging Face Hub operations using the `hf` CLI. Use when the user needs to download models/datasets/spaces, upload files to Hub repositories, create repos, manage local cache, or run compute jobs on HF infrastructure. Covers authentication, file transfers, repository creation, cache operations, and cloud compute.
name: hf-cli
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

cc @evalstate I have no strong opinion here, should we rename the skill to hugging-face-cli in huggingface_hub ?

Copy link
Collaborator

Choose a reason for hiding this comment

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

just note, we are about to fix all skills on this repo hugging-face-* to huggingface-*

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

is it okay if we keep the SKILL name hf-cli ? huggingface-cli is the name of the deprecated command and we want to avoid this name as much as possible since most LLMs still think that huggingface-cli still exists. wdyt?

Copy link
Collaborator

Choose a reason for hiding this comment

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

No really strong views on the name.

This skill is a little unusual in that we want:

  • To trigger when the user wants to achieve something with "hugging face" or "hub"
  • To (hopefully) intercept when the model would otherwise have used the deprecated "huggingface_hub" command.

For things following the spec it should look in the system prompt like:

<available_skills>

hf-cli
(Our description)
/home/.wherever/SKILL.md

</available_skills>

I think all the proposed names are reasonably good options that have a good chance of triggering the skill... I'd go for whatever looked aesthetically best :)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

very clear, thanks @evalstate! keeping hf-cli then, we can reassess if needed :)

Copy link
Collaborator Author

@hanouticelina hanouticelina Mar 6, 2026

Choose a reason for hiding this comment

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

btw is it okay to merge even though the validation workflow is failing? 🙈

Copy link
Collaborator

Choose a reason for hiding this comment

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

Updates ->

Rename directory to match skill name (spec requirement)
Update .claude/marketplace.json manually maintained metadata.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

LGTM thank you!

Update marketplace metadata to match name
@hanouticelina hanouticelina merged commit c8f3ecd into huggingface:main Mar 6, 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.

3 participants