@comfanion/usethis_todo 0.1.15-dev.0 → 0.1.15-dev.10

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, setNativeStorageBase } from "./tools"
+import { write, read, read_five, read_by_id, update } from "./tools"
 
 interface TodoPruneState {
   lastToolCallId: string | null
@@ -11,19 +11,7 @@ interface TodoPruneState {
 const pruneStates = new Map<string, TodoPruneState>()
 
 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.
-  // Wrapped in try-catch because client.path may not exist (sync TypeError).
-  try {
-    client?.path?.get()?.then((pathInfo: any) => {
-      const state = pathInfo?.data?.state
-      if (state) setNativeStorageBase(state)
-    }).catch(() => {})
-  } catch {
-    // client.path not available in this SDK version — fall back to guessed paths
-  }
-
-  // Ensure enhanced storage directory exists on init
+  // Ensure storage directory exists on init
   try {
     const todoDir = path.join(directory, ".opencode", "session-todo")
     await fs.mkdir(todoDir, { recursive: true })
@@ -51,67 +39,41 @@ const UsethisTodoPlugin: Plugin = async ({ directory, client }) => {
         "todoread",
       ])
 
-      // 1. Collect all TODO-related parts
+      // Collect all TODO-related tool parts
       const toolParts: { part: any; isLast: boolean }[] = []
-      const snapshotParts = new Set<any>()
-      let lastSnapshotPart: any = null
 
       for (const msg of messages) {
         for (const part of (msg.parts || [])) {
-          // Tool parts (contain both state.input and state.output)
           if (part.type === "tool" && prunedToolNames.has(part.tool) && part.state?.status === "completed") {
             toolParts.push({ part, isLast: part.callID === state.lastToolCallId })
           }
-          // User "## TODO" snapshot text parts
-          if (part.type === "text" && typeof part.text === "string" && part.text.startsWith("## TODO")) {
-            snapshotParts.add(part)
-            lastSnapshotPart = part
-          }
         }
       }
 
-      if (toolParts.length === 0 && snapshotParts.size === 0) return
-
-      // 2. Check if all tasks are done by parsing the latest snapshot
-      const lastSnapshotText = lastSnapshotPart?.text || ""
-      const doneMatch = lastSnapshotText.match(/\[(\d+)\/(\d+) done/)
-      const allDone = doneMatch && doneMatch[1] === doneMatch[2] && parseInt(doneMatch[1]) > 0
+      if (toolParts.length === 0) return
 
-      // 3. Prune old tool parts — clear BOTH input and output
+      // Prune old tool parts — clear BOTH input and output, keep only last
       for (const { part, isLast } of toolParts) {
-        if (allDone || !isLast) {
+        if (!isLast) {
           part.state.output = "[TODO pruned]"
           if (part.state.input) part.state.input = {}
         }
       }
-
-      // 4. Remove old "## TODO" snapshot parts entirely from messages
-      //    Keep only the last snapshot (or none if allDone)
-      const snapshotsToRemove = new Set(snapshotParts)
-      if (!allDone && lastSnapshotPart) {
-        snapshotsToRemove.delete(lastSnapshotPart)
-      }
-
-      if (snapshotsToRemove.size > 0) {
-        for (const msg of messages) {
-          if (!Array.isArray(msg.parts)) continue
-          msg.parts = msg.parts.filter((p: any) => !snapshotsToRemove.has(p))
-        }
-      }
-
-      // 5. Remove messages that became empty after part removal
-      output.messages = messages.filter((msg: any) => (msg.parts || []).length > 0)
     },
 
     tool: {
+      // Enhanced tools (original names)
       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,
     },
 
-    // UI niceties + publish snapshot into the chat
+    // Set nicer titles in TUI + track prune state
     "tool.execute.after": async (input, output) => {
       if (!input.tool.startsWith("usethis_todo_") && input.tool !== "todowrite" && input.tool !== "todoread") return
 
@@ -126,25 +88,47 @@ const UsethisTodoPlugin: Plugin = async ({ directory, client }) => {
       const out = output.output || ""
 
       // Set a nicer title in TUI
-      if (input.tool === "usethis_todo_write") {
+      if (input.tool === "usethis_todo_write" || input.tool === "todowrite") {
         const match = out.match(/\[(\d+)\/(\d+) done/)
-        output.title = match ? `📋 TODO: ${match[2]} tasks` : "📋 TODO updated"
+        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") {
+        output.title = match ? match[1] : "Task updated"
+      } else if (input.tool === "usethis_todo_read" || input.tool === "todoread") {
         const match = out.match(/\[(\d+)\/(\d+) done, (\d+) in progress\]/)
-        output.title = match ? `📋 TODO [${match[1]}/${match[2]} done]` : "📋 TODO list"
+        output.title = match ? `TODO [${match[1]}/${match[2]} done]` : "TODO list"
       } else if (input.tool === "usethis_todo_read_five") {
-        output.title = "📋 Next 5 tasks"
+        output.title = "Next 5 tasks"
       } else if (input.tool === "usethis_todo_read_by_id") {
-        output.title = "📋 Task details"
+        output.title = "Task details"
       }
 
-      // Native storage write handles sidebar updates now (via client.path.get()).
-      // The old session.prompt snapshot was causing "table chunks not found" errors.
+      // Publish snapshot into chat (helps when sidebar doesn't refresh)
+      const publishTools = new Set([
+        "usethis_todo_write",
+        "usethis_todo_update",
+        "usethis_todo_read",
+        "usethis_todo_read_five",
+        "usethis_todo_read_by_id",
+      ])
+
+      if (!publishTools.has(input.tool)) 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
+      }
     },
   }
 }
 
-export default UsethisTodoPlugin;
+export default UsethisTodoPlugin;

package/package.json

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

package/tools.ts

@@ -111,18 +111,6 @@ interface TodoGraph {
 // Storage — dual write
 // ============================================================================
 
-// Native storage base path (set by plugin init via client.path.get())
-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().
- * This ensures we write to the exact path OpenCode reads from for the sidebar.
- */
-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()
@@ -147,14 +135,8 @@ async function getNativeDataDirs(): Promise<string[]> {
   const dirs = new Set<string>()
 
   // 1) xdg-basedir (what OpenCode itself uses)
-  try {
-    const mod: any = await import("xdg-basedir")
-    if (mod?.xdgData && typeof mod.xdgData === "string") {
-      dirs.add(mod.xdgData)
-    }
-  } catch {
-    // ignore
-  }
+  // Removed dynamic import to avoid "chunk not found" errors in some environments
+  // relying on standard env vars and paths instead
 
   // 2) explicit XDG override
   if (process.env.XDG_DATA_HOME) {
@@ -169,15 +151,8 @@ async function getNativeDataDirs(): Promise<string[]> {
 }
 
 async function getNativePaths(sid: string): Promise<string[]> {
-  const file = `${sid || "current"}.json`
-
-  // Prefer the authoritative path from OpenCode server API
-  if (_nativeStorageBase) {
-    return [path.join(_nativeStorageBase, "storage", "todo", file)]
-  }
-
-  // Fallback: guess from well-known data dirs
   const baseDirs = await getNativeDataDirs()
+  const file = `${sid || "current"}.json`
   return baseDirs.map((base) => path.join(base, "opencode", "storage", "todo", file))
 }