@harborppm/cli 0.71.0 → 0.73.0

package/lib/cli.mjs

@@ -112,7 +112,8 @@ export const HELP = `harbor: Harbor from the terminal
   harbor hill [--program P]                   where each project is on its hill
   harbor hill PROJECT --at 0-100|top|done     move it: up to 50 is figuring it
                                               out, past 50 is getting it done
-  harbor mine                                 what's on you, late first
+  harbor mine                                 what's on you, late first, and
+                                              what you're also on
   harbor catchup [--since DATE]               while you were away: handed to
                                               you, decided, what moved
   harbor clock [NAME] [--zone ZONE|here|none] [--hours 9-17]
@@ -123,8 +124,11 @@ export const HELP = `harbor: Harbor from the terminal
                                               the first 20 (--all: every one);
                                               done ones only with --status done
   harbor show TASK                            one task (TASK: a key like RR-42),
-                                              with the date its owner's queue
-                                              suggests
+                                              who's on it, its files, and the
+                                              date its owner's queue suggests
+  harbor files PROJECT [--kind image|doc|link]
+                                              every file on a project's tasks,
+                                              newest first
   harbor open [TASK|PROJECT]                  open it on the website (or Today)
   harbor invite EMAIL [--viewer]              invite someone to your harbor
   harbor prompt [--shell SHELL] [--refresh]   due today and late, for your
@@ -157,7 +161,8 @@ export const HELP = `harbor: Harbor from the terminal
                                               --from: plan it from a finished
                                               project as that one really ran
   harbor add "TITLE" [--project P] [--assignee me|none|NAME]
-                     [--due DATE|pilot] [--priority low|medium|high|urgent]
+                     [--also "NAME, NAME"] [--due DATE|pilot]
+                     [--priority low|medium|high|urgent]
                      [--status S] [--comment TEXT] [--over-limit] [--dry-run]
                                               with no TITLE: one title a line
                                               from stdin; --due pilot takes the
@@ -167,7 +172,7 @@ export const HELP = `harbor: Harbor from the terminal
                                               move a task to In play, In review
                                               or Done
   harbor edit TASK [--title T] [--assignee me|none|NAME]
-                   [--due DATE|pilot|none]
+                   [--also "NAME, NAME"|none] [--due DATE|pilot|none]
                    [--priority P] [--status S] [--notes TEXT]
                    [--repeat weekly|biweekly|monthly|never]
                    [--waits-on TASK,TASK|none] [--confirm] [--over-limit]
