Improve jsonl[...] suite of scriptlets

gorhill · gorhill · commit ed9999efd646 · 2025-05-30T11:06:33.000-04:00
Reuse original line separator when reassembling JSONL lines.
diff --git a/src/js/resources/json-edit.js b/src/js/resources/json-edit.js
@@ -374,7 +374,8 @@ registerScriptlet(trustedJsonEditFetchResponse, {
 
 function jsonlEditFn(jsonp, text = '') {
     const safe = safeSelf();
-    const linesBefore = text.split(/\n+/);
+    const lineSeparator = /\r?\n/.exec(text)?.[0] || '\n';
+    const linesBefore = text.split(lineSeparator);
     const linesAfter = [];
     for ( const lineBefore of linesBefore ) {
         let obj;
@@ -387,9 +388,10 @@ function jsonlEditFn(jsonp, text = '') {
             linesAfter.push(lineBefore);
             continue;
         }
-        linesAfter.push(JSONPath.toJSON(obj, safe.JSON_stringify));
+        const lineAfter = safe.JSON_stringify(obj);
+        linesAfter.push(lineAfter);
     }
-    return linesAfter.join('\n');
+    return linesAfter.join(lineSeparator);
 }
 registerScriptlet(jsonlEditFn, {
     name: 'jsonl-edit.fn',
