@agentskit/cli 0.5.1 → 0.5.2

package/dist/index.cjs

@@ -1042,8 +1042,8 @@ var PROVIDER_ENV_KEYS = {
 };
 var PROVIDER_REACH_URLS = {
   openai: "https://api.openai.com/v1/models",
-  anthropic: "https://api.anthropic.com",
-  gemini: "https://generativelanguage.googleapis.com",
+  anthropic: "https://api.anthropic.com/v1/messages",
+  gemini: "https://generativelanguage.googleapis.com/v1beta/models",
   ollama: "http://localhost:11434/api/tags"
 };
 async function checkNodeVersion() {
@@ -1131,10 +1131,10 @@ async function checkProviderEnv(provider) {
   const value = process.env[envKey];
   if (!value) {
     return {
-      status: "fail",
+      status: "skip",
       name: `${provider} API key`,
-      detail: `${envKey} is not set`,
-      fix: `export ${envKey}=...`
+      detail: `${envKey} not set`,
+      fix: `export ${envKey}=... (only needed if you use ${provider})`
     };
   }
   if (value.length < 16) {
@@ -1150,7 +1150,11 @@ async function checkProviderEnv(provider) {
 async function checkProviderReachable(provider, fetchImpl = fetch, timeoutMs = 4e3) {
   const url = PROVIDER_REACH_URLS[provider];
   if (!url) {
-    return { status: "skip", name: `${provider} reachable`, detail: "No reachability check for this provider" };
+    return { status: "skip", name: `${provider} reachable`, detail: "No reachability check configured" };
+  }
+  const envKey = PROVIDER_ENV_KEYS[provider];
+  if (envKey && !process.env[envKey]) {
+    return { status: "skip", name: `${provider} reachable`, detail: "Skipped \u2014 no API key configured" };
   }
   const controller = new AbortController();
   const timer = setTimeout(() => controller.abort(), timeoutMs);
@@ -1158,12 +1162,18 @@ async function checkProviderReachable(provider, fetchImpl = fetch, timeoutMs = 4
     const res = await fetchImpl(url, {
       method: "GET",
       signal: controller.signal
-      // No auth — we just want to confirm DNS + network reach.
     });
+    if (res.status >= 200 && res.status < 400) {
+      return { status: "pass", name: `${provider} reachable`, detail: `${url} \u2192 ${res.status} OK` };
+    }
+    if (res.status === 401 || res.status === 403 || res.status === 405) {
+      return { status: "pass", name: `${provider} reachable`, detail: `${url} \u2192 ${res.status} (host reachable)` };
+    }
     return {
-      status: "pass",
+      status: "warn",
       name: `${provider} reachable`,
-      detail: `${url} \u2192 HTTP ${res.status}`
+      detail: `${url} \u2192 HTTP ${res.status}`,
+      fix: "Host reachable but returned unexpected status \u2014 check provider docs"
     };
   } catch (err) {
     const reason = err.name === "AbortError" ? `timeout after ${timeoutMs}ms` : err.message;
@@ -1221,41 +1231,69 @@ async function runDoctor(options = {}) {
   };
 }
 var ICON = {
-  pass: "\u2713",
-  warn: "!",
-  fail: "\u2717",
-  skip: "\xB7"
+  pass: "\u2714",
+  warn: "\u26A0",
+  fail: "\u2718",
+  skip: "\u25CB"
 };
 function renderReport(report, opts = {}) {
   const color = opts.color ?? true;
   const c = (code, text) => color ? `\x1B[${code}m${text}\x1B[0m` : text;
   const colorFor = {
     pass: (t) => c("32", t),
+    // green
     warn: (t) => c("33", t),
+    // yellow
     fail: (t) => c("31", t),
+    // red
     skip: (t) => c("90", t)
+    // dim
   };
   const lines = [];
   lines.push("");
-  lines.push(c("1", "agentskit doctor"));
+  lines.push(`  ${c("1;36", "\u26A1 AgentsKit Doctor")}`);
+  lines.push(`  ${c("90", "\u2500".repeat(50))}`);
   lines.push("");
+  const groups = {
+    "Environment": [],
+    "Providers": [],
+    "Network": []
+  };
   for (const r of report.results) {
-    const icon = colorFor[r.status](ICON[r.status]);
-    const name = r.name.padEnd(28);
-    const detail = r.detail ? c("90", r.detail) : "";
-    lines.push(`  ${icon}  ${name} ${detail}`);
-    if (r.fix && (r.status === "fail" || r.status === "warn")) {
-      lines.push(`     ${c("90", "\u21B3 " + r.fix)}`);
+    if (r.name.includes("reachable")) {
+      groups["Network"].push(r);
+    } else if (r.name.includes("API key")) {
+      groups["Providers"].push(r);
+    } else {
+      groups["Environment"].push(r);
     }
   }
-  lines.push("");
-  const summary = [
-    `${report.pass} pass`,
-    report.warn > 0 ? colorFor.warn(`${report.warn} warn`) : `${report.warn} warn`,
-    report.fail > 0 ? colorFor.fail(`${report.fail} fail`) : `${report.fail} fail`,
-    `${report.skip} skip`
-  ].join("  \xB7  ");
-  lines.push(`  ${c("1", "Summary:")} ${summary}`);
+  for (const [group, results] of Object.entries(groups)) {
+    if (results.length === 0) continue;
+    lines.push(`  ${c("1", group)}`);
+    for (const r of results) {
+      const icon = colorFor[r.status](ICON[r.status]);
+      const name = r.name.padEnd(28);
+      const detail = r.detail ? c("90", r.detail) : "";
+      lines.push(`    ${icon}  ${name} ${detail}`);
+      if (r.fix && r.status !== "pass") {
+        lines.push(`       ${c("90", "\u21B3 " + r.fix)}`);
+      }
+    }
+    lines.push("");
+  }
+  lines.push(`  ${c("90", "\u2500".repeat(50))}`);
+  const parts = [];
+  if (report.pass > 0) parts.push(colorFor.pass(`${report.pass} passed`));
+  if (report.warn > 0) parts.push(colorFor.warn(`${report.warn} warnings`));
+  if (report.fail > 0) parts.push(colorFor.fail(`${report.fail} failed`));
+  if (report.skip > 0) parts.push(colorFor.skip(`${report.skip} skipped`));
+  lines.push(`  ${parts.join("  \xB7  ")}`);
+  if (report.fail === 0) {
+    lines.push(`  ${c("32", "\u2714 Ready to build agents.")}`);
+  } else {
+    lines.push(`  ${c("31", "\u2718 Fix the issues above before continuing.")}`);
+  }
   lines.push("");
   return lines.join("\n");
 }