Skip to content

chore: replace js-yaml with yaml#7684

Merged
martinjagodic merged 2 commits intomainfrom
replace-js-yaml-with-yaml
Dec 17, 2025
Merged

chore: replace js-yaml with yaml#7684
martinjagodic merged 2 commits intomainfrom
replace-js-yaml-with-yaml

Conversation

@martinjagodic
Copy link
Member

Summary

This PR consolidates YAML parsing by replacing js-yaml with yaml (eemeli's library) across the codebase.

Motivation

  • We were maintaining two YAML libraries (js-yaml and yaml) with overlapping functionality
  • The yaml library is already used in core frontmatter parsing and provides richer features (better TypeScript support, parseDocument API for enhanced error handling, comment/anchor support)
  • Reduces bundle size and dependency surface area
  • Improves maintainability by having a single YAML implementation

Test plan

  • All existing tests pass

Checklist

Please add a x inside each checkbox:

A picture of a cute animal (not mandatory but encouraged)
🦙

@martinjagodic martinjagodic requested a review from a team as a code owner December 17, 2025 10:29
@github-actions github-actions bot added the type: chore work needed to keep the product and development running smoothly label Dec 17, 2025
@martinjagodic martinjagodic merged commit 809ab24 into main Dec 17, 2025
14 checks passed
@martinjagodic martinjagodic deleted the replace-js-yaml-with-yaml branch December 17, 2025 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: chore work needed to keep the product and development running smoothly

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants