Skip to content

chore(release): 5.0.0 - #1194

Merged
Mikola Lysenko (mikolalysenko) merged 7 commits into
mainfrom
release/v5.0.0
Oct 9, 2026
Merged

Mikola Lysenko (mikolalysenko) merged 7 commits into
mainfrom
release/v5.0.0

Conversation

@mikolalysenko

@mikolalysenko Mikola Lysenko (mikolalysenko) commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Version-bump PR for socket-patch 5.0.0 (runbook: docs/releasing.md step 2).

Two tests assumed the tree was at the published 4.0.0 and broke on the bump:

  • scripts/tests/test_release.py test_sync_main_on_a_working_tree_stamps_the_newest_tag tagged v4.0.0 over the live packaging; it now stamps a 4.0.0 baseline like the other stamp tests.
  • The vlt launcher leg on vlt ≤ 0.0.0-13: those vlt releases ignore the configured registry (vlt.json and --registry) and resolve from public npm, so the leg only passed because 4.0.0 is published. Verified against an unreachable registry (0.0.0-1/-11/-12/-13 install, 0.0.0-14 fails ECONNREFUSED). The era is marked non-hermetic in docs/testing/vlt-compatibility.md and the derived manifest skips the launcher leg there.

After merge, the release is: Actions → Release → Run workflow on main (optionally dry-run: true first), then approve the staged npm packages (platform packages first).

Before dispatching, confirm the registry trusted publishers point at the split workflows (docs/releasing.md → One-time registry setup): crates.io socket-patch-core/socket-patch-cli → publish-cargo.yml; npm main + 14 platform packages → publish-npm.yml. Neither workflow has ever run, and this couldn't be verified without registry owner credentials.

🤖 Generated with Claude Code


Note

Low Risk
Packaging version sync, changelog cut, and vlt/release test skips only; no new runtime behavior beyond what is already on main.

Overview
Release train step for socket-patch 5.0.0: rolls the accumulated [Unreleased] notes into a dated ## [5.0.0] — 2026-10-09 section (breaking CLI/JSON behavior, Gradle/JVM, ecosystem fixes, etc.) and bumps the workspace and npm packaging from 4.0.0 → 5.0.0 (Cargo.toml / Cargo.lock, main @socketsecurity/socket-patch package, 14 platform optional packages, and the npm lockfile).

Test harness adjustments so CI stays valid on the new tree: test_sync_main_on_a_working_tree_stamps_the_newest_tag seeds packaging with a 4.0.0 baseline instead of tagging over live 5.0.0 files; vlt ≤ 0.0.0-13 is documented and skipped for the agent launcher leg because those releases ignore vlt.json / --registry and always hit public npm (unpublished launcher version only exists on the harness registry). Adds registry_config_ignored(), a derived leg rule in vlt-leg-manifest.json, and an update to docs/testing/vlt-compatibility.md.

Reviewed by Cursor Bugbot for commit 75d69ae. Configure here.


Generated by Claude Code

Cut the [5.0.0] CHANGELOG section from [Unreleased], adding notes for
the 150 PRs merged since the CHANGELOG freeze (#848), and stamp 5.0.0
into Cargo.toml, Cargo.lock and the npm main + platform packages.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@socket-security-staging

socket-security-staging Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

@socket-security

socket-security Bot commented Oct 9, 2026 •

Copy link
Copy Markdown

The test copied the live packaging and tagged v4.0.0, so it failed once
the tree was stamped 5.0.0. copy_packaging's baseline= keeps it
independent of the checkout's version, like the other stamp tests.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
vlt <= 0.0.0-13 ignores vlt.json's registry (and --registry): installs
resolve from public npm. The launcher leg only passed there because
@socketsecurity/socket-patch@4.0.0 is published; at the unreleased 5.0.0
it cannot resolve. Verified against an unreachable registry: 0.0.0-1,
-11, -12, -13 still install, 0.0.0-14 fails ECONNREFUSED. Marks the era
non-hermetic in docs/testing/vlt-compatibility.md and adds the derived
skip rule for the launcher leg.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
…elease notes

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@mikolalysenko

Copy link
Copy Markdown
Collaborator Author

bugbot run


Generated by Claude Code

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 75d69ae. Configure here.

@mikolalysenko Mikola Lysenko (mikolalysenko) added the Ready for review Agent-verified: mergeable, CI green, Bugbot clean — awaiting human review label Oct 9, 2026
@mikolalysenko

Copy link
Copy Markdown
Collaborator Author

Ready for review (burn-down agent).

  • Head: 75d69ae
  • CI: 406/406 green (394 success, 12 skipped)
  • Bugbot: reviewed 75d69ae, no findings
  • Mergeable. Release PR, so the CHANGELOG.md cut is expected here.
  • Auto-merge is already armed; it will enqueue once a human approves.

Generated by Claude Code

@mikolalysenko

Copy link
Copy Markdown
Collaborator Author

[final reviewer] Reviewed 75d69ae06; no brief yet because the 5.0.0 notes are missing two user-facing fixes that are already on main and would ship in this release:

(771a543c, the #780/#1221 Gemfile.lock refactor, is arguably covered by the Maintenance line.) All three merged ~06:52 UTC, before this branch's last notes commit. The branch is 27 commits behind main but merges cleanly.

Everything else checks out: scripts/release-lint.sh --stable-only --tag-check passes on the head, test_release.py 82 OK, the Breaking section justifies the major bump, and no stray 4.0.0 is left except crates/socket-patch-node/npm/package.json:3 (private, unpublished, cosmetic). The package-lock.json drop of the 14 platform entries is expected from version-sync.sh.

Leaving the notes edit to the release agent. Once the two entries are in, I'll post the approval brief.


Generated by Claude Code

@mikolalysenko

Copy link
Copy Markdown
Collaborator Author

[final reviewer] Disarmed auto-merge on 75d69ae06. It had no final-review brief and no approval, and the open point from my 10:05 note still holds: the 5.0.0 notes leave out #1223 (1a44db8c) and #1228 (e03a666d). Both are already on main and would ship in this release. With auto-merge armed, an approval would have sent the release into the queue without those two entries.

Once the release agent adds them, I'll post the brief and re-arm auto-merge.


Generated by Claude Code

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

Labels

Ready for review Agent-verified: mergeable, CI green, Bugbot clean — awaiting human review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants