@dieulc/workflow 0.9.0 → 0.9.2

package/utils.ts

@@ -931,6 +931,48 @@ export function isPlanWritePath(p: string, cwd: string): boolean {
   return norm.includes(".pi/plans/") || norm.startsWith(cwdNorm + "/.pi/plans");
 }
 
+/**
+ * Whether two plan paths identify the same file.
+ *
+ * Plan targets arrive in several shapes: absolute Windows paths from a
+ * `tool_result`, relative paths from `/review <path>`, forward slashes from a
+ * handoff, and case differences from the host. A review-cycle boundary must
+ * not fire for a spelling difference, and must fire for a real move. Kept in
+ * this dependency-free module so the comparison is unit-testable; callers pass
+ * the session cwd for relative paths.
+ */
+export function samePlanPath(
+  a: string | null | undefined,
+  b: string | null | undefined,
+  cwd: string = process.cwd(),
+): boolean {
+  const normalize = (raw: string | null | undefined): string => {
+    let value = String(raw ?? "").trim();
+    if (!value) return "";
+    value = value.replace(/\\/g, "/");
+    const absolute =
+      value.startsWith("/") || /^[A-Za-z]:\//.test(value);
+    if (!absolute) {
+      value = `${String(cwd ?? "").replace(/\\/g, "/").replace(/\/+$/, "")}/${value}`;
+    }
+    const parts: string[] = [];
+    for (const part of value.split("/")) {
+      if (!part || part === ".") continue;
+      if (part === "..") {
+        if (parts.length > 0) parts.pop();
+        continue;
+      }
+      parts.push(part);
+    }
+    let out = parts.join("/");
+    if (value.startsWith("/")) out = `/${out}`;
+    // Windows drive letters are case-insensitive; keep POSIX case-sensitive.
+    out = out.replace(/^([A-Za-z]):/, (_m, drive: string) => `${drive.toUpperCase()}:`);
+    return process.platform === "win32" ? out.toLowerCase() : out;
+  };
+  return normalize(a) === normalize(b);
+}
+
 /** Inputs for deciding whether this run may hand a plan off. */
 export interface PlanHandoffEligibility {
   mode: "plan" | "build" | null;

package/workflow-status.ts

@@ -0,0 +1,81 @@
+/**
+ * Workflow footer segments (2026-10-31 UI plan, step 10).
+ *
+ * The extension used to publish one long `workflow` status containing the mode,
+ * progress, profile/role, commit/write policy and review text. Desktop hosts can
+ * only lay an arbitrary status string out as one column, and terminal Pi wraps
+ * the same line less efficiently. This module builds short, stable segments so
+ * each can be its own footer column.
+ *
+ * `workflow` stays the mode/progress string: it is what the Desktop parser
+ * (`src/shared/workflow-mode.ts`) and existing terminal Pi users already know.
+ * The other keys are additive.
+ */
+
+export const WORKFLOW_STATUS_KEYS = {
+  mode: "workflow",
+  profile: "workflow-profile",
+  policy: "workflow-policy",
+  review: "workflow-review",
+} as const;
+
+export interface WorkflowStatusInput {
+  mode: "build" | "plan" | null;
+  /** Plannotator's Plan sub-state, shown inside the mode segment. */
+  plannotatorActive?: boolean;
+  done: number;
+  total: number;
+  stale?: boolean;
+  profileLabel?: string;
+  roleLabel?: string;
+  commitMode: string;
+  /** Only Default mode has a write policy; null/undefined omits it. */
+  writeMode?: string | null;
+}
+
+export interface WorkflowStatusSegments {
+  mode: string;
+  profile: string;
+  policy: string;
+}
+
+export function buildWorkflowStatusSegments(
+  input: WorkflowStatusInput,
+): WorkflowStatusSegments {
+  const done = Number.isFinite(input.done) ? Math.max(0, input.done) : 0;
+  const total = Number.isFinite(input.total) ? Math.max(0, input.total) : 0;
+  const stale = input.stale ? " ⚠" : "";
+  const profile = [input.profileLabel ?? "", input.roleLabel ?? ""]
+    .map((part) => part.trim())
+    .filter(Boolean)
+    .join(" | ");
+  const policy =
+    `commits:${input.commitMode}` +
+    (input.writeMode ? ` · writes:${input.writeMode}` : "");
+
+  let mode: string;
+  if (input.mode === "build") {
+    mode = total > 0 ? `▶ build ${done}/${total}${stale}` : `▶ build${stale}`;
+  } else if (input.mode === "plan") {
+    const base = input.plannotatorActive
+      ? "⏸ plan — reviewing in browser"
+      : "⏸ plan";
+    mode = total > 0 ? `${base} ${done}/${total}${stale}` : `${base}${stale}`;
+  } else if (total > 0) {
+    // The default mode must stay parseable by the Desktop badge decoder.
+    mode = `◦ ${done}/${total}${stale}`;
+  } else {
+    mode = `◦ commits:${input.commitMode}${stale}`;
+  }
+
+  return { mode, profile, policy };
+}
+
+/** Sets a status only while it has text; `undefined` removes a stale key. */
+export function setWorkflowStatus(
+  ui: { setStatus?: (key: string, text: string | undefined) => void },
+  key: string,
+  text: string,
+): void {
+  ui.setStatus?.(key, text.length > 0 ? text : undefined);
+}