pacc 6.6.1 → 6.6.3

package/README.md

@@ -204,7 +204,7 @@ Type: [Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Globa
 *   `collection` **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)** 
 *   `private` **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)?** should the value be shown
 *   `credential` **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)?** any type of credential
-*   `persistent` **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)?** should we be stored
+*   `persistent` **[boolean](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Boolean)?** should we be stored (especially critical for credentials)
 *   `depends` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)?** name of an attribute we depend on
 *   `description` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)?** human readable
 *   `default` **any?** the default value
@@ -212,7 +212,7 @@ Type: [Object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Globa
 *   `get` **[Function](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Statements/function)?** get the value can be used to calculate default values
 *   `prepareValue` **[Function](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Statements/function)?** 
 *   `values` **[Set](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Set)\<any>?** allowed values
-*   `externalName` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)?** attrubute name used by external system
+*   `externalName` **[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)?** attribute name used by external system
 *   `env` **([Array](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Array)<[string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String)> | [string](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/String))?** environment variable(s) used to provide the value
 *   `additionalValues` **[object](https://developer.mozilla.org/docs/Web/JavaScript/Reference/Global_Objects/Object)?** other values to be set in case our attribute is set
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pacc",
-  "version": "6.6.1",
+  "version": "6.6.3",
   "publishConfig": {
     "access": "public",
     "provenance": true
@@ -39,7 +39,7 @@
   },
   "devDependencies": {
     "ava": "^6.4.1",
-    "browser-ava": "^2.3.47",
+    "browser-ava": "^2.3.49",
     "c8": "^10.1.3",
     "documentation": "^14.0.3",
     "semantic-release": "^25.0.2",

package/src/bytes.mjs

@@ -37,7 +37,7 @@ export function parseBytes(value) {
       const v = parseFloat(match[1]);
 
       if (match[2]) {
-        bytes += v * units[match[2]];
+        bytes += v * units[match[2].toLocaleLowerCase()];
       } else {
         bytes += v;
       }

package/src/common-attributes.mjs

@@ -10,7 +10,7 @@ import { types } from "./types.mjs";
  * @property {boolean} collection
  * @property {boolean} [private] should the value be shown
  * @property {boolean} [credential] any type of credential
- * @property {boolean} [persistent] should we be stored
+ * @property {boolean} [persistent] should we be stored (especially critical for credentials)
  * @property {string} [depends] name of an attribute we depend on
  * @property {string} [description] human readable
  * @property {any} [default] the default value
@@ -18,7 +18,7 @@ import { types } from "./types.mjs";
  * @property {Function} [get] get the value can be used to calculate default values
  * @property {Function} [prepareValue]
  * @property {Set<any>} [values] allowed values
- * @property {string} [externalName] attrubute name used by external system
+ * @property {string} [externalName] attribute name used by external system
  * @property {string[]|string} [env] environment variable(s) used to provide the value
  * @property {object} [additionalValues] other values to be set in case our attribute is set
  */

package/src/expand.mjs

@@ -6,14 +6,14 @@ const maxNestingLevel = 5;
  * Expand expressions inside of object graphs.
  * @param {any} object
  * @param {Object} context
- * @param {any} context.root
+ * @param {any} [context.root]
  * @param {function} [context.stopClass]
  * @param {string} [context.leadIn]
  * @param {string} [context.leadOut]
  * @returns {any}
  */
 export function expand(object, context = {}) {
-  const promises = [];
+  const /** @type {Array<Promise<any>>} */ promises = [];
 
   const leadIn = context.leadIn ?? "${";
   const leadOut = context.leadOut ?? "}";

package/src/types.mjs

@@ -61,10 +61,12 @@ export const types = {
   object: { name: "object", primitive: false }
 };
 
-function error(message) {
-  throw new Error(message);
-}
-
+/**
+ * 
+ * @param {string|undefined} type 
+ * @param {any} origin 
+ * @returns {Type}
+ */
 function raiseOnUnknownType(type, origin) {
   switch (typeof type) {
     case "string":
@@ -72,7 +74,7 @@ function raiseOnUnknownType(type, origin) {
         return types[type];
       }
     case "undefined":
-      error(`Unknown type ${type} in '${origin}'`);
+      throw new Error(`Unknown type ${type} in '${origin}'`, { cause: type });
   }
 
   return type;
@@ -174,7 +176,6 @@ export function resolveTypeLinks() {
 
 export function typeFactory(type, owner, data) {
   const factory = type.factoryFor?.(owner, data) || type.clazz;
-  //console.log(factory, type, owner, data);
   const object = new factory(owner);
 
   object.read(data);

package/types/common-attributes.d.mts

@@ -8,7 +8,7 @@
  * @property {boolean} collection
  * @property {boolean} [private] should the value be shown
  * @property {boolean} [credential] any type of credential
- * @property {boolean} [persistent] should we be stored
+ * @property {boolean} [persistent] should we be stored (especially critical for credentials)
  * @property {string} [depends] name of an attribute we depend on
  * @property {string} [description] human readable
  * @property {any} [default] the default value
@@ -16,7 +16,7 @@
  * @property {Function} [get] get the value can be used to calculate default values
  * @property {Function} [prepareValue]
  * @property {Set<any>} [values] allowed values
- * @property {string} [externalName] attrubute name used by external system
+ * @property {string} [externalName] attribute name used by external system
  * @property {string[]|string} [env] environment variable(s) used to provide the value
  * @property {object} [additionalValues] other values to be set in case our attribute is set
  */
@@ -213,7 +213,7 @@ export type AttributeDefinition = {
      */
     credential?: boolean;
     /**
-     * should we be stored
+     * should we be stored (especially critical for credentials)
      */
     persistent?: boolean;
     /**
@@ -242,7 +242,7 @@ export type AttributeDefinition = {
      */
     values?: Set<any>;
     /**
-     * attrubute name used by external system
+     * attribute name used by external system
      */
     externalName?: string;
     /**

package/types/expand.d.mts

@@ -2,14 +2,14 @@
  * Expand expressions inside of object graphs.
  * @param {any} object
  * @param {Object} context
- * @param {any} context.root
+ * @param {any} [context.root]
  * @param {function} [context.stopClass]
  * @param {string} [context.leadIn]
  * @param {string} [context.leadOut]
  * @returns {any}
  */
 export function expand(object: any, context?: {
-    root: any;
+    root?: any;
     stopClass?: Function;
     leadIn?: string;
     leadOut?: string;