opencode-translate 0.1.0 → 0.1.1

package/README.md

@@ -4,7 +4,7 @@
 
 ## What It Does
 
-- Activates once per root session when the first user message contains a trigger keyword such as `$en`.
+- Activates once per root session when any root-session user message contains a trigger keyword such as `$en`.
 - Translates user-authored text parts from `sourceLanguage` to English before the main LLM sees them.
 - Stores the original user text, plus a cached English translation in part metadata.
 - Shows a visible `→ EN: ...` preview under each translated user text part.
@@ -42,7 +42,7 @@ Hooks never throw. If the translator fails (network error, auth failure, provide
 1. Logs the error via `client.app.log` (visible with `verbose: true`).
 2. Emits a `⚠️ Translation failed: …` synthetic part.
 3. Falls back to sending the original (untranslated) user text to the model.
-4. On first-turn activation failure, it also rolls back activation so the next turn retries cleanly.
+4. On activation-turn failure, it also rolls back activation so the next turn retries cleanly.
 
 A stalled provider request is additionally bounded by a 60s hard timeout per translation call, so a hung upstream cannot block the OpenCode session.
 
@@ -75,7 +75,7 @@ opencode auth login anthropic
 export ANTHROPIC_API_KEY=...
 ```
 
-Start a brand-new session and put the trigger in the first message:
+Put the trigger in the message where translation should begin. Earlier messages in the session are left as-is:
 
 ```text
 $en 프로젝트 루트의 package.json을 읽고 요약해줘

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-translate",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "OpenCode plugin that lets the user chat in a configured source language while the main chat loop only sees English.",
   "type": "module",
   "main": "src/index.ts",

package/src/activation.ts

@@ -32,7 +32,12 @@ import {
 } from "./question-tool"
 import { createSyntheticPartID, createTranslator, hashText } from "./translator"
 
-const sessionStateCache = new Map<string, TranslateState | null>()
+const INACTIVE_ROOT_SESSION = "inactive-root"
+const INACTIVE_CHILD_SESSION = "inactive-child"
+
+type CachedSessionState = TranslateState | typeof INACTIVE_ROOT_SESSION | typeof INACTIVE_CHILD_SESSION
+
+const sessionStateCache = new Map<string, CachedSessionState>()
 const questionSnapshots = new Map<string, QuestionSnapshot>()
 const QUESTION_TOOL_ID = "question"
 
@@ -249,10 +254,26 @@ async function resolveSessionState(
 ): Promise<ResolvedSessionState> {
   const cached = sessionStateCache.get(sessionID)
   if (cached !== undefined) {
+    if (cached === INACTIVE_ROOT_SESSION) {
+      return {
+        sessionActive: false,
+        canActivate: true,
+        storedMessages: [],
+      }
+    }
+
+    if (cached === INACTIVE_CHILD_SESSION) {
+      return {
+        sessionActive: false,
+        canActivate: false,
+        storedMessages: [],
+      }
+    }
+
     return {
-      sessionActive: Boolean(cached),
+      sessionActive: true,
       canActivate: false,
-      state: cached ?? undefined,
+      state: cached,
       storedMessages: [],
     }
   }
@@ -265,7 +286,7 @@ async function resolveSessionState(
     }),
   )
   if (session.parentID != null) {
-    sessionStateCache.set(sessionID, null)
+    sessionStateCache.set(sessionID, INACTIVE_CHILD_SESSION)
     return { sessionActive: false, canActivate: false, storedMessages: [] }
   }
 
@@ -279,13 +300,13 @@ async function resolveSessionState(
   const state = extractStoredState(storedMessages)
   if (state) {
     sessionStateCache.set(sessionID, state)
-  } else if (storedMessages.length > 0) {
-    sessionStateCache.set(sessionID, null)
+  } else {
+    sessionStateCache.set(sessionID, INACTIVE_ROOT_SESSION)
   }
 
   return {
     sessionActive: Boolean(state),
-    canActivate: storedMessages.length === 0,
+    canActivate: !state,
     state: state ?? undefined,
     storedMessages,
   }
@@ -326,8 +347,6 @@ export function createHooks(ctx: PluginInput, rawOptions: PluginOptions = {}, de
             }
             activatedThisTurn = true
             sessionStateCache.set(input.sessionID, activeState)
-          } else {
-            sessionStateCache.set(input.sessionID, null)
           }
         }
 
@@ -424,7 +443,7 @@ export function createHooks(ctx: PluginInput, rawOptions: PluginOptions = {}, de
         // user-authored part, roll back activation so the next turn does a
         // clean retry instead of cementing broken state.
         if (activatedThisTurn && translationErrors.length > 0 && eligibleIndex === translationErrors.length) {
-          sessionStateCache.set(input.sessionID, null)
+          sessionStateCache.set(input.sessionID, INACTIVE_ROOT_SESSION)
           return
         }