@teambit/eslint 1.0.108 → 1.0.110

package/artifacts/preview/teambit_defender_eslint-preview.js

@@ -0,0 +1 @@
+!function(e,t){"object"==typeof exports&&"object"==typeof module?module.exports=t():"function"==typeof define&&define.amd?define([],t):"object"==typeof exports?exports["teambit.defender/eslint-preview"]=t():e["teambit.defender/eslint-preview"]=t()}(self,(()=>(()=>{"use strict";var e={66918:(e,t,o)=>{var r={id:"teambit.defender/eslint@1.0.110",homepage:"https://bit.cloud/teambit/defender/eslint",exported:!0};function n(){const e=i(o(87363));return n=function(){return e},e}function i(e){return e&&e.__esModule?e:{default:e}}Object.defineProperty(t,"__esModule",{value:!0}),t.Logo=void 0,n.__bit_component=r,i.__bit_component=r;const p=()=>n().default.createElement("div",{style:{height:"100%",display:"flex",justifyContent:"center"}},n().default.createElement("img",{style:{width:70},src:"https://static.bit.dev/brands/logo-eslint.svg"}));p.__bit_component=r,t.Logo=p},87363:e=>{e.exports=React}},t={};function o(r){var n=t[r];if(void 0!==n)return n.exports;var i=t[r]={exports:{}};return e[r](i,i.exports,o),i.exports}o.d=(e,t)=>{for(var r in t)o.o(t,r)&&!o.o(e,r)&&Object.defineProperty(e,r,{enumerable:!0,get:t[r]})},o.o=(e,t)=>Object.prototype.hasOwnProperty.call(e,t),o.r=e=>{"undefined"!=typeof Symbol&&Symbol.toStringTag&&Object.defineProperty(e,Symbol.toStringTag,{value:"Module"}),Object.defineProperty(e,"__esModule",{value:!0})};var r={};return(()=>{o.r(r),o.d(r,{compositions:()=>l,compositions_metadata:()=>u,overview:()=>f});var e={};o.r(e),o.d(e,{default:()=>c});var t=o(66918);o(87363);const n=MdxJsReact,i=TeambitMdxUiMdxScopeContext;var p=["components"];function s(){return s=Object.assign?Object.assign.bind():function(e){for(var t=1;t<arguments.length;t++){var o=arguments[t];for(var r in o)Object.prototype.hasOwnProperty.call(o,r)&&(e[r]=o[r])}return e},s.apply(this,arguments)}var a={},d="wrapper";function c(e){var t=e.components,o=function(e,t){if(null==e)return{};var o,r,n=function(e,t){if(null==e)return{};var o,r,n={},i=Object.keys(e);for(r=0;r<i.length;r++)o=i[r],t.indexOf(o)>=0||(n[o]=e[o]);return n}(e,t);if(Object.getOwnPropertySymbols){var i=Object.getOwnPropertySymbols(e);for(r=0;r<i.length;r++)o=i[r],t.indexOf(o)>=0||Object.prototype.propertyIsEnumerable.call(e,o)&&(n[o]=e[o])}return n}(e,p);return(0,n.mdx)(d,s({},a,o,{components:t,mdxType:"MDXLayout"}),(0,n.mdx)(i.MDXScopeProvider,{components:{},mdxType:"MDXScopeProvider"}))}c.isMDXComponent=!0;const l=[t],f=[e],u={compositions:[{displayName:"Logo",identifier:"Logo"}]}})(),r})()));

package/dist/{preview-1703647408454.js → preview-1703733956734.js}

