azdo-cli 0.2.0-develop.62 → 0.2.0-develop.71

package/README.md

@@ -53,7 +53,7 @@ azdo set-state 12345 "Active"
 
 | Command | Purpose | Common Flags |
 | --- | --- | --- |
-| `azdo get-item <id>` | Read a work item | `--short`, `--fields`, `--org`, `--project` |
+| `azdo get-item <id>` | Read a work item | `--short`, `--fields`, `--markdown`, `--org`, `--project` |
 | `azdo set-state <id> <state>` | Change work item state | `--json`, `--org`, `--project` |
 | `azdo assign <id> [name]` | Assign or unassign owner | `--unassign`, `--json`, `--org`, `--project` |
 | `azdo set-field <id> <field> <value>` | Update any field | `--json`, `--org`, `--project` |
@@ -75,6 +75,12 @@ azdo get-item 12345 --short
 
 # Include extra fields for this call
 azdo get-item 12345 --fields "System.Tags,Microsoft.VSTS.Common.Priority"
+
+# Convert rich text fields to markdown
+azdo get-item 12345 --markdown
+
+# Disable markdown even if config is on
+azdo get-item 12345 --no-markdown
 ```
 
 ```bash
@@ -89,6 +95,14 @@ azdo assign 12345 --unassign
 azdo set-field 12345 System.Title "Updated title"
 ```
 
+### Markdown Display
+
+The `get-item` command can convert HTML rich-text fields to readable markdown. Resolution order:
+
+1. `--markdown` / `--no-markdown` flag (highest priority)
+2. Config setting: `azdo config set markdown true`
+3. Default: off (HTML stripped to plain text)
+
 ### Markdown Field Commands
 
 ```bash
@@ -114,6 +128,9 @@ azdo config list
 # Interactive setup
 azdo config wizard
 
+# Enable markdown display for all get-item calls
+azdo config set markdown true
+
 # Set/get/unset values
 azdo config set fields "System.Tags,Custom.Priority"
 azdo config get fields

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) {
@@ -315,6 +336,13 @@ var SETTINGS = [
     type: "string[]",
     example: "System.Tags,Custom.Priority",
     required: false
+  },
+  {
+    key: "markdown",
+    description: "Convert rich text fields to markdown on display",
+    type: "boolean",
+    example: "true",
+    required: false
   }
 ];
 var VALID_KEYS = SETTINGS.map((s) => s.key);
