feat(cli): add deploy-config generator for plugin resources (no callers yet) - #519
Draft
MarioCadenas wants to merge 1 commit into
Draft
feat(cli): add deploy-config generator for plugin resources (no callers yet)#519MarioCadenas wants to merge 1 commit into
MarioCadenas wants to merge 1 commit into
Conversation
…rs yet) Standalone module reproducing what 'databricks apps init' renders, verified byte-for-byte against golden fixtures captured from a real scaffold: - config-plan: manifest resources -> app.yaml env (valueFrom=resourceKey), databricks.yml bundle variables + resource bindings. Per-type binding specs for the verified types (sql_warehouse, postgres). Platform-origin fields are excluded from app.yaml env (platform-injected at deploy). - config-writer: comment-preserving additive YAML patch via the 'yaml' package (never clobbers existing entries; idempotent), plus a 'databricks bundle validate' post-write gate. - Unverified resource types emit env entries but skip the databricks.yml binding with a warning rather than guessing. - __fixtures__/: analytics + lakebase configs captured from 'databricks apps init' as golden ground truth. - Adds 'yaml' dependency to packages/shared. 11 tests incl. byte-exact fixture comparison and idempotency. Wired into 'appkit add' in the follow-up commit. Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Stack 4/7 — base: `feat/registry-env-reconcile`
What
Standalone module reproducing what `databricks apps init` renders for resource bindings, verified byte-for-byte against golden fixtures captured from a real scaffold on dogfood.
Note for isolated review
`validateBundle`/`reportConfigWrite` are first used in stack 5/7 — no callers yet here (intentional stack seam).
Tests
11 tests incl. byte-exact fixture comparison and idempotency.
Part of the resource-aware `appkit add` stack (4/7).