@@ -1,5 +1,5 @@
-import * as compositions_0 from '/home/circleci/Library/Caches/Bit/capsules/8891be5ad3d35bfc38b9cd90c0e05b598a5a55af/teambit.defender_eslint@1.0.108/dist/eslint.composition.js';
-import * as overview_0 from '/home/circleci/Library/Caches/Bit/capsules/8891be5ad3d35bfc38b9cd90c0e05b598a5a55af/teambit.defender_eslint@1.0.108/dist/eslint.docs.mdx';
+import * as compositions_0 from '/home/circleci/Library/Caches/Bit/capsules/8891be5ad3d35bfc38b9cd90c0e05b598a5a55af/teambit.defender_eslint@1.0.110/dist/eslint.composition.js';
+import * as overview_0 from '/home/circleci/Library/Caches/Bit/capsules/8891be5ad3d35bfc38b9cd90c0e05b598a5a55af/teambit.defender_eslint@1.0.110/dist/eslint.docs.mdx';
 
 export const compositions = [compositions_0];
 export const overview = [overview_0];

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "@teambit/eslint",
-  "version": "1.0.108",
+  "version": "1.0.110",
   "homepage": "https://bit.cloud/teambit/defender/eslint",
   "main": "dist/index.js",
   "componentId": {
     "scope": "teambit.defender",
     "name": "eslint",
-    "version": "1.0.108"
+    "version": "1.0.110"
   },
   "dependencies": {
     "eslint": "7.32.0",
@@ -16,12 +16,12 @@
     "p-map-series": "2.1.0",
     "@babel/runtime": "7.23.2",
     "@teambit/harmony": "0.4.6",
-    "@teambit/component": "1.0.108",
-    "@teambit/linter": "1.0.108",
-    "@teambit/logger": "0.0.933",
-    "@teambit/cli": "0.0.840",
+    "@teambit/component": "1.0.110",
+    "@teambit/linter": "1.0.110",
+    "@teambit/logger": "0.0.935",
+    "@teambit/cli": "0.0.842",
     "@teambit/defender.eslint.config-mutator": "0.0.96",
-    "@teambit/workspace-config-files": "1.0.108"
+    "@teambit/workspace-config-files": "1.0.110"
   },
   "devDependencies": {
     "@types/eslint": "7.28.0",
@@ -30,7 +30,7 @@
     "@types/object-hash": "1.3.4",
     "@types/jest": "^29.2.2",
     "@types/testing-library__jest-dom": "^5.9.5",
-    "@teambit/harmony.envs.core-aspect-env": "0.0.13"
+    "@teambit/harmony.envs.core-aspect-env": "0.0.15"
   },
   "peerDependencies": {
     "react": "^17.0.0 || ^18.0.0",
@@ -40,11 +40,15 @@
   "optionalDependencies": {},
   "peerDependenciesMeta": {},
   "exports": {
-    "node": {
-      "require": "./dist/index.js",
-      "import": "./dist/esm.mjs"
+    ".": {
+      "node": {
+        "require": "./dist/index.js",
+        "import": "./dist/esm.mjs"
+      },
+      "default": "./dist/index.js"
     },
-    "default": "./dist/index.js"
+    "./dist/*": "./dist/*",
+    "./artifacts/*": "./artifacts/*"
   },
   "private": false,
   "engines": {

package/eslint.aspect.ts

@@ -1,5 +0,0 @@
-import { Aspect } from '@teambit/harmony';
-
-export const ESLintAspect = Aspect.create({
-  id: 'teambit.defender/eslint',
-});

package/eslint.linter.ts

@@ -1,258 +0,0 @@
-import fs from 'fs-extra';
-import path from 'path';
-import { flatten, compact } from 'lodash';
-import { Linter, LinterContext, LintResults, ComponentLintResult } from '@teambit/linter';
-import { ESLint as ESLintLib } from 'eslint';
-import mapSeries from 'p-map-series';
-import objectHash from 'object-hash';
-import { ComponentMap } from '@teambit/component';
-import { Logger } from '@teambit/logger';
-import { ESLintOptions } from './eslint.main.runtime';
-
-export class ESLintLinter implements Linter {
-  constructor(
-    private logger: Logger,
-
-    private options: ESLintOptions,
-
-    /**
-     * reference to the eslint module.
-     */
-    private ESLint?: any
-  ) {}
-
-  id = 'eslint-linter';
-  displayName = 'ESlint';
-
-  displayConfig() {
-    return JSON.stringify(this.options, null, 2);
-  }
-
-  async lint(context: LinterContext): Promise<LintResults> {
-    const longProcessLogger = this.logger.createLongProcessLogger('linting components', context.components.length);
-    const eslint = this.createEslint(this.options.config, this.ESLint);
-    if (this.options.tsConfig && context.rootDir) {
-      const tsConfigPath = this.createTempTsConfigFile(
-        context.rootDir,
-        context.componentsDirMap,
-        context.envRuntime.id,
-        this.options.tsConfig
-      );
-      if (this.options?.config?.overrideConfig?.parserOptions) {
-        this.options.config.overrideConfig.parserOptions.project = tsConfigPath;
-      }
-    }
-    const resultsP = mapSeries(context.components, async (component) => {
-      longProcessLogger.logProgress(
-        `component: ${component.id.toString()}, # of files: ${component.filesystem.files.length}`
-      );
-      const filesP = component.filesystem.files.map(async (file) => {
-        // TODO: now that we moved to lint files, maybe it's not required anymore
-        // The eslint api will not ignore extensions by default when using lintText, so we do it manually
-        if (!this.options.extensions?.includes(file.extname)) return undefined;
-        return file.path;
-      });
-
-      const files = compact(await Promise.all(filesP));
-      const lintResults = await eslint.lintFiles(files);
-
-      if (eslint && this.options.config.fix && lintResults) {
-        await ESLintLib.outputFixes(lintResults);
-      }
-
-      const results: ESLintLib.LintResult[] = compact(flatten(lintResults));
-      const formatter = await eslint.loadFormatter(this.options.formatter || 'stylish');
-      const output = formatter.format(results);
-      const {
-        totalErrorCount,
-        totalFatalErrorCount,
-        totalFixableErrorCount,
-        totalFixableWarningCount,
-        totalWarningCount,
-        componentsResults,
-      } = this.computeComponentResultsWithTotals(results);
-
-      return {
-        component,
-        output,
-        totalErrorCount,
-        totalFatalErrorCount,
-        totalFixableErrorCount,
-        totalFixableWarningCount,
-        totalWarningCount,
-        results: componentsResults,
-      };
-    });
-
-    const results = (await resultsP) as any as ComponentLintResult[];
-    const {
-      totalErrorCount,
-      totalFatalErrorCount,
-      totalFixableErrorCount,
-      totalFixableWarningCount,
-      totalWarningCount,
-      totalComponentsWithErrorCount,
-      totalComponentsWithFatalErrorCount,
-      totalComponentsWithFixableErrorCount,
-      totalComponentsWithFixableWarningCount,
-      totalComponentsWithWarningCount,
-    } = this.computeManyComponentsTotals(results);
-
-    return {
-      totalErrorCount,
-      totalFatalErrorCount,
-      totalFixableErrorCount,
-      totalFixableWarningCount,
-      totalWarningCount,
-      totalComponentsWithErrorCount,
-      totalComponentsWithFatalErrorCount,
-      totalComponentsWithFixableErrorCount,
-      totalComponentsWithFixableWarningCount,
-      totalComponentsWithWarningCount,
-      results,
-      errors: [],
-    };
-  }
-
-  private createTempTsConfigFile(
-    rootDir: string,
-    componentDirMap: ComponentMap<string>,
-    envId: string,
-    tsConfig: Record<string, any>
-  ): string {
-    const newTsConfig = {
-      ...tsConfig,
-    };
-    const compDirs: string[] = componentDirMap.toArray().map(([, compDir]) => compDir);
-    if (tsConfig.include) {
-      newTsConfig.include = flatten(
-        tsConfig.include.map((includedPath) => {
-          return compDirs.map((compDir) => `../../${compDir}/${includedPath}`);
-        })
-      );
-    }
-    if (tsConfig.exclude) {
-      newTsConfig.exclude = flatten(
-        tsConfig.exclude.map((excludedPath) => {
-          return compDirs.map((compDir) => `../../${compDir}/${excludedPath}`);
-        })
-      );
-    }
-    const cacheDir = getCacheDir(rootDir);
-    const hash = objectHash(newTsConfig);
-    // We save the tsconfig with hash here to avoid creating unnecessary tsconfig files
-    // this is very important as eslint will be able to cache the tsconfig file and will not need to create another program
-    // this affects performance dramatically
-    const tempTsConfigPath = path.join(cacheDir, `bit.tsconfig.eslint.${hash}.json`);
-    if (!fs.existsSync(tempTsConfigPath)) {
-      fs.outputJSONSync(tempTsConfigPath, newTsConfig, { spaces: 2 });
-    }
-    return tempTsConfigPath;
-  }
-
-  private computeComponentResultsWithTotals(results: ESLintLib.LintResult[]) {
-    let totalErrorCount = 0;
-    let totalFatalErrorCount = 0;
-    let totalFixableErrorCount = 0;
-    let totalFixableWarningCount = 0;
-    let totalWarningCount = 0;
-    const componentsResults = results.map((result) => {
-      totalErrorCount += result.errorCount ?? 0;
-      // @ts-ignore - missing from the @types/eslint lib
-      totalFatalErrorCount += result.fatalErrorCount ?? 0;
-      totalFixableErrorCount += result.fixableErrorCount ?? 0;
-      totalFixableWarningCount += result.fixableWarningCount ?? 0;
-      totalWarningCount += result.warningCount ?? 0;
-      return {
-        filePath: result.filePath,
-        errorCount: result.errorCount,
-        // @ts-ignore - missing from the @types/eslint lib
-        fatalErrorCount: result.fatalErrorCount,
-        fixableErrorCount: result.fixableErrorCount,
-        fixableWarningCount: result.fixableWarningCount,
-        warningCount: result.warningCount,
-        messages: result.messages,
-        raw: result,
-      };
-    });
-    return {
-      totalErrorCount,
-      totalFatalErrorCount,
-      totalFixableErrorCount,
-      totalFixableWarningCount,
-      totalWarningCount,
-      componentsResults,
-    };
-  }
-
-  private computeManyComponentsTotals(componentsResults: ComponentLintResult[]) {
-    let totalErrorCount = 0;
-    let totalFatalErrorCount = 0;
-    let totalFixableErrorCount = 0;
-    let totalFixableWarningCount = 0;
-    let totalWarningCount = 0;
-    let totalComponentsWithErrorCount = 0;
-    let totalComponentsWithFatalErrorCount = 0;
-    let totalComponentsWithFixableErrorCount = 0;
-    let totalComponentsWithFixableWarningCount = 0;
-    let totalComponentsWithWarningCount = 0;
-
-    componentsResults.forEach((result) => {
-      if (result.totalErrorCount) {
-        totalErrorCount += result.totalErrorCount;
-        totalComponentsWithErrorCount += 1;
-      }
-      // @ts-ignore - missing from the @types/eslint lib
-      if (result.totalFatalErrorCount) {
-        totalFatalErrorCount += result.totalFatalErrorCount;
-        totalComponentsWithFatalErrorCount += 1;
-      }
-      if (result.totalFixableErrorCount) {
-        totalFixableErrorCount += result.totalFixableErrorCount;
-        totalComponentsWithFixableErrorCount += 1;
-      }
-      if (result.totalFixableWarningCount) {
-        totalFixableWarningCount += result.totalFixableWarningCount;
-        totalComponentsWithFixableWarningCount += 1;
-      }
-      if (result.totalWarningCount) {
-        totalWarningCount += result.totalWarningCount;
-        totalComponentsWithWarningCount += 1;
-      }
-    });
-    return {
-      totalErrorCount,
-      totalFatalErrorCount,
-      totalFixableErrorCount,
-      totalFixableWarningCount,
-      totalWarningCount,
-      componentsResults,
-      totalComponentsWithErrorCount,
-      totalComponentsWithFatalErrorCount,
-      totalComponentsWithFixableErrorCount,
-      totalComponentsWithFixableWarningCount,
-      totalComponentsWithWarningCount,
-    };
-  }
-
-  /**
-   * Create the eslint instance by options that was already merged with context
-   * @param options
-   * @param ESLintModule
-   * @returns
-   */
-  private createEslint(options: ESLintLib.Options, ESLintModule?: any): ESLintLib {
-    // eslint-disable-next-line no-new
-    if (ESLintModule) new ESLintModule.ESLint(options);
-    return new ESLintLib(options);
-  }
-
-  version() {
-    if (this.ESLint) return this.ESLint.version;
-    return ESLintLib.version;
-  }
-}
-
-function getCacheDir(rootDir): string {
-  return path.join(rootDir, 'node_modules', '.cache');
-}

package/eslint.main.runtime.ts

@@ -1,112 +0,0 @@
-import { MainRuntime } from '@teambit/cli';
-// import { Linter as ESLinter, ESLint as ESLintLib } from 'eslint';
-import { ESLint as ESLintLib } from 'eslint';
-import { Linter, LinterContext, LinterMain } from '@teambit/linter';
-import { Logger, LoggerAspect, LoggerMain } from '@teambit/logger';
-import { EslintConfigMutator } from '@teambit/defender.eslint.config-mutator';
-import { WorkspaceConfigFilesMain } from '@teambit/workspace-config-files';
-import { ESLintAspect } from './eslint.aspect';
-import { ESLintLinter } from './eslint.linter';
-
-export type ESLintOptions = {
-  /**
-   * linter config for eslint.
-   */
-  // config: ESLinter.Config;
-  config: ESLintLib.Options;
-
-  /**
-   * specify to path to resolve eslint plugins from.
-   */
-  pluginPath?: string;
-
-  /**
-   * decide the formatter for the CLI output.
-   */
-  formatter?: string;
-
-  /**
-   * file types to lint.
-   */
-  extensions?: string[];
-
-  // TODO: improve type
-  /**
-   * typescript config for eslint.
-   * If you pass this, bit will auto generate a temp config file in `node_modules/.cache` and pass it to eslint.
-   * In case you have include/exclude props in the tsconfig, they will be changed to handle the fact that they are inside the node_modules/.cache folder.
-   * a `../../` will be added to the beginning of the path.
-   */
-  tsConfig?: Record<string, any>;
-};
-
-export type EslintConfigTransformContext = {
-  fix: boolean;
-};
-
-export type EslintConfigTransformer = (
-  config: EslintConfigMutator,
-  context: EslintConfigTransformContext
-) => EslintConfigMutator;
-
-export class ESLintMain {
-  constructor(private logger: Logger) {}
-
-  /**
-   * create a eslint linter instance.
-   * @param options eslint options.
-   * @param ESLintModule reference to an `eslint` module.
-   */
-  createLinter(
-    context: LinterContext,
-    options: ESLintOptions,
-    transformers: EslintConfigTransformer[] = [],
-    ESLintModule?: any
-  ): Linter {
-    const mergedOptions = getOptions(options, context);
-    const configMutator = new EslintConfigMutator(mergedOptions);
-    const transformerContext: EslintConfigTransformContext = { fix: !!context.fix };
-    const afterMutation = runTransformersWithContext(configMutator.clone(), transformers, transformerContext);
-
-    return new ESLintLinter(this.logger, afterMutation.raw, ESLintModule);
-  }
-
-  static runtime = MainRuntime;
-
-  static dependencies = [LoggerAspect];
-
-  static async provider([loggerExt]: [LoggerMain, WorkspaceConfigFilesMain, LinterMain]): Promise<ESLintMain> {
-    const logger = loggerExt.createLogger(ESLintAspect.id);
-    return new ESLintMain(logger);
-  }
-}
-
-ESLintAspect.addRuntime(ESLintMain);
-
-/**
- * get options for eslint.
- */
-function getOptions(options: ESLintOptions, context: LinterContext): ESLintOptions {
-  const mergedConfig: ESLintLib.Options = {
-    // @ts-ignore - this is a bug in the @types/eslint types
-    overrideConfig: options.config,
-    extensions: context.extensionFormats,
-    useEslintrc: false,
-    cwd: options.pluginPath,
-    fix: !!context.fix,
-    fixTypes: context.fixTypes,
-  };
-  return Object.assign({}, options, { config: mergedConfig, extensions: context.extensionFormats });
-}
-
-export function runTransformersWithContext(
-  config: EslintConfigMutator,
-  transformers: EslintConfigTransformer[] = [],
-  context: EslintConfigTransformContext
-): EslintConfigMutator {
-  if (!Array.isArray(transformers)) return config;
-  const newConfig = transformers.reduce((acc, transformer) => {
-    return transformer(acc, context);
-  }, config);
-  return newConfig;
-}

package/index.ts

@@ -1,10 +0,0 @@
-import { ESLintAspect } from './eslint.aspect';
-
-export { ESLintAspect };
-export type {
-  ESLintMain,
-  ESLintOptions,
-  EslintConfigTransformContext,
-  EslintConfigTransformer,
-} from './eslint.main.runtime';
-export default ESLintAspect;