fix: config.meta is not a dict attribute, use config.get('meta', {}) - #31
Merged
Matts52 merged 1 commit intoSep 22, 2026
Merged
Conversation
config.meta.get(x) crashes at runtime with:
'dbt.context.providers.RuntimeConfigObject object' has no attribute 'meta'
The live config context object exposed inside a materialization/relation
macro doesn't support .meta as a plain attribute -- only .get('key',
default) works. node.config (a rendered dict from the manifest, used
in cortex_skill/create.sql:136) is a different object and .meta access
there isn't affected, which is presumably why this shipped without being
caught locally.
Every model using the cortex_agent/cortex_skill/cortex_mcp_server
materializations fails to compile with this version. Confirmed against
dbt-sos's Slim CI: 13/17 cortex_agent + cortex_skill models errored,
100% reproducible, same stack trace in every case.
Fixes all 24 call sites across cortex_agent.sql, cortex_agent/create.sql,
cortex_skill/create.sql, and cortex_mcp_server/create.sql to use
config.get('meta', {}).get(x) instead -- the same pattern
dbt-cortex-search-service already uses correctly.
Matts52
approved these changes
Sep 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bug
Every
config.meta.get(x)call in v1.0.6 crashes at runtime:The live
configcontext object inside a materialization/relation macro doesn't expose.metaas a plain attribute — only.get('key', default)works.node.config(a rendered dict from the manifest, used correctly incortex_skill/create.sql:136) is a different object, which is presumably why that one line wasn't affected and this shipped without being caught locally.Confirmed against dbt-sos's Slim CI: every model using the
cortex_agent,cortex_skill, orcortex_mcp_servermaterializations fails to compile — 13/17 cortex_agent/cortex_skill models errored in a real build, 100% reproducible, same stack trace every time.Fix
All 24 call sites across
cortex_agent.sql,cortex_agent/create.sql,cortex_skill/create.sql, andcortex_mcp_server/create.sqlchanged fromconfig.meta.get(x)toconfig.get('meta', {}).get(x)— the same patterndbt-cortex-search-servicealready uses correctly (config.get('meta', default={})).No behavior change beyond the fix itself — same fallback-to-legacy-top-level-key logic on every call site, just using a
configaccessor that actually exists.