@ai-setting/roy-task-plugin 0.7.1 → 0.7.2

package/dist/claude-plugin/hooks/find-open-task.mjs

@@ -0,0 +1,66 @@
+/**
+ * `findOpenTask` — the Stop hook's "is a task still open?" lookup.
+ *
+ * Extracted from `stop-check.mjs` so it can be unit-tested without spawning a
+ * hook process (Task #4624, CV finding).
+ *
+ * ## Why the shape matters
+ *
+ * `TaskBackend.listTasks` is a *paged view*: it applies the status filter in
+ * memory, then slices `[offset, offset+limit)`. It promises neither an order
+ * nor that one page holds every match. So "take one page, then filter in the
+ * caller" silently misses an open task that sits behind a page of newer
+ * completed tasks — the exact regression this module exists to prevent.
+ */
+
+/**
+ * The statuses that count as "still open".
+ *
+ * Ordered by how recently a task is likely to have been touched; the search
+ * does not depend on the order because it scans every open status.
+ */
+export const OPEN_STATUSES = ["active", "todo"];
+
+/** Page size used while draining the open statuses. */
+const PAGE_SIZE = 500;
+
+/** Read an update timestamp from either roy shape (`updated_at` / `updatedAt`). */
+export function updatedAtOf(task) {
+  const raw = task?.updatedAt ?? task?.updated_at ?? 0;
+  if (typeof raw === "number") return raw;
+  const parsed = Date.parse(String(raw));
+  return Number.isNaN(parsed) ? 0 : parsed;
+}
+
+/**
+ * The most recently updated task that is still `todo` / `active`, or `null`.
+ *
+ * @param {{ listTasks: (options?: object) => Promise<{ tasks?: any[], total?: number }> }} backend
+ */
+export async function findOpenTask(backend) {
+  let newest = null;
+  let newestAt = Number.NEGATIVE_INFINITY;
+
+  for (const status of OPEN_STATUSES) {
+    let offset = 0;
+    for (;;) {
+      const page = await backend.listTasks({ status, limit: PAGE_SIZE, offset });
+      const tasks = page?.tasks ?? [];
+
+      for (const task of tasks) {
+        if (task.status !== status) continue;
+        const at = updatedAtOf(task);
+        if (at > newestAt || (at === newestAt && newest && task.id > newest.id)) {
+          newest = task;
+          newestAt = at;
+        }
+      }
+
+      offset += tasks.length;
+      const total = page?.total ?? offset;
+      if (tasks.length === 0 || offset >= total) break;
+    }
+  }
+
+  return newest;
+}

package/dist/claude-plugin/hooks/stop-check.mjs

@@ -39,6 +39,7 @@
  */
 
 import { createServer, defaultTaskDbPath, BridgeError } from "../mcp/bridge.mjs";
+import { findOpenTask } from "./find-open-task.mjs";
 
 /** Read stdin fully, with a hard cap so a huge transcript cannot stall us. */
 async function readStdin(limitBytes = 2 * 1024 * 1024) {
@@ -131,26 +132,6 @@ function parseTranscript(raw) {
   return entries;
 }
 
-/** The most recently updated task that is still open. */
-/**
- * Find the most recently touched task that is still open.
- *
- * Task #4619 — this used to be a hand-written SQL query against roy's own
- * database (`SELECT id, title, status FROM task WHERE status IN ('todo',
- * 'active') ORDER BY updated_at DESC LIMIT 1`). The *query* is task
- * semantics, so it belongs to core, not to the plugin.
- *
- * It is now expressed against core's `TaskBackend`, which owns the status
- * vocabulary and the ordering.
- */
-async function findOpenTask(backend) {
-  const result = await backend.listTasks({ limit: 200 });
-  const open = result.tasks
-    .filter((t) => t.status === "todo" || t.status === "active")
-    .sort((a, b) => (b.updatedAt ?? b.updated_at ?? 0) - (a.updatedAt ?? a.updated_at ?? 0));
-  return open[0] ?? null;
-}
-
 function approve() {
   process.stdout.write(JSON.stringify({ decision: "approve" }));
   process.exit(0);

package/dist/cli.js

@@ -2484,7 +2484,7 @@ function formatStatus(report) {
 }
 
 // ../../../packages/cli/src/main.ts
-var VERSION = "0.7.1";
+var VERSION = "0.7.2";
 async function main(argv) {
   const parsed = parseArgs(argv);
   switch (parsed.command) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ai-setting/roy-task-plugin",
-  "version": "0.7.1",
+  "version": "0.7.2",
   "type": "module",
   "main": "./dist/index.js",
   "scripts": {