opencode-translate 0.0.11 → 0.0.13

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-translate",
-  "version": "0.0.11",
+  "version": "0.0.13",
   "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

@@ -137,26 +137,14 @@ function mergeTranslatedMetadata(state: TranslateState, part: TextPartLike, engl
 // OpenCode's flag semantics, observed from packages/opencode and packages/ui:
 //   synthetic: true  -> hidden from the user UI, still sent to the LLM
 //   ignored: true    -> hidden from the LLM, still shown in the user UI
-// The translation preview, activation banner, and failure notices are
-// user-facing status/diagnostic parts that must not leak into the LLM
-// prompt, so they use synthetic:false + ignored:true.
-function createSyntheticTextPart(
-  sessionID: string,
-  messageID: string,
-  text: string,
-  metadata: Record<string, unknown>,
-): TextPartLike {
-  return {
-    id: createSyntheticPartID(),
-    sessionID,
-    messageID,
-    type: "text",
-    text,
-    synthetic: false,
-    ignored: true,
-    metadata,
-  }
-}
+//   both true        -> hidden from both (used for metadata-only marker
+//                       parts like the activation banner that exist purely
+//                       to carry state across reloads)
+// The plugin's user-facing status text (translation preview, activation
+// banner, failure notice) lives inline on the source-language user part
+// itself rather than as sibling parts, because OpenCode's
+// `UserMessageDisplay` renders only the first non-synthetic text part
+// per user message.
 
 // LLM-only text part: hidden from the TUI but the only LLM-visible
 // representation of the user's source-language text. The original
@@ -337,11 +325,21 @@ export function createHooks(ctx: PluginInput, rawOptions: PluginOptions = {}, de
         const nextParts: TextPartLike[] = []
         let eligibleIndex = 0
         const translationErrors: { part: TextPartLike; error: unknown }[] = []
+        // Track the first user-authored text part so the activation
+        // banner can be inlined onto it. OpenCode's `UserMessageDisplay`
+        // renders only the first non-synthetic text part per user
+        // message, so any standalone banner part is never visible in
+        // the TUI.
+        let firstUserTextPart: (TextPartLike & { text: string }) | undefined
 
         for (const part of output.parts as TextPartLike[]) {
           nextParts.push(part)
           if (!isUserAuthoredTextPart(part)) continue
 
+          if (firstUserTextPart === undefined) {
+            firstUserTextPart = part
+          }
+
           const currentEligibleIndex = eligibleIndex
           eligibleIndex += 1
           if (part.text.trim().length === 0) continue
@@ -355,26 +353,27 @@ export function createHooks(ctx: PluginInput, rawOptions: PluginOptions = {}, de
             })
 
             const sourceHash = hashText(part.text)
+            // Compute metadata against the ORIGINAL `part.text` so the
+            // stored `translate_source_hash` keeps a stable round-trip
+            // identity even after we splice the preview into the part
+            // text below.
             part.metadata = {
               ...(part.metadata ?? {}),
               ...mergeTranslatedMetadata(activeState, part, english),
             }
-            // Hide the user's source-language text from the LLM. The TUI
-            // still renders it because `ignored:true` only affects the
-            // user-side LLM serializer (`message-v2.ts:773`).
+            // Inline the `→ EN: ...` preview into the source-language
+            // part. OpenCode's `UserMessageDisplay`
+            // (packages/ui/src/components/message-part.tsx) renders only
+            // ONE text part per user message — the first non-synthetic —
+            // so a sibling `synthetic:false + ignored:true` preview part
+            // would never reach the screen. Combining them here keeps
+            // the original visible alongside the English twin while
+            // `ignored:true` still hides the whole thing from the LLM
+            // serializer (`message-v2.ts:773`); the LLM-only synthetic
+            // twin below carries the clean English prompt.
+            part.text = `${part.text}\n\n_→ EN: ${english}_`
             part.ignored = true
 
-            // UI-only preview so the user can verify the translation that
-            // was sent to the LLM.
-            nextParts.push(
-              createSyntheticTextPart(part.sessionID, part.messageID, `→ EN: ${english}`, {
-                translate_role: "translation_preview",
-                translate_nonce: activeState.translate_nonce,
-                translate_source_hash: sourceHash,
-                translate_part_index: currentEligibleIndex,
-              }),
-            )
-
             // LLM-only English twin. This is the actual prompt the model
             // sees in place of the now-`ignored` source-language part.
             nextParts.push(
@@ -386,22 +385,27 @@ export function createHooks(ctx: PluginInput, rawOptions: PluginOptions = {}, de
               }),
             )
           } catch (error) {
-            // Fall back to sending the original text to the LLM so the user
-            // still gets a response. Surface the error as a synthetic part.
+            // Translation failed. Append a visible warning to the
+            // source-language part (same UI constraint as the success
+            // path: only one text part per user message renders) and
+            // route the original, untranslated text to the LLM via a
+            // dedicated fallback twin so the model still gets a clean
+            // prompt without the warning text leaking into context.
             translationErrors.push({ part, error })
-            const wrapped = buildInboundTranslationError(activeState.translate_source_lang, normalizeReason(error))
+            const reason = normalizeReason(error)
+            const wrapped = buildInboundTranslationError(activeState.translate_source_lang, reason)
             await logError(client, wrapped)
+
+            const originalText = part.text
+            part.text = `${originalText}\n\n_⚠️ Translation failed: ${reason}. Original text was sent to the model._`
+            part.ignored = true
+
             nextParts.push(
-              createSyntheticTextPart(
-                part.sessionID,
-                part.messageID,
-                `⚠️ Translation failed: ${normalizeReason(error)}. Original text will be sent to the model.`,
-                {
-                  translate_role: "translation_failure",
-                  translate_nonce: activeState.translate_nonce,
-                  translate_part_index: currentEligibleIndex,
-                },
-              ),
+              createLlmOnlyTextPart(part.sessionID, part.messageID, originalText, {
+                translate_role: "llm_only_fallback",
+                translate_nonce: activeState.translate_nonce,
+                translate_part_index: currentEligibleIndex,
+              }),
             )
           }
         }
