Skip to content

Conversation

@RYGRIT
Copy link
Contributor

@RYGRIT RYGRIT commented Feb 3, 2026

fixes: #832

https://cards-dev.x.com/validator prompt

WARN: The image URL https://npmx.dev/__og-image__/static/og.png specified by the 'twitter:image:src' metatag may be restricted by the site's robots.txt file, which will prevent Twitter from fetching it.

I will add Allow: /__og-image__/* to the robots.txt file. This method didn't fix the problem either; I need help.

@vercel
Copy link

vercel bot commented Feb 3, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs.npmx.dev Error Error Feb 3, 2026 10:47pm
npmx.dev Ready Ready Preview, Comment Feb 3, 2026 10:47pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
npmx-lunaria Ignored Ignored Feb 3, 2026 10:47pm

Request Review

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 3, 2026

📝 Walkthrough

Walkthrough

Adds Open Graph and Twitter metadata mappings to the home page SEO setup: useSeoMeta now accepts ogTitle, twitterTitle, ogDescription and twitterDescription in addition to existing title and description, and sets twitterCard to a large image summary. Separately, public/robots.txt is updated to add an Allow: /__og-image__/* rule before the final Disallow: / entry. No control flow or error-handling changes.

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Linked Issues check ✅ Passed The PR successfully addresses issue #832 objectives by adding explicit OG/Twitter meta tags (ogTitle, twitterTitle, ogDescription, twitterDescription, twitterCard) and allowing OG image crawling via robots.txt.
Out of Scope Changes check ✅ Passed All changes are directly scoped to resolving the robots.txt warning and implementing explicit social card metadata as outlined in issue #832.
Description check ✅ Passed The pull request description directly addresses the changeset by referencing issue #832, explaining the Twitter Cards validator warning about robots.txt restrictions, and describing the solution (adding Allow: /og-image/* to robots.txt and explicit SEO meta tags).

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.


Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov
Copy link

codecov bot commented Feb 3, 2026

Codecov Report

❌ Patch coverage is 0% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
app/pages/index.vue 0.00% 4 Missing ⚠️

📢 Thoughts on this report? Let us know!

Comment on lines +39 to +44
ogTitle: () => $t('seo.home.title'),
twitterTitle: () => $t('seo.home.title'),
twitterCard: 'summary_large_image',
description: () => $t('seo.home.description'),
ogDescription: () => $t('seo.home.description'),
twitterDescription: () => $t('seo.home.description'),
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should probably be done on all pages, but we can merge this to test.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, that's what I was thinking too, and I'll continue to improve it. Since summary_large_image has remained unchanged, should we consider putting it into the nuxt.config.ts app configuration?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It works very well!

image

@danielroe danielroe enabled auto-merge February 3, 2026 22:46
@danielroe danielroe added this pull request to the merge queue Feb 3, 2026
Merged via the queue into npmx-dev:main with commit dcd81b0 Feb 3, 2026
14 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix: faild to render og on twitter (recently X)

2 participants