jtlt 0.13.0 → 0.15.0

package/.c8rc.json

@@ -9,6 +9,7 @@
     "test/**",
     "typings/**",
     "node_modules/**",
+    "**/*.d.ts",
     "**/*.config.js",
     "rollup.config.js",
     "eslint.config.js",

package/CHANGES.md

@@ -1,5 +1,23 @@
 # jtlt CHANGES
 
+## 0.15.0
+
+- feat: allow custom extensions
+- feat(types): `jtlt/context-extensions` `ContextExtensions` interface;
+  augment it via declaration merging so `extensions` helpers type-check on
+  `this` inside templates without suppressions
+- chore: bump codemirror/state, jamilih, and devDeps.
+
+## 0.14.0
+
+- feat: indexedDB JSONPath and XPath support
+- feat: async templates are awaited by default; new off-by-default `sync`
+  option throws instead (replaces `async`/`syncOnly`)
+- fix: `getKey()` resolves its `match` against the document root, so it now
+  works inside `forEach()`/`applyTemplates()` callbacks
+- fix(types): more precise typing (any -> unknown)
+- docs: lead with the `jtlt()` function; correct examples
+
 ## 0.13.0
 
 - fix(types): more precise typing

package/README.md

@@ -8,8 +8,6 @@ As with XSLT, allows for declarative, linear declaration of
 (recursive) templates and can be transformed into different
 formats (e.g., strings, JSON, or DOM objects).
 
