@prover-coder-ai/eslint-plugin-suggest-members 0.0.0

package/tests/rules/suggest-imports-exports.test.ts

@@ -0,0 +1,91 @@
+import { suggestExportsRule } from "../../src/rules/suggest-exports/index.js"
+import { suggestImportsRule } from "../../src/rules/suggest-imports/index.js"
+import { createRuleTester, resolveFixtureImportPath, resolveFixturePath } from "../utils/rule-tester.js"
+
+const ruleTester = createRuleTester()
+const filename = resolveFixturePath("consumer.ts")
+
+const validImportsCode = `
+  import { useState, useEffect } from "./modules/exports"
+  useState()
+  useEffect()
+`
+
+const invalidImportCode = `
+  import { useStae } from "./modules/exports"
+  useStae()
+`
+
+const exportsModulePath = resolveFixtureImportPath("modules/exports.ts")
+
+const invalidImportMessage =
+  `Export 'useStae' does not exist on type 'typeof import("${exportsModulePath}")'. Did you mean:\n` +
+  "  - useState(): number\n" +
+  "  - useMemo(): number\n" +
+  "  - useEffect(): void\n" +
+  "  - useCallback(): string"
+
+ruleTester.run("suggest-imports", suggestImportsRule, {
+  valid: [
+    {
+      filename,
+      code: validImportsCode
+    }
+  ],
+  invalid: [
+    {
+      filename,
+      code: invalidImportCode,
+      errors: [{ messageId: "suggestImports", data: { message: invalidImportMessage } }]
+    }
+  ]
+})
+
+const validExportModuleCode = `
+  export { useState } from "./modules/exports"
+`
+
+const validExportLocalCode = `
+  const formatGreeting = () => "ok"
+  export { formatGreeting }
+`
+
+const invalidExportModuleCode = `
+  export { useStae } from "./modules/exports"
+`
+
+const invalidExportLocalCode = `
+  const formatGreeting = () => "ok"
+  export { formatGree1ting }
+`
+
+const invalidExportModuleMessage = invalidImportMessage
+
+const invalidExportLocalMessage = "Cannot find name 'formatGree1ting'. Did you mean:\n" +
+  "  - formatGreeting(): string\n" +
+  "  - FormData: typeof undici.FormData"
+
+ruleTester.run("suggest-exports", suggestExportsRule, {
+  valid: [
+    {
+      filename,
+      code: validExportModuleCode
+    },
+    {
+      filename,
+      code: validExportLocalCode
+    }
+  ],
+  invalid: [
+    {
+      filename,
+      code: invalidExportModuleCode,
+      errors: [{ messageId: "suggestExports", data: { message: invalidExportModuleMessage } }]
+    },
+    {
+      filename,
+      code: invalidExportLocalCode,
+      errors: [{ messageId: "suggestExports", data: { message: invalidExportLocalMessage } }]
+    }
+  ]
+})

package/tests/rules/suggest-members.test.ts

