feat: add no-import-dist

Author: kirklin

package.json

@@ -85,7 +85,7 @@
     "eslint-plugin-i": "^2.29.1",
     "eslint-plugin-jsdoc": "^46.9.1",
     "eslint-plugin-jsonc": "^2.11.2",
-    "eslint-plugin-kirklin": "^1.0.0",
+    "eslint-plugin-kirklin": "^1.1.0",
     "eslint-plugin-markdown": "^3.0.1",
     "eslint-plugin-n": "^16.5.0",
     "eslint-plugin-no-only-tests": "^3.1.0",

pnpm-lock.yaml

@@ -15,7 +15,7 @@ export interface RuleOptions {
   /**
    * Skip prompts and use default values
    */
-  yes?: boolean
+  yes?: boolean;
 }
 
 export async function run(options: RuleOptions = {}) {

src/cli/run.ts

@@ -24,7 +24,7 @@ export async function imports(options: OptionsStylistic = {}): Promise<FlatConfi
         "import/no-webpack-loader-syntax": "error",
         "import/order": "error",
         "kirklin/import-dedupe": "error",
-        // 'kirklin/no-import-dist': 'error',
+        "kirklin/no-import-dist": "error",
         "kirklin/no-import-node-modules-by-path": "error",
 
         ...stylistic
@@ -38,7 +38,7 @@ export async function imports(options: OptionsStylistic = {}): Promise<FlatConfi
       files: ["**/bin/**/*", `**/bin.${GLOB_SRC_EXT}`],
       name: "kirklin:imports:bin",
       rules: {
-        // 'kirklin/no-import-dist': 'off',
+        "kirklin/no-import-dist": "off",
         "kirklin/no-import-node-modules-by-path": "off",
       },
     },

src/configs/imports.ts

@@ -51,7 +51,7 @@ export type Rules = WrapRuleConfig<
     EslintCommentsRules &
     // TODO: TOML rules
     {
-      "test/no-only-tests": RuleConfig<[]>
+      "test/no-only-tests": RuleConfig<[]>;
     }
   >
 >;
@@ -60,15 +60,15 @@ export type FlatConfigItem = Omit<FlatESLintConfigItem<Rules, false>, "plugins">
   /**
    * Custom name of each config item
    */
-  name?: string
+  name?: string;
 
   // Relax plugins type limitation, as most of the plugins did not have correct type info yet.
   /**
    * An object containing a name-value mapping of plugin names to plugin objects. When `files` is specified, these plugins are only available to the matching files.
    *
    * @see [Using plugins in your configuration](https://eslint.org/docs/latest/user-guide/configuring/configuration-files-new#using-plugins-in-your-configuration)
    */
-  plugins?: Record<string, any>
+  plugins?: Record<string, any>;
 };
 
 export type UserConfigItem = FlatConfigItem | Linter.FlatConfig;
