Conversation
xsh310
changed the title
ug skills and quiet download registration
xsh310
commented
Sep 18, 2026
xsh310
marked this pull request as ready for review
September 18, 2026 04:05
xsh310
added this pull request to stack #733
September 18, 2026 04:27
xsh310
force-pushed
the
skills-bare-configure-mcp
branch
2 times, most recently
from
September 18, 2026 05:18
58baa69 to
fdd55b2
Compare
…tration Bare `ug skills` now registers the schema-less skills MCP connection for the configured agents (replacing `ug configure skills` with no arguments), keeping any existing --mcp scope, and then prints the group help. On the first run it adds a short note on how to create a skill. Skill downloads no longer print the connection summary, so a successful registration can't bury the per-skill download failures the download step already reported; `--mcp` is unchanged. Co-authored-by: Isaac <no-reply@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
Co-authored-by: Isaac <no-reply@databricks.com>
A repeat `ug skills` still re-registers the connection but stays quiet, so the summary shows once at setup rather than on every invocation. Co-authored-by: Isaac <no-reply@databricks.com>
Add a `quiet` flag to setup_mcp_clients that drops the section header and the "Configuring for" line (keeping missing-client warnings), and pass it on repeat runs so a re-registration prints nothing at all. Co-authored-by: Isaac <no-reply@databricks.com>
Show the group help first, then register and print the connection summary and the create-skill note, so the actionable setup output lands at the bottom. Co-authored-by: Isaac <no-reply@databricks.com>
The download paths pass `quiet=True` to setup_mcp_clients so a download prints only its own progress and results (and any failures), never the skills MCP setup header or connection summary. Co-authored-by: Isaac <no-reply@databricks.com>
The lambda wrapping added in the prior commit tripped `ruff format --check` (enforced by tests/test_lint.py); reformat to satisfy it. No behavior change. Co-authored-by: Isaac <no-reply@databricks.com>
xsh310
force-pushed
the
skills-bare-configure-mcp
branch
from
September 18, 2026 15:31
fdd55b2 to
7399345
Compare
| agent skill directories. | ||
|
|
||
| ```bash | ||
| # Register the skills MCP connection (utility tools only) for your agents. |
Collaborator
There was a problem hiding this comment.
Do we need to mention this in the docs? Basically its quite everywhere, why mention it in the docs then?
Collaborator
There was a problem hiding this comment.
Also can ug skills display the help commands for skills?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed and why
ug configure skillsis being deprecated, so this adds a simple place to register the bare skills MCP connection: a bareug skills. It also tunes what gets printed when the skills MCP is configured, so a successful registration never buries download failures.ug skillsregisters the schema-less skills MCP connection (utility tools only) for the configured agents, keeps any client's existing--mcpscope, and then prints the group help it always showed. On the first run it also prints a short note on how to create a skill. When a connection already exists it re-registers (still printing the connection summary) but omits that create-skill note.ug skills addwithout--mcpor--skill, and the interactive picker) no longer print the "Skills MCP registered" summary, which had been burying the per-skill download failures the download step already reports.register_schemaless_skills_connectiongained aprint_summaryflag (default on) that these paths pass asFalse.--mcpstays exactly as it was.How do you know it works?
ug skillsfirst run (configures, shows help, prints the create-skill note); an already-configured run (re-registers, omits the note); a subcommand skipping the entrypoint's configuration; the three download paths suppressing the summary; andconfigure_bare_skills_mcp_commandreporting first run versus existing.uv run ruff checkis clean anduv run pytestpasses. The one failing test is a pre-existing live-gateway e2e case unrelated to this change.Note on the stack
This sits on top of #723 (its base is that PR's
skills-list-commandbranch, so the diff is just this one commit). GitHub does not allow a fork PR to join a gh stack, so the two carry no stack badge; once #723 merges, retarget this tomain.This pull request and its description were written by Isaac.