Repository navigation
chore(deps): apply open Renovate upgrades and fix pnpm 12 CI - #200
Merged
Merged
Conversation
Pin pnpm 12.14.0 with packageManager so action-setup stops floating to a latest that rejects fresh lockfile entries and unapproved install scripts. Keep the TypeScript 6 compiler API installed for tsup and eslint while TypeScript 7 provides tsc. Co-authored-by: Elone Hoo <hi@elonehoo.me>
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Applies the dependency upgrades from the nine open Renovate PRs in one branch, with a lockfile regenerated by pnpm 12.14.0 and the follow-on fixes required for CI to install, build, and test. The existing Renovate PRs are left open.
Upgrades
Where a minor PR and a major PR touched the same package, the major range is what landed.
@antfu/eslint-config^8.3.0 (superseded by chore(deps): update dependency @antfu/eslint-config to v9 - autoclosed #192),@types/node^24.19.2,bumpp^11.1.0 (superseded by chore(deps): update dependency bumpp to v12 - autoclosed #196),eslint^10.13.0,vue^3.5.44,@vitejs/plugin-vue^6.0.10,vite^8.3.4,@iconify-json/carbon^1.2.28,@unocss/reset^66.11.0,@vueuse/core^14.4.0 (superseded by chore(deps): update dependency @vueuse/core to v15 - autoclosed #198),https-localhost^4.7.2,jiti^2.7.0,unocss^66.11.0,unplugin-vue-components^32.1.0,workbox-window^7.4.1@antfu/eslint-config^9.5.1actions/checkoutv7 (CI and release)@typescript/native(npm:typescript@^7.0.2).tscis 7.0.2actions/setup-nodev7 (CI and release)bumpp^12.3.0chalk^6.0.1@vueuse/core^15.0.0tsup^9.0.0Why CI was red
pnpm/action-setupwithversion: latestinstalls pnpm 12. That release failspnpm installin two ways the Renovate lockfiles do not satisfy:minimumReleaseAgedefaults to 24 hours, so a lockfile generated the same day as a publish dies withERR_PNPM_MINIMUM_RELEASE_AGE_VIOLATION(eslint 10.13.0 on chore(deps): update all non-major dependencies - autoclosed #191).strictDepBuildsdefaults to true, so unapproved install scripts fail withERR_PNPM_IGNORED_BUILDS(esbuildon chore(deps): update dependency @antfu/eslint-config to v9 - autoclosed #192, chore(deps): update dependency typescript to v7 - autoclosed #194, chore(deps): update dependency bumpp to v12 - autoclosed #196, chore(deps): update dependency chalk to v6 - autoclosed #197, chore(deps): update dependency @vueuse/core to v15 - autoclosed #198, chore(deps): update dependency tsup to v9 - autoclosed #199).pnpm/action-setupv6 also errors when aversioninput is set andpackage.jsonpackageManageris set. pnpm is pinned aspackageManager: pnpm@12.14.0, Node is installed before pnpm, and the workflow no longer passesversion: latest.Other fixes
allowBuilds.esbuild: trueso the esbuild postinstall is allowed.minimumReleaseAge: 0so--no-frozen-lockfiledoes not reject a same-day publish.ignore-workspace-root-check,strict-peer-dependencies, andauto-install-peersmoved from.npmrcintopnpm-workspace.yaml, because pnpm 12 only reads auth and registry settings from.npmrc.@antfu/eslint-configv9 requiresshellEmulator: true,trustPolicy: no-downgrade, andminimumReleaseAgeExcludePrune: truein the workspace file, and a sorted key order.tsup9'srollup-plugin-dtsfails ontypescript@7.0.2, andtypescript-eslint(pulled in by the antfu config) throws on TS 7. Thetypescriptdependency isnpm:@typescript/typescript6@^6.0.2so tsup and eslint keep importing the TS 6 API.@typescript/nativeis TypeScript 7.0.2, and thetscbin is 7.0.2.Object.hasOwn()inisEmptyObject.pnpm run lint,pnpm run build, andpnpm run testpass locally on Node 22 and Node 26.12.0.pnpm install --frozen-lockfilepasses with pnpm 12.14.0.Not changed
test/still uses Vitest 3, Vite 6, and@vitest/coverage-v83. None of the open PRs bump them, and the test job passes on Node 26 without a Vitest major.Note
Medium Risk
Broad dev-dependency and pnpm/CI toolchain changes affect install, build, and lint everywhere, though runtime library behavior is unchanged aside from a minor
isEmptyObjectimplementation tweak.Reviewed by Cursor Bugbot for commit 5faf32a. Bugbot is set up for automated code reviews on this repo. Configure here.