Skip to content

Preserve RelayOrb card metadata per route#32

Open
khalidsaidi wants to merge 3 commits into
mainfrom
feat/rag-reliability-framing
Open

Preserve RelayOrb card metadata per route#32
khalidsaidi wants to merge 3 commits into
mainfrom
feat/rag-reliability-framing

Conversation

@khalidsaidi

Copy link
Copy Markdown
Owner

Summary

  • preserve summary_large_image and twitter:image on per-route metadata overrides
  • remove the unnecessary homepage metadata override so the root default stays authoritative
  • strengthen the production metadata verifier to assert twitter:card and twitter:image too

Testing

  • pnpm build

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: cbca1dd84a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

{
route: "/",
title: "RelayOrb — Tool Control Plane for AI Agents",
ogTitle: "RelayOrb — Tool Control Plane for AI Agents",

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Keep homepage OG title in sync

This expected value makes the new production verification step fail for /: the root page inherits openGraph.title and twitter.title from site/src/app/layout.tsx, and those are still RelayOrb - Tool Control Plane for AI Agents with a hyphen while the verifier asserts the em-dash title here. Because .github/workflows/deploy-site.yml now runs this script after every production deploy, site deployments will report failure even though the generated metadata matches the current layout values.

Useful? React with 👍 / 👎.

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.

1 participant