xslt-processor 5.0.0 → 5.0.1

package/README.md

@@ -119,7 +119,47 @@ const xslt = new Xslt(options);
 - `parameters` (`array`, default `[]`): external parameters that you want to use.
     - `name`: the parameter name;
     - `namespaceUri` (optional): the namespace;
-    - `value`: the value.
+    - `value`: the value. The type is preserved automatically:
+        - **string** values become `StringValue`;
+        - **number** values become `NumberValue` (usable in XPath arithmetic);
+        - **boolean** values become `BooleanValue` (usable in `xsl:if`/`xsl:when` tests);
+        - Note: in XPath/XSLT, any non-empty string is truthy, so the string "false" still behaves as true in tests;
+        - **`NodeSetValue`** instances are kept as-is (useful for passing additional documents);
+        - DOM nodes (objects with `nodeType`) are wrapped in a `NodeSetValue`;
+        - arrays of nodes are wrapped in a `NodeSetValue`.
+
+**`parameters` examples:**
+
+```js
+import { Xslt, XmlParser } from 'xslt-processor'
+
+// String parameter (default behavior)
+const xslt = new Xslt({ parameters: [
+  { name: 'title', value: 'Hello' }
+] });
+
+// Number parameter — works in XPath arithmetic ($count + 1 = 43)
+const xslt = new Xslt({ parameters: [
+  { name: 'count', value: 42 }
+] });
+
+// Boolean parameter — works in xsl:if / xsl:when tests
+const xslt = new Xslt({ parameters: [
+  { name: 'debug', value: true }
+] });
+
+// Node-set parameter — pass an additional document for cross-document lookups
+import { NodeSetValue } from 'xslt-processor/xpath/values'
+
+const xmlParser = new XmlParser();
+const lookupDoc = xmlParser.xmlParse('<lookup><entry key="a">Alpha</entry></lookup>');
+
+const xslt = new Xslt({ parameters: [
+  { name: 'lookup', value: new NodeSetValue([lookupDoc]) }
+] });
+// In XSLT: <xsl:value-of select="$lookup/lookup/entry[@key='a']"/>
+```
+
 - `fetchFunction` (`(uri: string) => Promise<string>`, optional): a custom function for loading external resources referenced by `<xsl:import>` and `<xsl:include>`. Receives the URI and must return the fetched content as a string. Defaults to the global `fetch` API. This is useful for:
     - Denying external loading entirely;
     - Loading from the local filesystem or other non-HTTP sources;

package/index.js

@@ -11867,28 +11867,37 @@ var NodeConverter = class {
    */
   convertVariables(exprContext) {
     const variables = {};
-    for (const [name, value] of Object.entries(exprContext.variables || {})) {
-      if (value && typeof value === "object" && "stringValue" in value) {
-        const nodeValue = value;
-        if (nodeValue.type === "node-set") {
-          variables[name] = value.nodeSetValue().map((n) => this.adaptXNode(n));
-        } else if (nodeValue.type === "string") {
-          variables[name] = value.stringValue();
-        } else if (nodeValue.type === "number") {
-          variables[name] = value.numberValue();
-        } else if (nodeValue.type === "boolean") {
-          variables[name] = value.booleanValue();
-        } else if (nodeValue.type === "map") {
-          variables[name] = nodeValue.value;
-        } else if (nodeValue.type === "array") {
-          variables[name] = nodeValue.value;
-        } else if (nodeValue.type === "function") {
-          variables[name] = nodeValue.value;
+    const contexts = [];
+    let ctx = exprContext;
+    while (ctx) {
+      contexts.push(ctx);
+      ctx = ctx.parent;
+    }
+    for (let i = contexts.length - 1; i >= 0; i--) {
+      const current = contexts[i];
+      for (const [name, value] of Object.entries(current.variables || {})) {
+        if (value && typeof value === "object" && "stringValue" in value) {
+          const nodeValue = value;
+          if (nodeValue.type === "node-set") {
+            variables[name] = value.nodeSetValue().map((n) => this.adaptXNode(n));
+          } else if (nodeValue.type === "string") {
+            variables[name] = value.stringValue();
+          } else if (nodeValue.type === "number") {
+            variables[name] = value.numberValue();
+          } else if (nodeValue.type === "boolean") {
+            variables[name] = value.booleanValue();
+          } else if (nodeValue.type === "map") {
+            variables[name] = nodeValue.value;
+          } else if (nodeValue.type === "array") {
+            variables[name] = nodeValue.value;
+          } else if (nodeValue.type === "function") {
+            variables[name] = nodeValue.value;
+          } else {
+            variables[name] = value.stringValue();
+          }
         } else {
-          variables[name] = value.stringValue();
+          variables[name] = value;
         }
-      } else {
-        variables[name] = value;
       }
     }
     return variables;
@@ -12481,7 +12490,7 @@ var ExprContext = class _ExprContext {
       this.xsltVersion,
       typeof opt_position !== "undefined" ? opt_position : this.position,
       this.decimalFormatSettings,
-      this.variables,
+      Object.create(this.variables || {}),
       this.knownNamespaces,
       this,
       this.caseInsensitive,