@storm-software/linting-tools 1.30.2 → 1.30.4

package/src/eslint/rules/import.ts

@@ -1,88 +0,0 @@
-import { Linter } from "eslint";
-
-const config: Linter.RulesRecord = {
-  /**
-   * Disallow non-import statements appearing before import statements.
-   *
-   * 🚫 Not fixable - https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/first.md
-   */
-  "import/first": "error",
-  /**
-   * Require a newline after the last import/require.
-   *
-   * 🔧 Fixable - https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/newline-after-import.md
-   */
-  "import/newline-after-import": "warn",
-  /**
-   * Disallow import of modules using absolute paths.
-   *
-   * 🚫 Not fixable - https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-absolute-path.md
-   */
-  "import/no-absolute-path": "error",
-  /**
-   * Disallow cyclical dependencies between modules.
-   *
-   * 🚫 Not fixable - https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-cycle.md
-   */
-  "import/no-cycle": "error",
-  /**
-   * Disallow default exports.
-   *
-   * 🚫 Not fixable - https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-default-export.md
-   */
-  "import/no-default-export": "error",
-  /**
-   * Disallow the use of extraneous packages.
-   *
-   * 🚫 Not fixable - https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-extraneous-dependencies.md
-   */
-  "import/no-extraneous-dependencies": [
-    "error",
-    { includeInternal: true, includeTypes: true }
-  ],
-  /**
-   * Disallow mutable exports.
-   *
-   * 🚫 Not fixable - https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-mutable-exports.md
-   */
-  "import/no-mutable-exports": "error",
-  /**
-   * Disallow importing packages through relative paths.
-   *
-   * 🚫 Not fixable - https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-relative-packages.md
-   */
-  "import/no-relative-packages": "warn",
-  /**
-   * Disallow a module from importing itself.
-   *
-   * 🚫 Not fixable - https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-self-import.md
-   */
-  "import/no-self-import": "error",
-  /**
-   * Ensures that there are no useless path segments.
-   *
-   * 🚫 Not fixable - https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/no-useless-path-segments.md
-   */
-  "import/no-useless-path-segments": ["error"],
-  /**
-   * Enforce a module import order convention.
-   *
-   * 🔧 Fixable - https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/order.md
-   */
-  "import/order": [
-    "warn",
-    {
-      groups: [
-        "builtin", // Node.js built-in modules
-        "external", // Packages
-        "internal", // Aliased modules
-        "parent", // Relative parent
-        "sibling", // Relative sibling
-        "index" // Relative index
-      ],
-      "newlines-between": "never"
-    }
-  ]
-};
-
-export default config;

package/src/eslint/rules/jsx-a11y.ts

@@ -1,8 +0,0 @@
-import { Linter } from "eslint";
-
-const config: Linter.RulesRecord = {
-  // This rule has been deprecated, but not yet removed.
-  "jsx-a11y/no-onchange": "off"
-};
-
-export default config;

package/src/eslint/rules/ts-docs.ts

@@ -1,12 +0,0 @@
-import { Linter } from "eslint";
-
-const config: Linter.RulesRecord = {
-  /**
-   * Require TSDoc comments conform to the TSDoc specification.
-   *
-   * 🚫 Not fixable - https://github.com/microsoft/tsdoc/tree/master/eslint-plugin
-   */
-  "tsdoc/syntax": "error"
-};
-
-export default config;

package/src/eslint/rules/unicorn.ts

@@ -1,23 +0,0 @@
-import { Linter } from "eslint";
-
-const config: Linter.RulesRecord = {
-  /**
-   * Require consistent filename case for all linted files.
-   *
-   * 🚫 Not fixable - https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/filename-case.md
-   */
-  "unicorn/filename-case": [
-    "error",
-    {
-      case: "kebabCase"
-    }
-  ],
-  /**
-   * Require using the `node:` protocol when importing Node.js built-in modules.
-   *
-   * 🔧 Fixable - https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-node-protocol.md
-   */
-  "unicorn/prefer-node-protocol": "warn"
-};
-
-export default config;

package/src/eslint/typescript/index.ts

