Bump facade CDN to 1.0.29#55
Closed
AndrewKirkovski wants to merge 3 commits into
Closed
Conversation
Web Host 1.0.29 release brings:
- proxy api default 401 → host.handleError('auth-expired') interceptor
with skipDefaultAuth per-request opt-out + cross-origin auto-skip
- fix for cold-cache "Proxy globals not found" race in host shells
(~1/3 fail rate on cold loads, host context only — iframe unaffected)
Refs updated in src/facade/{_index.yaml, README.md, Makefile,
config_handler_test.lua}.
Was used to confirm wippy.lock `replacements:` overrides hub-fetched vendor with the local checkout — the test query string appeared at runtime, proving the replace wires correctly. Removing it now that the smoke test is complete.
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.
Summary
fe_facade_urldefault and theMakefileWEB_HOST_CDNfromwebcomponents-1.0.26towebcomponents-1.0.291.0.27(prepared onfeat/host-config-layout-requirement, never merged) and1.0.28(superseded mid-PR by 1.0.29's hotfix)@wippy-fe/*workspace packages just published at0.0.29against the new hostWhat's in 1.0.29 vs 1.0.26
host.handleError('auth-expired')response interceptor. Single-flight per proxy-instance lifetime (a flood of in-flight requests after session expiry doesn't bounce 30 times). Per-request opt-out viaAxiosRequestConfig.skipDefaultAuth. Cross-origin auto-skip —api.get('https://api.openai.com/...')never leaks the Wippy token and a 401 from a third party never triggers our auth-expired bounce. Previously each child app had to wire its own 401 → login interceptor; in practice many didn't (keeper-v5's three apps shipped without any 401 handling and showed stale UI on session expiry).defineAsyncComponent'd, soapp.mount()returned before<VueAppGlobalConnector />(the only writer ofwindow[GLOBAL_API_PROVIDER]) mounted — the autoload-injected WC<script>tags fetched@wippy-fe/proxy.jsand hit its eager-getter throw on undefined globals. Fix relocatesregisterAutoloadComponents(...)into the connector's<script setup>immediately after the globals write, so they run as one synchronous atomic block.@wippy-fe/vite-pluginpackage (consumer-side build helper),@wippy-fe/sharedcross-boundary types,<wippy-loading>/<wippy-error>zero-dep components, deep-capture clipboard, modal-template registry + native<dialog>modal path, WS auth-failure → login redirect, lazy-WS lifecycle fixes.Files
src/facade/_index.yaml—fe_facade_urldefaultsrc/facade/Makefile—WEB_HOST_CDNconstant used bymake syncto pullloading.jsetc.src/facade/README.md— three doc referencessrc/facade/config_handler_test.lua— origin-extraction test fixtureCommits
1fa60c355e55b5310ea47Verification
The
live-yaml-test=1query-string marker added tofe_entry_path.default(in55e55b5) was used to confirm that the consumer-sidewippy.lockreplacements:block correctly overrides the hub-fetched.wippy/vendor/wippy/facade/_index.yamlwith this local checkout. The marker appeared at runtime whenwippy.exe runbooted in app-template — proving the replace wires correctly. Marker removed in310ea47. (Smoke test for the consumer-sidewippy.lockworkflow, not for the bump itself.)Note on branch name
Branch was originally named
chore/bump-facade-cdn-1.0.28before the 1.0.29 hotfix landed in gen-2-chat. The PR target version is now 1.0.29; branch name kept as-is to preserve PR continuity (no force-rename of remote refs).Test plan
facadepackage green (config_handler_test.luaupdated alongside source)make -C src/facade syncpulls freshloading.js,@wippy-fe/proxy.js, etc. fromwebcomponents-1.0.29https://web-host.wippy.ai/webcomponents-1.0.29/iframe.htmlonce gen-2-chat host is deployed at that tag🤖 Generated with Claude Code