Skip to content

fix: align markdown metadata base URLs#225

Merged
Kinfe123 merged 2 commits into
mainfrom
fix/fix-markdown-frontmatter-review
May 27, 2026
Merged

fix: align markdown metadata base URLs#225
Kinfe123 merged 2 commits into
mainfrom
fix/fix-markdown-frontmatter-review

Conversation

@Kinfe123
Copy link
Copy Markdown
Member

@Kinfe123 Kinfe123 commented May 27, 2026

  • fix: align markdown metadata base urls
  • fix: markdown frontmatter reviews

Summary by cubic

Fixes markdown frontmatter and aligns metadata base URLs so canonical and markdown URLs are consistent across adapters and respect configured base URLs.

  • Bug Fixes

    • Use resolveDocsMetadataBaseUrl for markdown origin in @farming-labs/astro, @farming-labs/nuxt, @farming-labs/svelte, @farming-labs/tanstack-start, and the Docs API.
    • Frontmatter now uses toYamlString; removes the local helper and fixes quoting for title, description, canonical_url, markdown_url, and last_updated.
    • Adds tests to confirm sitemap.baseUrl drives canonical Link headers and frontmatter URLs.
  • Dependencies

    • Bump example apps to 0.1.132 for @farming-labs/*.

Written for commit df94ed2. Summary will update on new commits. Review in cubic

@vercel
Copy link
Copy Markdown

vercel Bot commented May 27, 2026

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

Project Deployment Actions Updated (UTC)
docs-website Building Building Preview, Comment May 27, 2026 12:39pm

Copy link
Copy Markdown

@greptile-apps greptile-apps Bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@Kinfe123 Kinfe123 changed the title fix/fix markdown frontmatter review fix: align markdown metadata base URLs May 27, 2026
@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 27, 2026

Open in StackBlitz

@farming-labs/astro

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/astro@225

@farming-labs/astro-theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/astro-theme@225

@farming-labs/docs

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/docs@225

@farming-labs/theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/theme@225

@farming-labs/next

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/next@225

@farming-labs/nuxt

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/nuxt@225

@farming-labs/nuxt-theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/nuxt-theme@225

@farming-labs/svelte

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/svelte@225

@farming-labs/svelte-theme

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/svelte-theme@225

@farming-labs/tanstack-start

pnpm add https://pkg.pr.new/farming-labs/docs/@farming-labs/tanstack-start@225

commit: df94ed2

@Kinfe123 Kinfe123 merged commit 3fdcc20 into main May 27, 2026
9 of 10 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.

1 participant