Skip to content

Commit 7cd038f

Browse files
committed
fix: filter internal vite-plus aliases from published CLI overrides
The workspace pnpm-workspace.yaml overrides include npm: protocol redirections (vite→@voidzero-dev/vite-plus-core, vitest→@voidzero-dev/ vite-plus-test) that are internal to the workspace's Vite+ migration. These were being resolved and written into the published CLI package.json overrides, polluting the published artifact with dev-only aliases. Filter out any resolved overrides that use npm: protocol aliases, since those are always workspace-internal package redirections that should never be published to end users.
1 parent 3207da4 commit 7cd038f

1 file changed

Lines changed: 11 additions & 5 deletions

File tree

apps/server/scripts/cli.ts

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -231,6 +231,16 @@ const publishCmd = Command.make(
231231
const workspaceConfig = yield* readWorkspaceConfig();
232232
const workspaceCatalog = workspaceConfig.catalog ?? {};
233233
const workspaceOverrides = workspaceConfig.overrides ?? {};
234+
const resolvedOverrides = resolveCatalogDependencies(
235+
workspaceOverrides,
236+
workspaceCatalog,
237+
"apps/server",
238+
);
239+
// Filter out internal workspace aliases (npm: protocol redirections
240+
// like vite→vite-plus) that are irrelevant to published consumers.
241+
const publishOverrides = Object.fromEntries(
242+
Object.entries(resolvedOverrides).filter(([, v]) => !v.startsWith("npm:")),
243+
);
234244
const pkg: PackageJson = {
235245
name: serverPackageJson.name,
236246
repository: serverPackageJson.repository,
@@ -244,11 +254,7 @@ const publishCmd = Command.make(
244254
workspaceCatalog,
245255
"apps/server",
246256
),
247-
overrides: resolveCatalogDependencies(
248-
workspaceOverrides,
249-
workspaceCatalog,
250-
"apps/server",
251-
),
257+
overrides: publishOverrides,
252258
};
253259

254260
const original = yield* fs.readFileString(packageJsonPath);

0 commit comments

Comments
 (0)