eslint-plugin-conventions 4.2.2 → 4.2.4

package/src/rules/conventions/filename-case.js

@@ -1,17 +1,9 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.filenameCase = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
 const eslint_devkit_3 = require("@interlace/eslint-devkit");
-/**
- * Default uppercase filenames that are conventionally valid across all case conventions
- */
 const DEFAULT_ALLOWED_UPPERCASE_FILES = [
     'README',
     'LICENSE',
@@ -108,12 +100,9 @@ exports.filenameCase = (0, eslint_devkit_1.createRule)({
     create(context) {
         const [options] = context.options;
         const { case: caseType = 'kebabCase', ignore = [], allowedUppercaseFiles = DEFAULT_ALLOWED_UPPERCASE_FILES, allowedKebabCase = [], allowedSnakeCase = [], allowedCamelCase = [], allowedPascalCase = [], } = options || {};
-        // Convert to different case formats
-        // oxlint-disable-next-line consistent-function-scoping
         function toCamelCase(str) {
             return str.replace(/[-_](.)/g, (_, letter) => letter.toUpperCase());
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function toKebabCase(str) {
             return str
                 .replace(/([a-z])([A-Z])/g, '$1-$2')
@@ -124,32 +113,22 @@ exports.filenameCase = (0, eslint_devkit_1.createRule)({
             const camel = toCamelCase(str);
             return camel.charAt(0).toUpperCase() + camel.slice(1);
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function toSnakeCase(str) {
             return str
                 .replace(/([a-z])([A-Z])/g, '$1_$2')
                 .replace(/[\s-]+/g, '_')
                 .toLowerCase();
         }
-        // Check if filename matches the expected case pattern
-        // oxlint-disable-next-line consistent-function-scoping
         function isCamelCase(str) {
-            // camelCase: starts with lowercase, no separators, can have uppercase after first char
             return /^[a-z][a-zA-Z0-9]*$/.test(str);
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function isKebabCase(str) {
-            // kebab-case: all lowercase with hyphens as separators
             return /^[a-z][a-z0-9]*(-[a-z0-9]+)*$/.test(str);
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function isPascalCase(str) {
-            // PascalCase: starts with uppercase, no separators
             return /^[A-Z][a-zA-Z0-9]*$/.test(str);
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function isSnakeCase(str) {
-            // snake_case: all lowercase with underscores as separators
             return /^[a-z][a-z0-9]*(_[a-z0-9]+)*$/.test(str);
         }
         function matchesCase(str, expectedCase) {
@@ -166,11 +145,7 @@ exports.filenameCase = (0, eslint_devkit_1.createRule)({
                     return true;
             }
         }
-        // Generate suggested filename
         function getSuggestedName(basename, nameWithoutExt, targetCase) {
-            // Initialized to the untransformed name; every valid CaseType below
-            // overwrites it. (An unknown case never reaches here — matchesCase
-            // returns true for unknown cases, so no rename is ever suggested.)
             let transformed = nameWithoutExt;
             switch (targetCase) {
                 case 'camelCase':
@@ -186,26 +161,20 @@ exports.filenameCase = (0, eslint_devkit_1.createRule)({
                     transformed = toSnakeCase(nameWithoutExt);
                     break;
             }
-            // Add back the extension
             const ext = basename.slice(nameWithoutExt.length);
             return transformed + ext;
         }
         return {
             Program(node) {
-                // Get filename from context
                 const filename = context.filename;
                 if (!filename) {
                     return;
                 }
-                // Get just the filename part (without directory)
                 const basename = (0, eslint_devkit_3.getBasename)(filename);
-                // Remove all extensions for checking (handles .spec.ts, .test.tsx, etc.)
                 const nameWithoutExt = basename.replace(/(\.(spec|test|stories|story|e2e|d))*\.[^.]+$/, '');
-                // Skip if no name part (e.g., just extension or dotfile)
                 if (!nameWithoutExt) {
                     return;
                 }
-                // Check if filename should be ignored
                 for (const ignorePattern of ignore) {
                     if (typeof ignorePattern === 'string') {
                         if (basename === ignorePattern) {
@@ -218,16 +187,12 @@ exports.filenameCase = (0, eslint_devkit_1.createRule)({
                         }
                     }
                 }
-                // Skip allowed uppercase files (README, LICENSE, etc.) as they are conventionally valid
                 if (allowedUppercaseFiles.includes(nameWithoutExt)) {
                     return;
                 }
-                // Skip dotfiles (.eslintrc, .gitignore, etc.)
                 if (basename.startsWith('.')) {
                     return;
                 }
-                // Check if filename is allowed via case-specific overrides
-                // These allow specific files to use a different case than the global setting
                 if (allowedKebabCase.includes(nameWithoutExt) && matchesCase(nameWithoutExt, 'kebabCase')) {
                     return;
                 }
@@ -240,11 +205,10 @@ exports.filenameCase = (0, eslint_devkit_1.createRule)({
                 if (allowedPascalCase.includes(nameWithoutExt) && matchesCase(nameWithoutExt, 'pascalCase')) {
                     return;
                 }
-                // Check if filename matches the expected case
                 if (!matchesCase(nameWithoutExt, caseType)) {
                     const suggestedName = getSuggestedName(basename, nameWithoutExt, caseType);
                     context.report({
-                        node, // Report on the Program node
+                        node,
                         messageId: 'filenameCase',
                         data: {
                             case: caseType,
@@ -258,8 +222,6 @@ exports.filenameCase = (0, eslint_devkit_1.createRule)({
                                     suggestedName,
                                 },
                                 fix() {
-                                    // This is a file rename operation that can't be auto-fixed
-                                    // Just provide the suggestion
                                     return null;
                                 },
                             },
@@ -270,4 +232,3 @@ exports.filenameCase = (0, eslint_devkit_1.createRule)({
         };
     },
 });
-//# sourceMappingURL=filename-case.js.map

package/src/rules/conventions/no-commented-code.js

@@ -1,36 +1,23 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noCommentedCode = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
-/**
- * Check if a comment block contains code-like patterns
- */
 function looksLikeCode(comment, isBlockComment) {
-    // Remove comment markers for pattern matching
     let text = comment;
     if (isBlockComment) {
-        // Remove /* and */ markers and any leading * on each line
         text = text.replace(/^\s*\/\*+/, '').replace(/\*+\/\s*$/, '');
-        // Remove leading * from each line in block comments
         text = text
             .split('\n')
             .map((line) => line.replace(/^\s*\*+\s*/, ''))
             .join('\n');
     }
     else {
-        // Remove // markers
         text = text
             .split('\n')
             .map((line) => line.replace(/^\s*\/\/+\s*/, ''))
             .join('\n');
     }
-    // Split into lines and check each line
     const lines = text
         .split('\n')
         .map((line) => line.trim())
@@ -44,36 +31,24 @@ function looksLikeCode(comment, isBlockComment) {
         /^[a-zA-Z_$][a-zA-Z0-9_$]*\s*\(/,
         /^[{}[\]]/,
     ];
-    // Count how many lines look like code
     let codeLikeLines = 0;
-    // Check if any line looks like code (but not a TODO comment)
     for (const line of lines) {
-        // Skip TODO/FIXME comments
         if (/^(TODO|FIXME|HACK|XXX)/i.test(line)) {
             continue;
         }
-        // Check if line matches code patterns
         for (const pattern of codePatterns) {
             if (pattern.test(line)) {
                 codeLikeLines++;
-                break; // Count each line only once
+                break;
             }
         }
     }
-    // Consider it code if at least one line looks like code
     return codeLikeLines > 0;
 }
-/**
- * Count lines in a comment
- * For single-line comments, the value doesn't include newlines, so we count 1
- * For multi-line comments, we count the newlines in the value + 1
- */
 function countCommentLines(comment) {
     if (!comment)
         return 0;
     const newlineCount = (comment.match(/\n/g) || []).length;
-    // If there are newlines, it's a multi-line comment (newlineCount + 1 lines)
-    // If no newlines, it's a single-line comment (1 line)
     return newlineCount > 0 ? newlineCount + 1 : 1;
 }
 exports.noCommentedCode = (0, eslint_devkit_2.createRule)({
@@ -151,23 +126,17 @@ exports.noCommentedCode = (0, eslint_devkit_2.createRule)({
             return {};
         }
         const sourceCode = context.sourceCode;
-        /**
-         * Check comment nodes
-         */
         function checkComment(node) {
             try {
                 const commentText = node.value || '';
                 const isBlockComment = node.type === 'Block';
                 const lines = countCommentLines(commentText);
-                // Skip single-line comments if configured
                 if (ignoreSingleLine && lines === 1 && !isBlockComment) {
                     return;
                 }
-                // Skip if below minimum lines
                 if (lines < minLines) {
                     return;
                 }
-                // Check if it looks like code
                 if (looksLikeCode(commentText, isBlockComment)) {
                     context.report({
                         node,
@@ -180,7 +149,6 @@ exports.noCommentedCode = (0, eslint_devkit_2.createRule)({
                                 messageId: 'removeCode',
                                 fix: (fixer) => {
                                     try {
-                                        // Remove the entire comment
                                         return fixer.remove(node);
                                     }
                                     catch {
@@ -197,44 +165,36 @@ exports.noCommentedCode = (0, eslint_devkit_2.createRule)({
                 }
             }
             catch {
-                // Silently skip if there's an error processing the comment
                 return;
             }
         }
         return {
             Program() {
                 const comments = sourceCode.getAllComments();
-                // Group consecutive comments that look like code
                 const groupedComments = [];
                 let currentGroup = [];
                 for (let i = 0; i < comments.length; i++) {
                     const comment = comments[i];
                     const commentText = comment.value || '';
                     const isBlockComment = comment.type === 'Block';
-                    // Check if this comment looks like code
                     if (looksLikeCode(commentText, isBlockComment)) {
                         currentGroup.push(comment);
                     }
                     else {
-                        // If current group has comments, add it to grouped comments
                         if (currentGroup.length > 0) {
                             groupedComments.push([...currentGroup]);
                             currentGroup = [];
                         }
                     }
                 }
-                // Handle any remaining group
                 if (currentGroup.length > 0) {
                     groupedComments.push(currentGroup);
                 }
-                // Process each group
                 groupedComments.forEach((group) => {
                     if (group.length === 1) {
-                        // Single comment - use the standard checkComment function
                         checkComment(group[0]);
                     }
                     else {
-                        // Multiple consecutive comments - report as one error
                         const firstComment = group[0];
                         const totalLines = group.reduce((sum, comment) => {
                             const commentText = comment.value || '';
@@ -252,7 +212,6 @@ exports.noCommentedCode = (0, eslint_devkit_2.createRule)({
                                         messageId: 'removeCode',
                                         fix: (fixer) => {
                                             try {
-                                                // Remove the entire range of consecutive comments
                                                 return fixer.removeRange([
                                                     group[0].range[0],
                                                     group[group.length - 1].range[1],
@@ -276,4 +235,3 @@ exports.noCommentedCode = (0, eslint_devkit_2.createRule)({
         };
     },
 });
-//# sourceMappingURL=no-commented-code.js.map

package/src/rules/conventions/no-console-spaces.js

@@ -1,9 +1,4 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noConsoleSpaces = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -32,7 +27,6 @@ exports.noConsoleSpaces = (0, eslint_devkit_1.createRule)({
     },
     defaultOptions: [],
     create(context) {
-        // Console methods that join parameters with spaces
         const consoleMethods = new Set([
             'log',
             'debug',
@@ -44,13 +38,10 @@ exports.noConsoleSpaces = (0, eslint_devkit_1.createRule)({
             'group',
             'groupCollapsed',
         ]);
-        // oxlint-disable-next-line consistent-function-scoping
         function isInAllowedContext() {
-            // For simplicity, we'll skip the allow option for now
             return false;
         }
         function isConsoleMethodCall(node) {
-            // Check if this is a call to a console method
             if (node.callee.type === 'MemberExpression' &&
                 node.callee.object.type === 'Identifier' &&
                 node.callee.object.name === 'console' &&
@@ -60,31 +51,23 @@ exports.noConsoleSpaces = (0, eslint_devkit_1.createRule)({
             }
             return false;
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function getConsoleMethodName(node) {
-            // Safe extraction of console method name
             if (node.callee.type === 'MemberExpression' &&
                 node.callee.property.type === 'Identifier') {
                 return node.callee.property.name;
             }
             return 'console';
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function hasLeadingOrTrailingSpaces(text) {
-            // Check if string starts or ends with whitespace, but not if it's only whitespace
-            // Only flag if there's actual content with leading/trailing spaces
             const trimmed = text.trim();
             return trimmed.length > 0 && /^\s|\s$/.test(text);
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function hasLeadingOrTrailingSpacesInTemplate(text) {
-            // For template literals, even whitespace-only quasis should be flagged
             return /^\s|\s$/.test(text);
         }
         return {
             CallExpression(node) {
                 if (isConsoleMethodCall(node) && !isInAllowedContext()) {
-                    // Check each argument for leading/trailing spaces
                     for (const arg of node.arguments) {
                         if (arg.type === 'Literal' && typeof arg.value === 'string') {
                             if (hasLeadingOrTrailingSpaces(arg.value)) {
@@ -103,7 +86,6 @@ exports.noConsoleSpaces = (0, eslint_devkit_1.createRule)({
                             }
                         }
                         else if (arg.type === 'TemplateLiteral') {
-                            // Check template literal quasi values for leading/trailing spaces
                             let hasSpacesInTemplate = false;
                             for (const quasi of arg.quasis) {
                                 if (hasLeadingOrTrailingSpacesInTemplate(quasi.value.raw)) {
@@ -119,7 +101,6 @@ exports.noConsoleSpaces = (0, eslint_devkit_1.createRule)({
                                         method: getConsoleMethodName(node),
                                         arg: 'template literal with spaces',
                                     },
-                                    // Template literals are harder to fix automatically
                                     fix: null,
                                 });
                             }
@@ -130,4 +111,3 @@ exports.noConsoleSpaces = (0, eslint_devkit_1.createRule)({
         };
     },
 });
-//# sourceMappingURL=no-console-spaces.js.map

package/src/rules/conventions/no-json-schema-tags.js

@@ -1,17 +1,7 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noJsonSchemaTags = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
-/**
- * JSON Schema keywords that are commonly misused as JSDoc tags.
- * These have precise semantics in JSON Schema but are invalid
- * (or meaningless) in a JSDoc context.
- */
 const DEFAULT_FORBIDDEN_TAGS = new Set([
     'minimum',
     'maximum',
@@ -32,13 +22,8 @@ const DEFAULT_FORBIDDEN_TAGS = new Set([
     'contentMediaType',
     'contentEncoding',
 ]);
-/**
- * Parse a block comment to find `@tagName` occurrences and their positions.
- */
 function findForbiddenTags(commentText, forbiddenSet) {
     const results = [];
-    // Match @tagName patterns. We don't anchor at line start because block
-    // comments can have `* @tag …` or simply `@tag …` on any line.
     const tagRegex = /@([a-zA-Z][a-zA-Z0-9]*)\b/g;
     let match;
     while ((match = tagRegex.exec(commentText)) !== null) {
@@ -49,10 +34,6 @@ function findForbiddenTags(commentText, forbiddenSet) {
     }
     return results;
 }
-/**
- * Build a human-readable suggestion for how to express the constraint
- * in description text instead of a JSON Schema tag.
- */
 function getSuggestionText(tag) {
     const suggestions = {
         minimum: 'Include range in description, e.g. "(min: 1)" or "Range: 1-100"',
@@ -116,7 +97,6 @@ exports.noJsonSchemaTags = (0, eslint_devkit_1.createRule)({
     defaultOptions: [{ additionalForbiddenTags: [] }],
     create(context, [options = {}]) {
         const { additionalForbiddenTags = [] } = options;
-        // Build the full set of forbidden tags
         const forbiddenTags = new Set(DEFAULT_FORBIDDEN_TAGS);
         for (const tag of additionalForbiddenTags) {
             forbiddenTags.add(tag);
@@ -126,20 +106,17 @@ exports.noJsonSchemaTags = (0, eslint_devkit_1.createRule)({
             Program() {
                 const comments = sourceCode.getAllComments();
                 for (const comment of comments) {
-                    // Only inspect block comments (JSDoc is always /* … */)
                     if (comment.type !== 'Block') {
                         continue;
                     }
                     const commentText = comment.value;
                     const hits = findForbiddenTags(commentText, forbiddenTags);
                     for (const { tag, offset } of hits) {
-                        // Compute absolute source position.
-                        // comment.range[0] → opening `/*`, content starts at +2.
                         const absoluteStart = comment
                             .range[0] +
                             2 +
                             offset;
-                        const absoluteEnd = absoluteStart + 1 + tag.length; // `@` + tagName
+                        const absoluteEnd = absoluteStart + 1 + tag.length;
                         const suggestion = getSuggestionText(tag);
                         context.report({
                             loc: {
@@ -156,8 +133,6 @@ exports.noJsonSchemaTags = (0, eslint_devkit_1.createRule)({
                                     messageId: 'moveToDescription',
                                     data: { tag },
                                     fix: (fixer) => {
-                                        // Remove the entire @tag and its value (to end of line)
-                                        // Find the line boundaries
                                         const fullSource = sourceCode.getText();
                                         let lineStart = absoluteStart;
                                         while (lineStart > 0 && fullSource[lineStart - 1] !== '\n') {
@@ -168,7 +143,6 @@ exports.noJsonSchemaTags = (0, eslint_devkit_1.createRule)({
                                             fullSource[lineEnd] !== '\n') {
                                             lineEnd++;
                                         }
-                                        // Include the trailing newline if present
                                         if (lineEnd < fullSource.length && fullSource[lineEnd] === '\n') {
                                             lineEnd++;
                                         }
@@ -183,4 +157,3 @@ exports.noJsonSchemaTags = (0, eslint_devkit_1.createRule)({
         };
     },
 });
-//# sourceMappingURL=no-json-schema-tags.js.map

package/src/rules/conventions/no-magic-numbers.js

@@ -1,26 +1,14 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noMagicNumbers = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
-/** Numbers that are universally idiomatic in JS/TS and need no naming. */
 const DEFAULT_IGNORE = new Set([-1, 0, 1, 2]);
-/** Build a SCREAMING_SNAKE_CASE const name from a numeric value. */
 function constNameFor(value) {
-    // e.g. 5000 → MAGIC_5000 · -3 → MAGIC_NEG_3 · 1.5 → MAGIC_1_5
     const prefix = value < 0 ? 'MAGIC_NEG_' : 'MAGIC_';
     const digits = String(Math.abs(value)).replace('.', '_');
     return `${prefix}${digits}`;
 }
-/**
- * Walk up the AST to find the nearest statement ancestor that can be used
- * as the insertion point for a const declaration.
- */
 function nearestStatement(node) {
     const STATEMENT_TYPES = new Set([
         'ExpressionStatement', 'VariableDeclaration', 'ReturnStatement',
@@ -119,12 +107,10 @@ exports.noMagicNumbers = (0, eslint_devkit_1.createRule)({
                 BITWISE_OPS.has(parent.operator));
         }
         function isVariableDeclarator(node) {
-            // const FOO = 42 — the literal is the named constant itself
             const parent = node.parent;
             return parent?.type === 'VariableDeclarator';
         }
         function isExportedConst(node) {
-            // export const FOO = 42
             let current = node;
             while (current) {
                 if (current.type === 'ExportNamedDeclaration')
@@ -139,7 +125,6 @@ exports.noMagicNumbers = (0, eslint_devkit_1.createRule)({
             return false;
         }
         function isPropertyKey(node) {
-            // { 42: 'value' } — numeric key in object literal
             const parent = node.parent;
             return (parent?.type === 'Property' &&
                 parent.key === node);
@@ -149,15 +134,12 @@ exports.noMagicNumbers = (0, eslint_devkit_1.createRule)({
                 if (typeof node.value !== 'number')
                     return;
                 const value = node.value;
-                // Skip universally ignored values
                 if (isIgnoredNumber(value))
                     return;
-                // Skip NaN / Infinity (not numeric literals per se)
                 if (!isFinite(value))
                     return;
-                // Context-based skips
                 if (isVariableDeclarator(node))
-                    return; // const X = 42
+                    return;
                 if (isExportedConst(node))
                     return;
                 if (isArrayIndex(node))
@@ -184,7 +166,6 @@ exports.noMagicNumbers = (0, eslint_devkit_1.createRule)({
                                 const stmt = nearestStatement(node);
                                 if (!stmt)
                                     return null;
-                                // Determine indentation from the statement's first token.
                                 const firstToken = sourceCode.getFirstToken(stmt);
                                 if (!firstToken)
                                     return null;
@@ -202,4 +183,3 @@ exports.noMagicNumbers = (0, eslint_devkit_1.createRule)({
         };
     },
 });
-//# sourceMappingURL=no-magic-numbers.js.map

package/src/rules/conventions/no-raw-cross-property-href.js

@@ -1,16 +1,8 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.noRawCrossPropertyHref = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
-// Fixed list of cross-property hosts. Mirrors UTM_PHILOSOPHY.md's
-// cross-property travel matrix and ANALYTICS_PHILOSOPHY.md principle 1's
-// list of measured surfaces. Edit there and here together.
 const CROSS_PROPERTY_HOSTS = new Set([
     'ofriperetz.dev',
     'interlace.tools',
@@ -44,7 +36,6 @@ exports.noRawCrossPropertyHref = (0, eslint_devkit_1.createRule)({
     defaultOptions: [],
     create(context) {
         function parseHost(value) {
-            // Only absolute URLs with a hostname.
             if (!/^https?:\/\//i.test(value))
                 return null;
             try {
@@ -83,4 +74,3 @@ exports.noRawCrossPropertyHref = (0, eslint_devkit_1.createRule)({
         };
     },
 });
-//# sourceMappingURL=no-raw-cross-property-href.js.map

package/src/rules/conventions/prefer-code-point.js

@@ -1,9 +1,4 @@
 "use strict";
-/**
- * Copyright (c) 2025 Ofri Peretz
- * Licensed under the MIT License. Use of this source code is governed by the
- * MIT license that can be found in the LICENSE file.
- */
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.preferCodePoint = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -36,22 +31,14 @@ exports.preferCodePoint = (0, eslint_devkit_1.createRule)({
     },
     defaultOptions: [],
     create(context) {
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars, consistent-function-scoping
         function isInAllowedContext(node) {
-            // For simplicity, we'll skip the allow option for now
-            // This would require more complex logic to check comments/code context
             return false;
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function shouldIgnoreCall(node) {
-            // The unicorn rule flags ALL charCodeAt calls, but allows some contexts
             if (node.callee.type === 'MemberExpression') {
-                // Allow optional chaining
                 if (node.callee.optional) {
                     return true;
                 }
-                // Allow computed property access where the property is a variable (not literal)
-                // This means obj[method] is ignored, but obj['charCodeAt'] is still flagged
                 if (node.callee.computed &&
                     node.callee.property.type === 'Identifier') {
                     return true;
@@ -59,16 +46,12 @@ exports.preferCodePoint = (0, eslint_devkit_1.createRule)({
             }
             return false;
         }
-        // oxlint-disable-next-line consistent-function-scoping
         function isCharCodeAtCall(node) {
-            // Check if this is a call to charCodeAt method
             if (node.callee.type === 'MemberExpression') {
-                // Direct property access: obj.charCodeAt
                 if (node.callee.property.type === 'Identifier' &&
                     node.callee.property.name === 'charCodeAt') {
                     return true;
                 }
-                // Computed property access: obj['charCodeAt']
                 if (node.callee.computed &&
                     node.callee.property.type === 'Literal' &&
                     node.callee.property.value === 'charCodeAt') {
@@ -95,4 +78,3 @@ exports.preferCodePoint = (0, eslint_devkit_1.createRule)({
         };
     },
 });
-//# sourceMappingURL=prefer-code-point.js.map