web: Agent Landing Page A/B testing toolkit

[brunobergher]

To make it easier for us to iterate on the copy of landing pages for different agents, this PR generalizes the structure of the page and moves copy to content-only files which can be easily edited and referenced as variants (A, B, C, etc), falling back to A (which is always required) when an invalid value is passed (or its absent).

To try it out, go to /reviewer?v=A and /reviewer?v=B in the preview environment.
To understand how it's used, look into apps/web-roo-code/src/app/reviewer/page.tsx

This also tweaks the copy of Danny's proposed alternative for the Reviewer page, but mostly leaves it alone.

---

Important

Introduces A/B testing for agent landing pages with dynamic content selection and refactors content structure for consistency.

• A/B Testing Framework:
  • Introduces getContentVariant() in getContentVariant.ts to select content variants based on URL parameters.
  • Supports fallback to variant 'A' if no valid variant is specified.
• Content Structure:
  • Defines AgentPageContent interface in agent-page-content.ts for consistent content structure across agent pages.
  • Adds content-a.tsx, content-b.tsx, and content.ts for PR Fixer and PR Reviewer with different content variants.
• Page Components:
  • Updates page.tsx for pr-fixer and reviewer to use AgentLandingContent and dynamic content selection.
  • Removes ReviewerContent.tsx and PrFixerContent.tsx, consolidating into AgentLandingContent.tsx.
• UI Enhancements:
  • Refactors AgentLandingContent.tsx to dynamically render content based on AgentPageContent.
  • Updates agent-carousel.tsx to include links to PR Reviewer and PR Fixer pages.

^{This description was created by for a5c3571. You can customize this summary. It will automatically update as commits are pushed.}

[roomote]

Re-review complete. All previously flagged issues have been resolved.

• Fix unsafe non-null assertion on optional image properties in AgentLandingContent.tsx

Previous Reviews

• a5c3571: Review #1

_{Mention @roomote in a comment to trigger your PR Fixer agent and make changes to this pull request.}

[github-actions]

🚀 Preview deployed!

Your changes have been deployed to Vercel:

Preview URL: https://roo-code-website-mn8k0w2n7-roo-code.vercel.app

This preview will be updated automatically when you push new commits to this PR.

[roomote]

The non-null assertion on content.hero.image.url! is unsafe because both src and url are optional in the AgentPageContent interface. If a content variant is created with an image object that lacks both properties, this will pass undefined to the Next.js Image component and cause a runtime error. Consider either making at least one property required in the type definition or adding runtime validation.

_{Fix it with Roo Code or mention @roomote and request a fix.}

[brunobergher]

@roomote fix this by making the hero image url required, and consolidating all uses to use that property (not src)

[roomote]

Fixed the unsafe non-null assertion by making hero.image.url required and consolidating all uses. All local checks passed.

_{View commit | Revert commit}