Skip to content

docs: rewrite README to house standard - #6

Open
yakimoto wants to merge 1 commit into
mainfrom
docs/readme-refresh
Open

docs: rewrite README to house standard#6
yakimoto wants to merge 1 commit into
mainfrom
docs/readme-refresh

Conversation

@yakimoto

@yakimoto yakimoto commented Aug 5, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Rewrites the README to WAVE's house README standard (badges, links row) while keeping the existing accurate content (which was already grounded) — expands file-by-file description using the actual source comments in console/moq-wire-browser.js, moq-player.js, and session.js.
  • No new capabilities claimed; this repo has no capabilities.json, so no capability table is included.

Test plan

  • Preview renders correctly on GitHub
  • npm run serve still matches package.json's serve script

Note

Low Risk
Documentation-only README changes with no runtime or dependency impact.

Overview
Reformats README.md to WAVE’s house layout: centered title (crest-console), badge row, and a links line (Live · wave.online · github), plus a footer with company/docs links.

The technical content is expanded and clarified rather than replaced: each console/ file is described with more detail from the source (MoQ wire message types, WebCodecs/H.264/Chrome codec note, POST /api/v1/crest/session gateway proxy). The Run it section spells out the ?relay=&ns=&track= query params and notes the default relay wss://moq.wave.online. License line updates entity name to “WAVE Online, LLC”; open-core wording is tightened slightly (“client here” vs “client”).

Reviewed by Cursor Bugbot for commit ae11364. Configure here.

Review in cubic

Expand the README to WAVE's house style (badges, quick start, links row,
grounded status/usage sections) while keeping every capability claim
traceable to something actually in this repo (source, package.json,
capabilities.json, or the published npm/registry artifact).
@cursor

cursor Bot commented Aug 5, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_e6bc7c10-6063-4d95-a236-027518c4d67d)

@coderabbitai

coderabbitai Bot commented Aug 5, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You’ve reached a temporary PR review limit under our Fair Usage Limits Policy.

Your recent review volume is higher than typical usage, so adaptive limits are currently applied.

Next review available in: 2 minutes

Your organization has reached its usage spending cap. Adjust your spending cap in the billing tab.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: bd117039-9651-4864-9f04-1e0c4aa4a4d5

📥 Commits

Reviewing files that changed from the base of the PR and between a4676d6 and ae11364.

📒 Files selected for processing (1)
  • README.md

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

@qodo-code-review

Copy link
Copy Markdown

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

To customize comments, go to the Qodo configuration screen, or learn more in the docs.

Qodo Logo

@macroscopeapp

macroscopeapp Bot commented Aug 5, 2026

Copy link
Copy Markdown

Approvability

Verdict: Approved ae11364

Documentation-only change to README.md — reformatting, adding badges, and expanding descriptions. No code or runtime behavior affected.

You can customize Macroscope's approvability policy. Learn more.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review

Comment thread README.md
@qodo-code-review

Copy link
Copy Markdown

PR Summary by Qodo

docs: rewrite README to WAVE house standard (badges, links, grounded module notes)

📝 Documentation 🕐 10-20 Minutes

Grey Divider

AI Description

• Reformat README to WAVE house style (centered header, badges, links row).
• Expand “What this is” with grounded per-file behavior from in-repo sources.
• Clarify run instructions and keep scope/capability claims intentionally constrained.
Diagram

graph TD
  R[/"README.md"/] --> C["console/ overview"] --> W["moq-wire-browser.js"] --> P["moq-player.js"] --> S["session.js + index.html"]
Loading
High-Level Assessment

The following are alternative approaches to this PR:

1. Auto-generate README sections from source docstrings/comments
  • ➕ Reduces drift between README and implementation over time
  • ➕ Keeps “grounded claims” enforceable via CI checks
  • ➖ Adds tooling/CI complexity for a small repo
  • ➖ Generated prose can become less readable than curated text
2. Move detailed module descriptions to a /docs page and keep README minimal
  • ➕ Keeps README short and scan-friendly
  • ➕ Allows richer docs structure without bloating the landing page
  • ➖ Adds navigation/maintenance overhead
  • ➖ README becomes less self-contained for first-time users

Recommendation: The current curated README rewrite is the right tradeoff for now: it improves scanability (house style) while keeping every technical claim traceable to the repo. Consider auto-generation only if the module list and behavior descriptions start changing frequently.

Files changed (1) +33 / -12

Documentation (1) +33 / -12
README.mdRewrite README to house style with grounded per-module descriptions +33/-12

Rewrite README to house style with grounded per-module descriptions

• Replaces the existing header with a centered house-style block (project name, badges, link row) and adds section separators. Expands the “What this is” section to describe console modules in more concrete terms (wire codec message types, WebCodecs decode constraints, session creation via same-origin gateway). Clarifies run instructions and standardizes licensing/footer text.

README.md

@qodo-code-review

Copy link
Copy Markdown

Qodo Fixer

No findings are available for this PR yet. Findings appear here once Qodo has reviewed the PR.

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found across 1 file

Confidence score: 5/5

  • Automated review surfaced no issues in the provided summaries.
  • No files require special attention.
Architecture diagram
sequenceDiagram
    participant User as Browser User
    participant HTML as console/index.html
    participant UI as Console UI (session.js)
    participant Player as moq-player.js
    participant Wire as moq-wire-browser.js
    participant Gateway as WAVE Gateway (POST /api/v1/crest/session)
    participant Relay as MoQ Relay (wss://moq.wave.online)

    Note over User,Relay: Crest Console — Browser MoQ Player & Console

    User->>HTML: Open http://localhost:8080/?relay=&ns=&track=
    HTML->>HTML: Parse query params (?relay, ?ns, ?track)
    alt No relay param
        Note over HTML: Default relay: wss://moq.wave.online
    end

    HTML->>UI: Init console with params

    Note over UI,Relay: Step 1 — Open Crest Ingest Session

    UI->>Gateway: POST /api/v1/crest/session (same-origin proxy)
    Gateway-->>UI: Return session (namespace + track)

    UI->>UI: Store returned namespace/track

    Note over UI,Relay: Step 2 — Subscribe & Decode Track

    UI->>Player: subscribeToTrack(namespace, track, relayUrl)

    Player->>Relay: WebSocket connect (wss://moq.wave.online)

    Player->>Wire: Port MoQ draft-18 wire messages (SETUP, SUBSCRIBE, PUBLISH, FETCH, NAMESPACE)
    Wire-->>Player: Encode/decode message buffers (Uint8Array)

    Player->>Relay: Subscribe to track via MoQ protocol
    Relay-->>Player: H.264 access units over WebSocket

    Player->>Player: Reassemble H.264 Annex-B access units

    alt Chrome/Chromium with proprietary codecs
        Player->>Player: WebCodecs VideoDecoder API decode
        Player-->>HTML: Draw frames to <canvas>
    else Other browsers (no H.264 decode)
        Note over Player,HTML: H.264 decode unsupported — no video output
    end

    HTML-->>User: Display decoded video in canvas
Loading

Re-trigger cubic

Approval not submitted

This repository is configured to approve as @yakimoto, but that approval identity is unavailable.
GitHub does not allow a pull request author approval to count for their own PR.

cubic left this as a normal review comment instead of falling back to a Cubic App approval.

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.

1 participant