schub 0.1.3 → 0.1.4

package/src/components/StatusView.tsx

@@ -1,420 +0,0 @@
-import { dirname, normalize, sep } from "node:path";
-import { Box, Text, useInput } from "ink";
-import React from "react";
-import { type ChangeInfo, listChanges, updateChangeStatus } from "../changes";
-import { findSchubRoot, loadTaskDependencies, type TaskInfo, trimTaskTitle } from "../features/tasks";
-import { openInVsCode } from "../ide";
-import { launchOpencodeReview } from "../opencode";
-
-type StatusSortable = {
-  id: string;
-  title: string;
-  status: string;
-};
-
-type Shortcut = {
-  keyLabel: string;
-  label: string;
-};
-
-type StatusViewProps = {
-  refreshIntervalMs?: number;
-  onCopyId: (id: string) => void;
-  onReview?: (changeId: string) => void;
-  onShortcutsChange?: (shortcuts: Shortcut[]) => void;
-};
-
-const DEFAULT_REFRESH_INTERVAL_MS = 1000;
-const ACTIVE_TASK_STATUSES = ["blocked", "wip", "ready", "backlog"] as const;
-const CHANGE_TASK_STATUSES = [...ACTIVE_TASK_STATUSES, "done"] as const;
-
-type ChangeTaskCount = {
-  completed: number;
-  total: number;
-};
-
-const AUTO_MARK_STATUSES = new Set(["accepted", "wip"]);
-const REVIEW_SHORTCUT: Shortcut = { keyLabel: "r", label: "review" };
-
-const compareText = (left: string, right: string): number =>
-  left.localeCompare(right, undefined, { sensitivity: "base" });
-
-const sortByStatusThenTitle = (left: StatusSortable, right: StatusSortable): number => {
-  const statusCompare = compareText(left.status, right.status);
-  if (statusCompare !== 0) {
-    return statusCompare;
-  }
-
-  const titleCompare = compareText(left.title, right.title);
-  if (titleCompare !== 0) {
-    return titleCompare;
-  }
-
-  return compareText(left.id, right.id);
-};
-
-const isTaskItem = (item: ChangeInfo | TaskInfo): item is TaskInfo => {
-  const parts = normalize(item.path).split(sep);
-  return parts.includes("tasks");
-};
-
-const formatChangeId = (value: string) => {
-  const match = value.match(/^([Cc]\d{4})_/);
-  return match ? match[1].toUpperCase() : value;
-};
-
-const formatChangeTitle = (changeId: string, title: string) => {
-  const trimmedTitle = title.trim();
-  if (trimmedTitle !== changeId) {
-    return trimmedTitle;
-  }
-
-  const match = changeId.match(/^([Cc]\d{4})_(.+)$/);
-  if (!match) {
-    return trimmedTitle;
-  }
-
-  return match[2]
-    .split("-")
-    .map((word) => word.charAt(0).toUpperCase() + word.slice(1))
-    .join(" ");
-};
-
-const canAutoMarkChange = (status: string) => AUTO_MARK_STATUSES.has(status.trim().toLowerCase());
-
-const autoMarkChangesDone = (schubDir: string) => {
-  const allTasks = loadTaskDependencies(schubDir, CHANGE_TASK_STATUSES);
-  const changeTaskCounts = new Map<string, ChangeTaskCount>();
-
-  for (const task of allTasks) {
-    if (!task.changeId) {
-      continue;
-    }
-
-    const current = changeTaskCounts.get(task.changeId) ?? { completed: 0, total: 0 };
-    const completed = current.completed + (task.status === "done" ? 1 : 0);
-    changeTaskCounts.set(task.changeId, { completed, total: current.total + 1 });
-  }
-
-  if (changeTaskCounts.size === 0) {
-    return;
-  }
-
-  for (const change of listChanges(schubDir)) {
-    const counts = changeTaskCounts.get(change.id);
-    if (!counts || counts.total === 0 || counts.completed !== counts.total) {
-      continue;
-    }
-
-    if (canAutoMarkChange(change.status)) {
-      updateChangeStatus(schubDir, change.id, "Done");
-    }
-  }
-};
-
-const buildStatusData = (schubDir: string | null) => {
-  if (!schubDir) {
-    return {
-      pendingReview: [],
-      pendingImplementation: [],
-      pendingImplementationNoTasks: [],
-      drafts: [],
-      blocked: [],
-      wip: [],
-      ready: [],
-      backlog: [],
-      pendingImplementationCounts: new Map<string, string>(),
-    };
-  }
-
-  const allTasks = loadTaskDependencies(schubDir, CHANGE_TASK_STATUSES);
-
-  const blockedTasks: TaskInfo[] = [];
-  const wipTasks: TaskInfo[] = [];
-  const readyTasks: TaskInfo[] = [];
-  const backlogTasks: TaskInfo[] = [];
-
-  const activeChangeIds = new Set<string>();
-  const changeTaskCounts = new Map<string, ChangeTaskCount>();
-
-  for (const task of allTasks) {
-    if (task.status === "blocked") blockedTasks.push(task);
-    else if (task.status === "wip") wipTasks.push(task);
-    else if (task.status === "ready") readyTasks.push(task);
-    else if (task.status === "backlog") backlogTasks.push(task);
-
-    if (task.changeId && task.status !== "done") {
-      activeChangeIds.add(task.changeId);
-    }
-
-    if (task.changeId) {
-      const current = changeTaskCounts.get(task.changeId) ?? { completed: 0, total: 0 };
-      const completed = current.completed + (task.status === "done" ? 1 : 0);
-      changeTaskCounts.set(task.changeId, { completed, total: current.total + 1 });
-    }
-  }
-
-  const allChanges = listChanges(schubDir);
-  const pendingReviewChanges: ChangeInfo[] = [];
-  const pendingImplementationChanges: ChangeInfo[] = [];
-  const draftChanges: ChangeInfo[] = [];
-
-  for (const change of allChanges) {
-    const normalized = change.status.toLowerCase();
-    if (normalized.includes("review")) {
-      pendingReviewChanges.push(change);
-      continue;
-    }
-
-    const counts = changeTaskCounts.get(change.id);
-    const allTasksCompleted = Boolean(counts && counts.total > 0 && counts.completed === counts.total);
-
-    if (activeChangeIds.has(change.id)) {
-      pendingImplementationChanges.push(change);
-      continue;
-    }
-
-    if ((normalized.includes("implementing") || normalized.includes("accepted")) && !allTasksCompleted) {
-      pendingImplementationChanges.push(change);
-      continue;
-    }
-
-    if (normalized.includes("draft")) {
-      draftChanges.push(change);
-    }
-  }
-
-  const pendingImplementationWithTasks: ChangeInfo[] = [];
-  const pendingImplementationNoTasks: ChangeInfo[] = [];
-  const pendingImplementationCounts = new Map<string, string>();
-
-  for (const change of pendingImplementationChanges.sort(sortByStatusThenTitle)) {
-    const counts = changeTaskCounts.get(change.id);
-    if (counts && counts.total > 0) {
-      pendingImplementationWithTasks.push(change);
-      pendingImplementationCounts.set(change.id, `${counts.completed}/${counts.total}`);
-    } else {
-      pendingImplementationNoTasks.push(change);
-    }
-  }
-
-  return {
-    pendingReview: pendingReviewChanges.sort(sortByStatusThenTitle),
-    pendingImplementation: pendingImplementationWithTasks,
-    pendingImplementationNoTasks,
-    drafts: draftChanges.sort(sortByStatusThenTitle),
-    blocked: blockedTasks.sort(sortByStatusThenTitle),
-    wip: wipTasks.sort(sortByStatusThenTitle),
-    ready: readyTasks.sort(sortByStatusThenTitle),
-    backlog: backlogTasks.sort(sortByStatusThenTitle),
-    pendingImplementationCounts,
-  };
-};
-
-export default function StatusView({
-  refreshIntervalMs = DEFAULT_REFRESH_INTERVAL_MS,
-  onCopyId,
-  onReview = launchOpencodeReview,
-  onShortcutsChange,
-}: StatusViewProps) {
-  const schubDir = findSchubRoot();
-  const [, setRefreshTick] = React.useState(0);
-  const {
-    pendingReview,
-    pendingImplementation,
-    pendingImplementationNoTasks,
-    drafts,
-    blocked,
-    wip,
-    ready,
-    backlog,
-    pendingImplementationCounts,
-  } = buildStatusData(schubDir);
-  const repoRoot = schubDir ? dirname(schubDir) : "";
-
-  const allItems = [
-    ...pendingReview,
-    ...pendingImplementation,
-    ...pendingImplementationNoTasks,
-    ...drafts,
-    ...blocked,
-    ...wip,
-    ...ready,
-    ...backlog,
-  ];
-
-  const [selection, setSelection] = React.useState(0);
-  const totalItems = allItems.length;
-  const pendingReviewIds = new Set(pendingReview.map((change) => change.id));
-  const selectedItem = totalItems > 0 ? allItems[selection] : null;
-  const selectedReviewId =
-    selectedItem && !isTaskItem(selectedItem) && pendingReviewIds.has(selectedItem.id) ? selectedItem.id : null;
-  const lastReviewIdRef = React.useRef<string | null>(null);
-
-  React.useEffect(() => {
-    if (!onShortcutsChange) {
-      return;
-    }
-
-    if (lastReviewIdRef.current === selectedReviewId) {
-      return;
-    }
-
-    lastReviewIdRef.current = selectedReviewId;
-    onShortcutsChange(selectedReviewId ? [REVIEW_SHORTCUT] : []);
-  }, [onShortcutsChange, selectedReviewId]);
-
-  React.useEffect(() => {
-    if (!schubDir) {
-      return;
-    }
-
-    const interval = setInterval(() => {
-      autoMarkChangesDone(schubDir);
-      setRefreshTick((current) => current + 1);
-    }, refreshIntervalMs);
-
-    return () => {
-      clearInterval(interval);
-    };
-  }, [refreshIntervalMs, schubDir]);
-
-  React.useEffect(() => {
-    if (totalItems === 0) {
-      setSelection(0);
-      return;
-    }
-    setSelection((current) => Math.min(current, totalItems - 1));
-  }, [totalItems]);
-
-  useInput((input, key) => {
-    if (totalItems === 0) {
-      return;
-    }
-
-    if (key.downArrow) {
-      setSelection((current) => Math.min(current + 1, totalItems - 1));
-    }
-
-    if (key.upArrow) {
-      setSelection((current) => Math.max(current - 1, 0));
-    }
-
-    if (input === "o") {
-      const selectedItem = allItems[selection];
-      openInVsCode(repoRoot, selectedItem.path);
-    }
-
-    if (input === "c") {
-      const selectedItem = allItems[selection];
-      onCopyId(selectedItem.id);
-    }
-
-    if (input === "r" && selectedReviewId) {
-      onReview(selectedReviewId);
-    }
-  });
-
-  if (!schubDir) {
-    return (
-      <Box flexDirection="column">
-        <Text bold>Status</Text>
-        <Text color="red">No .schub directory found.</Text>
-      </Box>
-    );
-  }
-
-  if (totalItems === 0) {
-    return (
-      <Box flexDirection="column">
-        <Text color="gray">No active changes or tasks found.</Text>
-      </Box>
-    );
-  }
-
-  const renderRow = (item: ChangeInfo | TaskInfo, index: number, detail?: string) => {
-    const selected = index === selection;
-    const task = isTaskItem(item) ? item : null;
-    const title = task ? trimTaskTitle(task.title) : "";
-    const checklistIndicator =
-      task && task.status === "wip" && typeof task.checklistTotal === "number"
-        ? ` (${task.checklistRemaining ?? 0}/${task.checklistTotal})`
-        : "";
-    const changeTitle = task ? "" : formatChangeTitle(item.id, item.title);
-    const changeDetail = detail ? ` ${detail}` : "";
-    const displayTitle = task ? `${title}${checklistIndicator}` : `${changeTitle}${changeDetail}`.trim();
-    const displayId = task ? item.id : formatChangeId(item.id);
-
-    return (
-      <Box key={item.id} marginLeft={1}>
-        <Text color={selected ? "blue" : "gray"}>{selected ? "›" : " "}</Text>
-        <Box marginLeft={1}>
-          <Text color="white" bold={selected}>
-            {displayId}
-          </Text>
-          {displayTitle ? <Text color="gray"> {displayTitle}</Text> : null}
-        </Box>
-      </Box>
-    );
-  };
-
-  let currentIndex = 0;
-
-  const renderSection = (title: string, items: (ChangeInfo | TaskInfo)[], detailById?: Map<string, string>) => {
-    if (items.length === 0) return null;
-
-    const sectionStartIndex = currentIndex;
-    const sectionItems = items.map((item, i) => renderRow(item, sectionStartIndex + i, detailById?.get(item.id)));
-    currentIndex += items.length;
-
-    return (
-      <Box flexDirection="column" marginBottom={1}>
-        <Box marginBottom={0}>
-          <Text color="white">{title}</Text>
-        </Box>
-        {sectionItems}
-      </Box>
-    );
-  };
-
-  return (
-    <Box flexDirection="column">
-      <Box flexDirection="column" marginBottom={1}>
-        <Box marginBottom={1}>
-          <Text bold color="white">
-            Change Proposals
-          </Text>
-        </Box>
-        {renderSection("Pending Review", pendingReview)}
-        {renderSection("Pending Implementation", pendingImplementation, pendingImplementationCounts)}
-        {renderSection("No Tasks Defined", pendingImplementationNoTasks)}
-        {renderSection("Drafts", drafts)}
-        {pendingReview.length === 0 &&
-          pendingImplementation.length === 0 &&
-          pendingImplementationNoTasks.length === 0 &&
-          drafts.length === 0 && (
-            <Box marginLeft={1}>
-              <Text color="gray">No active proposals.</Text>
-            </Box>
-          )}
-      </Box>
-
-      <Box flexDirection="column">
-        <Box marginBottom={1}>
-          <Text bold color="white">
-            Tasks
-          </Text>
-        </Box>
-        {renderSection("Blocked", blocked)}
-        {renderSection("WIP", wip)}
-        {renderSection("Ready", ready)}
-        {renderSection("Backlog", backlog)}
-        {blocked.length === 0 && wip.length === 0 && ready.length === 0 && backlog.length === 0 && (
-          <Box marginLeft={1}>
-            <Text color="gray">No active tasks.</Text>
-          </Box>
-        )}
-      </Box>
-    </Box>
-  );
-}

