feat!: upgrade node to 24 and regain deployability#185
Conversation
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This comment was marked as outdated.
This comment was marked as outdated.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Noted observed bug from @marcel-at-glide 's observation: https://www.loom.com/share/8e1405c6b8764e689d299acd4c39d8d6 Claudio analysis:
Karl analysisI believe reasonably that this reflects an aging codebase, we should pursue next.js update but outside of this scope. Unless we have clear monitoring / user feedback that tells us that this is slow, this might be recency bias. I trust Claude's judgement and recommend we move forward. |
|
I vote to move forward |
|
The motion passes! |
We presently lack the ability to make change to the repo, because we are on Node 18 (actually we probably "got upgraded" over time, earlier art in the repo shows that we intend Node 14)
It is not possible to roll back to a version not on this list. As a part of this change, a one-way change from 18 to 24 was made. We can roll back down to 20, but cannot get back to version 18 or below.
The rest of this change supports deployability of the project, in a reasonably minimal way, however should take care to keep other dependencies up-to-date. (Next.js, react-dom, vercel, etc). Those are deferred and out of scope in favor of getting to a deployable state. This is supportive of the effort in #184 .
Summary
Test plan
🤖 Generated with Claude Code