Upgrade to 1.2.0 and remove Embabel repository

Author: johnsonr

.idea/jarRepositories.xml

@@ -18,7 +18,7 @@
 
     <properties>
         <java.version>21</java.version>
-        <embabel-agent.version>0.1.4</embabel-agent.version>
+        <embabel-agent.version>0.2.0</embabel-agent.version>
     </properties>
 
     <dependencies>
@@ -75,24 +75,25 @@
         </plugins>
     </build>
 
-    <repositories>
-        <repository>
-            <id>embabel-releases</id>
-            <url>https://repo.embabel.com/artifactory/libs-release</url>
-            <snapshots>
-                <enabled>false</enabled>
-            </snapshots>
-        </repository>
-        <repository>
-            <id>embabel-snapshots</id>
-            <url>https://repo.embabel.com/artifactory/libs-snapshot</url>
-            <releases>
-                <enabled>false</enabled>
-            </releases>
-            <snapshots>
-                <enabled>true</enabled>
-            </snapshots>
-        </repository>
-    </repositories>
+    <!-- Uncomment these lines to enable Embabel repositories and use snapshots -->
+    <!--    <repositories>-->
+    <!--        <repository>-->
+    <!--            <id>embabel-releases</id>-->
+    <!--            <url>https://repo.embabel.com/artifactory/libs-release</url>-->
+    <!--            <snapshots>-->
+    <!--                <enabled>false</enabled>-->
+    <!--            </snapshots>-->
+    <!--        </repository>-->
+    <!--        <repository>-->
+    <!--            <id>embabel-snapshots</id>-->
+    <!--            <url>https://repo.embabel.com/artifactory/libs-snapshot</url>-->
+    <!--            <releases>-->
+    <!--                <enabled>false</enabled>-->
+    <!--            </releases>-->
+    <!--            <snapshots>-->
+    <!--                <enabled>true</enabled>-->
+    <!--            </snapshots>-->
+    <!--        </repository>-->
+    <!--    </repositories>-->
 
 </project>

pom.xml

@@ -0,0 +1,9 @@
+#!/bin/bash
+
+# We only compile rather than run tests to avoid the need for an Open AI API key
+echo "Compiling in a Docker container to make sure dependencies can be downloaded"
+
+# Navigate to project root (two levels up from /scripts/diagnostic)
+cd "$(dirname "$0")/../.." || exit 1
+
+docker run --rm -v "$(pwd)":/app -w /app maven:3.9-eclipse-temurin-21 mvn clean compile

scripts/diagnostic/clean_compile.sh

@@ -18,7 +18,7 @@ void testWriteAndReviewAgent() {
         context.expectResponse(new WriteAndReviewAgent.Story("One upon a time Sir Galahad . . "));
 
         var agent = new WriteAndReviewAgent(200, 400);
-        var story = agent.craftStory(new UserInput("Tell me a story about a brave knight", Instant.now()), context);
+        var story = agent.craftStory(new UserInput("Tell me a story about a brave knight", Instant.now()), context.ai());
 
         var prompt = promptRunner.getLlmInvocations().getFirst().getMessages().getFirst().getContent();
         assertTrue(prompt.contains("knight"), "Expected prompt to contain 'knight'");
@@ -32,7 +32,7 @@ void testReview() {
         var story = new WriteAndReviewAgent.Story("Once upon a time, Sir Galahad...");
         var context = FakeOperationContext.create();
         context.expectResponse("A thrilling tale of bravery and adventure!");
-        var review = agent.reviewStory(userInput, story, context);
+        var review = agent.reviewStory(userInput, story, context.ai());
         var llmInvocation = context.getLlmInvocations().getFirst();
         var prompt = llmInvocation.getMessages().getFirst().getContent();
         assertTrue(prompt.contains("knight"), "Expected prompt to contain 'knight'");