azdo-cli 0.19.0-feature-037-api-error-surfacing.697 → 0.19.0-feature-037-api-error-surfacing.701

package/dist/index.js

@@ -81,8 +81,8 @@ function redactUrl(url) {
 var SECRET_ASSIGNMENT = String.raw`["']?\s*[:=]\s*["']?)[^"'\s,;&}]+`;
 var SECRET_IN_TEXT = [
   [/\b(Bearer|Basic)\s+[\w.~+/=-]{8,}/gi, `$1 ${REDACTED}`],
-  [new RegExp(`(\\b(?:access|refresh|api)[_-]?(?:token|key)\\b${SECRET_ASSIGNMENT}`, "gi"), `$1${REDACTED}`],
-  [new RegExp(`(\\b(?:token|pat|password|secret)\\b${SECRET_ASSIGNMENT}`, "gi"), `$1${REDACTED}`],
+  [new RegExp(String.raw`(\b(?:access|refresh|api)[_-]?(?:token|key)\b${SECRET_ASSIGNMENT}`, "gi"), `$1${REDACTED}`],
+  [new RegExp(String.raw`(\b(?:token|pat|password|secret)\b${SECRET_ASSIGNMENT}`, "gi"), `$1${REDACTED}`],
   [/\beyJ[A-Za-z0-9._~+/=-]{20,}/g, REDACTED],
   [/\b(?=[A-Za-z0-9]*\d)(?=[A-Za-z0-9]*[A-Za-z])[A-Za-z0-9]{40,}\b/g, REDACTED]
 ];
@@ -502,7 +502,9 @@ function describeFailureBody(body, contentType) {
     return null;
   }
   const redacted = redactBody(trimmed) ?? trimmed;
-  return truncateDetail(renderErrorFields(redacted) ?? redacted.slice(0, MAX_RAW_BODY_CHARS));
+  const fields = renderErrorFields(redacted);
+  if (fields !== null) return truncateDetail(fields);
+  return truncateDetail(redactText(redacted).slice(0, MAX_RAW_BODY_CHARS));
 }
 function truncateDetail(detail) {
   const collapsed = redactText(detail).replace(/\s+/g, " ").trim();
@@ -545,12 +547,12 @@ function writeTraceEntry(writer, url, init, response, responseBody) {
 async function traceResponse(url, init, response) {
   const writer = getActiveTraceWriter();
   if (!writer) return null;
-  let responseBody = "";
+  let responseBody = null;
   try {
     responseBody = await response.clone().text();
   } catch {
   }
-  writeTraceEntry(writer, url, init, response, responseBody);
+  writeTraceEntry(writer, url, init, response, responseBody ?? "");
   return responseBody;
 }
 async function fetchWithErrors(url, init) {
@@ -572,8 +574,9 @@ async function fetchWithErrors(url, init) {
   if (response.status === 401) throw new Error(withDetail("AUTH_FAILED", failureDetail));
   if (response.status === 403) throw new Error(withDetail("PERMISSION_DENIED", failureDetail));
   if (response.status === 404) {
-    const body = failureBodies.get(response);
-    throw new Error(body === void 0 ? "NOT_FOUND" : `NOT_FOUND | url=${url} | body=${body}`);
+    throw new Error(
+      failureDetail === null ? "NOT_FOUND" : `NOT_FOUND | url=${redactUrl(url)} | body=${failureDetail}`
+    );
   }
   if (contentType.toLowerCase().startsWith("text/html")) {
     throw new Error("AUTH_FAILED");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "azdo-cli",
-  "version": "0.19.0-feature-037-api-error-surfacing.697",
+  "version": "0.19.0-feature-037-api-error-surfacing.701",
   "description": "Azure DevOps CLI tool",
   "type": "module",
   "bin": {