@peerigon/configs 4.3.0-beta.1 → 4.3.0

package/CHANGELOG.md

@@ -1,3 +1,24 @@
+# [4.3.0](https://github.com/peerigon/configs/compare/v4.2.0...v4.3.0) (2025-03-24)
+
+### Bug Fixes
+
+- **eslint:** Turn of unicorn/consistent-function-scoping ([6570db3](https://github.com/peerigon/configs/commit/6570db32a4037f0e00d99cfacc95a024f9892b95))
+
+### Features
+
+- Compile JS code instead of just emitDeclarationOnly ([c7d29f7](https://github.com/peerigon/configs/commit/c7d29f700ee53aa6be5b16175b355f000fba778c))
+- **typescript:** Change base target to es2024 ([58b217a](https://github.com/peerigon/configs/commit/58b217ab0e4d64b7b969f1dbda748886f11009c5))
+
+# [4.3.0-beta.2](https://github.com/peerigon/configs/compare/v4.3.0-beta.1...v4.3.0-beta.2) (2025-03-23)
+
+### Bug Fixes
+
+- **eslint:** Turn of unicorn/consistent-function-scoping ([6570db3](https://github.com/peerigon/configs/commit/6570db32a4037f0e00d99cfacc95a024f9892b95))
+
+### Features
+
+- **typescript:** Change base target to es2024 ([58b217a](https://github.com/peerigon/configs/commit/58b217ab0e4d64b7b969f1dbda748886f11009c5))
+
 # [4.3.0-beta.1](https://github.com/peerigon/configs/compare/v4.2.0...v4.3.0-beta.1) (2025-03-12)
 
 ### Features

package/dist/eslint/lib/rule-options.js

@@ -4,17 +4,17 @@ const camelcase = {
     properties: "always",
 };
 export const ruleOptions = {
-    ["camelcase"]: camelcase,
-    ["max-lines"]: {
+    camelcase,
+    "max-lines": {
         max: 700,
         skipBlankLines: true,
         skipComments: true,
     },
-    ["no-unused-expressions"]: {
+    "no-unused-expressions": {
         allowShortCircuit: true,
         allowTernary: true,
     },
-    ["no-unused-vars"]: {
+    "no-unused-vars": {
         vars: "all",
         varsIgnorePattern: "^_",
         args: "after-used",
@@ -24,7 +24,7 @@ export const ruleOptions = {
         // This pattern is pretty common
         ignoreRestSiblings: true,
     },
-    ["@typescript-eslint/naming-convention"]: (() => {
+    "@typescript-eslint/naming-convention": (() => {
         const options = {
             default: {
                 selector: "default",

package/dist/eslint/rules/javascript.js

@@ -12,22 +12,20 @@ export const javascript = [
             // https://github.com/sindresorhus/eslint-plugin-unicorn/issues/896
             "unicorn/prevent-abbreviations": "off",
             "unicorn/filename-case": "off",
-            // The default of import-style is highly subjective and not always the best choice.
-            // E.g. bundlers are able to tree-shake named imports easier than default imports.
-            "unicorn/import-style": "off",
             "unicorn/no-nested-ternary": "off",
             "unicorn/no-null": "off",
             "unicorn/no-useless-undefined": "off",
             "unicorn/prefer-query-selector": "off",
             // Also turn off the following rules because they're too opinionated
-            "unicorn/require-array-join-separator": "off",
+            "unicorn/consistent-function-scoping": "off", // This rule forces to move arrow functions up the scope where it is often more readable to keep the function in the scope where it is used called.
+            "unicorn/import-style": "off", // The default of import-style is highly subjective and not always the best choice. E.g. bundlers are able to tree-shake named imports easier than default imports.
             "unicorn/no-array-for-each": "off",
             "unicorn/no-array-reduce": "off",
+            "unicorn/no-object-as-default-parameter": "off", // This rule also complains about the object when we already use destructuring. E.g. it would complain about the following pattern which is perfectly fine: { github = false, jsr = false } = { github: true, jsr: true }
+            "unicorn/no-single-promise-in-promise-methods": "off", // It makes sense to use Promise.all() with a single promise when we expect more to be added later. In that case we don't want to refactor the code, but just add the new promise.
             "unicorn/prefer-global-this": "off", // Too many false positives
             "unicorn/prefer-ternary": "off",
-            // It makes sense to use Promise.all() with a single promise when we expect more to be added later.
-            // In that case we don't want to refactor the code, but just add the new promise.
-            "unicorn/no-single-promise-in-promise-methods": "off",
+            "unicorn/require-array-join-separator": "off",
         },
     },
     {

package/dist/eslint/rules/typescript.js

@@ -20,7 +20,7 @@ export const typescript =
         globPatterns.typescriptJsx,
     ],
     plugins: {
-        ["prefer-arrow"]: preferArrow,
+        "prefer-arrow": preferArrow,
     },
     rules: {
         "@typescript-eslint/array-type": ["warn", { default: "generic" }],

package/dist/prettier/base.d.ts

@@ -1,5 +1,6 @@
 /**
- * Base config for Prettier.
+ * Provides the default configuration for Prettier with customized plugins and
+ * sorting rules.
  *
  * @type {import("prettier").Config}
  */

package/dist/prettier/base.js

@@ -1,33 +1,8 @@
 /**
- * ## Installation
+ * ## Base config for Prettier.
  *
- * ```sh
- * npm install prettier @peerigon/configs --save-dev
- * ```
- *
- * Then create a `prettier.config.js` next to your `package.json`:
- *
- * ```js
- * export { default } from "@peerigon/configs/prettier";
- * ```
- *
- * Recommended configuration in your `package.json` (using
- * [`npm-run-all2`](https://www.npmjs.com/package/npm-run-all2)):
- *
- * ```json
- * {
- *   "type": "module",
- *   "scripts": {
- *     "test": "run-p test:*",
- *     "test:format": "prettier --check ."
- *   }
- * }
- * ```
- *
- * ## Configuration
- *
- * Our config is entirely based on Prettier's default config. Besides that, it
- * also:
+ * Our base config is entirely based on Prettier's default config. Besides that,
+ * it also:
  *
  * - Auto-sorts `import` statements
  * - Formats JSDoc comments
@@ -35,17 +10,17 @@
  * - Formats and sorts CSS properties
  * - Sorts Tailwind CSS class names
  *
- * @module prettier
+ * @module prettierConfig
  */
 /**
  * @param {string} id
  * @returns {string}
  */
-const safeResolve = (id) => {
+function safeResolve(id) {
     return "resolve" in import.meta
         ? import.meta.resolve(id).slice("file://".length)
         : id;
-};
+}
 // Using safeResolve() here because the plugins might not be installed in the parent app/module
 // and we don't want to rely on the package manager to hoist the dependencies.
 const plugins = await Promise.all([
@@ -56,7 +31,8 @@ const plugins = await Promise.all([
     safeResolve("prettier-plugin-tailwindcss"),
 ]);
 /**
- * Base config for Prettier.
+ * Provides the default configuration for Prettier with customized plugins and
+ * sorting rules.
  *
  * @type {import("prettier").Config}
  */
@@ -72,4 +48,8 @@ export const config = {
     cssDeclarationSorterOrder: "smacss",
     cssDeclarationSorterKeepOverrides: false,
 };
+/**
+ * @type {import("prettier").Config}
+ * @see {config}
+ */
 export default config;

package/dist/semantic-release/cross-publish.d.ts

@@ -1,3 +1,7 @@
+/**
+ * @param {{ github: boolean; jsr: boolean }} options?
+ * @returns {import("semantic-release").Options}
+ */
 export function config({ github, jsr }?: {
     github: boolean;
     jsr: boolean;

package/dist/semantic-release/cross-publish.js

@@ -3,7 +3,7 @@ import { config as baseConfig } from "./base.js";
  * @param {{ github: boolean; jsr: boolean }} options?
  * @returns {import("semantic-release").Options}
  */
-export const config = ({ github = false, jsr = false } = { github: true, jsr: true }) => {
+export function config({ github = false, jsr = false } = { github: true, jsr: true }) {
     /** @type {import("semantic-release").PluginSpec[]} */
     const plugins = [];
     if (baseConfig.plugins) {
@@ -13,8 +13,12 @@ export const config = ({ github = false, jsr = false } = { github: true, jsr: tr
         plugins.push([
             "@semantic-release/exec",
             {
-                verifyConditionsCmd: 'echo "registry=https://npm.pkg.github.com/\n//npm.pkg.github.com/:_authToken=${process.env.GITHUB_TOKEN}" > /tmp/github.npmrc && npm whoami --userconfig /tmp/github.npmrc',
-                publishCmd: "npm publish --userconfig /tmp/github.npmrc --tag ${nextRelease.channel} --no-git-tag-version",
+                verifyConditionsCmd: 
+                // eslint-disable-next-line no-template-curly-in-string
+                'echo "registry=https://npm.pkg.github.com/\n//npm.pkg.github.com/:_authToken=${process.env.GITHUB_TOKEN}" > /tmp/github.npmrc && npm whoami --userconfig /tmp/github.npmrc',
+                publishCmd: 
+                // eslint-disable-next-line no-template-curly-in-string
+                "npm publish --userconfig /tmp/github.npmrc --tag ${nextRelease.channel} --no-git-tag-version",
                 successCmd: "rm /tmp/github.npmrc",
                 failCmd: "rm /tmp/github.npmrc",
             },
@@ -27,5 +31,5 @@ export const config = ({ github = false, jsr = false } = { github: true, jsr: tr
         ...baseConfig,
         plugins,
     };
-};
+}
 export default config();

package/dist/typescript/base.json

@@ -4,10 +4,10 @@
     "compilerOptions": {
         // == Target and module settings ==
         // Deliberately not using ESNext/NodeNext here to avoid breaking changes just by updating TypeScript.
-        "target": "es2022",
+        "target": "es2024",
         "module": "Preserve",
         "moduleDetection": "force",
-        "lib": ["es2022"],
+        "lib": ["es2024"],
         // == Strictness settings ==
         "strict": true,
         "noImplicitOverride": true,
@@ -31,8 +31,10 @@
         // Using noEmit true here because you should have a separate build config anyway
         "noEmit": true,
         "noPropertyAccessFromIndexSignature": true,
-        // Not setting skipLibCheck: true for now because it can hide type conflicts in dependencies
+        // Not setting skipLibCheck: true for now because it can hide type conflicts in dependencies.
         // You can override this in your app if you can't fix it with your package manager.
+        // If you want to learn more about the trade-offs,
+        // see https://www.testim.io/blog/typescript-skiplibcheck/
         "skipLibCheck": false,
         "verbatimModuleSyntax": true
     },

package/dist/typescript/js-lib.json

@@ -1,6 +1,8 @@
 {
     "extends": "./base.json",
     "compilerOptions": {
+        "target": "es2022",
+        "lib": ["es2022"],
         "declaration": true,
         "noEmit": false,
         "outDir": "${configDir}/dist"

package/dist/typescript/lib.json

@@ -1,6 +1,8 @@
 {
     "extends": "./base.json",
     "compilerOptions": {
+        "target": "es2022",
+        "lib": ["es2022"],
         "declaration": true,
         "declarationMap": true,
         "noEmit": false,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@peerigon/configs",
-  "version": "4.3.0-beta.1",
+  "version": "4.3.0",
   "description": "Configs for ESLint, Prettier, TypeScript & friends",
   "keywords": [
     "eslint",
@@ -54,8 +54,9 @@
     "test:styles:no-null": "cd eslint/styles/no-null.test; eslint --max-warnings 0 .",
     "test:styles:prefer-array-shorthand": "cd eslint/styles/prefer-array-shorthand.test; eslint --max-warnings 0 .",
     "test:styles:prefer-interface": "cd eslint/styles/prefer-interface.test; eslint --max-warnings 0 .",
+    "test:lint": "eslint --max-warnings 0 .",
     "test:types": "tsc --noEmit",
-    "test:prettier": "prettier --check .",
+    "test:format": "prettier --check .",
     "build": "run-s build:*",
     "build:clear": "rimraf dist",
     "build:tsc": "tsc",