Website roll fwd#366
Conversation
Dev to main
Dev > Main
Dev > Main
Dev to main
Dev > Main
removed extraneous return for Windows clippy
updated changelog for v0.4.5
Dev > Main
updated changelog for 0.4.6
Dev > Main
add workflow dispatch for prod wasm
Dev > Main
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (3)
📒 Files selected for processing (49)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 7782046. Configure here.
| <div className="max-w-sm"> | ||
| <p className="text-sm font-semibold">{label}</p> | ||
| {note ? <p className="mt-1 text-xs opacity-70">{note}</p> : null} | ||
| </div> |
There was a problem hiding this comment.
Placeholder developer notes visible to production users
High Severity
The Media component unconditionally renders its label and note props as visible text. The homepage passes developer-facing strings like "Final app/browser graphic goes here." and "MATLAB code snippet with a matrix math error underline. Put logos for Apple, NVIDIA and ARM GPU under." as note values. These internal design instructions are displayed directly to production users on the homepage hero and feature sections.
Additional Locations (2)
Reviewed by Cursor Bugbot for commit 7782046. Configure here.
3 participants
Note
Medium Risk
Medium risk because it introduces a new dynamic
/download/latestredirect flow with host/channel/platform detection and URL allowlisting, plus large website page refactors that could affect download funnels and navigation.Overview
Bumps all workspace crates and the TS bindings package from
0.4.1to0.4.8(including lockfiles) to align release versions.Adds a new dynamic Next.js endpoint
GET /download/latestthat selects dev vs prod channel based on host/env, infers platform from Client Hints/UA (with safe fallbacks), fetches alatest.jsonmanifest, allowlists redirect origins, and falls back to the/downloadchooser; includes unit tests for channel/platform/url validation.Reworks key marketing pages: the home page is rewritten around a parameterized
Hero, newMediaplaceholder blocks, and a client-sideDetectedDownloadLabel; the download page now emphasizes desktop downloads via/download/latestplus a separate CLI install section; navigation links are updated to point to/download/latest.Updates docs and sandbox launch plumbing: builtin docs now support and render
extended_capabilities, and workspace launch payloads/buttons add optionalagentPromptand improved analytics metadata.Refreshes pricing/footer UI and copy (e.g., removes Hobby cloud tier in the selector, adjusts CTAs to
/login, and updates messaging around desktop availability).Reviewed by Cursor Bugbot for commit 7782046. Bugbot is set up for automated code reviews on this repo. Configure here.