refactor

Author: baseballyama

Diff for: packages/eslint-plugin-svelte/src/rules/valid-compile.ts

@@ -4,6 +4,33 @@ import { getSvelteCompileWarnings } from '../shared/svelte-compile-warns/index.j
 import { getSourceCode } from '../utils/compat.js';
 import type { Position } from 'svelte-eslint-parser/lib/ast/common.js';
 
+const ignores: string[] = [
+	'missing-declaration',
+	// Svelte v4
+	'dynamic-slot-name',
+	// Svelte v5
+	'invalid-slot-name'
+] as const;
+
+const unusedSelectorWarnings: string[] = ['css_unused_selector', 'css-unused-selector'] as const;
+
+function isGlobalStyleNode(
+	globalStyleRanges: [Position, Position][],
+	start?: Position,
+	end?: Position
+) {
+	if (start == null || end == null) {
+		return false;
+	}
+	return globalStyleRanges.some(([rangeStart, rangeEnd]) => {
+		return (
+			(rangeStart.line < start.line ||
+				(rangeStart.line === start.line && rangeStart.column <= start.column)) &&
+			(end.line < rangeEnd.line || (end.line === rangeEnd.line && end.column <= rangeEnd.column))
+		);
+	});
+}
+
 export default createRule('valid-compile', {
 	meta: {
 		docs: {
@@ -40,32 +67,8 @@ export default createRule('valid-compile', {
 			: (warning) => warning;
 
 		const ignoreWarnings = Boolean(context.options[0]?.ignoreWarnings);
-
-		const ignores = [
-			'missing-declaration',
-			// Svelte v4
-			'dynamic-slot-name',
-			// Svelte v5
-			'invalid-slot-name'
-		];
-
-		const unusedSelectorWarnings = ['css_unused_selector', 'css-unused-selector'];
 		const globalStyleRanges: [Position, Position][] = [];
 
-		function isGlobalStyleNode(start?: Position, end?: Position) {
-			if (start == null || end == null) {
-				return false;
-			}
-			return globalStyleRanges.some(([rangeStart, rangeEnd]) => {
-				return (
-					(rangeStart.line < start.line ||
-						(rangeStart.line === start.line && rangeStart.column <= start.column)) &&
-					(end.line < rangeEnd.line ||
-						(end.line === rangeEnd.line && end.column <= rangeEnd.column))
-				);
-			});
-		}
-
 		/**
 		 * report
 		 */
@@ -74,7 +77,8 @@ export default createRule('valid-compile', {
 				if (
 					warn.code &&
 					(ignores.includes(warn.code) ||
-						(isGlobalStyleNode(warn.start, warn.end) && unusedSelectorWarnings.includes(warn.code)))
+						(isGlobalStyleNode(globalStyleRanges, warn.start, warn.end) &&
+							unusedSelectorWarnings.includes(warn.code)))
 				) {
 					continue;
 				}