Merge pull request #992 from xyloid/fix/react_instr_from_sig

fix(react): `ReAct` doesn't use `instructions` from the given `Signature`

Author: arnavsinghvi11

dspy/predict/react.py

@@ -28,11 +28,16 @@ def __init__(self, signature, max_iters=5, num_results=3, tools=None):
         inputs_ = ", ".join([f"`{k}`" for k in self.input_fields.keys()])
         outputs_ = ", ".join([f"`{k}`" for k in self.output_fields.keys()])
 
-        instr = [
+        instr = []
+        
+        if self.signature.instructions is not None:
+            instr.append(f"{self.signature.instructions}\n")
+        
+        instr.extend([
             f"You will be given {inputs_} and you will respond with {outputs_}.\n",
             "To do this, you will interleave Thought, Action, and Observation steps.\n",
             "Thought can reason about the current situation, and Action can be the following types:\n",
-        ]
+        ])
 
         self.tools["Finish"] = dspy.Example(
             name="Finish",

tests/predict/test_react.py

@@ -154,3 +154,16 @@ def test_custom_tools():
         "Thought 3: Even more thoughts\n\n"
         "Action 3: Finish[baz]"
     )
+
+
+def test_signature_instructions():
+    class ExampleSignature(dspy.Signature):
+        """You are going to generate output based on input."""
+
+        input = dspy.InputField()
+        output = dspy.OutputField()
+
+    react = dspy.ReAct(ExampleSignature)
+
+    assert react.react[0].signature.instructions is not None
+    assert react.react[0].signature.instructions.startswith("You are going to generate output based on input.")