@@ -415,13 +419,36 @@ export function createHooks(ctx: PluginInput, rawOptions: PluginOptions = {}, de
         }
 
         if (activatedThisTurn) {
-          nextParts.push(
-            createSyntheticTextPart(input.sessionID, output.message.id, createActivationBannerText(options), {
+          const bannerText = createActivationBannerText(options)
+          // Inline the activation banner into the first user-authored
+          // text part so it actually reaches the TUI (see comment above
+          // for the single-text-part rendering constraint). The banner
+          // sits underneath the inline `→ EN: ...` preview that was
+          // appended during translation, giving the user a one-time
+          // confirmation that translation mode just turned on.
+          if (firstUserTextPart !== undefined) {
+            firstUserTextPart.text = `${firstUserTextPart.text}\n\n_${bannerText}_`
+          }
+          // Also emit the banner as a metadata-only synthetic part so
+          // `extractStoredState`'s canonical `translate_role ===
+          // "activation_banner"` marker is preserved across reloads.
+          // `synthetic:true + ignored:true` keeps it hidden from both
+          // the TUI and the LLM serializer; it exists purely as a
+          // database row carrying state metadata.
+          nextParts.push({
+            id: createSyntheticPartID(),
+            sessionID: input.sessionID,
+            messageID: output.message.id,
+            type: "text",
+            text: bannerText,
+            synthetic: true,
+            ignored: true,
+            metadata: {
               ...activeState,
               translate_role: "activation_banner",
               translate_spec_version: SPEC_VERSION,
-            }),
-          )
+            },
+          })
         }
 
         output.parts.splice(0, output.parts.length, ...(nextParts as typeof output.parts))