Repository navigation
Conversation
Add the operation request pipeline: verify the caller with the workspace, refuse cross-site requests, parse the input, run the policy once and execute once, then validate and project the answer. The pipeline is not registered yet, so no request reaches it. Default outputs answer timestamps in RFC 3339, and span redaction matches the routed path. Signed-off-by: ditadi <victordperd@gmail.com>
ditadi
added this pull request to stack #693
October 23, 2026 15:01
Contributor
📦 Bundle size reportCompared against
|
| dist | raw | gzip |
|---|---|---|
| JS (runtime) | 1.3 MB (-4.5 KB) | 457 KB (-468 B) |
| Type declarations | 476 KB (+13 KB) | 172 KB (+3.9 KB) |
| Source maps | 2.5 MB (+7.2 KB) | 859 KB (+3.3 KB) |
| Other | 11 KB | 3.7 KB |
| Total | 4.2 MB (+15 KB) | 1.5 MB (+6.7 KB) |
Per-entry composition (own code — deps external (as shipped))
| Entry | Initial (gz) | Lazy (gz) | Total (gz) | node_modules (min) | Own code (min) |
|---|---|---|---|---|---|
. |
106 KB (+256 B) | 2.5 KB | 109 KB (+256 B) | external | 344 KB (+820 B) |
./beta |
102 KB (-516 B) | 485 B | 103 KB (-516 B) | external | 306 KB (-2.4 KB) |
./testing |
45 KB (+272 B) | 32 KB | 77 KB (+272 B) | external | 224 KB (+820 B) |
./tsdown |
520 B | 0 B | 520 B | external | 813 B |
./type-generator |
26 KB | 0 B | 26 KB | external | 74 KB |
Chunks:
| Entry | Chunk | Load | Size (gz) |
|---|---|---|---|
. |
index.js |
initial | 102 KB |
. |
utils.js |
initial | 4.6 KB |
. |
remote-tunnel-manager.js |
lazy | 2.5 KB |
./beta |
beta.js |
initial | 82 KB |
./beta |
modular.js |
initial | 6.0 KB |
./beta |
stream-manager.js |
initial | 5.9 KB |
./beta |
service-context.js |
initial | 4.2 KB |
./beta |
databricks.js |
initial | 3.3 KB |
./beta |
client.js |
initial | 647 B |
./beta |
index.js |
initial | 20 B |
./beta |
supervisor-api.js |
lazy | 192 B |
./beta |
databricks.js |
lazy | 178 B |
./beta |
index.js |
lazy | 115 B |
./testing |
manifest.js |
initial | 32 KB |
./testing |
index.js |
initial | 10 KB |
./testing |
wide-event-emitter.js |
initial | 2.9 KB |
./testing |
index.js |
lazy | 28 KB |
./testing |
remote-tunnel-manager.js |
lazy | 2.5 KB |
./testing |
utils.js |
lazy | 1.8 KB |
./tsdown |
index.js |
initial | 520 B |
./type-generator |
index.js |
initial | 26 KB |
@databricks/appkit-ui
npm tarball (packed): 395 KB (-2.3 KB) — gzipped download (dist + bin; excludes release-only docs/NOTICE).
| dist | raw | gzip |
|---|---|---|
| JS (runtime) | 427 KB (-1.9 KB) | 146 KB (-761 B) |
| Type declarations | 266 KB (-900 B) | 98 KB (-211 B) |
| Source maps | 849 KB (-4.4 KB) | 284 KB (-1.4 KB) |
| CSS | 16 KB | 3.2 KB |
| Total | 1.5 MB (-7.2 KB) | 531 KB (-2.4 KB) |
Per-entry composition (consumer bundle — deps bundled, peerDeps external)
| Entry | Initial (gz) | Lazy (gz) | Total (gz) | node_modules (min) | Own code (min) |
|---|---|---|---|---|---|
./js |
5.3 KB | 49 KB | 55 KB | 208 KB | 14 KB |
./js/beta |
2.1 KB (+15 B) | 0 B | 2.1 KB (+15 B) | 0 B | 5.0 KB (+22 B) |
./react |
433 KB | 49 KB | 481 KB | 1.3 MB | 177 KB |
./react/beta |
4.7 KB (-184 B) | 0 B | 4.7 KB (-184 B) | 0 B | 12 KB (-539 B) |
Chunks:
| Entry | Chunk | Load | Size (gz) |
|---|---|---|---|
./js |
index.js |
initial | 5.2 KB |
./js |
chunk |
initial | 120 B |
./js |
apache-arrow |
lazy | 49 KB |
./js/beta |
beta.js |
initial | 2.1 KB |
./react |
index.js |
initial | 431 KB |
./react |
tslib |
initial | 2.1 KB |
./react |
apache-arrow |
lazy | 49 KB |
./react/beta |
beta.js |
initial | 4.7 KB |
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 39974587375 -R databricks/appkit -n appkit-template-0.89.0-pr.b0cceb0-stack-database-stable-03-operation-authorization-689 -D appkit-pr-689 \
&& unzip -o "appkit-pr-689/appkit-template-0.89.0-pr.b0cceb0-stack-database-stable-03-operation-authorization-689.zip" -d "appkit-pr-689" \
&& databricks apps init --template "appkit-pr-689"The template pins |
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.
What did you change, and why?
Why: An operation must run only for a verified caller that its policy admits, and nothing it touches may reach telemetry.
Change (unregistered runtime):
resourceandallowonce, runexecuteonce, then validate and project the answer. Only a literaltrueadmits, and a throwing lookup or policy denies.x-forwarded-access-token, checked with the workspace Current UserMeAPI on every request. Local development uses the developer's credentials for same-origin loopback requests. The database connection stays the service principal.executereturned answers 500 withoutcome: "unknown". Default outputs answer timestamps in RFC 3339. Spans drop operation input from URLs, and everything below the operation span runs with tracing suppressed.Interface
The HTTP contract this handler implements. The last PR of this stack mounts it at
/api/database/operations/<name>.inputquery parameter holding JSON, in at most 8 KiB of query string. A write carries one JSON body of at most 1 MiB and no query string.executereceives{ actor, input, db, appkit, resource }, never the request, its headers, or the token.{ "error": "…" }{ "error": "…" }{ "error": "…" }{ "error": "…" }{ "error": "…", "details": [{ "path": ["changes"], "message": "Expected at least one change" }] }{ "error": "…" }authorizereturnsfalseor throws{ "error": "…" }authorizereturns{ deny }"forbidden", 404 for"not-found"{ "error": "…" }{ "error": "…" }execute{ "error": "…" }executereturned{ "error": "Database write may have completed", "outcome": "unknown" }How do you know it works?
Testing:
operation-runtime.test.tsserves the handler over real HTTP with a fake workspace and an in-memory data path. It covers 401 before any input is read, the 403 and 404 rules, one execution per request, transactions, the unknown write outcome, and sentinel tokens, inputs, and rows absent from spans and logs. Theoperation-actor,service-api,codecs, androuted-pathtests cover identity, the service API, timestamps, and path matching. The plugin, database, core, telemetry, utils, context, and shared database suites pass (131 files, 2239 tests), with the CRUD route suites unchanged.Stack created with GitHub Stacks CLI • Give Feedback 💬