eslint-plugin-ts-type-forge 0.8.10 → 0.8.11

package/dist/rules/no-side-effect-import.d.mts

@@ -1,6 +1,3 @@
-import type { TSESLint } from '@typescript-eslint/utils';
-type Options = readonly [];
-type MessageIds = 'removeSideEffectImport';
 /**
  * `import 'ts-type-forge';` binds nothing, and there is nothing for it to run:
  * the package ships declarations only, and its `exports` map offers no runtime
@@ -11,9 +8,7 @@ type MessageIds = 'removeSideEffectImport';
  *
  * The ambient globals are a separate entry point, `ts-type-forge/global`,
  * usually brought in through `compilerOptions.types` or a triple-slash
- * reference. This rule matches the bare specifier exactly, so that subpath is
- * left alone.
+ * reference, which the rule leaves alone.
  */
-export declare const noSideEffectImport: TSESLint.RuleModule<MessageIds, Options>;
-export {};
+export declare const noSideEffectImport: import("@typescript-eslint/utils/ts-eslint").RuleModule<"removeSideEffectImport", readonly [], unknown, import("@typescript-eslint/utils/ts-eslint").RuleListener>;
 //# sourceMappingURL=no-side-effect-import.d.mts.map

package/dist/rules/no-side-effect-import.d.mts.map

