Skip to content

skills: fix the download path (Files API place, securable vs bundle name) and prompt before fetching #253

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
xsh310 wants to merge 8 commits into
base: main
Choose a base branch
from
+432 −124
Open

skills: fix the download path (Files API place, securable vs bundle name) and prompt before fetching #253

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
415e7c6
skills: prompt before fetching so declined skills aren't downloaded
xsh310 Jul 31, 2026
e056755
skills: clarify download_skills docstring and ruff format
xsh310 Jul 31, 2026
0768e1a
skills: read skill bundles from the Files API `/Skills` place
xsh310 Aug 6, 2026
fe1295b
skills: address bundles by securable, name dirs by bundle name
xsh310 Aug 6, 2026
410f21f
skills: name SkillRef fields for the API, scope --skill to the securable
xsh310 Aug 6, 2026
94fd4af
skills: skip and warn when a finalized skill is missing either name
xsh310 Aug 6, 2026
8834d54
Merge branch 'main' into skills-dedup-before-fetch-main
xsh310 Aug 6, 2026
3f36cfd
skills: drop siblings that claim the same bundle name
xsh310 Aug 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions src/ucode/cli.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1931,9 +1931,9 @@ def configure_skills(
str | None,
typer.Option(
"--skill",
help="(download) Download only this comma-separated subset of skills (by leaf "
"name, e.g. `my-skill`) from the schema, instead of every skill. Requires a "
"single --location; not valid with --mcp.",
help="(download) Download only this comma-separated subset of skills (by "
"securable name, e.g. `my-skill`) from the schema, instead of every skill. "
"Requires a single --location; not valid with --mcp.",
),
] = None,
) -> None:
Expand Down
Loading
Loading