json-sort-cli 3.1.0 → 3.1.1

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [3.1.1](https://github.com/codsen/codsen/compare/json-sort-cli@3.1.0...json-sort-cli@3.1.1) (2023-04-18)
+
+### Bug Fixes
+
+- fix `-c` flag to correctly recognise different EOLs as per `-l` ([5af3ba1](https://github.com/codsen/codsen/commit/5af3ba16962c2dd001005780a3843ecf675389f4)), closes [#73](https://github.com/codsen/codsen/issues/73)
+
 # [3.1.0](https://github.com/codsen/codsen/compare/json-sort-cli@3.0.12...json-sort-cli@3.1.0) (2023-04-16)
 
 ### Features

package/cli.js

@@ -171,6 +171,8 @@ function readSortAndWriteOverFile(oneOfPaths) {
   return fs
     .readFile(oneOfPaths, "utf8")
     .then((filesContent) => {
+      let eolChar = resolveEolSetting(filesContent, cli.flags.lineEnding);
+
       let parsedJson;
       try {
         // try to parse JSON
@@ -200,7 +202,7 @@ function readSortAndWriteOverFile(oneOfPaths) {
             spaces: cli.flags.tabs
               ? "\t".repeat(indentationCount)
               : indentationCount,
-            EOL: resolveEolSetting(filesContent, cli.flags.lineEnding),
+            EOL: eolChar,
           }
         );
       } else {
@@ -219,6 +221,8 @@ function readSortAndWriteOverFile(oneOfPaths) {
           // we read it, now we return true if result differs after processing
 
           let stringified = JSON.stringify(
+            // The traversal below will mutate, not just traverse, -
+            // whatever you return from the callback below, gets written
             traverse(obj, (key, val) => {
               let current = val !== undefined ? val : key;
               if (isPlainObject(current)) {
@@ -230,7 +234,7 @@ function readSortAndWriteOverFile(oneOfPaths) {
                 current.length > 1 &&
                 current.every(isStr)
               ) {
-                // alphabetical sort
+                // alphabetical sort, this value gets written
                 return current.sort((a, b) => a.localeCompare(b));
               }
               return current;
@@ -238,7 +242,25 @@ function readSortAndWriteOverFile(oneOfPaths) {
             null,
             cli.flags.tabs ? "\t".repeat(indentationCount) : indentationCount
           );
-          return stringified.trimEnd() !== filesContent.trimEnd();
+
+          if (eolChar === "\r\n") {
+            // CRLF
+            stringified = stringified
+              .replaceAll(/(?<!\r)\n/g, "\r\n")
+              .replaceAll(/\r(?!\n)/g, "\n");
+          } else {
+            // LF or CR
+            stringified = stringified.replaceAll(/(?:\r?\n)|\r/g, eolChar);
+          }
+
+          return (
+            stringified
+              .replaceAll(
+                /\r\n/g,
+                resolveEolSetting(filesContent, cli.flags.lineEnding)
+              )
+              .trimEnd() !== filesContent.trimEnd()
+          );
         }
 
         // ELSE,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "json-sort-cli",
-  "version": "3.1.0",
+  "version": "3.1.1",
   "description": "Command line app to deep sort JSON files, retains package.json special key order",
   "keywords": [
     "app",