Skip to content

fix(deps): bump @tanstack/dom-vite to 0.1.0-alpha.8#850

Merged
tannerlinsley merged 1 commit intomainfrom
taren/bump-dom-vite-alpha8
Apr 22, 2026
Merged

fix(deps): bump @tanstack/dom-vite to 0.1.0-alpha.8#850
tannerlinsley merged 1 commit intomainfrom
taren/bump-dom-vite-alpha8

Conversation

@tannerlinsley
Copy link
Copy Markdown
Member

@tannerlinsley tannerlinsley commented Apr 22, 2026

Summary

Picks up TanStack/react@fb904fafix(react-dom): reverse-iterate placeChildrenInOrder.

Fixes these user-reported reorder bugs:

  • Application starter: clicking Analyze → picking a different idea chip swapped the Analyze and "I'm feeling lucky" buttons.
  • npm stats page: library dropdown items reordered after filter changes.

Root cause was the shim's forward iteration over the placement loop. When the starting DOM diverged from the target, each `insertBefore(doms[i], doms[i+1])` pulled doms[i] forward past nodes that should move behind it, cascading into the wrong final order.

Test plan

  • Reproduced the app-starter swap locally, verified fix clears it
  • Site search still returns results (no regression on last week's fix)
  • `pnpm husky` passes (format + tsc + lint + smoke 4/4)
  • 692/692 unit tests in the shim (688 prior + 4 new child-reorder regression tests)

Summary by CodeRabbit

  • Chores
    • Updated development dependencies to latest versions for improved build tooling compatibility.

Picks up fix(react-dom): reverse-iterate placeChildrenInOrder — fixes
the Analyze / "I'm feeling lucky" swap on the application starter and
the library dropdown reorder on the npm stats page.

Forward iteration of placeChildrenInOrder cascaded moves into the wrong
final order when the starting DOM diverged from the target. Reverse
iteration places each node before its already-final successor, so the
cascade can't happen.
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 22, 2026

Deploy Preview for tanstack ready!

Name Link
🔨 Latest commit 36aafce
🔍 Latest deploy log https://app.netlify.com/projects/tanstack/deploys/69e8e114d9e2b40008be04ea
😎 Deploy Preview https://deploy-preview-850--tanstack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 26
Accessibility: 90
Best Practices: 83
SEO: 97
PWA: 70
View the detailed breakdown and full score reports

To edit notification comments on pull requests, go to your Netlify project configuration.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 22, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 320da0a1-ec60-422f-bfaa-19589bc2de21

📥 Commits

Reviewing files that changed from the base of the PR and between 454493e and 36aafce.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (1)
  • package.json

📝 Walkthrough

Walkthrough

Updated the dev dependency @tanstack/dom-vite from version 0.1.0-alpha.7 to 0.1.0-alpha.8 in the package configuration. No other dependencies, scripts, or configurations were modified.

Changes

Cohort / File(s) Summary
Dependency Update
package.json
Bumped @tanstack/dom-vite devDependency from 0.1.0-alpha.7 to 0.1.0-alpha.8.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Possibly related PRs

Poem

A whisker twitch, a version bump so small,
Alpha.8 now answers the call,
The rabbit rejoices, ear to toe,
Dependencies dance and frameworks grow! 🐰✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch taren/bump-dom-vite-alpha8

Comment @coderabbitai help to get the list of available commands and usage tips.

@tannerlinsley tannerlinsley merged commit 43c791f into main Apr 22, 2026
6 of 8 checks passed
@tannerlinsley tannerlinsley deleted the taren/bump-dom-vite-alpha8 branch April 22, 2026 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant