@harborppm/cli 0.46.0 → 0.47.0

package/dist/app.mjs

@@ -2068,9 +2068,43 @@ function normalizeRaidItem(r) {
 		updatedAt: r.updatedAt ?? r.createdAt ?? now,
 		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 === "risk" && typeof r.happened === "boolean" ? { happened: r.happened } : {},
+		...kind === "decision" ? decisionFields(r) : {}
 	};
 }
+/** A revisit condition that still says something, or null. */
+function cleanRevisit(v) {
+	if (!v || typeof v !== "object") return null;
+	const r = v;
+	const str = (x) => typeof x === "string" && x ? x : null;
+	if (r.kind === "date" && str(r.on)) return {
+		kind: "date",
+		on: str(r.on).slice(0, 10)
+	};
+	if (r.kind === "risk" && str(r.riskId)) return {
+		kind: "risk",
+		riskId: str(r.riskId)
+	};
+	if (r.kind === "milestone" && str(r.milestoneId) && str(r.due)) return {
+		kind: "milestone",
+		milestoneId: str(r.milestoneId),
+		due: str(r.due).slice(0, 10)
+	};
+	return null;
+}
+/** A decision's own fields, kept only where they say something so older rows stay as they were. */
+function decisionFields(r) {
+	const out = {};
+	if (r.turnedDown?.trim()) out.turnedDown = r.turnedDown;
+	const tasks = ids$1(r.taskIds);
+	if (tasks.length) out.taskIds = tasks;
+	const risks = ids$1(r.riskIds);
+	if (risks.length) out.riskIds = risks;
+	const revisit = cleanRevisit(r.revisitIf);
+	if (revisit) out.revisitIf = revisit;
+	if (r.supersedesId) out.supersedesId = r.supersedesId;
+	return out;
+}
 /** Active first, then by score, then most recently touched. */
 function sortRaid(items) {
 	return [...items].sort((a, b) => {
@@ -2112,7 +2146,7 @@ function heatGrid(items) {
 function inHeatCell(item, probability, impact) {
 	return item.kind === "risk" && item.probability === probability && item.impact === impact;
 }
-var KIND_LABEL, STATUS_LABEL$1, STATUSES$1, PROBABILITY_LABEL, IMPACT_LABEL, FIELD_COPY;
+var KIND_LABEL, STATUS_LABEL$1, STATUSES$1, PROBABILITY_LABEL, IMPACT_LABEL, FIELD_COPY, ids$1;
 var init_raid = __esmMin((() => {
 	KIND_LABEL = {
 		risk: "Risk",
@@ -2182,6 +2216,7 @@ var init_raid = __esmMin((() => {
 			dueDate: "Decide by"
 		}
 	};
+	ids$1 = (v) => Array.isArray(v) ? [...new Set(v.filter((x) => typeof x === "string" && !!x))] : [];
 }));
 
 //#endregion
@@ -5435,6 +5470,85 @@ function freshlyOpened(data, projects, now) {
 	return out.sort((a, b) => (b.state.openedOn ?? "").localeCompare(a.state.openedOn ?? ""));
 }
 
+//#endregion
+//#region src/lib/decisions.ts
+/** A risk counts as come true when someone said so, or when it was turned into an issue. */
+function cameTrue(risk) {
+	return risk.happened === true || risk.kind === "issue";
+}
+/** "the risk “Vendor misses window” comes true", for "Revisit if …". */
+function revisitText(revisit, data) {
+	if (revisit.kind === "date") return `it's ${md$1(revisit.on)}`;
+	if (revisit.kind === "risk") {
+		const risk = byId(data.raidItems).get(revisit.riskId);
+		return risk ? `“${risk.title}” comes true` : "a risk that's since been removed comes true";
+	}
+	const m = byId(data.milestones).get(revisit.milestoneId);
+	return m ? `“${m.name}” slips past ${md$1(revisit.due)}` : "a milestone that's since been removed slips";
+}
+/** Why a decided decision is due another look today, in a sentence, or null while it isn't. */
+function revisitReason(item, data, today) {
+	const revisit = item.revisitIf;
+	if (item.kind !== "decision" || item.status !== "decided" || !revisit) return null;
+	if (revisit.kind === "date") return today >= revisit.on ? `${md$1(revisit.on)} came, the day set to look again.` : null;
+	if (revisit.kind === "risk") {
+		const risk = byId(data.raidItems).get(revisit.riskId);
+		return risk && cameTrue(risk) ? `The risk “${risk.title}” came true.` : null;
+	}
+	const m = byId(data.milestones).get(revisit.milestoneId);
+	if (!m || m.done) return null;
+	const due = m.dueDate.slice(0, 10);
+	if (due > revisit.due) return `“${m.name}” moved from ${md$1(revisit.due)} to ${md$1(due)}.`;
+	if (due < today) return `“${m.name}” is past its date, ${md$1(due)}.`;
+	return null;
+}
+/** The decision that replaced this one, if any. */
+function supersededBy(data, id) {
+	return data.raidItems.find((r) => r.kind === "decision" && r.supersedesId === id) ?? null;
+}
+/** Decided decisions in scope whose "revisit if" has happened, oldest decision first. */
+function decisionsToRevisit(data, inScope, today) {
+	const out = [];
+	for (const item of data.raidItems) {
+		if (item.kind !== "decision" || !item.revisitIf || !inScope(item)) continue;
+		const reason = revisitReason(item, data, today);
+		if (!reason) continue;
+		const where = item.projectId && byId(data.projects).get(item.projectId)?.name || item.programId && byId(data.programs).get(item.programId)?.name || item.portfolioId && byId(data.portfolios).get(item.portfolioId)?.name || null;
+		out.push({
+			item,
+			reason,
+			where
+		});
+	}
+	return out.sort((a, b) => (a.item.decidedAt ?? a.item.createdAt).localeCompare(b.item.decidedAt ?? b.item.createdAt));
+}
+/** The comment a "still stands" leaves, so the thread says it was looked at. */
+function stillStandsNote(reason) {
+	return reason ? `Looked at again (${reason.replace(/\.$/, "")}): it still stands.` : "Looked at again: it still stands.";
+}
+/** Fields a new decision copies from the one it replaces: where it sits, what it touches. */
+function successorOf(old) {
+	return {
+		kind: "decision",
+		title: old.title,
+		description: old.description,
+		portfolioId: old.portfolioId ?? null,
+		programId: old.programId ?? null,
+		projectId: old.projectId ?? null,
+		ownerId: old.ownerId,
+		taskIds: old.taskIds,
+		riskIds: old.riskIds,
+		supersedesId: old.id,
+		status: "proposed"
+	};
+}
+var md$1;
+var init_decisions = __esmMin((() => {
+	init_indexes();
+	init_date_text();
+	md$1 = (iso) => format(parseIso(iso.slice(0, 10)), "MMM d");
+}));
+
 //#endregion
 //#region src/lib/brief.ts
 init_date_fns();
@@ -5445,6 +5559,7 @@ init_work();
 init_indexes();
 init_blockers();
 init_clock();
+init_decisions();
 const BRIEF_QUIET = "Nothing late or at risk. Say what shipped — or enjoy the quiet week.";
 /** Late tasks worst first, each with what it holds up, so the letter says what to chase. */
 function lateLinesOf(data, tasks, now) {
@@ -5503,14 +5618,18 @@ function isHighRisk(r) {
 function isMajorIssue(r) {
 	return r.kind === "issue" && (r.impact != null ? r.impact >= 4 : r.severity === "high");
 }
-/** The risks and issues worth a line in the brief: live, and either a high-scored risk or a major issue. */
-function watchListOf(data, scope, limit = 5) {
-	const inScope = (r) => {
+/** Whether a register item sits inside the brief's portfolio (or anywhere, for all of them). */
+function inBriefScope(scope) {
+	return (r) => {
 		if (r.projectId) return scope.projectIds.has(r.projectId);
 		if (r.programId) return scope.programIds.has(r.programId);
 		if (r.portfolioId) return scope.portfolioId == null || r.portfolioId === scope.portfolioId;
 		return scope.portfolioId == null;
 	};
+}
+/** The risks and issues worth a line in the brief: live, and either a high-scored risk or a major issue. */
+function watchListOf(data, scope, limit = 5) {
+	const inScope = inBriefScope(scope);
 	const live = data.raidItems.filter((r) => r.kind !== "decision" && isActive(r) && inScope(r) && (r.kind === "risk" && isHighRisk(r) || isMajorIssue(r)));
 	return sortRaid(live).slice(0, limit).map((item) => ({
 		item,
@@ -5546,6 +5665,11 @@ function composeBriefBody(opts) {
 		}
 		lines.push("");
 	}
+	if (opts.revisit?.length) {
+		lines.push("Decisions to look at again");
+		for (const d of opts.revisit) lines.push(`• ${d.title}${d.where ? ` (${d.where})` : ""}: ${d.reason}`);
+		lines.push("");
+	}
 	const done = () => {
 		if (!opts.shipped?.length) return;
 		lines.push("Done this week");
@@ -5790,6 +5914,11 @@ function briefOf(data, opts = {}) {
 		said: saidThisWeek(data, now, portfolioId ? new Set(tasks.flatMap((t) => t.assigneeId ? [t.assigneeId] : [])) : null),
 		late: lateLinesOf(data, tasks, now),
 		opened: freshlyOpened(data, projects, now),
+		revisit: decisionsToRevisit(data, inBriefScope({
+			programIds,
+			projectIds,
+			portfolioId
+		}), todayISO(now)),
 		talk,
 		milestonesSoon,
 		milestonesLate,
@@ -5841,6 +5970,11 @@ function composeBriefBodyFrom(brief, data, workspace, all = false) {
 			project: b.project.name,
 			who: data.people.find((p) => p.id === b.note.authorId)?.name ?? null,
 			body: b.note.body
+		})),
+		revisit: brief.revisit.map((d) => ({
+			title: d.item.title,
+			reason: d.reason,
+			where: d.where
 		}))
 	});
 }
@@ -7685,6 +7819,7 @@ init_baseline();
 init_derived_health();
 init_date_text();
 init_format();
+init_decisions();
 init_swatches();
 init_raid();
 /** The people of this harbor to pick from, with "Nobody" first when that's allowed. */
@@ -8511,30 +8646,54 @@ function RaidPanel({ id, width, height, go, active, onClose }) {
 			label: "Score",
 			value: score != null ? String(score) : ""
 		}] : [],
-		...item.kind === "decision" ? [{
-			label: "Decided by",
-			value: person(item.decidedById) ?? "Not yet",
-			edit: {
-				kind: "pick",
-				choices: [{
-					label: "Not yet",
-					value: null
-				}, ...people.map((p) => ({
-					label: p.name,
-					value: p.id
-				}))],
-				initial: Math.max(0, people.findIndex((p) => p.id === item.decidedById) + 1),
-				onPick: (v) => up({ decidedById: v })
-			}
-		}, {
-			label: "Decided on",
-			value: item.decidedAt ? shortDay(item.decidedAt, today()) : "Not yet",
-			edit: {
-				kind: "date",
-				initial: item.decidedAt ?? null,
-				onSave: (v) => up({ decidedAt: v })
+		...item.kind === "decision" ? [
+			{
+				label: "Decided by",
+				value: person(item.decidedById) ?? "Not yet",
+				edit: {
+					kind: "pick",
+					choices: [{
+						label: "Not yet",
+						value: null
+					}, ...people.map((p) => ({
+						label: p.name,
+						value: p.id
+					}))],
+					initial: Math.max(0, people.findIndex((p) => p.id === item.decidedById) + 1),
+					onPick: (v) => up({ decidedById: v })
+				}
+			},
+			{
+				label: "Decided on",
+				value: item.decidedAt ? shortDay(item.decidedAt, today()) : "Not yet",
+				edit: {
+					kind: "date",
+					initial: item.decidedAt ?? null,
+					onSave: (v) => up({ decidedAt: v })
+				}
+			},
+			{
+				label: "Turned down",
+				value: (item.turnedDown ?? "").replace(/\s+/g, " "),
+				edit: {
+					kind: "text",
+					initial: item.turnedDown ?? "",
+					onSave: (v) => up({ turnedDown: v })
+				}
+			},
+			{
+				label: "Look again",
+				value: item.revisitIf ? revisitText(item.revisitIf, store.getState()) : "",
+				edit: {
+					kind: "date",
+					initial: item.revisitIf?.kind === "date" ? item.revisitIf.on : null,
+					onSave: (v) => up({ revisitIf: v ? {
+						kind: "date",
+						on: v
+					} : null })
+				}
 			}
-		}] : [],
+		] : [],
 		{
 			label: copy.dueDate,
 			value: shortDay(item.dueDate, today()),
@@ -14721,6 +14880,42 @@ function milestonesRaidSlice({ set, get, write }) {
 				updatedAt: stamp()
 			} : r) });
 		},
+		supersedeDecision: (id) => {
+			const old = get().raidItems.find((r) => r.id === id && r.kind === "decision");
+			if (!old) return null;
+			const now = stamp();
+			const row = normalizeRaidItem({
+				...successorOf(old),
+				id: uid("raid"),
+				createdById: get().currentUserId,
+				createdAt: now,
+				updatedAt: now
+			});
+			write({ raidItems: [...get().raidItems.map((r) => r.id === id ? {
+				...r,
+				status: "superseded",
+				updatedAt: now
+			} : r), row] });
+			get().log(`changed a decision — ${old.title}`);
+			return row.id;
+		},
+		decisionStillStands: (id, reason) => {
+			const item = get().raidItems.find((r) => r.id === id);
+			if (!item || item.kind !== "decision") return;
+			const now = stamp();
+			const update = {
+				id: uid("ru"),
+				authorId: get().currentUserId,
+				body: stillStandsNote(reason ?? null),
+				createdAt: now
+			};
+			write({ raidItems: get().raidItems.map((r) => r.id === id ? normalizeRaidItem({
+				...r,
+				revisitIf: null,
+				updates: [...r.updates, update],
+				updatedAt: now
+			}) : r) });
+		},
 		startPremortem: (projectId) => {
 			const project = get().projects.find((p) => p.id === projectId);
 			if (!project || project.premortem) return;
@@ -14864,6 +15059,7 @@ var init_milestones_raid = __esmMin((() => {
 	init_clock();
 	init_premortem();
 	init_raid();
+	init_decisions();
 	init_raid_history();
 	init_utils();
 	init_shared();
@@ -23121,6 +23317,7 @@ function voyageLog(data, projectId) {
 			guessed: true
 		};
 	});
+	const decisions = raid.filter((r) => r.kind === "decision" && r.status !== "proposed").sort((a, b) => (a.decidedAt ?? a.createdAt).localeCompare(b.decidedAt ?? b.createdAt));
 	const counts = /* @__PURE__ */ new Map();
 	for (const t of tasks) if (t.status === "done" && t.assigneeId) counts.set(t.assigneeId, (counts.get(t.assigneeId) ?? 0) + 1);
 	const people = new Map(data.people.map((p) => [p.id, p]));
@@ -23142,6 +23339,7 @@ function voyageLog(data, projectId) {
 		slips,
 		risks,
 		issues: issues.length,
+		decisions,
 		carriers,
 		budget: project.budget,
 		spent: project.spent,
@@ -23189,6 +23387,10 @@ function voyageStory(log) {
 		out.push(s.waitedOn ? `${late}; it waited on “${s.waitedOn.title}”, done ${md(s.waitedOn.completedAt)}.` : `${late}.`);
 	}
 	for (const r of log.risks.filter((x) => x.happened)) out.push(`The risk “${r.risk.title}” came true${r.guessed ? " (an issue names it)" : ""}.`);
+	if (log.decisions.length) {
+		const changed = log.decisions.filter((d) => d.status === "superseded").length;
+		out.push(`${plural(log.decisions.length, "decision was", "decisions were")} written down${changed ? `; ${changed === 1 ? "one was" : `${changed} were`} changed later` : ""}.`);
+	}
 	const top = log.carriers[0];
 	if (top && log.carriers.length > 1) out.push(`${top.person.name} closed ${plural(top.closed, "task")}, the most on the crew.`);
 	else if (top) out.push(`${top.person.name} closed ${plural(top.closed, "task")}.`);
@@ -23233,6 +23435,7 @@ init_hill();
 init_raid();
 init_priority();
 init_premortem();
+init_decisions();
 init_search();
 init_derived_health();
 init_format();
@@ -23440,10 +23643,7 @@ function raidOut(data, r) {
 		...text("contingency"),
 		...text("resolution"),
 		...text("rationale"),
-		...r.kind === "decision" ? {
-			decidedBy: nameOf(people, r.decidedById),
-			decidedOn: dayOf(r.decidedAt)
-		} : {},
+		...r.kind === "decision" ? decisionOut(data, r, people) : {},
 		...text("notes"),
 		updates: r.updates.map((u) => ({
 			by: nameOf(people, u.authorId) ?? "Someone",
@@ -23452,6 +23652,39 @@ function raidOut(data, r) {
 		}))
 	};
 }
+/** A decision's own fields: who and when, what was turned down, what it touches, when to look again. */
+function decisionOut(data, r, people) {
+	const tasks = byId(data.tasks);
+	const items = byId(data.raidItems);
+	const touches = [...(r.taskIds ?? []).flatMap((id) => {
+		const t = tasks.get(id);
+		if (!t) return [];
+		const key = taskKey(t, data.projects.find((p) => p.id === t.projectId));
+		return [key ? `${key} ${t.title}` : t.title];
+	}), ...(r.riskIds ?? []).flatMap((id) => {
+		const x = items.get(id);
+		return x ? [`${KIND_LABEL[x.kind]}: ${x.title}`] : [];
+	})];
+	const older = r.supersedesId ? items.get(r.supersedesId) : null;
+	const newer = supersededBy(data, r.id);
+	const due = revisitReason(r, data, todayISO());
+	return {
+		decidedBy: nameOf(people, r.decidedById),
+		decidedOn: dayOf(r.decidedAt),
+		...r.turnedDown?.trim() ? { turnedDown: r.turnedDown.trim() } : {},
+		...touches.length ? { touches } : {},
+		...r.revisitIf ? { revisitIf: revisitText(r.revisitIf, data) } : {},
+		...due ? { lookAgainNow: due } : {},
+		...older ? { replaces: {
+			id: older.id,
+			title: older.title
+		} } : {},
+		...newer ? { replacedBy: {
+			id: newer.id,
+			title: newer.title
+		} } : {}
+	};
+}
 const tool = (t) => t;
 /** A project's notes for later, as get_project gives them: sealed ones without their text unless they're the caller's. */
 function notesOf(ctx, p) {
@@ -24089,6 +24322,12 @@ const HARBOR_TOOLS = [
 					cameTrue: r.happened,
 					...r.guessed ? { guessed: true } : {}
 				})),
+				decisions: log.decisions.map((d) => ({
+					title: d.title,
+					on: dayOf(d.decidedAt),
+					...d.rationale?.trim() ? { why: d.rationale.trim() } : {},
+					...d.status === "superseded" ? { changedLater: true } : {}
+				})),
 				closedBy: log.carriers.map((c) => ({
 					who: c.person.name,
 					tasks: c.closed

package/package.json

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