Skip to content

[DCV-3760][DCV-3765] Profiles.yml refresh + partial parse fallback#19

Merged
sconley-datacoves merged 7 commits into
mainfrom
DCV-3760
Mar 26, 2026
Merged

[DCV-3760][DCV-3765] Profiles.yml refresh + partial parse fallback#19
sconley-datacoves merged 7 commits into
mainfrom
DCV-3760

Conversation

@jesusmgg
Copy link
Copy Markdown

@jesusmgg jesusmgg commented Mar 18, 2026

Summary

  • [DCV-3760] Track all potential profiles.yml paths (env var, project dir, ~/.dbt) in the watcher instead of only the resolved one at startup
  • [DCV-3760] Re-resolve profiles_dir dynamically on reparse so the server switches when a higher-priority profiles.yml appears or the active one is deleted
  • [DCV-3760] Re-resolve standard profiles_dir paths from state file on startup instead of using stale saved values. Custom (non-standard) paths are preserved.
  • [DCV-3765] When partial parsing fails after new source files are created (e.g. dbt-coves generate sources), create a fresh ManifestLoader and do a full reparse. Fixes linting/formatting not working on newly generated files until page reload.

Test plan

  • Modify ~/.dbt/profiles.yml, server reparses within 2s
  • Copy profiles.yml to project dir, server switches to it
  • Delete project dir profiles.yml, server falls back to ~/.dbt
  • Custom profiles_dir (non-standard path) preserved across reparse
  • Server recovers after fixing a broken profiles.yml
  • Run dbt-coves generate sources, linting works on new files without page reload
  • Existing file linting continues working after new files are generated
  • No watcher crash loop after generating sources

After merge

git tag v1.1.7 && git push origin v1.1.7

@jesusmgg jesusmgg self-assigned this Mar 18, 2026
@jesusmgg jesusmgg marked this pull request as draft March 20, 2026 03:51
@jesusmgg jesusmgg changed the title [DCV-3760] Refresh profiles.yml resolution on config reparse [DCV-3760][DCV-3765] Profiles.yml refresh + partial parse fallback Mar 25, 2026
@jesusmgg jesusmgg marked this pull request as ready for review March 25, 2026 12:45
@sconley-datacoves sconley-datacoves merged commit df0a0e4 into main Mar 26, 2026
3 checks passed
@sconley-datacoves sconley-datacoves deleted the DCV-3760 branch March 26, 2026 21:02
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.

2 participants