@aspruyt/xfg 2.1.0 → 2.1.2

package/dist/repository-processor.js

@@ -102,6 +102,9 @@ export class RepositoryProcessor {
             // Edge cases (repos with unusual git attributes on config files) are essentially nonexistent.
             const changedFiles = [];
             const diffStats = createDiffStats();
+            // Track pre-write actions for non-dry-run mode (issue #252)
+            // We need to know if a file was created vs updated BEFORE writing it
+            const preWriteActions = new Map();
             for (const file of repoConfig.files) {
                 const filePath = join(workDir, file.fileName);
                 const fileExistsLocal = existsSync(filePath);
@@ -148,7 +151,8 @@ export class RepositoryProcessor {
                     this.log.fileDiff(file.fileName, status, diffLines);
                 }
                 else {
-                    // Write the file
+                    // Write the file and store pre-write action for stats calculation
+                    preWriteActions.set(file.fileName, action);
                     this.gitOps.writeFile(file.fileName, fileContent);
                 }
             }
@@ -241,10 +245,9 @@ export class RepositoryProcessor {
                         if (!gitChangedFiles.has(file.fileName)) {
                             continue; // File didn't actually change
                         }
-                        const filePath = join(workDir, file.fileName);
-                        const action = existsSync(filePath)
-                            ? "update"
-                            : "create";
+                        // Use pre-write action (issue #252) - we stored whether file existed
+                        // BEFORE writing, which is the correct basis for create vs update
+                        const action = preWriteActions.get(file.fileName) ?? "update";
                         changedFiles.push({ fileName: file.fileName, action });
                     }
                     // Add any other files from git status that aren't already tracked
@@ -260,6 +263,21 @@ export class RepositoryProcessor {
                             : "create";
                         changedFiles.push({ fileName: gitFile, action });
                     }
+                    // Calculate diff stats from changedFiles (issue #252)
+                    for (const file of changedFiles) {
+                        switch (file.action) {
+                            case "create":
+                                incrementDiffStats(diffStats, "NEW");
+                                break;
+                            case "update":
+                                incrementDiffStats(diffStats, "MODIFIED");
+                                break;
+                            case "delete":
+                                incrementDiffStats(diffStats, "DELETED");
+                                break;
+                            // "skip" files are not counted in stats
+                        }
+                    }
                 }
             }
             if (!hasChanges) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aspruyt/xfg",
-  "version": "2.1.0",
+  "version": "2.1.2",
   "description": "CLI tool to sync JSON, JSON5, YAML, or text configuration files across multiple Git repositories via pull requests or direct push",
   "type": "module",
   "main": "dist/index.js",
@@ -58,7 +58,7 @@
     "yaml": "^2.4.5"
   },
   "devDependencies": {
-    "@types/node": "^25.0.7",
+    "@types/node": "^24.0.0",
     "c8": "^10.1.3",
     "tsx": "^4.15.0",
     "typescript": "^5.4.5"