@harborppm/cli 0.50.0 → 0.51.0

package/README.md

@@ -155,9 +155,11 @@ harbor risk "TITLE" --project P|--program P [--impact 1-5] [--probability 1-5]
 harbor issue "TITLE" --project P|--program P [--impact 1-5] [--owner NAME]
                     [--due DATE]
 harbor decision "TITLE" --project P|--program P [--owner NAME] [--due DATE]
+                       [--turned-down T] [--look-again DATE] [--supersedes "OLD"]
 harbor risks [words] [--kind risk,issue,decision] [--project P] [--closed]
 harbor raid "TITLE" [--status S] [--owner me|none|NAME] [--impact 1-5] [--probability 1-5]
-                   [--due DATE|none] [--update TEXT] [--mitigation T] [--resolution T] …
+                   [--due DATE|none] [--update TEXT] [--mitigation T] [--resolution T]
+                   [--turned-down T] [--look-again DATE|none] …
                                             one risk, issue or decision in full, or change it
 harbor review [WORDS] [--mine|--owner NAME] [--portfolio P] [--done]
                                             the review: programs worst first, with next moves

package/dist/app.mjs

@@ -2069,7 +2069,8 @@ function normalizeRaidItem(r) {
 		updates: Array.isArray(r.updates) ? r.updates : [],
 		scoreHistory: Array.isArray(r.scoreHistory) && r.scoreHistory.length ? r.scoreHistory : void 0,
 		...kind === "risk" && typeof r.happened === "boolean" ? { happened: r.happened } : {},
-		...kind === "decision" ? decisionFields(r) : {}
+		...kind === "decision" ? decisionFields(r) : {},
+		...ids$1(r.followerIds).length ? { followerIds: ids$1(r.followerIds) } : {}
 	};
 }
 /** A revisit condition that still says something, or null. */
