Skip to content

Conversation

@kennethkalmer
Copy link
Member

Summary

Configure Sandpack instances to use Ably's custom bundler instead of the default CodeSandbox bundler.

Changes

  • Updated SandpackProvider options to include bundlerURL: 'https://sandpack-production-366dfb2b55fc.herokuapp.com'
  • All code examples will now use the custom bundler for faster build times

Test plan

  • Verify code examples render correctly with the custom bundler
  • Test various example types (JavaScript, React, etc.)
  • Confirm bundling performance improvements

🤖 Generated with Claude Code

@coderabbitai
Copy link

coderabbitai bot commented Jan 20, 2026

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

@kennethkalmer kennethkalmer added the review-app Create a Heroku review app label Jan 20, 2026
@ably-ci ably-ci temporarily deployed to ably-docs-web-4870-cust-jhn26s January 20, 2026 11:48 Inactive
@kennethkalmer kennethkalmer changed the title Use custom Sandpack bundler [WEB-4870] Use custom Sandpack bundler Jan 20, 2026
@kennethkalmer kennethkalmer marked this pull request as ready for review January 21, 2026 10:02
@kennethkalmer kennethkalmer self-assigned this Jan 21, 2026
Copy link
Contributor

@GregHolmes GregHolmes left a comment

Choose a reason for hiding this comment

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

All good!
Only query I have is would this be better as an env var? Not a blocker though!

Configure Sandpack instances to use custom Ably bundler at
https://sandpack-production-366dfb2b55fc.herokuapp.com instead
of the default CodeSandbox bundler.

Co-Authored-By: Claude Sonnet 4.5 (1M context) <noreply@anthropic.com>
@GregHolmes GregHolmes force-pushed the WEB-4870-custom-sandpack-bundler branch from 8f49121 to e8d057b Compare January 21, 2026 10:09
@GregHolmes GregHolmes temporarily deployed to ably-docs-web-4870-cust-jhn26s January 21, 2026 10:09 Inactive
@kennethkalmer
Copy link
Member Author

Only query I have is would this be better as an env var? Not a blocker though!

@GregHolmes I was wondering the same thing, but honestly I don't see the value because I don't see this changing any time soon

@kennethkalmer kennethkalmer merged commit ad05099 into main Jan 21, 2026
7 checks passed
@kennethkalmer kennethkalmer deleted the WEB-4870-custom-sandpack-bundler branch January 21, 2026 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-app Create a Heroku review app

Development

Successfully merging this pull request may close these issues.

4 participants