yini-cli 1.3.0-beta → 1.3.1-beta

package/dist/commands/parseCommand.js

@@ -3,8 +3,23 @@ import fs from 'node:fs';
 import path from 'node:path';
 import YINI from 'yini-parser';
 import { getSerializer } from '../serializers/index.js';
-import { debugPrint, printObject, } from '../utils/print.js';
+import { debugPrint, printObject } from '../utils/print.js';
 // -------------------------------------------------------------------------
+const reportAction = (action, file, reason) => {
+    let txt = '';
+    if (reason) {
+        txt = `${action.padEnd(6)} "${file}" (${reason})`;
+    }
+    else {
+        txt = `${action.padEnd(6)} "${file}"`;
+    }
+    if (action === 'skip') {
+        console.warn(txt);
+    }
+    else {
+        console.log(txt);
+    }
+};
 /*
     TODO / SHOULD-DO:
 
@@ -91,24 +106,46 @@ const doParseFile = (file, commandOptions, outputFormat, outputFile = '') => {
     debugPrint('outputFormat = ' + outputFormat);
     const parseOptions = {
         strictMode: commandOptions.strict ?? false,
-        failLevel: preferredFailLevel,
-        includeMetadata: includeMetaData,
+        failLevel: commandOptions.bestEffort ? 'ignore-errors' : 'auto',
+        includeMetadata: false,
     };
     // If --best-effort then override fail-level.
-    if (commandOptions.bestEffort) {
-        parseOptions.failLevel = 'ignore-errors';
-    }
+    // if (commandOptions.bestEffort) {
+    //     parseOptions.failLevel = 'ignore-errors'
+    // }
     try {
         const parsed = YINI.parseFile(file, parseOptions);
         const serializer = getSerializer(outputFormat);
         const output = serializer.serialize(parsed);
         if (outputFile) {
             const resolved = path.resolve(outputFile);
-            enforceWritePolicy(file, resolved, commandOptions.overwrite);
+            const canWrite = enforceWritePolicy(file, resolved, commandOptions.overwrite);
+            if (!canWrite) {
+                if (commandOptions.verbose) {
+                    console.log(`skip    Skipping write to "${resolved}"`);
+                }
+                return;
+            }
+            // Double check, if the file was actually changed by comparing the contents.
+            if (fs.existsSync(resolved) && fs.statSync(resolved).isFile()) {
+                const existing = fs.readFileSync(resolved, 'utf-8');
+                // Only write the output file if the content actually changed.
+                // Prevents constantly showing meaningless rewrites, in some cases.
+                if (existing === output) {
+                    if (commandOptions.verbose) {
+                        // console.log(
+                        //     `skip    Output unchanged. Skipping write: "${resolved}"`,
+                        // )
+                        reportAction('skip', resolved, 'output unchanged');
+                    }
+                    return;
+                }
+            }
             // Write JSON output to file instead of stdout.
             fs.writeFileSync(resolved, output, 'utf-8');
             if (commandOptions.verbose) {
-                console.log(`Output written to file: "${outputFile}"`);
+                // console.log(`write    Output written to file: "${outputFile}"`)
+                reportAction('write', resolved);
             }
         }
         else {
@@ -123,19 +160,27 @@ const doParseFile = (file, commandOptions, outputFormat, outputFile = '') => {
 };
 const enforceWritePolicy = (srcPath, destPath, overwrite) => {
     if (!fs.existsSync(destPath)) {
-        return; // File does not exist, OK to write.
+        return true; // File does not exist, OK to write.
     }
     const srcStat = fs.statSync(srcPath);
     const destStat = fs.statSync(destPath);
-    const destIsNewer = destStat.mtimeMs >= srcStat.mtimeMs;
+    // Only strictly newer triggers skip overwrite.
+    const destIsNewer = destStat.mtimeMs > srcStat.mtimeMs;
     if (overwrite === true) {
-        return; // Explicit overwrite, OK.
+        return true; // Explicit overwrite, OK.
     }
     if (overwrite === false) {
         throw new Error(`File "${destPath}" already exists. Overwriting disabled (--no-overwrite).`);
     }
     // Default policy (overwrite undefined).
     if (destIsNewer) {
-        throw new Error(`Destination file "${destPath}" is newer than source. Use --overwrite to force.`);
+        // console.warn(
+        //     // `Destination file "${destPath}" is newer than source. Use --overwrite to force.`,
+        //     //`Warning: destination file "${destPath}" is newer than source. Skipping write. Use --overwrite to force.`,
+        //     `Warning: destination "${destPath}" is newer than source "${srcPath}". Skipping write. Use --overwrite to force.`,
+        // )
+        reportAction('skip', destPath, `newer than source "${srcPath}"`);
+        return false;
     }
+    return true;
 };

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "yini-cli",
-    "version": "1.3.0-beta",
+    "version": "1.3.1-beta",
     "description": "CLI tool for validating and converting YINI configuration files - an INI-like format with real structure, nested sections and strict or lenient modes.",
     "keywords": [
         "yini",