Add M1 macro property parsing

Author: sabino

PLAN.md

@@ -514,7 +514,7 @@ Exit criteria:
 
 - M0 is under PR review as the Zig `0.16.0` runtime scaffold.
 - M1 has started on stacked branches with native Zig artifact-first parser slices.
-- `dxt parse` now targets the supported Tier 0 subset: project name/model paths/seed paths/macro paths, SQL model discovery, CSV seed discovery, project macro discovery, docs block discovery, literal `ref` to models or seeds, literal `source`, literal `doc` in descriptions, inline `config(materialized=..., tags=...)`, known project macro call dependencies, narrow YAML model properties for scalar descriptions, simple columns, tags, materialization, disabled SQL models, dbt-shaped `unique`, `not_null`, `accepted_values`, and `relationships` generic test nodes, dependency maps, and deterministic partial `manifest.json`. YAML generic test arguments are currently supported for scalar values plus inline and block lists required by public Jaffle Shop DuckDB-style tests.
+- `dxt parse` now targets the supported Tier 0 subset: project name/model paths/seed paths/macro paths, SQL model discovery, CSV seed discovery, project macro discovery, macro property YAML for descriptions and arguments, docs block discovery, literal `ref` to models or seeds, literal `source`, literal `doc` in descriptions, inline `config(materialized=..., tags=...)`, known project macro call dependencies, narrow YAML model properties for scalar descriptions, simple columns, tags, materialization, disabled SQL models, dbt-shaped `unique`, `not_null`, `accepted_values`, and `relationships` generic test nodes, dependency maps, and deterministic partial `manifest.json`. YAML generic test arguments are currently supported for scalar values plus inline and block lists required by public Jaffle Shop DuckDB-style tests.
 - `dxt ls` now lists dbt-selectable resources from the same parser graph with stable text/JSON output and basic name, tag, path, resource type, and exact exclude filters; macros are emitted in artifacts but not exposed as `ls` resources.
-- Synthetic fixtures cover one model, model refs, seed refs, source refs, combined source/model YAML, inline config/tag selection, YAML model properties and columns, emitted `unique`, `not_null`, `accepted_values`, and `relationships` generic test nodes, project macro artifacts, configured `macro-paths` replacing the default macro directory, macro calls recorded in model and macro `depends_on.macros`, docs blocks with literal `doc` descriptions, disabled models, disabled ref diagnostics, unmatched model-property warnings, duplicate model and docs diagnostics, unsupported dynamic ref/doc diagnostics, missing doc diagnostics, malformed docs block diagnostics, and unsupported unknown macro-call diagnostics.
-- The current M1 manual gate parses the public Jaffle Shop DuckDB project into a partial manifest with SQL models, CSV seeds, docs blocks, project macros, and supported generic test nodes, including the Jaffle `accepted_values` and `relationships` tests. The next M1 slices should add published schema validation, stronger selector parity, package macro namespaces, macro property YAML, and deeper Jaffle artifact parity.
+- Synthetic fixtures cover one model, model refs, seed refs, source refs, combined source/model YAML, inline config/tag selection, YAML model properties and columns, emitted `unique`, `not_null`, `accepted_values`, and `relationships` generic test nodes, project macro artifacts and macro properties, configured `macro-paths` replacing the default macro directory, macro calls recorded in model and macro `depends_on.macros`, docs blocks with literal `doc` descriptions, disabled models, disabled ref diagnostics, unmatched model-property warnings, duplicate model and docs diagnostics, unsupported dynamic ref/doc diagnostics, missing doc diagnostics, malformed docs block diagnostics, and unsupported unknown macro-call diagnostics.
+- The current M1 manual gate parses the public Jaffle Shop DuckDB project into a partial manifest with SQL models, CSV seeds, docs blocks, project macros, and supported generic test nodes, including the Jaffle `accepted_values` and `relationships` tests. The next M1 slices should add published schema validation, stronger selector parity, package macro namespaces, and deeper Jaffle artifact parity.

src/project.zig

@@ -109,6 +109,7 @@ fn commandError(err: anyerror, stderr: *Io.Writer) ExitCode {
         error.DuplicateSeedName => stderr.writeAll("error: duplicate seed name in supported M1 parser subset\n") catch {},
         error.DuplicateDocName => stderr.writeAll("error: duplicate docs block name in supported M1 parser subset\n") catch {},
         error.DuplicateMacroName => stderr.writeAll("error: duplicate macro name in supported M1 parser subset\n") catch {},
+        error.DuplicateMacroProperty => stderr.writeAll("error: duplicate macro property patch in supported M1 parser subset\n") catch {},
         error.UnsupportedDynamicRef => stderr.writeAll("error: unsupported dynamic ref; M1 parser only supports literal ref calls\n") catch {},
         error.UnsupportedDynamicSource => stderr.writeAll("error: unsupported dynamic source; M1 parser only supports literal source calls\n") catch {},
         error.UnsupportedDynamicDoc => stderr.writeAll("error: unsupported dynamic doc; M1 parser only supports literal doc calls in descriptions\n") catch {},

src/root.zig

@@ -0,0 +1,5 @@
+name: duplicate_macro_property
+version: "1.0"
+profile: default
+model-paths: ["models"]
+target-path: target

tests/fixtures/duplicate_macro_property/dbt_project.yml

@@ -0,0 +1,3 @@
+{% macro format_id(column_name) %}
+    cast({{ column_name }} as varchar)
+{% endmacro %}

tests/fixtures/duplicate_macro_property/macros/format_id.sql

@@ -0,0 +1,7 @@
+version: 2
+
+macros:
+  - name: format_id
+    description: First patch.
+  - name: format_id
+    description: Second patch.

tests/fixtures/duplicate_macro_property/macros/schema.yml

@@ -0,0 +1 @@
+select {{ format_id("customer_id") }} as customer_id

tests/fixtures/duplicate_macro_property/models/customers.sql

@@ -0,0 +1,5 @@
+name: macro_properties
+version: "1.0"
+profile: default
+model-paths: ["models"]
+target-path: target

tests/fixtures/macro_properties/dbt_project.yml

@@ -0,0 +1,3 @@
+{% macro format_id(column_name) %}
+    cast({{ column_name }} as varchar)
+{% endmacro %}

tests/fixtures/macro_properties/macros/format_id.sql

@@ -0,0 +1,11 @@
+version: 2
+
+macros:
+  - name: format_id
+    description: Format an identifier expression.
+    arguments:
+      - name: column_name
+        type: string
+        description: Column expression to cast.
+      - name: optional_suffix
+        description: Optional suffix value.