Skip to content

Revert "Release 1378.0.0" - #11207

Merged
georgeweiler merged 1 commit into
mainfrom
revert/release-1378.0.0
Oct 29, 2026
Merged

georgeweiler merged 1 commit into
mainfrom
revert/release-1378.0.0

Conversation

@georgeweiler

@georgeweiler georgeweiler commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Explanation

Reverts #11205 (Release 1378.0.0), which was merged before its predecessor Release/1377.0.0 had been released. The release numbering assumed 1377 would land first, so this release has to come off main.

The revert restores:

Nothing was published. The main run for the merge failed on an unrelated rate-limit-controller test, so the Publish release job was skipped. No tags were created, and npm still lists @metamask/kyc-controller at 0.13.0.

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

Made with Cursor


Note

Low Risk
Version and changelog-only revert with no code changes; aligns main with unpublished release state.

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

@georgeweiler
georgeweiler requested review from a team as code owners October 29, 2026 20:36
@georgeweiler
georgeweiler deployed to default-branch October 29, 2026 20:37 — with GitHub Actions Active

@mcmire mcmire left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM!

@georgeweiler
georgeweiler added this pull request to the merge queue Oct 29, 2026
Merged via the queue into main with commit 818e359 Oct 29, 2026
50 checks passed
@georgeweiler
georgeweiler deleted the revert/release-1378.0.0 branch October 29, 2026 20:47
@xavier-brochard xavier-brochard mentioned this pull request Oct 30, 2026
1 of 4 tasks
@jiexi jiexi mentioned this pull request Oct 30, 2026
Naz-Ovh pushed a commit to 0x-fork/metamask-core that referenced this pull request Nov 4, 2026
## Explanation

Release **1378.0.0**. This PR bumps `@metamask/social-controllers` from
`3.14.1` to `3.14.2` (patch) to ship
[MetaMask#11209](MetaMask#11209), which passes the
realtime feed item's `authorComment` through from the backend instead of
deriving it from trade comments. The root monorepo version is bumped to
`1378.0.0`, as the release check requires.

`1378.0.0` is the next number after `1377.0.0`
([MetaMask#11203](MetaMask#11203)). An earlier
`1378.0.0` attempt
([MetaMask#11205](MetaMask#11205)) was reverted in
[MetaMask#11207](MetaMask#11207) before anything
was published, so there is no `1378.0.0` tag.

Packages deliberately left out of this release:

- `@metamask/base-controller`, `@metamask/base-data-service`,
`@metamask/controller-utils`, `@metamask/messenger`, and
`@metamask/profile-sync-controller` are marked `intentionally-skip` in
the release spec. `social-controllers` depends on them, but their
unreleased changes are test, tooling, and dev-dependency changes with no
consumer-facing API or behavior change.
- Other packages with unreleased changes on `main` (for example
`@metamask/kyc-controller`) are not part of this release.

## References

- Ships [MetaMask#11209](MetaMask#11209)
- Previous release:
[MetaMask#11203](MetaMask#11203)
- Reverted attempt:
[MetaMask#11205](MetaMask#11205),
[MetaMask#11207](MetaMask#11207)

## Checklist

- [ ] I've updated the test suite for new or updated code as appropriate
(N/A: version bump and changelog only)
- [ ] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate (N/A)
- [x] I've communicated my changes to consumers by [updating changelogs
for packages I've
changed](https://github.com/MetaMask/core/tree/main/docs/processes/updating-changelogs.md)
- [ ] I've introduced [breaking
changes](https://github.com/MetaMask/core/tree/main/docs/processes/breaking-changes.md)
in this PR and have prepared draft pull requests for clients and
consumer packages to resolve them (N/A: no breaking changes)

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> **Low Risk**
> Version and changelog updates only; consumer-facing behavior was
introduced in the earlier MetaMask#11209 merge, not in this diff.
> 
> **Overview**
> **Release-only PR** that bumps the monorepo to **`1378.0.0`** and
publishes **`@metamask/social-controllers` `3.14.2`**, with no source
changes in the diff.
> 
> The **`3.14.2`** changelog records the patch already merged in
[MetaMask#11209](MetaMask#11209): realtime feed
items now use the backend’s **`authorComment`** (missing stays missing,
**`null`** stays **`null`**, present values use the same validation as
the REST feed, invalid values drop the item) instead of deriving it from
trade comments.
> 
> <sup>Reviewed by [Cursor Bugbot](https://cursor.com/bugbot) for commit
20e085a. Bugbot is set up for automated
code reviews on this repo. Configure
[here](https://www.cursor.com/dashboard/bugbot).</sup>
<!-- /CURSOR_SUMMARY -->

Co-authored-by: Cursor <cursoragent@cursor.com>

This branch was successfully deployed

1 active deployment
default-branch — 9311ab84 Deployed Oct 29, 2026 by georgeweiler via Determine whether this PR is a release PR #6288
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