refactor: doctor defaults to live mode, remove setup command entirely (#263)

- Remove setup command completely (no backward compat needed)
- Doctor now runs live connectivity test by default
- Add --no-live flag to skip if needed
- Update SKILL.md docs

Author: jackwener

SKILL.md

@@ -244,8 +244,7 @@ opencli install <name>      # Auto-install an external CLI (e.g., gh, obsidian)
 opencli register <name>     # Register a local custom CLI for unified discovery
 opencli validate            # Validate all CLI definitions
 opencli validate bilibili   # Validate specific site
-opencli doctor              # Diagnose browser bridge connectivity (auto-starts daemon)
-opencli doctor --live       # Also test live browser connectivity
+opencli doctor              # Diagnose browser bridge (auto-starts daemon, includes live test)
 ```
 
 ### AI Agent Workflow

src/cli.ts

@@ -207,24 +207,14 @@ export function runCli(BUILTIN_CLIS: string, USER_CLIS: string): void {
   program
     .command('doctor')
     .description('Diagnose opencli browser bridge connectivity')
-    .option('--live', 'Test browser connectivity (requires Chrome running)', false)
+    .option('--no-live', 'Skip live browser connectivity test')
     .option('--sessions', 'Show active automation sessions', false)
     .action(async (opts) => {
       const { runBrowserDoctor, renderBrowserDoctorReport } = await import('./doctor.js');
       const report = await runBrowserDoctor({ live: opts.live, sessions: opts.sessions, cliVersion: PKG_VERSION });
       console.log(renderBrowserDoctorReport(report));
     });
 
-  program
-    .command('setup')
-    .description('(deprecated) Use "opencli doctor" instead')
-    .action(async () => {
-      console.log(chalk.yellow('⚠ "opencli setup" is deprecated. Running "opencli doctor" instead.\n'));
-      const { runBrowserDoctor, renderBrowserDoctorReport } = await import('./doctor.js');
-      const report = await runBrowserDoctor({ cliVersion: PKG_VERSION });
-      console.log(renderBrowserDoctorReport(report));
-    });
-
   program
     .command('completion')
     .description('Output shell completion script')

src/doctor.ts

@@ -121,7 +121,7 @@ export function renderBrowserDoctorReport(report: DoctorReport): string {
       : `failed (${report.connectivity.error ?? 'unknown'})`;
     lines.push(`${connIcon} Connectivity: ${detail}`);
   } else {
-    lines.push(`${chalk.dim('[SKIP]')} Connectivity: not tested (use --live)`);
+    lines.push(`${chalk.dim('[SKIP]')} Connectivity: skipped (--no-live)`);
   }
 
   if (report.sessions) {