refactor: introduce void 0 pkgs (#706)

Author: Mister-Hope

.github/renovate.json

@@ -1,10 +1,6 @@
 {
   "$schema": "https://docs.renovatebot.com/renovate-schema.json",
-  "extends": [
-    "config:recommended",
-    ":prHourlyLimitNone",
-    ":prConcurrentLimitNone"
-  ],
+  "extends": ["config:recommended", ":prHourlyLimitNone", ":prConcurrentLimitNone"],
   "dependencyDashboard": true,
   "schedule": ["every weekend"],
   "packageRules": [

.github/workflows/test.yml

@@ -37,11 +37,13 @@ jobs:
         run: pnpm install --frozen-lockfile
 
       - name: Build project and upload bundle
+        if: ${{ !cancelled() && matrix.node-version == 24 }}
         env:
           CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}
         run: pnpm run build
 
       - name: Linter test
+        if: ${{ !cancelled() && matrix.node-version == 24 }}
         run: pnpm lint
 
       - name: Unit test and save results

.husky/commit-msg

@@ -1,4 +1 @@
-#!/bin/sh
-. "$(dirname "$0")/_/husky.sh"
-
-pnpm commitlint --edit $1
+pnpm commitlint --edit $1

.husky/pre-commit

@@ -1,4 +1 @@
-#!/bin/sh
-. "$(dirname "$0")/_/husky.sh"
-
 pnpm nano-staged

.oxfmtrc.json

@@ -0,0 +1,4 @@
+{
+  "$schema": "./node_modules/oxfmt/configuration_schema.json",
+  "ignorePatterns": []
+}

.oxlintrc.json

@@ -0,0 +1,120 @@
+{
+  "$schema": "./node_modules/oxlint/configuration_schema.json",
+  "ignorePatterns": ["dist/**"],
+  "categories": {
+    "correctness": "error",
+    "suspicious": "warn",
+    "pedantic": "warn",
+    "perf": "warn",
+    "style": "warn"
+  },
+  "plugins": ["import", "unicorn", "oxc", "promise", "jsdoc", "vitest", "node"],
+  "rules": {
+    // we should not enforce capitalized comments
+    "capitalized-comments": "off",
+    // prefer to export declarations instead of exporting at the end
+    "group-exports": "off",
+    // this rule is too restrictive
+    "init-declarations": "off",
+    // allow continue in complicated loops
+    "no-continue": "off",
+    // warn about console usage but allow it
+    "no-console": "warn",
+    // type imports can be duplicated with value imports
+    "no-duplicate-imports": ["error", { "allowSeparateTypeImports": true }],
+    // inline comments shall be allowed
+    "no-inline-comments": "off",
+    "no-magic-numbers": "off",
+    // allow nested ternary for better readability in some cases
+    "no-nested-ternary": "off",
+    // allow null usage
+    "no-null": "off",
+    // ternary operator shall be allowed for simple expressions
+    "no-ternary": "off",
+    // normally we do not enforce numeric separator usage
+    "numeric-separators-style": "off",
+    // we prefer to sort imports with its pkg name
+    "sort-imports": "off",
+    // object keys are not supposed to be sorted in many cases
+    "sort-keys": "off",
+
+    // import plugin rules
+
+    "import/exports-last": "off",
+    // usually we just export vue components as default export without naming it
+    "import/no-anonymous-default-export": "off",
+    // a rule that shall not be enabled anyway
+    "import/no-named-export": "off",
+    // allow importing stylesheets
+    "import/no-unassigned-import": ["error", { "allow": ["**/*.css", "**/*.scss"] }],
+    // named export is preferred
+    "import/prefer-default-export": "off",
+
+    // jsdoc plugin rules
+
+    // duplicated typescript related jsdoc shall be warned
+    "jsdoc/check-tag-names": ["warn", { "typed": true }],
+    // we do not require types in jsdoc since we use TypeScript
+    "jsdoc/require-param-type": "off",
+    // we do not require types in jsdoc since we use TypeScript
+    "jsdoc/require-returns-type": "off",
+
+    "promise/always-return": "off",
+    // we sometimes need to create new Promise instance
+    "promise/avoid-new": "off",
+    // we prefer async/await syntax
+    "promise/prefer-await-to-then": "off",
+
+    // unicorn plugin rules
+    // filename should not be enforced strictly, e.g. Vue SFC files shall be PascalCased
+    // we shall expect a project level setting
+    "unicorn/filename-case": "off",
+    // toReversed can introduce memory overhead,
+    // in some chained calls it is better to use reverse as the "reversed" array is also a temp variable.
+    "unicorn/no-array-reverse": "off",
+    // toSorted can introduce memory overhead
+    // in some chained calls it is better to use reverse as the "sorted" array is also a temp variable.
+    "unicorn/no-array-sort": "off",
+    // sometimes nested ternary is considerable
+    "unicorn/no-nested-ternary": "warn",
+    // Array.at() is too modern for browser targets
+    "unicorn/prefer-at": "off",
+    // support starts with chrome71, firefox68. safari12.1, node12
+    // leaving it to warn for edge cases
+    "unicorn/prefer-global-this": "warn",
+
+    // stylistic rules with default being tweaked
+
+    "catch-error-name": ["warn", { "name": "err" }],
+    "curly": ["warn", "multi-line"],
+    "max-dependencies": ["warn", { "ignoreTypeImports": true, "max": 15 }],
+    "max-lines": ["warn", { "max": 500, "skipBlankLines": true, "skipComments": true }],
+    "max-lines-per-function": [
+      "warn",
+      { "max": 100, "skipBlankLines": true, "skipComments": true }
+    ],
+    "max-params": ["warn", 4],
+    "max-statements": ["warn", { "max": 30 }],
+
+    // per project settings
+    "id-length": "off"
+  },
+  "overrides": [
+    {
+      "files": ["**/*.ts"],
+      "plugins": ["typescript"],
+      "rules": {
+        // we make type casts only when necessary
+        // "typescript/no-unsafe-type-assertion": "off",
+        // skipping it for now, may enable it later
+        // "typescript/switch-exhaustiveness-check": "off",
+        // a lot of time we are just want to check existence
+        // "typescript/strict-boolean-expressions": "off"
+      }
+    },
+    {
+      "files": ["**/__tests__/**"],
+      "plugins": ["vitest"]
+    }
+  ]
+}

.vscode/extensions.json

@@ -0,0 +1,3 @@
+{
+  "recommendations": ["oxc.oxc-vscode", "vitest.explorer"]
+}

.vscode/settings.json

@@ -0,0 +1,4 @@
+{
+  "editor.defaultFormatter": "oxc.oxc-vscode",
+  "cSpell.words": ["crypted"]
+}

README.md

@@ -170,10 +170,7 @@ export const compare: (
  * @param salt Salt length to generate or salt to use, default to 10
  * @returns Resulting hash
  */
-export const hashSync: (
-  contentString: string,
-  salt?: string | number,
-) => string;
+export const hashSync: (contentString: string, salt?: string | number) => string;
 /**
  * Asynchronously generates a hash for the given string.
  *

__tests__/base64.spec.ts __tests__/base64.test.ts__tests__/base64.spec.ts renamed to __tests__/base64.test.ts

@@ -2,13 +2,13 @@ import { describe, expect, it } from "vitest";
 
 import { decodeBase64, encodeBase64 } from "../src/base64.js";
 
-describe("encodeBase64", () => {
+describe(encodeBase64, () => {
   it("should encode byte array to base64 string", () => {
     expect(
       encodeBase64(
         [
-          0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08, 0x09, 0x0a,
-          0x0b, 0x0c, 0x0d, 0x0e, 0x0f, 0x10,
+          0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08, 0x09, 0x0a, 0x0b, 0x0c, 0x0d, 0x0e,
+          0x0f, 0x10,
         ],
         16,
       ),
@@ -22,11 +22,11 @@ describe("encodeBase64", () => {
   });
 });
 
-describe("decodeBase64", () => {
+describe(decodeBase64, () => {
   it("should decode base64 string to byte array", () => {
     expect(decodeBase64("..CA.uOD/eaGAOmJB.yMBv.", 16)).toEqual([
-      0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08, 0x09, 0x0a, 0x0b,
-      0x0c, 0x0d, 0x0e, 0x0f,
+      0x00, 0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08, 0x09, 0x0a, 0x0b, 0x0c, 0x0d, 0x0e,
+      0x0f,
     ]);
   });