a2acalling 0.6.21 → 0.6.23

package/bin/cli.js

@@ -1126,6 +1126,40 @@ https://github.com/onthegonow/a2a_calling`;
     }
   },
 
+  config: (args) => {
+    const { A2AConfig } = require('../src/lib/config');
+    const config = new A2AConfig();
+    
+    const hostname = args.flags.hostname || args.flags.h;
+    const port = args.flags.port || args.flags.p;
+    const show = args.flags.show || args.flags.s || (!hostname && !port);
+    
+    if (show) {
+      const agent = config.getAgent();
+      console.log('A2A Configuration:\n');
+      console.log(`  Hostname: ${agent.hostname || '(not set)'}`);
+      console.log(`  Name: ${agent.name || '(not set)'}`);
+      console.log(`  Description: ${agent.description || '(not set)'}`);
+      const onboarding = config.getAll().onboarding || {};
+      console.log(`  Server port: ${onboarding.server_port || '(not running)'}`);
+      console.log(`  Onboarding step: ${onboarding.step || 'not started'}`);
+      return;
+    }
+    
+    const updates = {};
+    if (hostname) {
+      // Remove port from hostname if it's :80 (default)
+      const cleanHostname = hostname.replace(/:80$/, '');
+      updates.hostname = cleanHostname;
+      console.log(`  Hostname updated to: ${cleanHostname}`);
+    }
+    
+    if (Object.keys(updates).length > 0) {
+      config.setAgent(updates);
+      console.log('  ✅ Configuration saved.');
+    }
+  },
+
   server: (args) => {
     const explicitPort = args.flags.port || args.flags.p || process.env.PORT;
     if (explicitPort) {
@@ -1220,15 +1254,28 @@ https://github.com/onthegonow/a2a_calling`;
     }
 
     // ── Step 1: Port selection ───────────────────────────────────────────
-    // Scan ports 80, 3001-3020 and pick the first available one.
-    // Only show the selected port — agents don't need to see every candidate.
-    // Interactive users can override with a custom port; non-interactive
-    // auto-accepts the recommendation.
+    // Port 80 is strongly preferred (no firewall config needed for external access).
+    // If port 80 is available and bindable, use it. Otherwise fall back to 3001-3020.
     printSection('Port Configuration');
     const preferredPort = parsePort(args.flags.port || args.flags.p, null);
     const candidates = await inspectPorts(preferredPort);
     const availableCandidates = candidates.filter(c => c.available);
-    const recommendedPort = availableCandidates.length ? availableCandidates[0].port : null;
+    
+    // Strongly prefer port 80 if available
+    const port80Candidate = candidates.find(c => c.port === 80);
+    const port80Available = port80Candidate && port80Candidate.available;
+    
+    let recommendedPort;
+    if (port80Available) {
+      recommendedPort = 80;
+      console.log('  Port 80 is available — using it for easiest external access.');
+    } else if (availableCandidates.length) {
+      recommendedPort = availableCandidates[0].port;
+      console.log(`  Port 80 is in use. Using fallback port ${recommendedPort}.`);
+      console.log('  (Reverse proxy or firewall config will be needed for external access.)');
+    } else {
+      recommendedPort = null;
+    }
 
     if (!recommendedPort) {
       console.error('  Could not find a bindable port in the scan range.');
@@ -1240,9 +1287,14 @@ https://github.com/onthegonow/a2a_calling`;
       process.exit(1);
     }
 
-    console.log(`  Selected port: ${recommendedPort}`);
     let serverPort = recommendedPort;
-    const portChoice = await promptText(`Use port ${recommendedPort}? [Y/n/custom]: `, 'y');
+    
+    // If we got port 80, just confirm briefly. Otherwise allow override.
+    const portPrompt = port80Available 
+      ? `Use port 80? [Y/n]: `
+      : `Use port ${recommendedPort}? [Y/n/custom]: `;
+    const portChoice = await promptText(portPrompt, 'y');
+    
     if (!interactive) {
       serverPort = recommendedPort;
     } else if (!['', 'y', 'Y', 'yes', 'YES', 'ye'].includes(String(portChoice).trim())) {
@@ -1389,38 +1441,64 @@ https://github.com/onthegonow/a2a_calling`;
     console.log('  ✅ A2A server is running');
 
     if (externalIp) {
-      const verifyUrl = `http://${publicHost}/api/a2a/ping`;
-      if (serverPort !== 80) {
-        // Check what's using port 80
-        const port80Status = await isPortListening(80, '127.0.0.1', { timeoutMs: 250 });
+      if (serverPort === 80) {
+        // Port 80 — optimal setup, no extra config needed
+        console.log(`\n  ✅ Running on port 80 — external agents can reach you directly.`);
+        console.log(`  Invite hostname: ${externalIp}`);
+        // Update publicHost to not include port since 80 is default
+        publicHost = externalIp;
+      } else {
+        // Not on port 80 — need reverse proxy or firewall config
+        const { spawnSync } = require('child_process');
+        const hasNginx = spawnSync('which', ['nginx'], { encoding: 'utf8' }).status === 0;
+        const hasCaddy = spawnSync('which', ['caddy'], { encoding: 'utf8' }).status === 0;
         
-        console.log(`\n  ━━━ Reverse Proxy Setup ━━━`);
-        console.log(`  Server running on port ${serverPort}, but external callers expect port 80/443.`);
-        if (port80Status.listening) {
-          console.log(`  Port 80: in use (likely nginx, caddy, or another web server)`);
-        } else {
-          console.log(`  Port 80: available (but A2A is on ${serverPort})`);
+        console.log(`\n  ━━━ IMPORTANT: External Access Configuration ━━━`);
+        console.log(`\n  A2A server is on port ${serverPort}, but external callers expect port 80.`);
+        console.log(`  Port 80 is in use by ${hasNginx ? 'nginx' : hasCaddy ? 'Caddy' : 'another web server'}.`);
+        console.log(`\n  RECOMMENDED: Configure reverse proxy to route /api/a2a/* to port ${serverPort}`);
+        
+        if (hasNginx) {
+          console.log(`\n  ── nginx config (add to /etc/nginx/sites-available/default) ──`);
+          console.log(`  location /api/a2a/ {`);
+          console.log(`      proxy_pass http://127.0.0.1:${serverPort}/api/a2a/;`);
+          console.log(`      proxy_http_version 1.1;`);
+          console.log(`      proxy_set_header Host $host;`);
+          console.log(`      proxy_set_header X-Real-IP $remote_addr;`);
+          console.log(`  }`);
+          console.log(`  ────────────────────────────────────────────────────────────`);
+          console.log(`\n  To apply:`);
+          console.log(`    sudo nano /etc/nginx/sites-available/default`);
+          console.log(`    (add the location block inside your server {})`);
+          console.log(`    sudo nginx -t && sudo systemctl reload nginx`);
         }
-        console.log(`\n  RECOMMENDED: Configure your web server to proxy A2A requests:\n`);
-        console.log(`  For nginx, add to your server block:`);
-        console.log(`    location /api/a2a/ {`);
-        console.log(`      proxy_pass http://127.0.0.1:${serverPort}/api/a2a/;`);
-        console.log(`      proxy_http_version 1.1;`);
-        console.log(`      proxy_set_header Host $host;`);
-        console.log(`    }`);
-        console.log(`\n  For Caddy:`);
-        console.log(`    handle /api/a2a/* {`);
-        console.log(`      reverse_proxy 127.0.0.1:${serverPort}`);
-        console.log(`    }`);
-        console.log(`\n  After configuring, update the invite hostname to use port 80:`);
-        console.log(`    Current: ${publicHost}`);
-        console.log(`    Should be: ${externalIp} (without port, defaults to 80)`);
-        console.log(`\n  Or open port ${serverPort} directly in your firewall if no proxy is used.`);
-        console.log(`  ━━━━━━━━━━━━━━━━━━━━━━━━━━━`);
-        console.log(`\n  ACTION REQUIRED: Ask your owner if they want to configure a reverse proxy.`);
-        console.log(`  If they say yes, help them add the nginx/caddy config above.`);
-        console.log(`  If they say no or skip, the invite will use port ${serverPort} directly.`);
+        
+        if (hasCaddy) {
+          console.log(`\n  ── Caddy config ──`);
+          console.log(`  handle /api/a2a/* {`);
+          console.log(`      reverse_proxy 127.0.0.1:${serverPort}`);
+          console.log(`  }`);
+          console.log(`  ───────────────────`);
+        }
+        
+        console.log(`\n  After reverse proxy is configured:`);
+        console.log(`    Invite hostname will be: ${externalIp} (no port needed)`);
+        
+        console.log(`\n  ── Alternative: Direct firewall access ──`);
+        console.log(`  Open port ${serverPort} directly (less recommended):`);
+        console.log(`    sudo ufw allow ${serverPort}/tcp   # Ubuntu/Debian`);
+        console.log(`  Invite hostname will then be: ${publicHost}`);
+        
+        console.log(`\n  ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━`);
+        console.log(`\n  AGENT ACTION REQUIRED:`);
+        console.log(`  1. Configure reverse proxy (recommended) — add nginx/caddy config above`);
+        console.log(`  2. OR open firewall for port ${serverPort} — run the ufw/firewall-cmd command`);
+        console.log(`  3. OR skip (local-only, external agents cannot call)`);
+        console.log(`\n  Ask your owner which option to use, then proceed.`);
+        console.log(`  If reverse proxy is configured, update hostname with: a2a config --hostname ${externalIp}`);
       }
+      
+      const verifyUrl = `http://${publicHost}/api/a2a/ping`;
       console.log(`\n  Verify: curl -s ${verifyUrl}`);
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "a2acalling",
-  "version": "0.6.21",
+  "version": "0.6.23",
   "description": "Agent-to-agent calling for OpenClaw - A2A agent communication",
   "main": "src/index.js",
   "bin": {