@onlineapps/conn-orch-validator 2.0.11 → 2.0.13

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@onlineapps/conn-orch-validator",
-  "version": "2.0.11",
+  "version": "2.0.13",
   "description": "Validation orchestrator for OA Drive microservices - coordinates validation across all layers (base, infra, orch, business)",
   "main": "src/index.js",
   "scripts": {

package/src/ValidationOrchestrator.js

@@ -278,7 +278,7 @@ class ValidationOrchestrator {
       } else {
         const config = JSON.parse(fs.readFileSync(configFile, 'utf8'));
         if (!config.service?.name) errors.push('config.json missing service.name');
-        if (!config.service?.version) errors.push('config.json missing service.version');
+        // Note: service.version is now read from package.json (Single Source of Truth)
       }
 
       // Validate operations.json
@@ -478,7 +478,7 @@ class ValidationOrchestrator {
       }
     } else {
       console.error(`[ValidationOrchestrator] ❌ Validation FAILED (${duration}ms)`);
-      console.error(`[ValidationOrchestrator] Errors: ${JSON.stringify(results.errors, null, 2)}`);
+      console.error(`[ValidationOrchestrator] Errors: ${JSON.stringify(results.errors, null, 2)}(', ')}`);
     }
 
     return results;

package/src/validators/ServiceStructureValidator.js

@@ -159,12 +159,12 @@ class ServiceStructureValidator {
 
   /**
    * Validate config.json structure
+   * Note: service.version is now read from package.json (Single Source of Truth)
    */
   validateConfigStructure(config) {
-    // Required fields
+    // Required fields (version comes from package.json, not config.json)
     const requiredFields = [
       'service.name',
-      'service.version',
       'service.port'
     ];
 
@@ -200,18 +200,23 @@ class ServiceStructureValidator {
       }
     }
 
-    // Validate version format
-    if (config.service?.version) {
-      if (!/^\d+\.\d+\.\d+$/.test(config.service.version)) {
+    // Validate version format from package.json (Single Source of Truth)
+    const packagePath = path.join(this.serviceRoot, 'package.json');
+    try {
+      const pkg = JSON.parse(fs.readFileSync(packagePath, 'utf-8'));
+      const version = pkg.version;
+      if (version && !/^\d+\.\d+\.\d+$/.test(version)) {
         this.warnings.push({
           type: 'INVALID_VERSION',
-          path: 'conn-config/config.json',
-          field: 'service.version',
-          value: config.service.version,
-          message: `Version should follow semantic versioning: ${config.service.version}`,
+          path: 'package.json',
+          field: 'version',
+          value: version,
+          message: `Version should follow semantic versioning: ${version}`,
           fix: 'Use format: MAJOR.MINOR.PATCH (e.g., "1.0.0")'
         });
       }
+    } catch (e) {
+      // package.json validation is done separately
     }
   }