From 887b2802180f78911a2af8ac4fe8517f210dc65c Mon Sep 17 00:00:00 2001 From: John Marlo Lapiz <91590512+lazuee@users.noreply.github.com> Date: Thu, 16 Jan 2025 04:35:31 +0000 Subject: [PATCH 1/2] fix: change ResolvedConfig type to interface (#15473) --- packages/vite/src/node/config.ts | 135 ++++++++++++++++--------------- 1 file changed, 68 insertions(+), 67 deletions(-) diff --git a/packages/vite/src/node/config.ts b/packages/vite/src/node/config.ts index 507c580800325a..afcbfc76916bcb 100644 --- a/packages/vite/src/node/config.ts +++ b/packages/vite/src/node/config.ts @@ -532,73 +532,74 @@ export interface InlineConfig extends UserConfig { envFile?: false } -export type ResolvedConfig = Readonly< - Omit< - UserConfig, - | 'plugins' - | 'css' - | 'json' - | 'assetsInclude' - | 'optimizeDeps' - | 'worker' - | 'build' - | 'dev' - | 'environments' - | 'server' - | 'preview' - > & { - configFile: string | undefined - configFileDependencies: string[] - inlineConfig: InlineConfig - root: string - base: string - /** @internal */ - decodedBase: string - /** @internal */ - rawBase: string - publicDir: string - cacheDir: string - command: 'build' | 'serve' - mode: string - isWorker: boolean - // in nested worker bundle to find the main config - /** @internal */ - mainConfig: ResolvedConfig | null - /** @internal list of bundle entry id. used to detect recursive worker bundle. */ - bundleChain: string[] - isProduction: boolean - envDir: string - env: Record - resolve: Required & { - alias: Alias[] - } - plugins: readonly Plugin[] - css: ResolvedCSSOptions - json: Required - esbuild: ESBuildOptions | false - server: ResolvedServerOptions - dev: ResolvedDevEnvironmentOptions - /** @experimental */ - builder: ResolvedBuilderOptions | undefined - build: ResolvedBuildOptions - preview: ResolvedPreviewOptions - ssr: ResolvedSSROptions - assetsInclude: (file: string) => boolean - logger: Logger - createResolver: (options?: Partial) => ResolveFn - optimizeDeps: DepOptimizationOptions - /** @internal */ - packageCache: PackageCache - worker: ResolvedWorkerOptions - appType: AppType - experimental: ExperimentalOptions - environments: Record - /** @internal */ - fsDenyGlob: AnymatchFn - /** @internal */ - safeModulePaths: Set - } & PluginHookUtils -> +export interface ResolvedConfig + extends Readonly< + Omit< + UserConfig, + | 'plugins' + | 'css' + | 'json' + | 'assetsInclude' + | 'optimizeDeps' + | 'worker' + | 'build' + | 'dev' + | 'environments' + | 'server' + | 'preview' + > & { + configFile: string | undefined + configFileDependencies: string[] + inlineConfig: InlineConfig + root: string + base: string + /** @internal */ + decodedBase: string + /** @internal */ + rawBase: string + publicDir: string + cacheDir: string + command: 'build' | 'serve' + mode: string + isWorker: boolean + // in nested worker bundle to find the main config + /** @internal */ + mainConfig: ResolvedConfig | null + /** @internal list of bundle entry id. used to detect recursive worker bundle. */ + bundleChain: string[] + isProduction: boolean + envDir: string + env: Record + resolve: Required & { + alias: Alias[] + } + plugins: readonly Plugin[] + css: ResolvedCSSOptions + json: Required + esbuild: ESBuildOptions | false + server: ResolvedServerOptions + dev: ResolvedDevEnvironmentOptions + /** @experimental */ + builder: ResolvedBuilderOptions | undefined + build: ResolvedBuildOptions + preview: ResolvedPreviewOptions + ssr: ResolvedSSROptions + assetsInclude: (file: string) => boolean + logger: Logger + createResolver: (options?: Partial) => ResolveFn + optimizeDeps: DepOptimizationOptions + /** @internal */ + packageCache: PackageCache + worker: ResolvedWorkerOptions + appType: AppType + experimental: ExperimentalOptions + environments: Record + /** @internal */ + fsDenyGlob: AnymatchFn + /** @internal */ + safeModulePaths: Set + } & PluginHookUtils + > {} // inferred ones are omitted export const configDefaults = Object.freeze({ From 4dc672fc4ba5cbdb4ea46b1e526e56fbe84652ff Mon Sep 17 00:00:00 2001 From: John Marlo Lapiz <91590512+lazuee@users.noreply.github.com> Date: Thu, 16 Jan 2025 05:28:35 +0000 Subject: [PATCH 2/2] fix: correct identifier mappings for TransformResult in rollup and esbuild --- packages/vite/rollup.dts.config.ts | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/packages/vite/rollup.dts.config.ts b/packages/vite/rollup.dts.config.ts index 46306f0a22d6b5..f1ba14b3b1dd63 100644 --- a/packages/vite/rollup.dts.config.ts +++ b/packages/vite/rollup.dts.config.ts @@ -50,10 +50,10 @@ const identifierReplacements: Record> = { Plugin$1: 'rollup.Plugin', PluginContext$1: 'rollup.PluginContext', TransformPluginContext$1: 'rollup.TransformPluginContext', - TransformResult$2: 'rollup.TransformResult', + TransformResult$1: 'rollup.TransformResult', }, esbuild: { - TransformResult$1: 'esbuild_TransformResult', + TransformResult$2: 'esbuild_TransformResult', TransformOptions$1: 'esbuild_TransformOptions', BuildOptions$1: 'esbuild_BuildOptions', },