Repository navigation
improvement(library): single JSON-LD source, Markdown articles, and IndexNow pings - #9355
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…cs history fetch outcome in build logs
There was a problem hiding this comment.
All reported issues were addressed across 16 files
Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.
Turn on auto-fix | Re-trigger cubic
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
[Medium impact] The latest changes appear safe to merge; no new blocking issue was established. SummaryAdds Markdown library articles, consolidates structured data into JSON-LD, and submits changed library and comparison URLs to IndexNow after production cutover.
Reviews (5) · Last reviewed commit: "fix(library): noindex direct Markdown ar..." · Reviewed by Greptile |
… renamed pages, and give customer stories speakable hooks
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed
Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.
Turn on auto-fix | Re-trigger cubic
…ce its clone has no origin remote
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
All reported issues were addressed across 19 files
Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.
Requires human review: Auto-approval blocked because this review re-detected 1 unresolved issue already reported by Cubic.
Turn on auto-fix | Re-trigger cubic
…loyments and diff IndexNow from the last submitted revision
|
@cubic-dev-ai review this PR |
@waleedlatif1 I have started the AI code review. It will take a few minutes to complete. |
There was a problem hiding this comment.
No issues found across 21 files
Confidence score: 5/5
- Automated review surfaced no issues in the provided summaries.
- No files require special attention.
Heads up: you’re close to your flex budget. Increase your flex budget so reviews don’t pause.
Turn on auto-fix | Re-trigger cubic
Summary
BlogPosting(publisher as plain text) andFAQPagefrom the post layout and FAQ. Markup is unchanged apart from the attributes, an empty<meta>branch, and an empty FAQ wrapper div. Thespeakableselectors now point atdata-speakablehooks instead of the removeditempropsBlogPostinggets@id(<canonical>#article) andisPartOf→ the siteWebSitenode (#website). Authors link to their on-site profile (urlplus a shared@idthat the author page'sPersonnode uses too), and their external profile stays insameAs/library/<slug>.mdand at/library/<slug>forAccept: text/markdown(afterFiles rewrites, so/library/tagsand the feeds are untouched). Responses carryVary: Accept, areno-storelike the HTML page, and send a canonicalLinkheader. Drafts and unknown slugs return a Markdown 404. Post pages advertise<link rel="alternate" type="text/markdown">, and llms.txt notes the.mdconvention (matching docs)buildPostMetadatanow takes the section at every call siteapps/sim/public, andscripts/ping-indexnow.ts, which submits the library and comparison URLs whose source changed between the previous and newmainhead (renamed and deleted posts included, drafts never; every comparison when Sim's own profile changes). A newindexnowjob runs it aftertrigger-promoteconfirms the prod cutover. The job usescontinue-on-errorwith a 5-minute timeout, nothing depends on it, and it skips when nothing changedType of Change
Testing
scripts/ping-indexnow.test.ts(6 cases against a real git repo). Mutation-checked: removing the draft guard, rename handling, sim-profile fan-out, or profile-id lookup each turns a case redmainrelease: 5 library URLs, no request senttest-landing-seo-e2e.ts(CI's seo HTTP group):.mdandAcceptnegotiation, Vary, no-store, canonical Link, alternates,/library/tagsstill HTML, Markdown 404, no microdata,@id/isPartOf/author@idcross-page match, and speakable selectors resolve. The live site fails these checks before this change (verified with curl)content-routing.test.tsx,customers/page.test.tsx,lib/content/*(145 passing),dev-deployment-workflow,desktop-prune-workflowbun run lint, check:comment-hygiene, source-conventions, file-names, site-urls, library-content, client-boundary, boundaries, unused-exports, utils, test-patterns. type-check and check:audits are left to CIChecklist
test-auditauthoring gate)