bare-script 3.0.11 → 3.0.13

package/lib/library.js

@@ -40,6 +40,26 @@ const arrayCopyArgs = valueArgsModel([
 ]);
 
 
+// $function: arrayDelete
+// $group: Array
+// $doc: Delete an array element
+// $arg array: The array
+// $arg index: The index of the element to delete
+function arrayDelete(args) {
+    const [array, index] = valueArgsValidate(arrayDeleteArgs, args);
+    if (index >= array.length) {
+        throw new ValueArgsError('index', index);
+    }
+
+    array.splice(index, 1);
+}
+
+const arrayDeleteArgs = valueArgsModel([
+    {'name': 'array', 'type': 'array'},
+    {'name': 'index', 'type': 'number', 'integer': true, 'gte': 0}
+]);
+
+
 // $function: arrayExtend
 // $group: Array
 // $doc: Extend one array with another
@@ -2016,21 +2036,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,27 +2057,27 @@ 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'}
 ]);
 
 
 // The built-in script functions
 export const scriptFunctions = {
     arrayCopy,
+    arrayDelete,
     arrayExtend,
     arrayGet,
     arrayIndexOf,

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.13",
     "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.14",
         "jsdoc": "~4.0"
     }
 }