@vidyano-labs/virtual-service 0.4.2 → 0.5.0

package/README.md

@@ -144,7 +144,7 @@ Common attribute types supported by the virtual service:
 | `Date` | Dates only | `"2026-01-23"` |
 | `Byte` | Small integers (0-255) | `128` |
 
-> **Note:** Values are stored as strings in DTOs but converted to JavaScript types when accessed through `getValue()`. Boolean attributes accept both native booleans and string values `"True"`/`"False"`.
+> **Note:** Use native JavaScript types (boolean, number, Date, etc.) when setting attribute values. The `getValue<T>()` and `setValue<T>()` methods work directly with these types and support generics for type safety (e.g., `getValue<number>()`).
 
 ### Attribute Configuration
 
@@ -1142,8 +1142,8 @@ async onSave(obj: VirtualPersistentObject): Promise<VirtualPersistentObject> {
 | Method | Description |
 |--------|-------------|
 | `getAttribute(name)` | Get attribute by name |
-| `getAttributeValue(name)` | Get converted attribute value |
-| `setAttributeValue(name, value)` | Set attribute value with conversion |
+| `getAttributeValue<T>(name)` | Get attribute value |
+| `setAttributeValue<T>(name, value)` | Set attribute value |
 | `setValidationError(name, error)` | Set validation error (pass `null`/empty to clear) |
 | `setNotification(msg, type, duration?)` | Set notification |
 | `service` | Reference to the VirtualService instance |
@@ -1152,8 +1152,8 @@ async onSave(obj: VirtualPersistentObject): Promise<VirtualPersistentObject> {
 
 | Method | Description |
 |--------|-------------|
-| `getValue()` | Get converted value |
-| `setValue(value)` | Set value with conversion |
+| `getValue<T>()` | Get value |
+| `setValue<T>(value)` | Set value |
 | `setValidationError(error)` | Set validation error (pass `null`/empty to clear) |
 | `persistentObject` | Reference to the parent VirtualPersistentObject |
 | `service` | Reference to the VirtualService instance |

package/index.d.ts

@@ -93,13 +93,13 @@ type VirtualPersistentObjectAttributeWithReference = Dto.PersistentObjectAttribu
  */
 type VirtualPersistentObjectAttributeHelpers = {
     /**
-     * Gets the converted value of this attribute (e.g., Boolean as boolean, Int32 as number)
+     * Gets the value of this attribute
      */
-    getValue(): any;
+    getValue<T = any>(): T;
     /**
-     * Sets the value of this attribute with automatic type conversion
+     * Sets the value of this attribute
      */
-    setValue(value: any): void;
+    setValue<T = any>(value: T): void;
     /**
      * Sets a validation error on this attribute. Pass null/empty to clear.
      */
@@ -125,11 +125,11 @@ type VirtualPersistentObject = Omit<Dto.PersistentObjectDto, "queries"> & {
     /**
      * Gets the value of an attribute by name
      */
-    getAttributeValue(name: string): any;
+    getAttributeValue<T = any>(name: string): T;
     /**
      * Sets the value of an attribute by name
      */
-    setAttributeValue(name: string, value: any): void;
+    setAttributeValue<T = any>(name: string, value: T): void;
     /**
      * Sets a notification message on the persistent object
      */
@@ -735,14 +735,10 @@ declare class VirtualService extends Service {
     static set messages(value: Record<string, string>);
     /**
      * Converts a service string value to a primitive JavaScript type.
-     * Unlike DataType.fromServiceString, this returns number instead of BigNumber
+     * Unlike Service.fromServiceString, this returns number instead of BigNumber
      * for numeric types (Decimal, Double, Int64, etc.).
      */
-    static fromServiceValue(value: any, type: string): any;
-    /**
-     * Converts a primitive JavaScript value to a service string.
-     */
-    static toServiceValue(value: any, type: string): string;
+    static fromServiceString(value: string, typeName: string): any;
     /**
      * Gets a message by key with optional parameters.
      * Resolution: static messages → return key unchanged

package/index.js

@@ -1,27 +1,20 @@
 import { DataType, ServiceHooks, Service } from '@vidyano/core';
 
 /**
- * Value conversion utilities for the virtual service.
- * Wraps core DataType conversions to return JavaScript primitives instead of BigNumber.
+ * Internal value conversion utilities for the virtual service.
  *
- * @example
- * // Convert from service string to primitive
- * fromServiceValue("100.50", "Decimal")    // => 100.5 (number)
- * fromServiceValue("True", "Boolean")      // => true (boolean)
- * fromServiceValue("15-01-2024 10:30:00", "DateTime")  // => Date object
+ * NOTE: This file exists to avoid circular dependencies. Files like
+ * virtual-persistent-object.ts cannot import VirtualService directly
+ * (it would create a cycle), so they import these helpers instead.
  *
- * @example
- * // Convert from primitive to service string
- * toServiceValue(100.5, "Decimal")         // => "100.5"
- * toServiceValue(true, "Boolean")          // => "True"
- * toServiceValue(new Date(2024, 0, 15), "Date")  // => "15-01-2024 00:00:00"
+ * Public API: Use VirtualService.fromServiceString() and VirtualService.toServiceString().
  */
 /**
  * Converts a service string value to a primitive JavaScript type.
  * Unlike DataType.fromServiceString, this returns number instead of BigNumber
  * for numeric types (Decimal, Double, Int64, etc.).
  */
-function fromServiceValue(value, type) {
+function fromServiceString(value, type) {
     const result = DataType.fromServiceString(value, type);
     // Check for BigNumber (has toNumber method) and convert to number primitive
     if (result && typeof result.toNumber === "function")
@@ -31,7 +24,7 @@ function fromServiceValue(value, type) {
 /**
  * Converts a primitive JavaScript value to a service string.
  */
-function toServiceValue(value, type) {
+function toServiceString(value, type) {
     return DataType.toServiceString(value, type);
 }
 
@@ -389,7 +382,7 @@ class VirtualQueryRegistry {
             const rawValue = data[column.name];
             return {
                 key: column.name,
-                value: rawValue == null ? null : toServiceValue(rawValue, column.type),
+                value: rawValue == null ? null : toServiceString(rawValue, column.type),
                 objectId: data[column.name + "Id"],
                 typeHints: data[column.name + "$typeHints"]
             };
@@ -478,8 +471,9 @@ function createVirtualQuery(dto, config, service) {
     return proxy;
 }
 /**
- * Unwraps a VirtualQuery to get the underlying DTO
- * @param wrapped - The VirtualQuery to unwrap
+ * Unwraps a VirtualQuery to get the underlying DTO.
+ * Also accepts raw DTOs for recursive handling.
+ * @param wrapped - The VirtualQuery or QueryDto to unwrap
  * @returns The underlying QueryDto
  */
 function unwrapVirtualQuery(wrapped) {
@@ -528,13 +522,14 @@ function createVirtualQueryResultItem(dto, query) {
  * @returns A VirtualPersistentObjectAttribute that combines DTO properties with helper methods
  */
 function createVirtualPersistentObjectAttribute(attr, persistentObject, service) {
+    const internalAttr = attr;
     const helpers = {
         getValue() {
-            return fromServiceValue(attr.value, attr.type);
+            return internalAttr.value;
         },
         setValue(value) {
-            attr.value = toServiceValue(value, attr.type);
-            attr.isValueChanged = true;
+            internalAttr.value = value;
+            internalAttr.isValueChanged = true;
         },
         setValidationError(error) {
             attr.validationError = error || undefined;
@@ -546,7 +541,7 @@ function createVirtualPersistentObjectAttribute(attr, persistentObject, service)
             return service;
         }
     };
-    return new Proxy(attr, {
+    return new Proxy(internalAttr, {
         get(target, prop) {
             if (prop in helpers) {
                 const value = helpers[prop];
@@ -554,8 +549,19 @@ function createVirtualPersistentObjectAttribute(attr, persistentObject, service)
             }
             return target[prop];
         },
-        set(target, prop, value) {
-            target[prop] = value;
+        set(target, prop, newValue) {
+            if (prop === "value") {
+                const oldValue = target.value;
+                const hasChanged = oldValue instanceof Date && newValue instanceof Date
+                    ? oldValue.getTime() !== newValue.getTime()
+                    : oldValue !== newValue;
+                if (hasChanged) {
+                    target.value = newValue;
+                    target.isValueChanged = true;
+                }
+            }
+            else
+                target[prop] = newValue;
             return true;
         }
     });
@@ -623,15 +629,21 @@ function createVirtualPersistentObject(dto, service) {
     return proxy;
 }
 /**
- * Unwraps a VirtualPersistentObject to get the underlying DTO
- * Since the Proxy wraps the DTO, we can safely cast it back
- * @param wrapped - The VirtualPersistentObject to unwrap
- * @returns The underlying PersistentObjectDto
+ * Unwraps a VirtualPersistentObject to get a DTO suitable for wire transmission.
+ * Converts primitive values to service string format.
+ * Also accepts raw DTOs for recursive handling of parent/nested objects.
  */
 function unwrapVirtualPersistentObject(wrapped) {
-    // The wrapped object is a Proxy around the DTO
-    // We can return it as-is since the DTO is the target of the Proxy
-    return wrapped;
+    const dto = wrapped;
+    return {
+        ...dto,
+        attributes: dto.attributes?.map(attr => ({
+            ...attr,
+            value: attr.value != null ? toServiceString(attr.value, attr.type) : attr.value
+        })),
+        parent: dto.parent ? unwrapVirtualPersistentObject(dto.parent) : undefined,
+        queries: dto.queries?.map(q => unwrapVirtualQuery(q))
+    };
 }
 
 var _a;
@@ -858,6 +870,9 @@ class VirtualServiceHooks extends ServiceHooks {
         clientAttr.tab = configAttr.tab || "";
         clientAttr.column = configAttr.column;
         clientAttr.columnSpan = configAttr.columnSpan;
+        // Convert incoming service string to primitive
+        if (clientAttr.value != null)
+            clientAttr.value = fromServiceString(clientAttr.value, clientAttr.type);
         // Reference attribute properties - only set if explicitly configured
         if (configAttr.lookup) {
             const refAttr = clientAttr;
@@ -1703,17 +1718,11 @@ class VirtualService extends Service {
     }
     /**
      * Converts a service string value to a primitive JavaScript type.
-     * Unlike DataType.fromServiceString, this returns number instead of BigNumber
+     * Unlike Service.fromServiceString, this returns number instead of BigNumber
      * for numeric types (Decimal, Double, Int64, etc.).
      */
-    static fromServiceValue(value, type) {
-        return fromServiceValue(value, type);
-    }
-    /**
-     * Converts a primitive JavaScript value to a service string.
-     */
-    static toServiceValue(value, type) {
-        return toServiceValue(value, type);
+    static fromServiceString(value, typeName) {
+        return fromServiceString(value, typeName);
     }
     /**
      * Gets a message by key with optional parameters.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vidyano-labs/virtual-service",
-  "version": "0.4.2",
+  "version": "0.5.0",
   "description": "Virtual service implementation for testing Vidyano applications",
   "type": "module",
   "main": "index.js",
@@ -20,5 +20,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHash": "79e3626e489dad1001f311fcec3fb18e06b0a85a"
+  "gitHash": "3d37759c3014945a12137766692d4501d0eb465a"
 }