Conversation
- New /demo page with Game Boy emulator (@moq/boy) - Refresh nav: demo/blog/docs/github/discord icons - Rewrite index.mdx copy and restructure links - Drop unused hang/issues components from watch/publish - Bump @moq/publish to 0.2.3 and @moq/watch to 0.2.4 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 26 minutes and 27 seconds. ⌛ 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: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (19)
WalkthroughUpdated package.json dependencies (added a new library and bumped several deps/devDeps). Added 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 6
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@src/components/boy.tsx`:
- Line 5: The code constructs the demo relay URL without any JWT query param;
update the URL creation so it conditionally includes a JWT when present (e.g.,
read from import.meta.env.PUBLIC_DEMO_JWT or a passed-in demoToken) and add it
via url.searchParams.set('token', demoJwt) before using the URL; modify the
existing url variable creation (new URL("/demo",
import.meta.env.PUBLIC_RELAY_URL)) to append the token only if demoJwt is truthy
so demo broadcasts work when relay auth is enabled.
In `@src/layouts/global.astro`:
- Line 79: Update the img element that currently uses alt="Github" to use the
official brand capitalization alt="GitHub" so the <img src="/layout/github.svg"
class="w-32" alt="GitHub" /> tag in src/layouts/global.astro reflects the
correct brand name.
In `@src/pages/boy.astro`:
- Line 37: Update the user-facing copy string that contains "🔍
<strong>Discoverable</strong>: The CDN announces which games are available
<em>and</em> who wants to play. There's no custom orchistration layer; all
clients connect to <code>cdn.moq.dev</code> <b>ONLY</b>." by correcting the typo
"orchistration" to "orchestration" so the sentence reads "...There's no custom
orchestration layer..."; locate and edit that literal string in the template
(the text block in the page markup).
In `@src/pages/demo.mdx`:
- Around line 11-23: The Hang card currently nests an inner anchor inside the
outer anchor (outer <a ...> wrapping <article> and inner <a
href="https://hang.live">hang.live</a>), which is invalid HTML; remove the inner
anchor tag and replace it with plain text or a non-anchor element (e.g., <span>
or <div>) so only the outer <a> remains clickable, keeping the visible
"hang.live" text and styling intact within the article.
In `@src/pages/index.mdx`:
- Line 52: Replace the platform name "Github" with the correct capitalization
"GitHub" in the link text on the landing content (the string currently reading
"The [docs](https://doc.moq.dev) and [Github](https://github.com/moq-dev/moq) if
you're a nerd."); update the bracketed link label from "Github" to "GitHub"
within src/pages/index.mdx so the visible text uses the proper casing.
- Around line 21-23: Update the anchor href for the "open source" link so it
uses HTTPS instead of HTTP; locate the anchor with text "open source"
(href="http://github.com/moq-dev/hang.live") and change the scheme to
"https://github.com/moq-dev/hang.live" to avoid insecure redirects.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 550fceaf-8dea-4194-a7ef-87293c53cba9
⛔ Files ignored due to path filters (7)
bun.lockis excluded by!**/*.lockpublic/demo/bunny.pngis excluded by!**/*.pngpublic/demo/moqboy.svgis excluded by!**/*.svgpublic/layout/blog.svgis excluded by!**/*.svgpublic/layout/demo.svgis excluded by!**/*.svgpublic/layout/docs.svgis excluded by!**/*.svgpublic/layout/github.svgis excluded by!**/*.svg
📒 Files selected for processing (12)
package.jsonsrc/components/boy.tsxsrc/components/hang.astrosrc/components/issues.astrosrc/components/publish.tsxsrc/components/watch.tsxsrc/layouts/global.astrosrc/pages/boy.astrosrc/pages/demo.mdxsrc/pages/index.mdxsrc/pages/publish.astrosrc/pages/watch.astro
💤 Files with no reviewable changes (4)
- src/components/issues.astro
- src/components/hang.astro
- src/pages/watch.astro
- src/pages/publish.astro
| import "@moq/watch/support/element"; | ||
|
|
||
| export default function Boy() { | ||
| const url = new URL("/demo", import.meta.env.PUBLIC_RELAY_URL); |
There was a problem hiding this comment.
Add JWT query support to the demo relay URL.
Line 5 always uses /demo without a token. That breaks the repo requirement for demo broadcasts when relay auth is enabled.
🔧 Suggested fix
export default function Boy() {
- const url = new URL("/demo", import.meta.env.PUBLIC_RELAY_URL);
+ const token = import.meta.env.PUBLIC_RELAY_TOKEN;
+ const path = token ? `/demo?jwt=${encodeURIComponent(token)}` : "/demo";
+ const url = new URL(path, import.meta.env.PUBLIC_RELAY_URL);As per coding guidelines: "Support basic JWT authentication via query parameters for demo broadcasts".
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| const url = new URL("/demo", import.meta.env.PUBLIC_RELAY_URL); | |
| const token = import.meta.env.PUBLIC_RELAY_TOKEN; | |
| const path = token ? `/demo?jwt=${encodeURIComponent(token)}` : "/demo"; | |
| const url = new URL(path, import.meta.env.PUBLIC_RELAY_URL); |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/components/boy.tsx` at line 5, The code constructs the demo relay URL
without any JWT query param; update the URL creation so it conditionally
includes a JWT when present (e.g., read from import.meta.env.PUBLIC_DEMO_JWT or
a passed-in demoToken) and add it via url.searchParams.set('token', demoJwt)
before using the URL; modify the existing url variable creation (new
URL("/demo", import.meta.env.PUBLIC_RELAY_URL)) to append the token only if
demoJwt is truthy so demo broadcasts work when relay auth is enabled.
| <a href="/source"> | ||
| <img src="/layout/source.svg" class="w-32" alt="Source" /> | ||
| <a href="https://github.com/moq-dev/moq"> | ||
| <img src="/layout/github.svg" class="w-32" alt="Github" /> |
There was a problem hiding this comment.
Use official brand capitalization in alt text.
Line 79 should be alt="GitHub" (not Github).
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/layouts/global.astro` at line 79, Update the img element that currently
uses alt="Github" to use the official brand capitalization alt="GitHub" so the
<img src="/layout/github.svg" class="w-32" alt="GitHub" /> tag in
src/layouts/global.astro reflects the correct brand name.
|
|
||
| ### More Stuff | ||
| - The [blog](/blog) if you're a fan of hot takes. | ||
| - The [docs](https://doc.moq.dev) and [Github](https://github.com/moq-dev/moq) if you're a nerd. |
There was a problem hiding this comment.
Fix platform name capitalization.
Line 52 should use GitHub instead of Github.
🧰 Tools
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@src/pages/index.mdx` at line 52, Replace the platform name "Github" with the
correct capitalization "GitHub" in the link text on the landing content (the
string currently reading "The [docs](https://doc.moq.dev) and
[Github](https://github.com/moq-dev/moq) if you're a nerd."); update the
bracketed link label from "Github" to "GitHub" within src/pages/index.mdx so the
visible text uses the proper casing.
@moq/boy 0.2.3, @moq/publish 0.2.4, @moq/watch 0.2.5 (pulls in @moq/lite 0.2.0), plus astro, solid-js, biome, wrangler, and other dev dep bumps via bun update. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Each demo page now shows a jsDelivr-based copy/paste snippet for embedding the underlying @moq/* web component in any HTML page, auto-populated with the page's live relay URL and broadcast name. Pages moved from .astro to .mdx so the features list and surrounding prose is plain markdown. Interactive Solid components are wrapped in thin .astro files so client:only correctly skips SSR of the custom element registrations. Syntax highlighting via highlight.js (atom-one-dark) with a small global override so the theme doesn't double-stack backgrounds on top of the prose plugin's pre styling. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
♻️ Duplicate comments (1)
src/pages/index.mdx (1)
52-52:⚠️ Potential issue | 🟡 MinorCapitalize “GitHub” correctly in the link label.
Line 52 uses “Github”; the correct branding is “GitHub”.
🔧 Suggested fix
-- The [docs](https://doc.moq.dev) and [Github](https://github.com/moq-dev/moq) if you're a nerd. +- The [docs](https://doc.moq.dev) and [GitHub](https://github.com/moq-dev/moq) if you're a nerd.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/pages/index.mdx` at line 52, The link label currently uses the incorrect capitalization "Github"; update the text label to "GitHub" so the sentence reads "The [docs](https://doc.moq.dev) and [GitHub](https://github.com/moq-dev/moq) if you're a nerd." — locate and change the link label text in the index.mdx content where the Github link is defined.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Duplicate comments:
In `@src/pages/index.mdx`:
- Line 52: The link label currently uses the incorrect capitalization "Github";
update the text label to "GitHub" so the sentence reads "The
[docs](https://doc.moq.dev) and [GitHub](https://github.com/moq-dev/moq) if
you're a nerd." — locate and change the link label text in the index.mdx content
where the Github link is defined.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 4114f81e-b12e-4d6c-9758-f7346505c166
📒 Files selected for processing (1)
src/pages/index.mdx
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 participant
Summary
/demopage with Game Boy emulator powered by@moq/boyhang/issuescomponents from watch and publish pages@moq/publishto 0.2.3 and@moq/watchto 0.2.4Test plan
just devand click through /, /demo, /boy, /watch, /publishjust check🤖 Generated with Claude Code