Add isolated landing page in site/ - #59
Conversation
Co-authored-by: prath <prathamdby@users.noreply.github.com>
PR Agent ReviewNote Adds a new marketing landing page site/ for PR Agent, built with React 19, Vite, Tailwind CSS v4, and Motion. The site includes sections for hero, integrations, why, capabilities, how it works, self-hosting, social proof, FAQ, and a CTA footer. It is fully isolated from the main application with its own package.json, dependencies, and build configuration.
|
There was a problem hiding this comment.
Note
Full review is in the PR conversation. Expand below to copy fixes for your coding agent.
Fix all findings (agent prompt)
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate.
Repository: prathamdby/pr-agent
Pull request: #59
Head SHA: a7adb4d9dac3c43c6a4edd60bc36310e12683611
Findings:
[P1] @site/src/content.ts line 10
Update `GITHUB_URL` to the actual repository URL: `"https://github.com/prathamdby/pr-agent"`.
| { label: "FAQ", href: "#faq" }, | ||
| ] as const; | ||
|
|
||
| export const GITHUB_URL = "https://github.com/"; |
There was a problem hiding this comment.
P1 · GITHUB_URL points to github.com homepage, not the actual repo
site/src/content.ts · line 10
The exported GITHUB_URL constant is set to "https://github.com/" — the generic GitHub homepage. This value is used in Nav.tsx (line 46), Hero.tsx (line 68), and CtaFooter.tsx (lines 38, 44, 73) for all "View on GitHub" and "Get started" CTAs. Every external link directing users to the project will land on github.com instead of the actual repository (https://github.com/prathamdby/pr-agent). This makes the primary call-to-action across the entire marketing site non-functional.
Prompt to fix
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate.
Repository: prathamdby/pr-agent
Pull request: #59
Head SHA: a7adb4d9dac3c43c6a4edd60bc36310e12683611
[P1] @site/src/content.ts line 10
Update `GITHUB_URL` to the actual repository URL: `"https://github.com/prathamdby/pr-agent"`.
Co-authored-by: prath <prathamdby@users.noreply.github.com>
There was a problem hiding this comment.
Fix all findings (agent prompt)
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate.
Repository: prathamdby/pr-agent
Pull request: #59
Head SHA: 1c72d9e3dd6d4c43762fa7efd2062a282967b0ed
Findings:
[P1] @site/src/content.ts line 3
Change `GITHUB_URL` to `"https://github.com/prathamdby/pr-agent"` so that all GitHub links and CTA buttons point to the actual repository.
[P2] @site/src/components/demos/Pipeline.tsx line 60
Move the `last:hidden` class to the parent `<div className="relative flex gap-4 md:block">` instead, or conditionally render the connector only for non-last items using the array index.
| @@ -0,0 +1,103 @@ | |||
| import type { BrandName } from "./components/integrations.types"; | |||
|
|
|||
| export const GITHUB_URL = "https://github.com/"; | |||
There was a problem hiding this comment.
P1 · GITHUB_URL points to GitHub homepage, not the PR Agent repo
site/src/content.ts · line 3
The constant GITHUB_URL is set to "https://github.com/" instead of the actual repository URL https://github.com/prathamdby/pr-agent. This value is used in 6 link locations across three components: Nav (line 48), Hero (line 60), and CtaFooter (lines 32, 39, 69) plus the footer GitHub link. Every "GitHub", "View on GitHub", and "Get started" button on the site will navigate to github.com's homepage rather than the PR Agent repository. The root README and ADRs consistently reference github.com/prathamdby/pr-agent.
Prompt to fix
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate.
Repository: prathamdby/pr-agent
Pull request: #59
Head SHA: 1c72d9e3dd6d4c43762fa7efd2062a282967b0ed
[P1] @site/src/content.ts line 3
Change `GITHUB_URL` to `"https://github.com/prathamdby/pr-agent"` so that all GitHub links and CTA buttons point to the actual repository.
| <div | ||
| aria-hidden="true" | ||
| className="absolute left-[27px] top-14 h-[calc(100%-1rem)] w-px bg-line last:hidden md:hidden" | ||
| /> |
There was a problem hiding this comment.
P2 · last:hidden never matches for vertical Pipeline connector line
site/src/components/demos/Pipeline.tsx · line 60
The vertical connector <div> on line 60 uses last:hidden to hide the line on the last pipeline stage in mobile view. However, this <div> is the first child of its parent <div className="relative flex gap-4 md:block"> — there are sibling elements after it. The CSS :last-child pseudo-class only matches when the element is the last child of its parent, so last:hidden never activates. A dangling connector line will render below the final pipeline stage on mobile.
Prompt to fix
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate.
Repository: prathamdby/pr-agent
Pull request: #59
Head SHA: 1c72d9e3dd6d4c43762fa7efd2062a282967b0ed
[P2] @site/src/components/demos/Pipeline.tsx line 60
Move the `last:hidden` class to the parent `<div className="relative flex gap-4 md:block">` instead, or conditionally render the connector only for non-last items using the array index.
Summary
Adds a single-page marketing landing page for PR Agent, fully isolated inside a new
site/directory. It has its own dependencies, build, and config, and imports nothing from the parent application.The page explains the project for non-technical readers and beginners: what PR Agent does, how it works, why it is self-hosted, and answers to common questions. The design is dark, minimal, and detail-focused, with restrained Motion animation.
Tech stack
@tailwindcss/vite)site/src/components/ui(button, badge, card, accordion)motion/react) for entrance, stagger, the sliding showcase pill, and crossfadessimple-iconsbrand marksSections
Hero, integration wall, the problem, features bento, a tabbed "see it in action" showcase, how it works, self-hosting and privacy, social proof (stats + testimonials), FAQ, and a closing call to action.
Notes
site/public/shotsare copied fromdocs/readme/assets, so the page shows the real GitHub surfaces rather than mockups.Run it
cd site npm install npm run devPR Agent Description
PR Type
Enhancement
Description
site/with its own dependencies and buildChanges Diagram
File Walkthrough
Enhancement (20 files)
Site package manifest and dependencies
site/package.jsonHTML entry point with SEO metadata
site/index.htmlRoot App component composing all sections
site/src/App.tsxSticky navigation with scroll-aware styling
site/src/components/sections/Nav.tsxHero section with animated intro and screenshot frame
site/src/components/sections/Hero.tsxFeatures grid with staggered reveal and slash commands
site/src/components/sections/Features.tsxTabbed screenshot showcase with animated transitions
site/src/components/sections/Showcase.tsxFAQ accordion section
site/src/components/sections/Faq.tsxSelf-hosting section with guarantees and terminal preview
site/src/components/sections/SelfHosted.tsxBrowser-frame wrapper for product screenshots
site/src/components/Frame.tsxBrand logo SVG renderer from simple-icons
site/src/components/BrandLogo.tsxReusable scroll-triggered reveal animations
site/src/components/motion/Reveal.tsxCentralized marketing copy and data
site/src/content.tsTailwind v4 design tokens and global styles
site/src/index.cssButton component with CVA variants
site/src/components/ui/button.tsxAccordion UI primitive
site/src/components/ui/accordion.tsxPR Agent brand mark SVG
site/src/components/Mark.tsxVite configuration with React and Tailwind plugins
site/vite.config.tsSite-specific gitignore
site/.gitignorecn utility for class merging
site/src/lib/utils.ts