chore(deps): update dependency eslint-plugin-unicorn to v57 (#5852)

* chore(deps): update dependency eslint-plugin-unicorn to v57

* Change names to satisy linter

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Lukas Taegert-Atkinson <[email protected]>

Author: renovate[bot]

package-lock.json

@@ -159,7 +159,7 @@
     "eslint": "^9.21.0",
     "eslint-config-prettier": "^10.0.1",
     "eslint-plugin-prettier": "^5.2.3",
-    "eslint-plugin-unicorn": "^56.0.1",
+    "eslint-plugin-unicorn": "^57.0.0",
     "eslint-plugin-vue": "^9.32.0",
     "fixturify": "^3.0.0",
     "flru": "^1.0.2",

package.json

@@ -1,14 +1,12 @@
 import { version as rollupVersion } from 'package.json';
 import Bundle from '../Bundle';
 import Graph from '../Graph';
-import type { PluginDriver } from '../utils/PluginDriver';
-import { getSortedValidatedPlugins } from '../utils/PluginDriver';
 import { mkdir, writeFile } from '../utils/fs';
 import { catchUnfinishedHookActions } from '../utils/hookActions';
 import initWasm from '../utils/initWasm';
-import { getLogHandler } from '../utils/logHandler';
 import { getLogger } from '../utils/logger';
 import { LOGLEVEL_DEBUG, LOGLEVEL_INFO, LOGLEVEL_WARN } from '../utils/logging';
+import { getLogHandler } from '../utils/logHandler';
 import {
 	error,
 	logAlreadyClosed,
@@ -20,7 +18,9 @@ import { normalizeInputOptions } from '../utils/options/normalizeInputOptions';
 import { normalizeOutputOptions } from '../utils/options/normalizeOutputOptions';
 import { getOnLog, normalizeLog, normalizePluginOption } from '../utils/options/options';
 import { dirname, resolve } from '../utils/path';
-import { ANONYMOUS_OUTPUT_PLUGIN_PREFIX, ANONYMOUS_PLUGIN_PREFIX } from '../utils/pluginUtils';
+import type { PluginDriver } from '../utils/PluginDriver';
+import { getSortedValidatedPlugins } from '../utils/PluginDriver';
+import { ANONYMOUS_OUTPUT_PLUGIN_PREFIX, ANONYMOUS_PLUGIN_PREFIX } from '../utils/pluginNames';
 import { getTimings, initialiseTimers, timeEnd, timeStart } from '../utils/timers';
 import type {
 	InputOptions,

src/rollup/rollup.ts

@@ -1,6 +1,6 @@
 import type { PluginCache, SerializablePluginCache } from '../rollup/types';
 import { error, logAnonymousPluginCache, logDuplicatePluginName } from './logs';
-import { ANONYMOUS_OUTPUT_PLUGIN_PREFIX, ANONYMOUS_PLUGIN_PREFIX } from './pluginUtils';
+import { ANONYMOUS_OUTPUT_PLUGIN_PREFIX, ANONYMOUS_PLUGIN_PREFIX } from './pluginNames';
 
 export function createPluginCache(cache: SerializablePluginCache): PluginCache {
 	return {

src/utils/PluginCache.ts

@@ -15,7 +15,7 @@ import { error, logPluginError } from './logs';
 import { normalizeLog } from './options/options';
 import { parseAst } from './parseAst';
 import { createPluginCache, getCacheForUncacheablePlugin, NO_CACHE } from './PluginCache';
-import { ANONYMOUS_OUTPUT_PLUGIN_PREFIX, ANONYMOUS_PLUGIN_PREFIX } from './pluginUtils';
+import { ANONYMOUS_OUTPUT_PLUGIN_PREFIX, ANONYMOUS_PLUGIN_PREFIX } from './pluginNames';
 
 export function getPluginContext(
 	plugin: Plugin,

src/utils/PluginContext.ts

@@ -14,7 +14,7 @@ const fixturify = require('fixturify');
  * @type {import('../../src/rollup/types')} Rollup
  */
 const { rollup } = require('../../browser/dist/rollup.browser.js');
-const { assertFilesAreEqual, runTestSuiteWithSamples, compareError } = require('../utils.js');
+const { assertFilesAreEqual, runTestSuiteWithSamples, compareError } = require('../testHelpers.js');
 
 runTestSuiteWithSamples(
 	'browser',

src/utils/pluginUtils.ts src/utils/pluginNames.ts

@@ -1,4 +1,4 @@
-const { loader } = require('../../../utils.js');
+const { loader } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'bundles files for the browser',

test/browser/index.js

@@ -1,4 +1,4 @@
-const { loader } = require('../../../utils.js');
+const { loader } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'fails if a dependency cannot be resolved',

test/browser/samples/basic/_config.js

@@ -1,4 +1,4 @@
-const { loader } = require('../../../utils.js');
+const { loader } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'supports hashes in the browser build',

test/browser/samples/missing-dependency-resolution/_config.js

@@ -4,8 +4,8 @@ const path = require('node:path');
  */
 // @ts-expect-error not included in types
 const { rollup } = require('../../dist/rollup');
-const { compareLogs } = require('../utils');
-const { runTestSuiteWithSamples, assertDirectoriesAreEqual } = require('../utils.js');
+const { compareLogs } = require('../testHelpers');
+const { runTestSuiteWithSamples, assertDirectoriesAreEqual } = require('../testHelpers.js');
 
 const FORMATS = ['es', 'cjs', 'amd', 'system'];
 

test/browser/samples/supports-hashes/_config.js

@@ -1,4 +1,4 @@
-const { loader } = require('../../../../utils.js');
+const { loader } = require('../../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'deconflicts files against named files',

test/chunking-form/index.js

@@ -1,6 +1,6 @@
 const assert = require('node:assert');
 const path = require('node:path');
-const { getObject } = require('../../../utils');
+const { getObject } = require('../../../testHelpers');
 
 module.exports = defineTest({
 	description: 'associates empty modules with chunks if tree-shaking is disabled for them',

test/chunking-form/samples/emit-file/deconflict-against-named-files/_config.js

@@ -1,4 +1,4 @@
-const { loader } = require('../../../utils.js');
+const { loader } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'ignores queries and hashes for chunk names when preserving modules',

test/chunking-form/samples/empty-module-no-treeshake/_config.js

@@ -1,6 +1,6 @@
 const assert = require('node:assert');
 const { promises: fs } = require('node:fs');
-const { wait } = require('../../../utils');
+const { wait } = require('../../../testHelpers');
 
 const fsWriteFile = fs.writeFile;
 let currentWrites = 0;

test/chunking-form/samples/ignore-chunk-name-query/_config.js

@@ -1,5 +1,5 @@
 const assert = require('node:assert');
-const { replaceDirectoryInStringifiedObject } = require('../../../utils');
+const { replaceDirectoryInStringifiedObject } = require('../../../testHelpers');
 
 module.exports = defineTest({
 	description: 'replaces hashes when mutating chunk info in renderChunk',

test/chunking-form/samples/max-parallel-file-operations/_config.js

@@ -1,5 +1,5 @@
 const assert = require('node:assert');
-const { replaceDirectoryInStringifiedObject } = require('../../../utils');
+const { replaceDirectoryInStringifiedObject } = require('../../../testHelpers');
 
 module.exports = defineTest({
 	description:

test/chunking-form/samples/render-chunk-transform/_config.js

@@ -8,7 +8,7 @@ const {
 	normaliseOutput,
 	runTestSuiteWithSamples,
 	assertDirectoriesAreEqual
-} = require('../utils.js');
+} = require('../testHelpers.js');
 
 const cwd = process.cwd();
 const rollupBinary = `${path.resolve(__dirname, '../../dist/bin')}${path.sep}rollup`;

test/chunking-form/samples/render-chunk/_config.js

@@ -1,5 +1,5 @@
 // @ts-expect-error not included in types
-const { hasEsBuild } = require('../../../utils');
+const { hasEsBuild } = require('../../../testHelpers');
 
 module.exports = defineTest({
 	skip: !hasEsBuild,

test/cli/index.js

@@ -1,5 +1,5 @@
 // @ts-expect-error not included in types
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'tries to load .js config file if package type is "module"',

test/cli/samples/config-defineConfig-mjs/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'displays warnings when a config is loaded',

test/cli/samples/config-type-module/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'custom (plugin generated) code frame taking priority over pos generated one',

test/cli/samples/config-warnings/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'errors with plugin generated code frames also contain stack',

test/cli/samples/custom-frame-with-pos/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'throws if different types of entries are combined',

test/cli/samples/custom-frame/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'shows warning when multiple chunks empty',

test/cli/samples/duplicate-import-options/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'shows warning when chunk empty',

test/cli/samples/empty-chunk-multiple/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'errors on warnings with --failAfterWarnings',

test/cli/samples/empty-chunk/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'overrides the generatedCode option when using presets',

test/cli/samples/fail-after-warnings/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'prints error cause',

test/cli/samples/generated-code-unknown-preset/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'handles uncaught errors under watch',

test/cli/samples/handles-errors-cause/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'handles uncaught errors',

test/cli/samples/handles-uncaught-errors-under-watch/_config.js

@@ -1,5 +1,5 @@
 const path = require('node:path');
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'logs side effects',

test/cli/samples/handles-uncaught-errors/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'throws if a config in node_modules cannot be found',

test/cli/samples/log-side-effects/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../../utils.js');
+const { assertIncludes } = require('../../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'unknown CLI --plugin results in an error',

test/cli/samples/node-config-not-found/_config.js

@@ -1,5 +1,5 @@
 const path = require('node:path');
-const { assertIncludes } = require('../../../../utils.js');
+const { assertIncludes } = require('../../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'Throws when the plugin export cannot be found',

test/cli/samples/plugin/cannot-load/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../../utils.js');
+const { assertIncludes } = require('../../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'invalid CLI --plugin argument format',

test/cli/samples/plugin/export-not-found/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../../utils.js');
+const { assertIncludes } = require('../../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'does not use input as stdin on TTY interfaces',

test/cli/samples/plugin/invalid-argument/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../../utils.js');
+const { assertIncludes } = require('../../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'handles stdin errors',

test/cli/samples/stdin/no-stdin-tty/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'fails when using non-inline sourcemaps when bundling to stdout',

test/cli/samples/stdin/stdin-error/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'overrides the treeshake option when using presets',

test/cli/samples/stdout-only-inline-sourcemaps/_config.js

@@ -1,5 +1,5 @@
 const assert = require('node:assert');
-const { assertIncludes, assertDoesNotInclude, hasEsBuild } = require('../../../utils.js');
+const { assertIncludes, assertDoesNotInclude, hasEsBuild } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	skip: !hasEsBuild,

test/cli/samples/treeshake-unknown-preset/_config.js

@@ -1,6 +1,6 @@
 const assert = require('node:assert');
-const { assertDoesNotInclude } = require('../../../utils');
-const { assertIncludes } = require('../../../utils.js');
+const { assertDoesNotInclude } = require('../../../testHelpers');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description:

test/cli/samples/unfulfilled-hook-actions-error/_config.js

@@ -1,5 +1,5 @@
 const assert = require('node:assert');
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'show errors with non-zero exit code for unfulfilled async plugin actions on exit',

test/cli/samples/unfulfilled-hook-actions-manual-exit/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'use CLI --validate to test whether output is well formed',

test/cli/samples/unfulfilled-hook-actions/_config.js

@@ -1,6 +1,6 @@
 const { unlinkSync } = require('node:fs');
 const path = require('node:path');
-const { atomicWriteFileSync } = require('../../../utils');
+const { atomicWriteFileSync } = require('../../../testHelpers');
 
 let second;
 let third;

test/cli/samples/validate/_config.js

@@ -1,6 +1,6 @@
 const { unlinkSync } = require('node:fs');
 const path = require('node:path');
-const { atomicWriteFileSync } = require('../../../utils');
+const { atomicWriteFileSync } = require('../../../testHelpers');
 
 let mainFile;
 

test/cli/samples/wait-for-bundle-input-object/_config.js

@@ -1,6 +1,6 @@
 const { unlinkSync } = require('node:fs');
 const path = require('node:path');
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'displays warnings for broken sourcemaps',

test/cli/samples/wait-for-bundle-input/_config.js

@@ -1,6 +1,6 @@
 const { unlinkSync } = require('node:fs');
 const path = require('node:path');
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'displays warnings for broken sourcemaps',

test/cli/samples/warn-broken-sourcemap/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'warns for multiple circular dependencies',

test/cli/samples/warn-broken-sourcemaps/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'warns for circular dependencies',

test/cli/samples/warn-circular-multiple/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'warns when eval is used multiple times',

test/cli/samples/warn-circular/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'warns when eval is used',

test/cli/samples/warn-eval-multiple/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'warns about import and export related issues',

test/cli/samples/warn-eval/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'warns when there are multiple missing globals',

test/cli/samples/warn-import-export/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'warns when there is a missing global variable name',

test/cli/samples/warn-missing-global-multiple/_config.js

@@ -1,4 +1,4 @@
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'warns when mixed exports are used',

test/cli/samples/warn-missing-global/_config.js

@@ -1,5 +1,5 @@
 const path = require('node:path');
-const { assertIncludes } = require('../../../utils.js');
+const { assertIncludes } = require('../../../testHelpers.js');
 
 module.exports = defineTest({
 	description: 'aggregates warnings of different types',