dank-ai 1.0.37 → 1.0.38

package/lib/agent.js

@@ -16,8 +16,8 @@ class DankAgent {
     this.name = name;
     this.config = this._validateConfig(config);
     this.handlers = new Map();
-    // id is required and must be set via setId() - validated in _validateConfig
-    this.id = this.config.id;
+    // id is optional during construction - must be set via setId() before use
+    this.id = this.config.id || null;
     this.status = 'defined'; // defined, building, running, stopped, error
     this.containerId = null;
     this.createdAt = new Date().toISOString();
@@ -564,11 +564,39 @@ class DankAgent {
     return this;
   }
 
+  /**
+   * Validate that agent ID is set (required for all operations)
+   * @private
+   */
+  _validateId() {
+    if (!this.id || !this.config.id) {
+      throw new Error(
+        `Agent ID is required for agent "${this.name}". ` +
+        `Use .setId(uuidv4) to set a unique UUIDv4 identifier. ` +
+        `Example: createAgent('${this.name}').setId(require('uuid').v4())`
+      );
+    }
+    
+    // Validate UUIDv4 format
+    const trimmedId = this.id.trim();
+    if (!validateUUID(trimmedId)) {
+      throw new Error(`Agent ID must be a valid UUIDv4 for agent "${this.name}". Received: ${trimmedId}`);
+    }
+    
+    const uuidRegex = /^[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/i;
+    if (!uuidRegex.test(trimmedId)) {
+      throw new Error(`Agent ID must be a valid UUIDv4 for agent "${this.name}". Received: ${trimmedId}`);
+    }
+  }
+
   /**
    * Finalize agent configuration by auto-detecting features
    * This should be called before the agent is deployed
    */
   finalize() {
+    // Validate that ID is set before finalization (REQUIRED)
+    this._validateId();
+    
     this._autoDetectFeatures();
     return this;
   }
@@ -607,6 +635,9 @@ class DankAgent {
    * Get the complete agent configuration for serialization
    */
   toConfig() {
+    // Validate ID before serialization
+    this._validateId();
+    
     return {
       name: this.name,
       id: this.id,
@@ -640,28 +671,27 @@ class DankAgent {
    * Validate agent configuration
    */
   _validateConfig(config) {
-    // Validate ID separately (must be UUIDv4)
-    if (!config.id) {
-      throw new Error('Agent ID is required. Use .setId(uuidv4) to set a unique UUIDv4 identifier.');
-    }
-    
-    if (typeof config.id !== 'string') {
-      throw new Error('Agent ID must be a string');
-    }
-    
-    const trimmedId = config.id.trim();
-    if (!validateUUID(trimmedId)) {
-      throw new Error(`Agent ID must be a valid UUIDv4. Received: ${trimmedId}`);
-    }
-    
-    // Check if it's actually v4 (UUIDv4 has '4' in the version position)
-    const uuidRegex = /^[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/i;
-    if (!uuidRegex.test(trimmedId)) {
-      throw new Error(`Agent ID must be a valid UUIDv4. Received: ${trimmedId}`);
+    // ID is optional during construction - will be validated when setId() is called
+    // or when the agent is finalized/used
+    if (config.id) {
+      if (typeof config.id !== 'string') {
+        throw new Error('Agent ID must be a string');
+      }
+      
+      const trimmedId = config.id.trim();
+      if (!validateUUID(trimmedId)) {
+        throw new Error(`Agent ID must be a valid UUIDv4. Received: ${trimmedId}`);
+      }
+      
+      // Check if it's actually v4 (UUIDv4 has '4' in the version position)
+      const uuidRegex = /^[0-9a-f]{8}-[0-9a-f]{4}-4[0-9a-f]{3}-[89ab][0-9a-f]{3}-[0-9a-f]{12}$/i;
+      if (!uuidRegex.test(trimmedId)) {
+        throw new Error(`Agent ID must be a valid UUIDv4. Received: ${trimmedId}`);
+      }
     }
     
     const schema = Joi.object({
-      id: Joi.string().required(), // Required UUIDv4 - validated above
+      id: Joi.string().optional(), // Optional during construction, required via setId()
       
       llm: Joi.object({
         provider: Joi.string().valid(...SUPPORTED_LLMS).required(),

package/lib/cli/production-build.js

@@ -12,6 +12,15 @@ const analytics = require('../analytics');
  * Extract deployment metadata from an agent configuration
  */
 function extractAgentMetadata(agent, buildOptions, imageName) {
+  // Validate that agent ID is set (required)
+  if (!agent.id || !agent.config?.id) {
+    throw new Error(
+      `Agent ID is required for agent "${agent.name}". ` +
+      `Use .setId(uuidv4) to set a unique UUIDv4 identifier. ` +
+      `Example: createAgent('${agent.name}').setId(require('uuid').v4())`
+    );
+  }
+  
   const config = agent.config || {};
   const dockerConfig = config.docker || {};
   const communication = config.communication || {};

package/lib/config.js

@@ -120,6 +120,15 @@ class AgentConfig {
    * Generate environment variables for agent container
    */
   static generateContainerEnv(agent) {
+    // Validate that agent ID is set (required)
+    if (!agent.id || !agent.config?.id) {
+      throw new Error(
+        `Agent ID is required for agent "${agent.name}". ` +
+        `Use .setId(uuidv4) to set a unique UUIDv4 identifier. ` +
+        `Example: createAgent('${agent.name}').setId(require('uuid').v4())`
+      );
+    }
+    
     const env = {
       AGENT_NAME: agent.name,
       AGENT_ID: agent.id,

package/lib/docker/manager.js

@@ -1399,9 +1399,20 @@ class DockerManager {
     });
 
     // Finalize agent configuration (auto-detect features)
+    // This will validate that agent.id is set (required)
     agent.finalize();
 
     const imageName = `dank-agent-${agent.name.toLowerCase()}`;
+    
+    // Ensure agent.id is set (should be validated by finalize(), but double-check for safety)
+    if (!agent.id) {
+      throw new Error(
+        `Agent ID is required for agent "${agent.name}". ` +
+        `Use .setId(uuidv4) to set a unique UUIDv4 identifier. ` +
+        `Example: createAgent('${agent.name}').setId(require('uuid').v4())`
+      );
+    }
+    
     const containerName = `dank-${agent.name.toLowerCase()}-${agent.id
       .split("_")
       .pop()}`;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "dank-ai",
-  "version": "1.0.37",
+  "version": "1.0.38",
   "description": "Dank Agent Service - Docker-based AI agent orchestration platform",
   "main": "lib/index.js",
   "exports": {