eslint-plugin-conventions 4.2.3 → 4.2.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-conventions",
-  "version": "4.2.3",
+  "version": "4.2.4",
   "description": "ESLint plugin for team code conventions — enforces filename case, magic-number bans, commented-out code, expiring TODOs, and deprecated-API usage.",
   "type": "commonjs",
   "main": "./src/index.js",
@@ -22,10 +22,8 @@
   },
   "files": [
     "src/",
-    "dist/",
     "README.md",
-    "LICENSE",
-    "CHANGELOG.md"
+    "LICENSE"
   ],
   "keywords": [
     "eslint",
@@ -53,32 +51,22 @@
   },
   "dependencies": {
     "tslib": "^2.3.0",
-    "@interlace/eslint-devkit": "*"
-  },
-  "devDependencies": {
-    "@typescript-eslint/parser": "^8.46.2",
-    "@typescript-eslint/rule-tester": "^8.46.2"
+    "@interlace/eslint-devkit": "^1.6.1"
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/ofri-peretz/eslint",
+    "url": "git+https://github.com/ofri-peretz/eslint.git",
     "directory": "packages/eslint-plugin-conventions"
   },
   "homepage": "https://eslint.interlace.tools/docs/quality/plugin-conventions?utm_source=npm&utm_medium=referral&utm_campaign=eslint-plugin-conventions",
   "bugs": {
     "url": "https://github.com/ofri-peretz/eslint/issues"
   },
-  "scripts": {
-    "build": "tsx ../../scripts/build-package.ts",
-    "test": "vitest run",
-    "test:coverage": "vitest run --coverage",
-    "typecheck": "tsc -p tsconfig.lib.json --noEmit"
-  },
   "peerDependencies": {
     "eslint": "^8.0.0 || ^9.0.0 || ^10.0.0"
   },
   "funding": {
     "type": "github",
-    "url": "https://github.com/ofri-peretz/eslint"
+    "url": "git+https://github.com/ofri-peretz/eslint.git"
   }
 }

package/src/index.d.ts

