Merge pull request #408 from Opencode-DCP/dev

merge dev into master

Author: Tarquinen

.gitignore

@@ -40,4 +40,10 @@ test-update.ts
 docs/
 SCHEMA_NOTES.md
 
-repomix-output.xml
+repomix-output.xml
+
+# Python
+__pycache__/
+*.pyc
+*.pyo
+*.pyd

lib/hooks.ts

@@ -92,7 +92,12 @@ export function createSystemPromptHandler(
             return
         }
 
-        output.system.push(renderSystemPrompt(flags))
+        const newPrompt = renderSystemPrompt(flags)
+        if (output.system.length > 0) {
+            output.system[output.system.length - 1] += "\n\n" + newPrompt
+        } else {
+            output.system.push(newPrompt)
+        }
     }
 }
 

lib/messages/inject.ts

@@ -8,9 +8,10 @@ import {
     extractParameterKey,
     createSyntheticTextPart,
     createSyntheticToolPart,
-    isIgnoredUserMessage,
     appendMessageIdTagToToolOutput,
     findLastToolPart,
+    isIgnoredUserMessage,
+    rejectsTextParts,
 } from "./utils"
 import { getFilePathsFromParameters, isProtected } from "../protected-file-patterns"
 import { getLastUserMessage, isMessageCompacted } from "../shared-utils"
@@ -314,13 +315,22 @@ export const insertPruneToolContext = (
         )
         lastNonIgnoredMessage.parts.push(textPart)
     } else {
-        const toolPart = createSyntheticToolPart(
-            lastNonIgnoredMessage,
-            combinedContent,
-            modelId ?? "",
-            `${lastNonIgnoredMessage.info.id}:context`,
-        )
-        lastNonIgnoredMessage.parts.push(toolPart)
+        if (rejectsTextParts(modelId ?? "")) {
+            const toolPart = createSyntheticToolPart(
+                lastNonIgnoredMessage,
+                combinedContent,
+                modelId ?? "",
+                `${lastNonIgnoredMessage.info.id}:context`,
+            )
+            lastNonIgnoredMessage.parts.push(toolPart)
+        } else {
+            const textPart = createSyntheticTextPart(
+                lastNonIgnoredMessage,
+                combinedContent,
+                `${lastNonIgnoredMessage.info.id}:context`,
+            )
+            lastNonIgnoredMessage.parts.push(textPart)
+        }
     }
 }
 
@@ -365,6 +375,10 @@ export const insertMessageIdContext = (
             continue
         }
 
-        message.parts.push(createSyntheticToolPart(message, tag, toolModelId, messageIdSeed))
+        if (rejectsTextParts(toolModelId)) {
+            message.parts.push(createSyntheticToolPart(message, tag, toolModelId, messageIdSeed))
+        } else {
+            message.parts.push(createSyntheticTextPart(message, tag, messageIdSeed))
+        }
     }
 }

lib/messages/utils.ts

@@ -19,6 +19,11 @@ const isGeminiModel = (modelID: string): boolean => {
     return lowerModelID.includes("gemini")
 }
 
+export const rejectsTextParts = (modelID: string): boolean => {
+    const lowerModelID = modelID.toLowerCase()
+    return lowerModelID.includes("claude")
+}
+
 export const createSyntheticUserMessage = (
     baseMessage: WithParts,
     content: string,