[codex] Add site sharing metadata#1
Conversation
📝 WalkthroughWalkthroughAdds SEO and discoverability metadata to the site: ChangesSEO & Discoverability Setup
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Note Unit test generation is a public access feature. Expect some limitations and changes as we gather feedback and continue to improve it. Generating unit tests... This may take up to 20 minutes. |
There was a problem hiding this comment.
Pull request overview
Adds SEO and social sharing/discovery metadata for radical.codes so links unfurl correctly and crawlers can discover the site’s canonical URL and basic indexing resources.
Changes:
- Added canonical + author metadata, plus Open Graph and Twitter Card tags to
index.html. - Added a sitemap and robots policy under
public/for crawler discovery. - Added a web app manifest and social preview assets (
og-image.svgandog-image.png) underpublic/.
Reviewed changes
Copilot reviewed 4 out of 6 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
index.html |
Adds canonical URL, author, Open Graph, Twitter Card tags, and links the web manifest. |
public/sitemap.xml |
Introduces a basic sitemap for the site root URL. |
public/robots.txt |
Allows crawling and points crawlers at the sitemap. |
public/site.webmanifest |
Adds a basic web manifest for the site. |
public/og-image.svg |
Adds an accessible SVG source for the social preview image. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
✅ Created PR with unit tests: #10 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
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 `@public/site.webmanifest`:
- Around line 1-9: The web app manifest in site.webmanifest is missing the
required `icons` array property, which prevents proper browser installation
prompts and icon rendering. Add an `icons` array property to the manifest JSON
object containing one or more icon definitions, where each icon object specifies
a `src` path to an icon image file, the `sizes` it represents (such as 192x192
or 512x512), and the `type` indicating the image format (such as image/png).
This completes the manifest contract and enables browsers to display the
appropriate icons during installation and app launch.
🪄 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 Plus
Run ID: dbc0f12c-3a3a-43c1-939c-13969eab9650
⛔ Files ignored due to path filters (2)
public/og-image.pngis excluded by!**/*.pngpublic/og-image.svgis excluded by!**/*.svg
📒 Files selected for processing (4)
index.htmlpublic/robots.txtpublic/site.webmanifestpublic/sitemap.xml
| { | ||
| "name": "radical.codes", | ||
| "short_name": "radical.codes", | ||
| "description": "Ryan Gonyon's public proof surface for app rescue, production readiness, workflow automation, and dependable software systems.", | ||
| "start_url": "/", | ||
| "display": "minimal-ui", | ||
| "background_color": "#f8f8f4", | ||
| "theme_color": "#000000" | ||
| } |
There was a problem hiding this comment.
Add icons to complete the web app manifest contract.
On Line 1-9, the manifest omits the icons array. That leaves install identity incomplete and can prevent proper install prompts/icon rendering in browsers.
Proposed patch
{
"name": "radical.codes",
"short_name": "radical.codes",
"description": "Ryan Gonyon's public proof surface for app rescue, production readiness, workflow automation, and dependable software systems.",
"start_url": "/",
"display": "minimal-ui",
+ "icons": [
+ {
+ "src": "/icon-192.png",
+ "sizes": "192x192",
+ "type": "image/png"
+ },
+ {
+ "src": "/icon-512.png",
+ "sizes": "512x512",
+ "type": "image/png"
+ }
+ ],
"background_color": "`#f8f8f4`",
"theme_color": "`#000000`"
}📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| { | |
| "name": "radical.codes", | |
| "short_name": "radical.codes", | |
| "description": "Ryan Gonyon's public proof surface for app rescue, production readiness, workflow automation, and dependable software systems.", | |
| "start_url": "/", | |
| "display": "minimal-ui", | |
| "background_color": "#f8f8f4", | |
| "theme_color": "#000000" | |
| } | |
| { | |
| "name": "radical.codes", | |
| "short_name": "radical.codes", | |
| "description": "Ryan Gonyon's public proof surface for app rescue, production readiness, workflow automation, and dependable software systems.", | |
| "start_url": "/", | |
| "display": "minimal-ui", | |
| "icons": [ | |
| { | |
| "src": "/icon-192.png", | |
| "sizes": "192x192", | |
| "type": "image/png" | |
| }, | |
| { | |
| "src": "/icon-512.png", | |
| "sizes": "512x512", | |
| "type": "image/png" | |
| } | |
| ], | |
| "background_color": "`#f8f8f4`", | |
| "theme_color": "`#000000`" | |
| } |
🤖 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 `@public/site.webmanifest` around lines 1 - 9, The web app manifest in
site.webmanifest is missing the required `icons` array property, which prevents
proper browser installation prompts and icon rendering. Add an `icons` array
property to the manifest JSON object containing one or more icon definitions,
where each icon object specifies a `src` path to an icon image file, the `sizes`
it represents (such as 192x192 or 512x512), and the `type` indicating the image
format (such as image/png). This completes the manifest contract and enables
browsers to display the appropriate icons during installation and app launch.
|
Closing as superseded by #14. The durable sharing/discovery pieces from this PR have been adapted into the survivor branch with the current two-page site direction: manifest, robots, sitemap, canonical/social metadata, and JSON-LD. Keeping this closed avoids merging the older index.html state back over the new site. |
What changed
Adds public sharing/discovery metadata for radical.codes:
Why
The site is becoming the central public hub for Ryan's GitHub, Upwork, and proposal profile. Links should unfurl with the same positioning the site already leads with: app rescue, production readiness, workflow automation, and dependable software.
Validation
npm run buildpublic/intodist/Summary by CodeRabbit