package/src/ide.ts

@@ -1,7 +0,0 @@
-import { spawn } from "node:child_process";
-import { resolve } from "node:path";
-
-export const openInVsCode = (repoRoot: string, relativePath: string) => {
-  const targetPath = resolve(repoRoot, relativePath);
-  spawn("code", ["-g", targetPath], { stdio: "ignore" }).unref();
-};

package/templates/templates-parity.test.ts

@@ -1,45 +0,0 @@
-import { expect, test } from "bun:test";
-import { existsSync, readFileSync } from "node:fs";
-import { dirname, join, resolve } from "node:path";
-import { fileURLToPath } from "node:url";
-
-const testDir = dirname(fileURLToPath(import.meta.url));
-const cliDir = resolve(testDir, "..");
-const repoRoot = resolve(cliDir, "..", "..");
-
-const cliTemplates = [
-  join(cliDir, "templates", "create-proposal", "proposal-template.md"),
-  join(cliDir, "templates", "create-proposal", "adr-template.md"),
-  join(cliDir, "templates", "create-proposal", "cookbook-template.md"),
-  join(cliDir, "templates", "create-tasks", "task-template.md"),
-  join(cliDir, "templates", "setup-project", "project-overview-template.md"),
-  join(cliDir, "templates", "setup-project", "project-setup-template.md"),
-  join(cliDir, "templates", "setup-project", "project-wow-template.md"),
-  join(cliDir, "templates", "review-proposal", "review-me-template.md"),
-  join(cliDir, "templates", "review-proposal", "q&a-template.md"),
-];
-
-const legacyTemplates = [
-  join(repoRoot, "skills", "create-proposal", "assets", "proposal-template.md"),
-  join(repoRoot, "skills", "create-proposal", "assets", "adr-template.md"),
-  join(repoRoot, "skills", "create-tasks", "assets", "task-template.md"),
-  join(repoRoot, "skills", "setup-project", "assets", "project-overview-template.md"),
-  join(repoRoot, "skills", "setup-project", "assets", "project-setup-template.md"),
-  join(repoRoot, "skills", "setup-project", "assets", "project-wow-template.md"),
-  join(repoRoot, "skills", "setup-project", "assets", "review-me-template.md"),
-  join(repoRoot, "skills", "review-proposal", "assets", "review-me-template.md"),
-  join(repoRoot, "skills", "review-proposal", "assets", "q&a-template.md"),
-];
-
-test("cli templates exist and are non-empty", () => {
-  for (const templatePath of cliTemplates) {
-    const contents = readFileSync(templatePath, "utf8");
-    expect(contents.trim().length).toBeGreaterThan(0);
-  }
-});
-
-test("legacy skill template assets are removed", () => {
-  for (const legacyPath of legacyTemplates) {
-    expect(existsSync(legacyPath)).toBe(false);
-  }
-});