oc-transcript-save 0.2.0 → 0.2.2

package/index.ts

@@ -28,6 +28,7 @@ import {
 	toYaml,
 	truncateText,
 } from './transcript-save.ts'
+import path from 'node:path'
 
 const SERVICE_NAME = 'oc-transcript-save'
 
@@ -218,7 +219,7 @@ function resolveOutputPath(ctx: AnyCtx, snapshot: Snapshot): string {
 	const timestamp = formatFilenameTimestamp(time.created)
 	const id = sanitiseFilename(asString(session.id, 'unknown-session'))
 	const root = process.env.OC_TRANSCRIPT_SAVE_PATH || asString(ctx.location?.directory, '') || process.cwd()
-	return `${root.replace(/\/+$/, '')}/${timestamp}-${id}.md`
+	return path.join(root, `${timestamp}-${id}.md`)
 }
 
 async function fetchSnapshot(ctx: AnyCtx, sessionID: string): Promise<Snapshot> {
@@ -292,15 +293,14 @@ export const TranscriptSavePluginV2 = {
 			const sessionID = resolveSessionId(event)
 			if (!sessionID) return
 
-			if (type === 'session.status') {
-				const status = asObject(asObject(event.data).status)
-				const statusType = asString(status.type)
-				if (statusType === 'busy' || statusType === 'retry') return
-				if (statusType === 'idle') schedule(sessionID)
-				return
-			}
-
-			if (type === 'session.idle') {
+			// V2 replaced the ephemeral `session.status` / deprecated `session.idle`
+			// bus events (never delivered to plugins) with durable
+			// `session.execution.*` events. A turn ends in exactly one of these.
+			if (
+				type === 'session.execution.succeeded'
+				|| type === 'session.execution.failed'
+				|| type === 'session.execution.interrupted'
+			) {
 				schedule(sessionID)
 			}
 		}

package/package.json

@@ -1,7 +1,7 @@
 {
 	"name": "oc-transcript-save",
 	"description": "OpenCode plugin that autosaves full transcript in specified directory",
-	"version": "0.2.0",
+	"version": "0.2.2",
 	"type": "module",
 	"main": "./index.ts",
 	"module": "./index.ts",

package/transcript-save.ts

@@ -596,8 +596,20 @@ export async function atomicWrite(filePath: string, content: string): Promise<nu
 	try {
 		await rename(tmpPath, filePath)
 	} catch (error) {
-		await unlink(tmpPath).catch(() => undefined)
-		throw error
+		// POSIX rename atomically replaces the destination; Windows refuses to
+		// rename onto an existing file (EEXIST/EPERM). Remove it and retry.
+		if (process.platform === 'win32') {
+			await unlink(filePath).catch(() => undefined)
+			try {
+				await rename(tmpPath, filePath)
+			} catch (retryError) {
+				await unlink(tmpPath).catch(() => undefined)
+				throw retryError
+			}
+		} else {
+			await unlink(tmpPath).catch(() => undefined)
+			throw error
+		}
 	}
 
 	return Buffer.byteLength(normalised, 'utf8')