@@ -77,7 +77,7 @@ export interface OptionsFiles {
   /**
    * Override the `files` option to provide custom globs.
    */
-  files?: string[]
+  files?: string[];
 }
 
 export interface OptionsVue extends OptionsOverrides {
@@ -87,14 +87,14 @@ export interface OptionsVue extends OptionsOverrides {
    * @see https://github.com/kirklin/eslint-processor-vue-blocks
    * @default true
    */
-  sfcBlocks?: boolean | VueBlocksOptions
+  sfcBlocks?: boolean | VueBlocksOptions;
 
   /**
    * Vue version. Apply different rules set from `eslint-plugin-vue`.
    *
    * @default 3
    */
-  vueVersion?: 2 | 3
+  vueVersion?: 2 | 3;
 }
 
 export type OptionsTypescript =
@@ -107,14 +107,14 @@ export interface OptionsFormatters {
    *
    * Currently only support Prettier.
    */
-  css?: "prettier" | boolean
+  css?: "prettier" | boolean;
 
   /**
    * Enable formatting support for HTML.
    *
    * Currently only support Prettier.
    */
-  html?: "prettier" | boolean
+  html?: "prettier" | boolean;
 
   /**
    * Enable formatting support for Markdown.
@@ -123,26 +123,26 @@ export interface OptionsFormatters {
    *
    * When set to `true`, it will use Prettier.
    */
-  markdown?: "prettier" | "dprint" | boolean
+  markdown?: "prettier" | "dprint" | boolean;
 
   /**
    * Enable formatting support for GraphQL.
    */
-  graphql?: "prettier" | boolean
+  graphql?: "prettier" | boolean;
 
   /**
    * Custom options for Prettier.
    *
    * By default it's controlled by our own config.
    */
-  prettierOptions?: VendoredPrettierOptions
+  prettierOptions?: VendoredPrettierOptions;
 
   /**
    * Custom options for dprint.
    *
    * By default it's controlled by our own config.
    */
-  dprintOptions?: boolean
+  dprintOptions?: boolean;
 }
 
 export interface OptionsComponentExts {
@@ -152,60 +152,60 @@ export interface OptionsComponentExts {
    * @example ['vue']
    * @default []
    */
-  componentExts?: string[]
+  componentExts?: string[];
 }
 
 export interface OptionsTypeScriptParserOptions {
   /**
    * Additional parser options for TypeScript.
    */
-  parserOptions?: Partial<ParserOptions>
+  parserOptions?: Partial<ParserOptions>;
 
   /**
    * Glob patterns for files that should be type aware.
    * @default ['**\/*.{ts,tsx}']
    */
-  filesTypeAware?: string[]
+  filesTypeAware?: string[];
 }
 
 export interface OptionsTypeScriptWithTypes {
   /**
    * When this options is provided, type aware rules will be enabled.
    * @see https://typescript-eslint.io/linting/typed-linting/
    */
-  tsconfigPath?: string | string[]
+  tsconfigPath?: string | string[];
 }
 
 export interface OptionsHasTypeScript {
-  typescript?: boolean
+  typescript?: boolean;
 }
 
 export interface OptionsStylistic {
-  stylistic?: boolean | StylisticConfig
+  stylistic?: boolean | StylisticConfig;
 }
 
 export interface StylisticConfig extends Pick<StylisticCustomizeOptions, "indent" | "quotes" | "jsx" | "semi"> {
 }
 
 export interface OptionsOverrides {
-  overrides?: FlatConfigItem["rules"]
+  overrides?: FlatConfigItem["rules"];
 }
 
 export interface OptionsIsInEditor {
-  isInEditor?: boolean
+  isInEditor?: boolean;
 }
 
 export interface OptionsUnoCSS extends OptionsOverrides {
   /**
    * Enable attributify support.
    * @default true
    */
-  attributify?: boolean
+  attributify?: boolean;
   /**
    * Enable strict mode by throwing errors about blocklisted classes.
    * @default false
    */
-  strict?: boolean
+  strict?: boolean;
 }
 
 export interface OptionsConfig extends OptionsComponentExts {
@@ -217,12 +217,12 @@ export interface OptionsConfig extends OptionsComponentExts {
    * @see https://github.com/kirklin/eslint-config-flat-gitignore
    * @default true
    */
-  gitignore?: boolean | FlatGitignoreOptions
+  gitignore?: boolean | FlatGitignoreOptions;
 
   /**
    * Core rules. Can't be disabled.
    */
-  javascript?: OptionsOverrides
+  javascript?: OptionsOverrides;
 
   /**
    * Enable TypeScript support.
@@ -231,7 +231,7 @@ export interface OptionsConfig extends OptionsComponentExts {
    *
    * @default auto-detect based on the dependencies
    */
-  typescript?: boolean | OptionsTypescript
+  typescript?: boolean | OptionsTypescript;
 
   /**
    * Enable JSX related rules.
@@ -240,42 +240,42 @@ export interface OptionsConfig extends OptionsComponentExts {
    *
    * @default true
    */
-  jsx?: boolean
+  jsx?: boolean;
 
   /**
    * Enable test support.
    *
    * @default true
    */
-  test?: boolean | OptionsOverrides
+  test?: boolean | OptionsOverrides;
 
   /**
    * Enable Vue support.
    *
    * @default auto-detect based on the dependencies
    */
-  vue?: boolean | OptionsVue
+  vue?: boolean | OptionsVue;
 
   /**
    * Enable JSONC support.
    *
    * @default true
    */
-  jsonc?: boolean | OptionsOverrides
+  jsonc?: boolean | OptionsOverrides;
 
   /**
    * Enable YAML support.
    *
    * @default true
    */
-  yaml?: boolean | OptionsOverrides
+  yaml?: boolean | OptionsOverrides;
 
   /**
    * Enable TOML support.
    *
    * @default true
    */
-  toml?: boolean | OptionsOverrides
+  toml?: boolean | OptionsOverrides;
 
   /**
    * Enable linting for **code snippets** in Markdown.
@@ -284,14 +284,14 @@ export interface OptionsConfig extends OptionsComponentExts {
    *
    * @default true
    */
-  markdown?: boolean | OptionsOverrides
+  markdown?: boolean | OptionsOverrides;
 
   /**
    * Enable stylistic rules.
    *
    * @default true
    */
-  stylistic?: boolean | StylisticConfig
+  stylistic?: boolean | StylisticConfig;
 
   /**
    * Enable react rules.
@@ -303,7 +303,7 @@ export interface OptionsConfig extends OptionsComponentExts {
    *
    * @default false
    */
-  react?: boolean | OptionsOverrides
+  react?: boolean | OptionsOverrides;
 
   /**
    * Enable svelte rules.
@@ -313,7 +313,7 @@ export interface OptionsConfig extends OptionsComponentExts {
    *
    * @default false
    */
-  svelte?: boolean
+  svelte?: boolean;
 
   /**
    * Enable unocss rules.
@@ -323,7 +323,7 @@ export interface OptionsConfig extends OptionsComponentExts {
    *
    * @default false
    */
-  unocss?: boolean | OptionsUnoCSS
+  unocss?: boolean | OptionsUnoCSS;
 
   /**
    * Use external formatters to format files.
@@ -335,30 +335,30 @@ export interface OptionsConfig extends OptionsComponentExts {
    *
    * @default false
    */
-  formatters?: boolean | OptionsFormatters
+  formatters?: boolean | OptionsFormatters;
 
   /**
    * Control to disable some rules in editors.
    * @default auto-detect based on the process.env
    */
-  isInEditor?: boolean
+  isInEditor?: boolean;
 
   /**
    * Provide overrides for rules for each integration.
    *
    * @deprecated use `overrides` option in each integration key instead
    */
   overrides?: {
-    stylistic?: FlatConfigItem["rules"]
-    javascript?: FlatConfigItem["rules"]
-    typescript?: FlatConfigItem["rules"]
-    test?: FlatConfigItem["rules"]
-    vue?: FlatConfigItem["rules"]
-    jsonc?: FlatConfigItem["rules"]
-    markdown?: FlatConfigItem["rules"]
-    yaml?: FlatConfigItem["rules"]
-    toml?: FlatConfigItem["rules"]
-    react?: FlatConfigItem["rules"]
-    svelte?: FlatConfigItem["rules"]
-  }
+    stylistic?: FlatConfigItem["rules"];
+    javascript?: FlatConfigItem["rules"];
+    typescript?: FlatConfigItem["rules"];
+    test?: FlatConfigItem["rules"];
+    vue?: FlatConfigItem["rules"];
+    jsonc?: FlatConfigItem["rules"];
+    markdown?: FlatConfigItem["rules"];
+    yaml?: FlatConfigItem["rules"];
+    toml?: FlatConfigItem["rules"];
+    react?: FlatConfigItem["rules"];
+    svelte?: FlatConfigItem["rules"];
+  };
 }