test: add parseAgentTimeout tests to fix coverage regression

Extract inline timeout validation into exported parseAgentTimeout()
function and add 7 unit tests covering valid integers, zero, negative,
non-numeric, empty string, and large values.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: Mossaka

src/cli.test.ts

@@ -1,5 +1,5 @@
 import { Command } from 'commander';
-import { parseEnvironmentVariables, parseDomains, parseDomainsFile, escapeShellArg, joinShellArgs, parseVolumeMounts, isValidIPv4, isValidIPv6, parseDnsServers, validateAgentImage, isAgentImagePreset, AGENT_IMAGE_PRESETS, processAgentImageOption, processLocalhostKeyword, validateSkipPullWithBuildLocal, validateAllowHostPorts, validateFormat, validateApiProxyConfig, buildRateLimitConfig, validateRateLimitFlags } from './cli';
+import { parseEnvironmentVariables, parseDomains, parseDomainsFile, escapeShellArg, joinShellArgs, parseVolumeMounts, isValidIPv4, isValidIPv6, parseDnsServers, validateAgentImage, isAgentImagePreset, AGENT_IMAGE_PRESETS, processAgentImageOption, processLocalhostKeyword, validateSkipPullWithBuildLocal, validateAllowHostPorts, validateFormat, validateApiProxyConfig, buildRateLimitConfig, validateRateLimitFlags, parseAgentTimeout } from './cli';
 import { redactSecrets } from './redact-secrets';
 import * as fs from 'fs';
 import * as path from 'path';
@@ -1539,4 +1539,41 @@ describe('cli', () => {
       expect(result.error).toBeUndefined();
     });
   });
+
+  describe('parseAgentTimeout', () => {
+    it('should parse a valid positive integer', () => {
+      const result = parseAgentTimeout('30');
+      expect(result).toEqual({ minutes: 30 });
+    });
+
+    it('should parse single minute timeout', () => {
+      const result = parseAgentTimeout('1');
+      expect(result).toEqual({ minutes: 1 });
+    });
+
+    it('should return error for zero', () => {
+      const result = parseAgentTimeout('0');
+      expect(result).toEqual({ error: '--agent-timeout must be a positive integer (minutes)' });
+    });
+
+    it('should return error for negative value', () => {
+      const result = parseAgentTimeout('-5');
+      expect(result).toEqual({ error: '--agent-timeout must be a positive integer (minutes)' });
+    });
+
+    it('should return error for non-numeric string', () => {
+      const result = parseAgentTimeout('abc');
+      expect(result).toEqual({ error: '--agent-timeout must be a positive integer (minutes)' });
+    });
+
+    it('should return error for empty string', () => {
+      const result = parseAgentTimeout('');
+      expect(result).toEqual({ error: '--agent-timeout must be a positive integer (minutes)' });
+    });
+
+    it('should parse large timeout values', () => {
+      const result = parseAgentTimeout('1440');
+      expect(result).toEqual({ minutes: 1440 });
+    });
+  });
 });

src/cli.ts

@@ -422,6 +422,19 @@ export function validateAllowHostPorts(
   return { valid: true };
 }
 
+/**
+ * Parses and validates the --agent-timeout option
+ * @param value - The raw string value from the CLI option
+ * @returns The parsed timeout in minutes, or an error
+ */
+export function parseAgentTimeout(value: string): { minutes: number } | { error: string } {
+  const timeoutMinutes = parseInt(value, 10);
+  if (isNaN(timeoutMinutes) || timeoutMinutes <= 0) {
+    return { error: '--agent-timeout must be a positive integer (minutes)' };
+  }
+  return { minutes: timeoutMinutes };
+}
+
 /**
  * Parses and validates DNS servers from a comma-separated string
  * @param input - Comma-separated DNS server string (e.g., "8.8.8.8,1.1.1.1")
@@ -1114,13 +1127,13 @@ program
 
     // Parse and validate --agent-timeout
     if (options.agentTimeout !== undefined) {
-      const timeoutMinutes = parseInt(options.agentTimeout, 10);
-      if (isNaN(timeoutMinutes) || timeoutMinutes <= 0) {
-        logger.error('--agent-timeout must be a positive integer (minutes)');
+      const result = parseAgentTimeout(options.agentTimeout);
+      if ('error' in result) {
+        logger.error(result.error);
         process.exit(1);
       }
-      config.agentTimeout = timeoutMinutes;
-      logger.info(`Agent timeout set to ${timeoutMinutes} minutes`);
+      config.agentTimeout = result.minutes;
+      logger.info(`Agent timeout set to ${result.minutes} minutes`);
     }
 
     // Build rate limit config when API proxy is enabled