Refactor adapter instructions into abstract method

libs/core/kiln_ai/adapters/base_adapter.py

@@ -31,11 +31,10 @@ async def invoke(self, input: str) -> Dict | str:
     async def _run(self, input: str) -> Dict | str:
         pass
 
-    # override for provider specific instructions (e.g. json format instead of tool call)
+    # adapter specific instructions (e.g. tool calling, json format, etc)
+    @abstractmethod
     def adapter_specific_instructions(self) -> str | None:
-        if self._is_structured:
-            return "Always respond with a tool call. Never respond with a human readable message."
-        return None
+        pass
 
 
 class BasePromptBuilder(metaclass=ABCMeta):

libs/core/kiln_ai/adapters/langchain_adapters.py

@@ -51,6 +51,12 @@ def __init__(
             prompt_builder.adapter = self
             self.prompt_builder = prompt_builder
 
+    def adapter_specific_instructions(self) -> str | None:
+        # TODO: would be better to explicitly use bind_tools:tool_choice="task_response" here
+        if self._is_structured:
+            return "Always respond with a tool call. Never respond with a human readable message."
+        return None
+
     async def _run(self, input: str) -> Dict | str:
         # TODO cleanup
         prompt = self.prompt_builder.build_prompt(input)

libs/core/kiln_ai/adapters/test_structured_output.py

@@ -51,6 +51,9 @@ def __init__(self, kiln_task: models.Task, response: Dict | str | None):
     async def _run(self, input: str) -> Dict | str:
         return self.response
 
+    def adapter_specific_instructions(self) -> str | None:
+        return None
+
 
 async def test_mock_unstructred_response(tmp_path):
     task = build_structured_output_test_task(tmp_path)