move to more appropriate location, give function a more appropriate name

Author: Rich-Harris

packages/kit/src/exports/vite/index.js

@@ -37,7 +37,7 @@ import {
 import { import_peer } from '../../utils/import.js';
 import { compact } from '../../utils/array.js';
 import { build_remotes, treeshake_prerendered_remotes } from './build/build_remote.js';
-import { is_within_comment } from './comment_utils.js';
+import { should_ignore } from './static_analysis/utils.js';
 
 const cwd = process.cwd();
 
@@ -91,7 +91,7 @@ const warning_preprocessor = {
 		const basename = path.basename(filename);
 		if (basename.startsWith('+page.') || basename.startsWith('+layout.')) {
 			const match = content.match(options_regex);
-			if (match && match.index !== undefined && !is_within_comment(content, match.index)) {
+			if (match && match.index !== undefined && !should_ignore(content, match.index)) {
 				const fixed = basename.replace('.svelte', '(.server).js/ts');
 
 				const message =

packages/kit/src/exports/vite/comment_utils.js renamed to packages/kit/src/exports/vite/static_analysis/utils.js

@@ -1,10 +1,10 @@
 /**
- * Check if a match position is within a comment, accounting for strings and escaped characters
+ * Check if a match position is within a comment or a string
  * @param {string} content - The full content
  * @param {number} match_index - The index where the match starts
  * @returns {boolean} - True if the match is within a comment
  */
-export function is_within_comment(content, match_index) {
+export function should_ignore(content, match_index) {
 	// Track if we're inside different types of quotes and comments
 	let in_single_quote = false;
 	let in_double_quote = false;

packages/kit/src/exports/vite/comment_utils.spec.js renamed to packages/kit/src/exports/vite/static_analysis/utils.spec.js

@@ -1,6 +1,6 @@
 import { expect, test, vi } from 'vitest';
 import path from 'node:path';
-import { is_within_comment } from './comment_utils.js';
+import { should_ignore } from './utils.js';
 
 // Mock the colors module to avoid issues in tests
 vi.mock('kleur', () => ({
@@ -21,7 +21,7 @@ function should_warn_for_content(content, filename) {
 	const basename = path.basename(filename);
 	if (basename.startsWith('+page.') || basename.startsWith('+layout.')) {
 		const match = content.match(options_regex);
-		return match && match.index !== undefined && !is_within_comment(content, match.index);
+		return match && match.index !== undefined && !should_ignore(content, match.index);
 	}
 	return false;
 }