From 72a0e735866bb0ab0e44ab17ed24f32100727444 Mon Sep 17 00:00:00 2001 From: jdalton Date: Mon, 29 Dec 2025 09:03:33 -0500 Subject: [PATCH] fix(optimize): remove Node.js version filter from manifest entries Removes the semver-based Node version check that was filtering manifest entries, allowing all Socket registry overrides to be applied regardless of the Node version specified in package engines. --- src/commands/optimize/add-overrides.mts | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/src/commands/optimize/add-overrides.mts b/src/commands/optimize/add-overrides.mts index 914dab205..4de4ad6ae 100644 --- a/src/commands/optimize/add-overrides.mts +++ b/src/commands/optimize/add-overrides.mts @@ -109,21 +109,13 @@ export async function addOverrides( const depAliasMap = new Map() const depEntries = getDependencyEntries(pkgEnvDetails) - const manifestEntries = manifestNpmOverrides.filter(({ 1: data }) => - semver.satisfies( - // Roughly check Node range as semver.coerce will strip leading - // v's, carets (^), comparators (<,<=,>,>=,=), and tildes (~). - semver.coerce(data.engines.node)!, - pkgEnvDetails.pkgRequirements.node, - ), - ) const addingText = `Adding overrides to ${workspace}...` let loggedAddingText = false // Chunk package names to process them in parallel 3 at a time. await pEach( - manifestEntries, + manifestNpmOverrides, async ({ 1: data }) => { const { name: sockRegPkgName, package: origPkgName, version } = data const major = getMajor(version)!