fix: guard disposable and optional body

packages/runner/src/types/tasks.ts

@@ -1262,7 +1262,7 @@ export interface TestAttachment {
   /** File system path to the attachment */
   path?: string
   /** Inline attachment content as a string or raw binary data */
-  body?: string | Uint8Array
+  body?: string | Uint8Array | undefined
 }
 
 export interface Location {

packages/spy/optional-types.d.ts

@@ -0,0 +1,6 @@
+/* eslint-disable ts/ban-ts-comment */
+
+export interface Disposable {
+  // @ts-ignore -- Symbol.dispose might not be in user types
+  [Symbol.dispose]: () => void
+}

packages/spy/package.json

@@ -27,13 +27,17 @@
       "types": "./dist/index.d.ts",
       "default": "./dist/index.js"
     },
+    "./optional-types.js": {
+      "types": "./optional-types.d.ts"
+    },
     "./*": "./*"
   },
   "main": "./dist/index.js",
   "module": "./dist/index.js",
   "types": "./dist/index.d.ts",
   "files": [
-    "dist"
+    "dist",
+    "optional-types.d.ts"
   ],
   "scripts": {
     "build": "premove dist && rollup -c",

packages/spy/rollup.config.js

@@ -10,6 +10,7 @@ const external = [
   ...builtinModules,
   ...Object.keys(pkg.dependencies || {}),
   ...Object.keys(pkg.peerDependencies || {}),
+  '@vitest/spy/optional-types.js',
 ]
 
 const dtsUtils = createDtsUtils()

packages/spy/src/types.ts

@@ -1,3 +1,5 @@
+import type { Disposable } from '@vitest/spy/optional-types.js'
+
 export interface MockResultReturn<T> {
   type: 'return'
   /**

packages/vitest/optional-runtime-types.d.ts

@@ -0,0 +1,6 @@
+/* eslint-disable ts/ban-ts-comment */
+
+export interface Disposable {
+  // @ts-ignore -- Symbol.dispose might not be in user types
+  [Symbol.dispose]: () => void
+}

packages/vitest/package.json

@@ -48,6 +48,9 @@
     "./optional-types.js": {
       "types": "./optional-types.d.ts"
     },
+    "./optional-runtime-types.js": {
+      "types": "./optional-runtime-types.d.ts"
+    },
     "./src/*": "./src/*",
     "./globals": {
       "types": "./globals.d.ts"

packages/vitest/rollup.config.js

@@ -75,6 +75,7 @@ const external = [
   'node:console',
   'node:events',
   'inspector',
+  'vitest/optional-runtime-types.js',
   'vitest/optional-types.js',
   'vitest/browser',
   'vite/module-runner',

packages/vitest/src/integrations/vi.ts

@@ -6,6 +6,7 @@ import type {
   MaybePartiallyMockedDeep,
   MockInstance,
 } from '@vitest/spy'
+import type { Disposable } from 'vitest/optional-runtime-types.js'
 import type { RuntimeOptions, SerializedConfig } from '../runtime/config'
 import type { VitestMocker } from '../runtime/moduleRunner/moduleMocker'
 import type { MockFactoryWithHelper, MockOptions } from '../types/mocker'

test/cli/dts/exact-optional-property/src/runner.ts → test/cli/dts/exact-optional-property-no-node/src/runner.ts

@@ -1,4 +1,7 @@
 import type { VitestRunner } from '@vitest/runner'
+import type { FailureScreenshotArtifact } from '@vitest/runner/types'
 import { VitestTestRunner } from 'vitest/runners'
 
+type _Override = FailureScreenshotArtifact
+
 export class MyRunner extends VitestTestRunner implements VitestRunner {}

test/cli/dts/exact-optional-property-no-node/tsconfig.json

@@ -0,0 +1,17 @@
+{
+  "extends": ["../_shared/tsconfig.patch.json"],
+  "compilerOptions": {
+    "target": "ESNext",
+    "module": "ESNext",
+    "moduleResolution": "Bundler",
+    "types": [],
+    "strict": true,
+    "exactOptionalPropertyTypes": true,
+    "declaration": true,
+    "declarationMap": true,
+    "outDir": "dist",
+    "sourceMap": true,
+    "verbatimModuleSyntax": true,
+    "skipLibCheck": false
+  }
+}

test/cli/dts/exact-optional-property-node/package.json

@@ -0,0 +1,12 @@
+{
+  "name": "@vitest/test-integration-dts-exact-optional-property",
+  "type": "module",
+  "private": true,
+  "scripts": {
+    "test": "tsc -b"
+  },
+  "devDependencies": {
+    "@vitest/runner": "workspace:*",
+    "vitest": "workspace:*"
+  }
+}

test/cli/dts/exact-optional-property/src/reporter.ts → test/cli/dts/exact-optional-property-node/src/reporter.ts

@@ -1,5 +1,5 @@
 import type { File } from '@vitest/runner'
-import { DefaultReporter } from 'vitest/reporters'
+import { DefaultReporter } from 'vitest/node'
 
 export class MyReporter extends DefaultReporter {
   override reportTestSummary(files: File[], errors: unknown[], leakCount: number): void {

test/cli/dts/exact-optional-property/tsconfig.json → test/cli/dts/exact-optional-property-node/tsconfig.json

@@ -4,6 +4,7 @@
     "target": "ESNext",
     "module": "ESNext",
     "moduleResolution": "Bundler",
+    "types": [],
     "strict": true,
     "exactOptionalPropertyTypes": true,
     "declaration": true,

test/cli/dts/no-dispose/package.json

@@ -0,0 +1,12 @@
+{
+  "name": "@vitest/test-integration-dts-exact-optional-property",
+  "type": "module",
+  "private": true,
+  "scripts": {
+    "test": "tsc -b"
+  },
+  "devDependencies": {
+    "@vitest/runner": "workspace:*",
+    "vitest": "workspace:*"
+  }
+}

test/cli/dts/no-dispose/src/vitest.ts

@@ -0,0 +1,4 @@
+import { vi } from 'vitest'
+
+const _fn = vi.fn()
+const _mock = vi.doMock('./vitest.ts')

test/cli/dts/no-dispose/tsconfig.json

@@ -0,0 +1,17 @@
+{
+  "extends": ["../_shared/tsconfig.patch.json"],
+  "compilerOptions": {
+    "target": "es2020",
+    "lib": ["DOM", "ES2020"],
+    "module": "node16",
+    "types": [],
+    "strict": true,
+    "declaration": true,
+    "declarationMap": true,
+    "outDir": "dist",
+    "sourceMap": true,
+    "verbatimModuleSyntax": true,
+    "skipLibCheck": false
+  },
+  "include": ["src/*.ts"]
+}