@allocator-one/harness-cli 1.5.2 → 1.6.0

package/README.md

@@ -69,6 +69,24 @@ Example tool call: `issue_list` with `{"ready": true, "team": "ALL"}`
 returns the ready queue as JSON; `issue_claim` with `{"id": "IO-123"}`
 claims atomically (a conflict means someone else holds it).
 
+Issue creation resolves `--label` names within the target team first, then at
+workspace scope. Missing names are created at workspace scope before the issue
+is filed, and the final label ids travel in the single issue-create request, so
+a preparation or permission failure cannot leave an unlabeled issue behind.
+Creating a missing label requires Harness administrator standing.
+
+Label repair is additive and repeatable; it never implicitly replaces the
+whole set:
+
+```sh
+harness update IO-123 --add-label security --add-label needs-review
+harness update IO-123 --remove-label security
+```
+
+Both update flags accept a label name or id. Additions allow only labels from
+the issue's team or workspace. Removal can target an attached legacy foreign
+label; when attached normalized names are ambiguous, use the label id.
+
 Auth resolution is identical to the CLI. Without a credential, tools return
 an error telling the agent to run `harness login` in a terminal — the
 server never opens a browser itself.

package/dist/{chunk-CIW4OU5G.js → chunk-DQDNAG5E.js}

