azdo-cli 0.2.4 → 0.2.5

package/dist/index.js

@@ -42,6 +42,19 @@ async function fetchWithErrors(url, init) {
   if (response.status === 404) throw new Error("NOT_FOUND");
   return response;
 }
+async function readResponseMessage(response) {
+  try {
+    const body = await response.json();
+    if (typeof body.message === "string" && body.message.trim() !== "") {
+      return body.message.trim();
+    }
+  } catch {
+  }
+  return null;
+}
+function normalizeFieldList(fields) {
+  return Array.from(new Set(fields.map((f) => f.trim()).filter((f) => f.length > 0)));
+}
 function buildExtraFields(fields, requested) {
   const result = {};
   for (const name of requested) {
@@ -57,11 +70,18 @@ async function getWorkItem(context, id, pat, extraFields) {
     `https://dev.azure.com/${encodeURIComponent(context.org)}/${encodeURIComponent(context.project)}/_apis/wit/workitems/${id}`
   );
   url.searchParams.set("api-version", "7.1");
-  if (extraFields && extraFields.length > 0) {
-    const allFields = [...DEFAULT_FIELDS, ...extraFields];
+  const normalizedExtraFields = extraFields ? normalizeFieldList(extraFields) : [];
+  if (normalizedExtraFields.length > 0) {
+    const allFields = normalizeFieldList([...DEFAULT_FIELDS, ...normalizedExtraFields]);
     url.searchParams.set("fields", allFields.join(","));
   }
   const response = await fetchWithErrors(url.toString(), { headers: authHeaders(pat) });
+  if (response.status === 400) {
+    const serverMessage = await readResponseMessage(response);
+    if (serverMessage) {
+      throw new Error(`BAD_REQUEST: ${serverMessage}`);
+    }
+  }
   if (!response.ok) {
     throw new Error(`HTTP_${response.status}`);
   }
@@ -93,7 +113,7 @@ async function getWorkItem(context, id, pat, extraFields) {
     areaPath: data.fields["System.AreaPath"],
     iterationPath: data.fields["System.IterationPath"],
     url: data._links.html.href,
-    extraFields: extraFields && extraFields.length > 0 ? buildExtraFields(data.fields, extraFields) : null
+    extraFields: normalizedExtraFields.length > 0 ? buildExtraFields(data.fields, normalizedExtraFields) : null
   };
 }
 async function getWorkItemFieldValue(context, id, pat, fieldName) {
@@ -103,6 +123,12 @@ async function getWorkItemFieldValue(context, id, pat, fieldName) {
   url.searchParams.set("api-version", "7.1");
   url.searchParams.set("fields", fieldName);
   const response = await fetchWithErrors(url.toString(), { headers: authHeaders(pat) });
+  if (response.status === 400) {
+    const serverMessage = await readResponseMessage(response);
+    if (serverMessage) {
+      throw new Error(`BAD_REQUEST: ${serverMessage}`);
+    }
+  }
   if (!response.ok) {
     throw new Error(`HTTP_${response.status}`);
   }
@@ -127,12 +153,7 @@ async function updateWorkItem(context, id, pat, fieldName, operations) {
     body: JSON.stringify(operations)
   });
   if (response.status === 400) {
-    let serverMessage = "Unknown error";
-    try {
-      const body = await response.json();
-      if (body.message) serverMessage = body.message;
-    } catch {
-    }
+    const serverMessage = await readResponseMessage(response) ?? "Unknown error";
     throw new Error(`UPDATE_REJECTED: ${serverMessage}`);
   }
   if (!response.ok) {
@@ -421,7 +442,7 @@ function validateOrgProjectPair(options) {
     process.exit(1);
   }
 }
-function handleCommandError(err, id, context, scope = "write") {
+function handleCommandError(err, id, context, scope = "write", exit = true) {
   const error = err instanceof Error ? err : new Error(String(err));
   const msg = error.message;
   const scopeLabel = scope === "read" ? "Work Items (read)" : "Work Items (Read & Write)";
@@ -444,6 +465,10 @@ function handleCommandError(err, id, context, scope = "write") {
     process.stderr.write(
       "Error: Could not connect to Azure DevOps. Check your network connection.\n"
     );
+  } else if (msg.startsWith("BAD_REQUEST:")) {
+    const serverMsg = msg.replace("BAD_REQUEST: ", "");
+    process.stderr.write(`Error: Request rejected: ${serverMsg}
+`);
   } else if (msg.startsWith("UPDATE_REJECTED:")) {
     const serverMsg = msg.replace("UPDATE_REJECTED: ", "");
     process.stderr.write(`Error: Update rejected: ${serverMsg}
@@ -452,10 +477,21 @@ function handleCommandError(err, id, context, scope = "write") {
     process.stderr.write(`Error: ${msg}
 `);
   }
-  process.exit(1);
+  if (exit) {
+    process.exit(1);
+  } else {
+    process.exitCode = 1;
+  }
 }
 
 // src/commands/get-item.ts
+function parseRequestedFields(raw) {
+  if (raw === void 0) return void 0;
+  const source = Array.isArray(raw) ? raw : [raw];
+  const tokens = source.flatMap((entry) => entry.split(/[,\s]+/)).map((field) => field.trim()).filter((field) => field.length > 0);
+  if (tokens.length === 0) return void 0;
+  return Array.from(new Set(tokens));
+}
 function stripHtml(html) {
   let text = html;
   text = text.replace(/<h[1-6][^>]*>(.*?)<\/h[1-6]>/gi, "\n--- $1 ---\n");
@@ -515,12 +551,12 @@ function createGetItemCommand() {
       try {
         context = resolveContext(options);
         const credential = await resolvePat();
-        const fieldsList = options.fields ? options.fields.split(",").map((f) => f.trim()) : loadConfig().fields;
+        const fieldsList = options.fields !== void 0 ? parseRequestedFields(options.fields) : parseRequestedFields(loadConfig().fields);
         const workItem = await getWorkItem(context, id, credential.pat, fieldsList);
         const output = formatWorkItem(workItem, options.short ?? false);
         process.stdout.write(output + "\n");
       } catch (err) {
-        handleCommandError(err, id, context, "read");
+        handleCommandError(err, id, context, "read", false);
       }
     }
   );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "azdo-cli",
-  "version": "0.2.4",
+  "version": "0.2.5",
   "description": "Azure DevOps CLI tool",
   "type": "module",
   "bin": {