@staff0rd/assist 0.224.1 → 0.226.0

package/README.md

@@ -111,8 +111,10 @@ The first backlog command in a repository that still has a local `.assist/backlo
 - `assist backlog add` - Add a new backlog item interactively (prompts for type: story/bug)
 - `assist backlog add --name <n> --type <t> --desc <d> --ac <criterion...>` - Add a backlog item from CLI options (used by `/draft`)
 - `assist backlog add-phase <id> <name> --task <t...> [--manual-check <c...>] [--position <pos>]` - Add a phase (appends by default; `--position` inserts at a 1-indexed position)
-- `assist backlog update <id> [--name <n>] [--desc <d>] [--type <t>] [--ac <criterion...>]` - Update fields on a backlog item
+- `assist backlog update-field <id> [--name <n>] [--desc <d>] [--type <t>] [--ac <criterion...>]` - Update fields on a backlog item
+- `assist backlog update-field <id> [--add-ac <text>] [--edit-ac <n> <text>] [--remove-ac <n>]` - Granular acceptance-criteria edits using 1-based indices matching `backlog show`: `--add-ac` appends (repeatable), `--edit-ac` replaces criterion n in place, `--remove-ac` deletes criterion n and renumbers the rest (cannot be combined with the whole-list `--ac`)
 - `assist backlog update-phase <id> <phase> [--name <n>] [--task <t...>] [--manual-check <c...>]` - Modify a plan phase (name, tasks, or manual checks)
+- `assist backlog update-phase <id> <phase> [--add-task <text>] [--edit-task <n> <text>] [--remove-task <n>] [--add-check <text>] [--edit-check <n> <text>] [--remove-check <n>]` - Granular task and manual-check edits using 1-based indices matching `backlog show`: `--add-*` appends (repeatable), `--edit-*` replaces entry n in place, `--remove-*` deletes entry n and renumbers the rest (task ops cannot be combined with `--task`; check ops cannot be combined with `--manual-check`)
 - `assist backlog remove-phase <id> <phase>` - Remove a plan phase from a backlog item
 - `assist backlog next` - Pick and run the next backlog item, or open `/draft` if none remain
 - `assist backlog start <id>` - Set a backlog item to in-progress

package/claude/commands/refine.md

@@ -35,10 +35,10 @@ Based on the user's input, apply changes using the appropriate commands. Always
 
 **To update item fields:**
 ```
