bare-script 3.0.11 → 3.0.12

package/lib/library.js

@@ -2016,21 +2016,20 @@ function systemType([value = null]) {
 // $group: URL
 // $doc: Encode a URL
 // $arg url: The URL string
-// $arg extra: Optional (default is true). If true, encode extra characters for wider compatibility.
 // $return: The encoded URL string
 function urlEncode(args) {
-    const [url, extra] = valueArgsValidate(urlEncodeArgs, args);
+    const [url] = valueArgsValidate(urlEncodeArgs, args);
     let urlEncoded = encodeURI(url);
-    if (extra) {
-        // Replace ')' with '%29' for Markdown links
-        urlEncoded = urlEncoded.replaceAll(')', '%29');
-    }
+
+    // Encode '(' and ')' (for Markdown links)
+    urlEncoded = urlEncoded.replaceAll('(', '%28');
+    urlEncoded = urlEncoded.replaceAll(')', '%29');
+
     return urlEncoded;
 }
 
 const urlEncodeArgs = valueArgsModel([
-    {'name': 'url', 'type': 'string'},
-    {'name': 'extra', 'type': 'boolean', 'default': true}
+    {'name': 'url', 'type': 'string'}
 ]);
 
 
@@ -2038,21 +2037,20 @@ const urlEncodeArgs = valueArgsModel([
 // $group: URL
 // $doc: Encode a URL component
 // $arg url: The URL component string
-// $arg extra: Optional (default is true). If true, encode extra characters for wider compatibility.
 // $return: The encoded URL component string
 function urlEncodeComponent(args) {
-    const [url, extra] = valueArgsValidate(urlEncodeComponentArgs, args);
+    const [url] = valueArgsValidate(urlEncodeComponentArgs, args);
     let urlEncoded = encodeURIComponent(url);
-    if (extra) {
-        // Replace ')' with '%29' for Markdown links
-        urlEncoded = urlEncoded.replaceAll(')', '%29');
-    }
+
+    // Encode '(' and ')' (for Markdown links)
+    urlEncoded = urlEncoded.replaceAll('(', '%28');
+    urlEncoded = urlEncoded.replaceAll(')', '%29');
+
     return urlEncoded;
 }
 
 const urlEncodeComponentArgs = valueArgsModel([
-    {'name': 'url', 'type': 'string'},
-    {'name': 'extra', 'type': 'boolean', 'default': true}
+    {'name': 'url', 'type': 'string'}
 ]);
 
 

package/lib/value.js

@@ -313,7 +313,7 @@ export function valueArgsValidate(fnArgs, args, errorReturnValue = null) {
 
     // Extra arguments?
     if (args.length > fnArgsLength) {
-        args.splice(fnArgsLength);
+        throw new ValueArgsError(null, args.length, errorReturnValue);
     }
 
     return args;
@@ -331,12 +331,18 @@ export class ValueArgsError extends Error {
     /**
      * Create a BareScript runtime error
      *
-     * @param {string} argName - The function argument name
+     * @param {string} argName - The function argument name. If `arg_name` is null, there are too many arguments,
+     *     and `arg_value` is the number of arguments.
      * @param {*} argValue - The function argument value
      * @param {*} [returnValue = null] - The function's error return value
      */
     constructor(argName, argValue, returnValue = null) {
-        const message = `Invalid "${argName}" argument value, ${valueJSON(argValue)}`;
+        let message;
+        if (argName === null) {
+            message = `Too many arguments (${valueJSON(argValue)})`;
+        } else {
+            message = `Invalid "${argName}" argument value, ${valueJSON(argValue)}`;
+        }
         super(message);
         this.name = this.constructor.name;
         this.returnValue = returnValue;

package/package.json

@@ -1,7 +1,7 @@
 {
     "type": "module",
     "name": "bare-script",
-    "version": "3.0.11",
+    "version": "3.0.12",
     "description": "BareScript; a lightweight scripting and expression language",
     "keywords": [
         "expression",
@@ -30,8 +30,8 @@
         "schema-markdown": "~1.2"
     },
     "devDependencies": {
-        "c8": "~9.1",
-        "eslint": "~9.2",
+        "c8": "~10.1",
+        "eslint": "~9.9",
         "jsdoc": "~4.0"
     }
 }