Skip to content

refactor: use gray-matter for parsing frontmatter#1168

Merged
danielroe merged 2 commits intomainfrom
refactor/graymatter
Feb 7, 2026
Merged

refactor: use gray-matter for parsing frontmatter#1168
danielroe merged 2 commits intomainfrom
refactor/graymatter

Conversation

@danielroe
Copy link
Copy Markdown
Member

No description provided.

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 7, 2026

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

Project Deployment Actions Updated (UTC)
npmx.dev Ready Ready Preview, Comment Feb 7, 2026 8:01pm
npmx.dev-vercel Ready Ready Preview, Comment Feb 7, 2026 8:01pm
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
docs.npmx.dev Ignored Ignored Preview Feb 7, 2026 8:01pm
npmx-lunaria Ignored Ignored Feb 7, 2026 8:01pm

Request Review

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 7, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 7, 2026

📝 Walkthrough

Walkthrough

This pull request replaces the custom frontmatter parsing in modules/standard-site-sync.ts with the gray-matter library, switching from a readFileSync + parseBasicFrontmatter flow to using gray-matter's read and taking frontmatter from the result's data property. The parseBasicFrontmatter unit test file was removed. The project dependency gray-matter@4.0.3 was added. Other synchronisation logic (normalisation, validation, hashing, downstream PUT) is unchanged.

Possibly related PRs

🚥 Pre-merge checks | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request lacks any description from the author, making it impossible to assess the stated intent or rationale for the changes. Add a pull request description explaining the purpose of replacing the custom frontmatter parser with gray-matter, including benefits and any breaking changes or migration notes.

✏️ 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
  • Commit unit tests in branch refactor/graymatter

No actionable comments were generated in the recent review. 🎉


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

@danielroe danielroe merged commit 24094a1 into main Feb 7, 2026
22 checks passed
@danielroe danielroe deleted the refactor/graymatter branch February 7, 2026 20:16
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.

2 participants