@@ -0,0 +1,98 @@
+import { suggestMembersRule } from "../../src/rules/suggest-members/index.js"
+import { createRuleTester, resolveFixturePath } from "../utils/rule-tester.js"
+
+const ruleTester = createRuleTester()
+const filename = resolveFixturePath("consumer.ts")
+const memberAccessMessage =
+  "Property 'nmae' does not exist on type '{ name: string; count: number; }'. Did you mean:\n" +
+  "  - name: string"
+const memberOptionalAccessMessage = "Property 'nmae' does not exist on type '{ name: string; }'. Did you mean:\n" +
+  "  - name: string"
+const memberPatternMessage = "Property 'na1me' does not exist on type 'Named'. Did you mean:\n" +
+  "  - name: string"
+const memberLiteralMessage = "Property 'kin1d' does not exist on type 'Named'. Did you mean:\n" +
+  "  - kind: \"named\""
+
+ruleTester.run("suggest-members", suggestMembersRule, {
+  valid: [
+    {
+      filename,
+      code: `
+        const obj = { name: "ok", count: 1 }
+        obj.name
+        obj.count
+      `
+    },
+    {
+      filename,
+      code: `
+        const obj = { name: "ok" }
+        obj["name"]
+      `
+    },
+    {
+      filename,
+      code: `
+        const obj = { name: "ok" }
+        obj?.name
+      `
+    }
+  ],
+  invalid: [
+    {
+      filename,
+      code: `
+        const obj = { name: "ok", count: 1 }
+        obj.nmae
+      `,
+      errors: [{
+        messageId: "suggestMembers",
+        data: {
+          message: memberAccessMessage
+        }
+      }]
+    },
+    {
+      filename,
+      code: `
+        const obj = { name: "ok" }
+        obj?.nmae
+      `,
+      errors: [{
+        messageId: "suggestMembers",
+        data: {
+          message: memberOptionalAccessMessage
+        }
+      }]
+    },
+    {
+      filename,
+      code: `
+        type Named = { readonly name: string }
+        const variant: Named = { name: "ok" }
+        const { na1me } = variant
+        void na1me
+      `,
+      errors: [{
+        messageId: "suggestMembers",
+        data: {
+          message: memberPatternMessage
+        }
+      }]
+    },
+    {
+      filename,
+      code: `
+        type Named = { readonly kind: "named"; readonly name: string }
+        const variant: Named = { kin1d: "named", name: "ok" }
+        void variant
+      `,
+      errors: [{
+        messageId: "suggestMembers",
+        data: {
+          message: memberLiteralMessage
+        }
+      }]
+    }
+  ]
+})

package/tests/rules/suggest-missing-names.test.ts

@@ -0,0 +1,35 @@
+import { suggestMissingNamesRule } from "../../src/rules/suggest-missing-names/index.js"
+import { createRuleTester, resolveFixturePath } from "../utils/rule-tester.js"
+
+const ruleTester = createRuleTester()
+const filename = resolveFixturePath("consumer.ts")
+const missingNameMessage = "Cannot find name 'formatGreeting'. Did you mean:\n" +
+  "  - formatGree1ting(name: string): string\n" +
+  "  - FormData: typeof undici.FormData"
+
+ruleTester.run("suggest-missing-names", suggestMissingNamesRule, {
+  valid: [
+    {
+      filename,
+      code: `
+        const formatGreeting = (name: string) => name
+        formatGreeting("ok")
+      `
+    }
+  ],
+  invalid: [
+    {
+      filename,
+      code: `
+        const formatGree1ting = (name: string) => name
+        formatGreeting("ok")
+      `,
+      errors: [{
+        messageId: "suggestMissingNames",
+        data: {
+          message: missingNameMessage
+        }
+      }]
+    }
+  ]
+})

package/tests/rules/suggest-module-paths.test.ts

@@ -0,0 +1,54 @@
+import { suggestModulePathsRule } from "../../src/rules/suggest-module-paths/index.js"
+import { createRuleTester, resolveFixturePath } from "../utils/rule-tester.js"
+
+const ruleTester = createRuleTester()
+const filename = resolveFixturePath("consumer.ts")
+const localModulePathMessage = "Cannot find module \"./module-paths/alhpa\". Did you mean:\n" +
+  "  - ./module-paths/alpha\n" +
+  "  - ./module-paths/beta"
+const externalModulePathMessage =
+  "Cannot find module 'eff1ect' or its corresponding type declarations. Did you mean:\n" +
+  "  - effect\n" +
+  "  - @effect/cli\n" +
+  "  - @effect/rpc\n" +
+  "  - @effect/sql"
+
+ruleTester.run("suggest-module-paths", suggestModulePathsRule, {
+  valid: [
+    {
+      filename,
+      code: `
+        import { alpha } from "./module-paths/alpha"
+        alpha
+      `
+    }
+  ],
+  invalid: [
+    {
+      filename,
+      code: `
+        import { alpha } from "./module-paths/alhpa"
+        alpha
+      `,
+      errors: [{
+        messageId: "suggestModulePaths",
+        data: {
+          message: localModulePathMessage
+        }
+      }]
+    },
+    {
+      filename,
+      code: `
+        import { pipe } from "eff1ect"
+        pipe
+      `,
+      errors: [{
+        messageId: "suggestModulePaths",
+        data: {
+          message: externalModulePathMessage
+        }
+      }]
+    }
+  ]
+})

