schub 0.1.2 → 0.1.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "schub",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "type": "module",
   "bin": {
     "schub": "./src/index.ts"
@@ -20,15 +20,18 @@
     "test": "bun test"
   },
   "dependencies": {
+    "@inkjs/ui": "^2.0.0",
     "chalk": "^5.6.2",
     "ink": "^6.6.0",
     "react": "^19.2.3",
-    "react-devtools-core": "^6.1.2"
+    "react-devtools-core": "^6.1.2",
+    "yargs": "^18.0.0"
   },
   "devDependencies": {
     "@types/bun": "latest",
     "@types/node": "^24.3.0",
     "@types/react": "^19.2.8",
+    "@types/yargs": "^17.0.35",
     "ink-testing-library": "^4.0.0"
   }
 }

package/skills/create-proposal/SKILL.md

@@ -11,7 +11,7 @@ $ARGUMENTS
 
 ## Workflow
 
-1. Derive a concise, verb-led `change-id` suffix from the request (kebab-case: `add-`, `update-`, `remove-`, `refactor-`, `fix-`) and prefix it with `C` + the next 3-digit sequence + `_` (e.g., `C001_add-user-auth`).
+1. Derive a concise, verb-led `change-id` suffix from the request (kebab-case: `add-`, `update-`, `remove-`, `refactor-`, `fix-`) and prefix it with `C` + the next 4-digit sequence + `_` (e.g., `C0001_add-user-auth`).
 2. Run `npx schub changes create --change-id "<change-id>" --title "<title>" --input "<user prompt verbatim>"` to scaffold the proposal template.
    - Supported flags: `--change-id`, `--title`, `--input`, `--overwrite`.
 3. Update the generated proposal with concrete, testable statements.
@@ -31,3 +31,7 @@ $ARGUMENTS
 - (OPTIONAL) Contracts: `.schub/changes/<change-id>/contracts.md`
 - (OPTIONAL) Research: `.schub/changes/<change-id>/research.md`
 - (OPTIONAL) ADR: `.schub/changes/<change-id>/adr.md`
+
+## Notes
+
+- **Do not start the implementation** only edit the `changes` folder.

package/skills/create-tasks/SKILL.md

@@ -3,7 +3,7 @@ name: create-tasks
 description: "Break an accepted proposal into actionable task files under `.schub/tasks/` using the task template. Use when asked to create tasks for a change-id."
 ---
 
-## User Input (should contain change-id or shorthand `C###`)
+## User Input (should contain change-id or shorthand `C####`)
 
 ```text
 $ARGUMENTS
@@ -11,13 +11,13 @@ $ARGUMENTS
 
 ## Workflow
 
-1. If the user provides shorthand `C###`, resolve it to the matching `C###_<suffix>` change folder (error if ambiguous). Confirm `.schub/changes/<change-id>/proposal.md` exists and that it is marked accepted. If it is missing, ask the user to run the create-proposal skill first. If it is still a draft, ask the user to review it first.
+1. If the user provides shorthand `C####`, resolve it to the matching `C####_<suffix>` change folder (error if ambiguous). Confirm `.schub/changes/<change-id>/proposal.md` exists and that it is marked accepted. If it is missing, ask the user to run the create-proposal skill first. If it is still a draft, ask the user to review it first.
 2. Read the proposal and other files in the <change-id> folder.
 3. Derive tasks, each task should be:
    - Small enough to implement in one sitting.
    - Independently testable.
    - Explicit about file paths and affected components.
-4. Assign a unique task id using `T###` (ex: `T001`) and avoid collisions with existing tasks.
+4. Assign a unique task id using `T####` (ex: `T0001`) and avoid collisions with existing tasks.
 5. Run `npx schub tasks create --change-id "<change-id>" --title "<task title>"` to scaffold tasks.
    - Repeat `--title` for multiple tasks.
    - Optional flags: `--status "<status>"` (default: `backlog`), `--overwrite`.
