Skip to content

chore(deps): converge oxfmt on ^0.61.0 (WR-0495) - #199

Merged
town-crier-announce[bot] merged 1 commit into
mainfrom
chore/oxfmt-0.61-convergence
Jul 29, 2026
Merged

chore(deps): converge oxfmt on ^0.61.0 (WR-0495)#199
town-crier-announce[bot] merged 1 commit into
mainfrom
chore/oxfmt-0.61-convergence

Conversation

@Goosterhof

Copy link
Copy Markdown
Contributor

What

Converges oxfmt on ^0.61.0 as part of the fleet-wide alignment tracked by WR-0495.

The fleet had drifted into a three-way split^0.59 / ^0.60 / ^0.61 across ten repos. Because ^0.x locks the minor in npm semver, ^0.59.0 genuinely refuses 0.60.0; the ranges have to be moved explicitly, which is why dependabot had an open oxfmt PR on most of these repos.

Why this is safe — measured, not assumed

WR-0495 was originally written on the assumption that "an oxfmt minor can reflow files fleet-wide", which justified a cautious one-repo-at-a-time rollout. That assumption was tested and does not hold for this hop.

oxfmt@<old> --check vs oxfmt@0.61.0 --check, each run against this repo's own format:check target (not a guessed path):

  • 9 of 10 repos: byte-identical output. Same flagged set, both versions.
  • 1 repo (codebook): a single cosmetic SCSS reflow — 0.61 collapses a multi-line tint(...) call onto one line. Folded into that repo's own PR.

Positive control (§ Null-Result Gate — a clean result is only meaningful if the instrument is known to fire): a deliberately misformatted file was introduced, --check correctly flagged it, and the file was restored. The check is not silently passing.

The diff

Manifest + lockfile only. Every added lockfile entry was verified to be an oxfmt package — no unrelated dependency moved.

Verification

  • format:check — run after the bump, result recorded in the commit message
  • Pre-commit gates (Pint / Rector / oxlint / oxfmt / vue-tsc, as applicable per repo) — passed

🤖 Generated with Claude Code

https://claude.ai/code/session_012QpHumQkpoipXd6oMuy1aB

Part of the fleet oxfmt convergence. The fleet had split three ways
(^0.59 / ^0.60 / ^0.61); `^0.x` locks the minor in npm semver, so the
ranges must move explicitly.

Verified output-identical before bumping: `oxfmt@0.59.0 --check` and
`oxfmt@0.61.0 --check` flag the same file set on this repo's own
format target. format:check verified after bump. Diff is manifest +
lockfile only.

Partially supersedes dependabot #189 (oxfmt leg only; its oxlint
1.74 -> 1.76 leg is unaffected and remains open).

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012QpHumQkpoipXd6oMuy1aB
@Goosterhof Goosterhof added the Agent Review Requested Requesting review of specialized AI review agents. label Jul 28, 2026
@Goosterhof
Goosterhof requested a review from a team as a code owner July 28, 2026 21:00
@Goosterhof Goosterhof added the Agent Review Requested Requesting review of specialized AI review agents. label Jul 28, 2026

@dmooibroek dmooibroek 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.

Clean. Verified the containment claim independently: root package.json moves only oxfmt ^0.59.0 -> ^0.61.0 in devDependencies, and the lockfile diff touches zero non-oxfmt node_modules/ entries. Checked the publish angle for this repo specifically — no published sub-package declares oxfmt in dependencies or peerDependencies, so the range change is not consumer-visible on the npm registry.

@town-crier-announce
town-crier-announce Bot merged commit 193fd6a into main Jul 29, 2026
5 checks passed
@town-crier-announce
town-crier-announce Bot deleted the chore/oxfmt-0.61-convergence branch July 29, 2026 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Agent Review Requested Requesting review of specialized AI review agents.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants