pi-smart-fetch 0.3.4 → 0.3.5

package/dist/index.js

@@ -10087,42 +10087,54 @@ function buildUserFacingFetchErrorSummary(error) {
       return "No readable content could be extracted from the page.";
     case "processing_error":
       return "The response could not be processed.";
-    case "network_error":
+    case "network_error": {
+      if (/dns error/i.test(error.error)) {
+        return "DNS error \u2014 could not resolve the hostname.";
+      }
+      if (/connection failed|connection refused|unreachable/i.test(error.error)) {
+        return "Connection failed \u2014 the server is unreachable.";
+      }
+      if (/tls|ssl/i.test(error.error)) {
+        return "TLS/SSL error \u2014 certificate may be invalid.";
+      }
       return "The request failed before a usable response was returned.";
+    }
     default:
       return error.error;
   }
 }
 function buildFetchErrorResponseText(error) {
   const lines = [`Error: ${error.error}`];
-  const metadata = buildHeader([
-    ["URL", error.url],
-    ["Final URL", error.finalUrl],
-    ["Phase", error.phase ? describeErrorPhase(error.phase) : void 0],
-    [
-      "Timeout",
-      error.timeoutMs ? formatDurationMs(error.timeoutMs) : void 0
-    ],
-    [
-      "HTTP status",
-      error.statusCode ? `${error.statusCode}${error.statusText ? ` ${error.statusText}` : ""}` : void 0
-    ],
-    ["Mime type", error.mimeType],
-    [
-      "Content-Length",
-      error.contentLength !== void 0 ? `${error.contentLength} bytes (${formatByteCount(error.contentLength)})` : void 0
-    ],
-    [
-      "Downloaded before failure",
-      error.downloadedBytes !== void 0 ? `${error.downloadedBytes} bytes (${formatByteCount(error.downloadedBytes)})` : void 0
-    ],
-    [
-      "Suggested timeoutMs",
-      error.code === "timeout" ? suggestRetryTimeoutMs(error) : void 0
-    ]
-  ]);
-  if (metadata) {
-    lines.push("", metadata);
+  if (error.code === "timeout" || error.code === "http_error" || error.code === "download_error") {
+    const metadata = buildHeader([
+      ["URL", error.url],
+      ["Final URL", error.finalUrl],
+      ["Phase", error.phase ? describeErrorPhase(error.phase) : void 0],
+      [
+        "Timeout",
+        error.timeoutMs ? formatDurationMs(error.timeoutMs) : void 0
+      ],
+      [
+        "HTTP status",
+        error.statusCode ? `${error.statusCode}${error.statusText ? ` ${error.statusText}` : ""}` : void 0
+      ],
+      ["Mime type", error.mimeType],
+      [
+        "Content-Length",
+        error.contentLength !== void 0 ? `${error.contentLength} bytes (${formatByteCount(error.contentLength)})` : void 0
+      ],
+      [
+        "Downloaded before failure",
+        error.downloadedBytes !== void 0 ? `${error.downloadedBytes} bytes (${formatByteCount(error.downloadedBytes)})` : void 0
+      ],
+      [
+        "Suggested timeoutMs",
+        error.code === "timeout" ? suggestRetryTimeoutMs(error) : void 0
+      ]
+    ]);
+    if (metadata) {
+      lines.push("", metadata);
+    }
   }
   if (error.code === "timeout") {
     lines.push(
@@ -10656,6 +10668,31 @@ function isTimeoutError(error) {
   const message = error instanceof Error ? error.message : String(error);
   return /timed out|timeout|deadline exceeded|abort(?:ed)?/i.test(message);
 }
+function isDnsError(error) {
+  const message = error instanceof Error ? error.message : String(error);
+  return /dns error|failed to lookup address|nodename nor servname provided|name resolution failed/i.test(
+    message
+  );
+}
+function isConnectError(error) {
+  const message = error instanceof Error ? error.message : String(error);
+  return /client error \(connect\)|connection refused|tcp connect error|connection reset|network unreachable|no route to host/i.test(
+    message
+  );
+}
+function isTlsError(error) {
+  const message = error instanceof Error ? error.message : String(error);
+  return /ssl.*error|tls.*error|bad certificate|certificate.*invalid|unknown.*issuer/i.test(
+    message
+  );
+}
+function extractHostname(url) {
+  try {
+    return new URL(url).hostname;
+  } catch {
+    return url;
+  }
+}
 function buildTimeoutError(context) {
   const targetUrl = context.finalUrl ?? context.url;
   const timeoutLabel = `${context.timeoutMs}ms`;
@@ -10733,14 +10770,47 @@ function buildThrownFetchError(error, context) {
   if (isTimeoutError(error)) {
     return buildTimeoutError(context);
   }
+  const hostname = extractHostname(context.url);
+  if (isDnsError(error)) {
+    return {
+      error: `DNS error: failed to lookup address for ${hostname}. Check the URL and try again.`,
+      code: "network_error",
+      phase: "connecting",
+      retryable: false,
+      url: context.url,
+      finalUrl: context.finalUrl
+    };
+  }
+  if (isConnectError(error)) {
+    return {
+      error: `Connection failed to ${hostname}. The server may be unreachable or blocking requests.`,
+      code: "network_error",
+      phase: "connecting",
+      retryable: true,
+      url: context.url,
+      finalUrl: context.finalUrl
+    };
+  }
+  if (isTlsError(error)) {
+    return {
+      error: `TLS/SSL error connecting to ${hostname}. The server's certificate may be invalid.`,
+      code: "network_error",
+      phase: "connecting",
+      retryable: false,
+      url: context.url,
+      finalUrl: context.finalUrl
+    };
+  }
+  const isConnectLevel = isDnsError(error) || isConnectError(error) || isTlsError(error);
+  const effectivePhase = isConnectLevel ? "connecting" : context.phase;
   const message = error instanceof Error ? error.message : String(error);
   const targetUrl = context.finalUrl ?? context.url;
-  const phaseDescription = context.phase === "loading" ? "downloading the response" : context.phase === "waiting" ? "waiting for the server response" : context.phase === "connecting" ? "connecting" : "fetching";
+  const phaseDescription = effectivePhase === "loading" ? "downloading the response" : effectivePhase === "waiting" ? "waiting for the server response" : effectivePhase === "connecting" ? "connecting" : "fetching";
   return {
-    error: context.phase === "processing" ? `Failed while processing the response from ${targetUrl}: ${message}` : `Request failed while ${phaseDescription} for ${targetUrl}: ${message}`,
-    code: context.phase === "processing" ? "processing_error" : context.phase === "loading" && context.mimeType ? "download_error" : "network_error",
-    phase: context.phase,
-    retryable: context.phase !== "processing",
+    error: effectivePhase === "processing" ? `Failed while processing the response from ${targetUrl}: ${message}` : `Request failed while ${phaseDescription} for ${targetUrl}: ${message}`,
+    code: effectivePhase === "processing" ? "processing_error" : effectivePhase === "loading" && context.mimeType ? "download_error" : "network_error",
+    phase: effectivePhase,
+    retryable: effectivePhase !== "processing",
     timeoutMs: context.timeoutMs,
     url: context.url,
     finalUrl: context.finalUrl,