@@ -619,7 +619,6 @@ async function version(config) {
 ` };
 }
 async function create(ctx, opts) {
-  const labels = wantedLabels(opts);
   if (opts.title === void 0) {
     throw new UsageError("usage: harness create --title <title> [--team <key>] \u2026");
   }
@@ -627,6 +626,8 @@ async function create(ctx, opts) {
   const team = requiredTeam(ctx, opts.team);
   const priority = optionalPriority(opts.priority);
   const dueDate = optionalDueDate(opts.dueDate);
+  const labels = requestedLabelNames(wantedLabels(opts));
+  const labelIds = await prepareCreateLabelIds(ctx, team, labels);
   let body = { title: opts.title };
   body = putOpt(body, "description", opts.description);
   body = putOpt(body, "design", opts.design);
@@ -636,23 +637,14 @@ async function create(ctx, opts) {
   body = putPresent(body, "priority", priority);
   body = putDueDate(body, dueDate);
   body = putAssignee(body, opts.assignee);
+  if (labelIds.length > 0) body["label_ids"] = labelIds;
   const created = await ctx.client.post(`/api/v1/teams/${team}/issues`, body);
   const issue = created.data;
-  try {
-    const labeled = await applyLabels(ctx, issue, labels);
-    return {
-      json: labeled,
-      human: `Created ${str2(labeled["identifier"])}: ${clean(labeled["title"])}
+  return {
+    json: issue,
+    human: `Created ${str2(issue["identifier"])}: ${clean(issue["title"])}
 `
-    };
-  } catch (error) {
-    if (error instanceof HttpError || error instanceof TransportError) {
-      throw new CommandError(
-        `created ${str2(issue["identifier"])}, but applying labels failed: ${errorText(error)}`
-      );
-    }
-    throw error;
-  }
+  };
 }
 async function update(ctx, identifier, opts) {
   requireIdentifiers([identifier]);
@@ -666,10 +658,15 @@ async function update(ctx, identifier, opts) {
   }
   const priority = optionalPriority(opts.priority);
   const dueDate = optionalDueDate(opts.dueDate);
-  let issue = null;
+  const addLabels = requestedLabelInputs(opts.addLabel ?? []);
+  const removeLabels = requestedLabelInputs(opts.removeLabel ?? []);
+  const labelPlan = addLabels.length > 0 || removeLabels.length > 0 ? await prepareLabelUpdate(ctx, identifier, addLabels, removeLabels) : null;
+  let issue = labelPlan?.issue ?? null;
+  let wrote = false;
   if (opts.claim === true) {
     const body = await ctx.client.post(`/api/v1/issues/${identifier}/claim`);
     issue = body.data;
+    wrote = true;
   }
   let patch = {};
   patch = putOpt(patch, "title", opts.title);
@@ -692,15 +689,26 @@ ${opts.appendNotes}`;
   if (Object.keys(patch).length > 0) {
     const body = await ctx.client.patch(`/api/v1/issues/${identifier}`, patch);
     issue = body.data;
+    wrote = true;
   }
   if (opts.state !== void 0) {
     issue = await setState(ctx, identifier, opts.state);
+    wrote = true;
+  }
+  if (labelPlan?.changed === true) {
+    const body = await ctx.client.put(`/api/v1/issues/${identifier}/labels`, {
+      label_ids: labelPlan.ids
+    });
+    issue = body.data;
+    wrote = true;
   }
   if (issue === null) {
     throw new UsageError("usage: harness update <id> --claim | --title \u2026 | --state <name> | \u2026");
   }
-  return { json: issue, human: `Updated ${str2(issue["identifier"])}
-` };
+  const human = wrote ? `Updated ${str2(issue["identifier"])}
+` : `Labels already up to date for ${str2(issue["identifier"])}
+`;
+  return { json: issue, human };
 }
 async function close(ctx, identifiers, opts) {
   if (identifiers.length === 0) {
@@ -902,27 +910,137 @@ function validateType(type) {
     throw new UsageError(`not an issue type: ${type} (expected ${ISSUE_TYPES.join(", ")})`);
   }
 }
-async function applyLabels(ctx, issue, names) {
-  if (names.length === 0) return issue;
-  const labelsBody = await ctx.client.get("/api/v1/labels");
-  const byName = new Map(labelsBody.data.map((label) => [str2(label["name"]), label["id"]]));
+var UUID_FORMAT = /^[0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12}$/i;
+function requestedLabelNames(names) {
+  const unique = /* @__PURE__ */ new Map();
+  for (const name of requestedLabelInputs(names)) {
+    const normalized = normalizeLabelName(name);
+    if (!unique.has(normalized)) unique.set(normalized, name);
+  }
+  return [...unique.values()];
+}
+function requestedLabelInputs(values) {
+  return values.map((value) => {
+    const trimmed = value.trim();
+    if (trimmed === "") throw new UsageError("label names and ids cannot be empty");
+    return trimmed;
+  });
+}
+function normalizeLabelName(value) {
+  return str2(value).trim().toLowerCase();
+}
+async function prepareCreateLabelIds(ctx, teamKey, names) {
+  if (names.length === 0) return [];
+  const teams = await ctx.client.get("/api/v1/teams");
+  const target = teams.data.find(
+    (team) => str2(team["key"]).toLowerCase() === teamKey.toLowerCase()
+  );
+  if (target === void 0) throw new CommandError(`No team matches key "${teamKey}".`);
+  const teamId = str2(target["id"]);
+  if (teamId === "") throw new CommandError(`team ${teamKey} carried no id`);
+  return prepareApplicableLabelIds(ctx, teamId, names);
+}
+async function prepareApplicableLabelIds(ctx, teamId, names) {
+  let labels = (await ctx.client.get("/api/v1/labels")).data;
   const ids = [];
   for (const name of names) {
-    const existing = byName.get(name);
-    if (existing !== void 0) {
-      ids.push(existing);
-    } else {
-      const created = await ctx.client.post("/api/v1/labels", {
-        name,
-        color: "gray"
-      });
-      ids.push(created.data["id"]);
+    const resolved = await resolveOrCreateApplicableLabel(ctx, labels, name, teamId);
+    labels = resolved.catalog;
+    ids.push(labelId(resolved.label));
+  }
+  return [...new Set(ids)];
+}
+function applicableLabel(labels, input, teamId) {
+  const normalized = normalizeLabelName(input);
+  const named = labels.filter((label) => normalizeLabelName(label["name"]) === normalized);
+  const teamMatches = named.filter((label) => label["team_id"] === teamId);
+  if (teamMatches.length > 1) {
+    throw new CommandError(`more than one team label matches "${input}"; use a label id`);
+  }
+  if (teamMatches.length === 1) return teamMatches[0];
+  const workspaceMatches = named.filter(
+    (label) => label["team_id"] === null || label["team_id"] === void 0
+  );
+  if (workspaceMatches.length > 1) {
+    throw new CommandError(`more than one workspace label matches "${input}"; use a label id`);
+  }
+  return workspaceMatches[0];
+}
+async function resolveOrCreateApplicableLabel(ctx, catalog, name, teamId) {
+  const existing = applicableLabel(catalog, name, teamId);
+  if (existing !== void 0) return { label: existing, catalog };
+  try {
+    const created = await ctx.client.post("/api/v1/labels", {
+      name,
+      color: "gray"
+    });
+    return { label: created.data, catalog: [...catalog, created.data] };
+  } catch (error) {
+    if (!(error instanceof HttpError) || error.status !== 422) throw error;
+    const refreshed = (await ctx.client.get("/api/v1/labels")).data;
+    const winner = applicableLabel(refreshed, name, teamId);
+    if (winner === void 0) throw error;
+    return { label: winner, catalog: refreshed };
+  }
+}
+function labelId(label) {
+  const id = str2(label["id"]);
+  if (id === "") throw new CommandError("the label response carried no id");
+  return id;
+}
+async function prepareLabelUpdate(ctx, identifier, addInputs, removeInputs) {
+  const issue = (await ctx.client.get(`/api/v1/issues/${identifier}`)).data;
+  const attached = Array.isArray(issue["labels"]) ? issue["labels"] : [];
+  const originalIds = attached.map(labelId);
+  const removed = /* @__PURE__ */ new Set();
+  for (const input of removeInputs) {
+    const idInput = UUID_FORMAT.test(input);
+    const exact = attached.find((label) => {
+      const id = labelId(label);
+      return idInput ? id.toLowerCase() === input.toLowerCase() : id === input;
+    });
+    if (exact !== void 0) {
+      removed.add(labelId(exact));
+      continue;
+    }
+    const normalized = normalizeLabelName(input);
+    const matches = attached.filter((label) => normalizeLabelName(label["name"]) === normalized);
+    if (matches.length > 1) {
+      throw new CommandError(`more than one attached label matches "${input}"; remove by label id`);
+    }
+    if (matches.length === 1) removed.add(labelId(matches[0]));
+  }
+  const ids = originalIds.filter((id) => !removed.has(id));
+  if (addInputs.length > 0) {
+    const team = issue["team"] ?? {};
+    const teamId = str2(team["id"]);
+    if (teamId === "") throw new CommandError(`issue ${identifier} carried no team id`);
+    let catalog = (await ctx.client.get("/api/v1/labels")).data;
+    for (const input of addInputs) {
+      const idInput = UUID_FORMAT.test(input);
+      let chosen = idInput ? catalog.find((label) => labelId(label).toLowerCase() === input.toLowerCase()) : void 0;
+      if (chosen !== void 0) {
+        const owner = chosen["team_id"];
+        if (owner !== null && owner !== void 0 && owner !== teamId) {
+          throw new CommandError(
+            `label ${input} belongs to another team and cannot be added to ${identifier}`
+          );
+        }
+      } else {
+        if (idInput) throw new CommandError(`No label matches id "${input}".`);
+        const resolved = await resolveOrCreateApplicableLabel(ctx, catalog, input, teamId);
+        chosen = resolved.label;
+        catalog = resolved.catalog;
+      }
+      const id = labelId(chosen);
+      if (!ids.includes(id)) ids.push(id);
     }
   }
-  const body = await ctx.client.put(`/api/v1/issues/${str2(issue["identifier"])}/labels`, {
-    label_ids: ids
-  });
-  return body.data;
+  return {
+    issue,
+    ids,
+    changed: ids.length !== originalIds.length || ids.some((id, index) => id !== originalIds[index])
+  };
 }
 async function fetchIssues(ctx, params, opts) {
   if (opts.all === true && opts.page !== void 0) {

package/dist/cli.js

@@ -70,7 +70,7 @@ import {
   version,
   workflowStates,
   writeCredentials
-} from "./chunk-CIW4OU5G.js";
+} from "./chunk-DQDNAG5E.js";
 
 // src/cli.ts
 import { createInterface as createInterface2 } from "readline";
@@ -1208,10 +1208,15 @@ function requireNonBlankTitle(text, titleFile) {
     titleFile === void 0 ? "--title is empty \u2014 an issue needs a title" : `--title-file ${titleFile} produced an empty title`
   );
 }
-program.command("create").description("create an issue").option("--title <text>", "issue title (required)").option("--description <text>", "problem statement").option("--design <text>", "design notes").option("--notes <text>", "working notes").option("--acceptance <text>", "acceptance criteria").option("--title-file <path>", 'read the title from a file ("-" for stdin)').option("--description-file <path>", 'read the description from a file ("-" for stdin)').option("--design-file <path>", 'read the design notes from a file ("-" for stdin)').option("--notes-file <path>", 'read the notes from a file ("-" for stdin)').option("--acceptance-file <path>", 'read the acceptance criteria from a file ("-" for stdin)').option(
+program.command("create").description("create an issue atomically after resolving requested labels").option("--title <text>", "issue title (required)").option("--description <text>", "problem statement").option("--design <text>", "design notes").option("--notes <text>", "working notes").option("--acceptance <text>", "acceptance criteria").option("--title-file <path>", 'read the title from a file ("-" for stdin)').option("--description-file <path>", 'read the description from a file ("-" for stdin)').option("--design-file <path>", 'read the design notes from a file ("-" for stdin)').option("--notes-file <path>", 'read the notes from a file ("-" for stdin)').option("--acceptance-file <path>", 'read the acceptance criteria from a file ("-" for stdin)').option(
   "--type <type>",
   "task | bug | feature | epic (applied as a type/ label)"
-).option("--priority <name>", "urgent | high | medium | low | none").option("--due-date <date>", "YYYY-MM-DD").option("--parent <id>", "parent epic").option("--assignee <who>", "me, a user id, an email, or a full name").option("--team <key>", "team key (or HARNESS_TEAM)").option("--label <name>", "label to apply (repeatable)", collect, []).option("--json", "emit JSON").action(async (opts) => {
+).option("--priority <name>", "urgent | high | medium | low | none").option("--due-date <date>", "YYYY-MM-DD").option("--parent <id>", "parent epic").option("--assignee <who>", "me, a user id, an email, or a full name").option("--team <key>", "team key (or HARNESS_TEAM)").option(
+  "--label <name>",
+  "resolve team-first, then workspace; create missing workspace labels before the issue (repeatable)",
+  collect,
+  []
+).option("--json", "emit JSON").action(async (opts) => {
   await withConfig(opts.json === true, true, (ctx) => {
     if (opts.title === void 0 && opts.titleFile === void 0) {
       throw new UsageError(
@@ -1227,6 +1232,16 @@ program.command("create").description("create an issue").option("--title <text>"
 program.command("update").description("update an issue; --claim takes it atomically").argument("[id]").option("--claim", "claim atomically and move to the first started state").option("--state <name>", "workflow state by name or type").option("--title <text>").option("--description <text>").option("--design <text>").option("--notes <text>", "replace the notes").option("--append-notes <text>", "append to the notes").option("--acceptance <text>").option("--title-file <path>", 'read the title from a file ("-" for stdin)').option("--description-file <path>", 'read the description from a file ("-" for stdin)').option("--design-file <path>", 'read the design notes from a file ("-" for stdin)').option("--notes-file <path>", 'replace the notes from a file ("-" for stdin)').option("--append-notes-file <path>", 'append to the notes from a file ("-" for stdin)').option("--acceptance-file <path>", 'read the acceptance criteria from a file ("-" for stdin)').option("--priority <name>", "urgent | high | medium | low | none").option("--due-date <date>", 'YYYY-MM-DD; "" clears').option("--parent <id>").option(
   "--assignee <who>",
   'me, a user id, an email, or a full name; "" unassigns'
+).option(
+  "--add-label <name-or-id>",
+  "add a label without replacing existing labels (repeatable)",
+  collect,
+  []
+).option(
+  "--remove-label <name-or-id>",
+  "remove an attached label without replacing others (repeatable)",
+  collect,
+  []
 ).option("--json", "emit JSON").action(
   async (id, opts) => {
     await withConfig(opts.json === true, true, (ctx) => {
@@ -1552,7 +1567,7 @@ program.command("upload").description("upload a file; prints markdown to referen
   });
 });
 program.command("mcp").description("run the MCP server over stdio (for agents)").action(async () => {
-  const { startMcpServer } = await import("./server-J2V5JXFY.js");
+  const { startMcpServer } = await import("./server-WBZ5OPHK.js");
   await startMcpServer();
 });
 program.command("login").description("sign in via the browser (infra.one session)").option("--url <url>", "host to sign in to").option("--no-browser", "print the URL and prompt for a pasted token instead").option(

package/dist/{server-J2V5JXFY.js → server-WBZ5OPHK.js}

@@ -50,7 +50,7 @@ import {
   upload,
   userList,
   workflowStates
-} from "./chunk-CIW4OU5G.js";
+} from "./chunk-DQDNAG5E.js";
 
 // src/mcp/server.ts
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
@@ -189,13 +189,17 @@ var TOOLS = [
       due_date: dateField,
       state: z.string().optional().describe("workflow state by name or type"),
       parent: z.string().optional(),
-      assignee: z.string().optional().describe('"" unassigns')
+      assignee: z.string().optional().describe('"" unassigns'),
+      add_labels: z.array(z.string()).optional().describe("label names or ids to add without replacing existing labels"),
+      remove_labels: z.array(z.string()).optional().describe("attached label names or ids to remove without replacing others")
     },
     handler: (ctx, args) => update(ctx, args["id"], {
       ...args,
       acceptance: args["acceptance_criteria"],
       appendNotes: args["append_notes"],
-      dueDate: args["due_date"]
+      dueDate: args["due_date"],
+      addLabel: args["add_labels"],
+      removeLabel: args["remove_labels"]
     })
   },
   {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@allocator-one/harness-cli",
-  "version": "1.5.2",
+  "version": "1.6.0",
   "description": "CLI and MCP server for Harness, allocator.one's issue tracker",
   "type": "module",
   "license": "MIT",

package/spec/openapi.json

@@ -1038,6 +1038,14 @@
                     "format": "date",
                     "type": "string"
                   },
+                  "label_ids": {
+                    "description": "Optional label IDs from the workspace or the same team; attached atomically with the issue",
+                    "items": {
+                      "format": "uuid",
+                      "type": "string"
+                    },
+                    "type": "array"
+                  },
                   "notes": {
                     "description": "Working notes (markdown)",
                     "type": "string"