Repository navigation
admin-ui preload: finish rollout after DEV deploy test (follow-up to #2748) #2749
Description
Follow-up to the Component-preload prototype in #2748 (branch worktree-admin-preload-prototype), which enables the preload build for one tile (puzzles) only. These items are blocked on first merging #2748 to DEV and doing a real DEV deploy, because the open questions can only be answered on the deployed approuter.
Gate: DEV deploy test of #2748
- Full
npm run deploy -- --env dev(NO--skip-build, NO-mscoping — admin-UI is bundle-gated). - On DEV
/admin-ui/, open the Puzzles tile and confirm in DevTools Network that it loadsComponent-preload.js(~1 request) and NOT ~10 per-module requests. - Confirm the preload loads under the approuter CSP (no
unsafe-eval;sap.ui.require.preloadis CSP-safe but verify no console ModuleError). - Confirm the crossword builder's Web Worker (
lib/fill-worker.js) still starts —new Worker(sap.ui.require.toUrl(...))resolves the standalone file, not the preload.
Blocked on that gate passing
- Update Step 3.5 bundle-gate (
scripts/check-shipped-admin-bundle.cjs): it diffs shipped admin files against source and will flag preloaded output (minified +Component-preload.js) as drift. Teach it to compare the built output for preload components, or exempt them. - Fan out
PRELOAD_COMPONENTSto the remaining heavy tiles once the pattern is proven. Candidates by source size:secrets,joule,concepts,homepage,shelf-definitions,verb-definitions,topicClusters,kgCommunities,tags. Add incrementally; each is independently revertible by removing it from the set. - Decide whether to ship
-dbgsources +.mapfiles (debug fallback, larger approuter footprint) or strip them from the deployed static dir. - Note the new failure mode in gotchas: with minification on, syntactically-invalid controller JS now fails at the
ui5 buildminify step for preload components (previously only atmbtminify during deploy). - Commit an e2e spec exercising a preloaded tile (advisory nudge fires on
app/**changes; real coverage is the post-DEV-deploye2ejob).
Prototype measurement (puzzles): ~10 authed approuter round-trips → ~2; 77 KB unminified → 39.6 KB minified bundle.
Activity
jung-thomas commentedon Oct 13, 2026
Update — DEV deploy test passed; bundle-gate item already done
Tom manually ran the Puzzles tile on DEV and confirmed it's fine. Verified on `origin/DEV` (merge `e90dcd21`):
Caveat on verification: `/admin-ui/` is XSUAA-gated and `cf ssh` to `tutorials-dev-approuter` is denied, so the deployed-artifact half rests on Tom's manual confirmation; the source half is confirmed on the DEV branch.
Remaining (now building out the full solution):
jung-thomas commentedon Oct 13, 2026
Full fan-out built out in #2754 (branch
worktree-admin-preload-fanout), targeting DEV:discoverComponents(); full build = 51/51 preload, 0 fallbacks.-dbg+.map— pruned inbuild-component.cjs; 0 shipped. Worker file kept.test/e2e/admin-preload.test.jsasserts#puzzlesfetchesComponent-preload.js(200), no module error.Plus hardening not in the original list: silent-but-logged fault isolation (one bad tile → raw-copy fallback, build stays green), and the 40 config files (20
ui5.yaml+ 20package.json+ 16 camelCase name fixes) tiles needed to build standalone.Open item for post-merge DEV deploy: run the e2e spec against DEV to confirm the preload loads under the approuter CSP (the one thing not verifiable pre-deploy). Once that passes, #2749 can close.
Merge pull request #2754 from sap-tutorials/worktree-admin-preload-fa…