@comfanion/usethis_todo 0.1.15-dev.12 → 0.1.15-dev.13

package/package.json

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

package/tools.ts

@@ -150,26 +150,22 @@ async function getNativeDataDirs(): Promise<string[]> {
   return [...dirs]
 }
 
-async function getStoragePath(sid: string): Promise<string> {
+async function getStoragePaths(sid: string): Promise<string[]> {
   const file = `${sid || "current"}.json`
+  const paths: string[] = []
 
-  // 1. Prefer authoritative path from OpenCode server
+  // 1. Authoritative path from OpenCode server
   if (_nativeStorageBase) {
-    return path.join(_nativeStorageBase, "storage", "todo", file)
+    paths.push(path.join(_nativeStorageBase, "storage", "todo", file))
   }
 
-  // 2. Fallback: guess from well-known data dirs (first one that exists or default)
+  // 2. Well-known data dirs (fallbacks)
   const baseDirs = await getNativeDataDirs()
-  // Try to find one that exists
   for (const base of baseDirs) {
-    try {
-      await fs.access(base)
-      return path.join(base, "opencode", "storage", "todo", file)
-    } catch {}
+    paths.push(path.join(base, "opencode", "storage", "todo", file))
   }
   
-  // Default to first one
-  return path.join(baseDirs[0], "opencode", "storage", "todo", file)
+  return [...new Set(paths)] // Unique paths
 }
 
 // ============================================================================
@@ -217,21 +213,21 @@ function toNativePriority(priority: string): string {
 
 async function readTodos(sid: string, directory?: string): Promise<Todo[]> {
   try {
-    const storagePath = await getStoragePath(sid)
-    const raw = JSON.parse(await fs.readFile(storagePath, "utf-8"))
-    if (!Array.isArray(raw)) return []
-    
-    return raw.map((t: any) => {
-      // Restore our rich status from shadow field if present
-      const realStatus = fromNativeStatus(t.status, t.usethisStatus)
-      const realPriority = t.usethisPriority || t.priority // Fallback if not shadowed
-
-      return normalizeTodo({
-        ...t,
-        status: realStatus,
-        priority: realPriority
-      })
-    })
+    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 []
   }
@@ -243,25 +239,17 @@ function getEnhancedPath(sid: string, directory?: string): string {
 }
 
 async function writeTodos(todos: Todo[], sid: string, directory?: string): Promise<void> {
-  const storagePath = await getStoragePath(sid)
+  const storagePaths = await getStoragePaths(sid)
   const enhancedPath = getEnhancedPath(sid, directory)
   
-  // Prepare for storage:
-  // 1. Set native-compatible fields (status, priority)
-  // 2. Store real values in shadow fields (usethisStatus, usethisPriority)
-  // 3. Keep all other fields (blockedBy, etc)
-  
   const storageTodos = todos.map(t => {
     const nativeStatus = toNativeStatus(t.status)
     const nativePriority = toNativePriority(t.priority)
     
     return {
       ...t,
-      // Native fields for Sidebar compatibility
       status: nativeStatus,
       priority: nativePriority,
-      
-      // Shadow fields (our source of truth)
       usethisStatus: t.status,
       usethisPriority: t.priority
     }
@@ -269,16 +257,26 @@ async function writeTodos(todos: Todo[], sid: string, directory?: string): Promi
 
   const json = JSON.stringify(storageTodos, null, 2)
 
-  // 1. Write to Global Storage (for Sidebar)
-  await fs.mkdir(path.dirname(storagePath), { recursive: true })
-  await fs.writeFile(storagePath, json, "utf-8")
+  // 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. Write to Local Storage (for User visibility/Git)
   try {
     await fs.mkdir(path.dirname(enhancedPath), { recursive: true })
     await fs.writeFile(enhancedPath, json, "utf-8")
-  } catch {
-    // ignore local write errors (non-fatal)
+    await logAction(directory || "", "debug", `Writing to local: ${enhancedPath}`)
+  } catch (e) {
+    await logAction(directory || "", "error", `Local write failed: ${String(e)}`)
   }
 }