@@ -401,19 +401,12 @@ def download_managed_skills_on_launch(
401401) -> list [str ]:
402402 """Download admin-published skills to disk so the agent's ``/skills`` lists them.
403403
404- Runs on the managed launch path (see ``cli._apply_managed_skills``), where the
405- managed config only *registers* the skills MCP connection -- nothing writes the
406- skill bundles to ``.claude/skills`` / ``.agents/skills``, so the agent's on-disk
407- ``/skills`` picker stays empty. For each ``<catalog>.<schema>`` location this
408- lists the schema's finalized skills and writes only those whose bundle is not
409- already on disk, so a launch where everything is already downloaded costs one
410- cheap listing call per schema and writes nothing.
411-
412- Existing skills are left untouched with no overwrite prompt: the launch path
413- must never block on input, and a developer's own same-named skill must never be
414- clobbered silently. Everything is best-effort -- a failure listing a schema or
415- fetching a bundle warns and is skipped, and the function never raises -- so it
416- can never block the launch. Returns the bundle names newly written to disk.
404+ Runs on the managed launch path: the config only registers the skills MCP
405+ connection, so nothing else writes the bundles that ``/skills`` reads. Writes
406+ only skills not already on disk -- no overwrite prompt, so the launch never
407+ blocks on input and a developer's own same-named skill is never clobbered.
408+ Best-effort and never raises, so it can't block the launch. Returns the bundle
409+ names newly written.
417410 """
418411 roots = skill_dir_roots (path )
419412 written : list [str ] = []
0 commit comments