fix: recipe params not being replaced all the time (#4207)

ui/desktop/src/components/ChatInput.tsx

@@ -288,17 +288,34 @@ export default function ChatInput({
     setHasUserTyped(false);
   }, [initialValue]); // Keep only initialValue as a dependency
 
+  // Track if we've already set the recipe prompt to avoid re-setting it
+  const hasSetRecipePromptRef = useRef(false);
+
   // Handle recipe prompt updates
   useEffect(() => {
-    // If recipe is accepted and we have an initial prompt, and no messages yet, set the prompt
+    // If recipe is accepted and we have an initial prompt, and no messages yet, and we haven't set it before
-    if (recipeAccepted && initialPrompt && messages.length === 0 && !displayValue.trim()) {
+    if (
+      recipeAccepted &&
+      initialPrompt &&
+      messages.length === 0 &&
+      !hasSetRecipePromptRef.current
+    ) {
       setDisplayValue(initialPrompt);
       setValue(initialPrompt);
+      hasSetRecipePromptRef.current = true;
       setTimeout(() => {
         textAreaRef.current?.focus();
       }, 0);
     }
-  }, [recipeAccepted, initialPrompt, messages.length, displayValue]);
+    // we don't need hasSetRecipePromptRef in the dependency array because it is a ref that persists across renders
+  }, [recipeAccepted, initialPrompt, messages.length]);
+
+  // Reset the recipe prompt flag when the recipe changes or messages are added
+  useEffect(() => {
+    if (messages.length > 0 || !recipeAccepted || !initialPrompt) {
+      hasSetRecipePromptRef.current = false;
+    }
+  }, [recipeAccepted, initialPrompt, messages.length]);
 
   // Draft functionality - load draft if no initial value or recipe
   useEffect(() => {

ui/desktop/src/hooks/useRecipeManager.ts

@@ -1,7 +1,7 @@
 import { useEffect, useMemo, useState, useRef } from 'react';
 import { createRecipe, Recipe, scanRecipe } from '../recipe';
 import { Message, createUserMessage } from '../types/message';
-import { updateSystemPromptWithParameters } from '../utils/providerUtils';
+import { updateSystemPromptWithParameters, substituteParameters } from '../utils/providerUtils';
 import { useChatContext } from '../contexts/ChatContext';
 
 interface LocationState {
@@ -119,18 +119,6 @@ export const useRecipeManager = (messages: Message[], locationState?: LocationSt
     setReadyForAutoUserPrompt(true);
   }, []);
 
-  // Substitute parameters in prompt
-  const substituteParameters = (prompt: string, params: Record<string, string>): string => {
-    let substitutedPrompt = prompt;
-
-    for (const key in params) {
-      // Escape special characters in the key (parameter) and match optional whitespace
-      const regex = new RegExp(`{{\\s*${key.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')}\\s*}}`, 'g');
-      substitutedPrompt = substitutedPrompt.replace(regex, params[key]);
-    }
-    return substitutedPrompt;
-  };
-
   // Get the recipe's initial prompt (always return the actual prompt, don't modify based on conversation state)
   const initialPrompt = useMemo(() => {
     if (!recipeConfig?.prompt || !recipeAccepted || recipeConfig?.isScheduledExecution) {

ui/desktop/src/utils/providerUtils.ts

@@ -51,7 +51,7 @@ There may be (but not always) some tools mentioned in the instructions which you
 `;
 
 // Helper function to substitute parameters in text
-const substituteParameters = (text: string, params: Record<string, string>): string => {
+export const substituteParameters = (text: string, params: Record<string, string>): string => {
   let substitutedText = text;
 
   for (const key in params) {