Skip to content

fix: robots and lazy load playground - #4671

Open
ematipico wants to merge 1 commit into
mainfrom
chore/improve-site
Open

ematipico wants to merge 1 commit into
mainfrom
chore/improve-site

Conversation

@ematipico

Copy link
Copy Markdown
Member

Summary

I quickly vibe-coded the following things:

  • exclude the playground from the bots
  • make the playground lazy-load itself via a button. The WASM files are huge, so if an agent or a but arrives, at least we don't load all those assets.

These suggestions come from people at Netlify.

@netlify

netlify Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for biomejs ready!

Name Link
🔨 Latest commit 6d0a1af
🔍 Latest deploy log https://app.netlify.com/projects/biomejs/deploys/6adb8b72dfc24ff65735e572
😎 Deploy Preview https://deploy-preview-4671--biomejs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@dyc3

dyc3 commented Oct 23, 2026

Copy link
Copy Markdown
Contributor

make sure to fix the ci

@coderabbitai

coderabbitai Bot commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
📥 Commits

Reviewing files that changed from the base of the PR and between 64c8641 and 6d0a1af.

📒 Files selected for processing: 4
  • public/robots.txt
  • src/playground/PlaygroundLoader.tsx
  • src/playground/components/LoadingScreen.tsx
  • src/styles/playground/index.css

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The robots file now disallows crawling of /playground/. In the playground, the loading screen offers a button before loading starts. When the button is used, PlaygroundLoader requests loading and creates the Biome and Prettier workers. The loading screen and button styles have also changed.

Suggested reviewers: dyc3

Priority: ⬇️ Low

Change: Other

Merge Risk: ⚪ Minimal · up to 6d0a1

The playground now avoids starting its worker and WASM loading on an unrequested visit, with no established merge-blocking risk.

Pre-merge checks: Passed 4
Check name Status Explanation
Description check Passed The description clearly relates to both changes: excluding the playground from crawlers and loading it on demand to avoid unnecessary WASM downloads.
Title check Passed The title clearly summarises the two main changes: updating robots.txt and lazy-loading the playground.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches: 💡 1
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
🛠️ Fix failing CI checks: 💡
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

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.

2 participants