@@ -1,144 +0,0 @@
-import { Linter } from "eslint";
-import {
-  RESTRICTED_GLOBALS,
-  RESTRICTED_MODULES,
-  RESTRICTED_SYNTAX
-} from "../constants";
-import importRules from "../rules/import";
-import stormRules from "../rules/storm";
-import tsDocsRules from "../rules/ts-docs";
-import unicornRules from "../rules/unicorn";
-
-const config: Linter.Config = {
-  root: true,
-  overrides: [
-    {
-      files: ["*.ts", "*.tsx"],
-      extends: ["plugin:@nx/typescript"],
-      rules: {}
-    }
-  ],
-  parser: "@typescript-eslint/parser",
-  extends: [
-    "eslint:recommended",
-    "plugin:@typescript-eslint/recommended",
-    "prettier"
-  ],
-  plugins: ["sonarjs", "unicorn", "promise", "import", "eslint-plugin-tsdoc"],
-  rules: {
-    // Disallows if statements as the only statement in else blocks
-    // https://eslint.org/docs/rules/no-lonely-if
-    "no-lonely-if": "error",
-    // Disallows the use of console
-    // https://eslint.org/docs/rules/no-console
-    "no-console": "error",
-    // Requires method and property shorthand syntax for object literals
-    // https://eslint.org/docs/rules/object-shorthand
-    "object-shorthand": ["error", "always"],
-    // Disallows loops with a body that allows only one iteration
-    // https://eslint.org/docs/rules/no-unreachable-loop
-    "no-unreachable-loop": "error",
-    "sonarjs/no-one-iteration-loop": "off", // similar to 'no-unreachable-loop' but reports less cases
-    "prefer-arrow-callback": ["error", { allowNamedFunctions: true }],
-
-    "sonarjs/no-unused-collection": "error",
-    "sonarjs/no-identical-conditions": "error",
-    "sonarjs/no-inverted-boolean-check": "error",
-    "sonarjs/no-use-of-empty-return-value": "error",
-    "sonarjs/no-gratuitous-expressions": "error",
-    "sonarjs/no-nested-switch": "error",
-    "unicorn/no-lonely-if": "error",
-    "sonarjs/no-collapsible-if": "off", // same as 'unicorn/no-lonely-if'
-    "unicorn/no-array-push-push": "error",
-    "unicorn/no-instanceof-array": "error",
-    "unicorn/no-empty-file": "error",
-    "unicorn/no-useless-fallback-in-spread": "error",
-    "unicorn/prefer-array-find": "error",
-    "unicorn/no-useless-spread": "error",
-    "unicorn/prefer-includes": "error",
-
-    // Disallows specified syntax
-    // https://eslint.org/docs/rules/no-restricted-syntax
-    "no-restricted-syntax": ["error", ...RESTRICTED_SYNTAX],
-    "no-else-return": ["error", { allowElseIf: false }],
-    "promise/no-nesting": "error",
-
-    "import/extensions": ["error", "ignorePackages"], // Bob when bundling requires to have `.js` extension
-    "import/no-default-export": "error",
-    "import/prefer-default-export": "off", // disable opposite of 'import/no-default-export'
-    "unicorn/filename-case": "error",
-
-    "@typescript-eslint/no-unused-vars": [
-      "error",
-      {
-        argsIgnorePattern: "^_",
-        varsIgnorePattern: "^_" // allow underscores in destructuring
-      }
-    ],
-
-    // Enforce the style of numeric separators by correctly grouping digits
-    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/numeric-separators-style.md
-    "unicorn/numeric-separators-style": "error",
-    // Prefer using the node: protocol when importing Node.js builtin modules
-    // https://github.com/sindresorhus/eslint-plugin-unicorn/blob/main/docs/rules/prefer-node-protocol.md
-    "unicorn/prefer-node-protocol": "error",
-    // Reports any imports that come after non-import statements
-    // https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/first.md
-    "import/first": "error",
-    // Disallow shorthand type conversions
-    // https://eslint.org/docs/latest/rules/no-implicit-coercion
-    "no-implicit-coercion": [
-      "error",
-      {
-        disallowTemplateShorthand: true,
-        // in TypeScript `!!` is preferable https://www.typescriptlang.org/docs/handbook/2/narrowing.html#truthiness-narrowing
-        boolean: false
-      }
-    ],
-    // Disallow specified modules when loaded by `import` declarations
-    // https://github.com/eslint-community/eslint-plugin-n/blob/master/docs/rules/no-restricted-import.md
-    "n/no-restricted-import": ["error", RESTRICTED_MODULES],
-    // Disallow specified modules when loaded by require
-    // https://github.com/eslint-community/eslint-plugin-n/blob/master/docs/rules/no-restricted-require.md
-    "n/no-restricted-require": ["error", RESTRICTED_MODULES],
-    "no-restricted-modules": "off", // deprecated in favor of corresponding rules from `eslint-plugin-n`
-
-    // Disallow specified global variables
-    // https://eslint.org/docs/latest/rules/no-restricted-globals
-    "no-restricted-globals": ["error", ...RESTRICTED_GLOBALS],
-
-    "@typescript-eslint/no-explicit-any": "error",
-    "prefer-const": ["error", { destructuring: "all" }],
-    "import/no-duplicates": "error",
-    "import/newline-after-import": "off", // prettified by prettier-plugin-sort-imports
-    "prefer-object-has-own": "error",
-    "logical-assignment-operators": [
-      "error",
-      "always",
-      { enforceForIfStatements: true }
-    ],
-    "@typescript-eslint/prefer-optional-chain": "error",
-    yoda: "error",
-    "unicorn/prefer-export-from": ["error", { ignoreUsedVariables: true }],
-    "promise/no-multiple-resolved": "error",
-    "unicorn/prefer-logical-operator-over-ternary": "error",
-    "no-unused-expressions": "off",
-    "@typescript-eslint/no-unused-expressions": "error",
-    "no-negated-condition": "off",
-    "unicorn/no-negated-condition": "error",
-    "unicorn/no-array-for-each": "error",
-    "unicorn/prefer-string-trim-start-end": "error",
-    "no-self-compare": "error",
-    eqeqeq: ["error", "always", { null: "ignore" }],
-    "import/no-useless-path-segments": "error",
-    "require-await": "off",
-    "no-return-await": "off",
-
-    ...importRules,
-    ...unicornRules,
-    ...tsDocsRules,
-    ...stormRules
-  }
-};
-
-export default config;

