Skip to content

Fix WASM path order to avoid 404 in console#361

Merged
JakeSCahill merged 5 commits intomainfrom
fix/wasm-path-order
Feb 5, 2026
Merged

Fix WASM path order to avoid 404 in console#361
JakeSCahill merged 5 commits intomainfrom
fix/wasm-path-order

Conversation

@JakeSCahill
Copy link
Copy Markdown
Contributor

@JakeSCahill JakeSCahill commented Feb 4, 2026 •
edited
Loading

Summary

  • Swaps the order of WASM paths in the mini-playground loader to try the production path (/blobl.wasm) first
  • Avoids a 404 console error on production when the UI bundle path (/_/blobl.wasm) is tried first and fails

Test plan

  • Test mini-playground on local preview (gulp preview)
  • Verify no 404 errors appear in console on production after deploy

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 4, 2026 •
edited
Loading

✅ Deploy Preview for docs-ui ready!

Name Link
🔨 Latest commit 3314141
🔍 Latest deploy log https://app.netlify.com/projects/docs-ui/deploys/6984a9c2129d01286cf72428
😎 Deploy Preview https://deploy-preview-361--docs-ui.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse		 1 paths audited
Performance: 28 (no change from production)
Accessibility: 93 (no change from production)
Best Practices: 100 (no change from production)
SEO: 89 (no change from production)
PWA: -
View the detailed breakdown and full score reports

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

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 4, 2026 •
edited
Loading

Important

Review skipped

Auto incremental 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.

  • 🔍 Trigger a full review
📝 Walkthrough

Walkthrough

The change reorders the WASM path resolution logic in the loadBloblangWasm function. The function now attempts to load the WASM file from the production/Netlify path first (siteRoot + '/blobl.wasm'), followed by the UI Preview path (rootPath + '/blobl.wasm'). A clarifying comment was added to document this priority ordering.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: reordering WASM paths to prevent 404 errors in the console.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
Description check ✅ Passed The pull request description clearly relates to the changeset, explaining the WASM path reordering and the production 404 error being addressed.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/wasm-path-order

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 and usage tips.

@JakeSCahill JakeSCahill requested a review from a team February 4, 2026 20:03
paulohtb6
paulohtb6 approved these changes Feb 5, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@paulohtb6 paulohtb6 left a comment

Choose a reason for hiding this comment

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

great changes

@JakeSCahill JakeSCahill merged commit c5c2e90 into main Feb 5, 2026
7 checks passed
@JakeSCahill JakeSCahill deleted the fix/wasm-path-order branch February 5, 2026 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@paulohtb6 paulohtb6 paulohtb6 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JakeSCahill @paulohtb6