Skip to content

Add Product → Hero 3D playbook landing page at /hero-3d-playbook - #86

Open
salinsug wants to merge 1 commit into
mainfrom
hero-3d-playbook
Open

Add Product → Hero 3D playbook landing page at /hero-3d-playbook#86
salinsug wants to merge 1 commit into
mainfrom
hero-3d-playbook

Conversation

@salinsug

@salinsug salinsug commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

What

Adds a standalone Product Photo → Hero 3D playbook page for the Livepeer Agent, served at /hero-3d-playbook.

Approach

  • Self-contained HTML bundle at public/hero-3d-playbook.html (assets are inlined and decoded to blob: URLs at runtime — no external asset files needed).
  • Kept outside the app/ design system intentionally — off-brand, standalone playbook page.
  • A rewrite in next.config.ts maps the clean /hero-3d-playbook route to the file.

Same pattern as the /agent-alpha (#84) and /consistency-cheatsheet (#85) pages.

Verification

Rendered locally on the dev server at /hero-3d-playbook — title, heading, stat tiles, and all 4 images load; no console errors.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Added support for accessing the 3D playbook through the /hero-3d-playbook URL.
    • The page now loads correctly from its static HTML content.

Self-contained HTML bundle served from public/, kept outside the app/
design system. Rewrite maps the clean /hero-3d-playbook route to the
file. Assets are inlined and decoded to blob URLs at runtime.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@salinsug
salinsug requested a review from adamsoffer as a code owner July 22, 2026 18:53
@vercel

vercel Bot commented Jul 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
website Ready Ready Preview, Comment Jul 22, 2026 6:54pm

Request Review

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

Next.js configuration now rewrites /hero-3d-playbook to /hero-3d-playbook.html from the public/ directory.

Changes

Hero 3D Playbook Route

Layer / File(s) Summary
Add hero 3D playbook rewrite
next.config.ts
Adds an asynchronous rewrite mapping /hero-3d-playbook to /hero-3d-playbook.html.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: adamsoffer

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: adding the Hero 3D playbook landing page at /hero-3d-playbook.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch hero-3d-playbook

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

ESLint install timed out. The project may have too many dependencies for the sandbox.


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.

@coderabbitai coderabbitai 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@next.config.ts`:
- Around line 23-27: Relocate the standalone hero 3D playbook HTML artifact from
the root of public into the appropriate permitted public subdirectory, then
update the rewrite destination in the configuration block for source
"/hero-3d-playbook" to match its new location. Do not amend the guideline;
preserve the existing route behavior.
🪄 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: defaults

Review profile: CHILL

Plan: Pro

Run ID: 83613feb-c26e-4259-9754-92294f962498

📥 Commits

Reviewing files that changed from the base of the PR and between 35a669b and 2ae25c0.

📒 Files selected for processing (2)
  • next.config.ts
  • public/hero-3d-playbook.html
👮 Files not reviewed due to content moderation or server errors (1)
  • public/hero-3d-playbook.html

Comment thread next.config.ts
Comment on lines +23 to +27
// Standalone Product → Hero 3D playbook, served as a raw HTML file from
// public/hero-3d-playbook.html (kept outside the app/ design system).
{
source: "/hero-3d-playbook",
destination: "/hero-3d-playbook.html",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win

Resolve the public asset placement guideline violation.

This rewrite targets the root-level public/hero-3d-playbook.html, but the repository requires static assets under public/images/, public/videos/, or public/fonts/. Relocate the artifact and update destination, or explicitly amend the guideline for standalone HTML pages before merging.

As per coding guidelines, public/**/* static assets must be placed in public/images/, public/videos/, or public/fonts/.

🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

In `@next.config.ts` around lines 23 - 27, Relocate the standalone hero 3D
playbook HTML artifact from the root of public into the appropriate permitted
public subdirectory, then update the rewrite destination in the configuration
block for source "/hero-3d-playbook" to match its new location. Do not amend the
guideline; preserve the existing route behavior.

Source: Coding guidelines

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