feat: add biome (#1281)

* feat: biome

* chore: test biome

* chore: update code

* chore: update code

* chore: update changeset

* chore: update biome rules

* chore: enable biome lint

* chore: remove eslint and prettier

* chore: remove eslint and prettier

* chore: test lint-staged

* chore: test lint-staged

* chore: test lint-staged

* chore: test lint-staged

* chore: test lint-staged

* chore: test lint-staged

* chore: test lint-staged

* chore: update test lint-staged

* chore: update test lint-staged

* chore: update test lint-staged

* chore: update test lint-staged

* chore: update husky

* chore: update husky

* chore: update husky

* chore: update husky

* chore: update husky

* chore: update husky

.changeset/sweet-stingrays-crash.md

@@ -0,0 +1,2 @@
+---
+---

.github/workflows/lint.yaml

@@ -22,7 +22,7 @@ jobs:
  run: cargo fmt
  - name: Run cargo clippy
  run: cargo clippy
- eslint:
+ lint:
  name: TS Code Lint
  runs-on: ubuntu-latest
  steps:
@@ -32,5 +32,5 @@ jobs:
  node-version: 18
  - name: Install Dependencies
  run: npm install -g [email protected] && pnpm i --frozen-lockfile
- - name: Run eslint
- run: npx eslint packages
+ - name: Run lint
+ run: npx biome check --no-errors-on-unmatched --files-ignore-unknown=true

.husky/pre-commit

@@ -0,0 +1 @@
+npx lint-staged

.pnpmfile.cjs

@@ -4,15 +4,17 @@ function readPackage(pkg, context) {
  // Replace [email protected] with [email protected]
  pkg.peerDependencies = {
  vue: pkg.version
- }
- context.log(`${pkg.name}@${pkg.version} => vue@${pkg.version} in peerDependencies`)
+ };
+ context.log(
+ `${pkg.name}@${pkg.version} => vue@${pkg.version} in peerDependencies`
+ );
  }
- 
- return pkg
+
+ return pkg;
 }
 
 module.exports = {
  hooks: {
  readPackage
  }
-}
+};

biome.json

@@ -0,0 +1,138 @@
+{
+ "$schema": "https://biomejs.dev/schemas/1.7.2/schema.json",
+ "vcs": {
+ "clientKind": "git",
+ "enabled": true,
+ "useIgnoreFile": true,
+ "defaultBranch": "main"
+ },
+ "files": {
+ "ignore": [
+ "crates/**",
+ "*.vue",
+ "examples/**",
+ "tsconfig.base.json",
+ "tsconfig.json",
+ "cspell.json",
+ "e2e/**",
+ "*.e2e.ts",
+ "*.d.ts",
+ "packages/runtime/**",
+ "packages/runtime-plugin-import-meta/**",
+ "packages/create-farm/templates/**",
+ "packages/create-farm-plugin/templates/**",
+ "rust-plugins/**",
+ "js-plugins/vue",
+ "js-plugins/less",
+ "js-plugins/sass"
+ ],
+ "ignoreUnknown": true
+ },
+ "formatter": {
+ "enabled": true,
+ "formatWithErrors": false,
+ "indentStyle": "space",
+ "indentWidth": 2,
+ "lineEnding": "lf",
+ "lineWidth": 80,
+ "attributePosition": "auto"
+ },
+ "organizeImports": {
+ "enabled": true,
+ "include": ["./**/*.js", "./**/*.ts", "./**/*.mjs"]
+ },
+ "linter": {
+ "enabled": true,
+ "rules": {
+ "recommended": false,
+ "complexity": {
+ "noBannedTypes": "error",
+ "noExtraBooleanCast": "error",
+ "noMultipleSpacesInRegularExpressionLiterals": "error",
+ "noUselessCatch": "error",
+ "noUselessTypeConstraint": "error",
+ "noWith": "error"
+ },
+ "correctness": {
+ "noConstAssign": "error",
+ "noConstantCondition": "error",
+ "noEmptyCharacterClassInRegex": "error",
+ "noEmptyPattern": "error",
+ "noGlobalObjectCalls": "error",
+ "noInnerDeclarations": "error",
+ "noInvalidConstructorSuper": "error",
+ "noNewSymbol": "error",
+ "noNonoctalDecimalEscape": "error",
+ "noPrecisionLoss": "error",
+ "noSelfAssign": "error",
+ "noSetterReturn": "error",
+ "noSwitchDeclarations": "error",
+ "noUndeclaredVariables": "warn",
+ "noUnreachable": "error",
+ "noUnreachableSuper": "error",
+ "noUnsafeFinally": "error",
+ "noUnsafeOptionalChaining": "error",
+ "noUnusedLabels": "error",
+ "noUnusedVariables": "warn",
+ "useIsNan": "error",
+ "useValidForDirection": "error",
+ "useYield": "error"
+ },
+ "style": {
+ "noInferrableTypes": "error",
+ "noNamespace": "error",
+ "noNonNullAssertion": "warn",
+ "useAsConstAssertion": "error"
+ },
+ "suspicious": {
+ "noAsyncPromiseExecutor": "error",
+ "noCatchAssign": "error",
+ "noClassAssign": "error",
+ "noCompareNegZero": "error",
+ "noControlCharactersInRegex": "off",
+ "noDebugger": "error",
+ "noDuplicateCase": "error",
+ "noDuplicateClassMembers": "error",
+ "noDuplicateObjectKeys": "error",
+ "noDuplicateParameters": "error",
+ "noEmptyBlockStatements": "off",
+ "noExplicitAny": "off",
+ "noExtraNonNullAssertion": "error",
+ "noFallthroughSwitchClause": "error",
+ "noFunctionAssign": "error",
+ "noGlobalAssign": "error",
+ "noImportAssign": "error",
+ "noMisleadingCharacterClass": "error",
+ "noMisleadingInstantiator": "error",
+ "noPrototypeBuiltins": "warn",
+ "noRedeclare": "error",
+ "noShadowRestrictedNames": "error",
+ "noUnsafeNegation": "error",
+ "useGetterReturn": "error",
+ "useNamespaceKeyword": "error",
+ "useValidTypeof": "error"
+ }
+ },
+ "ignore": [
+ "**/binding/**",
+ "**/templates/**",
+ "**/dist/**",
+ "**/node_modules/**",
+ "**/*.config.js",
+ "**/*.config.ts"
+ ]
+ },
+ "javascript": {
+ "formatter": {
+ "jsxQuoteStyle": "single",
+ "quoteProperties": "asNeeded",
+ "trailingComma": "none",
+ "semicolons": "always",
+ "arrowParentheses": "always",
+ "bracketSpacing": true,
+ "bracketSameLine": false,
+ "quoteStyle": "single",
+ "attributePosition": "auto"
+ }
+ }
+}

