Conversation
✅ Deploy Preview for sunny44-builder ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
✅ Deploy Preview for base44-platform-starter ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
ozsay
force-pushed
the
codex/white-label-builder-socket
branch
from
September 17, 2026 10:32
2d810d8 to
ec485ac
Compare
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
Replace periodic chat polling in
examples/white-label-minimalwith the platform SDK builder socket. The browser subscribes before loading its initial conversation, applies streamed message replacements and image resolutions, and lets the SDK resume from applied cursors after transport reconnects. There is no periodic app/conversation timer.@base44/sdkto the published preview aliasnpm:@base44-preview/sdk@0.8.48-pr.286.7d59d14from Add platform browser client for white-label sockets javascript-sdk#286, verified to containbuilder.initandrefreshToken.Validation
npm run minimal:test: 37 server, identity, stream, conversation and question tests passed.npm run minimal:typecheckandnpm run minimal:build: passed.git diff --checkpasses.Rollout limitations
The current service-user token also authorizes HTTP operations. The read-only socket and token-route ownership check do not narrow the token itself to one app or to socket-only access. Replace this temporary exchange when the browser-specific credential solution lands.
The backend must accept this token on the white-label endpoint and enable the workspace flag. The pending verifier still prevents real deployed connections until integrated. Verification here uses fixture sockets and mocked credentials, not a deployed backend. A fresh snapshot plus buffered events is not an atomic history API and may briefly reapply state already present in the snapshot. Queue/task events are consumed for cursor continuity; this minimal UI renders progress from app/message updates.
Intent (for reviewers & PR Autofix)
What this PR does
Use the new platform client SDK's builder module for minimal-example chat updates, preserving existing HTTP mutations and the readable getting-started structure.
Key decisions & why
What NOT to touch
The SDK preview pin is intentional until its client entry point is released. Do not restore periodic polling or silently fall back to the legacy socket. Keep the existing app-ownership and same-origin checks for token retrieval and HTTP actions.
Tradeoffs / follow-ups
Integrate the dedicated browser credential and verify the real backend before rollout. This PR changes only the minimal example, not the main starter app or the SDK/backend PRs.