Skip to content

feat: Add option to use Oxfmt for formatting changelog#198

Merged
Mrtenz merged 4 commits intomainfrom
mrtenz/oxfmt-changelog
Apr 15, 2026
Merged

feat: Add option to use Oxfmt for formatting changelog#198
Mrtenz merged 4 commits intomainfrom
mrtenz/oxfmt-changelog

Conversation

@Mrtenz
Copy link
Copy Markdown
Member

@Mrtenz Mrtenz commented Apr 14, 2026

This adds a new optional --formatter option to the CLI, which can be set to either "oxfmt" or "prettier" (default). This allows the generated changelog to be formatted with Oxfmt.


Note

Medium Risk
Changes the release workflow to thread a new formatter parameter through changelog generation/execution paths and upgrades @metamask/auto-changelog, which could alter changelog output or release-branch automation behavior.

Overview
Adds a new --formatter CLI option (default prettier) to control how changelogs are formatted, supporting both prettier and oxfmt across the CLI and interactive UI workflows.

Threads this formatter selection through determineInitialParameters, followMonorepoWorkflow/UI execution, and executeReleasePlan/package update helpers, replacing the previous hardcoded Prettier formatting with a getFormatter switch.

Updates dependencies to @metamask/auto-changelog@^6.1.0, adds optional peer deps for oxfmt/prettier, and adjusts tests (including mocking dynamic imports) plus a new Oxfmt-specific changelog expectation.

Reviewed by Cursor Bugbot for commit 8624d54. Bugbot is set up for automated code reviews on this repo. Configure here.

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Apr 14, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​metamask/​auto-changelog@​4.0.0 ⏵ 6.1.096 -2100100 +194 +8100

View full report

@socket-security
Copy link
Copy Markdown

socket-security Bot commented Apr 14, 2026

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

Ignoring alerts on:

  • @octokit/request@8.4.1
  • @metamask/auto-changelog@6.1.0

View full report

@Mrtenz
Copy link
Copy Markdown
Member Author

Mrtenz commented Apr 14, 2026

@SocketSecurity ignore npm/@metamask/auto-changelog@6.1.0
@SocketSecurity ignore npm/@octokit/request@8.4.1

Network access used to talk to the GitHub API.

@Mrtenz Mrtenz marked this pull request as ready for review April 14, 2026 11:45
@Mrtenz Mrtenz requested a review from a team as a code owner April 14, 2026 11:45
Comment thread src/project.ts
Comment thread src/main.ts
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 381fa8b. Configure here.

Comment thread src/project.test.ts
@Mrtenz
Copy link
Copy Markdown
Member Author

Mrtenz commented Apr 14, 2026

@metamaskbot publish-preview

Copy link
Copy Markdown
Contributor

@mcmire mcmire left a comment

Choose a reason for hiding this comment

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

LGTM!

@Mrtenz Mrtenz merged commit 54fc385 into main Apr 15, 2026
16 checks passed
@Mrtenz Mrtenz deleted the mrtenz/oxfmt-changelog branch April 15, 2026 14:37
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