dbt-labs / dbt-labs/dbt-agent-skills

Check that skills are relevant for all adapters

Open
#5 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
710
Forks
62
Avg merge
1d 18h
Merged PRs (30d)
5

Description

### Description

Seeing this comment https://github.com/dbt-labs/llm-skills/blob/b57f4dd49759b5352aa45a94b642dc3600269555/dbt-commands/writing-data-tests/SKILL.md?plain=1#L141

I think that we need to be cautious about some recommendations that could apply to some adapters but not all.

In this case, for BigQuery a `max` would still scan the entire table (and potentially cost thousands of $)

- we should review that skills apply to all adapters ; and in case they don't add this information to the skills
- we should maybe add skills dedicated to some adapters to mention the best practices -- e.g.
- on BQ, we likely don't want to call `dbt show` with a `where` if there is no partition in the model
- on Snowflake, the metadata queries are "free" and we can have quite a few

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.