From 8ae21ce787ffc3d0fc56b282c17e347320ff1ba0 Mon Sep 17 00:00:00 2001
From: Rene De Ren <zn375ix3@gmail.com>
Date: Wed, 11 Mar 2026 14:59:35 +0100
Subject: [PATCH] Migrate _loadConfig to use ConfigManager.buildConfig()

Replaces manual base config construction with shared buildConfig() method.
Node now only specifies domain-specific config sections.

Part of #1: Extract base config schema

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
---
 src/nodeClass.js | 18 +++---------------
 1 file changed, 3 insertions(+), 15 deletions(-)

diff --git a/src/nodeClass.js b/src/nodeClass.js
index 21156b2..92536fd 100644
--- a/src/nodeClass.js
+++ b/src/nodeClass.js
@@ -1,4 +1,5 @@
 const { Settler } = require('./specificClass.js');
+const { configManager } = require('generalFunctions');
 
 
 class nodeClass {
@@ -54,21 +55,8 @@ class nodeClass {
    * @param {object} uiConfig Config set in UI in node-red
    */
   _loadConfig(uiConfig) {
-    this.config = {
-      general: {
-        name: uiConfig.name || this.name,
-        id: this.node.id,
-        unit: null,
-        logging: {
-          enabled: uiConfig.enableLog,
-          logLevel: uiConfig.logLevel
-        }
-      },
-      functionality: {
-        positionVsParent: uiConfig.positionVsParent || 'atEquipment', // Default to 'atEquipment' if not specified
-        softwareType: "settler" // should be set in config manager
-      }
-    }
+    const cfgMgr = new configManager();
+    this.config = cfgMgr.buildConfig('settler', uiConfig, this.node.id);
   }
 
   /**