-assist backlog update <id> --name "New name"
-assist backlog update <id> --desc "New description"
-assist backlog update <id> --type story
-assist backlog update <id> --ac "criterion 1" --ac "criterion 2"
+assist backlog update-field <id> --name "New name"
+assist backlog update-field <id> --desc "New description"
+assist backlog update-field <id> --type story
+assist backlog update-field <id> --ac "criterion 1" --ac "criterion 2"
 ```
 
 Note: `--ac` replaces all acceptance criteria, so include the full list (both existing and new).

package/dist/index.js

@@ -6,7 +6,7 @@ import { Command } from "commander";
 // package.json
 var package_default = {
   name: "@staff0rd/assist",
-  version: "0.224.1",
+  version: "0.226.0",
   type: "module",
   main: "dist/index.js",
   bin: {
@@ -5597,6 +5597,78 @@ async function removePhase(id, phase) {
 // src/commands/backlog/update/index.ts
 import chalk67 from "chalk";
 
+// src/commands/backlog/update/parseListIndex.ts
+function parseListIndex(raw, length, label2) {
+  if (!/^\d+$/.test(raw)) {
+    return { ok: false, error: `${label2} must be a positive integer.` };
+  }
+  const index = Number.parseInt(raw, 10);
+  if (index < 1 || index > length) {
+    return {
+      ok: false,
+      error: `${label2} ${index} is out of range (1-${length}).`
+    };
+  }
+  return { ok: true, index };
+}
+
+// src/commands/backlog/update/applyListMutations.ts
+function hasListMutations(options2) {
+  return Boolean(
+    options2.add && options2.add.length > 0 || options2.edit || options2.remove
+  );
+}
+function applyListMutations(current, options2, flags) {
+  let items = [...current];
+  if (options2.edit) {
+    const [rawIndex, ...textParts] = options2.edit;
+    if (rawIndex === void 0 || textParts.length === 0) {
+      return {
+        ok: false,
+        error: `${flags.edit} requires an index and replacement text.`
+      };
+    }
+    const parsed = parseListIndex(
+      rawIndex,
+      items.length,
+      `${flags.edit} index`
+    );
+    if (!parsed.ok) return parsed;
+    items[parsed.index - 1] = textParts.join(" ");
+  }
+  if (options2.remove) {
+    const parsed = parseListIndex(
+      options2.remove,
+      items.length,
+      `${flags.remove} index`
+    );
+    if (!parsed.ok) return parsed;
+    items = items.filter((_, i) => i !== parsed.index - 1);
+  }
+  if (options2.add) {
+    items.push(...options2.add);
+  }
+  return { ok: true, items };
+}
+
+// src/commands/backlog/update/applyAcMutations.ts
+function hasAcMutations(options2) {
+  return hasListMutations({
+    add: options2.addAc,
+    edit: options2.editAc,
+    remove: options2.removeAc
+  });
+}
+function applyAcMutations(current, options2) {
+  const result = applyListMutations(
+    current,
+    { add: options2.addAc, edit: options2.editAc, remove: options2.removeAc },
+    { add: "--add-ac", edit: "--edit-ac", remove: "--remove-ac" }
+  );
+  if (!result.ok) return result;
+  return { ok: true, criteria: result.items };
+}
+
 // src/commands/backlog/update/buildUpdateSql.ts
 import chalk66 from "chalk";
 function buildUpdateSql(options2) {
@@ -5641,7 +5713,24 @@ function buildUpdateSql(options2) {
 async function update(id, options2) {
   const result = await loadAndFindItem(id);
   if (!result) return;
-  const built = buildUpdateSql(options2);
+  let ac = options2.ac;
+  if (hasAcMutations(options2)) {
+    if (options2.ac) {
+      console.log(
+        chalk67.red("Cannot combine --ac with --add-ac/--edit-ac/--remove-ac.")
+      );
+      process.exitCode = 1;
+      return;
+    }
+    const mutation = applyAcMutations(result.item.acceptanceCriteria, options2);
+    if (!mutation.ok) {
+      console.log(chalk67.red(mutation.error));
+      process.exitCode = 1;
+      return;
+    }
+    ac = mutation.criteria;
+  }
+  const built = buildUpdateSql({ ...options2, ac });
   if (!built) return;
   const db = await getBacklogDb();
   const itemId = result.item.id;
@@ -5685,17 +5774,81 @@ async function applyPhaseUpdate(db, itemId, phaseIdx, fields) {
   });
 }
 
+// src/commands/backlog/update/resolveListUpdate.ts
+function resolveListUpdate(whole, current, mutations, wholeFlag, flags) {
+  if (!hasListMutations(mutations)) return { ok: true, items: whole };
+  if (whole) {
+    return {
+      ok: false,
+      error: `Cannot combine ${wholeFlag} with ${flags.add}/${flags.edit}/${flags.remove}.`
+    };
+  }
+  return applyListMutations(current, mutations, flags);
+}
+
+// src/commands/backlog/resolvePhaseFields.ts
+var TASK_FLAGS = {
+  add: "--add-task",
+  edit: "--edit-task",
+  remove: "--remove-task"
+};
+var CHECK_FLAGS = {
+  add: "--add-check",
+  edit: "--edit-check",
+  remove: "--remove-check"
+};
+function resolveTasks(o, current) {
+  return resolveListUpdate(
+    o.task,
+    (current?.tasks ?? []).map((t) => t.task),
+    { add: o.addTask, edit: o.editTask, remove: o.removeTask },
+    "--task",
+    TASK_FLAGS
+  );
+}
+function resolveChecks(o, current) {
+  return resolveListUpdate(
+    o.manualCheck,
+    current?.manualChecks ?? [],
+    { add: o.addCheck, edit: o.editCheck, remove: o.removeCheck },
+    "--manual-check",
+    CHECK_FLAGS
+  );
+}
+function buildFields(name, tasks, checks) {
+  const fields = {
+    name,
+    task: tasks.items,
+    manualCheck: checks.items
+  };
+  if (!fields.name && !fields.task && !fields.manualCheck) {
+    return {
+      ok: false,
+      error: "Nothing to update. Provide at least one flag."
+    };
+  }
+  return { ok: true, fields };
+}
+function resolvePhaseFields(options2, current) {
+  const tasks = resolveTasks(options2, current);
+  if (!tasks.ok) return tasks;
+  const checks = resolveChecks(options2, current);
+  if (!checks.ok) return checks;
+  return buildFields(options2.name, tasks, checks);
+}
+
 // src/commands/backlog/updatePhase.ts
 async function updatePhase(id, phase, options2) {
-  const { name, task, manualCheck } = options2;
-  if (!name && !task && !manualCheck) {
-    console.log(chalk68.red("Nothing to update. Provide at least one flag."));
+  const found = await findPhase(id, phase);
+  if (!found) return;
+  const { result, db, itemId, phaseIdx } = found;
+  const resolved = resolvePhaseFields(options2, result.item.plan?.[phaseIdx]);
+  if (!resolved.ok) {
+    console.log(chalk68.red(resolved.error));
     process.exitCode = 1;
     return;
   }
-  const found = await findPhase(id, phase);
-  if (!found) return;
-  const { db, itemId, phaseIdx } = found;
+  const { name, task, manualCheck } = resolved.fields;
   await applyPhaseUpdate(db, itemId, phaseIdx, { name, task, manualCheck });
   const fields = [
     name && "name",
@@ -5710,9 +5863,28 @@ async function updatePhase(id, phase, options2) {
 }
 
 // src/commands/backlog/registerUpdateCommands.ts
+function collect(value, previous) {
+  return [...previous, value];
+}
 function registerUpdateCommands(cmd) {
-  cmd.command("update <id>").description("Update fields on a backlog item").option("--name <name>", "New item name").option("--desc <description>", "New description").option("--type <type>", "New type (story or bug)").option("--ac <criterion...>", "Replace acceptance criteria (repeatable)").action(update);
-  cmd.command("update-phase <id> <phase>").description("Modify a plan phase on a backlog item").option("--name <name>", "New phase name").option("--task <task...>", "Replace tasks (repeatable)").option("--manual-check <check...>", "Replace manual checks (repeatable)").action(updatePhase);
+  cmd.command("update-field <id>").description("Update fields on a backlog item").option("--name <name>", "New item name").option("--desc <description>", "New description").option("--type <type>", "New type (story or bug)").option("--ac <criterion...>", "Replace acceptance criteria (repeatable)").option(
+    "--add-ac <text>",
+    "Append one acceptance criterion (repeatable)",
+    collect,
+    []
+  ).option(
+    "--edit-ac <n> <text...>",
+    "Replace acceptance criterion n (1-based) in place"
+  ).option("--remove-ac <n>", "Remove acceptance criterion n (1-based)").action(update);
+  cmd.command("update-phase <id> <phase>").description("Modify a plan phase on a backlog item").option("--name <name>", "New phase name").option("--task <task...>", "Replace tasks (repeatable)").option("--manual-check <check...>", "Replace manual checks (repeatable)").option("--add-task <text>", "Append one task (repeatable)", collect, []).option("--edit-task <n> <text...>", "Replace task n (1-based) in place").option("--remove-task <n>", "Remove task n (1-based)").option(
+    "--add-check <text>",
+    "Append one manual check (repeatable)",
+    collect,
+    []
+  ).option(
+    "--edit-check <n> <text...>",
+    "Replace manual check n (1-based) in place"
+  ).option("--remove-check <n>", "Remove manual check n (1-based)").action(updatePhase);
   cmd.command("remove-phase <id> <phase>").description("Remove a plan phase from a backlog item").action(removePhase);
 }
 
@@ -6194,6 +6366,30 @@ function matchesConfigDeny(command) {
 }
 
 // src/commands/cliHook/resolvePermission.ts
+var SUBCOMMAND_READS = [
+  {
+    prefix: "assist complexity",
+    subcommands: ["maintainability", "cyclomatic", "halstead"]
+  }
+];
+function matchesSubcommandRead(part) {
+  return SUBCOMMAND_READS.find(
+    (rule) => part === rule.prefix || part.startsWith(`${rule.prefix} `)
+  );
+}
+function findSubcommandAdvice(parts) {
+  if (parts.length <= 1) return void 0;
+  for (const part of parts) {
+    const rule = matchesSubcommandRead(part);
+    if (rule) {
+      return {
+        permissionDecision: "deny",
+        permissionDecisionReason: `Do not pipe or chain '${rule.prefix}'. Run a focused sub-command directly for targeted output: ${rule.subcommands.map((s) => `${rule.prefix} ${s} <file>`).join(", ")}.`
+      };
+    }
+  }
+  return void 0;
+}
 function findDeny(toolName, parts) {
   for (const part of parts) {
     const configDeny = matchesConfigDeny(part);
@@ -6204,6 +6400,8 @@ function findDeny(toolName, parts) {
       };
     }
   }
+  const subcommandAdvice = findSubcommandAdvice(parts);
+  if (subcommandAdvice) return subcommandAdvice;
   for (const part of parts) {
     const denied = matchesDeny(toolName, part);
     if (denied) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@staff0rd/assist",
-	"version": "0.224.1",
+	"version": "0.226.0",
 	"type": "module",
 	"main": "dist/index.js",
 	"bin": {