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

Merged
xsh310 merged 13 commits into from
Aug 11, 2026
+421 −134
Merged

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
13 commits
Select commit Hold shift + click to select a range
17268d4
skills: prompt before fetching so declined skills aren't downloaded
xsh310 Jul 31, 2026
eeb9fd7
skills: clarify download_skills docstring and ruff format
xsh310 Jul 31, 2026
d5a373a
skills: read skill bundles from the Files API `/Skills` place
xsh310 Aug 6, 2026
9745034
skills: address bundles by securable, name dirs by bundle name
xsh310 Aug 6, 2026
e37d4b5
skills: name SkillRef fields for the API, scope --skill to the securable
xsh310 Aug 6, 2026
ce18a41
skills: skip and warn when a finalized skill is missing either name
xsh310 Aug 6, 2026
a378fa9
skills: drop siblings that claim the same bundle name
xsh310 Aug 6, 2026
cc6666d
skills: drop the client-side name check, let the server own naming
xsh310 Aug 7, 2026
104c623
skills: name the skip count in the download summary
xsh310 Aug 7, 2026
aaa60ef
skills: fix stale download docstring and rename _write_bundle param
xsh310 Aug 11, 2026
163b4ef
Merge branch 'main' into skills-dedup-before-fetch-main
xsh310 Aug 11, 2026
c9a677c
Merge branch 'main' into skills-dedup-before-fetch-main
xsh310 Aug 11, 2026
5f976fa
Merge branch 'main' into skills-dedup-before-fetch-main
xsh310 Aug 11, 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 @@ -2377,9 +2377,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