fix(browser): allow importing BrowserCommand if no browser package is installed (#8666)

Author: sheremet-va

packages/browser/context.d.ts

@@ -1,5 +1,5 @@
 import { SerializedConfig } from 'vitest'
-import { StringifyOptions } from 'vitest/internal/browser'
+import { StringifyOptions, BrowserCommands } from 'vitest/internal/browser'
 import { ARIARole } from './aria-role.js'
 import {} from './matchers.js'
 
@@ -17,11 +17,6 @@ export type BufferEncoding =
   | 'binary'
   | 'hex'
 
-export interface FsOptions {
-  encoding?: BufferEncoding
-  flag?: string | number
-}
-
 export interface CDPSession {
   // methods are defined by the provider type augmentation
 }
@@ -168,19 +163,6 @@ export interface ScreenshotMatcherOptions<
   timeout?: number
 }
 
-export interface BrowserCommands {
-  readFile: (
-    path: string,
-    options?: BufferEncoding | FsOptions
-  ) => Promise<string>
-  writeFile: (
-    path: string,
-    content: string,
-    options?: BufferEncoding | (FsOptions & { mode?: number | string })
-  ) => Promise<void>
-  removeFile: (path: string) => Promise<void>
-}
-
 export interface UserEvent {
   /**
    * Creates a new user event instance. This is useful if you need to keep the
@@ -421,7 +403,7 @@ export interface LocatorByRoleOptions extends LocatorOptions {
 
 interface LocatorScreenshotOptions extends Omit<ScreenshotOptions, 'element'> {}
 
-interface LocatorSelectors {
+export interface LocatorSelectors {
   /**
    * Creates a way to locate an element by its [ARIA role](https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Roles), [ARIA attributes](https://developer.mozilla.org/en-US/docs/Web/Accessibility/ARIA/Attributes) and [accessible name](https://developer.mozilla.org/en-US/docs/Glossary/Accessible_name).
    * @see {@link https://vitest.dev/guide/browser/locators#getbyrole}

packages/browser/src/node/commands/fs.ts

@@ -1,5 +1,5 @@
+import type { BrowserCommands } from 'vitest/browser'
 import type { BrowserCommand, TestProject } from 'vitest/node'
-import type { BrowserCommands } from '../../../context'
 import fs, { promises as fsp } from 'node:fs'
 import { basename, dirname, resolve } from 'node:path'
 import mime from 'mime/lite'

packages/browser/src/node/project.ts

@@ -1,6 +1,7 @@
 import type { StackTraceParserOptions } from '@vitest/utils/source-map'
 import type { ViteDevServer } from 'vite'
 import type { ParsedStack, SerializedConfig, TestError } from 'vitest'
+import type { BrowserCommands } from 'vitest/browser'
 import type {
   BrowserCommand,
   BrowserCommandContext,
@@ -10,7 +11,6 @@ import type {
   TestProject,
   Vitest,
 } from 'vitest/node'
-import type { BrowserCommands } from '../../context'
 import type { ParentBrowserProject } from './projectParent'
 import { existsSync } from 'node:fs'
 import { readFile } from 'node:fs/promises'

packages/vitest/browser/context.d.ts

@@ -4,3 +4,4 @@ export * from '@vitest/browser-playwright/context'
 export * from '@vitest/browser-webdriverio/context'
 // @ts-ignore -- @vitest/browser-preview might not be installed
 export * from '@vitest/browser-preview/context'
+export { BrowserCommands, FsOptions } from 'vitest/internal/browser'

packages/vitest/package.json

@@ -188,7 +188,7 @@
     "tinyglobby": "catalog:",
     "tinypool": "^2.0.0",
     "tinyrainbow": "catalog:",
-    "vite": "^6.0.0 || ^7.0.0-0",
+    "vite": "^6.0.0 || ^7.0.0",
     "why-is-node-running": "^2.3.0"
   },
   "devDependencies": {
@@ -205,7 +205,6 @@
     "@types/picomatch": "^4.0.2",
     "@types/prompts": "^2.4.9",
     "@types/sinonjs__fake-timers": "^8.1.5",
-    "@vitest/browser-playwright": "workspace:*",
     "acorn-walk": "catalog:",
     "birpc": "catalog:",
     "cac": "catalog:",

packages/vitest/src/public/browser.ts

@@ -23,6 +23,24 @@ export {
   getOriginalPosition,
 } from '@vitest/utils/source-map'
 export { getSafeTimers, setSafeTimers } from '@vitest/utils/timers'
+
+export interface FsOptions {
+  encoding?: BufferEncoding
+  flag?: string | number
+}
+
+export interface BrowserCommands {
+  readFile: (
+    path: string,
+    options?: BufferEncoding | FsOptions
+  ) => Promise<string>
+  writeFile: (
+    path: string,
+    content: string,
+    options?: BufferEncoding | (FsOptions & { mode?: number | string })
+  ) => Promise<void>
+  removeFile: (path: string) => Promise<void>
+}
 /**
  * @internal
  */