Merge pull request #3 from techdivision/develop

Fix: Remove duplicate default export causing double worklogs

Author: wagnert

package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-time-tracking",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Automatic time tracking plugin for OpenCode - tracks session duration and tool usage to CSV",
   "main": "src/Plugin.ts",
   "types": "src/Plugin.ts",

src/Plugin.ts

@@ -72,5 +72,3 @@ export const plugin: Plugin = async ({
 
   return hooks
 }
-
-export default plugin

src/hooks/EventHook.ts

@@ -60,10 +60,10 @@ async function extractSummaryTitle(
  * @returns The event hook function
  *
  * @remarks
-  * Handles two types of events:
-   *
-   * 1. **message.part.updated** - Tracks token usage from step-finish parts
-   * 2. **session.idle** - Finalizes and exports the session
+ * Handles two types of events:
+ *
+ * 1. **message.part.updated** - Tracks token usage from step-finish parts
+ * 2. **session.idle** - Finalizes and exports the session
  *
  * @example
  * ```typescript
@@ -77,6 +77,9 @@ export function createEventHook(
   csvWriter: CsvWriter,
   client: OpencodeClient
 ) {
+  // Track sessions that are currently being processed to prevent duplicates
+  const processingSessions = new Set<string>()
+
   return async ({ event }: { event: Event }): Promise<void> => {
     // Track token usage from step-finish events
     if (event.type === "message.part.updated") {
@@ -105,13 +108,33 @@ export function createEventHook(
         return
       }
 
+      // Debug: Log all session.idle events
+      console.log(`[TimeTracking] ${new Date().toISOString()} Received session.idle for session ${sessionID}, processingSessions size before: ${processingSessions.size}`)
+
+      // Prevent duplicate processing if this session is already being handled
+      if (processingSessions.has(sessionID)) {
+        console.log(`[TimeTracking] ${new Date().toISOString()} Skipping duplicate session.idle for session ${sessionID}`)
+        return
+      }
+
+      // Mark this session as being processed
+      processingSessions.add(sessionID)
+      console.log(`[TimeTracking] ${new Date().toISOString()} Added session ${sessionID} to processingSessions, size now: ${processingSessions.size}`)
+
       const session = sessionManager.get(sessionID)
 
       if (!session || session.activities.length === 0) {
         sessionManager.delete(sessionID)
+        processingSessions.delete(sessionID)
         return
       }
 
+      // Delete session immediately to prevent duplicate processing
+      sessionManager.delete(sessionID)
+
+      // Debug: Log if we're processing this session
+      console.log(`[TimeTracking] ${new Date().toISOString()} Processing session.idle for session ${sessionID}, activities: ${session.activities.length}, processingSessions: [${Array.from(processingSessions).join(', ')}]`)
+
       const endTime = Date.now()
       const durationSeconds = Math.round((endTime - session.startTime) / 1000)
 
@@ -155,9 +178,10 @@ export function createEventHook(
             variant: "error",
           },
         })
-      }
+        }
 
-      sessionManager.delete(sessionID)
-    }
+        // Remove from processing set when done
+        processingSessions.delete(sessionID)
+      }
   }
 }