Skip to content

Cleanup: Remove debug logging and unnecessary code #4

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
wagnert merged 1 commit into from
Jan 3, 2026
Merged

Cleanup: Remove debug logging and unnecessary code #4

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
28 changes: 2 additions & 26 deletions src/hooks/EventHook.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,9 +77,6 @@ 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") {
Expand Down Expand Up @@ -108,33 +105,13 @@ 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)

Expand Down Expand Up @@ -180,8 +157,7 @@ export function createEventHook(
})
}

// Remove from processing set when done
processingSessions.delete(sessionID)
}
sessionManager.delete(sessionID)
}
}
}