Skip to content

Release/918.0.0#8478

Merged
jeffsmale90 merged 5 commits intomainfrom
release/918.0.0
Apr 16, 2026
Merged

Release/918.0.0#8478
jeffsmale90 merged 5 commits intomainfrom
release/918.0.0

Conversation

@jeffsmale90
Copy link
Copy Markdown
Contributor

@jeffsmale90 jeffsmale90 commented Apr 15, 2026

Explanation

Releases

  • @metamask/gator-permissions-controller@4.0.0
  • @metamask/signature-controller@39.2.0

Breaking change to @metamask-gator-permissions-controller:

  • BREAKING: Add status to PermissionInfoWithMetadata type, resolved from onchain data (#8445)

References

Checklist

  • I've updated the test suite for new or updated code as appropriate
  • I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
  • I've communicated my changes to consumers by updating changelogs for packages I've changed
  • I've introduced breaking changes in this PR and have prepared draft pull requests for clients and consumer packages to resolve them

Note

Medium Risk
Primarily a version/dependency release PR, but it propagates a breaking type change (PermissionInfoWithMetadata.status) via a major bump that can break downstream TypeScript consumers.

Overview
Cuts the 918.0.0 release by bumping the root monorepo version and publishing @metamask/gator-permissions-controller 4.0.0 (noted as BREAKING due to adding a status field to PermissionInfoWithMetadata, resolved from on-chain data).

Updates the dependency chain to consume the new major: @metamask/signature-controller is released as 39.2.0 and now depends on @metamask/gator-permissions-controller@^4.0.0, and @metamask/shield-controller updates to @metamask/signature-controller@^39.2.0; lockfile and changelogs are updated accordingly.

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

@jeffsmale90 jeffsmale90 requested review from a team as code owners April 15, 2026 22:19
Copy link
Copy Markdown
Contributor

@mj-kiwi mj-kiwi left a comment

Choose a reason for hiding this comment

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

Two format issues.

Comment thread packages/shield-controller/CHANGELOG.md Outdated
Comment thread packages/signature-controller/CHANGELOG.md Outdated
- remove errant "
- add comma between list of changes

Co-authored-by: MJ Kiwi <me@mj.kiwi>
@jeffsmale90 jeffsmale90 enabled auto-merge April 16, 2026 04:04
### Changed

- Bump `@metamask/gator-permissions-controller` from `^0.6.0` to `^0.8.0` ([#7274](https://github.com/MetaMask/core/pull/7274)), ([#7330](https://github.com/MetaMask/core/pull/7330))
- Bump `@metamask/gator-permissions-controller` from `^0.6.0` to `^0.8.0`, ([#7274](https://github.com/MetaMask/core/pull/7274), [#7330](https://github.com/MetaMask/core/pull/7330))
Copy link
Copy Markdown
Member

@OGPoyraz OGPoyraz Apr 16, 2026

Choose a reason for hiding this comment

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

Why did we adjusted previous release changelogs here and below? Are these really corrections or unintended changes?

Copy link
Copy Markdown
Contributor Author

@jeffsmale90 jeffsmale90 Apr 16, 2026

Choose a reason for hiding this comment

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

That is strange - I had a look, and it's the release branch automation resolving an issue with the original changelog (each change contributing to the line was wrapped in individual () rather than the entire list). We could revert it, but the next time we create a release it's going to do the same thing.

I repeated the release branch creation and it fixed it again. I think we will see these on a number of CHANGELOG files.

Copy link
Copy Markdown
Member

@OGPoyraz OGPoyraz Apr 16, 2026

Choose a reason for hiding this comment

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

This might be because of new linter.

My only concern was

- **BREAKING:** Bump `@metamask/keyring-controller` peer dependency from `^18.0.0` to `^19.0.0` ([#4195](https://github.com/MetaMask/core/pull/4195))

line because it changes PR number.

@jeffsmale90 jeffsmale90 added this pull request to the merge queue Apr 16, 2026
Merged via the queue into main with commit b39384c Apr 16, 2026
345 checks passed
@jeffsmale90 jeffsmale90 deleted the release/918.0.0 branch April 16, 2026 07:54
geositta added a commit that referenced this pull request Apr 16, 2026
…sition-live-price-sync

* origin/main:
  fix(transaction-pay-controller): resolve correct networkClientId for source chain in relay execute (#8492)
  Release/920.0.0 (#8494)
  chore: bump `accounts` deps (#8464)
  feat: adds auth to social controllers (#8485)
  Release/919.0.0 (#8482)
  feat(seedless-onboarding-controller): generic encryptor result type aligned with KeyringController TO-686 (#8411)
  Release/918.0.0 (#8478)
  chore: add periodic check for spl tokens (#8400)
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.

4 participants