package/tests/utils/rule-tester.ts

@@ -0,0 +1,41 @@
+import path from "node:path"
+import { fileURLToPath } from "node:url"
+
+import * as tsParser from "@typescript-eslint/parser"
+import { RuleTester } from "@typescript-eslint/rule-tester"
+import * as vitest from "vitest"
+
+RuleTester.afterAll = vitest.afterAll
+RuleTester.it = vitest.it
+RuleTester.itOnly = vitest.it.only
+RuleTester.describe = vitest.describe
+
+export const tsconfigRootDir = path.join(
+  path.dirname(fileURLToPath(import.meta.url)),
+  "..",
+  ".."
+)
+
+export const createRuleTester = () =>
+  new RuleTester({
+    languageOptions: {
+      parser: tsParser,
+      parserOptions: {
+        projectService: {
+          allowDefaultProject: ["*.ts*"],
+          defaultProject: "tsconfig.json"
+        },
+        tsconfigRootDir
+      }
+    }
+  })
+
+export const resolveFixturePath = (relativePath: string): string =>
+  path.join(tsconfigRootDir, "tests", "fixtures", relativePath)
+
+const normalizePath = (value: string): string => value.replaceAll("\\", "/")
+
+const stripExtension = (value: string): string => value.replace(/\.[^/.]+$/, "")
+
+export const resolveFixtureImportPath = (relativePath: string): string =>
+  stripExtension(normalizePath(resolveFixturePath(relativePath)))

package/tsconfig.build.json

@@ -0,0 +1,13 @@
+{
+	"extends": "./tsconfig.json",
+	"compilerOptions": {
+		"rootDir": "src",
+		"outDir": "lib",
+		"declaration": true,
+		"declarationMap": true,
+		"sourceMap": true,
+		"types": ["node"]
+	},
+	"include": ["src/**/*.ts"],
+	"exclude": ["tests", "dist", "node_modules"]
+}

package/tsconfig.json

@@ -0,0 +1,22 @@
+{
+	"extends": "../../tsconfig.base.json",
+	"compilerOptions": {
+		"rootDir": ".",
+		"outDir": "dist",
+		"types": ["vitest"],
+		"baseUrl": ".",
+		"paths": {
+			"@/*": ["src/*"]
+		}
+	},
+	"include": [
+		"src/**/*",
+		"tests/**/*",
+		"scripts/**/*",
+		"types/**/*.d.ts",
+		"vite.config.ts",
+		"vitest.config.ts",
+		"eslint.config.mts"
+	],
+	"exclude": ["dist", "node_modules"]
+}

package/types/eslint-plugins.d.ts

@@ -0,0 +1,15 @@
+declare module "eslint-plugin-sort-destructure-keys" {
+	import type { ESLint } from "eslint";
+
+	const plugin: ESLint.Plugin;
+	export default plugin;
+}
+
+declare module "@eslint-community/eslint-plugin-eslint-comments/configs" {
+	import type { Linter } from "eslint";
+
+	const configs: {
+		readonly recommended: Linter.Config;
+	};
+	export default configs;
+}

package/vite.config.ts

@@ -0,0 +1,33 @@
+import { defineConfig } from "vite"
+import tsconfigPaths from "vite-tsconfig-paths"
+
+const baseUrl = new URL(".", import.meta.url)
+const normalizePath = (value: string): string =>
+  value.startsWith("/") && value.includes(":") ? value.slice(1) : value
+const resolvePath = (relative: string): string =>
+  normalizePath(decodeURIComponent(new URL(relative, baseUrl).pathname))
+
+export default defineConfig({
+  plugins: [tsconfigPaths()],
+  publicDir: false,
+  resolve: {
+    alias: {
+      "@": resolvePath("src")
+    }
+  },
+  build: {
+    target: "node20",
+    outDir: "dist",
+    sourcemap: true,
+    ssr: "src/rules/main.ts",
+    rollupOptions: {
+      output: {
+        format: "es",
+        entryFileNames: "main.js"
+      }
+    }
+  },
+  ssr: {
+    target: "node"
+  }
+})