@@ -348,7 +376,7 @@ function saveConfig(config) {
 }
 function validateKey(key) {
   if (!VALID_KEYS.includes(key)) {
-    throw new Error(`Unknown setting key "${key}". Valid keys: org, project, fields`);
+    throw new Error(`Unknown setting key "${key}". Valid keys: ${VALID_KEYS.join(", ")}`);
   }
 }
 function getConfigValue(key) {
@@ -361,6 +389,11 @@ function setConfigValue(key, value) {
   const config = loadConfig();
   if (value === "") {
     delete config[key];
+  } else if (key === "markdown") {
+    if (value !== "true" && value !== "false") {
+      throw new Error(`Invalid value "${value}" for markdown. Must be "true" or "false".`);
+    }
+    config.markdown = value === "true";
   } else if (key === "fields") {
     config.fields = value.split(",").map((s) => s.trim());
   } else {
@@ -399,6 +432,61 @@ function resolveContext(options) {
   );
 }
 
+// src/services/md-convert.ts
+import { NodeHtmlMarkdown } from "node-html-markdown";
+
+// src/services/html-detect.ts
+var HTML_TAG_REGEX = /<\/?([a-z][a-z0-9]*)\b/gi;
+var HTML_TAGS = /* @__PURE__ */ new Set([
+  "p",
+  "br",
+  "div",
+  "span",
+  "strong",
+  "em",
+  "b",
+  "i",
+  "u",
+  "a",
+  "ul",
+  "ol",
+  "li",
+  "h1",
+  "h2",
+  "h3",
+  "h4",
+  "h5",
+  "h6",
+  "table",
+  "tr",
+  "td",
+  "th",
+  "img",
+  "pre",
+  "code"
+]);
+function isHtml(content) {
+  let match;
+  HTML_TAG_REGEX.lastIndex = 0;
+  while ((match = HTML_TAG_REGEX.exec(content)) !== null) {
+    if (HTML_TAGS.has(match[1].toLowerCase())) {
+      return true;
+    }
+  }
+  return false;
+}
+
+// src/services/md-convert.ts
+function htmlToMarkdown(html) {
+  return NodeHtmlMarkdown.translate(html);
+}
+function toMarkdown(content) {
+  if (isHtml(content)) {
+    return htmlToMarkdown(content);
+  }
+  return content;
+}
+
 // src/services/command-helpers.ts
 function parseWorkItemId(idStr) {
   const id = Number.parseInt(idStr, 10);
@@ -421,7 +509,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 +532,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 +544,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");
@@ -472,7 +575,24 @@ function stripHtml(html) {
   text = text.replace(/\n{3,}/g, "\n\n");
   return text.trim();
 }
-function formatWorkItem(workItem, short) {
+function convertRichText(html, markdown) {
+  if (!html) return "";
+  return markdown ? toMarkdown(html) : stripHtml(html);
+}
+function formatExtraFields(extraFields, markdown) {
+  return Object.entries(extraFields).map(([refName, value]) => {
+    const fieldLabel = refName.includes(".") ? refName.split(".").pop() : refName;
+    const displayValue = markdown ? toMarkdown(value) : value;
+    return `${fieldLabel.padEnd(13)}${displayValue}`;
+  });
+}
+function summarizeDescription(text, label) {
+  const descLines = text.split("\n").filter((l) => l.trim() !== "");
+  const firstThree = descLines.slice(0, 3);
+  const suffix = descLines.length > 3 ? "\n..." : "";
+  return [`${label("Description:")}${firstThree.join("\n")}${suffix}`];
+}
+function formatWorkItem(workItem, short, markdown = false) {
   const lines = [];
   const label = (name) => name.padEnd(13);
   lines.push(`${label("ID:")}${workItem.id}`);
@@ -486,19 +606,12 @@ function formatWorkItem(workItem, short) {
   }
   lines.push(`${label("URL:")}${workItem.url}`);
   if (workItem.extraFields) {
-    for (const [refName, value] of Object.entries(workItem.extraFields)) {
-      const fieldLabel = refName.includes(".") ? refName.split(".").pop() : refName;
-      lines.push(`${fieldLabel.padEnd(13)}${value}`);
-    }
+    lines.push(...formatExtraFields(workItem.extraFields, markdown));
   }
   lines.push("");
-  const descriptionText = workItem.description ? stripHtml(workItem.description) : "";
+  const descriptionText = convertRichText(workItem.description, markdown);
   if (short) {
-    const descLines = descriptionText.split("\n").filter((l) => l.trim() !== "");
-    const firstThree = descLines.slice(0, 3);
-    const truncated = descLines.length > 3;
-    const descSummary = firstThree.join("\n") + (truncated ? "\n..." : "");
-    lines.push(`${label("Description:")}${descSummary}`);
+    lines.push(...summarizeDescription(descriptionText, label));
   } else {
     lines.push("Description:");
     lines.push(descriptionText);
@@ -507,7 +620,7 @@ function formatWorkItem(workItem, short) {
 }
 function createGetItemCommand() {
   const command = new Command("get-item");
-  command.description("Retrieve an Azure DevOps work item by ID").argument("<id>", "work item ID").option("--org <org>", "Azure DevOps organization").option("--project <project>", "Azure DevOps project").option("--short", "show abbreviated output").option("--fields <fields>", "comma-separated additional field reference names").action(
+  command.description("Retrieve an Azure DevOps work item by ID").argument("<id>", "work item ID").option("--org <org>", "Azure DevOps organization").option("--project <project>", "Azure DevOps project").option("--short", "show abbreviated output").option("--fields <fields>", "comma-separated additional field reference names").option("--markdown", "convert rich text fields to markdown").action(
     async (idStr, options) => {
       const id = parseWorkItemId(idStr);
       validateOrgProjectPair(options);
@@ -515,12 +628,13 @@ 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);
+        const markdownEnabled = options.markdown ?? loadConfig().markdown ?? false;
+        const output = formatWorkItem(workItem, options.short ?? false, markdownEnabled);
         process.stdout.write(output + "\n");
       } catch (err) {
-        handleCommandError(err, id, context, "read");
+        handleCommandError(err, id, context, "read", false);
       }
     }
   );
@@ -821,63 +935,6 @@ function createSetFieldCommand() {
 
 // src/commands/get-md-field.ts
 import { Command as Command7 } from "commander";
-
-// src/services/md-convert.ts
-import { NodeHtmlMarkdown } from "node-html-markdown";
-
-// src/services/html-detect.ts
-var HTML_TAG_REGEX = /<\/?([a-z][a-z0-9]*)\b/gi;
-var HTML_TAGS = /* @__PURE__ */ new Set([
-  "p",
-  "br",
-  "div",
-  "span",
-  "strong",
-  "em",
-  "b",
-  "i",
-  "u",
-  "a",
-  "ul",
-  "ol",
-  "li",
-  "h1",
-  "h2",
-  "h3",
-  "h4",
-  "h5",
-  "h6",
-  "table",
-  "tr",
-  "td",
-  "th",
-  "img",
-  "pre",
-  "code"
-]);
-function isHtml(content) {
-  let match;
-  HTML_TAG_REGEX.lastIndex = 0;
-  while ((match = HTML_TAG_REGEX.exec(content)) !== null) {
-    if (HTML_TAGS.has(match[1].toLowerCase())) {
-      return true;
-    }
-  }
-  return false;
-}
-
-// src/services/md-convert.ts
-function htmlToMarkdown(html) {
-  return NodeHtmlMarkdown.translate(html);
-}
-function toMarkdown(content) {
-  if (isHtml(content)) {
-    return htmlToMarkdown(content);
-  }
-  return content;
-}
-
-// src/commands/get-md-field.ts
 function createGetMdFieldCommand() {
   const command = new Command7("get-md-field");
   command.description("Get a work item field value, converting HTML to markdown").argument("<id>", "work item ID").argument("<field>", "field reference name (e.g., System.Description)").option("--org <org>", "Azure DevOps organization").option("--project <project>", "Azure DevOps project").action(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "azdo-cli",
-  "version": "0.2.0-develop.62",
+  "version": "0.2.0-develop.71",
   "description": "Azure DevOps CLI tool",
   "type": "module",
   "bin": {