@comfanion/usethis_todo 0.1.15-dev.9 → 0.1.16-dev.1

package/index.ts

@@ -2,7 +2,7 @@ import type { Plugin } from "@opencode-ai/plugin"
 import path from "path"
 import fs from "fs/promises"
 
-import { write, read, read_five, read_by_id, update } from "./tools"
+import { write, read, read_five, read_by_id, update, setNativeStorageBase } from "./tools"
 
 interface TodoPruneState {
   lastToolCallId: string | null
@@ -10,7 +10,14 @@ interface TodoPruneState {
 
 const pruneStates = new Map<string, TodoPruneState>()
 
-const UsethisTodoPlugin: Plugin = async ({ directory }) => {
+const UsethisTodoPlugin: Plugin = async ({ directory, client }) => {
+  // Resolve the authoritative state path from OpenCode server (non-blocking).
+  // Must NOT await — server may block until plugin init completes → deadlock.
+  client.path.get().then((pathInfo: any) => {
+    const state = pathInfo?.data?.state
+    if (state) setNativeStorageBase(state)
+  }).catch(() => {})
+
   // Ensure storage directory exists on init
   try {
     const todoDir = path.join(directory, ".opencode", "session-todo")
@@ -62,20 +69,18 @@ const UsethisTodoPlugin: Plugin = async ({ directory }) => {
     },
 
     tool: {
-      // Enhanced tools (original names)
+      // Enhanced tools only — NO native overrides
+      // Native todowrite/todoread stay untouched → Bus.publish fires → sidebar works
       usethis_todo_write: write,
       usethis_todo_read: read,
       usethis_todo_read_five: read_five,
       usethis_todo_read_by_id: read_by_id,
       usethis_todo_update: update,
-      // Override native tools — same implementation, native names
-      todowrite: write,
-      todoread: read,
     },
 
-    // Set nicer titles in TUI + track prune state
+    // Set nicer titles in TUI + track prune state (enhanced tools only)
     "tool.execute.after": async (input, output) => {
-      if (!input.tool.startsWith("usethis_todo_") && input.tool !== "todowrite" && input.tool !== "todoread") return
+      if (!input.tool.startsWith("usethis_todo_")) return
 
       // Update prune state with latest call ID
       const sessionID = input.sessionID
@@ -88,13 +93,13 @@ const UsethisTodoPlugin: Plugin = async ({ directory }) => {
       const out = output.output || ""
 
       // Set a nicer title in TUI
-      if (input.tool === "usethis_todo_write" || input.tool === "todowrite") {
+      if (input.tool === "usethis_todo_write") {
         const match = out.match(/\[(\d+)\/(\d+) done/)
         output.title = match ? `TODO: ${match[2]} tasks` : "TODO updated"
       } else if (input.tool === "usethis_todo_update") {
         const match = out.match(/^✅ (.+)$/m)
         output.title = match ? match[1] : "Task updated"
-      } else if (input.tool === "usethis_todo_read" || input.tool === "todoread") {
+      } else if (input.tool === "usethis_todo_read") {
         const match = out.match(/\[(\d+)\/(\d+) done, (\d+) in progress\]/)
         output.title = match ? `TODO [${match[1]}/${match[2]} done]` : "TODO list"
       } else if (input.tool === "usethis_todo_read_five") {
@@ -102,6 +107,24 @@ const UsethisTodoPlugin: Plugin = async ({ directory }) => {
       } else if (input.tool === "usethis_todo_read_by_id") {
         output.title = "Task details"
       }
+
+      // Publish TODO snapshot into chat for write/update ops
+      const isWriteOp = input.tool === "usethis_todo_write"
+        || input.tool === "usethis_todo_update"
+      if (!isWriteOp) return
+
+      const text = ["## TODO", "", out].join("\n")
+      try {
+        await client?.session?.prompt?.({
+          path: { id: input.sessionID },
+          body: {
+            noReply: true,
+            parts: [{ type: "text", text }],
+          },
+        })
+      } catch {
+        // non-fatal
+      }
     },
   }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@comfanion/usethis_todo",
-  "version": "0.1.15-dev.9",
+  "version": "0.1.16-dev.1",
   "description": "OpenCode plugin: enhanced TODO tools (dual storage + dependency graph)",
   "type": "module",
   "main": "./index.ts",

package/tools.ts

@@ -1,5 +1,5 @@
 /**
- * TODO Tool with Dependencies & Priority — v3 (dual storage)
+ * TODO Tool with Dependencies & Priority — v3 (unified storage)
  *
  * 4 commands:
  *   usethis_todo_write({ todos: [...] })    - create/update TODO list
@@ -9,15 +9,13 @@
  *   usethis_todo_update(id, field, value)   - update any task field
  *
  * Storage:
- *   Enhanced: .opencode/session-todos/{sid}.json  (title, blockedBy, graph)
- *   Native:  ~/.local/share/opencode/storage/todo/{sid}.json  (TUI display)
+ *   Unified: Native OpenCode storage (TUI compatible)
+ *   Path resolved via client.path.get()
  *
  * Features:
- * - Hierarchical IDs: E01-S01-T01
- * - Dependencies: blockedBy field
- * - Priority: CRIT | HIGH | MED | LOW (auto-sorted)
- * - Graph: shows available, blocked, parallel tasks
- * - Dual write: native OpenCode storage for TUI integration
+ *   - Stores FULL enhanced schema (blockedBy, priority, etc) in the native file
+ *   - Maps statuses to native values (pending/completed) for Sidebar compatibility
+ *   - Preserves original statuses in `usethisStatus` field
  */
 
 import { tool } from "@opencode-ai/plugin"
@@ -79,6 +77,10 @@ interface Todo {
   createdAt?: number
   updatedAt?: number
   
+  // Shadow fields for native compatibility
+  usethisStatus?: string    // The REAL status (todo, ready, etc)
+  usethisPriority?: string  // The REAL priority (CRIT, HIGH, etc)
+
   // Mind extensions (backward compatible - optional)
   files?: TodoFile[]                // Associated files
   tools?: TodoTool[]                // Associated tool calls
@@ -93,13 +95,6 @@ interface Todo {
   }
 }
 
-interface NativeTodo {
-  id: string
-  content: string         // "title: content" combined
-  status: string          // pending | in_progress | completed | cancelled
-  priority: string        // high | medium | low
-}
-
 interface TodoGraph {
   todos: Todo[]
   available: string[]
@@ -108,19 +103,24 @@ interface TodoGraph {
 }
 
 // ============================================================================
-// Storage — dual write
+// Storage — Dual Write
 // ============================================================================
 
+let _nativeStorageBase: string | null = null
+
+/**
+ * Set the native storage base path (OpenCode's state directory).
+ * Called once during plugin initialization with the result of client.path.get().
+ */
+export function setNativeStorageBase(statePath: string): void {
+  _nativeStorageBase = statePath
+}
+
 // Resolve project directory (context.directory may be undefined via MCP)
 function dir(directory?: string): string {
   return directory || process.env.OPENCODE_PROJECT_DIR || process.cwd()
 }
 
-// Enhanced storage path (project-local)
-function getEnhancedPath(sid: string, directory?: string): string {
-  return path.join(dir(directory), ".opencode", "session-todo", `${sid || "current"}.json`)
-}
-
 async function logAction(directory: string, action: string, details: string): Promise<void> {
   try {
     const logPath = path.join(dir(directory), ".opencode", "todo.log")
@@ -150,71 +150,133 @@ async function getNativeDataDirs(): Promise<string[]> {
   return [...dirs]
 }
 
-async function getNativePaths(sid: string): Promise<string[]> {
-  const baseDirs = await getNativeDataDirs()
+async function getStoragePaths(sid: string): Promise<string[]> {
   const file = `${sid || "current"}.json`
-  return baseDirs.map((base) => path.join(base, "opencode", "storage", "todo", file))
-}
+  const paths: string[] = []
 
-// Map our format → native format
-function toNative(todo: Todo): NativeTodo {
-  // Status mapping: our → native
-  const statusMap: Record<string, string> = {
-    todo: "pending",
-    in_progress: "in_progress",
-    ready: "in_progress",       // native has no "ready"
-    finished: "in_progress",    // back-compat
-    done: "completed",          // native uses "completed" not "done"
-    cancelled: "cancelled",
+  // 1. Authoritative path from OpenCode server
+  if (_nativeStorageBase) {
+    paths.push(path.join(_nativeStorageBase, "storage", "todo", file))
   }
-  // Priority mapping: CRIT/HIGH/MED/LOW → high/medium/low
-  const prioMap: Record<string, string> = {
-    CRIT: "high",
-    HIGH: "high",
-    MED: "medium",
-    LOW: "low",
+
+  // 2. Well-known data dirs (fallbacks)
+  const baseDirs = await getNativeDataDirs()
+  for (const base of baseDirs) {
+    paths.push(path.join(base, "opencode", "storage", "todo", file))
   }
+  
+  return [...new Set(paths)] // Unique paths
+}
 
-  const deps = todo.blockedBy?.length ? ` [← ${todo.blockedBy.join(", ")}]` : ""
-  const desc = todo.description?.trim() ? ` — ${todo.description.trim()}` : ""
-  const rel = todo.releases?.length ? ` [rel: ${todo.releases.join(", ")}]` : ""
+// ============================================================================
+// Mapping Logic (Enhanced <-> Native)
+// ============================================================================
 
-  return {
-    id: todo.id,
-    content: `${todo.content}${desc}${rel}${deps}`,
-    status: statusMap[todo.status] || "pending",
-    priority: prioMap[todo.priority] || "medium",
+function toNativeStatus(status: string): string {
+  switch (status) {
+    case "todo": return "pending"
+    case "in_progress": return "in_progress"
+    case "ready": return "in_progress" // Sidebar has no 'ready'
+    case "done": return "completed"
+    case "cancelled": return "cancelled"
+    default: return "pending"
   }
 }
 
+function fromNativeStatus(nativeStatus: string, originalStatus?: string): string {
+  // If we have the original status stored, prefer it
+  if (originalStatus) return originalStatus
+
+  // Otherwise map back (lossy for ready/todo distinction)
+  switch (nativeStatus) {
+    case "pending": return "todo"
+    case "in_progress": return "in_progress"
+    case "completed": return "done"
+    case "cancelled": return "cancelled"
+    default: return "todo"
+  }
+}
+
+function toNativePriority(priority: string): string {
+  switch (priority) {
+    case "CRIT": return "high"
+    case "HIGH": return "high"
+    case "MED": return "medium"
+    case "LOW": return "low"
+    default: return "medium"
+  }
+}
+
+// ============================================================================
+// Read / Write
+// ============================================================================
+
 async function readTodos(sid: string, directory?: string): Promise<Todo[]> {
   try {
-    const raw = JSON.parse(await fs.readFile(getEnhancedPath(sid, directory), "utf-8"))
-    if (!Array.isArray(raw)) return []
-    return raw.map((t: any) => normalizeTodo(t))
+    const storagePaths = await getStoragePaths(sid)
+    // Try to read from any available path, starting with authoritative
+    for (const storagePath of storagePaths) {
+      try {
+        const raw = JSON.parse(await fs.readFile(storagePath, "utf-8"))
+        if (Array.isArray(raw)) {
+          return raw.map((t: any) => {
+            const realStatus = fromNativeStatus(t.status, t.usethisStatus)
+            const realPriority = t.usethisPriority || t.priority
+            return normalizeTodo({ ...t, status: realStatus, priority: realPriority })
+          })
+        }
+      } catch { continue }
+    }
+    return []
   } catch {
     return []
   }
 }
 
+// Enhanced storage path (project-local) - restored for visibility/backup
+function getEnhancedPath(sid: string, directory?: string): string {
+  return path.join(dir(directory), ".opencode", "session-todo", `${sid || "current"}.json`)
+}
+
 async function writeTodos(todos: Todo[], sid: string, directory?: string): Promise<void> {
-  // 1. Enhanced storage (our full format)
+  const storagePaths = await getStoragePaths(sid)
   const enhancedPath = getEnhancedPath(sid, directory)
-  await fs.mkdir(path.dirname(enhancedPath), { recursive: true })
-  await fs.writeFile(enhancedPath, JSON.stringify(todos, null, 2), "utf-8")
+  
+  const storageTodos = todos.map(t => {
+    const nativeStatus = toNativeStatus(t.status)
+    const nativePriority = toNativePriority(t.priority)
+    
+    return {
+      ...t,
+      status: nativeStatus,
+      priority: nativePriority,
+      usethisStatus: t.status,
+      usethisPriority: t.priority
+    }
+  })
+
+  const json = JSON.stringify(storageTodos, null, 2)
+
+  // 1. Write to Global Storages (for Sidebar)
+  await Promise.allSettled(
+    storagePaths.map(async (p) => {
+      try {
+        await fs.mkdir(path.dirname(p), { recursive: true })
+        await fs.writeFile(p, json, "utf-8")
+        await logAction(directory || "", "debug", `Writing to global: ${p}`)
+      } catch (e) {
+        await logAction(directory || "", "error", `Global write failed for ${p}: ${String(e)}`)
+      }
+    })
+  )
 
-  // 2. Native storage (for TUI display)
-  const nativeTodos = todos.map(toNative)
+  // 2. Write to Local Storage (for User visibility/Git)
   try {
-    const nativePaths = await getNativePaths(sid)
-    await Promise.allSettled(
-      nativePaths.map(async (nativePath) => {
-        await fs.mkdir(path.dirname(nativePath), { recursive: true })
-        await fs.writeFile(nativePath, JSON.stringify(nativeTodos, null, 2), "utf-8")
-      }),
-    )
-  } catch {
-    // Native write failure is non-fatal
+    await fs.mkdir(path.dirname(enhancedPath), { recursive: true })
+    await fs.writeFile(enhancedPath, json, "utf-8")
+    await logAction(directory || "", "debug", `Writing to local: ${enhancedPath}`)
+  } catch (e) {
+    await logAction(directory || "", "error", `Local write failed: ${String(e)}`)
   }
 }
 
@@ -319,6 +381,10 @@ function normalizeTodo(input: any): Todo {
   if (input.searches) normalized.searches = input.searches
   if (input.context) normalized.context = input.context
   if (input.cleanup) normalized.cleanup = input.cleanup
+  
+  // Preserve shadow fields
+  if (input.usethisStatus) normalized.usethisStatus = input.usethisStatus
+  if (input.usethisPriority) normalized.usethisPriority = input.usethisPriority
 
   return normalized
 }