Repository navigation
Tiny Sunny: show why Base44 refused live updates - #53
Closed
amitbardos wants to merge 1 commit into
Closed
amitbardos wants to merge 1 commit into
amitbardos wants to merge 1 commit into
Conversation
Opening a socket session can be refused for a missing apps:watch scope, a workspace without white-label sockets, or an app the key isn't granted. The example reported all of them as a missing scope and the UI showed a generic message; it now passes Base44's reason through to the chat. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
✅ Deploy Preview for base44-platform-starter ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for sunny44-builder ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Contributor
Author
|
The files this changes (useBuilderSocket.ts, socket-session.ts) are no longer on main; Tiny Sunny's live updates now go through the SDK (#79). |
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.
After #52, tiny.sunny44.com still pauses live updates with "needs a workspace key with the apps:watch scope", even with the scope added.
POST /api/service/socket-sessionsreturns 403 for three reasons (apperbackend/app/whitelabel_sockets/api.py):scope_required: the key lacksapps:watchwhitelabel_sockets_disabled: the workspace doesn't have bothbase44-for-platformsandwhitelabel-websocketsapp_not_allowed: the key isn't granted the app (watch is an app-targeted scope)The example mapped every 401/403 to the scope message, and the chat showed only "Live updates paused". It now passes Base44's
error.messagethrough, and the chat shows it when it comes from our server. Socket errors keep the generic message.Unit tests, typecheck and the 22 Playwright tests pass.
🤖 Generated with Claude Code