package/src/manypkg/index.ts

@@ -1,179 +0,0 @@
-/* eslint-disable @typescript-eslint/no-explicit-any */
-import { checks } from "@manypkg/cli/src/checks";
-import { Options } from "@manypkg/cli/src/checks/utils";
-import { ExitError } from "@manypkg/cli/src/errors";
-import { npmTagAll } from "@manypkg/cli/src/npm-tag";
-import { runCmd } from "@manypkg/cli/src/run";
-import { upgradeDependency } from "@manypkg/cli/src/upgrade";
-import { install, writePackage } from "@manypkg/cli/src/utils";
-import { Package, Packages, getPackages } from "@manypkg/get-packages";
-import pLimit from "p-limit";
-import spawn from "spawndamnit";
-
-type RootPackage = Package & {
-  packageJson: {
-    manypkg?: Options;
-  };
-};
-type PackagesWithConfig = Packages & {
-  rootPackage?: RootPackage;
-};
-
-const defaultOptions = {
-  defaultBranch: "main"
-};
-
-const runChecks = (
-  allWorkspaces: Map<string, Package>,
-  rootWorkspace: RootPackage | undefined,
-  shouldFix: boolean,
-  options: Options
-) => {
-  let hasErrored = false;
-  let requiresInstall = false;
-  const ignoredRules = new Set(options.ignoredRules || []);
-  for (const [ruleName, check] of Object.entries(checks)) {
-    if (ignoredRules.has(ruleName)) {
-      continue;
-    }
-
-    if (check.type === "all") {
-      for (const [, workspace] of allWorkspaces) {
-        const errors = check.validate(
-          workspace,
-          allWorkspaces,
-          rootWorkspace,
-          options
-        );
-        if (shouldFix && check.fix !== undefined) {
-          for (const error of errors) {
-            const output = check.fix(error as any, options) || {
-              requiresInstall: false
-            };
-            if (output.requiresInstall) {
-              requiresInstall = true;
-            }
-          }
-        } else {
-          for (const error of errors) {
-            hasErrored = true;
-            console.error(check.print(error as any, options));
-          }
-        }
-      }
-    }
-    if (check.type === "root" && rootWorkspace) {
-      const errors = check.validate(rootWorkspace, allWorkspaces, options);
-      if (shouldFix && check.fix !== undefined) {
-        for (const error of errors) {
-          const output = check.fix(error as any, options) || {
-            requiresInstall: false
-          };
-          if (output.requiresInstall) {
-            requiresInstall = true;
-          }
-        }
-      } else {
-        for (const error of errors) {
-          hasErrored = true;
-          console.error(check.print(error as any, options));
-        }
-      }
-    }
-  }
-  return { requiresInstall, hasErrored };
-};
-
-const execLimit = pLimit(4);
-
-async function execCmd(args: string[]) {
-  const { packages } = await getPackages(process.cwd());
-  let highestExitCode = 0;
-  await Promise.all(
-    packages.map(pkg => {
-      return execLimit(async () => {
-        const { code } = await spawn(args[0], args.slice(1), {
-          cwd: pkg.dir,
-          stdio: "inherit"
-        });
-        highestExitCode = Math.max(code, highestExitCode);
-      });
-    })
-  );
-  throw new ExitError(highestExitCode);
-}
-
-export async function runManypkg(
-  manypkgType: string,
-  manypkgArgs: string[],
-  manypkgFix = true
-) {
-  if (manypkgType === "exec") {
-    return execCmd(manypkgArgs.slice(0));
-  }
-  if (manypkgType === "run") {
-    return runCmd(manypkgArgs.slice(0), process.cwd());
-  }
-  if (manypkgType === "upgrade") {
-    return upgradeDependency(manypkgArgs.slice(0));
-  }
-  if (manypkgType === "npm-tag") {
-    return npmTagAll(manypkgArgs.slice(0));
-  }
-
-  if (manypkgType !== "check" && manypkgType !== "fix") {
-    console.error(
-      `command ${manypkgType} not found, only check, exec, run, upgrade, npm-tag and fix exist`
-    );
-    process.exit(1);
-  }
-
-  const { packages, rootPackage, rootDir } = (await getPackages(
-    process.env["STORM_WORKSPACE_ROOT"]
-      ? process.env["STORM_WORKSPACE_ROOT"]
-      : process.cwd()
-  )) as PackagesWithConfig;
-
-  const options: Options = {
-    ...defaultOptions,
-    ...rootPackage?.packageJson.manypkg
-  };
-
-  const packagesByName = new Map<string, Package>(
-    packages.map(x => [x.packageJson.name, x])
-  );
-
-  console.log(`🔍 Checking ${packages.length} packages...`);
-  if (rootPackage) {
-    packagesByName.set(rootPackage.packageJson.name, rootPackage);
-  }
-  const { hasErrored, requiresInstall } = runChecks(
-    packagesByName,
-    rootPackage,
-    manypkgFix,
-    options
-  );
-
-  if (manypkgFix) {
-    await Promise.all(
-      [...packagesByName].map(async ([_, workspace]) => {
-        writePackage(workspace);
-      })
-    );
-    if (requiresInstall) {
-      await install("pnpm", rootDir);
-    }
-
-    if (hasErrored) {
-      console.log("🎉 Fixed workspace packages!");
-    } else {
-      console.log("🎉 Workspace packages are valid!");
-    }
-  } else if (hasErrored) {
-    console.info(
-      "⚠️ The above errors may be fixable if the --manypkg-fix flag is used"
-    );
-  } else {
-    console.log("🎉 Workspace packages are valid!");
-  }
-}