@@ -14719,6 +14720,24 @@ var init_assignment = __esmMin((() => {}));
 //#region src/lib/slices/tasks.ts
 function tasksSlice({ set, get, write }) {
 	return {
+		followWork: (personId, taskIds, raidIds = []) => {
+			const tasks = new Set(taskIds);
+			const raid = new Set(raidIds);
+			const add = (ids) => ids?.includes(personId) ? ids : [...ids ?? [], personId];
+			write({
+				tasks: get().tasks.map((t) => tasks.has(t.id) && t.assigneeId !== personId ? {
+					...t,
+					followerIds: add(t.followerIds)
+				} : t),
+				raidItems: get().raidItems.map((r) => raid.has(r.id) && r.ownerId !== personId ? {
+					...r,
+					followerIds: add(r.followerIds),
+					updatedAt: stamp()
+				} : r)
+			});
+			const who = get().people.find((p) => p.id === personId);
+			get().log(`added ${who?.name ?? "someone"} as a second pair of hands on ${taskIds.length + raidIds.length} item${taskIds.length + raidIds.length === 1 ? "" : "s"}`, "/people");
+		},
 		addTask: (t, opts) => {
 			if (!opts?.allowOverCap) {
 				if (columnWip(get(), t.projectId, t.status, 1)) return "";

package/lib/cli.mjs

@@ -142,6 +142,8 @@ export const HELP = `harbor: Harbor from the terminal
   harbor issue "TITLE" --project P|--program P [--impact 1-5] [--owner NAME]
                        [--due DATE]
   harbor decision "TITLE" --project P|--program P [--owner NAME] [--due DATE]
+                         [--turned-down T] [--look-again DATE]
+                         [--supersedes "OLD DECISION"]
   harbor risks [words] [--kind risk,issue,decision] [--project P] [--closed]
                                               the register, worst first
                                               (--closed adds closed ones)
@@ -149,7 +151,7 @@ export const HELP = `harbor: Harbor from the terminal
                      [--probability 1-5] [--due DATE|none] [--update TEXT]
                      [--title T] [--description T] [--mitigation T]
                      [--contingency T] [--resolution T] [--rationale T]
-                     [--notes T]
+                     [--notes T] [--turned-down T] [--look-again DATE|none]
                                               one risk, issue or decision in
                                               full, or change it
   harbor review [WORDS] [--mine|--owner NAME] [--portfolio P] [--done]
@@ -743,7 +745,8 @@ const DETAIL = {
     "  Also --owner me|none|NAME, --impact 1-5, --probability 1-5 (risks),",
     "  --due DATE|none (review by, or decide by), --title, --description,",
     "  --mitigation and --contingency (risks), --resolution (issues and",
-    "  decisions), --rationale (decisions) and --notes. TITLE is its title or",
+    "  decisions), --rationale, --turned-down and --look-again DATE|none",
+    "  (decisions) and --notes. TITLE is its title or",
     "  enough of it to pick one out; harbor risks lists them. Changes several",
     "  at once, as one change.",
   ],
@@ -794,6 +797,12 @@ const DETAIL = {
     '  harbor decision "TITLE" --project P    writes: logs a decision to make',
     '  harbor decision "TITLE" --program P    writes: logs one on the program',
     "      [--owner NAME] [--due DATE] (the date to decide by)",
+    "      [--turned-down T] (what was turned down, and why not)",
+    "      [--look-again DATE] (when it comes back to the top of the brief)",
+    '  harbor decision "TITLE" --supersedes "OLD"',
+    "                                         writes: a decision that replaces an",
+    "                                         earlier one, which reads as",
+    "                                         superseded",
   ],
 };
 
@@ -850,6 +859,9 @@ function distance(a, b) {
 /** A mistyped command, with the one it was probably meant to be. */
 export function noCommand(name) {
   if (name.startsWith("-")) return `${name} goes after a command, like harbor mine ${name}.`;
+  // A plural kind reads the register, not the command that writes one.
+  if (["decisions", "issues"].includes(name))
+    return `No command "${name}". harbor risks --kind ${name.slice(0, -1)} lists them.`;
   const near = COMMANDS.filter((c) => distance(name, c) <= (c.length > 4 ? 2 : 1));
   const guess = near.length === 1 ? ` Did you mean harbor ${near[0]}?` : "";
   return `No command "${name}".${guess} Run harbor help.`;
@@ -1963,7 +1975,7 @@ export async function run(argv, io = {}) {
           includeClosed: Boolean(values.closed),
           limit: 50,
         },
-        (r) => raidList(r, positionals.join(" ")),
+        (r) => raidList(r, positionals.join(" "), kind),
       );
     }
     case "report": {
@@ -2292,6 +2304,8 @@ export async function run(argv, io = {}) {
             "update",
             "decided-by",
             "decided-on",
+            "turned-down",
+            "look-again",
             ...fields,
           ],
         ),
@@ -2309,6 +2323,9 @@ export async function run(argv, io = {}) {
         decidedBy: values["decided-by"],
         decidedOn:
           values["decided-on"] === undefined ? undefined : nullable(dueDate(values["decided-on"])),
+        turnedDown: values["turned-down"],
+        revisitOn:
+          values["look-again"] === undefined ? undefined : nullable(dueDate(values["look-again"])),
         update: values.update,
         ...Object.fromEntries(fields.map((f) => [f, values[f]])),
       };
@@ -2370,11 +2387,15 @@ export async function run(argv, io = {}) {
         spec(
           [],
           ["project", "program", "owner", "due"].concat(
-            command === "risk" ? ["impact", "probability"] : command === "issue" ? ["impact"] : [],
+            command === "risk"
+              ? ["impact", "probability"]
+              : command === "issue"
+                ? ["impact"]
+                : ["supersedes", "turned-down", "look-again"],
           ),
         ),
       );
-      if (!values.project && !values.program) {
+      if (!values.project && !values.program && !values.supersedes) {
         throw new HarborError("Say where it belongs: --project NAME or --program NAME.");
       }
       return call(
@@ -2389,8 +2410,14 @@ export async function run(argv, io = {}) {
           impact: level(values.impact, "impact"),
           probability: level(values.probability, "probability"),
           due: dueDate(values.due),
+          supersedes: values.supersedes,
+          turnedDown: values["turned-down"],
+          revisitOn: values["look-again"] === undefined ? undefined : dueDate(values["look-again"]),
         },
-        (r) => `Logged the ${command} “${r.raised.title}”.`,
+        (r) =>
+          r.replaces
+            ? `Logged the decision “${r.raised.title}”; it replaces “${r.replaces.title}”.`
+            : `Logged the ${command} “${r.raised.title}”.`,
       );
     }
     default:

package/lib/format.mjs

@@ -246,12 +246,19 @@ export function todayLocal(d = new Date()) {
 }
 
 /** The register, worst first: kind, title, then status, scores, owner and where it sits. */
-export function raidList(r, words = "") {
+export function raidList(r, words = "", kinds = []) {
   const want = words.trim().toLowerCase();
   const items = (r.items ?? []).filter(
     (x) => !want || `${x.title} ${x.description ?? ""}`.toLowerCase().includes(want),
   );
-  if (!items.length) return want ? "Nothing in the register matches." : "The register is clear.";
+  if (!items.length) {
+    if (want) return "Nothing in the register matches.";
+    const only =
+      kinds?.length === 1
+        ? { risk: "risks", issue: "issues", decision: "decisions" }[kinds[0]]
+        : null;
+    return only ? `No ${only} in the register.` : "The register is clear.";
+  }
   const lines = items.map((x) => {
     const bits = [
       x.status?.toLowerCase(),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@harborppm/cli",
-  "version": "0.50.0",
+  "version": "0.51.0",
   "description": "Harbor from the terminal: a full-screen app with everything, plus commands for scripts and agents.",
   "keywords": [
     "harbor",