@@ -1 +1 @@
-{"version":3,"file":"no-side-effect-import.d.mts","sourceRoot":"","sources":["../../src/rules/no-side-effect-import.mts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,QAAQ,EAAY,MAAM,0BAA0B,CAAC;AAInE,KAAK,OAAO,GAAG,SAAS,EAAE,CAAC;AAE3B,KAAK,UAAU,GAAG,wBAAwB,CAAC;AAE3C;;;;;;;;;;;;GAYG;AACH,eAAO,MAAM,kBAAkB,EAAE,QAAQ,CAAC,UAAU,CAAC,UAAU,EAAE,OAAO,CA8B9D,CAAC"}
+{"version":3,"file":"no-side-effect-import.d.mts","sourceRoot":"","sources":["../../src/rules/no-side-effect-import.mts"],"names":[],"mappings":"AAGA;;;;;;;;;;;GAWG;AACH,eAAO,MAAM,kBAAkB,oKAK7B,CAAC"}

package/dist/rules/no-side-effect-import.mjs

@@ -1,4 +1,4 @@
-import { Arr } from 'ts-data-forge';
+import { createNoSideEffectImport } from 'eslint-rule-forge';
 import { TS_TYPE_FORGE_MODULE } from './constants.mjs';
 
 
@@ -12,44 +12,10 @@ import { TS_TYPE_FORGE_MODULE } from './constants.mjs';
 
 
 
-
-export const noSideEffectImport = {
-    meta: {
-        type: 'problem',
-        docs: {
-            description: `Remove a side-effect-only \`import '${TS_TYPE_FORGE_MODULE}';\`, which binds no name and has no runtime entry point to resolve to. \`${TS_TYPE_FORGE_MODULE}/global\` is left alone.`,
-        },
-        fixable: 'code',
-        schema: [],
-        messages: {
-            removeSideEffectImport: `\`import '${TS_TYPE_FORGE_MODULE}';\` binds no name, and \`${TS_TYPE_FORGE_MODULE}\` ships types only — it has no runtime entry point, so the import throws when the module is loaded. Remove it; the ambient globals come from \`${TS_TYPE_FORGE_MODULE}/global\`, not from this.`,
-        },
-    },
-    create: (context) => ({
-        ImportDeclaration: (node) => {
-            if (node.source.value !== TS_TYPE_FORGE_MODULE ||
-                !Arr.isEmpty(node.specifiers)) {
-                return;
-            }
-            context.report({
-                node,
-                messageId: 'removeSideEffectImport',
-                fix: (fixer) => removeStatement(fixer, context.sourceCode, node),
-            });
-        },
-    }),
-    defaultOptions: [],
-};
-
-
-
-
-
-const removeStatement = (fixer, sourceCode, node) => {
-    const restOfLine = /^[^\S\n]*\n/u.exec(sourceCode.getText().slice(node.range[1]));
-    return fixer.removeRange([
-        node.range[0],
-        node.range[1] + (restOfLine?.[0].length ?? 0),
-    ]);
-};
+export const noSideEffectImport = createNoSideEffectImport({
+    moduleName: TS_TYPE_FORGE_MODULE,
+    type: 'problem',
+    description: `Remove a side-effect-only \`import '${TS_TYPE_FORGE_MODULE}';\`, which binds no name and has no runtime entry point to resolve to. \`${TS_TYPE_FORGE_MODULE}/global\` is left alone.`,
+    message: `\`import '${TS_TYPE_FORGE_MODULE}';\` binds no name, and \`${TS_TYPE_FORGE_MODULE}\` ships types only — it has no runtime entry point, so the import throws when the module is loaded. Remove it; the ambient globals come from \`${TS_TYPE_FORGE_MODULE}/global\`, not from this.`,
+});
 //# sourceMappingURL=no-side-effect-import.mjs.map

package/dist/rules/no-side-effect-import.mjs.map

@@ -1 +1 @@
-{"version":3,"file":"no-side-effect-import.mjs","sourceRoot":"","sources":["../../src/rules/no-side-effect-import.mts"],"names":[],"mappings":"AACA,OAAO,EAAE,GAAG,EAAE,MAAM,eAAe,CAAC;AACpC,OAAO,EAAE,oBAAoB,EAAE,MAAM,iBAAiB,CAAC;AAMvD;;;;;;;;;;;;GAYG;AACH,MAAM,CAAC,MAAM,kBAAkB,GAA6C;IAC1E,IAAI,EAAE;QACJ,IAAI,EAAE,SAAS;QACf,IAAI,EAAE;YACJ,WAAW,EAAE,uCAAuC,oBAAoB,6EAA6E,oBAAoB,0BAA0B;SACpM;QACD,OAAO,EAAE,MAAM;QACf,MAAM,EAAE,EAAE;QACV,QAAQ,EAAE;YACR,sBAAsB,EAAE,aAAa,oBAAoB,6BAA6B,oBAAoB,mJAAmJ,oBAAoB,2BAA2B;SAC7S;KACF;IAED,MAAM,EAAE,CAAC,OAAO,EAAE,EAAE,CAAC,CAAC;QACpB,iBAAiB,EAAE,CAAC,IAAI,EAAE,EAAE;YAC1B,IACE,IAAI,CAAC,MAAM,CAAC,KAAK,KAAK,oBAAoB;gBAC1C,CAAC,GAAG,CAAC,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,EAC7B,CAAC;gBACD,OAAO;YACT,CAAC;YAED,OAAO,CAAC,MAAM,CAAC;gBACb,IAAI;gBACJ,SAAS,EAAE,wBAAwB;gBACnC,GAAG,EAAE,CAAC,KAAK,EAAE,EAAE,CAAC,eAAe,CAAC,KAAK,EAAE,OAAO,CAAC,UAAU,EAAE,IAAI,CAAC;aACjE,CAAC,CAAC;QACL,CAAC;KACF,CAAC;IACF,cAAc,EAAE,EAAE;CACV,CAAC;AAEX,uEAAuE;AAEvE;;;GAGG;AACH,MAAM,eAAe,GAAG,CACtB,KAAyB,EACzB,UAA+B,EAC/B,IAAmB,EACD,EAAE;IACpB,MAAM,UAAU,GAAG,cAAc,CAAC,IAAI,CACpC,UAAU,CAAC,OAAO,EAAE,CAAC,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAC1C,CAAC;IAEF,OAAO,KAAK,CAAC,WAAW,CAAC;QACvB,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC;QACb,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,UAAU,EAAE,CAAC,CAAC,CAAC,CAAC,MAAM,IAAI,CAAC,CAAC;KAC9C,CAAC,CAAC;AACL,CAAC,CAAC"}
+{"version":3,"file":"no-side-effect-import.mjs","sourceRoot":"","sources":["../../src/rules/no-side-effect-import.mts"],"names":[],"mappings":"AAAA,OAAO,EAAE,wBAAwB,EAAE,MAAM,mBAAmB,CAAC;AAC7D,OAAO,EAAE,oBAAoB,EAAE,MAAM,iBAAiB,CAAC;AAEvD;;;;;;;;;;;GAWG;AACH,MAAM,CAAC,MAAM,kBAAkB,GAAG,wBAAwB,CAAC;IACzD,UAAU,EAAE,oBAAoB;IAChC,IAAI,EAAE,SAAS;IACf,WAAW,EAAE,uCAAuC,oBAAoB,6EAA6E,oBAAoB,0BAA0B;IACnM,OAAO,EAAE,aAAa,oBAAoB,6BAA6B,oBAAoB,mJAAmJ,oBAAoB,2BAA2B;CAC9R,CAAC,CAAC"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-ts-type-forge",
-  "version": "0.8.10",
+  "version": "0.8.11",
   "private": false,
   "description": "ESLint rules that steer type declarations toward ts-type-forge idioms.",
   "keywords": [
@@ -36,7 +36,7 @@
   ],
   "dependencies": {
     "@typescript-eslint/utils": "~8.71.1",
-    "eslint-rule-forge": "^0.2.0",
+    "eslint-rule-forge": "^0.3.0",
     "ts-data-forge": "~14.9.0",
     "ts-type-forge": "9.2.2"
   },
@@ -44,9 +44,9 @@
     "@typescript-eslint/parser": "8.71.1",
     "@typescript-eslint/rule-tester": "8.71.1",
     "dedent": "1.7.2",
-    "eslint-config-typed": "5.19.0",
-    "eslint-plugin-ts-data-forge": "0.10.0",
-    "eslint-plugin-ts-fortress": "1.3.5",
+    "eslint-config-typed": "5.19.1",
+    "eslint-plugin-ts-data-forge": "0.10.1",
+    "eslint-plugin-ts-fortress": "1.3.6",
     "prettier": "3.9.9",
     "prettier-plugin-organize-imports": "4.3.0",
     "prettier-plugin-packagejson": "3.0.2",

package/src/rules/no-side-effect-import.mts

@@ -1,11 +1,6 @@
-import type { TSESLint, TSESTree } from '@typescript-eslint/utils';
-import { Arr } from 'ts-data-forge';
+import { createNoSideEffectImport } from 'eslint-rule-forge';
 import { TS_TYPE_FORGE_MODULE } from './constants.mjs';
 
-type Options = readonly [];
-
-type MessageIds = 'removeSideEffectImport';
-
 /**
  * `import 'ts-type-forge';` binds nothing, and there is nothing for it to run:
  * the package ships declarations only, and its `exports` map offers no runtime
@@ -16,58 +11,11 @@ type MessageIds = 'removeSideEffectImport';
  *
  * The ambient globals are a separate entry point, `ts-type-forge/global`,
  * usually brought in through `compilerOptions.types` or a triple-slash
- * reference. This rule matches the bare specifier exactly, so that subpath is
- * left alone.
+ * reference, which the rule leaves alone.
  */
-export const noSideEffectImport: TSESLint.RuleModule<MessageIds, Options> = {
-  meta: {
-    type: 'problem',
-    docs: {
-      description: `Remove a side-effect-only \`import '${TS_TYPE_FORGE_MODULE}';\`, which binds no name and has no runtime entry point to resolve to. \`${TS_TYPE_FORGE_MODULE}/global\` is left alone.`,
-    },
-    fixable: 'code',
-    schema: [],
-    messages: {
-      removeSideEffectImport: `\`import '${TS_TYPE_FORGE_MODULE}';\` binds no name, and \`${TS_TYPE_FORGE_MODULE}\` ships types only — it has no runtime entry point, so the import throws when the module is loaded. Remove it; the ambient globals come from \`${TS_TYPE_FORGE_MODULE}/global\`, not from this.`,
-    },
-  },
-
-  create: (context) => ({
-    ImportDeclaration: (node) => {
-      if (
-        node.source.value !== TS_TYPE_FORGE_MODULE ||
-        !Arr.isEmpty(node.specifiers)
-      ) {
-        return;
-      }
-
-      context.report({
-        node,
-        messageId: 'removeSideEffectImport',
-        fix: (fixer) => removeStatement(fixer, context.sourceCode, node),
-      });
-    },
-  }),
-  defaultOptions: [],
-} as const;
-
-/* eslint-disable @typescript-eslint/prefer-readonly-parameter-types */
-
-/**
- * Removes a statement along with the rest of its line, so that deleting it does
- * not leave a blank line where it was.
- */
-const removeStatement = (
-  fixer: TSESLint.RuleFixer,
-  sourceCode: TSESLint.SourceCode,
-  node: TSESTree.Node,
-): TSESLint.RuleFix => {
-  const restOfLine = /^[^\S\n]*\n/u.exec(
-    sourceCode.getText().slice(node.range[1]),
-  );
-
-  return fixer.removeRange([
-    node.range[0],
-    node.range[1] + (restOfLine?.[0].length ?? 0),
-  ]);
-};
+export const noSideEffectImport = createNoSideEffectImport({
+  moduleName: TS_TYPE_FORGE_MODULE,
+  type: 'problem',
+  description: `Remove a side-effect-only \`import '${TS_TYPE_FORGE_MODULE}';\`, which binds no name and has no runtime entry point to resolve to. \`${TS_TYPE_FORGE_MODULE}/global\` is left alone.`,
+  message: `\`import '${TS_TYPE_FORGE_MODULE}';\` binds no name, and \`${TS_TYPE_FORGE_MODULE}\` ships types only — it has no runtime entry point, so the import throws when the module is loaded. Remove it; the ambient globals come from \`${TS_TYPE_FORGE_MODULE}/global\`, not from this.`,
+});