exprify 1.0.3 → 1.0.6

package/src/utils/store.js

@@ -1,8 +1,7 @@
 export function createUnitsStore(initial = {}) {
-  let units = { ...initial};
-
-  // ---------- Helpers ----------
+  let units = { ...initial };
 
+  // Helpers
   function getAllUnitsFlat() {
     const result = new Set();
 
@@ -19,7 +18,6 @@ export function createUnitsStore(initial = {}) {
 
           // Avoid duplicate like "m" vs "meter"
           if (unitLower !== keyLower) {
-            // Optional: only single-word units
             if (unitLower.split(/\s+/).length === 1) {
               result.add(unitLower);
             }
@@ -41,6 +39,9 @@ export function createUnitsStore(initial = {}) {
     return Array.from(result);
   }
 
+  /**
+   * @param {string} input
+   */
   function findUnit(input) {
     input = input.toLowerCase();
 
@@ -53,7 +54,7 @@ export function createUnitsStore(initial = {}) {
           u.unit?.toLowerCase() === input ||
           u.symbol?.toLowerCase() === input
         ) {
-          return { type, key , data: u};
+          return { type, key, data: u };
         }
       }
     }
@@ -61,68 +62,89 @@ export function createUnitsStore(initial = {}) {
     return null;
   }
 
-  // ---------- Core Convert ----------
-
+  /**
+   * @param {number} value
+   * @param {any} fromUnit
+   * @param {any} toUnit
+   */
   function convert(value, fromUnit, toUnit) {
-      const from = findUnit(fromUnit);
-      const to = findUnit(toUnit);
+    const from = findUnit(fromUnit);
+    const to = findUnit(toUnit);
 
-      if (!from) throw new Error(`Unknown unit: ${fromUnit}`);
-      if (!to) throw new Error(`Unknown unit: ${toUnit}`);
-
-      if (from.type !== to.type) {
-        throw new Error(`Cannot convert ${fromUnit} to ${toUnit} (${to.data.unit || to.key}). ${from.data.unit || from.key} conversion units like ${Object.keys(units[from.type]).join(", ")}`);
-      }
-
-      const result = value * (from.data.value / to.data.value);
+    if (!from) {
+      throw new Error(`Unknown unit: ${fromUnit}`);
+    }
+    if (!to) {
+      throw new Error(`Unknown unit: ${toUnit}`);
+    }
 
-      return { value: result, unit: to.key };
+    if (from.type !== to.type) {
+      throw new Error(
+        `Cannot convert ${fromUnit} to ${toUnit} (${to.data.unit || to.key}). ${from.data.unit || from.key} conversion units like ${Object.keys(units[from.type]).join(', ')}`
+      );
     }
 
-  // ---------- Public API ----------
+    const result = value * (from.data.value / to.data.value);
 
+    return { value: result, unit: to.key };
+  }
+
+  // Public API
   return {
     // Get all units
     getUnits: () => units,
 
-    // Replace all units
-    setUnits: (newUnits) => {
+    setUnits: (/** @type {{}} */ newUnits) => {
       units = { ...newUnits };
     },
 
-    // Update single type
-    updateType: (type, data) => {
+    updateType: (/** @type {string | number} */ type, /** @type {any} */ data) => {
       units[type] = { ...units[type], ...data };
     },
 
-    // Add new unit
-    addUnit: (type, key, unitObj) => {
-      if (!units[type]) units[type] = {};
+    addUnit: (
+      /** @type {string | number} */ type,
+      /** @type {string | number} */ key,
+      /** @type {any} */ unitObj
+    ) => {
+      if (!units[type]) {
+        units[type] = {};
+      }
       units[type][key] = unitObj;
     },
+    // Unit-aware arithmetic: unify operands to same unit type, then apply operator
+    /**
+     * @param {string} op
+     * @param {{ unit: any; value: any; }} left
+     * @param {{ unit: any; value: number; }} right
+     */
     compute(op, left, right) {
+      const isUnit = (/** @type {any} */ v) =>
+        v && typeof v === 'object' && 'value' in v && 'unit' in v;
 
-      const isUnit = (v) =>
-        v && typeof v === "object" && "value" in v && "unit" in v;
-
-      const apply = (a, b) => {
+      const apply = (/** @type {any} */ a, /** @type {any} */ b) => {
         switch (op) {
-          case "+": return a + b;
-          case "-": return a - b;
-          case "*": return a * b;
-          case "/": return a / b;
-          case "%": return a % b;
-          case "^": return Math.pow(a, b);
+          case '+':
+            return a + b;
+          case '-':
+            return a - b;
+          case '*':
+            return a * b;
+          case '/':
+            return a / b;
+          case '%':
+            return a % b;
+          case '^':
+            return Math.pow(a, b);
         }
       };
 
       // BOTH UNIT
       if (isUnit(left) && isUnit(right)) {
-
         const from = this.findUnit(right.unit);
         const to = this.findUnit(left.unit);
 
-        if (from.type !== to.type) {
+        if (!from || !to || from.type !== to.type) {
           throw new Error(`Cannot operate on different unit types`);
         }
 
@@ -132,47 +154,47 @@ export function createUnitsStore(initial = {}) {
         const result = apply(left.value, r);
 
         // multiplication/division produce compound units
-        if (op === "*") {
+        if (op === '*') {
           return { value: result, unit: left.unit };
         }
 
-        if (op === "/") {
+        if (op === '/') {
           return { value: result, unit: left.unit };
         }
 
-        if (op === "^") {
+        if (op === '^') {
           return { value: result, unit: left.unit };
         }
 
         return { value: result, unit: left.unit };
       }
 
-      // ================= LEFT UNIT =================
+      // LEFT UNIT
       if (isUnit(left) && !isUnit(right)) {
         const result = apply(left.value, right);
 
         return { value: result, unit: left.unit };
       }
 
-      // ================= RIGHT UNIT =================
+      // RIGHT UNIT
       if (!isUnit(left) && isUnit(right)) {
         const result = apply(left, right.value);
 
-        if (op === "/") {
+        if (op === '/') {
           return { value: result, unit: right.unit };
         }
 
         return { value: result, unit: right.unit };
       }
 
-      // ================= NORMAL =================
+      // NORMAL
       return apply(left, right);
     },
-    // Convert
+
     convert,
 
     // Search helpers
     getAllUnitsFlat,
-    findUnit
+    findUnit,
   };
 }

package/src/variables/store.js

@@ -2,18 +2,19 @@ const validVarName = /^[a-zA-Z_$][a-zA-Z0-9_$]*$/;
 
 export function createVarStore(initial = {}) {
   let store = Object.create(null);
-  
 
   for (const key in initial) {
     store[key] = initial[key];
   }
 
   return {
+    /**
+     * @param {string} name
+     * @param {number | undefined} value
+     */
     set(name, value, { override = true } = {}) {
-
-      // Name validation
-      if (typeof name !== "string" || !name) {
-        throw new Error("Variable name must be a non-empty string");
+      if (typeof name !== 'string' || !name) {
+        throw new Error('Variable name must be a non-empty string');
       }
 
       if (!validVarName.test(name)) {
@@ -26,50 +27,52 @@ export function createVarStore(initial = {}) {
       }
 
       // Prevent overwrite (optional)
-      if (!override && name in variablesDB) {
+      if (!override && name in store) {
         throw new Error(`Variable '${name}' already exists`);
       }
 
       store[name] = value;
     },
 
-    //get variable
+    /**
+     * @param {string | number} name
+     */
     get(name) {
       return store[name];
     },
 
-    // check existence
+    /**
+     * @param {any} name
+     */
     has(name) {
       return Object.prototype.hasOwnProperty.call(store, name);
     },
 
-    // remove variable
+    /**
+     * @param {string | number} name
+     */
     remove(name) {
       delete store[name];
     },
 
-    // get all variables (snapshot)
     all() {
       return { ...store };
     },
 
-    // clear all
     clear() {
       store = Object.create(null);
     },
 
-    // merge multiple variables
     merge(obj = {}) {
       for (const key in obj) {
         store[key] = obj[key];
       }
     },
 
-    // clone store (for scoped instances) 
     clone() {
       return createVarStore(store);
-    }
+    },
   };
 }
 
-export default { createVarStore };
+export default { createVarStore };