chore(deps): TanStack SSR packages and jest-dom 7 - #56
Merged
Conversation
Held back from the previous sweep because these drive SSR and no gate executes the server bundle, so they needed a manual render check rather than a green test run. Taking both to latest also collapses @tanstack/router-core, which the lockfile carried at 1.171.14 and 1.171.15 at the same time, down to a single 1.171.15. That is why this removes more lockfile lines than it adds. - @tanstack/react-router 1.170.17 -> 1.170.18 (apps/web) - @tanstack/react-start 1.168.27 -> 1.168.34 (apps/web) Supersedes Dependabot #53, which proposed react-start 1.168.33. Verified locally: routeTree.gen.ts does not drift after tsr generate, the dev server renders a 200 whose SSR output differs from the previous release only by a per-render match id, and build, lint, typecheck, and test all pass.
…peer v7 makes @testing-library/dom a required peer and raises the Node floor to 22. apps/web imports @testing-library/jest-dom/vitest in its test setup but never declared @testing-library/dom, relying on @testing-library/react to drag it in, so this adds the explicit declaration alongside the bump. packages/ui already declared it. Nothing else in v7 touches this repo. The matchers in use are toBeInTheDocument, toHaveAttribute, toHaveTextContent, toHaveClass, toContainElement, and toBeDisabled, all stable core matchers. - @testing-library/jest-dom 6.9.1 -> 7.0.0 (apps/web, packages/ui) - @testing-library/dom ^10.4.1 declared in apps/web Supersedes Dependabot #54, which bumped the version without adding the peer. Verified locally: pnpm install reports no unmet peers, the 18 component tests in packages/ui that use these matchers pass, and build, lint, typecheck, and test all pass.
This was referenced Aug 2, 2026
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.
Picks up two of the three updates held back from #52. Supersedes #53 and #54. The third, jsdom 30 (#55), is blocked on the local Node version and is explained at the bottom.
What this changes
b804f2bTanStack router and start.@tanstack/react-router 1.170.18,@tanstack/react-start 1.168.34. These were held back from #52 because they drive SSR and no gate executes the server bundle, so they needed a manual render check rather than a green test run. Taking both to latest also collapses@tanstack/router-core, which the lockfile carried at 1.171.14 and 1.171.15 simultaneously, down to a single 1.171.15. That is why the commit removes more lockfile lines than it adds.b783bc7@testing-library/jest-dom 7. The one breaking change in v7 is that@testing-library/dombecomes a required peer, with the Node floor raised to 22.apps/webimports@testing-library/jest-dom/vitestin its test setup but never declared@testing-library/dom, relying on@testing-library/reactto drag it in, so this adds the explicit declaration alongside the bump.packages/uialready declared it. Dependabot's #54 bumped the version without adding the peer.Nothing else in v7 touches this repo. The matchers in use are
toBeInTheDocument,toHaveAttribute,toHaveTextContent,toHaveClass,toContainElement, andtoBeDisabled, all stable core matchers.Verification
pnpm install --frozen-lockfileindividually, so neither carries a lockfile out of sync with its manifests.TURBO_FORCE=true,pnpm test:e2epasses,pnpm auditreports no known vulnerabilities.apps/web/src/routeTree.gen.tsdoes not drift aftertsr generate.pnpm installreports no unmet peers after the jest-dom bump.packages/uithat actually exercise the jest-dom matchers pass.Not included: jsdom 30 (#55)
jsdom@30.0.1declaresengines: ^22.22.2 || ^24.15.0 || >=26.0.0. The local Node is v24.8.0, which satisfies none of those ranges, since^24.15.0means>=24.15.0 <25.0.0. Bumping Node to 24.15 or later would unblock it. There is noenginesfield or.nvmrcin the repo pinning a version, so this is a local toolchain decision rather than a repo constraint.Also still outstanding:
typescript 7.0.2. Dependabot has not proposed it, and correctly so.typescript-eslint@8.65.0declares a peer oftypescript: >=4.8.4 <6.1.0, so TypeScript 7 is not installable until typescript-eslint widens that range.