declapract-typescript-ehmpathy 0.43.7 → 0.43.9

package/dist/practices/cicd-service/best-practice/package.json

@@ -1,6 +1,6 @@
 {
   "devDependencies": {
-    "declastruct": "@declapract{check.minVersion('1.4.0')}",
+    "declastruct": "@declapract{check.minVersion('1.4.3')}",
     "declastruct-aws": "@declapract{check.minVersion('1.0.3')}",
     "declastruct-unix-network": "@declapract{check.minVersion('1.0.0')}"
   }

package/dist/practices/git/best-practice/.gitattributes

@@ -1,2 +1,3 @@
-# exclude package-lock from git diff; https://stackoverflow.com/a/72834452/3068233
+# exclude package locks from git diff; https://stackoverflow.com/a/72834452/3068233
+pnpm-lock.json -diff
 package-lock.json -diff

package/dist/practices/git/best-practice/.gitattributes.declapract.ts

@@ -1,6 +1,56 @@
-import { FileCheckType } from 'declapract';
+import { FileCheckType, type FileFixFunction } from 'declapract';
 
 /**
  * check that they're contained in the file
  */
 export const check: FileCheckType = FileCheckType.CONTAINS;
+
+/**
+ * the header comment for the package lock exclusions section
+ */
+const HEADER =
+  '# exclude package locks from git diff; https://stackoverflow.com/a/72834452/3068233';
+
+/**
+ * the entries that should appear under the header
+ */
+const ENTRIES = ['pnpm-lock.json -diff', 'package-lock.json -diff'];
+
+/**
+ * fix by ensuring all entries are present under the header
+ */
+export const fix: FileFixFunction = (contents) => {
+  // if no contents, create the file with the header and entries
+  if (!contents) {
+    return { contents: [HEADER, ...ENTRIES].join('\n') + '\n' };
+  }
+
+  const lines = contents.split('\n');
+
+  // find the header line index
+  const headerIndex = lines.findIndex((line) => line.trim() === HEADER);
+
+  // if header not found, append the whole section at the end
+  if (headerIndex === -1) {
+    const newContent =
+      contents.trimEnd() + '\n\n' + [HEADER, ...ENTRIES].join('\n') + '\n';
+    return { contents: newContent };
+  }
+
+  // find which entries are missing after the header
+  const missingEntries = ENTRIES.filter((entry) => !contents.includes(entry));
+
+  // if no missing entries, nothing to fix
+  if (missingEntries.length === 0) {
+    return { contents };
+  }
+
+  // insert missing entries right after the header
+  const newLines = [
+    ...lines.slice(0, headerIndex + 1),
+    ...missingEntries,
+    ...lines.slice(headerIndex + 1),
+  ];
+
+  return { contents: newLines.join('\n') };
+};

package/dist/practices/lint/best-practice/biome.json

@@ -12,7 +12,8 @@
       "!**/coverage",
       "!**/node_modules",
       "!**/.artifact",
-      "!**/.generated/**"
+      "!**/.generated/**",
+      "!**/.test*/**"
     ]
   },
   "formatter": {
@@ -27,7 +28,13 @@
       "trailingCommas": "all"
     }
   },
-  "assist": { "actions": { "source": { "organizeImports": "on" } } },
+  "assist": {
+    "actions": {
+      "source": {
+        "organizeImports": "on"
+      }
+    }
+  },
   "linter": {
     "enabled": true,
     "rules": {
@@ -36,13 +43,19 @@
         "noExcessiveCognitiveComplexity": "warn"
       },
       "correctness": {
-        "noUnusedImports": { "level": "error", "fix": "safe" },
+        "noUnusedImports": {
+          "level": "error",
+          "fix": "safe"
+        },
         "noUnusedVariables": "warn",
         "useExhaustiveDependencies": "off"
       },
       "style": {
         "noDefaultExport": "error",
-        "noNonNullAssertion": { "level": "warn", "fix": "none" },
+        "noNonNullAssertion": {
+          "level": "warn",
+          "fix": "none"
+        },
         "useImportType": "error",
         "noParameterAssign": "off"
       },

package/dist/practices/node/best-practice/package.json

@@ -6,6 +6,7 @@
   "homepage": "https://github.com/@declapract{variable.organizationName}/@declapract{variable.projectName}",
   "bugs": "https://github.com/@declapract{variable.organizationName}/@declapract{variable.projectName}/issues",
   "scripts": {
+    "fix": "npm run fix:format && npm run fix:lint",
     "prepush": "npm run test && npm run build",
     "preversion": "npm run prepush",
     "postversion": "git push origin HEAD --tags --no-verify"

package/dist/practices/typescript/best-practice/tsconfig.json

@@ -14,7 +14,9 @@
     "sourceMap": true,
     "outDir": "dist",
     "target": "es2020",
-    "incremental": true
+    "incremental": true,
+    "module": "commonjs", // enable usage of esm modules without a strict boundary; todo, cutover to be esm native
+    "moduleResolution": "node"
   },
   "include": [
     "**/*.ts"

package/package.json

@@ -2,7 +2,7 @@
   "name": "declapract-typescript-ehmpathy",
   "author": "ehmpathy",
   "description": "declapract best practices declarations for typescript",
-  "version": "0.43.7",
+  "version": "0.43.9",
   "license": "MIT",
   "main": "src/index.js",
   "repository": "ehmpathy/declapract-typescript-ehmpathy",
@@ -42,7 +42,7 @@
   },
   "dependencies": {
     "chalk": "4.1.2",
-    "domain-objects": "0.31.0",
+    "domain-objects": "0.31.3",
     "expect": "29.4.2",
     "flat": "5.0.2",
     "helpful-errors": "1.5.3",
@@ -61,15 +61,15 @@
     "@trivago/prettier-plugin-sort-imports": "4.3.0",
     "@tsconfig/node-lts-strictest": "18.12.1",
     "@types/flat": "5.0.2",
-    "@types/node": "22.15.21",
     "@types/jest": "29.4.0",
+    "@types/node": "22.15.21",
     "@typescript-eslint/eslint-plugin": "7.8.0",
     "@typescript-eslint/parser": "7.8.0",
     "babel-jest": "30.2.0",
     "core-js": "3.26.1",
     "cz-conventional-changelog": "3.3.0",
-    "declastruct": "1.3.0",
-    "declastruct-github": "1.0.3",
+    "declastruct": "1.4.3",
+    "declastruct-github": "1.0.5",
     "depcheck": "1.4.3",
     "eslint": "8.56.0",
     "eslint-config-airbnb-typescript": "18.0.0",