Skip to content

chore(deps): upgrade dependencies to latest stable - #241

Merged
prakashchoudhary07 merged 3 commits into
developfrom
chore/upgrade-dependencies-latest
Sep 8, 2026
Merged

prakashchoudhary07 merged 3 commits into
developfrom
chore/upgrade-dependencies-latest

Conversation

@AnujChhikara

@AnujChhikara AnujChhikara commented Sep 3, 2026 •

Copy link
Copy Markdown
Contributor

Date: 3 Sep, 2026

Developer Name: @AnujChhikara


Issue Ticket Number

Tech Doc Link

  • NA

Business Doc Link

  • NA

Description

Move the toolchain to pnpm 11 / Node 24 LTS and bring every dependency to latest stable, except six held back to keep this a dependency bump rather than a refactor: typescript, react-day-picker, lucide-react, eslint, @eslint/js and eslint-plugin-react-hooks.

  • pnpm-workspace.yaml: onlyBuiltDependencies -> allowBuilds (pnpm 11)
  • tsconfig: drop deprecated baseUrl, make paths tsconfig-relative (TS 6)
  • vite.config: drop manualChunks, removed by rolldown in Vite 8
  • utils: NodeJS.Timeout -> ReturnType (@types/node 26)
  • mockServiceWorker.js regenerated by msw's postinstall

Documentation Updated?

  • Yes
  • No

Under Feature Flag

  • Yes
  • No

Database Changes

  • Yes
  • No

Breaking Changes

  • Yes
  • No

Development Tested?

  • Yes
  • No

Screenshots

Screenshot 1
Screen.Recording.2026-09-03.at.6.08.24.PM.mov

Test Coverage

Screenshot 1

Additional Notes

Move the toolchain to pnpm 11 / Node 24 LTS and bring every dependency to
latest stable, except six held back to keep this a dependency bump rather
than a refactor: typescript, react-day-picker, lucide-react, eslint,
@eslint/js and eslint-plugin-react-hooks.

- pnpm-workspace.yaml: onlyBuiltDependencies -> allowBuilds (pnpm 11)
- tsconfig: drop deprecated baseUrl, make paths tsconfig-relative (TS 6)
- vite.config: drop manualChunks, removed by rolldown in Vite 8
- utils: NodeJS.Timeout -> ReturnType<typeof setTimeout> (@types/node 26)
- mockServiceWorker.js regenerated by msw's postinstall
@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Sep 3, 2026 •

Copy link
Copy Markdown

Deploying todo-frontend with  Cloudflare Pages  Cloudflare Pages

Latest commit: 7bc83fb
Status: ✅  Deploy successful!
Preview URL: https://6ccc61f6.todo-frontend-76p.pages.dev
Branch Preview URL: https://chore-upgrade-dependencies-l.todo-frontend-76p.pages.dev

View logs

@coderabbitai

coderabbitai Bot commented Sep 3, 2026 •

Copy link
Copy Markdown

Review Change Stack

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Team

Run ID: 229e2f0f-8f11-43ab-94cc-41aac9536f28

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Team

Run ID: 1c25348b-b429-4d34-bcfc-5d1e27ab384c

📥 Commits

Reviewing files that changed from the base of the PR and between d997d87 and bbc98de.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (7)
  • .github/workflows/build-and-test.yml
  • package.json
  • pnpm-workspace.yaml
  • public/mockServiceWorker.js
  • src/lib/utils.ts
  • tsconfig.json
  • vite.config.ts
💤 Files with no reviewable changes (1)
  • vite.config.ts

Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review.


Summary by CodeRabbit

  • Chores

    • Updated the application’s underlying runtime, build tools, testing tools, and supporting libraries.
    • Updated development workflows to use newer Node.js and pnpm versions.
    • Improved compatibility across different runtime environments.
  • Bug Fixes

    • Improved handling of streamed responses in the local request-mocking environment.
    • Updated request timing and response processing for more reliable development behavior.
  • Build

    • Simplified production build configuration while continuing to generate source maps.

Walkthrough

The pull request updates Node.js and pnpm versions, refreshes dependencies, changes pnpm build permissions, upgrades MSW request handling, adjusts TypeScript path resolution, updates timer typing, and removes Vite manual chunk configuration.

Changes

Toolchain modernization

Layer / File(s) Summary
Toolchain and build configuration
.github/workflows/build-and-test.yml, package.json, pnpm-workspace.yaml, vite.config.ts
CI and Volta now use Node.js 24 and pnpm 11. Runtime and development dependencies are updated. pnpm uses allowBuilds, and Vite no longer defines manual chunks.

Mock service worker request flow

Layer / File(s) Summary
Request timing and response serialization
public/mockServiceWorker.js
MSW is upgraded to 2.15.0. Requests include interception timestamps. Event-stream responses skip cloning, and response metadata is serialized from the original response.

Compatibility settings

Layer / File(s) Summary
Type and path compatibility
src/lib/utils.ts, tsconfig.json
debounce uses ReturnType<typeof setTimeout>. The @/* alias resolves through ./src/* without baseUrl.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: ⚪ Minimal · up to bbc98

The update modernizes the Node/pnpm toolchain, dependencies, build permissions, TypeScript configuration, and MSW worker behavior. No concrete current-head merge-blocking risk remains.

Sequence Diagram(s)

sequenceDiagram
  participant FetchEvent
  participant handleRequest
  participant getResponse
  participant Client
  FetchEvent->>handleRequest: Record and pass requestInterceptedAt
  handleRequest->>getResponse: Pass requestInterceptedAt
  getResponse->>Client: Send REQUEST with interceptedAt
  getResponse->>Client: Send RESPONSE with serialized metadata
Loading

Poem

A rabbit checks the toolchain bright
Node hops to twenty-four tonight
pnpm carries scripts with care
Mocked streams drift through cleaner air
Tiny timers type just right
The burrow builds by morning light

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files. (4 skipped: 4 …
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title clearly identifies the main change: dependency and toolchain upgrades. It is concise and relevant to the changeset.
Description check ✅ Passed The description accurately covers the toolchain updates, dependency changes, configuration updates, and regenerated mock service worker.
Full details: Docstring Coverage

Explanation

Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 2 functions across 2 files. (4 skipped: 4 unsupported.)

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/upgrade-dependencies-latest

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.

Comment thread vite.config.ts
build: {
outDir: 'dist',
sourcemap: true,
rollupOptions: {

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

As part of upgrading Vite from v7 to v8, the existing manualChunks config stopped working because Vite 8 uses Rolldown and no longer accepts our object-based configuration.

Comment thread public/mockServiceWorker.js
MayankBansal12
MayankBansal12 previously approved these changes Sep 4, 2026
@AnujChhikara AnujChhikara self-assigned this Sep 5, 2026
Comment thread pnpm-workspace.yaml Outdated
Comment thread pnpm-workspace.yaml
@prakashchoudhary07
prakashchoudhary07 merged commit 6f5beaf into develop Sep 8, 2026
3 checks passed
@prakashchoudhary07
prakashchoudhary07 deleted the chore/upgrade-dependencies-latest branch September 8, 2026 20:25
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.

3 participants