@syncbridge/common 0.5.12 → 0.6.0

package/classes/base-element.js

@@ -12,8 +12,10 @@ export class BaseElement extends Runnable {
         this.logger = init.logger;
         this.name = init.name;
         this.values = init.values;
-        if (this.logger)
-            this.on('error', err => this.logger?.error(err));
+        this.on('error', err => this.logger?.error(err));
+        this.on('values-updated', (properties) => {
+            this.logger?.info(`Configuration changed. Updated properties: ${properties.join(',')}`);
+        });
     }
     get status() {
         return this._context.calculatedStatus || this._context.status;

package/constants.js

@@ -1,4 +1,4 @@
-export const version = '0.5.12';
+export const version = '0.6.0';
 export const OWN_ELEMENT_METADATA = Symbol.for('OWN_ELEMENT_METADATA');
 export const COMPONENT_OPTIONS = Symbol.for('COMPONENT_OPTIONS');
 export const PROCESSOR_OPTIONS = Symbol.for('PROCESSOR_OPTIONS');

package/index.d.ts

@@ -16,5 +16,5 @@ export * from './models-document.js';
 export * from './processor-factory.js';
 export * from './registry/extension-package.js';
 export * from './registry/extension-registry.js';
-export * from './utils/metadata-utils.js';
+export * from './utils/materialize-metadata.js';
 export * from './utils/profile-utils.js';

package/index.js

@@ -16,5 +16,5 @@ export * from './models-document.js';
 export * from './processor-factory.js';
 export * from './registry/extension-package.js';
 export * from './registry/extension-registry.js';
-export * from './utils/metadata-utils.js';
+export * from './utils/materialize-metadata.js';
 export * from './utils/profile-utils.js';

package/package.json

@@ -1,16 +1,17 @@
 {
   "name": "@syncbridge/common",
-  "version": "0.5.12",
+  "version": "0.6.0",
   "description": "SyncBridge Common utilities",
   "author": "Panates Inc",
   "license": "MIT",
   "dependencies": {
     "@jsopen/objects": "^2.1.1",
+    "fast-deep-equal": "^3.1.3",
     "node-events-async": "^1.5.0",
     "reflect-metadata": "^0.2.2",
     "semver": "^7.7.4",
     "ts-gems": "^3.11.3",
-    "valgen": "^5.19.4"
+    "valgen": "^5.19.5"
   },
   "peerDependencies": {
     "@opra/common": ">=1.17.1 <2"

package/processor-factory.js

@@ -1,7 +1,8 @@
+import equal from 'fast-deep-equal';
 import { StackExecutor } from './classes/stack-executor.js';
 import { initializeModelsDocument } from './models-document.js';
 import { ExtensionRegistry } from './registry/extension-registry.js';
-import { materializeMetadata } from './utils/metadata-utils.js';
+import { materializeMetadata } from './utils/materialize-metadata.js';
 export var ProcessorFactory;
 (function (ProcessorFactory) {
     ProcessorFactory.METADATA_KEY = Symbol('METADATA');
@@ -49,11 +50,12 @@ export var ProcessorFactory;
                     await signalUpdates(component);
                 }
             }
-            if (ctx.valueUpdates.has(obj)) {
-                await obj.emitAsyncSafe('values-updated', ctx.valueUpdates.get(obj));
+            const properties = ctx.valueUpdates.get(obj);
+            if (properties?.length) {
+                await obj.emitAsyncSafe('values-updated', properties);
             }
         };
-        signalUpdates(processor);
+        await signalUpdates(processor);
         return {
             issues: ctx.stackExecutor.issues,
         };
@@ -93,7 +95,6 @@ export var ProcessorFactory;
                         oldMetadata?.className &&
                         oldMetadata?.className !== newMetadata.className)
                         throw new Error('Can not change component class while processor running. You should stop it first.');
-                    ExtensionRegistry.getComponent(childMetadata.className);
                     /** Validate sub components */
                     if (childMetadata.components) {
                         _validateComponents({
@@ -125,14 +126,14 @@ export var ProcessorFactory;
         });
     }
     async function _configureComponent(ctx, key, childMetadata) {
-        const { newMetadata, newProfile, oldMetadata, oldProfile, curPath, owner, processor, } = ctx;
+        const { newProfile, oldMetadata, oldProfile, curPath, owner, processor } = ctx;
         const oldChildMetadata = oldMetadata?.components?.[key];
         const oldChildProfile = oldProfile?.components?.[key];
         const childProfile = newProfile.components[key];
         const componentPath = curPath ? curPath + '/' + key : key;
         let componentInstance = owner.components?.[key];
-        /** Create the component only if the processor stopped */
-        if (processor.stopped) {
+        /** Create the component only if the owner stopped */ // todo implement hotplug components
+        if (owner.stopped) {
             const compReg = ExtensionRegistry.getComponent(childMetadata.className);
             /** Create the component instance */
             const compClass = await compReg.getClass();
@@ -150,26 +151,30 @@ export var ProcessorFactory;
         }
         else {
             componentInstance.values = {};
-            const patchValues = (target, valPath, newValues, newVariables, oldVariables, oldValues) => {
-                Object.assign(target, newValues);
-                const updatedValues = [];
-                for (const [varKey, varMeta] of Object.entries(newMetadata)) {
-                    const keyPath = valPath ? valPath + '.' + varKey : varKey;
-                    /** Update only hotplug variables */
-                    if (varMeta?.hotPlug !== true)
-                        continue;
-                    if (varMeta.type === 'nested') {
-                        target[varKey] = {};
-                        patchValues(target[varKey], keyPath, newValues[varKey], newVariables[varKey]?.variables || {}, oldVariables?.[varKey]?.variables, oldValues?.[varKey]);
-                        continue;
+            const patchValues = (target, valPath, newValues, newVariables, oldValues, oldVariables) => {
+                if (oldValues)
+                    Object.assign(target, oldValues);
+                const updatedValues = ctx.valueUpdates.get(componentInstance) || [];
+                ctx.valueUpdates.set(componentInstance, updatedValues);
+                if (newVariables) {
+                    for (const [varKey, varMeta] of Object.entries(newVariables)) {
+                        const keyPath = valPath ? valPath + '.' + varKey : varKey;
+                        /** Update only hotplug variables */
+                        if (varMeta?.hotPlug !== true)
+                            continue;
+                        if (varMeta.type === 'nested') {
+                            target[varKey] = {};
+                            patchValues(target[varKey], keyPath, newValues[varKey], newVariables[varKey]?.variables || {}, oldValues?.[varKey], oldVariables?.[varKey]?.variables);
+                            continue;
+                        }
+                        if (!equal(target[varKey], newValues[varKey])) {
+                            updatedValues.push(keyPath);
+                            target[varKey] = newValues[varKey];
+                        }
                     }
-                    updatedValues.push(keyPath);
-                    target[varKey] = newValues[varKey];
                 }
-                if (updatedValues.length)
-                    ctx.valueUpdates.set(componentInstance, updatedValues);
             };
-            patchValues(componentInstance.values, '', childProfile.values || {}, childMetadata.variables || {}, oldChildMetadata?.variables, oldProfile?.values);
+            patchValues(componentInstance.values, '', childProfile.values || {}, childMetadata.variables || {}, oldChildProfile?.values, oldChildMetadata?.variables);
         }
         /** Create the child logger */
         componentInstance.logger = owner.logger?.child({