use meta.dune.public for table visibility config

Author: a-monteiro

README.md

@@ -152,6 +152,34 @@ select * from dune.dune__tmp_.dbt_template_view_model
 
 All templates are in `models/templates/`.
 
+## Table Visibility
+
+By default, all tables are **private** — only your team can query them. To make a table publicly accessible (visible and queryable by anyone on Dune), set `meta.dune.public: true` in the model config:
+
+```sql
+{{ config(
+    alias = 'my_model'
+    , materialized = 'table'
+    , meta = {"dune": {"public": true}}
+    , properties = {
+        "partitioned_by": "ARRAY['block_date']"
+    }
+) }}
+```
+
+To make all models in a folder public, set it in `dbt_project.yml`:
+
+```yaml
+models:
+  my_project:
+    public_models:
+      +meta:
+        dune:
+          public: true
+```
+
+Visibility is only applied in the `prod` target and has no effect in development. Views are not supported.
+
 ## GitHub Actions
 
 ### CI Workflow (Pull Requests)

macros/dune_dbt_overrides/properties.sql

@@ -1,20 +1,4 @@
-{#
-  Override of the dbt-trino adapter's properties() macro to inject dune.public
-  into extra_properties at CREATE TABLE time.
-
-  Configure per model via config():
-    , dune_public = true   -- make table publicly visible on Dune
-    , dune_public = false  -- (default) keep table private
-
-  Or set for an entire folder in dbt_project.yml:
-    models:
-      my_project:
-        public_models:
-          +dune_public: true
-
-  Only runs in prod. On incremental (non-full-refresh) runs, set_table_visibility
-  handles it via ALTER TABLE instead.
-#}
+{# Override of the dbt-trino adapter's properties() macro to inject dune.public into extra_properties at CREATE TABLE time. #}
 {% macro properties(temporary=False) %}
   {%- set _properties = config.get('properties') -%}
   {%- set table_format = config.get('table_format') -%}
@@ -58,8 +42,7 @@
     {%- endif -%}
   {%- endif -%}
 
-  {#-- Inject dune.public into extra_properties at CREATE time (prod only) --#}
-  {%- set dune_public = config.get('dune_public') -%}
+  {%- set dune_public = config.get('meta', {}).get('dune', {}).get('public') -%}
   {%- if dune_public is not none and target.name == 'prod' -%}
     {%- set visibility_value = 'true' if dune_public else 'false' -%}
     {%- set extra_props_sql = "map_from_entries(ARRAY[ROW('dune.public', '" ~ visibility_value ~ "')])" -%}

macros/dune_dbt_overrides/set_table_visibility.sql

@@ -7,30 +7,12 @@
   ])
 {%- endmacro -%}
 
-{#
-  set_table_visibility: post-hook that sets dune.public on incremental (non-full-refresh) runs.
-
-  For table materializations and incremental full-refreshes, extra_properties is set
-  at CREATE TABLE time via the overridden properties() macro. This hook handles the
-  incremental case where no CREATE is issued (INSERT/MERGE only).
-
-  Configure per model via config():
-    , dune_public = true   -- make table publicly accessible on Dune (visible and queryable by anyone)
-    , dune_public = false  -- (default) keep table private
-
-  Or set for an entire folder in dbt_project.yml:
-    models:
-      my_project:
-        public_models:
-          +dune_public: true
-
-  Only runs in prod. Views are not supported.
-#}
+{# post-hook that keeps dune.public in sync on incremental (non-full-refresh) runs where no CREATE TABLE is issued #}
 {% macro set_table_visibility(this, materialization) %}
 {%- if target.name == 'prod'
     and materialization == 'incremental'
     and not flags.FULL_REFRESH -%}
-  {%- set dune_public = config.get('dune_public', false) -%}
+  {%- set dune_public = config.get('meta', {}).get('dune', {}).get('public', false) -%}
   {%- set properties = {'dune.public': 'true' if dune_public else 'false'} -%}
   ALTER TABLE {{ this }}
     SET PROPERTIES extra_properties = {{ dune_properties(properties) }}

models/templates/dbt_template_append_incremental_model.sql

@@ -9,7 +9,7 @@
     alias = 'dbt_template_append_incremental_model'
     , materialized = 'incremental'
     , incremental_strategy = 'append'
-    , dune_public = false
+    , meta = {"dune": {"public": false}}
     , properties = {
         "partitioned_by": "ARRAY['block_date']"
     }

models/templates/dbt_template_delete_insert_incremental_model.sql

@@ -4,7 +4,7 @@
     , incremental_strategy = 'delete+insert'
     , unique_key = ['block_number', 'block_date']
     , incremental_predicates = ["block_date >= now() - interval '1' day"]
-    , dune_public = false
+    , meta = {"dune": {"public": false}}
     , properties = {
         "partitioned_by": "ARRAY['block_date']"
     }

models/templates/dbt_template_merge_incremental_model.sql

@@ -4,7 +4,7 @@
     , incremental_strategy = 'merge'
     , unique_key = ['block_number', 'block_date']
     , incremental_predicates = ["DBT_INTERNAL_DEST.block_date >= now() - interval '1' day"]
-    , dune_public = false
+    , meta = {"dune": {"public": false}}
     , properties = {
         "partitioned_by": "ARRAY['block_date']"
     }

models/templates/dbt_template_table_model.sql

@@ -1,7 +1,7 @@
 {{ config(
     alias = 'dbt_template_table_model'
     , materialized = 'table'
-    , dune_public = false
+    , meta = {"dune": {"public": false}}
     , properties = {
         "partitioned_by": "ARRAY['block_date']"
     }