Skip to content

fix(check-manifest): track root freshness inputs - #178

Merged
eitsupi merged 1 commit into
mainfrom
fix/manifest-freshness-project-inputs
Aug 29, 2026
Merged

fix(check-manifest): track root freshness inputs#178
eitsupi merged 1 commit into
mainfrom
fix/manifest-freshness-project-inputs

Conversation

@eitsupi

@eitsupi eitsupi commented Aug 29, 2026

Copy link
Copy Markdown
Owner

Summary

  • include root dbt_project.yml in manifest freshness checks
  • include vars.yml when it exists
  • share one freshness-input collector between summary and check-manifest
  • sort and deduplicate inputs when resource paths overlap the project root

Behavior

A root project input is reported stale only when its mtime is newer than manifest.json. The optional vars.yml file is ignored when absent and is not treated as a deleted manifest resource.

check-manifest remains a direct project/manifest validator and does not depend on the analyzed manifest cache.

Verification

  • cargo test --workspace --all-targets --all-features
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cargo fmt --all
  • portable integration coverage for newer, older, missing, and duplicate root inputs

@eitsupi eitsupi changed the title fix(manifest): track root freshness inputs fix: track root freshness inputs Aug 29, 2026
@eitsupi
eitsupi marked this pull request as ready for review August 29, 2026 12:28
@eitsupi eitsupi changed the title fix: track root freshness inputs fix(check-manifest): track root freshness inputs Aug 29, 2026
@eitsupi
eitsupi merged commit 9e6e368 into main Aug 29, 2026
12 checks passed
@eitsupi
eitsupi deleted the fix/manifest-freshness-project-inputs branch August 29, 2026 12:39
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.

1 participant