Skip to content

馃殌 chore(release): prepare supervision 0.2.10 - #148

Merged
cfviotti merged 1 commit into
mainfrom
release/0.2.10
Oct 17, 2026
Merged

cfviotti merged 1 commit into
mainfrom
release/0.2.10

Conversation

@cfviotti

Copy link
Copy Markdown
Contributor

Description

Prepare supervision 0.2.10 so consumers can install the already merged annotation smoothing, consistent mask outlines, focus and label fixes, bounded polygon preparation, and paused-mask refinement from #132.

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation or example update
  • Refactor or maintenance
  • Performance improvement

Validation

  • npm run verify (3,582 Vitest tests plus repository contracts, types and builds)
  • npm run package:tarball
  • npm run package:tarball:smoke with a clean external consumer
  • npm run package:publish:dry-run

Notes For Reviewers

Version metadata and matching documentation only. The release candidate is based on merged main at 1e7325d; #146 and #147 are not included.

The candidate version was unused on npm when prepared. Publication remains a separate protected workflow from main after the release PR is approved and merged.

@cfviotti

Copy link
Copy Markdown
Contributor Author

Mega Review

Status: REVIEW-CLEAN @ 671c20c 路 full first round 路 converged after 1 round

The manifest, workspace lock entry, docs toolbar and migration pins consistently prepare supervision@0.2.10. No runtime source, dependencies, exports or publishing workflow changed. The portable archive contains the bundled core and no workspace-relative install dependency.

Caveats

Three pinned Codex lanes, outside Claude and the parent applied all thirteen lenses to this four-file delta. Codex independently compared every lockfile property; Claude inspected its changed region and materialized diff rather than fully parsing it. All four GitHub feedback feeds are complete and no prior key exists. Ticket compliance: not requested. Merged implementation changes are the release base, not re-reviewed here.

Separate implementation checks passed: full verification (3,582 Vitest tests, contracts, types and builds), portable packaging, 12 clean-consumer checks, and publish dry run. Current CI is green on this exact head. Review-clean does not replace those checks; nothing has been published.

Review Ledger

Open findings

  • None.

Prior key statuses

  • None; first round.

Unresolved prior open findings

  • None.

Deliberate tradeoffs

  • None introduced by this delta.

Refuted

  • Dry-run bundled-file count means core is missing: the actual archive contains node_modules/supervision-js-core/dist/index.js, lists bundled core in its manifest and passes the external-consumer smoke suite.
  • Source file:../core becomes an invalid consumer dependency: the unchanged portable packer embeds core and rewrites its staged manifest; the inspected archive has no file: install dependency.

Implausible

  • None.

Pre-existing

  • Main's docs deployment can precede manual npm publication, temporarily advertising an unused version. The committed release procedure already requires that order; publication remains a separate protected step. No process expansion in this release PR.

@cfviotti cfviotti added the LGTMegaReview 馃 Approved by Jarbas Mega Review label Oct 17, 2026
@cfviotti
cfviotti requested a review from a team October 17, 2026 00:42
@cfviotti cfviotti self-assigned this Oct 17, 2026
@cfviotti
cfviotti marked this pull request as ready for review October 17, 2026 00:42
@cfviotti
cfviotti merged commit 3c9badb into main Oct 17, 2026
3 checks passed
@cfviotti
cfviotti deleted the release/0.2.10 branch October 17, 2026 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LGTMegaReview 馃 Approved by Jarbas Mega Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants