-
Notifications
You must be signed in to change notification settings - Fork 2.5k
feat(web): fill missing SEO-related values #7096
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| import type { MetadataRoute } from "next" | ||
| import { SEO } from "@/lib/seo" | ||
|
|
||
| export default function robots(): MetadataRoute.Robots { | ||
| return { | ||
| rules: { | ||
| userAgent: "*", | ||
| allow: "/", | ||
| }, | ||
| sitemap: `${SEO.url}/sitemap.xml`, | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The robots.ts file references a sitemap at Consider either:
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yes, ideally this PR should wait for the mentioned PR |
||
| host: SEO.url, | ||
| } | ||
| } | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,30 @@ | ||
| const SITE_URL = process.env.NEXT_PUBLIC_SITE_URL ?? "https://roocode.com" | ||
|
|
||
| export const SEO = { | ||
| url: SITE_URL, | ||
| name: "Roo Code", | ||
| title: "Roo Code – Your AI-Powered Dev Team in VS Code", | ||
| description: | ||
| "Roo Code puts an entire AI dev team right in your editor, outpacing closed tools with deep project-wide context, multi-step agentic coding, and unmatched developer-centric flexibility.", | ||
| locale: "en_US", | ||
| ogImage: { | ||
| url: "/android-chrome-512x512.png", | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The ogImage URL uses a relative path. For Open Graph images to work properly across all platforms (especially social media previews), consider using an absolute URL: Alternatively, you could make it dynamic:
Contributor
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. this is already handled by nextjs, it automatically prepends the site url |
||
| width: 512, | ||
| height: 512, | ||
| alt: "Roo Code Logo", | ||
| }, | ||
| keywords: [ | ||
elianiva marked this conversation as resolved.
Show resolved
Hide resolved
|
||
| "Roo Code", | ||
| "AI coding agent", | ||
| "VS Code extension", | ||
| "AI pair programmer", | ||
| "software development", | ||
| "agentic coding", | ||
| "code refactoring", | ||
| "debugging", | ||
| ], | ||
| category: "technology", | ||
| twitterCard: "summary_large_image" as const, | ||
| } as const | ||
|
|
||
| export type SeoConfig = typeof SEO | ||
Uh oh!
There was an error while loading. Please reload this page.