Skip to content

Conversation

@malcolndandaro
Copy link
Collaborator

Add reference file for UC skill related to stuff

  • access control/grants
  • delta sharing
  • storage credentials, external locations, connections
  • objects (tables, volumes, catalogs & schemas)
  • tags
  • profiling and anomaly detection

Copy link
Collaborator

@dustinvannoy-db dustinvannoy-db left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like a lot of lines for a skill. Can we try to have it only use SQL when available, only use CLI when available, and point to docs links instead of having every example in the markdowns? Or maybe have it separate out as script files that can be referenced when needed?

If the MCP server has functions to simplify this then it should be able to use those instead and we can remove even more examples here.

@malcolndandaro malcolndandaro changed the title add UC reference files for: objects, grants, credentials, tags, deta sharing add UC reference files for: objects, grants, credentials, tags, delta sharing Jan 29, 2026
@malcolndandaro
Copy link
Collaborator Author

malcolndandaro commented Jan 29, 2026

This seems like a lot of lines for a skill. Can we try to have it only use SQL when available, only use CLI when available, and point to docs links instead of having every example in the markdowns? Or maybe have it separate out as script files that can be referenced when needed?

If the MCP server has functions to simplify this then it should be able to use those instead and we can remove even more examples here.

@dustinvannoy-db
Yeah i think we can make it simpler. Should the priority be (when available): MCP -> SQL -> SDK -> CLI?

@calreynolds
Copy link
Collaborator

@malcolndandaro yup I think that flow makes sense (subject to testing).

Generally highest level -> lowest level in order of priority is perfect 👍

@dustinvannoy-db
Copy link
Collaborator

dustinvannoy-db commented Jan 29, 2026 via email

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.

3 participants