@@ -25,8 +25,9 @@ $ARGUMENTS
    - Priority (P1/P2/P3)
    - Parallelizable (yes/no)
    - Goal, scope, steps, acceptance, and evidence
+   - If the task is blocked, set `blocked_reason` in the task frontmatter
 7. When defining the acceptance, include relevant tests, e2e commands and documentation updates.
-8. Resolve blockers by check all existing tasks that are not done. If another task is a blocker, add it to **Depends on**.
+8. Resolve blockers by check all existing tasks that are not done. If another task is a blocker, add it to `depends_on` in frontmatter.
 9. Stop after the task files are created. Do not implement code or modify the plan or supporting artifacts.
 
 ## Output Locations

package/skills/implement-task/SKILL.md

@@ -9,6 +9,11 @@ description: "Implement a single task end-to-end: locate the task file by id, mo
 $ARGUMENTS
 ```
 
+## Task Implement Modes
+
+`schub tasks implement` defaults to `worktree` mode. It creates `.schub/worktrees/<task-id>` on branch `task/<task-id>` and launches Opencode from that worktree.
+Use `--mode none` to skip worktree creation and run from the repo root. Use `--worktree-root` to override the base worktree path.
+
 ## Workflow
 
 1. **Identify the task**
@@ -23,7 +28,7 @@ $ARGUMENTS
 
 3. **Move the task before work**
    - If starting work: move to `.schub/tasks/wip/` (create folder if needed).
-   - If blocked: move to `.schub/tasks/blocked/` and record the blocker in the task file.
+   - If blocked: move to `.schub/tasks/blocked/` and set `blocked_reason` in the task frontmatter.
    - If complete: move to `.schub/tasks/done/` after confirming all checklists are checked.
    - Keep the task in **exactly one** status folder.
 

package/skills/review-proposal/SKILL.md

@@ -3,7 +3,7 @@ name: review-proposal
 description: "Run a review session for a proposal by creating a list of open questions, updating the proposal, adding a Q&A section, and deleting REVIEW_ME.md on completion. Use when asked to review proposals."
 ---
 
-## User Input (change-id or shorthand `C###`)
+## User Input (change-id or shorthand `C####`)
 
 ```text
 $ARGUMENTS
@@ -11,7 +11,7 @@ $ARGUMENTS
 
 ## Workflow
 
-0. If the user provides shorthand `C###`, resolve it to the matching `C###_<suffix>` change folder (error if ambiguous). Mark the proposal Status as "In Review"
+0. If the user provides shorthand `C####`, resolve it to the matching `C####_<suffix>` change folder (error if ambiguous). Mark the proposal Status as "In Review"
 1. Review the proposal by checking the [MISSING INFORMATION] tags and the potential issues listed.
 2. If there are issues, or missing information, run `npx schub review create --change-id "<change-id>"` to scaffold `.schub/changes/<change-id>/REVIEW_ME.md`.
 3. Triage each item:
@@ -35,3 +35,4 @@ $ARGUMENTS
 
 - Avoid expanding scope beyond missing information in the proposal.
 - Focus review on high-stakes issues; place details into assumptions instead of asking.
+- **Do not start the implementation** only edit the `changes` folder.

package/skills/update-roadmap/SKILL.md

@@ -0,0 +1,23 @@
+---
+name: update-roadmap
+description: "Add, list, or propose items from the project roadmap in .schub/roadmap.md. Use when asked to update or promote roadmap user stories."
+---
+
+## User Input
+
+```text
+$ARGUMENTS
+```
+
+## Workflow
+
+1. Ensure `.schub/roadmap.md` exists. If missing, ask the user to run `npx schub init` first.
+2. To view items, run `npx schub roadmap list` and confirm the index of the target story.
+3. To add a story, run `npx schub roadmap add "<story>"`.
+4. To create a proposal from a story, run `npx schub roadmap propose <index>`.
+5. Confirm the roadmap entry now includes the `C####` proposal prefix.
+
+## Output Locations
+
+- `.schub/roadmap.md`
+- `.schub/changes/<change-id>/proposal.md`

