Sorting: fix encodeLargeNumbers to respect comma values (#171)

Author: thim81

CHANGELOG.md

@@ -1,5 +1,9 @@
 ## unreleased
 
+## [1.27.2] - 2025-07-04
+
+- Sorting: fix encodeLargeNumbers to respect comma values (#170)
+
 ## [1.27.1] - 2025-07-04
 
 - Sorting: ignore casing when sorting component properties (#165)

test/util-file.test.js

@@ -395,6 +395,12 @@ describe('openapi-format CLI file tests', () => {
       const output = encodeLargeNumbers(input);
       expect(output).toBe(' "large: "123456789.123456789===",\n key": "123",\n "other": "456"');
     });
+
+    test('should not modify numbers that are part of a comma separated string', () => {
+      const input = 'location: 10.12345678912345, 10.12345678912345';
+      const output = encodeLargeNumbers(input);
+      expect(output).toBe(input);
+    });
   });
 
   describe('addQuotesToRefInString function', () => {

test/yaml-big-numbers/input.yaml

@@ -0,0 +1,39 @@
+openapi: 3.0.0
+info:
+  title: TEST API
+  version: 1.0.0
+paths:
+  /test:
+    post:
+      requestBody:
+        content:
+          application/json:
+            schema:
+              type: object
+              properties:
+                location: string
+                test_property: string
+            examples:
+              Example1:
+                $ref: '#/components/examples/Test1'
+              Example2:
+                $ref: '#/components/examples/Test2'
+      responses:
+        '201':
+          description: Succcess
+          content:
+            application/json:
+              schema:
+                type: object
+                properties:
+                  type: string
+components:
+  examples:
+    Test1:
+      value:
+        location: 10.12345678912345, 10.12345678912345
+        test_property: test
+    Test2:
+      value:
+        location: 10.12345678912345, 10.12345678912345
+        test_property: test

test/yaml-big-numbers/options.yaml

@@ -0,0 +1,3 @@
+verbose: true
+output: output.yaml
+no-sort: false

test/yaml-big-numbers/output.yaml

@@ -0,0 +1,39 @@
+openapi: 3.0.0
+info:
+  title: TEST API
+  version: 1.0.0
+paths:
+  /test:
+    post:
+      requestBody:
+        content:
+          application/json:
+            schema:
+              type: object
+              properties:
+                location: string
+                test_property: string
+            examples:
+              Example1:
+                $ref: '#/components/examples/Test1'
+              Example2:
+                $ref: '#/components/examples/Test2'
+      responses:
+        '201':
+          description: Succcess
+          content:
+            application/json:
+              schema:
+                type: object
+                properties:
+                  type: string
+components:
+  examples:
+    Test1:
+      value:
+        location: '10.12345678912345, 10.12345678912345'
+        test_property: test
+    Test2:
+      value:
+        location: '10.12345678912345, 10.12345678912345'
+        test_property: test

utils/file.js

@@ -279,7 +279,7 @@ async function getRemoteFile(filePath) {
  */
 function encodeLargeNumbers(inputContent) {
   // Convert large number value safely before parsing
-  const regexEncodeLargeNumber = /: ([0-9]+(\.[0-9]+)?)\b(?!\.[0-9])(,|\n)/g; // match > : 123456789.123456789
+  const regexEncodeLargeNumber = /: ([0-9]+(\.[0-9]+)?)(?!\.[0-9])(,(?!\s*[0-9])|\n)/g; // match > : 123456789.123456789
   return inputContent.replace(regexEncodeLargeNumber, rawInput => {
     const endChar = rawInput.endsWith(',') ? ',' : '\n';
     const rgx = new RegExp(endChar, 'g');