Skip to content

Handle unsupported DuckDB metadata freshness - #215

Merged
sabino merged 1 commit into
mainfrom
agent/issue-210-compat-duckdb-unsupported-metadata-source-freshn
Jun 21, 2026
Merged

Handle unsupported DuckDB metadata freshness#215
sabino merged 1 commit into
mainfrom
agent/issue-210-compat-duckdb-unsupported-metadata-source-freshn

Conversation

@sabino

@sabino sabino commented Jun 21, 2026

Copy link
Copy Markdown
Owner

Summary

  • Return a dbt-grounded unsupported metadata freshness reason for DuckDB sources with freshness but no loaded_at_field or loaded_at_query.
  • Keep Sources v3 runtime-error rows for those sources while continuing selected loaded-at-field and loaded-at-query freshness checks.
  • Update focused CLI coverage plus compatibility, plan, and changelog notes.

Linked Issue

Closes #210

Scope

  • In scope: DuckDB metadata-only source freshness unsupported runtime-error behavior and focused Sources v3 schema coverage.
  • Out of scope: DuckDB metadata freshness execution, adapter capability matrices, get_relation_last_modified, non-DuckDB behavior, source-status selector changes.

Source Grounding

dxt Ownership

  • Zig modules: src/project/source_freshness.zig, src/project.zig.
  • Artifacts: sources.json Sources v3 runtime-error rows.
  • Commands: dxt source freshness.

Validation

  • zig build using isolated ignored cache paths because the default local Zig cache hit a mount rename PermissionDenied.
  • zig build test full suite: attempted, interrupted after extended silence inside the Zig test runner on the busy worker host.
  • Focused native Zig test: zig test src/project/source_freshness.zig --cache-dir .agent/runs/issue-210-zig-test-cache/local --global-cache-dir .agent/runs/issue-210-zig-test-cache/global.
  • Focused pytest: python -m pytest tests/test_cli.py::test_source_freshness_reports_unsupported_duckdb_metadata_and_continues -q.
  • Public Jaffle gate: not needed for this unsupported-DuckDB diagnostic slice.
  • python scripts/check_runtime_boundary.py.
  • python scripts/check_public_safety.py.
  • git diff --check and zig fmt --check src/project/source_freshness.zig src/project.zig.

Agent Evidence

Risks

  • Runtime boundary: product behavior stays in Zig; Python change is focused black-box CLI/schema test coverage.
  • Public safety: scan passed.
  • Artifact parity: runtime-error row remains Sources v3-shaped with only unique_id, error, and status.
  • Branch overlap: rebased on current origin/main before final validation and push.
  • Follow-up: real metadata freshness execution still needs an explicit Zig adapter capability/API design and a safe upstream-backed adapter implementation.

@sabino
sabino marked this pull request as ready for review June 21, 2026 18:57
@sabino
sabino merged commit 557f109 into main Jun 21, 2026
6 checks passed
@sabino
sabino deleted the agent/issue-210-compat-duckdb-unsupported-metadata-source-freshn branch June 21, 2026 18:57
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.

[compat] DuckDB unsupported metadata source freshness parity

1 participant