jtlt 0.24.0 → 0.24.1

package/src/JSONJoiningTransformer.js

@@ -488,10 +488,9 @@ class JSONJoiningTransformer extends AbstractJoiningTransformer {
     // If children provided as array, copy (may be primitives or nested JML)
     if (Array.isArray(childNodes) && childNodes.length) {
       jmlChildren.push(...childNodes.map((childNode) => {
-        if (typeof childNode === 'string') {
-          return this._replaceCharacterMaps(childNode);
-        }
-        return childNode;
+        return typeof childNode === 'string'
+          ? this._replaceCharacterMaps(childNode)
+          : childNode;
       }));
     }
 
@@ -537,7 +536,7 @@ class JSONJoiningTransformer extends AbstractJoiningTransformer {
             standalone
           };
         }
-        /* c8 ignore stop */
+        /* c8 ignore stop -- See above */
 
         const doc = {$document: {
           ...(xmlDeclaration ? {xmlDeclaration} : {}),
@@ -641,20 +640,22 @@ class JSONJoiningTransformer extends AbstractJoiningTransformer {
     }
 
     const pending = this._pendingNamespaceMap.get(prefix);
-    if (pending) {
-      const top = /** @type {ElementInfo} */ (this._elementStack.at(-1));
-      const {attsObj} = top;
-
-      if (!attsObj.xmlns) {
-        attsObj.xmlns = {};
-      }
+    if (!pending) {
+      return;
+    }
 
-      /** @type {Record<string, string>} */
-      (attsObj.xmlns)[pending.prefix === '#default' ? '' : pending.prefix] =
-        this._replaceCharacterMaps(pending.namespaceURI);
+    const top = /** @type {ElementInfo} */ (this._elementStack.at(-1));
+    const {attsObj} = top;
 
-      this._pendingNamespaceMap.delete(prefix);
+    if (!attsObj.xmlns) {
+      attsObj.xmlns = {};
     }
+
+    /** @type {Record<string, string>} */
+    (attsObj.xmlns)[pending.prefix === '#default' ? '' : pending.prefix] =
+      this._replaceCharacterMaps(pending.namespaceURI);
+
+    this._pendingNamespaceMap.delete(prefix);
   }
   /**
    * Adds/updates an attribute for the most recently open element built via
@@ -882,7 +883,7 @@ class JSONJoiningTransformer extends AbstractJoiningTransformer {
           standalone
         };
       }
-      /* c8 ignore stop */
+      /* c8 ignore stop -- See above */
 
       resultDoc = {$document: {
         ...(xmlDeclaration ? {xmlDeclaration} : {}),
@@ -921,10 +922,9 @@ class JSONJoiningTransformer extends AbstractJoiningTransformer {
    */
   _usePropertySets (obj, psName) {
     // Merge the named property set (if present) into the provided object
-    if (Object.hasOwn(this.propertySets, psName)) {
-      return Object.assign(obj, this.propertySets[psName]);
-    }
-    return obj;
+    return Object.hasOwn(this.propertySets, psName)
+      ? Object.assign(obj, this.propertySets[psName])
+      : obj;
   }
 }
 

package/src/JSONPathTransformer.js

@@ -34,10 +34,9 @@ class JSONPathTransformer {
        */
       template (value) {
         // Emit property names for the current object context
-        if (value && typeof value === 'object' && !Array.isArray(value)) {
-          return Object.keys(value).join('');
-        }
-        return '';
+        return value && typeof value === 'object' && !Array.isArray(value)
+          ? Object.keys(value).join('')
+          : '';
       }
     },
     transformObjects: {

package/src/JSONPathTransformerContext.js

@@ -319,10 +319,8 @@ class JSONPathTransformerContext {
     /* const results = */ this._getJoiningTransformer();
     const modeMatchedTemplates = this._templates.filter((templateObj) => {
       // Exclude named-only templates (those with name but no path)
-      if (templateObj.name && !templateObj.path) {
-        return false;
-      }
-      return ((mode && mode === templateObj.mode) ||
+      return (!templateObj.name || templateObj.path) &&
+        ((mode && mode === templateObj.mode) ||
         (!mode && !templateObj.mode));
     });
 
@@ -382,10 +380,7 @@ class JSONPathTransformerContext {
         if (Number.isNaN(an)) {
           return Number(order);
         }
-        if (Number.isNaN(bn)) {
-          return -order;
-        }
-        return (an - bn) * order;
+        return Number.isNaN(bn) ? -order : (an - bn) * order;
       }
       // text
       /* c8 ignore next 2 -- all null/undefined tested; OR short-circuit */
@@ -460,7 +455,7 @@ class JSONPathTransformerContext {
           if (!templateObj.path) {
             return false;
           }
-          /* c8 ignore stop */
+          /* c8 ignore stop -- See above */
           const queryResult = /** @type {any[]} */ (
             (/** @type {any} */ (jsonpath))({
               path: JSONPathTransformer.makeJSONPathAbsolute(
@@ -554,7 +549,7 @@ class JSONPathTransformerContext {
             : (that._config.specificityPriorityResolver && b.path
               ? that._config.specificityPriorityResolver(b.path)
               : 0);
-          /* c8 ignore stop */
+          /* c8 ignore stop -- See above */
 
           if (aPriority === bPriority) {
             that._triggerEqualPriorityError();
@@ -676,13 +671,13 @@ class JSONPathTransformerContext {
               // @ts-expect-error -- _openTagState: StringJoiningTransformer
               joiner._openTagState = false;
             }
-            /* c8 ignore stop */
+            /* c8 ignore stop -- See above */
             joiner.append(resolvedRet);
           }
           that._parent = parent;
           // Matches reached here always carry a parentProperty; the sync
           // path covers the root-node fallback.
-          /* c8 ignore next */
+          /* c8 ignore next -- See comment just above */
           that._parentProperty = (parentProperty ?? that._parentProperty);
           that._currPath = _oldPath;
           return undefined;
@@ -700,7 +695,7 @@ class JSONPathTransformerContext {
           // @ts-expect-error -- _openTagState only on StringJoiningTransformer
           joiner._openTagState = false;
         }
-        /* c8 ignore stop */
+        /* c8 ignore stop -- See above */
         joiner.append(
           /** @type {string|Node|*} */ (ret)
         );
@@ -910,10 +905,7 @@ class JSONPathTransformerContext {
         if (Number.isNaN(an)) {
           return Number(order);
         }
-        if (Number.isNaN(bn)) {
-          return -order;
-        }
-        return (an - bn) * order;
+        return Number.isNaN(bn) ? -order : (an - bn) * order;
       }
       /* c8 ignore next 2 -- all null/undefined tested; OR short-circuit */
       const aStr = aVal === null || aVal === undefined ? '' : String(aVal);
@@ -1257,23 +1249,25 @@ class JSONPathTransformerContext {
         currentGroup.push(m.value);
         const endMatch = evalInContext(groupEndingWith, m.value);
 
-        if (endMatch) {
-          const prevContext = this._contextObj;
-          const prevParams = this._params;
-          const prevVars = this.vars;
-          this.vars = {};
-          try {
-            this._contextObj = currentGroup;
-            /** @type {any} */ (this)._currentGroup = currentGroup;
-            cb.call(this, null, currentGroup, this);
-          } finally {
-            this._contextObj = prevContext;
-            this._params = prevParams;
-            this.vars = prevVars;
-            delete /** @type {any} */ (this)._currentGroup;
-          }
-          currentGroup = [];
+        if (!endMatch) {
+          continue;
         }
+
+        const prevContext = this._contextObj;
+        const prevParams = this._params;
+        const prevVars = this.vars;
+        this.vars = {};
+        try {
+          this._contextObj = currentGroup;
+          /** @type {any} */ (this)._currentGroup = currentGroup;
+          cb.call(this, null, currentGroup, this);
+        } finally {
+          this._contextObj = prevContext;
+          this._params = prevParams;
+          this.vars = prevVars;
+          delete /** @type {any} */ (this)._currentGroup;
+        }
+        currentGroup = [];
       }
 
       // Process last group if not ended
@@ -1341,10 +1335,7 @@ class JSONPathTransformerContext {
         if (Number.isNaN(an)) {
           return order;
         }
-        if (Number.isNaN(bn)) {
-          return -order;
-        }
-        return (an - bn) * order;
+        return Number.isNaN(bn) ? -order : (an - bn) * order;
       }
       /* c8 ignore next 2 -- all null/undefined tested; OR short-circuit */
       const aStr = aVal === null || aVal === undefined ? '' : String(aVal);
@@ -1642,12 +1633,11 @@ class JSONPathTransformerContext {
      * @returns {string} - Captured group or empty string
      */
     const getRegexGroup = (groupNumber) => {
-      if (!currentCapturedGroups ||
+      return !currentCapturedGroups ||
           groupNumber < 0 ||
-          groupNumber >= currentCapturedGroups.length) {
-        return '';
-      }
-      return currentCapturedGroups[groupNumber] || '';
+          groupNumber >= currentCapturedGroups.length
+        ? ''
+        : currentCapturedGroups[groupNumber] || '';
     };
 
     // Save previous context to restore later
@@ -1750,7 +1740,7 @@ class JSONPathTransformerContext {
         // For non-functions, attempting structuredClone again would rethrow;
         // use shallow clone to retain non-serializable props like Symbols.
         clone = Array.isArray(val) ? [...val] : {...val};
-        /* c8 ignore stop */
+        /* c8 ignore stop -- See above */
       }
     } else {
       // Primitives/functions copied by value/reference semantics naturally.
@@ -1782,7 +1772,7 @@ class JSONPathTransformerContext {
       }
     } else { /* c8 ignore start -- primitive branch attribution variance */
       clone = src; // Primitive/function - nothing to shallow clone
-    } /* c8 ignore stop */
+    } /* c8 ignore stop -- See above */
     this._getJoiningTransformer().append(clone);
     return this;
   }
@@ -1841,10 +1831,9 @@ class JSONPathTransformerContext {
     if (Object.hasOwn(this.vars, name)) {
       return {has: true, value: this.vars[name]};
     }
-    if (Object.hasOwn(this._runtimeParams, name)) {
-      return {has: true, value: this._runtimeParams[name]};
-    }
-    return {has: false, value: undefined};
+    return Object.hasOwn(this._runtimeParams, name)
+      ? {has: true, value: this._runtimeParams[name]}
+      : {has: false, value: undefined};
   }
 
   /**
@@ -2778,15 +2767,9 @@ class JSONPathTransformerContext {
       }
       // Single item; apply scalar truthiness
       const single = val[0];
-      if (single && typeof single === 'object') {
-        return true;
-      }
-      return Boolean(single);
+      return single && typeof single === 'object' ? true : Boolean(single);
     }
-    if (val && typeof val === 'object') {
-      return true;
-    }
-    return Boolean(val);
+    return val && typeof val === 'object' ? true : Boolean(val);
   }
 
   /**
@@ -2837,10 +2820,9 @@ class JSONPathTransformerContext {
       return null;
     }
     const right = this._parseLiteral(m.groups.right.trim());
-    if (!right) {
-      return null;
-    }
-    return {left: m.groups.left, op: m.groups.op, right: right.value};
+    return !right
+      ? null
+      : {left: m.groups.left, op: m.groups.op, right: right.value};
   }
 
   /**
@@ -2871,10 +2853,7 @@ class JSONPathTransformerContext {
     if (str === 'null') {
       return {value: null};
     }
-    if (str === 'undefined') {
-      return {value: undefined};
-    }
-    return null;
+    return str === 'undefined' ? {value: undefined} : null;
   }
 
   /**

package/src/StringJoiningTransformer.js

@@ -562,7 +562,7 @@ class StringJoiningTransformer extends AbstractJoiningTransformer {
           standalone ? ` standalone="yes"` : ''
         }?>\n`;
       }
-      /* c8 ignore stop */
+      /* c8 ignore stop -- See above */
 
       const doctype = bareDoctype ||
         doctypePublic !== undefined || doctypeSystem !== undefined
@@ -733,13 +733,15 @@ class StringJoiningTransformer extends AbstractJoiningTransformer {
    */
   _flushPendingNamespace (prefix) {
     const pending = this._pendingNamespaceMap.get(prefix);
-    if (pending) {
-      this.append(' ' + (
-        pending.prefix === '#default' ? 'xmlns' : 'xmlns:' + pending.prefix
-      ) + '="' +
-          this._replaceCharacterMaps(pending.namespaceURI) + '"');
-      this._pendingNamespaceMap.delete(prefix);
+    if (!pending) {
+      return;
     }
+
+    this.append(' ' + (
+      pending.prefix === '#default' ? 'xmlns' : 'xmlns:' + pending.prefix
+    ) + '="' +
+        this._replaceCharacterMaps(pending.namespaceURI) + '"');
+    this._pendingNamespaceMap.delete(prefix);
   }
 
   /**

package/src/XPathTransformer.js

@@ -50,10 +50,9 @@ class XPathTransformer {
         return {path: template[0], template: template[1]};
       }
       // Normalize 'match' to 'path' for XSLT compatibility
-      if (template.match && !template.path) {
-        return {...template, path: template.match};
-      }
-      return template;
+      return template.match && !template.path
+        ? {...template, path: template.match}
+        : template;
     });
     this.templates.forEach((template) => {
       if ('name' in template && template.name &&