Skip to content

Commit 33d0538

Browse files
authored
fix: remove unused exports (#7618)
1 parent 95d7a33 commit 33d0538

File tree

25 files changed

+213
-91
lines changed

25 files changed

+213
-91
lines changed

packages/browser/rollup.config.js

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ const external = [
1313
...Object.keys(pkg.dependencies),
1414
...Object.keys(pkg.peerDependencies || {}),
1515
/^@?vitest(\/|$)/,
16+
'@vitest/browser/utils',
1617
'worker_threads',
1718
'node:worker_threads',
1819
'vite',
@@ -98,6 +99,7 @@ export default () =>
9899
file: 'dist/context.js',
99100
format: 'esm',
100101
},
102+
external: ['@vitest/browser/utils'],
101103
plugins: [
102104
oxc({
103105
transform: { target: 'node18' },

packages/browser/src/client/tester/logger.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
import { format, stringify } from 'vitest/utils'
1+
import { format, stringify } from 'vitest/internal/browser'
22
import { getConfig } from '../utils'
33
import { rpc } from './rpc'
44
import { getBrowserRunner } from './runner'

packages/browser/src/client/tester/public-utils.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
import type { Locator, LocatorSelectors } from '@vitest/browser/context'
2-
import type { StringifyOptions } from 'vitest/utils'
2+
import type { StringifyOptions } from 'vitest/internal/browser'
33
import { page } from '@vitest/browser/context'
44
import { asLocator } from 'ivya'
5-
import { stringify } from 'vitest/utils'
5+
import { stringify } from 'vitest/internal/browser'
66

77
export function getElementLocatorSelectors(element: Element): LocatorSelectors {
88
const locator = page.elementLocator(element)

packages/browser/src/client/tester/rpc.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import type { VitestBrowserClient } from '@vitest/browser/client'
2-
import { getSafeTimers } from 'vitest/utils'
2+
import { getSafeTimers } from 'vitest/internal/browser'
33

44
const { get } = Reflect
55

packages/browser/src/client/tester/runner.ts

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,14 @@ import type { VitestExecutor } from 'vitest/execute'
44
import type { VitestBrowserClientMocker } from './mocker'
55
import { globalChannel, onCancel } from '@vitest/browser/client'
66
import { page, userEvent } from '@vitest/browser/context'
7-
import { loadDiffConfig, loadSnapshotSerializers, takeCoverageInsideWorker } from 'vitest/browser'
7+
import {
8+
loadDiffConfig,
9+
loadSnapshotSerializers,
10+
originalPositionFor,
11+
takeCoverageInsideWorker,
12+
TraceMap,
13+
} from 'vitest/internal/browser'
814
import { NodeBenchmarkRunner, VitestTestRunner } from 'vitest/runners'
9-
import { originalPositionFor, TraceMap } from 'vitest/utils'
1015
import { createStackString, parseStacktrace } from '../../../../utils/src/source-map'
1116
import { executor, getWorkerState } from '../utils'
1217
import { rpc } from './rpc'

packages/browser/src/client/tester/snapshot.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import type { VitestBrowserClient } from '@vitest/browser/client'
2+
import type { ParsedStack } from 'vitest/internal/browser'
23
import type { SnapshotEnvironment } from 'vitest/snapshot'
3-
import type { ParsedStack } from 'vitest/utils'
4-
import { originalPositionFor, TraceMap } from 'vitest/utils'
4+
import { originalPositionFor, TraceMap } from 'vitest/internal/browser'
55

66
export class VitestBrowserSnapshotEnvironment implements SnapshotEnvironment {
77
private sourceMaps = new Map<string, any>()

packages/browser/src/client/tester/tester.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import {
99
startCoverageInsideWorker,
1010
startTests,
1111
stopCoverageInsideWorker,
12-
} from 'vitest/browser'
12+
} from 'vitest/internal/browser'
1313
import { executor, getBrowserState, getConfig, getWorkerState } from '../utils'
1414
import { setupDialogsSpy } from './dialog'
1515
import { setupConsoleLogSpy } from './logger'

packages/browser/src/node/plugin.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -190,8 +190,7 @@ export default (parentServer: ParentBrowserProject, base = '/'): Plugin[] => {
190190

191191
const exclude = [
192192
'vitest',
193-
'vitest/utils',
194-
'vitest/browser',
193+
'vitest/internal/browser',
195194
'vitest/runners',
196195
'@vitest/browser',
197196
'@vitest/browser/client',

packages/browser/utils.d.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
// fortunately, the file is quite small
44

55
import { LocatorSelectors, Locator } from '@vitest/browser/context'
6-
import { StringifyOptions } from 'vitest/utils'
6+
import { StringifyOptions } from 'vitest/internal/browser'
77

88
export type PrettyDOMOptions = Omit<StringifyOptions, 'maxLength'>
99

packages/utils/src/display.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ interface Options {
2121
stylize: (value: string, styleType: string) => string
2222
}
2323

24-
type LoupeOptions = Partial<Options>
24+
export type LoupeOptions = Partial<Options>
2525

2626
const {
2727
AsymmetricMatcher,

packages/utils/src/index.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ export {
44
objDisplay,
55
stringify,
66
} from './display'
7-
export type { StringifyOptions } from './display'
7+
export type { LoupeOptions, StringifyOptions } from './display'
88

99
export {
1010
assertTypes,

packages/vitest/package.json

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@
5858
"types": "./dist/workers.d.ts",
5959
"import": "./dist/workers.js"
6060
},
61-
"./browser": {
61+
"./internal/browser": {
6262
"types": "./dist/browser.d.ts",
6363
"default": "./dist/browser.js"
6464
},
@@ -74,10 +74,6 @@
7474
"types": "./dist/environments.d.ts",
7575
"default": "./dist/environments.js"
7676
},
77-
"./utils": {
78-
"types": "./dist/utils.d.ts",
79-
"default": "./dist/utils.js"
80-
},
8177
"./config": {
8278
"types": "./config.d.ts",
8379
"require": "./dist/config.cjs",

packages/vitest/rollup.config.js

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ const entries = {
2828
'mocker': 'src/public/mocker.ts',
2929
'spy': 'src/integrations/spy.ts',
3030
'coverage': 'src/public/coverage.ts',
31-
'utils': 'src/public/utils.ts',
3231
'execute': 'src/public/execute.ts',
3332
'reporters': 'src/public/reporters.ts',
3433
// TODO: advanced docs
@@ -55,7 +54,6 @@ const dtsEntries = {
5554
suite: 'src/public/suite.ts',
5655
config: 'src/public/config.ts',
5756
coverage: 'src/public/coverage.ts',
58-
utils: 'src/public/utils.ts',
5957
execute: 'src/public/execute.ts',
6058
reporters: 'src/public/reporters.ts',
6159
mocker: 'src/public/mocker.ts',

packages/vitest/src/integrations/run-once.ts

Lines changed: 0 additions & 52 deletions
This file was deleted.

packages/vitest/src/integrations/utils.ts

Lines changed: 0 additions & 9 deletions
This file was deleted.

packages/vitest/src/node/test-run.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
import type { File as RunnerTestFile, TaskEventPack, TaskResultPack, TaskUpdateEvent } from '@vitest/runner'
2-
import type { SerializedError } from '../public/utils'
2+
import type { SerializedError } from '@vitest/utils'
33
import type { UserConsoleLog } from '../types/general'
44
import type { Vitest } from './core'
55
import type { TestProject } from './project'

packages/vitest/src/public/browser.ts

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,3 +11,14 @@ export {
1111
} from '../runtime/setup-common'
1212
export { collectTests, processError, startTests } from '@vitest/runner'
1313
export * as SpyModule from '@vitest/spy'
14+
export {
15+
format,
16+
getSafeTimers,
17+
inspect,
18+
stringify,
19+
} from '@vitest/utils'
20+
export type { LoupeOptions, ParsedStack, StringifyOptions } from '@vitest/utils'
21+
export {
22+
originalPositionFor,
23+
TraceMap,
24+
} from '@vitest/utils/source-map'

packages/vitest/src/public/index.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -107,9 +107,7 @@ export type {
107107
} from '../api/types'
108108
export { assert, chai, createExpect, expect, should } from '../integrations/chai'
109109
export { inject } from '../integrations/inject'
110-
export { isFirstRun, runOnce } from '../integrations/run-once'
111110

112-
export { getRunningMode, isWatchMode } from '../integrations/utils'
113111
export { vi, vitest } from '../integrations/vi'
114112
export type { VitestUtils } from '../integrations/vi'
115113
export { bench } from '../runtime/benchmark'

packages/vitest/src/public/utils.ts

Lines changed: 0 additions & 2 deletions
This file was deleted.

packages/vitest/src/runtime/setup-common.ts

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,9 @@ import type { SerializedConfig } from './config'
55
import type { VitestExecutor } from './execute'
66
import { addSerializer } from '@vitest/snapshot'
77
import { setSafeTimers } from '@vitest/utils'
8-
import { resetRunOnceCounter } from '../integrations/run-once'
98

109
let globalSetup = false
1110
export async function setupCommonEnv(config: SerializedConfig): Promise<void> {
12-
resetRunOnceCounter()
1311
setupDefines(config.defines)
1412
setupEnv(config.env)
1513

pnpm-lock.yaml

Lines changed: 11 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

test/browser/test/utils.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ import { commands } from '@vitest/browser/context'
22
import { prettyDOM } from '@vitest/browser/utils'
33
import { afterEach, expect, it, test } from 'vitest'
44

5-
import { inspect } from 'vitest/utils'
5+
import { inspect } from 'vitest/internal/browser'
66

77
afterEach(() => {
88
document.body.innerHTML = ''

test/core/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@
3636
"vite-node": "workspace:*",
3737
"vitest": "workspace:*",
3838
"vitest-environment-custom": "file:./vitest-environment-custom",
39+
"vitest-package-exports": "^0.1.1",
3940
"vue": "catalog:",
4041
"zustand": "^4.5.6"
4142
}

0 commit comments

Comments
 (0)