Repository navigation
feat(appkit): add dev-only design pages plugin and ui-only template - #680
MarioCadenas wants to merge 42 commits into
Conversation
📦 Bundle size reportCompared against
|
| dist | raw | gzip |
|---|---|---|
| JS (runtime) | 1.3 MB (+56 KB) | 480 KB (+20 KB) |
| Type declarations | 467 KB (+2.6 KB) | 170 KB (+1.2 KB) |
| Source maps | 2.6 MB (+96 KB) | 891 KB (+32 KB) |
| Other | 11 KB | 3.7 KB |
| Total | 4.4 MB (+155 KB) | 1.5 MB (+54 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 (+65 B) | 2.5 KB | 109 KB (+65 B) | external | 344 KB (+77 B) |
./beta |
115 KB (+12 KB) | 482 B | 116 KB (+12 KB) | external | 343 KB (+33 KB) |
./testing |
45 KB (-1 B) | 32 KB (+74 B) | 77 KB (+73 B) | external | 224 KB (+175 B) |
./tsdown |
520 B | 0 B | 520 B | external | 813 B |
./type-generator |
26 KB (+31 B) | 0 B | 26 KB (+31 B) | external | 74 KB (+42 B) |
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 | 95 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 | 176 B |
./beta |
index.js |
lazy | 114 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): 402 KB (+5.0 KB) — gzipped download (dist + bin; excludes release-only docs/NOTICE).
| dist | raw | gzip |
|---|---|---|
| JS (runtime) | 431 KB (+2.0 KB) | 148 KB (+1003 B) |
| Type declarations | 269 KB (+1.8 KB) | 99 KB (+934 B) |
| Source maps | 857 KB (+4.1 KB) | 288 KB (+2.0 KB) |
| CSS | 16 KB | 3.2 KB |
| Other | 19 KB (+19 KB) | 1.9 KB (+1.9 KB) |
| Total | 1.6 MB (+27 KB) | 540 KB (+5.8 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 | 0 B | 2.1 KB | 0 B | 5.0 KB |
./react |
433 KB (+68 B) | 49 KB | 481 KB (+68 B) | 1.3 MB (+3 B) | 178 KB (+126 B) |
./react/beta |
4.9 KB (+73 B) | 0 B | 4.9 KB (+73 B) | 0 B | 13 KB (+171 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.9 KB |
|
Review 1 · Fail · 3 critical, 2 major Does the change do what the task asked? No. Gate: major and worse must be fixed. The canvas, multi-selection, notes, persistence across a real restart, and production 404 work, and the automated suite passes. The handoff is not reliable: normal selections can lack source locations, contain stale props, and include screenshots missing visible content. Valid wrapped components are also omitted, and concurrent reads can falsely report no pending selections. Required
|
|
Review 2 · Pass Does the change do what the task asked? Yes. Gate: major and worse must be fixed. The five earlier findings are fixed, and I found no regressions from those fixes. Browser checks against the built UI package, the full test suite, typechecking, and read-only lint/format checks passed. There are no remaining findings in this round. No findings. |
af1f4a2 to
aa65b0d
Compare
MarioCadenas
changed the title
|
Review 5 · Fail · 1 critical, 1 major Does the change do what the task asked? No. Gate: major and worse must be fixed. The design canvas, selection handoff, persistence and production 404 behavior work in the exercised cases, and all 50 targeted tests pass. The screenshot requirement is incomplete: selecting an absolutely positioned element can silently save an entirely blank image. The added CLI also omits declared resource defaults from deployment variables in an unusual supported case. Required
|
🤖 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 41685913132 -R databricks/appkit -n appkit-template-0.90.0-pr.b70e378-work-1-design-plugin-dev-only-design-680 -D appkit-pr-680 \
&& unzip -o "appkit-pr-680/appkit-template-0.90.0-pr.b70e378-work-1-design-plugin-dev-only-design-680.zip" -d "appkit-pr-680" \
&& databricks apps init --template "appkit-pr-680"The template pins |
aa65b0d to
4d00fbf
Compare
|
Review 6 · Pass Does the change do what the task asked? Yes. Gate: major and worse must be fixed. Both previous findings are fixed at the reviewed commit. Chrome reproduced the blank screenshot with the previous code and verified the saved JPEG with the fix; actual plugin-add runs preserved manifest defaults in deployment YAML. The focused regression checks passed, with no remaining findings in this round. No findings. |
4d00fbf to
acf5c0f
Compare
|
Review 7 · Fail · 3 critical Does the change do what the task asked? No. Gate: major and worse must be fixed. The main canvas, source tracing, persisted handoff, production 404 behavior, and ui-only template work in the exercised paths. Three browser-proven faults remain: open dialogs disable the picker, completing a send discards selections added during that send, and screenshots omit typed values in uncontrolled inputs. The repository was left unchanged. Required
|
|
Review 8 · Pass Does the change do what the task asked? Yes. Gate: major and worse must be fixed. F8, F9 and F10 are fixed: browser checks confirmed modal selection, preservation of later picks and edited notes during a pending send, and screenshots containing current form values. Focused tests and nearby runtime checks passed, with no remaining findings in this review round. The repository was left unchanged. No findings. |
9bdf7b9 to
ad6dc6e
Compare
|
Review 10 · Fail · 3 critical Does the change do what the task asked? No. Gate: major and worse must be fixed. The change does not meet the full task: the running app has no selection interface, open dialogs block selection, and clicking an SVG path saves a blank screenshot. The normal canvas handoff, source metadata, persistence across a server restart, and production 404 behavior worked locally. All 71 tests exercised passed, including the previously failing React type test. Required
|
|
Review 11 · Fail · 2 critical Does the change do what the task asked? No. Gate: major and worse must be fixed. F12 and F13 are fixed and worked in Chromium, but the change does not yet meet the supplied task. F11 remains against the original running-app selection requirement; the claimed later scope change was not confirmed. Pressing the advertised Escape shortcut after selecting dialog content also closes the dialog and silently loses its screenshot when sent. Required
|
MarioCadenas
changed the title
|
Review 12 · Fail · 2 critical, 1 major Does the change do what the task asked? No. Gate: major and worse must be fixed. The design pages and ui-only starter work, including hot reload and production 404s. The change does not meet the supplied task because element selection and persistent agent handoff are absent. An unrelated unfinished draft can also break an otherwise valid design page. Required
|
|
Review 13 · Fail · 2 critical Does the change do what the task asked? No. Gate: major and worse must be fixed. The broken-draft loading bug is fixed, including live browser rendering and hot reload beside an unfinished page. The current task still explicitly requires multi-selection with notes and a persisted agent handoff; both remain absent despite their intentional removal, so the change does not meet all stated requirements. Required
|
adfa702 to
b5f6314
Compare
|
Review 14 · Fail · 1 critical, 2 major Does the change do what the task asked? No. Gate: major and worse must be fixed. The change does not meet the full task: the required element selection and persistent agent handoff are absent. Design page rendering, navigation and production 404s work; 83 targeted tests pass and both changed TypeScript packages typecheck. Browser testing also reproduced false readiness after a render error and a broken route for pages/index.tsx. Required
|
b5f6314 to
48d89da
Compare
|
Review 15 · Fail · 1 critical, 1 major Does the change do what the task asked? No. Gate: major and worse must be fixed. The required selection and persistent agent-handoff workflow is still missing. The render-error fix works for edits to the index page, but readiness remains incorrectly true when an imported component fails after a hot edit. F20 is fixed, and all 49 focused design and server tests pass. Required
|
0341ad5 to
2d7a963
Compare
|
Review 20 · Pass · 1 minor, 1 trivial Does the change do what the task asked? Yes. Gate: major and worse must be fixed. Taking the human's later scope decisions into account (selection handling moved to the builder's inspector, drafts served with a hydrator), the branch does what was asked: /design is dev-only and returns 404 otherwise, drafts render real appkit-ui blocks, errors are reported, the props schema is published, and the ui-only template is generated. All 67 design-plugin tests and the use-design-ready tests pass. One edge case is left: the dark-mode token sync follows the system color scheme and ignores a draft that forces its own theme. Not required (below the gate)
|
|
Review 21 · Fail · 1 major Does the change do what the task asked? Yes. Gate: major and worse must be fixed. This round's additions (browserless status fields, theme.css, shipped schema, chart data types, template vitest inline) are in place and the design and chart tests pass. One claim is overstated: Required
|
- design() plugin (beta): in development serves a /design canvas that renders every PascalCase export under client/src/design/, and injects a click-to-select picker into the canvas and every app page. Picks are traced via the existing data-source transform plus the React fiber (component, owner stack, props), screenshotted, and persisted with an optional note to .appkit/design-selection.json (GET/POST/DELETE /api/design/selection). Outside development it mounts nothing and /design returns 404. - Plugins can contribute dev-server Vite plugins via an optional vitePlugins() hook. - appkit-ui/react/beta: DesignCanvas, DesignPicker, mountDesignPicker. - Template: gated client/src/design example, .appkit/ gitignored, and a ui-only app template (server + design). Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
|
Review 22 · Pass Does the change do what the task asked? Yes. Gate: major and worse must be fixed. F23 is fixed. compileGraph now runs transformRequest on the index module and then walks the module graph breadth-first, transforming every local script import. It skips node_modules and virtual modules, stops at 500 modules, and names the failing file. The new tests run against a real Vite server and pass, and nothing else in the design suite broke. No findings. |
Charts rendered at a fixed 300px by default and overflowed shorter cards. Without a height prop a chart now fills its parent's height (height: 100%) with a 160px minimum for parents sized by their content; an explicit height still sets an exact size. Applies to the chart, its wrapper and the loading / waiting states. Docs and tests (including jsdom sizing checks). Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
…and a stable URL - Hydrator: a data-appkit block still streaming in shows a skeleton (sized by kind), replaced in place by the rendered component with a 200 ms fade-in; both off with prefers-reduced-motion. - /design/drafts/__hydrator.js?draft=<name> is a stable URL for the hydrator, and /design/drafts/theme.css for the theme, so an app builder can include them in drafts it streams itself. Both are listed in /design/__status. The plugin's endpoints and Vite's module CORS allow the configured builderOrigin; the hydrator posts to its own origin (not via new URL(path, import.meta.url), which Vite rewrites to /@fs/...), and errors of drafts without a file are listed as streamed drafts. - Fix: react-router for /design pages is detected with Vite's resolver and must be 7+; Node's require.resolve could find a hoisted react-router@5 and the page then never rendered. - Docs and tests (incl. a parse check of the generated hydrator). Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
|
Review 24 · Fail · 1 major Does the change do what the task asked? Yes. Gate: major and worse must be fixed. The design plugin code in this round holds up: the 82 design tests pass, including the streaming, skeleton and hydrator-parse tests, and the workspace typecheck is clean. But the browser demo handed to the human does not render. Its hand-written Vite server leaves out appkit-ui's @/js alias, so every component import fails with a Vite error overlay, and the agent says it never opened the page in a browser. Required
|
The drafts folder was added to Vite's watcher. A changed .html file matches no module, so Vite sent a full reload, and in middleware mode (how the server plugin runs Vite) that reloads every open page on each write: a draft mid-stream restarted and the app reloaded too. Drafts are now watched with fs.watch, and only the appkit:draft-changed notice is sent. Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
|
Review 25 · Pass Does the change do what the task asked? Yes. Gate: major and worse must be fixed. F24 is fixed. The streaming demo at localhost:5199 now shows the full Trips dashboard (KPI cards, bar chart, table), with no Vite error overlay, no console errors and a single page load. The tests for the draft-watcher fix pass. No findings. |
ECharts advances its entry animation on animation frames, which the browser
pauses in hidden or background frames, such as a builder's preview. The axes
and labels were drawn while the series stayed frozen part-drawn or empty
(a blank line, a donut without its ring, a bar chart without bars). Charts
now draw their final state at once; options={{ animation: true }} turns the
animation back on.
Co-authored-by: Isaac <no-reply@databricks.com>
Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
…undle
The check for react-router 7 resolved it through Vite, which returns the
pre-bundled node_modules/.vite/deps/react-router.js. That folder's
package.json has no name, so the check failed, the page entry was built
without a router, and a page using <Link> crashed ("Cannot destructure
property 'basename'") and never painted. It now uses Node's resolution from
the design page, which finds the app's own copy.
Co-authored-by: Isaac <no-reply@databricks.com>
Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
…, render once - A block alone in its element fills that element's height, so an agent's sized box sizes the chart (it rendered at the 160px minimum). Next to a heading it keeps its own height and stays inside the card. - A chart's title is a heading above the chart instead of ECharts' title, which was drawn over the axes and legend. The schema says to leave it out when the container has its own heading. - A block that has rendered is never mounted again, also when the hydrator runs twice, and a block re-parsed with the same JSON doesn't fade in again. Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
|
Review 26 · Pass · 1 trivial Does the change do what the task asked? Yes. Gate: major and worse must be fixed. The round-17 fixes are in: charts default to animation:false and options can turn it back on, the react-router check now uses Node's resolver, and draft blocks size to their container with the title rendered as a heading. The design plugin, the ui-only template and the tests are all present, and the design and chart suites, typecheck and lint pass. I could not reproduce the hidden-iframe chart freeze in a real browser here. Not required (below the gate)
|
This reverts commit 509a53f. With the entry animation off, the first click on a chart was lost on the playground's metric views page (only a second click selected the bar), failing its integration test. Frozen part-drawn charts only stay frozen while the browser keeps animation frames paused (a background or occluded window); they finish when frames resume. Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
….json A block with data-src="mock#<key>" takes mock[<key>] from drafts/mock.json, served at /design/drafts/mock.json (also to the builder origins) and linked from /design/__status as `mock`. An array becomes the block's data (rows for a DataTable), an object is merged into its props; inline props win and the block may hold no JSON. The page fetches the file once; until it's there and parses, the blocks show their skeleton and the page asks again every second (errors after 30 s). A missing key fails just that block. Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
|
Review 28 · Pass · 1 minor, 1 trivial Does the change do what the task asked? Yes. Gate: major and worse must be fixed. The data-src/mock.json increment is small and looks correct. Blocks wait as skeletons, the file is fetched once and retried, inline props win, a missing key fails only its own block, and the server route sends no-store and CORS for builder origins. The design plugin tests pass; the one gap is that the 30-second give-up path is never exercised. Not required (below the gate)
|
…rrors
- data-src="mock#<key>" looks the key up in mock.json's "datasets" object
first, then at the top level: the drafts skill writes
{ mock, datasets: { <name>: rows } }, so every block failed to find its key.
- The fade-in is a CSS animation instead of a class set on an animation
frame. Frames don't run in a background tab or hidden frame, so blocks
(and "Couldn't render" boxes) stayed invisible there.
- /design/__status reports a draft's errors as null with checked: false
until a page has rendered its current version and handled every block,
instead of [] (which read as "no errors", e.g. right after a restart).
Co-authored-by: Isaac <no-reply@databricks.com>
Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
|
Review 29 · Fail · 1 major Does the change do what the task asked? Yes. Gate: major and worse must be fixed. The three round-21 fixes are in the code and read correctly: mock#key now checks datasets then the top level, the fade-in is a plain CSS animation that needs no animation frame, and /design/__status reports checked:false with errors:null until a page reports checked. The 94 design-plugin tests pass. One real risk is left in the new checked handshake: the reset, error and checked reports go out as separate, independent fetch POSTs, so the server can receive them out of order. Required
|
The hydrator's reset, error and checked reports were independent fetch POSTs, so the status endpoint could get them out of order: a late reset wiped `checked` and the errors, and an early `checked` briefly showed errors: []. Each report now starts once the one before it has finished. Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
|
Review 30 · Pass Does the change do what the task asked? Yes. Gate: major and worse must be fixed. F28 is fixed. The hydrator now chains its status POSTs on a single promise, so the reset, each error and checked reach the server in order, and a failed request doesn't stop the ones after it. The new test passes on the fix and fails on the old drafts.ts, so it really does catch the bug. No findings. |
…esign-plugin-dev-only-design-
|
Review 31 · Pass Does the change do what the task asked? Yes. Gate: major and worse must be fixed. This round had no earlier findings to re-check. The latest fix (3e4b35c) sends the draft status reports one after another, and the design plugin's tests pass. I found no new critical fault. No findings. |
For apps that skip /design (no data), the builder loads the app's / in a hidden frame and reveals it on appkit:app-ready. The app-ready script's last wait was two animation frames, which never run in a display:none frame, so the signal never came. On the app's pages it now waits for two frames or 100 ms, whichever comes first. /design keeps its two-frame wait. The draft-watcher test now writes again until the watcher is live: macOS's file events can miss writes made just after a watch starts. Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
…th-mo' into work/1-design-plugin-dev-only-design- Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
|
Review 32 · Pass Does the change do what the task asked? Yes. Gate: major and worse must be fixed. This round's change is small: the app-ready post now waits for two animation frames or 100 ms, whichever comes first, so a builder that preloads / in a display:none frame still gets the signal. The new tests pass on the fix and both fail on the pre-fix vite-plugin.ts, which shows they catch the bug. The design plugin suite (97 tests) and the repo-wide typecheck are green. I found nothing new that needs fixing. No findings. |
…esign-plugin-dev-only-design-
bundle-size-baseline.json: took main's (v0.90.0 release) baseline; this branch's appkit tarball grows 3.8% against it, within the 5% budget. Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
|
Review 33 · Pass Does the change do what the task asked? Yes. Gate: major and worse must be fixed. This round only merged origin/main into the branch. bundle-size-baseline.json now matches main byte for byte, nothing in the design plugin changed in the merge, all 12 PR checks pass on the merge commit, and the 100 design tests pass locally. I found nothing new. No findings. |
…ILDER_ORIGIN A scaffolded app calls design() with no options, so no builder origin was set and the plugin never added the app-ready script: a builder waiting for appkit:app-ready on / (apps that skip /design) waited forever. Without a builderOrigin option, design() now reads APPKIT_DESIGN_BUILDER_ORIGIN (one origin, or several separated by commas), which the builder's runtime sets. It is checked like the option; an explicit builderOrigin wins. Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
…th-mo' into work/1-design-plugin-dev-only-design- Co-authored-by: Isaac <no-reply@databricks.com> Signed-off-by: MarioCadenas <MarioCadenas@users.noreply.github.com>
|
Review 34 · Pass · 1 minor Does the change do what the task asked? Yes. Gate: major and worse must be fixed. The round's change does what it says: a bare design() reads APPKIT_DESIGN_BUILDER_ORIGIN, validates it like the option, and an explicit builderOrigin still wins. The design plugin suite passes at the head commit, and the merge commit changes no files. One narrow gap: a bad variable value throws even in production, where the plugin is supposed to do nothing. Not required (below the gate)
|
Adds a dev-only
design()plugin (beta) that serves design pages at full size, and an emptyui-onlyapp template (server + design) for quick PoCs.What it does
/designrenders the default export ofclient/src/design/index.tsx./design/<name>rendersclient/src/design/pages/<name>.tsx. Each page renders on its own at full size with the app's styles (client/src/index.css) and hot reload. Anything else under/designreturns 404.react-router(the template ships it), design pages share a router whose basename is/design, so<Link to="/orders">stays at/design/orders. The requested page is imported directly and other pages load only when navigated to, so an unfinished draft breaks only its own URL.GET /design/__statusreturns{ ready, pages }.readymeansclient/src/design/index.tsxexists, isn't the template placeholder (export const designPlaceholder = true), and has painted since it last changed. After first paint (and after each hot update), the page posts{ type: "appkit:design-ready", version: 1, page, pages }towindow.parent(elsewindow.opener), only to the configuredbuilderOrigin(bare origins only;"*"is rejected).index.htmland the/designpage declarecolor-scheme: light dark; the theme already followsprefers-color-scheme.data-source="file:line:col"attributes, so the app builder's own element inspector can map a click back to source. The plugin has no picker or selection store of its own./designreturns 404. The manifest is deliberately notdevOnly: a dropped plugin couldn't claim/design, and the SPA fallback would serve the app'sindex.htmlfor it with a 200.vitePlugins()method. That's how the design plugin serves its pages.ui-onlyentry intools/generate-app-templates.ts(features:design), a gatedclient/src/design/index.tsxsample page, and a README line.How I checked it
/designand/design/ordersrendered their pages full size withdata-sourceattributes, and no page errors./design/missing,/design/a/band/design/orders.tsxreturned 404, and/api/design/selectionreturned 404 (no API)..jsx, missing pages, paths outside/design) and the production 404.pnpm build,pnpm docs:build,pnpm -r typecheck,pnpm check, knip andpnpm test(312 files) all pass. The bundle baseline is refreshed.Not done
ui-onlywasn't generated withdatabricks apps init(needs the CLI and network); only the template files changed.Task: KIT-1 (app-kit/1)
This pull request and its description were written by Isaac.