@tamyla/clodo-framework 4.1.0 → 4.2.0

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+# [4.2.0](https://github.com/tamylaa/clodo-framework/compare/v4.1.0...v4.2.0) (2026-01-31)
+
+
+### Features
+
+* **validation:** integrate ConfigurationValidator into ValidationHandler; add post-generation validation in ServiceOrchestrator; tests ([#5](https://github.com/tamylaa/clodo-framework/issues/5)) ([2889864](https://github.com/tamylaa/clodo-framework/commit/28898641bac4c2d6d206e12dbba8216e1de386a3))
+
 # [4.1.0](https://github.com/tamylaa/clodo-framework/compare/v4.0.15...v4.1.0) (2026-01-31)
 
 

package/dist/service-management/ServiceOrchestrator.js

@@ -131,15 +131,52 @@ export class ServiceOrchestrator {
         outputPath: options.outputDir || this.outputPath,
         middlewareStrategy: this.middlewareStrategy
       });
-      return {
-        success: true,
-        serviceId: generationResult.serviceId || null,
-        servicePath: generationResult.servicePath || null,
-        serviceName: generationResult.serviceName || coreInputs.serviceName,
-        fileCount: generationResult.fileCount || (generationResult.generatedFiles || []).length,
-        generatedFiles: generationResult.generatedFiles || [],
-        warnings: validation.warnings || []
-      };
+
+      // Post-generation validation using ValidationHandler
+      const validationHandler = options.customConfig ? new ValidationHandler({
+        customConfig: options.customConfig
+      }) : this.validationHandler;
+      try {
+        const postValidation = await validationHandler.validateService(generationResult.servicePath);
+        if (!postValidation.valid && !options.force) {
+          return {
+            success: false,
+            errors: postValidation.issues || [],
+            warnings: validation.warnings || [],
+            validationReport: postValidation
+          };
+        }
+        return {
+          success: true,
+          serviceId: generationResult.serviceId || null,
+          servicePath: generationResult.servicePath || null,
+          serviceName: generationResult.serviceName || coreInputs.serviceName,
+          fileCount: generationResult.fileCount || (generationResult.generatedFiles || []).length,
+          generatedFiles: generationResult.generatedFiles || [],
+          warnings: validation.warnings || [],
+          validationReport: postValidation
+        };
+      } catch (valErr) {
+        // If validation step throws, surface error unless forced
+        if (!options.force) {
+          return {
+            success: false,
+            errors: [valErr.message]
+          };
+        }
+        return {
+          success: true,
+          serviceId: generationResult.serviceId || null,
+          servicePath: generationResult.servicePath || null,
+          serviceName: generationResult.serviceName || coreInputs.serviceName,
+          fileCount: generationResult.fileCount || (generationResult.generatedFiles || []).length,
+          generatedFiles: generationResult.generatedFiles || [],
+          warnings: validation.warnings || [],
+          validationReport: {
+            error: valErr.message
+          }
+        };
+      }
     } catch (error) {
       return {
         success: false,

package/dist/service-management/handlers/ValidationHandler.js

@@ -6,8 +6,7 @@
 import fs from 'fs/promises';
 import path from 'path';
 import { FrameworkConfig } from '../../utils/framework-config.js';
-// import { ConfigurationValidator } from '../../lib/shared/utils/configuration-validator.js';
-
+import { ConfigurationValidator } from '../../security/ConfigurationValidator.js';
 export class ValidationHandler {
   constructor(options = {}) {
     this.strict = options.strict || false;
@@ -94,17 +93,33 @@ export class ValidationHandler {
     issues.push(...wranglerValidation.issues);
 
     // Run comprehensive configuration validation using ConfigurationValidator
-    // Temporarily disabled due to import issues
-    // try {
-    //   const configValidation = await ConfigurationValidator.validateServiceConfig(servicePath);
-    //   if (!configValidation.isValid) {
-    //     issues.push(...configValidation.errors);
-    //     issues.push(...configValidation.warnings.map(w => `Warning: ${w}`));
-    //   }
-    // } catch (error) {
-    //   issues.push(`Configuration validation failed: ${error.message}`);
-    // }
-
+    try {
+      // Determine manifest path candidates and select first that exists
+      const manifestCandidates = ['clodo-service-manifest.json', 'service-manifest.json', 'manifest.json'];
+      let manifestPath = null;
+      for (const candidate of manifestCandidates) {
+        const candidatePath = path.join(servicePath, candidate);
+        try {
+          await fs.access(candidatePath);
+          manifestPath = candidatePath;
+          break;
+        } catch {
+          // Not found, continue
+        }
+      }
+      const wranglerPath = path.join(servicePath, 'wrangler.toml');
+      if (manifestPath) {
+        const configValidation = ConfigurationValidator.validateServiceConfig(manifestPath, wranglerPath);
+        if (!configValidation.valid) {
+          issues.push(...(configValidation.issues || []).map(i => `Configuration mismatch: ${i.message || JSON.stringify(i)}`));
+        }
+      } else {
+        // No manifest found — warn but do not block validation
+        issues.push('Warning: No service manifest found (clodo-service-manifest.json) — skipping manifest↔wrangler validation');
+      }
+    } catch (error) {
+      issues.push(`Configuration validation step failed: ${error.message}`);
+    }
     return {
       valid: issues.length === 0,
       issues,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tamyla/clodo-framework",
-  "version": "4.1.0",
+  "version": "4.2.0",
   "description": "Reusable framework for Clodo-style software architecture on Cloudflare Workers + D1",
   "type": "module",
   "sideEffects": [