package/tsconfig.json

@@ -1,24 +0,0 @@
-{
-  "extends": "../../tsconfig.base.json",
-  "compilerOptions": {
-    "module": "ESNext",
-    "forceConsistentCasingInFileNames": true,
-    "strict": false,
-    "allowJs": true,
-    "noImplicitOverride": true,
-    "noPropertyAccessFromIndexSignature": false,
-    "noImplicitReturns": true,
-    "noFallthroughCasesInSwitch": true,
-    "esModuleInterop": true
-  },
-  "files": [],
-  "include": [],
-  "references": [
-    {
-      "path": "./tsconfig.lib.json"
-    },
-    {
-      "path": "./tsconfig.spec.json"
-    }
-  ]
-}

package/tsconfig.lib.json

@@ -1,11 +0,0 @@
-{
-  "extends": "./tsconfig.json",
-  "compilerOptions": {
-    "target": "es2022",
-    "outDir": "../../dist/out-tsc",
-    "declaration": true,
-    "types": ["node"]
-  },
-  "include": ["src/**/*.ts", "src/**/*.js", "bin/**/*"],
-  "exclude": ["jest.config.ts", "src/**/*.spec.ts", "src/**/*.test.ts"]
-}

package/tsconfig.spec.json

@@ -1,13 +0,0 @@
-{
-  "extends": "./tsconfig.json",
-  "compilerOptions": {
-    "outDir": "../../dist/out-tsc",
-    "types": ["jest", "node"]
-  },
-  "include": [
-    "jest.config.ts",
-    "src/**/*.test.ts",
-    "src/**/*.spec.ts",
-    "src/**/*.d.ts"
-  ]
-}