Add Databricks notebook for installing Genie Code skills#433
Add Databricks notebook for installing Genie Code skills#433Praneeth16 wants to merge 3 commits intodatabricks-solutions:mainfrom
Conversation
…pace Non-technical Genie Code users can now import this notebook into their workspace and run it to install skills, without needing a local terminal or the Databricks CLI. The notebook uses the Databricks Python SDK (pre-installed in all runtimes) to download skills from GitHub and upload them to /Users/<username>/.assistant/skills/. - New: databricks-skills/install_genie_code_skills.py (Databricks notebook) - Updated: README.md with notebook install option in Genie Code section Co-authored-by: Isaac
The APX repo stores files at skills/apx/SKILL.md (flat), not in a skill-name subdirectory like the other repos. Added source_path parameter to install_skill() to handle this, and moved APX extra files to its own mapping matching get_apx_skill_extra_files() in install_skills.sh. Co-authored-by: Isaac
|
I like the approach for V1. I think we need to make an update for v2 to dynamically generate the install skills list instead of hardcoded in the notebook as this introduces 2 places (NB and install script) where we have to maintain the skill install lists. One change first though. The verify cell is not working. Please see the below change I had to make. The current cell gives "Could not list skills: 'ObjectInfo' object has no attribute 'is_directory'" when executed. from databricks.sdk import WorkspaceClient w = WorkspaceClient() try: |
Replaces the hardcoded skill registries (DATABRICKS_SKILLS, MLFLOW_SKILLS, APX_SKILLS and their extra-file mappings) with dynamic discovery using the GitHub Git Trees API. This eliminates the need to maintain duplicate skill lists across install_skills.sh and this notebook. Also fixes the verify cell which used the non-existent `is_directory` attribute on ObjectInfo — now uses `object_type` comparison instead. Co-authored-by: Isaac
|
@scottDBX1886 thanks for the suggestions! I've implemented them and made the skills discovery dynamic based on GitHub. |
|
New notebook looks really good. Dynamic skills retrieval is perfect. Cell 7 looks good. |
scottDBX1886
left a comment
There was a problem hiding this comment.
All looks good. great PR
Summary
databricks-skills/install_genie_code_skills.py) that installs Genie Code skills from within a workspace, no local terminal needed/Users/<username>/.assistant/skills/install_skills.shexactly (validated programmatically)Test plan
ast.parse)urllib,json,posixpath,databricks.sdk)install_skills.sh: 26 Databricks + 8 MLflow + 1 APXinstall_skills.shfor all 20 Databricks + 5 MLflow + 1 APX entriesfe-vm-cme-condeworkspace: 35 skills installed successfully in 59 seconds/Users/<username>/.assistant/skills/contains all 35 skill directories viadatabricks workspace listThis pull request was AI-assisted by Isaac.