@@ -447,7 +452,7 @@ const REPEAT_WORD = { weekly: "every week", biweekly: "every two weeks", monthly
 const WEEKDAYS = ["sunday", "monday", "tuesday", "wednesday", "thursday", "friday", "saturday"];
 
 /** The one open project a task can go in (First voyage aside), or why --project is needed. */
-async function onlyProject(client, harbor) {
+async function onlyProject(client, harbor, say = "--project NAME") {
   const r = await client.call("list_projects", {
     harbor,
     health: ["on-track", "at-risk", "off-track", "paused"],
@@ -459,7 +464,7 @@ async function onlyProject(client, harbor) {
     throw new HarborError('This harbor has no open project yet; harbor project "NAME" --program P makes one.');
   const names = open.slice(0, 3).map((p) => p.name);
   const more = open.length > 3 || r.total > 5 ? ", …" : "";
-  throw new HarborError(`Say which project: --project NAME (${names.join(", ")}${more}).`);
+  throw new HarborError(`Say which project: ${say} (${names.join(", ")}${more}).`);
 }
 
 const WEEKDAY_NAME = WEEKDAYS.map((d) => d[0].toUpperCase() + d.slice(1));
@@ -608,7 +613,7 @@ export function hint(message, project) {
     )
     .replace(
       /^Nothing to change: pass a field to update\.$/,
-      "Nothing to change. Say what with --title, --assignee, --due, --priority, --status, --notes, --repeat or --waits-on.",
+      "Nothing to change. Say what with --title, --assignee, --also, --due, --priority, --status, --notes, --repeat or --waits-on.",
     )
     .replace(
       /^This connection was allowed to read only\..*$/s,
@@ -642,6 +647,24 @@ export function voyageForTerminal(r, project = r.project) {
   };
 }
 
+/** --also "Sam, Aya" as the names; none (or nobody) leaves only the owner. */
+export function alsoList(value) {
+  const v = value.trim();
+  if (/^(none|nobody)$/i.test(v)) return [];
+  const names = v
+    .split(",")
+    .map((x) => x.trim())
+    .filter(Boolean);
+  if (!names.length) throw new HarborError('Say who: --also "NAME, NAME", or --also none.');
+  return names;
+}
+
+/** One file: its name and kind, a link's address, who added it and when. */
+export function fileText(f) {
+  const kind = { image: "image", doc: "document", link: "link" }[f.kind] ?? f.kind;
+  return `${f.name} (${kind}${f.url ? `, ${f.url}` : ""}) · ${f.addedBy ?? "someone"}, ${f.added}`;
+}
+
 /** Commands whose forms do different things: one line each, saying which read and which write. */
 const DETAIL = {
   add: [
@@ -654,7 +677,8 @@ const DETAIL = {
     "  --project can be left out when the harbor has one open project (First",
     "  voyage aside); the task goes there.",
     "",
-    "  --assignee me|none|NAME, --due DATE|pilot, --priority low|medium|high|urgent,",
+    "  --assignee me|none|NAME, --also \"NAME, NAME\" (others on it with the",
+    "  owner), --due DATE|pilot, --priority low|medium|high|urgent,",
     "  --status S, --comment TEXT and --over-limit apply to every task added, so",
     "  ten lines with --comment add the same comment to all ten. More than 100",
     "  lines is harbor import's job.",
@@ -1194,11 +1218,13 @@ const DETAIL = {
     '  harbor risk "TITLE" --project P        writes: logs a risk on the project',
     '  harbor risk "TITLE" --program P        writes: logs one on the program',
     "      [--impact 1-5] [--probability 1-5] [--owner NAME] [--due DATE]",
+    "  With neither, it goes in the harbor's one open project (First voyage aside).",
   ],
   issue: [
     '  harbor issue "TITLE" --project P       writes: logs an issue on the project',
     '  harbor issue "TITLE" --program P       writes: logs one on the program',
     "      [--impact 1-5] [--owner NAME] [--due DATE]",
+    "  With neither, it goes in the harbor's one open project (First voyage aside).",
   ],
   decision: [
     '  harbor decision "TITLE" --project P    writes: logs a decision to make',
@@ -1206,6 +1232,7 @@ const DETAIL = {
     "      [--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)",
+    "  With neither, it goes in the harbor's one open project (First voyage aside).",
     '  harbor decision "TITLE" --supersedes "OLD"',
     "                                         writes: a decision that replaces an",
     "                                         earlier one, which reads as",
@@ -1288,6 +1315,9 @@ const OTHER_NAMES = {
   longpole: "harbor why PROJECT shows what's holding it up.",
   "long-pole": "harbor why PROJECT shows what's holding it up.",
   pilot: "harbor show TASK says the date its owner's queue suggests; --due pilot on add or edit takes it.",
+  milestone: "Milestones belong to a program: in the app (harbor), open the program and press m; or on the website.",
+  milestones: "The app's program screen lists them: harbor, then 4, and Enter on a program.",
+  decide: 'harbor decision "TITLE" --project P logs a decision.',
   dock: "harbor try PROJECT is the dry dock.",
   "dry-dock": "harbor try PROJECT is the dry dock.",
 };
@@ -2082,6 +2112,25 @@ export async function run(argv, io = {}) {
       out(asJson ? jsonText(result) : plain(taskList(result, opts)));
       return 0;
     }
+    case "files": {
+      const { positionals, values, harbor } = parse(spec([], ["kind"]));
+      const project = text(positionals, "the project");
+      const kind = values.kind?.trim().toLowerCase().replace(/s$/, "");
+      if (kind !== undefined && !["image", "doc", "link"].includes(kind))
+        throw new HarborError("--kind is image, doc or link.");
+      return call("list_files", { harbor, project, kind, limit: 200 }, (r) =>
+        r.files.length
+          ? [
+              `${r.project}: ${r.total} file${r.total === 1 ? "" : "s"}${kind ? ` (${kind}s)` : ""}`,
+              "",
+              ...r.files.map(
+                (f) => `${fileText(f)}  on ${f.task.key ?? f.task.id} ${f.task.title}`,
+              ),
+              ...(r.total > r.files.length ? [`…and ${r.total - r.files.length} more.`] : []),
+            ].join("\n")
+          : `No ${kind ? `${kind}s` : "files"} on ${r.project}'s tasks yet. Attach them to a task on the website.`,
+      );
+    }
     case "show": {
       const { positionals, harbor } = parse(spec());
       return call("get_task", { harbor, task: one(positionals, "task key, like RR-42") }, (t) => {
@@ -2091,6 +2140,7 @@ export async function run(argv, io = {}) {
             { ...opts, width: undefined },
           ),
         ];
+        if (t.alsoOn?.length) lines.push(`Also on it: ${t.alsoOn.join(", ")}.`);
         if (t.repeats) lines.push(`Repeats ${REPEAT_WORD[t.repeats] ?? t.repeats}.`);
         if (t.pilotDue?.fits) lines.push(`${t.pilotDue.says}, so the due date holds.`);
         else if (t.pilotDue) lines.push(...pilotLines({ pilot: t.pilotDue }, t));
@@ -2113,6 +2163,7 @@ export async function run(argv, io = {}) {
               (w) => `  ${w.key ?? w.id}  ${w.title} (${STATUS[w.status] ?? w.status})`,
             ),
           );
+        if (t.files?.length) lines.push("", "Files:", ...t.files.map((f) => `  ${fileText(f)}`));
         if (t.comments.length)
           lines.push(
             "",
@@ -2331,7 +2382,7 @@ export async function run(argv, io = {}) {
       const { positionals, values, harbor } = parse(
         spec(
           ["over-limit", "dry-run"],
-          ["project", "assignee", "due", "priority", "status", "comment"],
+          ["project", "assignee", "also", "due", "priority", "status", "comment"],
         ),
       );
       // No --project: the harbor's one open project, if that's all there is.
@@ -2407,6 +2458,7 @@ export async function run(argv, io = {}) {
             title,
             // Yours unless you say otherwise, as with n in the app.
             assignee: values.assignee ?? "me",
+            also: values.also === undefined ? undefined : alsoList(values.also),
             due: duePilot(values.due) ? undefined : dueDate(values.due),
             suggestedDue: duePilot(values.due),
             priority: values.priority,
@@ -2520,7 +2572,17 @@ export async function run(argv, io = {}) {
       const { positionals, values, harbor } = parse(
         spec(
           ["confirm", "over-limit"],
-          ["title", "assignee", "due", "priority", "status", "notes", "repeat", "waits-on"],
+          [
+            "title",
+            "assignee",
+            "also",
+            "due",
+            "priority",
+            "status",
+            "notes",
+            "repeat",
+            "waits-on",
+          ],
         ),
       );
       const waits = values["waits-on"];
@@ -2531,6 +2593,7 @@ export async function run(argv, io = {}) {
           task: one(positionals, "task key, like RR-42"),
           title: values.title,
           assignee: values.assignee,
+          also: values.also === undefined ? undefined : alsoList(values.also),
           due:
             values.due === undefined || duePilot(values.due)
               ? undefined
@@ -2555,6 +2618,11 @@ export async function run(argv, io = {}) {
         (r) =>
           [
             `Updated: ${taskLine(r.updated, opts)}`,
+            values.also !== undefined
+              ? r.updated.alsoOn?.length
+                ? `Also on it: ${r.updated.alsoOn.join(", ")}.`
+                : "Only the owner is on it now."
+              : null,
             r.updated.waitsOn ? `Waits on ${r.updated.waitsOn.join(", ")}.` : null,
             r.updated.repeat ? `Repeats ${REPEAT_WORD[r.updated.repeat]}.` : null,
             ...(values.due && !duePilot(values.due) ? pilotLines(r, r.updated) : []),
@@ -3407,8 +3475,9 @@ export async function run(argv, io = {}) {
           ),
         ),
       );
+      // Neither: the harbor's one open project, as harbor add does.
       if (!values.project && !values.program && !values.supersedes) {
-        throw new HarborError("Say where it belongs: --project NAME or --program NAME.");
+        values.project = await onlyProject(client, harbor, "--project NAME or --program NAME");
       }
       return call(
         "log_raid",

package/lib/format.mjs

@@ -39,7 +39,7 @@ export function taskLine(t, { today, mine = false, width } = {}) {
     STATUS[t.status] ?? t.status,
     t.due ? (late ? `late, due ${t.due}` : soon ? `due ${soon}` : `due ${t.due}`) : null,
     t.priority && t.priority !== "medium" ? t.priority : null,
-    mine ? null : (t.assignee ?? null),
+    mine ? null : t.assignee ? `${t.assignee}${t.alsoOn?.length ? ` +${t.alsoOn.length}` : ""}` : null,
     t.project ?? null,
     t.waitingOn ? `waits on ${t.waitingOn}` : null,
   ].filter(Boolean);
@@ -72,6 +72,8 @@ export function myWork(w, opts) {
   if (!w.you) return w.note ?? "Nothing is yours in this harbor yet.";
   const out = [`${w.you}: ${w.openTasks} open${w.late ? `, ${w.late} late` : ""}`];
   if (w.tasks?.length) out.push("", ...w.tasks.map((t) => taskLine(t, { ...opts, mine: true })));
+  if (w.alsoOn?.length)
+    out.push("", "Also on (others own these):", ...w.alsoOn.map((t) => `  ${taskLine(t, opts)}`));
   if (w.waitingOnYou?.length) {
     out.push("", "Waiting on you:");
     for (const x of w.waitingOnYou)

package/package.json

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