@aifabrix/builder 2.0.7 → 2.0.8

package/lib/app-run.js

@@ -281,11 +281,11 @@ async function startContainer(appName, composePath, port, config = null) {
   await execAsync(`docker-compose -f "${composePath}" up -d`, { env });
   logger.log(chalk.green(`✓ Container aifabrix-${appName} started`));
 
-  // Wait for health check - detect actual mapped port from Docker
-  // Don't pass port so waitForHealthCheck will auto-detect it from container
+  // Wait for health check using host port
+  // Port is the host port (CLI --port or config.port, NOT localPort)
   const healthCheckPath = config?.healthCheck?.path || '/health';
   logger.log(chalk.blue(`Waiting for application to be healthy at http://localhost:${port}${healthCheckPath}...`));
-  await waitForHealthCheck(appName, 90, null, config);
+  await waitForHealthCheck(appName, 90, port, config);
 }
 
 /**
@@ -343,7 +343,8 @@ async function runApp(appName, options = {}) {
     }
 
     // Check port availability
-    const port = options.port || config.build?.localPort || config.port || 3000;
+    // Host port: CLI --port if provided, otherwise port from variables.yaml (NOT localPort)
+    const port = options.port || config.port || 3000;
     const portAvailable = await checkPortAvailable(port);
     if (!portAvailable) {
       throw new Error(`Port ${port} is already in use. Try --port <alternative>`);

package/lib/secrets.js

@@ -257,6 +257,22 @@ async function generateEnvFile(appName, secretsPath, environment = 'local', forc
   const resolved = await resolveKvReferences(template, secrets, environment, resolvedSecretsPath);
 
   fs.writeFileSync(envPath, resolved, { mode: 0o600 });
+
+  // Update PORT variable in container .env file to use port (from variables.yaml)
+  // Note: containerPort is ONLY used for Docker Compose port mapping, NOT for PORT env variable
+  // The application inside container listens on PORT env variable, which should be 'port' from variables.yaml
+  if (fs.existsSync(variablesPath)) {
+    const variablesContent = fs.readFileSync(variablesPath, 'utf8');
+    const variables = yaml.load(variablesContent);
+    const port = variables.port || 3000;
+
+    // Update PORT in container .env file to use port (NOT containerPort, NOT localPort)
+    let envContent = fs.readFileSync(envPath, 'utf8');
+    envContent = envContent.replace(/^PORT\s*=\s*.*$/m, `PORT=${port}`);
+    fs.writeFileSync(envPath, envContent, { mode: 0o600 });
+  }
+
+  // Process and copy to envOutputPath if configured (uses localPort for copied file)
   processEnvVariables(envPath, variablesPath);
 
   return envPath;

package/lib/utils/compose-generator.js

@@ -110,10 +110,9 @@ function buildServiceConfig(appName, config, port) {
   // Container port: build.containerPort > config.port
   const containerPortValue = config.build?.containerPort || config.port || port;
 
-  // Host port: use options.port if provided (CLI --port), otherwise use config.port
-  // Container port: use containerPort if exists, otherwise port
+  // Host port: use port parameter (already calculated from CLI --port or config.port in generateDockerCompose)
   // Note: build.localPort is ONLY used for .env file PORT variable (for local PC dev), NOT for Docker Compose
-  const hostPort = port !== config.port ? port : (config.port || port);
+  const hostPort = port;
 
   return {
     app: buildAppConfig(appName, config),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aifabrix/builder",
-  "version": "2.0.7",
+  "version": "2.0.8",
   "description": "AI Fabrix Local Fabric & Deployment SDK",
   "main": "lib/index.js",
   "bin": {