refactor: remove direct imports from rollup (#7751)

Author: sheremet-va

packages/browser/src/node/plugins/pluginContext.ts

@@ -1,4 +1,4 @@
-import type { PluginContext } from 'rollup'
+import type { Rollup } from 'vite'
 import type { Plugin } from 'vitest/config'
 import type { ParentBrowserProject } from '../projectParent'
 import { fileURLToPath } from 'node:url'
@@ -28,7 +28,7 @@ export default function BrowserContext(globalServer: ParentBrowserProject): Plug
 }
 
 async function generateContextFile(
-  this: PluginContext,
+  this: Rollup.PluginContext,
   globalServer: ParentBrowserProject,
 ) {
   const commands = Object.keys(globalServer.commands)

packages/mocker/src/node/dynamicImportPlugin.ts

@@ -32,7 +32,6 @@ export function dynamicImportPlugin(options: DynamicImportPluginOptions = {}): P
 }
 
 export interface DynamicImportInjectorResult {
-  ast: Rollup.ProgramNode
   code: string
   map: SourceMap
 }
@@ -45,7 +44,7 @@ export function injectDynamicImport(
 ): DynamicImportInjectorResult | undefined {
   const s = new MagicString(code)
 
-  let ast: any
+  let ast: ReturnType<Rollup.PluginContext['parse']>
   try {
     ast = parse(code)
   }
@@ -75,7 +74,6 @@ export function injectDynamicImport(
   })
 
   return {
-    ast,
     code: s.toString(),
     map: s.generateMap({ hires: 'boundary', source: id }),
   }

packages/mocker/src/node/esmWalker.ts

@@ -61,7 +61,7 @@ export function isNodeInPattern(node: _Node): node is Property {
  * Except this is using acorn AST
  */
 export function esmWalker(
-  root: Rollup.ProgramNode,
+  root: ReturnType<Rollup.PluginContext['parse']>,
   { onIdentifier, onImportMeta, onDynamicImport, onCallExpression }: Visitors,
 ): void {
   const parentStack: Node[] = []

packages/mocker/src/node/hoistMocksPlugin.ts

@@ -9,7 +9,6 @@ import type {
   VariableDeclaration,
 } from 'estree'
 import type { SourceMap } from 'magic-string'
-import type { RollupAstNode } from 'rollup'
 import type { Plugin, Rollup } from 'vite'
 import type { Node, Positioned } from './esmWalker'
 import { findNodeAround } from 'acorn-walk'
@@ -125,7 +124,6 @@ const regexpHoistable
 const hashbangRE = /^#!.*\n/
 
 export interface HoistMocksResult {
-  ast: Rollup.ProgramNode
   code: string
   map: SourceMap
 }
@@ -149,7 +147,7 @@ export function hoistMocks(
 
   const s = new MagicString(code)
 
-  let ast: Rollup.ProgramNode
+  let ast: ReturnType<Rollup.PluginContext['parse']>
   try {
     ast = parse(code)
   }
@@ -175,7 +173,7 @@ export function hoistMocks(
   const idToImportMap = new Map<string, string>()
 
   const imports: {
-    node: RollupAstNode<ImportDeclaration>
+    node: Positioned<ImportDeclaration>
     id: string
   }[] = []
 
@@ -549,7 +547,6 @@ export function hoistMocks(
   }
 
   return {
-    ast,
     code: s.toString(),
     map: s.generateMap({ hires: 'boundary', source: id }),
   }

packages/mocker/src/node/resolver.ts

@@ -1,5 +1,4 @@
-import type { PartialResolvedId } from 'rollup'
-import type { ResolvedConfig as ViteConfig, ViteDevServer } from 'vite'
+import type { Rollup, ResolvedConfig as ViteConfig, ViteDevServer } from 'vite'
 import { existsSync, readFileSync } from 'node:fs'
 import { isAbsolute, join, resolve } from 'pathe'
 import { cleanUrl } from '../utils'
@@ -117,7 +116,7 @@ export class ServerMockResolver {
     return this.resolveModule(rawId, resolved)
   }
 
-  private resolveModule(rawId: string, resolved: PartialResolvedId | null) {
+  private resolveModule(rawId: string, resolved: Rollup.PartialResolvedId | null) {
     const id = resolved?.id || rawId
     const external
       = !isAbsolute(id) || isModuleDirectory(this.options, id) ? rawId : null

packages/vitest/src/node/viteLogger.ts

@@ -1,5 +1,4 @@
-import type { RollupError } from 'rollup'
-import type { LogErrorOptions, Logger, LoggerOptions, LogLevel, LogType } from 'vite'
+import type { LogErrorOptions, Logger, LoggerOptions, LogLevel, LogType, Rollup } from 'vite'
 import type { Logger as VitestLogger } from './logger'
 import colors from 'tinyrainbow'
 
@@ -41,7 +40,7 @@ export function createViteLogger(
   level: LogLevel = 'info',
   options: LoggerOptions = {},
 ): Logger {
-  const loggedErrors = new WeakSet<Error | RollupError>()
+  const loggedErrors = new WeakSet<Error | Rollup.RollupError>()
   const { prefix = '[vite]', allowClearScreen = true } = options
   const thresh = LogLevels[level]
   const canClearScreen

test/core/test/injector-esm.test.ts

@@ -1,4 +1,4 @@
-import { parseAst } from 'rollup/parseAst'
+import { parseAst } from 'vite'
 import { expect, test } from 'vitest'
 import { injectDynamicImport } from '../../../packages/mocker/src/node/dynamicImportPlugin'
 

test/core/test/injector-mock.test.ts

@@ -1,6 +1,6 @@
 import type { HoistMocksPluginOptions } from '../../../packages/mocker/src/node/hoistMocksPlugin'
 import { stripVTControlCharacters } from 'node:util'
-import { parseAst } from 'rollup/parseAst'
+import { parseAst } from 'vite'
 import { describe, expect, it, test } from 'vitest'
 import { generateCodeFrame } from 'vitest/src/node/error.js'
 import { hoistMocks } from '../../../packages/mocker/src/node/hoistMocksPlugin'