Skip to content

Conversation

@danielroe
Copy link
Member

@danielroe danielroe commented Oct 27, 2025

This reverts commit bc4b293.

πŸ”— Linked issue

❓ Type of change

  • πŸ“– Documentation (updates to the documentation or readme)
  • 🐞 Bug fix (a non-breaking change that fixes an issue)
  • πŸ‘Œ Enhancement (improving an existing functionality)
  • ✨ New feature (a non-breaking change that adds functionality)
  • 🧹 Chore (updates to the build process or auxiliary tools and libraries)
  • ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

πŸ“š Description

migrating to fetch-based handler seems to have caused a number of regressions, due to incomplete support for fetching from sockets on some platforms.

we'll update later once listhen has been rebased on srvx

this isn't a pure revert - we switch to http-proxy-3 which has been adopted by vite

@github-actions
Copy link
Contributor

github-actions bot commented Oct 27, 2025

πŸ“¦ Bundle Size Comparison

πŸ“‰ nuxi

Metric Base Head Diff
Rendered 5171.46 KB 4376.28 KB -795.18 KB (-15.38%)

πŸ“‰ nuxt-cli

Metric Base Head Diff
Rendered 134.10 KB 126.17 KB -7.93 KB (-5.91%)

➑️ create-nuxt

Metric Base Head Diff
Rendered 2321.85 KB 2321.85 KB 0.00 KB (0.00%)

@pkg-pr-new
Copy link

pkg-pr-new bot commented Oct 27, 2025

nuxt-cli-playground

npm i https://pkg.pr.new/create-nuxt@1104
npm i https://pkg.pr.new/nuxi@1104
npm i https://pkg.pr.new/@nuxt/cli@1104

commit: 9d8001a

@codspeed-hq
Copy link

codspeed-hq bot commented Oct 27, 2025

CodSpeed Performance Report

Merging #1104 will not alter performance

Comparing refactor/proxy (9d8001a) with main (caa1a6d)

Summary

βœ… 2 untouched

@danielroe danielroe marked this pull request as ready for review October 27, 2025 20:28
@danielroe danielroe merged commit 3afd02b into main Oct 27, 2025
12 checks passed
@danielroe danielroe deleted the refactor/proxy branch October 27, 2025 20:29
@github-actions github-actions bot mentioned this pull request Oct 27, 2025
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.

2 participants