Skip to content

chore: introduce DevelopmentTokenSource and deprecate SandboxTokenSource - #1149

Open
hiroshihorie wants to merge 2 commits into
mainfrom
hiroshi/development-token-source
Open

chore: introduce DevelopmentTokenSource and deprecate SandboxTokenSource#1149
hiroshihorie wants to merge 2 commits into
mainfrom
hiroshi/development-token-source

Conversation

@hiroshihorie

Copy link
Copy Markdown
Member

What

Port of livekit/client-sdk-swift#1077. LiveKit Cloud renamed the sandbox token server to the development token server, this aligns the API:

  • DevelopmentTokenSource({required String id}) is the new name, same endpoint and header
  • SandboxTokenSource remains as a deprecated subclass. Unlike Swift, a typealias is not enough here because the old constructor takes sandboxId: while the new one takes id:, so the subclass keeps existing call sites compiling unchanged
  • Doc references updated to the development token server wording

Also excludes build/ from analysis, Xcode vendors Swift package checkouts there when SPM is enabled (same fix already applied to components-flutter and the agent starter).

Notes

  • Changeset: minor type="added"
  • The JS SDK has the same concept (TokenSource.sandboxTokenServer) and has not been renamed yet, tracked separately

Port of livekit/client-sdk-swift#1077. LiveKit Cloud renamed the
sandbox token server to development token server. SandboxTokenSource
stays as a deprecated subclass so the old sandboxId constructor keeps
compiling. Also exclude build directories from analysis, Xcode vendors
Swift package checkouts there when SPM is enabled.
@hiroshihorie
hiroshihorie marked this pull request as ready for review August 4, 2026 07:08

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

Comment thread lib/src/token_source/development.dart Outdated
Comment thread lib/livekit_client.dart
export 'src/token_source/endpoint.dart';
export 'src/token_source/custom.dart';
export 'src/token_source/caching.dart';
export 'src/token_source/sandbox.dart';

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this break clients using the old path?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's only renamed, and should be ok.

Co-authored-by: lukasIO <mail@lukasseiler.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants