schub 0.1.2 → 0.1.4

package/src/tui/components/ProposalDetailView.test.tsx

@@ -0,0 +1,169 @@
+import { afterEach, expect, test } from "bun:test";
+import { mkdirSync, mkdtempSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { render as inkRender } from "ink-testing-library";
+import ProposalDetailView from "./ProposalDetailView";
+
+const ansiPattern = new RegExp(`${String.fromCharCode(27)}[[0-9;]*m`, "g");
+const stripAnsi = (value: string) => value.replace(ansiPattern, "");
+
+const renders: Array<() => void> = [];
+const render = (...args: Parameters<typeof inkRender>) => {
+  const rendered = inkRender(...args);
+  renders.push(rendered.unmount);
+  return rendered;
+};
+
+afterEach(() => {
+  for (const unmount of renders) {
+    unmount();
+  }
+  renders.length = 0;
+});
+
+const writeProposal = (changesRoot: string, changeId: string, content: string) => {
+  const changeDir = join(changesRoot, changeId);
+  mkdirSync(changeDir, { recursive: true });
+  writeFileSync(join(changeDir, "proposal.md"), content, "utf8");
+};
+
+const writeTask = (schubDir: string, status: string, id: string, slug: string, body: string) => {
+  const tasksRoot = join(schubDir, "tasks", status);
+  mkdirSync(tasksRoot, { recursive: true });
+  writeFileSync(join(tasksRoot, `${id}_${slug}.md`), body, "utf8");
+};
+
+const findLine = (output: string, id: string) => output.split("\n").find((line) => line.includes(id)) ?? "";
+
+test("proposal detail view renders metadata and summary", () => {
+  const originalCwd = process.cwd();
+  const base = mkdtempSync(join(tmpdir(), "schub-proposal-detail-"));
+  const schubDir = join(base, ".schub");
+  const changesRoot = join(schubDir, "changes");
+  const changeId = "C1200_detail-view";
+  const proposal = `---\nchange_id: ${changeId}\ncreated: 2026-01-10\nstatus: Draft\ninput: TUI detail view\n---\n# Proposal - Detail View\n\n## Summary\n\nAdd a detail view in the TUI.\n\n## Why\n\nBecause.\n`;
+
+  mkdirSync(changesRoot, { recursive: true });
+  writeProposal(changesRoot, changeId, proposal);
+
+  try {
+    process.chdir(base);
+    const { lastFrame } = render(<ProposalDetailView changeId={changeId} />);
+    const output = stripAnsi(lastFrame() ?? "");
+
+    expect(output).toContain("Change ID:");
+    expect(output).toContain(changeId);
+    expect(output).toContain("Created:");
+    expect(output).toContain("2026-01-10");
+    expect(output).toContain("Status:");
+    expect(output).toContain("Draft");
+    expect(output).toContain("Input:");
+    expect(output).toContain("TUI detail view");
+    expect(output).toContain("Summary");
+    expect(output).toContain("Add a detail view in the TUI.");
+  } finally {
+    process.chdir(originalCwd);
+  }
+});
+
+test("proposal detail view shows summary fallback when missing", () => {
+  const originalCwd = process.cwd();
+  const base = mkdtempSync(join(tmpdir(), "schub-proposal-summary-"));
+  const schubDir = join(base, ".schub");
+  const changesRoot = join(schubDir, "changes");
+  const changeId = "C1201_missing-summary";
+  const proposal = `---\nchange_id: ${changeId}\ncreated: 2026-01-11\nstatus: Draft\ninput: None\n---\n# Proposal - Missing Summary\n\n## Why\n\nBecause.\n`;
+
+  mkdirSync(changesRoot, { recursive: true });
+  writeProposal(changesRoot, changeId, proposal);
+
+  try {
+    process.chdir(base);
+    const { lastFrame } = render(<ProposalDetailView changeId={changeId} />);
+    const output = stripAnsi(lastFrame() ?? "");
+
+    expect(output).toContain("No summary provided.");
+  } finally {
+    process.chdir(originalCwd);
+  }
+});
+
+test("proposal detail view shows empty state when tasks are missing", () => {
+  const originalCwd = process.cwd();
+  const base = mkdtempSync(join(tmpdir(), "schub-proposal-empty-tasks-"));
+  const schubDir = join(base, ".schub");
+  const changesRoot = join(schubDir, "changes");
+  const changeId = "C1202_no-tasks";
+  const proposal = `---\nchange_id: ${changeId}\ncreated: 2026-01-12\nstatus: Draft\ninput: Nothing\n---\n# Proposal - Empty Tasks\n\n## Summary\n\nStill working.\n`;
+
+  mkdirSync(changesRoot, { recursive: true });
+  writeProposal(changesRoot, changeId, proposal);
+
+  try {
+    process.chdir(base);
+    const { lastFrame } = render(<ProposalDetailView changeId={changeId} />);
+    const output = stripAnsi(lastFrame() ?? "");
+
+    expect(output).toContain("No tasks found.");
+  } finally {
+    process.chdir(originalCwd);
+  }
+});
+
+test("proposal detail view highlights the selected task", () => {
+  const originalCwd = process.cwd();
+  const base = mkdtempSync(join(tmpdir(), "schub-proposal-selection-"));
+  const schubDir = join(base, ".schub");
+  const changesRoot = join(schubDir, "changes");
+  const changeId = "C1203_task-selection";
+  const proposal = `---\nchange_id: ${changeId}\ncreated: 2026-01-13\nstatus: Accepted\ninput: Selection test\n---\n# Proposal - Task Selection\n\n## Summary\n\nFocus on tasks.\n`;
+
+  mkdirSync(changesRoot, { recursive: true });
+  writeProposal(changesRoot, changeId, proposal);
+  writeTask(schubDir, "backlog", "T0100", "first-task", `---\nchange_id: ${changeId}\n---\n# Task: T0100 First Task\n`);
+  writeTask(schubDir, "ready", "T0101", "second-task", `---\nchange_id: ${changeId}\n---\n# Task: T0101 Second Task\n`);
+
+  try {
+    process.chdir(base);
+    const { lastFrame } = render(<ProposalDetailView changeId={changeId} selection={1} />);
+    const output = stripAnsi(lastFrame() ?? "");
+    const backlogLine = findLine(output, "T0100");
+    const readyLine = findLine(output, "T0101");
+
+    expect(output).toContain("Backlog");
+    expect(output).toContain("Ready");
+    expect(backlogLine).not.toContain("›");
+    expect(readyLine).toContain("›");
+  } finally {
+    process.chdir(originalCwd);
+  }
+});
+
+test("proposal detail view sorts tasks alphabetically", () => {
+  const originalCwd = process.cwd();
+  const base = mkdtempSync(join(tmpdir(), "schub-proposal-sorted-"));
+  const schubDir = join(base, ".schub");
+  const changesRoot = join(schubDir, "changes");
+  const changeId = "C1204_task-sorted";
+  const proposal = `---\nchange_id: ${changeId}\ncreated: 2026-01-13\nstatus: Accepted\ninput: Sorting test\n---\n# Proposal - Task Sorting\n\n## Summary\n\nSort tasks alphabetically.\n`;
+
+  mkdirSync(changesRoot, { recursive: true });
+  writeProposal(changesRoot, changeId, proposal);
+  writeTask(schubDir, "backlog", "T0100", "zulu-task", `---\nchange_id: ${changeId}\n---\n# Task: T0100 Zulu Task\n`);
+  writeTask(schubDir, "backlog", "T0200", "alpha-task", `---\nchange_id: ${changeId}\n---\n# Task: T0200 Alpha Task\n`);
+
+  try {
+    process.chdir(base);
+    const { lastFrame } = render(<ProposalDetailView changeId={changeId} />);
+    const output = stripAnsi(lastFrame() ?? "");
+    const alphaIndex = output.indexOf("T0200");
+    const zuluIndex = output.indexOf("T0100");
+
+    expect(alphaIndex).toBeGreaterThanOrEqual(0);
+    expect(zuluIndex).toBeGreaterThanOrEqual(0);
+    expect(alphaIndex).toBeLessThan(zuluIndex);
+  } finally {
+    process.chdir(originalCwd);
+  }
+});

package/src/tui/components/ProposalDetailView.tsx

@@ -0,0 +1,166 @@
+import { Box, Text } from "ink";
+import { readChangeDetail } from "../../changes";
+import {
+  findSchubRoot,
+  listTasksForChange,
+  TASK_STATUSES,
+  type TaskInfo,
+  type TaskStatus,
+  trimTaskTitle,
+} from "../../features/tasks";
+
+type ProposalDetailViewProps = {
+  changeId: string;
+  selection?: number;
+  startDir?: string;
+};
+
+type TaskGroup = {
+  label: string;
+  status: TaskStatus;
+  items: TaskInfo[];
+};
+
+const SUMMARY_FALLBACK = "No summary provided.";
+
+const compareText = (left: string, right: string) => left.localeCompare(right, undefined, { sensitivity: "base" });
+
+const sortTaskItems = (tasks: TaskInfo[]) =>
+  tasks.sort((left, right) => {
+    const leftTitle = trimTaskTitle(left.title) || left.id;
+    const rightTitle = trimTaskTitle(right.title) || right.id;
+    const titleCompare = compareText(leftTitle, rightTitle);
+    if (titleCompare !== 0) {
+      return titleCompare;
+    }
+    return compareText(left.id, right.id);
+  });
+
+const TASK_STATUS_LABELS: Record<TaskStatus, string> = {
+  backlog: "Backlog",
+  ready: "Ready",
+  wip: "WIP",
+  blocked: "Blocked",
+  done: "Done",
+  archived: "Archived",
+};
+
+const buildTaskGroups = (tasks: TaskInfo[]): TaskGroup[] => {
+  const groups = new Map<TaskStatus, TaskInfo[]>();
+  for (const status of TASK_STATUSES) {
+    groups.set(status, []);
+  }
+
+  for (const task of tasks) {
+    const group = groups.get(task.status);
+    if (group) {
+      group.push(task);
+    }
+  }
+
+  for (const group of groups.values()) {
+    sortTaskItems(group);
+  }
+
+  return TASK_STATUSES.map((status) => ({
+    status,
+    label: TASK_STATUS_LABELS[status],
+    items: groups.get(status) ?? [],
+  })).filter((group) => group.items.length > 0);
+};
+
+export default function ProposalDetailView({ changeId, selection = 0, startDir }: ProposalDetailViewProps) {
+  const resolvedStartDir = startDir ?? process.env.SCHUB_CWD ?? process.cwd();
+  const schubDir = findSchubRoot(resolvedStartDir);
+
+  if (!schubDir) {
+    return (
+      <Box flexDirection="column">
+        <Text color="red">No .schub directory found.</Text>
+      </Box>
+    );
+  }
+
+  const detail = readChangeDetail(schubDir, changeId);
+  const tasks = listTasksForChange(schubDir, detail.changeId);
+  const summary = detail.summary.trim() || SUMMARY_FALLBACK;
+  const taskGroups = buildTaskGroups(tasks);
+
+  const renderMetadataRow = (label: string, value: string) => (
+    <Box>
+      <Text color="white">{label}:</Text>
+      <Text color="gray"> {value}</Text>
+    </Box>
+  );
+
+  const renderTaskRow = (task: TaskInfo, index: number) => {
+    const selected = index === selection;
+    const title = trimTaskTitle(task.title);
+    const displayTitle = title ? ` ${title}` : "";
+
+    return (
+      <Box key={task.id} marginLeft={1}>
+        <Text color={selected ? "blue" : "gray"}>{selected ? "›" : " "}</Text>
+        <Box marginLeft={1}>
+          <Text color="white" bold={selected}>
+            {task.id}
+          </Text>
+          {displayTitle ? <Text color="gray">{displayTitle}</Text> : null}
+        </Box>
+      </Box>
+    );
+  };
+
+  let currentIndex = 0;
+
+  const renderTaskGroup = (group: TaskGroup) => {
+    const groupStartIndex = currentIndex;
+    const rows = group.items.map((task, index) => renderTaskRow(task, groupStartIndex + index));
+    currentIndex += group.items.length;
+
+    return (
+      <Box key={group.status} flexDirection="column" marginBottom={1}>
+        <Box marginBottom={0}>
+          <Text color="white">{group.label}</Text>
+        </Box>
+        {rows}
+      </Box>
+    );
+  };
+
+  return (
+    <Box flexDirection="column">
+      <Box marginBottom={1}>
+        <Text bold color="white">
+          Proposal Detail
+        </Text>
+      </Box>
+      <Box flexDirection="column" marginBottom={1}>
+        {renderMetadataRow("Change ID", detail.changeId)}
+        {renderMetadataRow("Created", detail.created)}
+        {renderMetadataRow("Status", detail.status)}
+        {renderMetadataRow("Input", detail.input)}
+      </Box>
+      <Box flexDirection="column" marginBottom={1}>
+        <Text color="white">Summary</Text>
+        <Box marginLeft={1}>
+          <Text color="gray">{summary}</Text>
+        </Box>
+      </Box>
+      <Box flexDirection="column">
+        <Box marginBottom={1}>
+          <Text bold color="white">
+            Tasks
+          </Text>
+        </Box>
+        {taskGroups.length === 0 ? (
+          <Box marginLeft={1}>
+            <Text color="gray">No tasks found.</Text>
+          </Box>
+        ) : (
+          taskGroups.map((group) => renderTaskGroup(group))
+        )}
+      </Box>
+    </Box>
+  );
+}

package/src/tui/components/RoadmapView.test.tsx

@@ -0,0 +1,85 @@
+import { afterEach, expect, test } from "bun:test";
+import { mkdirSync, mkdtempSync, readFileSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { render as inkRender } from "ink-testing-library";
+import RoadmapView from "./RoadmapView";
+
+const ansiPattern = new RegExp(`${String.fromCharCode(27)}[[0-9;]*m`, "g");
+const stripAnsi = (value: string) => value.replace(ansiPattern, "");
+
+const renders: Array<() => void> = [];
+const render = (...args: Parameters<typeof inkRender>) => {
+  const rendered = inkRender(...args);
+  renders.push(rendered.unmount);
+  return rendered;
+};
+
+afterEach(() => {
+  for (const unmount of renders) {
+    unmount();
+  }
+  renders.length = 0;
+});
+
+const nextTick = () => new Promise((resolve) => setTimeout(resolve, 0));
+
+const createRoadmapWorkspace = () => {
+  const base = mkdtempSync(join(tmpdir(), "schub-roadmap-view-"));
+  const schubDir = join(base, ".schub");
+  mkdirSync(schubDir, { recursive: true });
+  const roadmapPath = join(schubDir, "roadmap.md");
+  writeFileSync(roadmapPath, ["## Roadmap", "", "- **C0001**: As a user, I want to sign in.", ""].join("\n"), "utf8");
+  return { base, roadmapPath };
+};
+
+test("roadmap view renders items with proposal refs", async () => {
+  const { base } = createRoadmapWorkspace();
+  const { lastFrame } = render(<RoadmapView startDir={base} />);
+  await nextTick();
+  const output = stripAnsi(lastFrame() ?? "");
+
+  expect(output).toContain("Roadmap");
+  expect(output).toContain("C0001");
+  expect(output).toContain("As a user, I want to sign in.");
+});
+
+test("roadmap view sorts stories alphabetically", async () => {
+  const base = mkdtempSync(join(tmpdir(), "schub-roadmap-sorted-"));
+  const schubDir = join(base, ".schub");
+  mkdirSync(schubDir, { recursive: true });
+  const roadmapPath = join(schubDir, "roadmap.md");
+  writeFileSync(
+    roadmapPath,
+    ["## Roadmap", "", "- **C0001**: Zulu story.", "- **C0002**: Alpha story.", ""].join("\n"),
+    "utf8",
+  );
+
+  const { lastFrame } = render(<RoadmapView startDir={base} />);
+  await nextTick();
+  const output = stripAnsi(lastFrame() ?? "");
+  const alphaIndex = output.indexOf("Alpha story.");
+  const zuluIndex = output.indexOf("Zulu story.");
+
+  expect(alphaIndex).toBeGreaterThanOrEqual(0);
+  expect(zuluIndex).toBeGreaterThanOrEqual(0);
+  expect(alphaIndex).toBeLessThan(zuluIndex);
+});
+
+test("roadmap view adds a story from direct input", async () => {
+  const { base, roadmapPath } = createRoadmapWorkspace();
+  const rendered = render(<RoadmapView refreshIntervalMs={50} startDir={base} />);
+  await nextTick();
+  await nextTick();
+
+  const story = "As a tester, I want to explore.";
+  for (const char of story) {
+    rendered.stdin.write(char);
+    await nextTick();
+  }
+  rendered.stdin.write("\n");
+  await new Promise((resolve) => setTimeout(resolve, 20));
+
+  const content = readFileSync(roadmapPath, "utf8");
+  expect(content).toContain("- **<PROPOSAL_REF>**: As a tester, I want to explore.");
+});