declapract-typescript-ehmpathy 0.47.37 → 0.47.38

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

@@ -2,7 +2,10 @@ import type { FileCheckFunction, FileFixFunction } from 'declapract';
 import { dedupe } from 'domain-objects';
 import expect from 'expect';
 
-const expectedIgnores = [
+/**
+ * ignores that can be sorted alphabetically (no order dependencies)
+ */
+const ignoresSortable = [
   '*.log',
   '*.tsbuildinfo',
   '.artifact', // deployment artifacts from `simple-artifact-builder` are produced here
@@ -17,15 +20,38 @@ const expectedIgnores = [
   '*.bak.*', // backup files
   'coverage',
   'dist',
-  'node_modules',
 ].sort();
 
+/**
+ * ignores that must appear in a specific order (negations must follow their targets)
+ *
+ * note
+ * - node_modules under .test* dirs are test fixtures (cloned repos via genTempDir)
+ * - gitignore negation requires the negated pattern to come after the original
+ */
+const ignoresOrdered = [
+  'node_modules',
+  '!.test*/**/node_modules',
+  '!.test*/**/node_modules/**',
+];
+
 const defineExpectedContents = (contents: string | null): string => {
-  const ignoresAlreadyDefined = contents ? contents.split('\n') : [];
-  const finalLines = dedupe([...ignoresAlreadyDefined, ...expectedIgnores])
-    .sort() // sorted
-    .filter((line) => !!line); // without empty lines
-  return [...finalLines.sort(), ''].join('\n');
+  // parse ignores from file
+  const ignoresFromFile = contents ? contents.split('\n') : [];
+
+  // separate into sortable vs ordered
+  const orderedPatterns = new Set(ignoresOrdered);
+  const ignoresFromFileSortable = ignoresFromFile.filter(
+    (line) => !!line && !orderedPatterns.has(line),
+  );
+
+  // combine and sort the sortable ignores
+  const sortedIgnores = dedupe([...ignoresFromFileSortable, ...ignoresSortable])
+    .sort()
+    .filter((line) => !!line);
+
+  // append ordered ignores at the end (order preserved)
+  return [...sortedIgnores, ...ignoresOrdered, ''].join('\n');
 };
 
 export const check: FileCheckFunction = (contents) => {

package/package.json

@@ -2,7 +2,7 @@
   "name": "declapract-typescript-ehmpathy",
   "author": "ehmpathy",
   "description": "declapract best practices declarations for typescript",
-  "version": "0.47.37",
+  "version": "0.47.38",
   "license": "MIT",
   "main": "src/index.js",
   "repository": "ehmpathy/declapract-typescript-ehmpathy",