azdo-cli 0.2.0-hotfix-0-2-5.63 → 0.2.4

package/dist/index.js

@@ -42,19 +42,6 @@ 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) {
@@ -70,18 +57,11 @@ 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");
-  const normalizedExtraFields = extraFields ? normalizeFieldList(extraFields) : [];
-  if (normalizedExtraFields.length > 0) {
-    const allFields = normalizeFieldList([...DEFAULT_FIELDS, ...normalizedExtraFields]);
+  if (extraFields && extraFields.length > 0) {
+    const allFields = [...DEFAULT_FIELDS, ...extraFields];
     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}`);
   }
@@ -113,7 +93,7 @@ async function getWorkItem(context, id, pat, extraFields) {
     areaPath: data.fields["System.AreaPath"],
     iterationPath: data.fields["System.IterationPath"],
     url: data._links.html.href,
-    extraFields: normalizedExtraFields.length > 0 ? buildExtraFields(data.fields, normalizedExtraFields) : null
+    extraFields: extraFields && extraFields.length > 0 ? buildExtraFields(data.fields, extraFields) : null
   };
 }
 async function getWorkItemFieldValue(context, id, pat, fieldName) {
@@ -123,12 +103,6 @@ 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}`);
   }
@@ -153,7 +127,12 @@ async function updateWorkItem(context, id, pat, fieldName, operations) {
     body: JSON.stringify(operations)
   });
   if (response.status === 400) {
-    const serverMessage = await readResponseMessage(response) ?? "Unknown error";
+    let serverMessage = "Unknown error";
+    try {
+      const body = await response.json();
+      if (body.message) serverMessage = body.message;
+    } catch {
+    }
     throw new Error(`UPDATE_REJECTED: ${serverMessage}`);
   }
   if (!response.ok) {
@@ -442,7 +421,7 @@ function validateOrgProjectPair(options) {
     process.exit(1);
   }
 }
-function handleCommandError(err, id, context, scope = "write", exit = true) {
+function handleCommandError(err, id, context, scope = "write") {
   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)";
@@ -465,10 +444,6 @@ function handleCommandError(err, id, context, scope = "write", exit = true) {
     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}
@@ -477,21 +452,10 @@ function handleCommandError(err, id, context, scope = "write", exit = true) {
     process.stderr.write(`Error: ${msg}
 `);
   }
-  if (exit) {
-    process.exit(1);
-  } else {
-    process.exitCode = 1;
-  }
+  process.exit(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");
@@ -551,12 +515,12 @@ function createGetItemCommand() {
       try {
         context = resolveContext(options);
         const credential = await resolvePat();
-        const fieldsList = options.fields !== void 0 ? parseRequestedFields(options.fields) : parseRequestedFields(loadConfig().fields);
+        const fieldsList = options.fields ? options.fields.split(",").map((f) => f.trim()) : 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", false);
+        handleCommandError(err, id, context, "read");
       }
     }
   );

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "azdo-cli",
-  "version": "0.2.0-hotfix-0-2-5.63",
+  "version": "0.2.4",
   "description": "Azure DevOps CLI tool",
   "type": "module",
   "bin": {