commitlint.config.js

@@ -1,3 +1,3 @@
 module.exports = {
  extends: ['@commitlint/config-conventional']
-}
+};

cspell.json

@@ -65,10 +65,11 @@
  "icns",
  "idents",
  "IHDR",
- "JIDA",
  "importee",
  "indicatif",
+ "Instantiator",
  "jfif",
+ "JIDA",
  "jridgewell",
  "jsnext",
  "kolorist",
@@ -92,6 +93,7 @@
  "nanospinner",
  "napi",
  "NAPI",
+ "Nonoctal",
  "normpath",
  "npmlog",
  "onwarn",

js-plugins/dts/farm.config.mjs

@@ -1,30 +1,30 @@
-import { builtinModules } from "module";
+import { builtinModules } from 'module';
 
 /**
  * @type {import('@farmfe/core').UserConfig}
  */
 export default {
  compilation: {
  input: {
- index: "./src/index.ts",
+ index: './src/index.ts'
  },
  output: {
- path: "build",
- entryFilename: "[entryName].cjs",
- targetEnv: "node",
- format: "cjs",
+ path: 'build',
+ entryFilename: '[entryName].cjs',
+ targetEnv: 'node',
+ format: 'cjs'
  },
  partialBundling: {
  enforceResources: [
  {
- name: "index.js",
- test: [".+"],
- },
- ],
+ name: 'index.js',
+ test: ['.+']
+ }
+ ]
  },
  minify: false,
  sourcemap: false,
  persistentCache: false,
- presetEnv: false,
+ presetEnv: false
  }
 };

js-plugins/dts/package.json

@@ -35,8 +35,5 @@
  "@farmfe/core": "workspace:*",
  "@types/fs-extra": "^11.0.1"
  },
- "files": [
- "dist",
- "build"
- ]
-}
+ "files": ["dist", "build"]
+}

js-plugins/dts/src/context.ts

@@ -1,10 +1,10 @@
+import os from 'node:os';
+import { relative, resolve } from 'node:path';
 import type { UserConfig } from '@farmfe/core';
-import { DefaultLogger } from './logger.js';
 import chalk from 'chalk';
 import glob from 'fast-glob';
-import os from 'node:os';
-import { relative, resolve } from 'node:path';
 import { CompilerOptions, Project, SourceFile } from 'ts-morph';
+import { DefaultLogger } from './logger.js';
 import {
  ensureAbsolute,
  ensureArray,
@@ -40,7 +40,7 @@ export default class Context {
  noEmitOnError: false,
  skipDiagnostics: true,
  copyDtsFiles: false,
- afterDiagnostic: () => {}
+ afterDiagnostic: () => ({})
  };
 
  const userOptions = mergeObjects(defaultOption, options);
@@ -58,7 +58,7 @@ export default class Context {
  : this.config.output?.path;
  const aliasesExclude = userOptions?.aliasesExclude ?? [];
  const tsConfigPath = resolveAbsolutePath(userOptions.tsconfigPath, root);
- libFolderPath = libFolderPath && ensureAbsolute(libFolderPath, root);
+ const folderPath = libFolderPath && ensureAbsolute(libFolderPath, root);
  const compilerOptions = userOptions.compilerOptions ?? {};
 
  const mergeCompilerOptions = {
@@ -75,7 +75,7 @@ export default class Context {
  } as CompilerOptions),
  tsConfigFilePath: tsConfigPath,
  skipAddingFilesFromTsConfig: true,
- libFolderPath
+ libFolderPath: folderPath
  };
  this.project = new Project(mergeCompilerOptions);
  const tsConfigOptions = getTsConfig(
@@ -108,8 +108,8 @@ export default class Context {
  (isRegExp(find)
  ? find.toString() === alias.toString()
  : isRegExp(alias)
- ? find.match(alias)?.[0]
- : find === alias)
+  ? find.match(alias)?.[0]
+  : find === alias)
  )
  );
  }

js-plugins/dts/src/index.ts

@@ -1,8 +1,8 @@
 import type { JsPlugin } from '@farmfe/core';
 
 import Context from './context.js';
-import { tryToReadFileSync } from './utils.js';
 import { pluginName } from './options.js';
+import { tryToReadFileSync } from './utils.js';
 
 import type { DtsPluginOptions } from './types.js';
 

js-plugins/dts/src/types.ts

@@ -1,4 +1,4 @@
-import type { ts, Diagnostic } from 'ts-morph';
+import type { Diagnostic, ts } from 'ts-morph';
 
 export interface DtsPluginOptions {
  /**