@adbayb/stack 2.26.0 → 2.28.0

package/configs/eslint/constants.js

@@ -1,6 +1,4 @@
-import { cwd } from "node:process";
-
-export const CWD = cwd();
+export const CWD = process.cwd();
 
 export const JAVASCRIPT_EXTENSIONS = ["**/*.{js,jsx,cjs,mjs}"];
 

package/configs/eslint/index.js

@@ -7,6 +7,7 @@ import { config as sonarConfig } from "./presets/sonar.js";
 import { config as reactConfig } from "./presets/react.js";
 import { config as overridableConfig } from "./presets/overridable.js";
 import { config as nodeConfig } from "./presets/node.js";
+import { config as markdownConfig } from "./presets/markdown.js";
 import { config as jsdocConfig } from "./presets/jsdoc.js";
 import { config as importConfig } from "./presets/import.js";
 import { config as eslintConfig } from "./presets/eslint.js";
@@ -20,6 +21,7 @@ export default createConfig(
 	...typescriptConfig,
 	...importConfig,
 	...jsdocConfig,
+	...markdownConfig,
 	...nodeConfig,
 	...reactConfig,
 	...sonarConfig,

package/configs/eslint/presets/import.js

@@ -19,7 +19,6 @@ export const config = [
 			"import-x/no-anonymous-default-export": "error",
 			"import-x/no-cycle": "error",
 			"import-x/no-default-export": "error",
-			"import-x/no-deprecated": "error",
 			"import-x/no-duplicates": "error",
 			"import-x/no-empty-named-blocks": "error",
 			"import-x/no-extraneous-dependencies": "error",

package/configs/eslint/presets/markdown.js

@@ -0,0 +1,3 @@
+import { flat } from "eslint-plugin-mdx";
+
+export const config = [flat];

package/configs/eslint/presets/node.js

@@ -14,6 +14,7 @@ export const config = [
 		rules: {
 			"n/callback-return": "error",
 			"n/exports-style": ["error", "module.exports"],
+			"n/hashbang": "error",
 			"n/no-exports-assign": "error",
 			"n/no-path-concat": "error",
 			"n/no-process-env": [
@@ -29,9 +30,9 @@ export const config = [
 				"error",
 				{ allowExperimental: true },
 			],
-			"n/prefer-global/buffer": ["error", "never"],
+			"n/prefer-global/buffer": ["error", "always"],
 			"n/prefer-global/console": ["error", "always"],
-			"n/prefer-global/process": ["error", "never"],
+			"n/prefer-global/process": ["error", "always"],
 			"n/prefer-global/text-decoder": ["error", "always"],
 			"n/prefer-global/text-encoder": ["error", "always"],
 			"n/prefer-global/url": ["error", "always"],

package/configs/eslint/presets/react.js

@@ -22,6 +22,7 @@ export const config = [
 			"react/dom/no-missing-iframe-sandbox": "error",
 			"react/dom/no-namespace": "error",
 			"react/dom/no-render": "error",
+			"react/dom/no-render-return-value": "error",
 			"react/dom/no-script-url": "error",
 			"react/dom/no-unsafe-iframe-sandbox": "error",
 			"react/dom/no-unsafe-target-blank": "error",
@@ -60,6 +61,7 @@ export const config = [
 			"react/no-unstable-context-value": "error",
 			"react/no-unstable-default-props": "error",
 			"react/no-use-context": "error",
+			"react/no-useless-forward-ref": "error",
 			"react/no-useless-fragment": "error",
 			"react/prefer-shorthand-boolean": "error",
 			"react/prefer-shorthand-fragment": "error",

package/configs/eslint/presets/uncategorized.js

@@ -1,6 +1,5 @@
 import sortKeysCustomOrderPlugin from "eslint-plugin-sort-keys-custom-order";
 import dependPlugin from "eslint-plugin-depend";
-import { FlatCompat } from "@eslint/eslintrc";
 
 import { JAVASCRIPT_LIKE_EXTENSIONS } from "../constants.js";
 
@@ -41,11 +40,4 @@ export const config = [
 			],
 		},
 	},
-	...new FlatCompat().extends("plugin:mdx/recommended").map((mdxConfig) => ({
-		...mdxConfig,
-		files: ["**/*.{md,mdx}"],
-		settings: {
-			"mdx/code-blocks": true,
-		},
-	})),
 ];

package/configs/eslint/presets/unicorn.js

@@ -1,9 +1,9 @@
-/* eslint-disable unicorn/string-content */
 import unicornPlugin from "eslint-plugin-unicorn";
 
 import { JAVASCRIPT_LIKE_EXTENSIONS } from "../constants.js";
 
 export const config = [
+	// TODO: Add new rules https://github.com/sindresorhus/eslint-plugin-unicorn/releases/tag/v60.0.0
 	{
 		files: JAVASCRIPT_LIKE_EXTENSIONS,
 		plugins: {
@@ -37,6 +37,7 @@ export const config = [
 			"unicorn/new-for-builtins": "error",
 			"unicorn/no-abusive-eslint-disable": "error",
 			"unicorn/no-array-callback-reference": "error",
+			"unicorn/no-array-reverse": "error",
 			"unicorn/no-await-in-promise-methods": "error",
 			"unicorn/no-console-spaces": "error",
 			"unicorn/no-document-cookie": "error",
@@ -58,6 +59,7 @@ export const config = [
 			"unicorn/no-unreadable-array-destructuring": "error",
 			"unicorn/no-unreadable-iife": "error",
 			"unicorn/no-unused-properties": "error",
+			"unicorn/no-useless-error-capture-stack-trace": "error",
 			"unicorn/no-useless-fallback-in-spread": "error",
 			"unicorn/no-useless-length-check": "error",
 			"unicorn/no-useless-promise-resolve-reject": "error",
@@ -71,6 +73,7 @@ export const config = [
 			"unicorn/prefer-array-index-of": "error",
 			"unicorn/prefer-array-some": "error",
 			"unicorn/prefer-blob-reading-methods": "error",
+			"unicorn/prefer-class-fields": "error",
 			"unicorn/prefer-code-point": "error",
 			"unicorn/prefer-date-now": "error",
 			"unicorn/prefer-dom-node-append": "error",
@@ -117,25 +120,12 @@ export const config = [
 			"unicorn/prevent-abbreviations": [
 				"error",
 				{
-					allowList: {
-						props: true,
-						Props: true,
-					},
+					ignore: [/^props/i, /^ref/i],
 				},
 			],
 			"unicorn/relative-url-style": ["error", "always"],
 			"unicorn/require-array-join-separator": "error",
 			"unicorn/require-number-to-fixed-digits-argument": "error",
-			"unicorn/string-content": [
-				"error",
-				{
-					patterns: {
-						"->": "→",
-						"\\.\\.\\.": "…",
-						"^http:\\/\\/": String.raw`^https:\/\/`,
-					},
-				},
-			],
 			"unicorn/switch-case-braces": "error",
 			"unicorn/text-encoding-identifier-case": "error",
 			"unicorn/throw-new-error": "error",

package/dist/index.js

@@ -1,5 +1,4 @@
 import { helpers, termost } from 'termost';
-import process from 'node:process';
 import { resolve, join } from 'node:path';
 import { createRequire } from 'node:module';
 import { writeFile, chmod, mkdir, symlink, rm } from 'node:fs/promises';
@@ -196,7 +195,7 @@ const ESLINT_EXTENSIONS = [
     "mdx"
 ];
 
-var version = "2.26.0";
+var version = "2.28.0";
 
 const createWatchCommand = (program)=>{
     program.command({
@@ -285,7 +284,7 @@ const createInstallCommand = (program)=>{
     }).task({
         label: label$4("Install `git.commit-msg` hook"),
         async handler () {
-            await installGitHook("commit-msg", getStackCommand("check --only commit", false));
+            await installGitHook("commit-msg", getStackCommand("check --filter commit", false));
         }
     }).task({
         label: label$4("Check the package manager"),
@@ -589,7 +588,7 @@ const PRESERVE_FILES = [
     "node_modules"
 ];
 
-const checkTypes = async ()=>{
+const checkType = async ()=>{
     try {
         return await helpers.exec("pnpm --parallel exec tsc --noEmit");
     } catch (error) {
@@ -597,9 +596,9 @@ const checkTypes = async ()=>{
     }
 };
 
-const checkPackages = async ()=>{
+const checkDependency = async ()=>{
     const stdout = await helpers.exec("pnpm recursive ls --json");
-    const checkPackagesVersionMismatch = createPackagesVersionMismatchChecker();
+    const checkDependencyVersionMismatch = createPackagesVersionMismatchChecker();
     const packages = JSON.parse(stdout).map((package_)=>{
         const packagePath = join(package_.path, "package.json");
         assert(package_.name, ()=>createPackageError(`\`${packagePath}\` must have a name field.`));
@@ -616,19 +615,19 @@ const checkPackages = async ()=>{
     });
     for (const package_ of packages){
         // Check version mismatches to guarantee a single copy for a given package in the monorepo (use case: prevent singleton-like issues with React contexts)
-        checkPackagesVersionMismatch(package_);
+        checkDependencyVersionMismatch(package_);
         // Check version range accordingly to our dependency guidelines (ie. dev dependencies must be pinned and dependencies must have caret)
-        checkPackagesVersionRange(package_);
+        checkDependencyVersionRange(package_);
     }
 };
-const checkPackagesVersionRange = ({ name, dependencies, devDependencies, peerDependencies })=>{
+const checkDependencyVersionRange = ({ name, dependencies, devDependencies, peerDependencies })=>{
     for (const dependencyName of Object.keys(devDependencies)){
         const version = devDependencies[dependencyName];
         assertVersion(version, {
             name: dependencyName,
             consumedBy: name
         });
-        if (version !== "workspace:*" && !/^\d/.test(version)) throw createPackageError(`As a dev dependency, \`${dependencyName}\` version must be fixed (or set as "workspace:*" for local packages) to reduce accidental breaking change risks due to an implicit semver upgrade.`, {
+        if (version !== "workspace:*" && !isExcluded(version) && !/^\d/.test(version)) throw createPackageError(`As a dev dependency, \`${dependencyName}\` version must be fixed (or set as "workspace:*" for local packages) to reduce accidental breaking change risks due to an implicit semver upgrade.`, {
             name: dependencyName,
             consumedBy: name
         });
@@ -639,7 +638,7 @@ const checkPackagesVersionRange = ({ name, dependencies, devDependencies, peerDe
             name: dependencyName,
             consumedBy: name
         });
-        if (version !== "workspace:^" && hasNoCaret(version)) throw createPackageError(`As a dependency, \`${dependencyName}\` version must be prefixed with a caret (or set as "workspace:^" for local packages) to optimize the size (whether of installation or bundle output) on the consumer side.`, {
+        if (version !== "workspace:^" && !hasCaret(version) && !isExcluded(version)) throw createPackageError(`As a dependency, \`${dependencyName}\` version must be prefixed with a caret (or set as "workspace:^" for local packages) to optimize the size (whether of installation or bundle output) on the consumer side.`, {
             name: dependencyName,
             consumedBy: name
         });
@@ -650,7 +649,7 @@ const checkPackagesVersionRange = ({ name, dependencies, devDependencies, peerDe
             name: dependencyName,
             consumedBy: name
         });
-        if (hasNoCaret(version)) /*
+        if (!hasCaret(version) && !isExcluded(version)) /*
 			 * Why disallowing workspace protocol as a version resolver?
 			 * To reduce the update frequency needs consumer-side and guarantee on our side the minimum compatible version,
 			 * the best practice should be to keeping an explicit number version which represents the lowest compatible version from an API perspective.
@@ -699,12 +698,12 @@ function assertVersion(version, { name, consumedBy }) {
             consumedBy
         }));
 }
-const isPreReleaseVersion = (version)=>/\d+\.\d+\.\d+-(alpha|beta|experimental|next|rc).*/.exec(version);
-const hasNoCaret = (version)=>!isPreReleaseVersion(version) && !version.startsWith("^");
-
-const checkLinter = eslint({
-    isFixMode: false
-});
+const isExcluded = (version)=>{
+    const isPreReleaseVersion = /\d+\.\d+\.\d+-(alpha|beta|experimental|next|rc).*/.exec(version);
+    const isNpmProtocol = version.startsWith("npm:");
+    return isNpmProtocol || isPreReleaseVersion;
+};
+const hasCaret = (version)=>version.startsWith("^");
 
 const checkCommit = async ()=>{
     try {
@@ -714,26 +713,30 @@ const checkCommit = async ()=>{
     }
 };
 
+const checkCode = eslint({
+    isFixMode: false
+});
+
 const ONLY_VALUES = [
     "commit",
-    "linter",
-    "packages",
-    "types"
+    "code",
+    "dependency",
+    "type"
 ];
 const createCheckCommand = (program)=>{
     program.command({
         name: "check",
         description: "Check code health (static analysis)"
     }).option({
-        key: "only",
-        name: "only",
-        description: `Run only one specified task (accepted value: ${ONLY_VALUES.join(", ")})`,
+        key: "filter",
+        name: "filter",
+        description: `Filter the compliance check to run (accepted value: ${ONLY_VALUES.join(", ")})`,
         defaultValue: undefined
     }).task({
         handler (_, argv) {
             logCheckableFiles(argv.operands);
         },
-        skip: ifOnlyDefinedAndNotEqualTo("linter")
+        skip: ifFilterDefinedAndNotEqualTo("code")
     }).task({
         label: label("Prepare the project"),
         async handler () {
@@ -742,46 +745,46 @@ const createCheckCommand = (program)=>{
                 hasLiveOutput: false
             });
         },
-        skip ({ only }) {
-            return only === "commit"; // No need to build if only commit is checked
+        skip ({ filter }) {
+            return filter === "commit"; // No need to build if only commit is checked
         }
     }).task({
-        label: label("Check package guidelines"),
+        label: label("Check dependency compliance"),
         async handler () {
-            await checkPackages();
+            await checkDependency();
         },
-        skip: ifOnlyDefinedAndNotEqualTo("packages")
+        skip: ifFilterDefinedAndNotEqualTo("dependency")
     }).task({
-        label: label("Check linter rules"),
+        label: label("Check code compliance"),
         async handler (_, argv) {
             const filenames = argv.operands;
-            await checkLinter(filenames);
+            await checkCode(filenames);
         },
-        skip: ifOnlyDefinedAndNotEqualTo("linter")
+        skip: ifFilterDefinedAndNotEqualTo("code")
     }).task({
-        label: label("Check types"),
+        label: label("Check type compliance"),
         async handler () {
-            await checkTypes();
+            await checkType();
         },
         skip (context, argv) {
-            return ifOnlyDefinedAndNotEqualTo("types")(context) || !hasDependency("typescript") || /**
+            return ifFilterDefinedAndNotEqualTo("type")(context) || !hasDependency("typescript") || /**
 					 * For now, skip type-checking if some files are passed down.
 					 * @see https://github.com/microsoft/TypeScript/issues/27379
 					 */ argv.operands.length > 0;
         }
     }).task({
-        label: label("Check commit"),
+        label: label("Check commit compliance"),
         async handler () {
             await checkCommit();
         },
-        skip ({ only }) {
-            return only !== "commit";
+        skip (context) {
+            return context.filter !== "commit";
         }
     });
 };
 const label = (message)=>`${message} 🧐`;
-const ifOnlyDefinedAndNotEqualTo = (only)=>(context)=>{
-        return context.only !== undefined && context.only !== only;
+const ifFilterDefinedAndNotEqualTo = (filter)=>(context)=>{
+        return context.filter !== undefined && context.filter !== filter;
     };
 
 const createBuildCommand = (program)=>{

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@adbayb/stack",
-  "version": "2.26.0",
+  "version": "2.28.0",
   "description": "My opinionated JavaScript-based toolchain",
   "keywords": [
     "stack",
@@ -40,39 +40,38 @@
   ],
   "dependencies": {
     "@changesets/changelog-github": "^0.5.1",
-    "@changesets/cli": "^2.29.4",
+    "@changesets/cli": "^2.29.5",
     "@commitlint/cli": "^19.8.1",
     "@commitlint/config-conventional": "^19.8.1",
-    "@eslint-react/eslint-plugin": "^1.52.2",
-    "@eslint/compat": "^1.3.0",
-    "@eslint/eslintrc": "^3.3.1",
-    "@stylistic/eslint-plugin": "^4.4.1",
-    "@vitest/eslint-plugin": "^1.2.7",
-    "eslint": "^9.29.0",
-    "eslint-config-prettier": "^10.1.5",
-    "eslint-import-resolver-typescript": "^4.4.3",
+    "@eslint-react/eslint-plugin": "^1.52.3",
+    "@eslint/compat": "^1.3.1",
+    "@stylistic/eslint-plugin": "^5.2.2",
+    "@vitest/eslint-plugin": "^1.3.3",
+    "eslint": "^9.32.0",
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-import-resolver-typescript": "^4.4.4",
     "eslint-plugin-depend": "^1.2.0",
-    "eslint-plugin-import-x": "^4.15.2",
+    "eslint-plugin-import-x": "^4.16.1",
     "eslint-plugin-jest-formatting": "^3.1.0",
-    "eslint-plugin-jsdoc": "^51.0.3",
-    "eslint-plugin-mdx": "^3.5.0",
-    "eslint-plugin-n": "^17.20.0",
-    "eslint-plugin-prettier": "^5.5.0",
+    "eslint-plugin-jsdoc": "^51.4.1",
+    "eslint-plugin-mdx": "^3.6.2",
+    "eslint-plugin-n": "^17.21.0",
+    "eslint-plugin-prettier": "^5.5.3",
     "eslint-plugin-react-hooks": "^5.2.0",
-    "eslint-plugin-sonarjs": "^3.0.3",
+    "eslint-plugin-sonarjs": "^3.0.4",
     "eslint-plugin-sort-keys-custom-order": "^2.2.1",
-    "eslint-plugin-unicorn": "^59.0.1",
+    "eslint-plugin-unicorn": "^60.0.0",
     "fdir": "^6.4.6",
-    "globals": "^16.2.0",
-    "prettier": "^3.5.3",
-    "prettier-plugin-packagejson": "^2.5.15",
-    "termost": "^1.4.0",
-    "turbo": "^2.5.4",
+    "globals": "^16.3.0",
+    "prettier": "^3.6.2",
+    "prettier-plugin-packagejson": "^2.5.19",
+    "termost": "^1.8.0",
+    "turbo": "^2.5.5",
     "typescript": "^5.8.3",
-    "typescript-eslint": "^8.34.1"
+    "typescript-eslint": "^8.38.0"
   },
   "devDependencies": {
-    "@types/node": "22.15.32",
+    "@types/node": "22.16.5",
     "quickbundle": "2.13.0"
   },
   "publishConfig": {