package/src/changes.test.ts

@@ -0,0 +1,166 @@
+import { expect, test } from "bun:test";
+import { existsSync, mkdirSync, mkdtempSync, readFileSync, writeFileSync } from "node:fs";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { archiveChange, createChange, listChangeOverview, readChangeDetail } from "./changes";
+
+const createSchubRoot = () => {
+  const base = mkdtempSync(join(tmpdir(), "schub-change-archive-"));
+  const schubDir = join(base, ".schub");
+  mkdirSync(schubDir, { recursive: true });
+  return { schubDir };
+};
+
+const seedProposal = (schubDir: string, changeId: string, status: string) => {
+  const changeDir = join(schubDir, "changes", changeId);
+  mkdirSync(changeDir, { recursive: true });
+  const proposal = `---\nstatus: ${status}\n---\n# Proposal - Seed\n`;
+  writeFileSync(join(changeDir, "proposal.md"), proposal, "utf8");
+};
+
+const seedArchivedProposal = (schubDir: string, changeId: string, status: string) => {
+  const changeDir = join(schubDir, "archive", "changes", changeId);
+  mkdirSync(changeDir, { recursive: true });
+  const proposal = `---\nstatus: ${status}\n---\n# Proposal - Seed\n`;
+  writeFileSync(join(changeDir, "proposal.md"), proposal, "utf8");
+};
+
+const writeProposalContent = (schubDir: string, changeId: string, proposal: string) => {
+  const changeDir = join(schubDir, "changes", changeId);
+  mkdirSync(changeDir, { recursive: true });
+  writeFileSync(join(changeDir, "proposal.md"), proposal, "utf8");
+};
+
+test("createChange rejects duplicate prefixes", () => {
+  const { schubDir } = createSchubRoot();
+  seedProposal(schubDir, "C0001_existing-change", "Draft");
+
+  expect(() =>
+    createChange(schubDir, {
+      changeId: "C0001_new-change",
+      title: "New Change",
+    }),
+  ).toThrow("Change prefix 'C0001' already exists");
+});
+
+test("archiveChange updates proposal and moves directory", () => {
+  const { schubDir } = createSchubRoot();
+  const changeId = "C0001_archive-change";
+  seedProposal(schubDir, changeId, "Accepted");
+
+  archiveChange(schubDir, changeId);
+
+  const archivePath = join(schubDir, "archive", "changes", changeId);
+  expect(existsSync(join(schubDir, "changes", changeId))).toBe(false);
+  expect(existsSync(archivePath)).toBe(true);
+
+  const proposalPath = join(archivePath, "proposal.md");
+  const updated = readFileSync(proposalPath, "utf8");
+  expect(updated).toContain("status: Archived");
+});
+
+test("archiveChange rejects archive collisions", () => {
+  const { schubDir } = createSchubRoot();
+  const changeId = "C0001_archive-change";
+  seedProposal(schubDir, changeId, "Accepted");
+
+  const archivePath = join(schubDir, "archive", "changes", changeId);
+  mkdirSync(archivePath, { recursive: true });
+  const archivedProposal = join(archivePath, "proposal.md");
+  writeFileSync(archivedProposal, "---\nstatus: Archived\n---\n# Proposal - Existing\n", "utf8");
+
+  expect(() => archiveChange(schubDir, changeId)).toThrow(`Archive already exists: ${archivePath}`);
+
+  const activeProposal = readFileSync(join(schubDir, "changes", changeId, "proposal.md"), "utf8");
+  expect(activeProposal).toContain("status: Accepted");
+  expect(readFileSync(archivedProposal, "utf8")).toContain("# Proposal - Existing");
+});
+
+test("listChangeOverview includes archived proposals with normalized ordering", () => {
+  const { schubDir } = createSchubRoot();
+  seedProposal(schubDir, "C0001_draft-change", "draft");
+  seedProposal(schubDir, "C0002_review-change", "Pending review");
+  seedProposal(schubDir, "C0003_accepted-change", "ACCEPTED");
+  seedArchivedProposal(schubDir, "C0004_implementing-change", "Implementing");
+  seedArchivedProposal(schubDir, "C0005_done-change", "Done");
+  seedArchivedProposal(schubDir, "C0006_archived-change", "Archived");
+  seedProposal(schubDir, "C0007_custom-change", "needs input");
+
+  const changes = listChangeOverview(schubDir);
+  const changeMap = new Map(changes.map((change) => [change.id, change]));
+
+  expect(changeMap.get("C0006_archived-change")?.statusLabel).toBe("Archived");
+  expect(changeMap.get("C0001_draft-change")?.statusLabel).toBe("Draft");
+  expect(changeMap.get("C0007_custom-change")?.statusLabel).toBe("Needs Input");
+  expect(changeMap.get("C0003_accepted-change")?.statusOrder).toBe(2);
+
+  const groupedStatuses = new Map<string, number>();
+  for (const change of changes) {
+    groupedStatuses.set(change.statusLabel, change.statusOrder);
+  }
+
+  const orderedLabels = Array.from(groupedStatuses.entries())
+    .map(([label, order]) => ({ label, order }))
+    .sort((left, right) => left.order - right.order || left.label.localeCompare(right.label))
+    .map((group) => group.label);
+
+  expect(orderedLabels).toEqual([
+    "Draft",
+    "Pending Review",
+    "Accepted",
+    "Implementing",
+    "Done",
+    "Archived",
+    "Needs Input",
+  ]);
+});
+
+test("readChangeDetail parses proposal metadata and summary", () => {
+  const { schubDir } = createSchubRoot();
+  const changeId = "C0100_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`;
+
+  writeProposalContent(schubDir, changeId, proposal);
+
+  const detail = readChangeDetail(schubDir, changeId);
+
+  expect(detail.changeId).toBe(changeId);
+  expect(detail.created).toBe("2026-01-10");
+  expect(detail.status).toBe("Draft");
+  expect(detail.input).toBe("TUI detail view");
+  expect(detail.summary).toBe("Add a detail view in the TUI.");
+});
+
+test("readChangeDetail accepts variable-length prefixes", () => {
+  const { schubDir } = createSchubRoot();
+  const shortId = "C1_short-change";
+  const longId = "C12345_long-change";
+  const proposal = (changeId: string) =>
+    `---\nchange_id: ${changeId}\ncreated: 2026-01-12\nstatus: Draft\ninput: None\n---\n# Proposal - Prefix\n\n## Summary\n\nPrefix test.\n`;
+
+  writeProposalContent(schubDir, shortId, proposal(shortId));
+  writeProposalContent(schubDir, longId, proposal(longId));
+
+  expect(readChangeDetail(schubDir, shortId).changeId).toBe(shortId);
+  expect(readChangeDetail(schubDir, longId).changeId).toBe(longId);
+});
+
+test("readChangeDetail falls back when summary is blank or missing", () => {
+  const { schubDir } = createSchubRoot();
+  const blankId = "C0101_blank-summary";
+  const missingId = "C0102_missing-summary";
+
+  writeProposalContent(
+    schubDir,
+    blankId,
+    `---\nchange_id: ${blankId}\ncreated: 2026-01-11\nstatus: Draft\ninput: None\n---\n# Proposal - Blank Summary\n\n## Summary\n\n   \n\n## Why\n\nBecause.\n`,
+  );
+  writeProposalContent(
+    schubDir,
+    missingId,
+    `---\nchange_id: ${missingId}\ncreated: 2026-01-11\nstatus: Draft\ninput: None\n---\n# Proposal - Missing Summary\n\n## Why\n\nBecause.\n`,
+  );
+
+  expect(readChangeDetail(schubDir, blankId).summary).toBe("No summary provided.");
+  expect(readChangeDetail(schubDir, missingId).summary).toBe("No summary provided.");
+});