Repository navigation
Conversation
Signed-off-by: ditadi <victordperd@gmail.com>
… 001 Signed-off-by: ditadi <victordperd@gmail.com>
Contributor
🤖 AppKit PR bot🔬 Run evalsStart an eval for this PR from the evals-monitor app: Go to Evals Monitor → 📦 Try this PR's app templateScaffolds a new app from this PR's SDK build. Run it in any folder (requires the GitHub CLI — gh run download 41109402593 -R databricks/appkit -n appkit-template-0.89.0-pr.635ba31-stack-database-dev-flow-00-design-677 -D appkit-pr-677 \
&& unzip -o "appkit-pr-677/appkit-template-0.89.0-pr.635ba31-stack-database-dev-flow-00-design-677.zip" -d "appkit-pr-677" \
&& databricks apps init --template "appkit-pr-677"The template pins |
This was referenced Oct 19, 2026
Signed-off-by: ditadi <victordperd@gmail.com>
This was referenced Oct 20, 2026
ditadi
marked this pull request as ready for review
October 26, 2026 02:04
The design doc promised behavior the corrected stack does not ship. It now describes what the code does: - The tooling export is @databricks/appkit/beta/database-tooling, beta and internal to the AppKit CLI. - migrate, introspect, clean and init hold config/database/.appkit-db.lock. init holds one lock for its whole run, and init --schema none, seed, verify and clean --delete-branch take none. The doc says how to recover a stale lock file. - init --schema none hands off nothing, and schema public is never handed off. - Values in variable-overrides.json count for the deploy boundary, and an unresolved branch or database reference refuses every write. - Introspect keeps .private() on declared columns, writes new columns private, and refuses a schema.ts that does not load or a foreign key it does not declare. - verify --deployed compares the committed migrations with the source snapshot of the active deployment, and fails when it cannot read it. - Migrations log one line per run. There is no span and no metric. - #675 keeps the plugin and appkit db hidden. A reveal is a later, separate change. The appkit db check section stays as it is until #678 settles. Signed-off-by: ditadi <victordperd@gmail.com>
…he stack code Signed-off-by: ditadi <victordperd@gmail.com>
… and prompts Apply the review of the design doc. The pooler refusal belongs to migrate on start, the migrator reconciles without the lock before it takes it, the deploy guard reads each target with and without variable-overrides.json, and init asks only for --schema clean and an ambiguous rename. The reveal also moves the database export into the generated barrel, check refuses a file that is not a drizzle-kit snapshot, and the project is not a databricks.yml variable. Signed-off-by: ditadi <victordperd@gmail.com>
This branch has not been deployed
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
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.
🥞 Stacked PR (generated by Stack)
Use this link to review incremental changes.
What did you change, and why?
The DatabasePlugin reads tables but never creates them. Getting from
databricks apps initto a deployed table today takes hand-run SQL, grants and console steps on the branch the app deploys to.This PR adds design 001, Database development flow, as
designs/001_DatabaseDevelopmentFlow.md. Comments and edits happen in the Google Doc, and changes there are synced to this file. The rest of this stack implements the design: #654, #655, #666, #670, #672, #674, #675 and #678.This revision matches the doc to the corrected stack. The doc now says that:
@databricks/appkit/beta/database-tooling, which is beta and internal to the AppKit CLImigrate,introspect,cleanandinitholdconfig/database/.appkit-db.lock, andseed,verify,check,clean --delete-branchandinit --schema nonetake no lockinit --schema nonehands off nothing, and schemapublicis never handed offvariable-overrides.json, and an unresolved branch or database reference refuses every write.private()on declared columns, writes new columns private, and refuses aschema.tsthat does not load or a foreign key it does not declareverify --deployedcompares the committed migrations with the source snapshot of the active deployment, and fails when it cannot read that snapshotappkit db checkfails on a snapshot file that is not a drizzle-kit snapshothidden: trueandappkit dbhidden fromappkit --help, and offering the plugin inapps initis a later, separate changeHow do you know it works?
Docs only. A review checked each changed statement against the code of #654 to #678.