trekoon 0.1.4 → 0.1.5

package/.agents/skills/trekoon/SKILL.md

@@ -61,7 +61,11 @@ Use long flags (`--status`, `--description`, etc.) and ALWAYS append `--toon` to
 
 ## 1) Status Management
 
-### Valid Statuses
+### Status values
+
+Trekoon accepts any non-empty status string.
+
+Recommended statuses for consistent workflows:
 
 | Status | Meaning |
 |--------|---------|
@@ -69,7 +73,7 @@ Use long flags (`--status`, `--description`, etc.) and ALWAYS append `--toon` to
 | `in_progress` | Actively being worked on |
 | `done` | Completed successfully |
 
-Note: `in-progress` (hyphenated) is equivalent to `in_progress`.
+Note: `in-progress` (hyphenated) is treated the same as `in_progress` for default list ordering/filtering.
 
 ### When to Change Status
 
@@ -165,7 +169,7 @@ trekoon epic show <id> --all --toon
 trekoon task show <id> --all --toon
 ```
 
-- `epic list` / `task list` defaults:
+- `epic list` / `task list` / `subtask list` defaults:
   - open work only (`in_progress`, `in-progress`, `todo`)
   - prioritized as `in_progress`/`in-progress` first, then `todo`
   - default limit `10`

package/README.md

@@ -77,10 +77,10 @@ Human view options:
 
 - List and show commands default to table output in human mode.
 - Use `--view compact` to restore compact pipe output.
-- `epic list` and `task list` support `--view table|compact`.
+- `epic list`, `task list`, and `subtask list` support `--view table|compact`.
 - `epic show` and `task show` support `--view table|compact|tree|detail`.
 
-List defaults and filters (`epic list`, `task list`):
+List defaults and filters (`epic list`, `task list`, `subtask list`):
 
 - Default scope: open work only (`in_progress`, `in-progress`, `todo`)
 - Default limit: `10`

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "trekoon",
-  "version": "0.1.4",
+  "version": "0.1.5",
   "description": "AI-first local issue tracker CLI.",
   "license": "MIT",
   "type": "module",

package/src/commands/help.ts

@@ -36,7 +36,7 @@ const COMMAND_HELP: Record<string, string> = {
   task:
     "Usage: trekoon task <subcommand> [options] (list defaults: open statuses + limit 10; list flags: --status <csv> | --limit <n> | --all | --view table|compact; show: compact=task summary, tree=hierarchy, detail=descriptions, and --all defaults to detail in machine modes; update bulk flags: --all | --ids <csv> with --append <text> and/or --status <status>)",
   subtask:
-    "Usage: trekoon subtask <subcommand> [options] (list supports --view table|compact; update bulk flags: --all | --ids <csv> with --append <text> and/or --status <status>)",
+    "Usage: trekoon subtask <subcommand> [options] (list defaults: open statuses + limit 10; list flags: --task <id> | --status <csv> | --limit <n> | --all | --view table|compact; update bulk flags: --all | --ids <csv> with --append <text> and/or --status <status>)",
   dep: "Usage: trekoon dep <subcommand> [options]",
   events: "Usage: trekoon events prune [--dry-run] [--archive] [--retention-days <n>]",
   migrate: "Usage: trekoon migrate <status|rollback> [--to-version <n>]",

package/src/commands/quickstart.ts

@@ -17,7 +17,7 @@ const QUICKSTART_TEXT = [
   "3) Task details and description",
   "- Human list and show views default to table format.",
   "- Alternate list view: add --view compact.",
-  "- task/epic list defaults: open work only (in_progress/in-progress, todo), max 10.",
+  "- task/epic/subtask list defaults: open work only (in_progress/in-progress, todo), max 10.",
   "- Filter list by status: --status in_progress,todo (CSV).",
   "- Change page size: --limit <n>. Show all statuses and all rows with --all.",
   "- --all cannot be combined with --status or --limit.",

package/src/commands/subtask.ts

@@ -1,4 +1,4 @@
-import { hasFlag, parseArgs, readEnumOption, readMissingOptionValue, readOption } from "./arg-parser";
+import { hasFlag, parseArgs, parseStrictPositiveInt, readEnumOption, readMissingOptionValue, readOption } from "./arg-parser";
 
 import { DomainError, type SubtaskRecord } from "../domain/types";
 import { TrackerDomain } from "../domain/tracker-domain";
@@ -12,6 +12,8 @@ function formatSubtask(subtask: SubtaskRecord): string {
 }
 
 const VIEW_MODES = ["table", "compact"] as const;
+const DEFAULT_SUBTASK_LIST_LIMIT = 10;
+const DEFAULT_OPEN_SUBTASK_STATUSES = ["in_progress", "in-progress", "todo"] as const;
 
 function parseIdsOption(rawIds: string | undefined): string[] {
   if (rawIds === undefined) {
@@ -24,6 +26,45 @@ function parseIdsOption(rawIds: string | undefined): string[] {
     .filter((value) => value.length > 0);
 }
 
+function parseStatusCsv(rawStatuses: string | undefined): string[] | undefined {
+  if (rawStatuses === undefined) {
+    return undefined;
+  }
+
+  return rawStatuses
+    .split(",")
+    .map((value) => value.trim())
+    .filter((value) => value.length > 0);
+}
+
+function subtaskStatusPriority(status: string): number {
+  if (status === "in_progress" || status === "in-progress") {
+    return 0;
+  }
+
+  if (status === "todo") {
+    return 1;
+  }
+
+  return 2;
+}
+
+function filterSortAndLimitSubtasks(
+  subtasks: readonly SubtaskRecord[],
+  statuses: readonly string[] | undefined,
+  limit: number | undefined,
+): SubtaskRecord[] {
+  const allowedStatuses = statuses === undefined ? undefined : new Set(statuses);
+  const filtered = allowedStatuses === undefined ? [...subtasks] : subtasks.filter((subtask) => allowedStatuses.has(subtask.status));
+  const sorted = [...filtered].sort((left, right) => subtaskStatusPriority(left.status) - subtaskStatusPriority(right.status));
+
+  if (limit === undefined) {
+    return sorted;
+  }
+
+  return sorted.slice(0, limit);
+}
+
 function appendLine(existing: string, line: string): string {
   return existing.length > 0 ? `${existing}\n${line}` : line;
 }
@@ -116,6 +157,8 @@ export async function runSubtask(context: CliContext): Promise<CliResult> {
       case "list": {
         const missingListOption =
           readMissingOptionValue(parsed.missingOptionValues, "view") ??
+          readMissingOptionValue(parsed.missingOptionValues, "status", "s") ??
+          readMissingOptionValue(parsed.missingOptionValues, "limit", "l") ??
           readMissingOptionValue(parsed.missingOptionValues, "task", "t");
         if (missingListOption !== undefined) {
           return failMissingOptionValue("subtask.list", missingListOption);
@@ -123,6 +166,10 @@ export async function runSubtask(context: CliContext): Promise<CliResult> {
 
         const rawView: string | undefined = readOption(parsed.options, "view");
         const view = readEnumOption(parsed.options, VIEW_MODES, "view");
+        const includeAll = hasFlag(parsed.flags, "all");
+        const rawStatuses = readOption(parsed.options, "status", "s");
+        const rawLimit = readOption(parsed.options, "limit", "l");
+
         if (rawView !== undefined && view === undefined) {
           return failResult({
             command: "subtask.list",
@@ -135,8 +182,64 @@ export async function runSubtask(context: CliContext): Promise<CliResult> {
           });
         }
 
+        if (includeAll && rawStatuses !== undefined) {
+          return failResult({
+            command: "subtask.list",
+            human: "Use either --all or --status, not both.",
+            data: { code: "invalid_input", flags: ["all", "status"] },
+            error: {
+              code: "invalid_input",
+              message: "--all and --status are mutually exclusive",
+            },
+          });
+        }
+
+        if (includeAll && rawLimit !== undefined) {
+          return failResult({
+            command: "subtask.list",
+            human: "Use either --all or --limit, not both.",
+            data: { code: "invalid_input", flags: ["all", "limit"] },
+            error: {
+              code: "invalid_input",
+              message: "--all and --limit are mutually exclusive",
+            },
+          });
+        }
+
+        const statuses = parseStatusCsv(rawStatuses);
+        if (rawStatuses !== undefined && statuses !== undefined && statuses.length === 0) {
+          return failResult({
+            command: "subtask.list",
+            human: "Provide at least one status with --status.",
+            data: { code: "invalid_input", status: rawStatuses },
+            error: {
+              code: "invalid_input",
+              message: "Invalid --status value",
+            },
+          });
+        }
+
+        const parsedLimit = parseStrictPositiveInt(rawLimit);
+        if (Number.isNaN(parsedLimit)) {
+          return failResult({
+            command: "subtask.list",
+            human: "Invalid --limit value. Use an integer >= 1.",
+            data: { code: "invalid_input", limit: rawLimit },
+            error: {
+              code: "invalid_input",
+              message: "Invalid --limit value",
+            },
+          });
+        }
+
         const taskId: string | undefined = readOption(parsed.options, "task", "t") ?? parsed.positional[1];
-        const subtasks = domain.listSubtasks(taskId);
+        const selectedStatuses = includeAll
+          ? undefined
+          : statuses ?? [...DEFAULT_OPEN_SUBTASK_STATUSES];
+        const selectedLimit = includeAll
+          ? undefined
+          : parsedLimit ?? DEFAULT_SUBTASK_LIST_LIMIT;
+        const subtasks = filterSortAndLimitSubtasks(domain.listSubtasks(taskId), selectedStatuses, selectedLimit);
         const listView = view ?? "table";
         const human =
           subtasks.length === 0