-
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your contribution! I've reviewed the SEO improvements and they look good overall. The centralized SEO configuration is a great approach. I've left some suggestions inline to help improve the implementation, particularly around the missing sitemap and Open Graph image URL.
| userAgent: "*", | ||
| allow: "/", | ||
| }, | ||
| sitemap: `${SEO.url}/sitemap.xml`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The robots.ts file references a sitemap at ${SEO.url}/sitemap.xml, but there's no sitemap.ts file included in this PR. This could cause issues as search engines will look for the sitemap but won't find it.
Consider either:
- Adding a sitemap.ts file in this PR
- Coordinating with PR feat: implement sitemap generation in TypeScript and remove XML file #6206 which implements sitemap generation
- Temporarily removing the sitemap reference until it's implemented
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yes, ideally this PR should wait for the mentioned PR
| "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", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The 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:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is already handled by nextjs, it automatically prepends the site url
daniel-lxs
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! Looks good
* main: (70 commits) fix: use native Ollama API instead of OpenAI compatibility layer (RooCodeInc#7137) feat: add support for OpenAI gpt-5-chat-latest model (RooCodeInc#7058) Make enhance with task history default to true (RooCodeInc#7140) Bump cloud version to 0.16.0 (RooCodeInc#7135) Release: v1.51.0 (RooCodeInc#7130) Add an API for resuming tasks by ID (RooCodeInc#7122) Add support for task page event population (RooCodeInc#7117) fix: add type check before calling .match() on diffItem.content (RooCodeInc#6905) (RooCodeInc#6906) Fix: Enable save button for provider dropdown and checkbox changes (RooCodeInc#7113) fix: Use cline.cwd as primary source for workspace path in codebaseSearchTool (RooCodeInc#6902) Hotfix multiple folder workspace checkpoint (RooCodeInc#6903) fix: prevent XML entity decoding in diff tools (RooCodeInc#7107) (RooCodeInc#7108) Refactor task execution system: improve call stack management (RooCodeInc#7035) Changeset version bump (RooCodeInc#7104) feat(web): fill missing SEO-related values (RooCodeInc#7096) Update contributors list (RooCodeInc#6883) Release v3.25.15 (RooCodeInc#7103) fix: add /evals page to sitemap generation (RooCodeInc#7102) feat: implement sitemap generation in TypeScript and remove XML file (RooCodeInc#6206) fix: reset condensing state when switching tasks (RooCodeInc#6922) ...
Related GitHub Issue
Partially Closes: #5231
This should also help with #6206
If this gets approved I will also continue doing the same for https://github.com/RooCodeInc/Roo-Code-Docs/ since it seems to be suffering from the same issue.
Roo Code Task Context (Optional)
https://app.roocode.com/share/8821ffaa-d861-4a05-b31f-df92ab49d0ed
Description
This PR adds necessary SEO values needed to help with website indexing. This should also add a nice preview when people share RooCode's website making it look more legitimate. Currently it's missing a lot of values.
Test Procedure
You can check it by using a tool called
check-site-meta. Runnpx check-site-metaand see the updated version.Pre-Submission Checklist
Screenshots / Videos
Here's a screenshot comparing before and after to save you some time.
BEFORE

AFTER

Documentation Updates
Additional Notes
If it's really needed I could also probably add dynamic og image generation using something like https://github.com/vercel/satori so it looks better when people share the link, especially for the docs.
Get in Touch
@elianiva on Discord
Important
Enhances SEO metadata across multiple pages using a centralized
SEOconfiguration, improving website indexing and sharing previews.SEOconfiguration inseo.tsfor centralized metadata management.metadatainenterprise/page.tsx,evals/page.tsx,privacy/page.tsx, andterms/page.tsxto useSEOvalues for title, description, openGraph, and twitter metadata.robots.tsto define robots.txt rules and sitemap location usingSEO.url.layout.tsxto useSEOfor metadata, ensuring consistent site-wide SEO settings.This description was created by
for 9b2c1da. You can customize this summary. It will automatically update as commits are pushed.