Refresh repo dependencies#417
Merged
Merged
Conversation
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.
Why:
Dependabot currently has a narrow lockfile PR for
brace-expansion(#416), while the repo has a broader set of transitive dependency alerts onmain. This consolidates the feasible dependency refresh into one tested PR so we can clear the lockfile noise together instead of spending CI/review cycles on one-off bumps.What:
@biomejs/biome,@types/node,@vitest/coverage-v8,jest-diff,knip, andvitest.p-queue,nock, and shared@types/noderanges.brace-expansion,@hono/node-server,path-to-regexp,picomatch,tar,minimatch,ajv,qs, andglob.tempdependency and replaces it with native Node temp-directory helpers so the deprecatedrimraf@2/glob@7/inflightchain drops out of the audit..changeset/deps-refresh.mdwith patch release notes for@transloadit/node,transloadit, and@transloadit/mcp-server; generated package changelogs will be updated by the post-merge Version Packages PR.Held back:
got: stayed on14.6.6. Thegotv15 release requires Node.js 22 and switches to nativeFormData; this repo still publishes@transloadit/node/transloaditfor Node 20+, and the attempted bump failed unit tests withNon-native FormData is not supported. Source: https://github.com/sindresorhus/got/releases/tag/v15.0.0zodin@transloadit/node/transloadit: stayed on3.25.76. The attempted v4 bump failed during SDK schema import on Zod internals (def.shape is not a function), and the Zod v4 migration guide documents broad breaking changes plus undocumented internal API changes. Source: https://zod.dev/v4/changelog@transloadit/*workspace ranges were left alone; Yarn reports multiple workspace-vs-registry upgrade strategies there, and this repo’s release/version flow owns those ranges.Post-review release path:
# Releasessection for@transloadit/node,transloadit, and@transloadit/mcp-server, ensure CI is green, then squash-merge that version PR.npm view @transloadit/node version,npm view transloadit version, andnpm view @transloadit/mcp-server version.Validation:
corepack yarn check(rerun after changeset)corepack yarn verify:full(rerun after changeset; passed on rerun after one transient notify-url-relay network-test failure)corepack yarn workspace @transloadit/node test:unitcorepack yarn npm audit -A -R --severity low(No audit suggestions)corepack yarn install --immutablegit diff --check --cached~/code/dotfiles/bin/council.ts review(No issues found.)