Skip to content

fix(typescript): parse CRLF changesets in release tooling - #2528

Open
wtxadmin-oss wants to merge 1 commit into
mcp-use:mainfrom
wtxadmin-oss:fix/typescript/release-changeset-crlf
Open

wtxadmin-oss wants to merge 1 commit into
mcp-use:mainfrom
wtxadmin-oss:fix/typescript/release-changeset-crlf

Conversation

@wtxadmin-oss

@wtxadmin-oss wtxadmin-oss commented Sep 12, 2026

Copy link
Copy Markdown

Summary

  • normalize CRLF line endings before parsing changeset frontmatter
  • exercise internal peer-range preparation with both LF and CRLF changesets
  • add a regression test proving CRLF changesets participate in Canary baseline preflight
  • include the required TypeScript changeset

The change stays at the local parsing boundary and does not alter release publication or workflow gates.

Validation

  • pnpm test:release-channel — 25 passed
  • pnpm exec prettier --check scripts/release-channel.mjs scripts/release-channel.test.mjs .changeset/calm-crabs-handle-crlf.md
  • pnpm exec eslint scripts/release-channel.mjs scripts/release-channel.test.mjs
  • pre-commit checks passed

Closes #2527


Summary by cubic

Fixes release tooling so it recognizes changesets with CRLF line endings, which previously caused Windows-authored changes to be missed during Canary baseline checks.

  • Normalizes CRLF to LF before parsing changeset frontmatter in release-channel.mjs.
  • Adds tests covering internal peer-range prep and Canary baseline preflight for both LF and CRLF changesets.
  • Includes a patch changeset for mcp-use.

Closes #2527.

Written for commit cc92e78. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 3 files

Re-trigger cubic

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Release preparation silently ignores changesets with CRLF line endings

1 participant