ante-erp-cli 1.6.6 → 1.6.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ante-erp-cli",
-  "version": "1.6.6",
+  "version": "1.6.7",
   "description": "Comprehensive CLI tool for managing ANTE ERP self-hosted installations",
   "type": "module",
   "bin": {

package/src/commands/install.js

@@ -557,7 +557,8 @@ Support: support@ante.ph
           const composeFile = join(config.installDir, 'docker-compose.yml');
 
           try {
-            // Run seed command in the backend container
+            // Run production seed command in the backend container
+            // Uses seed:prod which compiles TypeScript and runs with node (production-safe)
             await execa('docker', [
               'compose',
               '-f', composeFile,
@@ -566,7 +567,7 @@ Support: support@ante.ph
               'backend',
               'npm',
               'run',
-              'seed'
+              'seed:prod'
             ]);
           } catch (error) {
             // Seeding is optional, just log warning

package/src/commands/set-domain.js

@@ -9,7 +9,7 @@ import { configureNginx, requiresNginx } from '../utils/nginx.js';
 import { pullImages, stopServices, startServices, waitForServiceHealthy } from '../utils/docker.js';
 
 /**
- * Validate URL format
+ * Validate URL format with proper hostname and protocol checking
  * @param {string} url - URL to validate
  * @returns {boolean|string} True if valid, error message if invalid
  */
@@ -19,16 +19,49 @@ function validateUrl(url) {
   }
 
   try {
-    // Allow http://IP:PORT format
-    if (url.match(/^https?:\/\/.+/)) {
-      return true;
+    const parsed = new URL(url);
+
+    // Check protocol
+    if (parsed.protocol !== 'http:' && parsed.protocol !== 'https:') {
+      return 'URL must use http:// or https:// protocol';
     }
-    return 'URL must start with http:// or https://';
-  } catch {
-    return 'Invalid URL format';
+
+    // Check hostname exists and is not empty
+    if (!parsed.hostname || parsed.hostname.trim() === '') {
+      return 'URL must have a valid hostname (e.g., staging-api.ante.ph or 143.198.91.153)';
+    }
+
+    // Check for common mistakes (triple slashes)
+    if (url.includes('///') && !url.match(/^https?:\/\/[^/]/)) {
+      return 'URL has invalid format - check for extra slashes after protocol';
+    }
+
+    return true;
+  } catch (error) {
+    return `Invalid URL format: ${error.message}`;
   }
 }
 
+/**
+ * Sanitize URL by normalizing common formatting issues
+ * @param {string} url - URL to sanitize
+ * @returns {string} Sanitized URL
+ */
+function sanitizeUrl(url) {
+  if (!url) return url;
+
+  // Normalize protocol slashes (https:/// -> https://)
+  url = url.replace(/^(https?):\/\/+/, '$1://');
+
+  // Remove trailing slashes from the URL
+  url = url.replace(/\/+$/, '');
+
+  // Remove any accidental leading slashes before hostname
+  url = url.replace(/^(https?:\/\/)\/+/, '$1');
+
+  return url;
+}
+
 /**
  * Read current value from .env file
  * @param {string} envPath - Path to .env file
@@ -189,21 +222,22 @@ export async function setDomain(options) {
           {
             type: 'input',
             name: 'frontendUrl',
-            message: 'Frontend domain/URL (e.g., https://app.example.com or http://139.59.110.232:8080):',
+            message: 'Frontend URL:\n  Examples: https://staging.ante.ph or http://143.198.91.153:8080\n  Enter URL',
             default: currentFrontendUrl || 'http://localhost:8080',
             validate: validateUrl
           },
           {
             type: 'input',
             name: 'apiUrl',
-            message: 'API domain/URL (e.g., https://api.example.com or http://139.59.110.232:3001):',
+            message: 'Backend API URL:\n  Examples: https://staging-api.ante.ph or http://143.198.91.153:3001\n  Enter URL',
             default: currentApiUrl || 'http://localhost:3001',
             validate: validateUrl
           }
         ]);
 
-        frontendUrl = answers.frontendUrl;
-        apiUrl = answers.apiUrl;
+        // Sanitize URLs to fix common formatting issues
+        frontendUrl = sanitizeUrl(answers.frontendUrl);
+        apiUrl = sanitizeUrl(answers.apiUrl);
       }
     } else {
       // Non-interactive mode
@@ -215,6 +249,10 @@ export async function setDomain(options) {
         process.exit(1);
       }
 
+      // Sanitize URLs to fix common formatting issues
+      frontendUrl = sanitizeUrl(frontendUrl);
+      apiUrl = sanitizeUrl(apiUrl);
+
       const frontendValidation = validateUrl(frontendUrl);
       const apiValidation = validateUrl(apiUrl);