@@ -1,175 +1,17 @@
-/**
- * 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.
- */
-import type { TSESLint } from '@interlace/eslint-devkit';
-export declare const rules: {
-    'no-commented-code': TSESLint.RuleModule<"commentedCode" | "removeCode" | "useVersionControl", [(import("./rules/conventions/no-commented-code").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'expiring-todo-comments': TSESLint.RuleModule<"expiringTodoComment" | "invalidTodoCondition" | "multipleTodoConditions", [(import("./rules/conventions/expiring-todo-comments").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'prefer-code-point': TSESLint.RuleModule<"preferCodePoint", [], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'prefer-dom-node-text-content': TSESLint.RuleModule<"preferDomNodeTextContent", [], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'no-console-spaces': TSESLint.RuleModule<"noConsoleSpaces", [], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'no-deprecated-api': TSESLint.RuleModule<"deprecatedAPI" | "useReplacement", [(import("./rules/deprecation/no-deprecated-api").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'prefer-dependency-version-strategy': TSESLint.RuleModule<"preferStrategy" | "invalidStrategy", [(import("./rules/conventions/prefer-dependency-version-strategy").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'filename-case': TSESLint.RuleModule<"filenameCase", [(import("./rules/conventions/filename-case").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'consistent-existence-index-check': TSESLint.RuleModule<"consistentExistenceCheck", [(import("./rules/conventions/consistent-existence-index-check").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'no-json-schema-tags': TSESLint.RuleModule<"jsonSchemaTag" | "moveToDescription", [(import("./rules/conventions/no-json-schema-tags").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'require-data-testid': TSESLint.RuleModule<"missingDataTestId" | "dynamicDataTestId", [(import("./rules/conventions/require-data-testid").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'utm-taxonomy': TSESLint.RuleModule<"invalidUtmSource" | "invalidUtmMedium", [], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'no-raw-cross-property-href': TSESLint.RuleModule<"rawCrossPropertyHref", [], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'analytics-event-naming': TSESLint.RuleModule<"invalidEventName" | "interpolatedEventName", [], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-    'no-magic-numbers': TSESLint.RuleModule<"noMagicNumber" | "extractConst", [(import("./rules/conventions/no-magic-numbers").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-        name: string;
-    };
-};
+// Minimal declaration written by scripts/build-package.ts.
+//
+// tsc's generated entry declaration inlined every inferred rule-option type
+// (up to 166 kB per plugin) for an API no consumer calls directly. This types
+// what a flat config actually touches. Per-rule option types remain available
+// from the "./types" subpath export where a plugin provides one.
+import type { Linter, Rule } from 'eslint';
+
+export declare const rules: Record<string, Rule.RuleModule>;
+export declare const configs: Record<string, Linter.Config>;
 export declare const plugin: {
-    meta: {
-        name: string;
-        version: string;
-    };
-    rules: {
-        'no-commented-code': TSESLint.RuleModule<"commentedCode" | "removeCode" | "useVersionControl", [(import("./rules/conventions/no-commented-code").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'expiring-todo-comments': TSESLint.RuleModule<"expiringTodoComment" | "invalidTodoCondition" | "multipleTodoConditions", [(import("./rules/conventions/expiring-todo-comments").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'prefer-code-point': TSESLint.RuleModule<"preferCodePoint", [], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'prefer-dom-node-text-content': TSESLint.RuleModule<"preferDomNodeTextContent", [], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'no-console-spaces': TSESLint.RuleModule<"noConsoleSpaces", [], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'no-deprecated-api': TSESLint.RuleModule<"deprecatedAPI" | "useReplacement", [(import("./rules/deprecation/no-deprecated-api").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'prefer-dependency-version-strategy': TSESLint.RuleModule<"preferStrategy" | "invalidStrategy", [(import("./rules/conventions/prefer-dependency-version-strategy").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'filename-case': TSESLint.RuleModule<"filenameCase", [(import("./rules/conventions/filename-case").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'consistent-existence-index-check': TSESLint.RuleModule<"consistentExistenceCheck", [(import("./rules/conventions/consistent-existence-index-check").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'no-json-schema-tags': TSESLint.RuleModule<"jsonSchemaTag" | "moveToDescription", [(import("./rules/conventions/no-json-schema-tags").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'require-data-testid': TSESLint.RuleModule<"missingDataTestId" | "dynamicDataTestId", [(import("./rules/conventions/require-data-testid").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'utm-taxonomy': TSESLint.RuleModule<"invalidUtmSource" | "invalidUtmMedium", [], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'no-raw-cross-property-href': TSESLint.RuleModule<"rawCrossPropertyHref", [], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'analytics-event-naming': TSESLint.RuleModule<"invalidEventName" | "interpolatedEventName", [], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-        'no-magic-numbers': TSESLint.RuleModule<"noMagicNumber" | "extractConst", [(import("./rules/conventions/no-magic-numbers").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-            name: string;
-        };
-    };
-};
-export declare const configs: {
-    recommended: {
-        plugins: {
-            conventions: {
-                meta: {
-                    name: string;
-                    version: string;
-                };
-                rules: {
-                    'no-commented-code': TSESLint.RuleModule<"commentedCode" | "removeCode" | "useVersionControl", [(import("./rules/conventions/no-commented-code").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'expiring-todo-comments': TSESLint.RuleModule<"expiringTodoComment" | "invalidTodoCondition" | "multipleTodoConditions", [(import("./rules/conventions/expiring-todo-comments").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'prefer-code-point': TSESLint.RuleModule<"preferCodePoint", [], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'prefer-dom-node-text-content': TSESLint.RuleModule<"preferDomNodeTextContent", [], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'no-console-spaces': TSESLint.RuleModule<"noConsoleSpaces", [], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'no-deprecated-api': TSESLint.RuleModule<"deprecatedAPI" | "useReplacement", [(import("./rules/deprecation/no-deprecated-api").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'prefer-dependency-version-strategy': TSESLint.RuleModule<"preferStrategy" | "invalidStrategy", [(import("./rules/conventions/prefer-dependency-version-strategy").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'filename-case': TSESLint.RuleModule<"filenameCase", [(import("./rules/conventions/filename-case").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'consistent-existence-index-check': TSESLint.RuleModule<"consistentExistenceCheck", [(import("./rules/conventions/consistent-existence-index-check").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'no-json-schema-tags': TSESLint.RuleModule<"jsonSchemaTag" | "moveToDescription", [(import("./rules/conventions/no-json-schema-tags").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'require-data-testid': TSESLint.RuleModule<"missingDataTestId" | "dynamicDataTestId", [(import("./rules/conventions/require-data-testid").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'utm-taxonomy': TSESLint.RuleModule<"invalidUtmSource" | "invalidUtmMedium", [], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'no-raw-cross-property-href': TSESLint.RuleModule<"rawCrossPropertyHref", [], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'analytics-event-naming': TSESLint.RuleModule<"invalidEventName" | "interpolatedEventName", [], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                    'no-magic-numbers': TSESLint.RuleModule<"noMagicNumber" | "extractConst", [(import("./rules/conventions/no-magic-numbers").Options | undefined)?], unknown, TSESLint.RuleListener> & {
-                        name: string;
-                    };
-                };
-            };
-        };
-        rules: {
-            'conventions/no-commented-code': "warn";
-            'conventions/expiring-todo-comments': "warn";
-            'conventions/no-deprecated-api': "warn";
-            'conventions/utm-taxonomy': "error";
-            'conventions/no-raw-cross-property-href': "error";
-            'conventions/analytics-event-naming': "error";
-            'conventions/no-magic-numbers': "warn";
-        };
-    };
+  meta: { name: string; version: string };
+  rules: Record<string, Rule.RuleModule>;
 };
-export default plugin;
+
+declare const _default: typeof plugin;
+export default _default;

package/src/index.js

@@ -1,12 +1,6 @@
 "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.configs = exports.plugin = exports.rules = void 0;
-// Conventions rules
 const no_commented_code_1 = require("./rules/conventions/no-commented-code");
 const expiring_todo_comments_1 = require("./rules/conventions/expiring-todo-comments");
 const prefer_code_point_1 = require("./rules/conventions/prefer-code-point");
@@ -42,7 +36,7 @@ exports.rules = {
 exports.plugin = {
     meta: {
         name: 'eslint-plugin-conventions',
-        version: '4.2.3',
+        version: '4.2.4',
     },
     rules: exports.rules,
 };
@@ -63,4 +57,3 @@ exports.configs = {
     },
 };
 exports.default = exports.plugin;
-//# sourceMappingURL=index.js.map

package/src/lib/eslint-plugin-conventions.js

@@ -4,4 +4,3 @@ exports.eslintPluginConventions = eslintPluginConventions;
 function eslintPluginConventions() {
     return 'eslint-plugin-conventions';
 }
-//# sourceMappingURL=eslint-plugin-conventions.js.map

package/src/oxlint.d.ts

@@ -1,20 +1,17 @@
-/**
- * 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.
- */
-/**
- * oxlint sub-export.
- *
- * Consumers wire oxlint to this entry via:
- *   { "jsPlugins": ["eslint-plugin-conventions/oxlint"] }
- *
- * oxlint's JS plugin loader does `require('eslint-plugin-conventions/oxlint')` and reads
- * `.rules` off the result. The compiled output of `export = plugin` is
- * `module.exports = plugin`, which gives oxlint the plugin object directly
- * (`{ meta, rules }`) — no wrapper, no `.default` indirection.
- *
- * Generated by scripts/generate-oxlint-shims.mjs. Do not edit by hand.
- */
-import { plugin } from './index';
-export = plugin;
+// Minimal declaration written by scripts/build-package.ts.
+//
+// tsc's generated entry declaration inlined every inferred rule-option type
+// (up to 166 kB per plugin) for an API no consumer calls directly. This types
+// what a flat config actually touches. Per-rule option types remain available
+// from the "./types" subpath export where a plugin provides one.
+import type { Linter, Rule } from 'eslint';
+
+export declare const rules: Record<string, Rule.RuleModule>;
+export declare const configs: Record<string, Linter.Config>;
+export declare const plugin: {
+  meta: { name: string; version: string };
+  rules: Record<string, Rule.RuleModule>;
+};
+
+declare const _default: typeof plugin;
+export default _default;

package/src/oxlint.js

@@ -1,22 +1,3 @@
 "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.
- */
-/**
- * oxlint sub-export.
- *
- * Consumers wire oxlint to this entry via:
- *   { "jsPlugins": ["eslint-plugin-conventions/oxlint"] }
- *
- * oxlint's JS plugin loader does `require('eslint-plugin-conventions/oxlint')` and reads
- * `.rules` off the result. The compiled output of `export = plugin` is
- * `module.exports = plugin`, which gives oxlint the plugin object directly
- * (`{ meta, rules }`) — no wrapper, no `.default` indirection.
- *
- * Generated by scripts/generate-oxlint-shims.mjs. Do not edit by hand.
- */
 const index_1 = require("./index");
 module.exports = index_1.plugin;
-//# sourceMappingURL=oxlint.js.map

package/src/rules/conventions/analytics-event-naming.js

@@ -1,15 +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.analyticsEventNaming = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
-// ANALYTICS_PHILOSOPHY.md principle 4 → fixed verb list. Edit there and
-// here together.
 const ACTION_VERBS = [
     'click',
     'submit',
@@ -61,15 +54,12 @@ exports.analyticsEventNaming = (0, eslint_devkit_1.createRule)({
     defaultOptions: [],
     create(context) {
         function isCaptureLike(node) {
-            // <obj>.capture(...) — PostHog
-            // <obj>.track(...)   — Segment / Mixpanel / Amplitude
             if (node.callee.type === 'MemberExpression' &&
                 node.callee.property.type === 'Identifier' &&
                 (node.callee.property.name === 'capture' ||
                     node.callee.property.name === 'track')) {
                 return true;
             }
-            // bare track(...) — our primitive
             if (node.callee.type === 'Identifier' &&
                 node.callee.name === 'track') {
                 return true;
@@ -101,4 +91,3 @@ exports.analyticsEventNaming = (0, eslint_devkit_1.createRule)({
         };
     },
 });
-//# sourceMappingURL=analytics-event-naming.js.map

package/src/rules/conventions/consistent-existence-index-check.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.consistentExistenceIndexCheck = void 0;
 const eslint_devkit_1 = require("@interlace/eslint-devkit");
@@ -48,7 +43,6 @@ exports.consistentExistenceIndexCheck = (0, eslint_devkit_1.createRule)({
         const { preferred = 'in' } = options || {};
         function reportInconsistentCheck(node, currentMethod, object, property) {
             let fix;
-            // Only provide fixes for standalone expressions, not when part of larger expressions
             const parent = node.parent;
             const isStandaloneExpression = !parent ||
                 parent.type === 'ExpressionStatement' ||
@@ -93,12 +87,7 @@ exports.consistentExistenceIndexCheck = (0, eslint_devkit_1.createRule)({
             });
         }
         return {
-            // Check for hasOwnProperty calls
             CallExpression(node) {
-                // Direct hasOwnProperty calls: obj.hasOwnProperty(prop). Reindented
-                // to match the sibling `if` blocks below — the original 6-space
-                // indent here suggested this branch was at a different scope level
-                // (CodeQL: `js/misleading-indentation-after-control-statement`).
                 if (node.callee.type === 'MemberExpression' &&
                     node.callee.property.type === 'Identifier' &&
                     node.callee.property.name === 'hasOwnProperty' &&
@@ -106,7 +95,6 @@ exports.consistentExistenceIndexCheck = (0, eslint_devkit_1.createRule)({
                     preferred !== 'hasOwnProperty') {
                     reportInconsistentCheck(node, 'hasOwnProperty', node.callee.object, node.arguments[0]);
                 }
-                // Object.prototype.hasOwnProperty.call(obj, prop)
                 if (node.callee.type === 'MemberExpression' &&
                     node.callee.object.type === 'MemberExpression' &&
                     node.callee.object.object.type === 'MemberExpression' &&
@@ -122,7 +110,6 @@ exports.consistentExistenceIndexCheck = (0, eslint_devkit_1.createRule)({
                     preferred !== 'hasOwnProperty') {
                     reportInconsistentCheck(node, 'Object.prototype.hasOwnProperty.call', node.arguments[0], node.arguments[1]);
                 }
-                // Object.hasOwn(obj, prop)
                 if (node.callee.type === 'MemberExpression' &&
                     node.callee.object.type === 'Identifier' &&
                     node.callee.object.name === 'Object' &&
@@ -133,7 +120,6 @@ exports.consistentExistenceIndexCheck = (0, eslint_devkit_1.createRule)({
                     reportInconsistentCheck(node, 'Object.hasOwn', node.arguments[0], node.arguments[1]);
                 }
             },
-            // Check for 'in' operator usage
             BinaryExpression(node) {
                 if (node.operator === 'in' && preferred !== 'in') {
                     reportInconsistentCheck(node, 'in', node.right, node.left);
@@ -142,4 +128,3 @@ exports.consistentExistenceIndexCheck = (0, eslint_devkit_1.createRule)({
         };
     },
 });
-//# sourceMappingURL=consistent-existence-index-check.js.map

package/src/rules/conventions/expiring-todo-comments.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.expiringTodoComments = void 0;
 exports.parseVersionParts = parseVersionParts;
@@ -19,12 +14,6 @@ const eslint_devkit_1 = require("@interlace/eslint-devkit");
 const eslint_devkit_2 = require("@interlace/eslint-devkit");
 const eslint_devkit_3 = require("@interlace/eslint-devkit");
 const eslint_devkit_4 = require("@interlace/eslint-devkit");
-/**
- * Normalize a version specifier (`^1.2.3`, `24.x`, `18`) into numeric
- * major/minor/patch parts. Wildcard or non-numeric segments become 0 —
- * without this, `package.json#engines.node: "24.x"` would compare as
- * equal to everything, making every `>=` engine TODO falsely match.
- */
 function parseVersionParts(version) {
     const cleaned = version.replace(/^[\^~>=<]+/, '').trim();
     const parts = cleaned
@@ -44,9 +33,7 @@ function compareVersions(version1, version2) {
     return v1.patch - v2.patch;
 }
 function checkDateCondition(dateStr) {
-    // Invalid date strings produce an Invalid Date; comparisons against it
-    // are always false, so malformed input safely reads as "not expired".
-    const conditionDate = new Date(dateStr + 'T00:00:00Z'); // UTC
+    const conditionDate = new Date(dateStr + 'T00:00:00Z');
     const now = new Date();
     return now >= conditionDate;
 }
@@ -76,7 +63,6 @@ function checkEngineVersionCondition(pkg, engine, operator, targetVersion) {
     const nodeVersion = pkg?.engines?.node;
     if (!nodeVersion)
         return false;
-    // Simple version comparison for engines
     const currentVersion = nodeVersion.replace(/^[\^~>=<]+/, '');
     const comparison = compareVersions(currentVersion, targetVersion);
     switch (operator) {
@@ -104,25 +90,19 @@ function checkDependencyCondition(pkg, packageName, present) {
     return present ? hasPackage : hasPackage;
 }
 function parseTodoCondition(commentValue, term) {
-    // Match patterns like: TODO [condition]: message
-    // or TODO (@author) [condition]: message
-    // Also handle block comments with * prefix: * TODO [condition]: message
     const todoRegex = new RegExp(`\\*?\\s*${term}\\s*(?:\\([^)]+\\))?\\s*\\[([^\\]]+)\\]\\s*:\\s*(.+)$`, 'im');
     const match = commentValue.match(todoRegex);
     if (!match)
         return null;
     const conditionStr = match[1];
     const rest = match[2];
-    // Split multiple conditions (shouldn't happen, but handle gracefully)
     const conditions = conditionStr.split(/\s*,\s*/);
     return { conditions, rest };
 }
 function validateCondition(condition) {
-    // Date condition: YYYY-MM-DD
     if (/^\d{4}-\d{2}-\d{2}$/.test(condition)) {
         return { type: 'date', value: condition };
     }
-    // Package version condition: >=1.0.0, >2.0.0, etc.
     const packageVersionMatch = condition.match(/^([><]=?|=|==)\s*([\d.]+)$/);
     if (packageVersionMatch) {
         return {
@@ -131,13 +111,11 @@ function validateCondition(condition) {
             value: packageVersionMatch[2],
         };
     }
-    // Engine version condition: engine:node@>=8, engine:node@>20.0.0
     const engineVersionMatch = condition.match(/^engine:(\w+)@([><]=?)\s*([\d.]+)$/);
     if (engineVersionMatch) {
         const engine = engineVersionMatch[1];
-        // Validate engine name
         if (!['node', 'npm', 'yarn', 'pnpm'].includes(engine)) {
-            return null; // Invalid engine, let it be caught as invalid condition
+            return null;
         }
         return {
             type: 'engine-version',
@@ -145,7 +123,6 @@ function validateCondition(condition) {
             operator: engineVersionMatch[2],
         };
     }
-    // Dependency condition: +package-name or -package-name
     const depMatch = condition.match(/^([+-])(.+)$/);
     if (depMatch) {
         return {
@@ -156,10 +133,6 @@ function validateCondition(condition) {
     }
     return null;
 }
-/**
- * Evaluate an already-parsed condition. `getPkg` is called lazily so that
- * date-only conditions never touch the filesystem.
- */
 function checkParsedCondition(getPkg, parsed) {
     if (!parsed)
         return false;
@@ -241,7 +214,6 @@ exports.expiringTodoComments = (0, eslint_devkit_1.createRule)({
     create(context) {
         const [options] = context.options;
         const { terms = ['TODO', 'FIXME', 'XXX'] } = options || {};
-        // Cache package.json data
         let packageJson = null;
         function loadPackageJson() {
             if (packageJson !== null) {
@@ -258,7 +230,6 @@ exports.expiringTodoComments = (0, eslint_devkit_1.createRule)({
                 if (!parsed)
                     continue;
                 const { conditions, rest } = parsed;
-                // Check for multiple conditions
                 if (conditions.length > 1) {
                     context.report({
                         loc: comment.loc,
@@ -271,7 +242,6 @@ exports.expiringTodoComments = (0, eslint_devkit_1.createRule)({
                     continue;
                 }
                 const condition = conditions[0];
-                // Validate condition format
                 const parsedCondition = validateCondition(condition);
                 if (!parsedCondition) {
                     context.report({
@@ -284,7 +254,6 @@ exports.expiringTodoComments = (0, eslint_devkit_1.createRule)({
                     });
                     continue;
                 }
-                // Check if condition has expired
                 if (checkParsedCondition(loadPackageJson, parsedCondition)) {
                     context.report({
                         loc: comment.loc,
@@ -296,7 +265,6 @@ exports.expiringTodoComments = (0, eslint_devkit_1.createRule)({
                         },
                     });
                 }
-                // Only process one term per comment
                 break;
             }
         }
@@ -313,4 +281,3 @@ exports.expiringTodoComments = (0, eslint_devkit_1.createRule)({
         };
     },
 });
-//# sourceMappingURL=expiring-todo-comments.js.map