chore: fix lint errors

sapphi-red · sapphi-red · commit 30d59612069f · 2025-03-31T18:11:21.000+09:00
diff --git a/.eslintrc.cjs b/.eslintrc.cjs
@@ -50,7 +50,7 @@ module.exports = defineConfig({
     'n/no-extraneous-import': [
       'error',
       {
-        allowModules: ['vite', 'less', 'sass', 'vitest'],
+        allowModules: ['vite', 'less', 'sass', 'vitest', '@playwright/test'],
       },
     ],
     'n/no-extraneous-require': [
@@ -127,7 +127,7 @@ module.exports = defineConfig({
       },
     },
     {
-      files: ['playground/**'],
+      files: ['playground/**', 'packages/plugin-react-swc/playground/**'],
       rules: {
         'n/no-extraneous-import': 'off',
         'n/no-extraneous-require': 'off',
@@ -150,7 +150,7 @@ module.exports = defineConfig({
       },
     },
     {
-      files: ['playground/**'],
+      files: ['playground/**', 'packages/plugin-react-swc/playground/**'],
       excludedFiles: '**/__tests__/**',
       rules: {
         'no-undef': 'off',
diff --git a/packages/plugin-react-swc/playground/decorators/__tests__/decorators.spec.ts b/packages/plugin-react-swc/playground/decorators/__tests__/decorators.spec.ts
@@ -1,5 +1,5 @@
 import { expect, test } from '@playwright/test'
-import { setupDevServer, setupBuildAndPreview } from '../../utils.ts'
+import { setupBuildAndPreview, setupDevServer } from '../../utils.ts'
 
 test('Decorators build', async ({ page }) => {
   const { testUrl, server } = await setupBuildAndPreview('decorators')
diff --git a/packages/plugin-react-swc/playground/decorators/src/App.tsx b/packages/plugin-react-swc/playground/decorators/src/App.tsx
@@ -1,4 +1,5 @@
-import { ComponentClass, Component } from 'react'
+import type { ComponentClass } from 'react'
+import { Component } from 'react'
 
 function decorated(target: ComponentClass) {
   const original = target.prototype.render
diff --git a/packages/plugin-react-swc/playground/emotion-plugin/__tests__/emotion-plugin.spec.ts b/packages/plugin-react-swc/playground/emotion-plugin/__tests__/emotion-plugin.spec.ts
@@ -1,9 +1,9 @@
 import { expect, test } from '@playwright/test'
 import {
-  setupDevServer,
+  expectColor,
   setupBuildAndPreview,
+  setupDevServer,
   setupWaitForLogs,
-  expectColor,
 } from '../../utils.ts'
 
 test('Emotion plugin build', async ({ page }) => {
diff --git a/packages/plugin-react-swc/playground/emotion/__tests__/emotion.spec.ts b/packages/plugin-react-swc/playground/emotion/__tests__/emotion.spec.ts
@@ -1,9 +1,9 @@
 import { expect, test } from '@playwright/test'
 import {
-  setupDevServer,
+  expectColor,
   setupBuildAndPreview,
+  setupDevServer,
   setupWaitForLogs,
-  expectColor,
 } from '../../utils.ts'
 
 test('Emotion build', async ({ page }) => {
diff --git a/packages/plugin-react-swc/playground/hmr/__tests__/hmr.spec.ts b/packages/plugin-react-swc/playground/hmr/__tests__/hmr.spec.ts
@@ -1,7 +1,7 @@
 import { expect, test } from '@playwright/test'
 import {
-  setupDevServer,
   setupBuildAndPreview,
+  setupDevServer,
   setupWaitForLogs,
 } from '../../utils.ts'
 
diff --git a/packages/plugin-react-swc/playground/mdx/__tests__/mdx.spec.ts b/packages/plugin-react-swc/playground/mdx/__tests__/mdx.spec.ts
@@ -1,7 +1,7 @@
 import { expect, test } from '@playwright/test'
 import {
-  setupDevServer,
   setupBuildAndPreview,
+  setupDevServer,
   setupWaitForLogs,
 } from '../../utils.ts'
 
diff --git a/packages/plugin-react-swc/playground/react-18/__tests__/react-18.spec.ts b/packages/plugin-react-swc/playground/react-18/__tests__/react-18.spec.ts
@@ -1,7 +1,7 @@
 import { expect, test } from '@playwright/test'
 import {
-  setupDevServer,
   setupBuildAndPreview,
+  setupDevServer,
   setupWaitForLogs,
 } from '../../utils.ts'
 
diff --git a/packages/plugin-react-swc/playground/styled-components/__tests__/styled-components.spec.ts b/packages/plugin-react-swc/playground/styled-components/__tests__/styled-components.spec.ts
@@ -1,9 +1,9 @@
 import { expect, test } from '@playwright/test'
 import {
-  setupDevServer,
+  expectColor,
   setupBuildAndPreview,
+  setupDevServer,
   setupWaitForLogs,
-  expectColor,
 } from '../../utils.ts'
 
 test('styled-components build', async ({ page }) => {
diff --git a/packages/plugin-react-swc/playground/ts-lib/__tests__/ts-lib.spec.ts b/packages/plugin-react-swc/playground/ts-lib/__tests__/ts-lib.spec.ts
@@ -1,5 +1,5 @@
 import { expect, test } from '@playwright/test'
-import { setupDevServer, setupBuildAndPreview } from '../../utils.ts'
+import { setupBuildAndPreview, setupDevServer } from '../../utils.ts'
 
 test('TS lib build', async ({ page }) => {
   const { testUrl, server } = await setupBuildAndPreview('ts-lib')
diff --git a/packages/plugin-react-swc/playground/utils.ts b/packages/plugin-react-swc/playground/utils.ts
@@ -1,12 +1,12 @@
-import { expect, type Locator, type Page } from '@playwright/test'
+import { readFileSync, writeFileSync } from 'node:fs'
+import { type Locator, type Page, expect } from '@playwright/test'
 import {
+  build,
   createServer,
   loadConfigFromFile,
   mergeConfig,
   preview,
-  build,
 } from 'vite'
-import { readFileSync, writeFileSync } from 'fs'
 
 export const setupWaitForLogs = async (page: Page) => {
   let logs: string[] = []
@@ -56,7 +56,7 @@ export const setupDevServer = async (name: string) => {
     ) => {
       const path = `${root}/${name}`
       let content = readFileSync(path, 'utf-8')
-      for (let [search, replace] of replacements) {
+      for (const [search, replace] of replacements) {
         if (!content.includes(search)) {
           throw new Error(`'${search}' not found in ${name}`)
         }
diff --git a/packages/plugin-react-swc/playground/worker/__tests__/worker.spec.ts b/packages/plugin-react-swc/playground/worker/__tests__/worker.spec.ts
@@ -1,7 +1,7 @@
 import { test } from '@playwright/test'
 import {
-  setupDevServer,
   setupBuildAndPreview,
+  setupDevServer,
   setupWaitForLogs,
 } from '../../utils.ts'
 
diff --git a/packages/plugin-react-swc/playwright.config.ts b/packages/plugin-react-swc/playwright.config.ts
@@ -1,5 +1,5 @@
 import { fileURLToPath } from 'node:url'
-import { devices, type PlaywrightTestConfig } from '@playwright/test'
+import { type PlaywrightTestConfig, devices } from '@playwright/test'
 import fs from 'fs-extra'
 
 const tempDir = fileURLToPath(new URL('playground-temp', import.meta.url))
diff --git a/packages/plugin-react-swc/scripts/bundle.ts b/packages/plugin-react-swc/scripts/bundle.ts
@@ -1,6 +1,6 @@
-import { rmSync, writeFileSync, copyFileSync } from 'node:fs'
+import { copyFileSync, rmSync, writeFileSync } from 'node:fs'
 import { execSync } from 'node:child_process'
-import { build, type BuildOptions, context } from 'esbuild'
+import { type BuildOptions, build, context } from 'esbuild'
 
 import packageJSON from '../package.json'
 
diff --git a/packages/plugin-react-swc/src/index.ts b/packages/plugin-react-swc/src/index.ts
@@ -1,17 +1,17 @@
-import { readFileSync } from 'fs'
-import { dirname, join } from 'path'
-import { fileURLToPath } from 'url'
-import type { SourceMapPayload } from 'module'
+import { readFileSync } from 'node:fs'
+import { dirname, join } from 'node:path'
+import { fileURLToPath } from 'node:url'
+import type { SourceMapPayload } from 'node:module'
+import { createRequire } from 'node:module'
 import {
+  type JscTarget,
   type Output,
   type ParserConfig,
   type ReactConfig,
-  type JscTarget,
-  transform,
   type Options as SWCOptions,
+  transform,
 } from '@swc/core'
-import type { PluginOption, UserConfig, BuildOptions } from 'vite'
-import { createRequire } from 'module'
+import type { BuildOptions, PluginOption, UserConfig } from 'vite'
 
 const runtimePublicPath = '/@react-refresh'
 
@@ -20,13 +20,16 @@ injectIntoGlobalHook(window);
 window.$RefreshReg$ = () => {};
 window.$RefreshSig$ = () => (type) => type;`
 
+/* eslint-disable no-restricted-globals */
 const _dirname =
   typeof __dirname !== 'undefined'
     ? __dirname
     : dirname(fileURLToPath(import.meta.url))
 const resolve = createRequire(
   typeof __filename !== 'undefined' ? __filename : import.meta.url,
 ).resolve
+/* eslint-enable no-restricted-globals */
+
 const reactCompRE = /extends\s+(?:React\.)?(?:Pure)?Component/
 const refreshContentRE = /\$Refresh(?:Reg|Sig)\$\(/
 
@@ -265,7 +268,7 @@ const transformWithOptions = async (
     const message: string = e.message
     const fileStartIndex = message.indexOf('╭─[')
     if (fileStartIndex !== -1) {
-      const match = message.slice(fileStartIndex).match(/:(\d+):(\d+)]/)
+      const match = message.slice(fileStartIndex).match(/:(\d+):(\d+)\]/)
       if (match) {
         e.line = match[1]
         e.column = match[2]
diff --git a/packages/plugin-react-swc/src/refresh-runtime.js b/packages/plugin-react-swc/src/refresh-runtime.js
@@ -1,3 +1,6 @@
+/* global window */
+/* eslint-disable eqeqeq, prefer-const, @typescript-eslint/no-empty-function */
+
 /*! Copyright (c) Meta Platforms, Inc. and affiliates. **/
 /**
  * This is simplified pure-js version of https://github.com/facebook/react/blob/main/packages/react-refresh/src/ReactFreshRuntime.js
@@ -511,7 +514,7 @@ function isLikelyComponentType(type) {
           // This looks like a class.
           return false
         }
-        // eslint-disable-next-line no-proto
+
         if (type.prototype.__proto__ !== Object.prototype) {
           // It has a superclass.
           return false