package/vitest.config.ts

@@ -0,0 +1,87 @@
+// CHANGE: Migrate from Jest to Vitest with mathematical equivalence
+// WHY: Faster execution, native ESM, Effect integration via @effect/vitest
+// QUOTE(ТЗ): "Проект использует Effect + функциональную парадигму"
+// REF: Migration from jest.config.mjs
+// PURITY: SHELL (configuration only)
+// INVARIANT: ∀ test: behavior_jest ≡ behavior_vitest
+// EFFECT: Effect<TestReport, never, TestEnvironment>
+// COMPLEXITY: O(n) test execution where n = |test_files|
+
+import tsconfigPaths from "vite-tsconfig-paths"
+import { defineConfig } from "vitest/config"
+
+const baseUrl = new URL(".", import.meta.url)
+const normalizePath = (value: string): string =>
+  value.startsWith("/") && value.includes(":") ? value.slice(1) : value
+const resolvePath = (relative: string): string =>
+  normalizePath(decodeURIComponent(new URL(relative, baseUrl).pathname))
+
+export default defineConfig({
+  plugins: [tsconfigPaths()], // Resolves @/* paths from tsconfig
+  test: {
+    // CHANGE: Native ESM support without experimental flags
+    // WHY: Vitest designed for ESM, no need for --experimental-vm-modules
+    // INVARIANT: Deterministic test execution without side effects
+    globals: false, // IMPORTANT: Use explicit imports for type safety
+    environment: "node",
+    testTimeout: 15000,
+
+    // CHANGE: Match Jest's test file patterns
+    // INVARIANT: Same test discovery as Jest
+    include: ["tests/**/*.{test,spec}.ts"],
+    exclude: ["node_modules", "dist", "dist-test"],
+
+    // CHANGE: Coverage with 100% threshold for CORE (same as Jest)
+    // WHY: CORE must maintain mathematical guarantees via complete coverage
+    // INVARIANT: coverage_vitest ≥ coverage_jest ∧ ∀ f ∈ CORE: coverage(f) = 100%
+    coverage: {
+      provider: "v8", // Faster than babel (istanbul), native V8 coverage
+      reporter: ["text", "json", "html"],
+      include: ["src/**/*.ts"],
+      exclude: [
+        "src/**/*.test.ts",
+        "src/**/*.spec.ts",
+        "src/**/__tests__/**",
+        "scripts/**/*.ts"
+      ],
+      // CHANGE: Maintain exact same thresholds as Jest
+      // WHY: Enforce 100% coverage for CORE, 10% minimum for SHELL
+      // INVARIANT: ∀ f ∈ src/core/**/*.ts: all_metrics(f) = 100%
+      // NOTE: Vitest v8 provider collects coverage for all matched files by default
+      thresholds: {
+        "src/core/**/*.ts": {
+          branches: 100,
+          functions: 100,
+          lines: 100,
+          statements: 100
+        },
+        global: {
+          branches: 10,
+          functions: 10,
+          lines: 10,
+          statements: 10
+        }
+      }
+    },
+
+    // CHANGE: Faster test execution via thread pooling
+    // WHY: Vitest uses worker threads by default (faster than Jest's processes)
+    // COMPLEXITY: O(n/k) where n = tests, k = worker_count
+    // NOTE: Vitest runs tests in parallel by default, no additional config needed
+
+    // CHANGE: Clear mocks between tests (Jest equivalence)
+    // WHY: Prevent test contamination, ensure test independence
+    // INVARIANT: ∀ test_i, test_j: independent(test_i, test_j) ⇒ no_shared_state
+    clearMocks: true,
+    mockReset: true,
+    restoreMocks: true
+    // CHANGE: Disable globals to enforce explicit imports
+    // WHY: Type safety, explicit dependencies, functional purity
+    // NOTE: Tests must import { describe, it, expect } from "vitest"
+  },
+  resolve: {
+    alias: {
+      "@": resolvePath("src")
+    }
+  }
+})