-***Beta state!!!***
-
 See the [Demo](https://brettz9.github.io/jtlt/demo/).
 
 ## Credits
@@ -29,9 +27,56 @@ See the [test file](./test/browser/index.html).
 
 ## Basic usage
 
-### Node
+The quickest way to run a transform is the **`jtlt()`** function. Give it a
+config object; it runs the transform and returns a `Promise` that resolves to
+the result:
+
+```js
+import {jtlt} from 'jtlt';
+
+const data = {title: 'Hello', items: ['a', 'b']};
+
+const templates = [
+  {path: '$', template () {
+    this.applyTemplates('$.title');
+    this.applyTemplates('$.items[*]');
+  }},
+  {path: '$.title', template (v) {
+    this.element('h1', {}, [], () => this.text(v));
+  }},
+  {path: '$.items[*]', template (v) {
+    this.element('li', {}, [], () => this.text(v));
+  }}
+];
+
+const out = await jtlt({data, templates, outputType: 'string'});
+// -> <h1>Hello</h1><li>a</li><li>b</li>
+```
+
+Templates may be `async` (for example to `await this.indexedDB(...)`);
+`jtlt()` awaits them automatically. Pass `sync: true` to forbid asynchronous
+templates (a template that then returns a Promise throws).
+
+The same call works in Node and the browser (in the browser you must also
+load the dependencies — see the [test file](./test/browser/index.html)). For
+XML/HTML sources, add `engineType: 'xpath'` — see
+[Quick start (XML source with XPath)](#quick-start-xml-source-with-xpath).
+
+### `jtlt()` vs `JTLT.create()`
+
+`jtlt()` is a thin, Promise-returning wrapper around the lower-level
+`JTLT` class. Prefer `jtlt()`. Reach for `JTLT.create()` /
+`new JTLT()` only when you need the instance itself, `autostart: false`, or
+to drive `.transform(mode)` yourself.
 
-### Browser
+| | `jtlt(config)` | `JTLT.create(config)` |
+| --- | --- | --- |
+| Returns | a `Promise` of the result | a `JTLT` instance |
+| Result delivery | the resolved value | a required `success` callback (also returned by `.transform()`) |
+| Async templates | awaited automatically | awaited automatically; `.transform()` returns a Promise |
+
+Anywhere below that shows `JTLT.create({…}).transform(mode)` can instead be
+written `await jtlt({…, mode})`.
 
 ## API
 
@@ -39,16 +84,20 @@ See the [docs](docs/API.md). A high‑level overview is below.
 
 ## API overview
 
-JTLT has two layers:
+Run a transform with the **`jtlt(config)`** function (Promise-returning,
+recommended) or the lower-level **`JTLT`** class (`JTLT.create(config)` /
+`new JTLT(config)`, which delivers the result through a required `success`
+callback). Under the hood JTLT has two layers:
 
 - Engine (template application):
     - JSONPathTransformer: Applies templates to JSON by matching JSONPath selectors (and optional modes), resolving priority, and invoking the winning template. Falls back to built‑in default rules when no user template matches.
     - JSONPathTransformerContext: The execution context passed to templates. It mirrors the joiner API (e.g., string(), object(), array()) so templates can emit results. It also provides helpers like applyTemplates(), callTemplate(), valueOf(), variable(), and forEach().
     - XPathTransformer (experimental): Applies templates to
       XML/HTML DOM by matching XPath selectors (and optional modes).
-      Supports three evaluation modes: version 1 (native
-      XPathEvaluator), version 2 (via xpath2.js), and version 3 (via fontoxpath).
-      Falls back to built‑in default rules when no template matches.
+      Supports three evaluation modes: version `1` (native
+      XPathEvaluator), version `2` (via xpath2.js), and version `3.1` (via
+      fontoxpath). Falls back to built‑in default rules when no template
+      matches.
     - XPathTransformerContext (experimental): Execution context for
       XPath. Offers get(), forEach(), valueOf(), variable(), key()
       and the same joiner helpers as the JSONPath context.
@@ -69,16 +118,16 @@ JTLT has two layers:
 
 ### Common joiner methods
 
-- append(value): Central sink. Based on context, concatenates to string, pushes to array, or assigns to an object property.
-- get(): Return the accumulated result.
-- object(obj?, cb?, usePropertySets?, propSets?): Enter object context; optionally seed from an object or build via cb.
-- array(arr?, cb?): Enter array context; optionally seed from an array or build via cb.
-- string(str, cb?): Emit a string value (no HTML escaping). In String joiner, optional cb lets you compose nested fragments before emitting.
-- number(num), boolean(bool), null(), undefined() (JS mode only), nonfiniteNumber(NaN|Infinity), function(fn) (JS mode only): Emit primitives/functions.
-- element(name, attrs?, children?, cb?): Build elements (String and DOM joiners). In String joiner, uses Jamilih under the hood to serialize; in DOM joiner, creates Elements.
-- attribute(name, value, avoidEscape?): Add attributes to the most recently opened element (String joiner) or to the current Element (DOM joiner).
-- text(txt): Emit text content. In String joiner, escapes & and <, and closes an open start tag if needed.
-- plainText(str): Raw, no‑escape append that bypasses context routing in the String joiner (always writes to top‑level buffer). In DOM/JSON joiners, it maps to text()/string() respectively.
+- `append(value)`: Central sink. Based on context, concatenates to string, pushes to array, or assigns to an object property.
+- `get()`: Return the accumulated result.
+- `object(obj?, cb?, usePropertySets?, propSets?)`: Enter object context; optionally seed from an object or build via cb.
+- `array(arr?, cb?)`: Enter array context; optionally seed from an array or build via cb.
+- `string(str, cb?)`: Emit a string value (no HTML escaping). In String joiner, optional cb lets you compose nested fragments before emitting.
+- `number(num), boolean(bool), null(), undefined()` (JS mode only), `nonfiniteNumber(NaN|Infinity), function(fn)` (JS mode only): Emit primitives/functions.
+- `element(name, attrs?, children?, cb?)`: Build elements (String and DOM joiners). In String joiner, uses Jamilih under the hood to serialize; in DOM joiner, creates Elements.
+- `attribute(name, value, avoidEscape?)`: Add attributes to the most recently opened element (String joiner) or to the current Element (DOM joiner).
+- `text(txt)`: Emit text content. In String joiner, escapes & and <, and closes an open start tag if needed.
+- `plainText(str)`: Raw, no‑escape append that bypasses context routing in the String joiner (always writes to top‑level buffer). In DOM/JSON joiners, it maps to text()/string() respectively.
 
 ### string() vs text() vs plainText() (String joiner)
 
@@ -95,56 +144,32 @@ Provide joiningConfig when constructing JTLT:
 - joiningConfig.xmlElements: Switch element() to XML serialization mode in the String joiner.
 - joiningConfig.preEscapedAttributes: Skip escaping attribute values in the String joiner.
 
-## Quick start (JSON source)
-
-```js
-import {jtlt} from 'jtlt';
-
-const data = {title: 'Hello', items: ['a', 'b']};
-
-const templates = [
-  {path: '$', template () {
-    this.applyTemplates();
-  }},
-  {path: '$.title', template (v) {
-    this.string('<h1>', () => this.text(v));
-    this.string('</h1>');
-  }},
-  {path: '$.items[*]', template (v) {
-    this.element('li', {}, [], () => this.text(v));
-  }}
-];
-
-const out = await jtlt({data, templates, outputType: 'string'});
-
-console.log(out);
-```
-
-Notes:
+## Notes on the basic example
 
 - Modes let you organize multiple passes or output targets.
-- You can also call templates by name via this.callTemplate('name').
-- For DOM output, use outputType: 'dom'. For JSON output, use 'json'.
+- You can also call templates by name via `this.callTemplate('name')`.
+- For DOM output, use `outputType: 'dom'`. For JSON output, use `'json'`
+  (the default is `'string'`).
 
-### Quick start (XML source with XPath)
+## Quick start (XML source with XPath)
 
 You can run templates against XML/HTML using XPath instead of JSONPath.
 
 - `data` should be a Document or Element (e.g., from `DOMParser` with
   `text/xml`).
-- `xpathVersion`: `1` uses native XPath (browser like). `2` uses
-  `xpath2.js` for XPath 2.0‑style evaluation. and `3.1` uses `fontoxpath`
-  for XPath 3.1.
+- `xpathVersion`: `1` uses native XPath (browser‑like). `2` uses
+  `xpath2.js` for XPath 2.0‑style evaluation. `3.1` uses `fontoxpath` for
+  XPath 3.1. Default is `1`.
 - In version 2, some functions may be missing; prefer simple path
   expressions. Use version 1 for standard XPath 1.0 function support.
 
-Example (string output) using the JTLT facade with XPath:
+Example (string output) with `jtlt()` and the XPath engine:
 
 ```js
 import {JSDOM} from 'jsdom';
 import {jtlt} from 'jtlt';
 
-const {window} = new JSDOM('<!doctype><html><body></body></html>');
+const {window} = new JSDOM('<!doctype html><html><body></body></html>');
 const parser = new window.DOMParser();
 const doc = parser.parseFromString(
   '<root><item>a</item><item>b</item></root>', 'text/xml'
@@ -160,8 +185,7 @@ const templates = [
   {
     path: '//item',
     template (n) {
-      this.string('<li>', () => this.text(n.textContent));
-      this.string('</li>');
+      this.element('li', {}, [], () => this.text(n.textContent));
     }
   }
 ];
@@ -171,8 +195,7 @@ const out = await jtlt({
   templates,
   outputType: 'string',
   engineType: 'xpath',
-  xpathVersion: 1, // or 2
-  success: (res) => res
+  xpathVersion: 1 // or 2, or 3.1
 });
 // -> <li>a</li><li>b</li>
 ```
@@ -182,15 +205,14 @@ const out = await jtlt({
 If you just want to run a single, non-recursive query (similar to an XQuery "for … where … return …"), you can skip defining templates and use `forQuery` to seed a root function that iterates a JSONPath and emits results.
 
 - `forQuery` takes the same arguments you’d pass to `this.forEach(select, cb)`: an absolute JSONPath selector and a callback invoked for each match.
-- You can set variables via `this.variable(name, select)` and use plain JavaScript `if` for conditions (there is no dedicated `this.if`).
+- The callback runs once per match with `this` bound to that match, so use plain JavaScript `if` for conditions (there is no dedicated `this.if`).
 
-Example: collect item names whose price meets a threshold, using a variable sourced from the root.
+Example: collect item names whose price is at least 10.
 
 ```js
-import JTLT from 'jtlt';
+import {jtlt} from 'jtlt';
 
 const data = {
-  threshold: 10,
   items: [
     {name: 'A', price: 8},
     {name: 'B', price: 12},
@@ -198,37 +220,29 @@ const data = {
   ]
 };
 
-const jtlt = JTLT.create({
+const result = await jtlt({
   data,
   outputType: 'json', // Top-level result will be a JSON array
   // forQuery mirrors: this.forEach(select, cb)
   forQuery: [
     '$.items[*]',
     function (item) {
-      // Set a reusable variable from the root context
-      this.variable('threshold', '$.threshold');
-      const {threshold} = this.vars;
-
       // Use normal JS conditionals (no this.if helper)
-      if (item.price >= threshold) {
+      if (item.price >= 10) {
         // In JSON output mode, appending a string pushes into
         //   the top-level array
         this.string(item.name);
       }
     }
-  ],
-  // success receives the final result; return it for convenience
-  success: (out) => out
+  ]
 });
-
-const result = jtlt.transform();
 // result => ['B', 'C']
 ```
 
 Tips:
 
 - For string output, set `outputType: 'string'` and emit with `this.text()`/`this.string()` in the callback.
-- `this.variable(name, select)` evaluates the JSONPath against the current context (root for `forQuery`), storing it in `this.vars[name]`.
+- `forQuery`'s callback context is the matched item, not the root — to use a value from the root (e.g. a `threshold`), use a root template instead: `this.variable('threshold', '$.threshold')` then `this.forEach('$.items[*]', cb)` (see the FLWOR example below).
 - If you need multiple passes or richer logic, switch to named templates and modes.
 
 ## FLWOR-style (XQuery) example
@@ -238,7 +252,7 @@ You can express the essentials of a FLWOR expression (For, Let, Where, Order by,
 Scenario: list book titles whose price is at/above a threshold, ordered by price descending and then title ascending.
 
 ```js
-import JTLT from 'jtlt';
+import {jtlt} from 'jtlt';
 
 const data = {
   threshold: 10,
@@ -274,8 +288,7 @@ const templates = [
   }}
 ];
 
-const out = JTLT.create({data, templates, outputType: 'string'}).
-  transform('html');
+const out = await jtlt({data, templates, outputType: 'string', mode: 'html'});
 
 // -> <ul><li>Brave New</li><li>Cobalt</li><li>Delta</li></ul>
 console.log(out);
@@ -297,7 +310,7 @@ You can model a join across two arrays (e.g., orders ↔ customers) using two `f
 Example: render an HTML list of orders annotated with customer names.
 
 ```js
-import JTLT from 'jtlt';
+import {jtlt} from 'jtlt';
 
 const data = {
   customers: [
@@ -333,10 +346,11 @@ const templates = [
   }}
 ];
 
-const out = JTLT.create({
-  data, templates, outputType: 'string'
-}).transform('html');
-// -> <ul><li>Bob — Keyboard</li><li>Alice — Mouse</li></ul>
+const out = await jtlt({
+  data, templates, outputType: 'string', mode: 'html'
+});
+// sorted by date ascending:
+// -> <ul><li>Alice — Mouse</li><li>Bob — Keyboard</li></ul>
 console.log(out);
 ```
 
@@ -352,7 +366,7 @@ Notes:
 Define an index once, then perform O(1) lookups from another sequence when rendering. If no match is found, `getKey()` returns the current context (`this`) as a sentinel; check for that to skip safely.
 
 ```js
-import JTLT from 'jtlt';
+import {jtlt} from 'jtlt';
 
 const data = {
   customers: [
@@ -383,9 +397,9 @@ const templates = [
   }}
 ];
 
-const out = JTLT.create({
-  data, templates, outputType: 'string'
-}).transform('html');
+const out = await jtlt({
+  data, templates, outputType: 'string', mode: 'html'
+});
 // -> <ul><li>Bob: Keyboard</li></ul>
 console.log(out);
 ```
@@ -416,7 +430,7 @@ Differences / current limitations:
   allow a particuluar subset of JavaScript.
 - Stylesheet composition/precedence: no `xsl:import`/`xsl:include` equivalents; only basic priority and modes.
 - Schema awareness: no type-aware processing (a major XSLT/XQuery feature).
-- Multi-output (`xsl:result-document`): not built-in; pick one output type per transform.
+- One output type per transform, though `document()` / `resultDocument()` can emit several documents of that type within a run.
 
 ## Differences between an exact equivalence with XSLT
 

package/demo/calltemplate-params-demo.js

@@ -1,17 +1,17 @@
 /**
  * Demo: Using valueOf() to access parameters in callTemplate
  *
- * This demonstrates the new feature where parameters passed via callTemplate
+ * This demonstrates the feature where parameters passed via callTemplate
  * can be accessed within the template using valueOf({select: '$paramName'})
  * instead of having to receive them as function parameters.
  */
 
 /* eslint-disable no-console -- Demo file */
 
-import JTLT from '../src/index-node.js';
+import {jtlt} from '../src/index-node.js';
 
 console.log('=== Demo 1: Named parameters ===');
-JTLT.create({
+console.log(await jtlt({
   data: {
     users: [
       {name: 'Alice', role: 'Admin'},
@@ -50,15 +50,12 @@ JTLT.create({
         this.string(')\n');
       }
     }
-  ],
-  success (result) {
-    console.log(result);
-    console.log('\n');
-  }
-});
+  ]
+}));
+console.log('\n');
 
 console.log('=== Demo 2: Indexed parameters (no names) ===');
-JTLT.create({
+console.log(await jtlt({
   data: {value: 'Test'},
   outputType: 'string',
   templates: [
@@ -88,15 +85,12 @@ JTLT.create({
         this.string('\n');
       }
     }
-  ],
-  success (result) {
-    console.log(result);
-    console.log('\n');
-  }
-});
+  ]
+}));
+console.log('\n');
 
 console.log('=== Demo 3: Nested callTemplate ===');
-JTLT.create({
+console.log(await jtlt({
   data: {company: 'ACME Corp'},
   outputType: 'string',
   templates: [
@@ -135,8 +129,5 @@ JTLT.create({
         this.string('\n');
       }
     }
-  ],
-  success (result) {
-    console.log(result);
-  }
-});
+  ]
+}));

package/demo/codemirror.esm.js

@@ -7879,7 +7879,7 @@ class Chunk {
         this.value = value;
         this.maxPoint = maxPoint;
     }
-    get length() { return this.to[this.to.length - 1]; }
+    get length() { return last(this.to); }
     // Find the index of the given position and side. Use the ranges'
     // `from` pos when `end == false`, `to` when `end == true`.
     findIndex(pos, side, end, startAt = 0) {
@@ -7902,9 +7902,9 @@ class Chunk {
             if (f(this.from[i] + offset, this.to[i] + offset, this.value[i]) === false)
                 return false;
     }
-    map(offset, changes) {
+    map(offset, changes, basePos, baseSide, spill) {
         let value = [], from = [], to = [], newPos = -1, maxPoint = -1;
-        for (let i = 0; i < this.value.length; i++) {
+        iter: for (let i = 0; i < this.value.length; i++) {
             let val = this.value[i], curFrom = this.from[i] + offset, curTo = this.to[i] + offset, newFrom, newTo;
             if (curFrom == curTo) {
                 let mapped = changes.mapPos(curFrom, val.startSide, val.mapMode);
@@ -7929,9 +7929,29 @@ class Chunk {
                 newPos = newFrom;
             if (val.point)
                 maxPoint = Math.max(maxPoint, newTo - newFrom);
-            value.push(val);
-            from.push(newFrom - newPos);
-            to.push(newTo - newPos);
+            if ((newFrom - basePos || val.startSide - baseSide) >= 0) {
+                value.push(val);
+                from.push(newFrom - newPos);
+                to.push(newTo - newPos);
+                basePos = newTo;
+                baseSide = val.endSide;
+            }
+            else {
+                if (newFrom == newTo) { // Try to reorder points to fit in here
+                    for (let i = value.length; i > 0; i--) {
+                        if ((newFrom - (to[i - 1] + newPos) || val.startSide - value[i - 1].endSide) >= 0) {
+                            value.splice(i, 0, val);
+                            from.splice(i, 0, newFrom - newPos);
+                            to.splice(i, 0, newTo - newPos);
+                            continue iter;
+                        }
+                        if ((newFrom - (from[i - 1] + newPos) || val.endSide - value[i - 1].startSide) > 0)
+                            break;
+                    }
+                }
+                // Otherwise, spill into a new layer
+                spill(newFrom, newTo, val);
+            }
         }
         return { mapped: value.length ? new Chunk(from, to, value, maxPoint) : null, pos: newPos };
     }
@@ -8018,7 +8038,7 @@ class RangeSet {
         while (cur.value || i < add.length) {
             if (i < add.length && (cur.from - add[i].from || cur.startSide - add[i].value.startSide) >= 0) {
                 let range = add[i++];
-                if (!builder.addInner(range.from, range.to, range.value))
+                if (!builder.addInner(range.from, range.to, range.value, false))
                     spill.push(range);
             }
             else if (cur.rangeIndex == 1 && cur.chunkIndex < this.chunk.length &&
@@ -8029,7 +8049,7 @@ class RangeSet {
             }
             else {
                 if (!filter || filterFrom > cur.to || filterTo < cur.from || filter(cur.from, cur.to, cur.value)) {
-                    if (!builder.addInner(cur.from, cur.to, cur.value))
+                    if (!builder.addInner(cur.from, cur.to, cur.value, false))
                         spill.push(Range.create(cur.from, cur.to, cur.value));
                 }
                 cur.next();
@@ -8045,6 +8065,12 @@ class RangeSet {
         if (changes.empty || this.isEmpty)
             return this;
         let chunks = [], chunkPos = [], maxPoint = -1;
+        let spilled;
+        let spill = (from, to, value) => {
+            if (!spilled)
+                spilled = new RangeSetBuilder();
+            spilled.addRange(from, to, value, false);
+        };
         for (let i = 0; i < this.chunk.length; i++) {
             let start = this.chunkPos[i], chunk = this.chunk[i];
             let touch = changes.touchesRange(start, start + chunk.length);
@@ -8054,7 +8080,9 @@ class RangeSet {
                 chunkPos.push(changes.mapPos(start));
             }
             else if (touch === true) {
-                let { mapped, pos } = chunk.map(start, changes);
+                let [prevPos, prevSide] = !chunks.length ? [-1, -1]
+                    : [last(chunkPos) + last(chunks).length, last(last(chunks).value).endSide];
+                let { mapped, pos } = chunk.map(start, changes, prevPos, prevSide, spill);
                 if (mapped) {
                     maxPoint = Math.max(maxPoint, mapped.maxPoint);
                     chunks.push(mapped);
@@ -8063,6 +8091,8 @@ class RangeSet {
             }
         }
         let next = this.nextLayer.map(changes);
+        if (spilled)
+            next = spilled.finishInner(next);
         return chunks.length == 0 ? next : new RangeSet(chunkPos, chunks, next || RangeSet.empty, maxPoint);
     }
     /**
@@ -8204,7 +8234,7 @@ class RangeSet {
     static join(sets) {
         if (!sets.length)
             return RangeSet.empty;
-        let result = sets[sets.length - 1];
+        let result = last(sets);
         for (let i = sets.length - 2; i >= 0; i--) {
             for (let layer = sets[i]; layer != RangeSet.empty; layer = layer.nextLayer)
                 result = new RangeSet(layer.chunkPos, layer.chunk, result, Math.max(layer.maxPoint, result.maxPoint));
@@ -8216,6 +8246,7 @@ class RangeSet {
 The empty set of ranges.
 */
 RangeSet.empty = /*@__PURE__*/new RangeSet([], [], null, -1);
+function last(arr) { return arr[arr.length - 1]; }
 function lazySort(ranges) {
     if (ranges.length > 1)
         for (let prev = ranges[0], i = 1; i < ranges.length; i++) {
@@ -8266,16 +8297,20 @@ class RangeSetBuilder {
     Add a range. Ranges should be added in sorted (by `from` and
     `value.startSide`) order.
     */
-    add(from, to, value) {
-        if (!this.addInner(from, to, value))
-            (this.nextLayer || (this.nextLayer = new RangeSetBuilder)).add(from, to, value);
+    add(from, to, value) { this.addRange(from, to, value, true); }
+    /**
+    @internal
+    */
+    addRange(from, to, value, strict) {
+        if (!this.addInner(from, to, value, strict))
+            (this.nextLayer || (this.nextLayer = new RangeSetBuilder)).addRange(from, to, value, strict);
     }
     /**
     @internal
     */
-    addInner(from, to, value) {
+    addInner(from, to, value, strict) {
         let diff = from - this.lastTo || value.startSide - this.last.endSide;
-        if (diff <= 0 && (from - this.lastFrom || value.startSide - this.last.startSide) < 0)
+        if (strict && diff <= 0 && (from - this.lastFrom || value.startSide - this.last.startSide) < 0)
             throw new Error("Ranges must be added sorted by `from` position and `startSide`");
         if (diff < 0)
             return false;

package/demo/vendor/jhtml/src/SAJJ/SAJJ.ObjectArrayDelegator.js

@@ -7,13 +7,12 @@ import SAJJ from './SAJJ.js';
 /* eslint-enable jsdoc/reject-any-type -- Arbitrary */
 
 /**
-* @abstract
-* @class
-* @todo Might add an add() method which defines how to combine result values
-*  (so as to allow for other means besides string concatenation)
-*/
+ * @abstract
+ * @class
+ * @todo Might add an add() method which defines how to combine result values
+ *  (so as to allow for other means besides string concatenation)
+ */
 class ObjectArrayDelegator extends SAJJ {
-  /* eslint-disable jsdoc/require-returns-check -- Abstract */
   /**
    * @returns {AnyDelegated}
    */
@@ -76,7 +75,6 @@ class ObjectArrayDelegator extends SAJJ {
   ) {
     throw new Error('Abstract');
   }
-  /* eslint-enable jsdoc/require-returns-check -- Abstract */
 
   // It is probably not necessary to override the defaults for the following
   //   two methods and perhaps not any of the others either
@@ -109,17 +107,19 @@ class ObjectArrayDelegator extends SAJJ {
         }
       } else {
         for (const key in value) {
-          if (Object.hasOwn(value, key)) {
-            this.currentKey = key;
-            this.currentObject = value[key];
-            keyVals.push(
-              this.keyValueHandler(
-                value[key], key, value, parentKey,
-                parentObjectArrayBool, false, i
-              )
-            );
-            i++;
+          if (!Object.hasOwn(value, key)) {
+            continue;
           }
+
+          this.currentKey = key;
+          this.currentObject = value[key];
+          keyVals.push(
+            this.keyValueHandler(
+              value[key], key, value, parentKey,
+              parentObjectArrayBool, false, i
+            )
+          );
+          i++;
         }
       }
     }