Skip to content

Conversation

@MayankBansal12
Copy link
Member

@MayankBansal12 MayankBansal12 commented Jan 16, 2026

Date: 16-01-2026

Developer Name: @MayankBansal12


Issue Ticket Number

Description

  • Updates node version to node 22.x from 18.x
    (Lint, build, test and dev all are working after updating)

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

working proof
Screencast.from.2026-01-17.04-09-05.mp4

build logs

Screenshot from 2026-01-17 04-13-18

dev logs

Screenshot from 2026-01-17 04-13-33

Test Coverage

test run summary image

Additional Notes

  • node wasn't updated to latest 24.x because of incompatibility with storybook
  • node version bump was necessary to support deployment on vercel, i have tried self hosting with updated node version and it's working fine
  • seperate issue ticket Upgrade project dependencies to latest versions #1387 has been created for major packages upgrade

@vercel
Copy link

vercel bot commented Jan 16, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
status-app Ready Ready Preview, Comment Jan 16, 2026 10:34pm

@coderabbitai
Copy link

coderabbitai bot commented Jan 16, 2026

Walkthrough

Updated Node.js engine version requirements in package.json from 18.x to 24.x, including Volta version specification from 18.14.2 to 24.0.0. This change standardizes the Node.js runtime across the project to version 24.

Changes

Cohort / File(s) Summary
Node.js Version Update
package.json
Updated engines.node from "18.x" to "24.x" and volta.node from "18.14.2" to "24.0.0"

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested reviewers

  • AnujChhikara
  • iamitprakash

Poem

🐰 From eighteen hops to twenty-four we leap,
Node versions updated, promises we keep,
Volta follows suit with steady stride,
Modern runtimes now our guide! 🚀

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The PR title states the update is to Node 22.x, but the actual changes update Node to 24.x, creating a misleading discrepancy between title and implementation. Update the PR title to 'chore: bump node version to 24.x to support vercel deployment' to accurately reflect the changes made.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The pull request description clearly addresses the changeset, describing the Node.js version update from 18.x to 24.x with testing confirmation and specific context.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 2

🤖 Fix all issues with AI agents
In `@package.json`:
- Around line 109-110: Update the Node version pinned in the package.json
"volta" section from "24.0.0" to "24.13.0" and bump the TypeScript Node type
definitions by updating the `@types/node` package entry (in devDependencies or
dependencies) to "24.10.4"; ensure you only change the version strings for the
"volta"."node" field and the "@types/node" dependency (no other changes).
- Around line 98-100: The package.json engines entry currently pins "node":
"24.x" which is incompatible with Storybook 6.4; either revert the engines field
to a supported Node line (e.g., change the "node" value from "24.x" to "22.x")
or upgrade Storybook (update the Storybook package entries in devDependencies
from 6.4 to a current release that documents Node 24 support) and verify
Next.js/Jest compatibility; locate the "engines" object and the Storybook
entries in package.json (devDependencies like `@storybook/`* or storybook
packages) and apply one of these two changes consistently, then run the
build/test matrix to confirm compatibility.

@MayankBansal12 MayankBansal12 force-pushed the chore/update-node-version branch from fb62953 to 0668408 Compare January 16, 2026 22:32
@MayankBansal12 MayankBansal12 changed the title chore: update node version to 24.x chore: update node version to 22.x Jan 16, 2026
@MayankBansal12 MayankBansal12 changed the title chore: update node version to 22.x chore: bump node version to 22.x to support vercel deployment Jan 16, 2026
@iamitprakash iamitprakash merged commit 2185b76 into develop Jan 17, 2026
4 checks passed
@iamitprakash iamitprakash deleted the chore/update-node-version branch January 17, 2026 18:14
@MayankBansal12 MayankBansal12 mentioned this pull request Jan 17, 2026
10 tasks
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.

4 participants