Skip to content

Fix install.sh skill dir bug + add MCP tool references to jobs and UC skills#284

Open
CheeYuTan wants to merge 1 commit intodatabricks-solutions:mainfrom
CheeYuTan:fix/small-fixes-bundle
Open

Fix install.sh skill dir bug + add MCP tool references to jobs and UC skills#284
CheeYuTan wants to merge 1 commit intodatabricks-solutions:mainfrom
CheeYuTan:fix/small-fixes-bundle

Conversation

@CheeYuTan
Copy link
Contributor

Summary

Consolidates PRs #272, #261, and #259 into a single right-sized PR.

File Change Source PR
install.sh Fix Claude skill dir using = instead of += — overwrites dirs from other tools #272
databricks-jobs/SKILL.md Add MCP Tool Integration section with manage_jobs and manage_job_runs examples #261
databricks-unity-catalog/SKILL.md Expand MCP Tool Integration with 8 governance tools (manage_uc_objects, grants, tags, storage, connections, security_policies, monitors, sharing) #259
auth.py, test_sql.py Fix pre-existing ruff format issues to pass CI

Test plan

  • install.sh bug: when TOOLS="claude cursor", the claude case now appends (+=) instead of overwriting (=) the dirs array, so cursor's directory is preserved
  • Jobs MCP tool examples match the actual manage_jobs and manage_job_runs tool signatures
  • UC MCP tool examples match the actual tool signatures for all 8 governance tools
  • ruff format --check passes locally
  • ruff check passes locally

…bs and UC skills

Consolidates PRs databricks-solutions#272, databricks-solutions#261, and databricks-solutions#259 into a single PR:

- install.sh: Fix Claude skill directory using = instead of +=, which
  overwrites directories from previously processed tools
- databricks-jobs: Add MCP Tool Integration section with manage_jobs
  and manage_job_runs usage examples
- databricks-unity-catalog: Expand MCP Tool Integration with 8 governance
  tool references (manage_uc_objects, grants, tags, storage, connections,
  security_policies, monitors, sharing)
- Fix pre-existing ruff format issues in auth.py and test_sql.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant