Remove insight feature from digest

Author: johanmha

src/main/java/com/codebrief/model/Digest.java

@@ -19,6 +19,5 @@ public class Digest {
     private List<NewsItem> topUpdates;
     private List<NewsItem> quickMentions;
     private List<NewsItem> communityBuzz;
-    private String insight;
     private String summary;
 }

src/main/java/com/codebrief/service/GeminiService.java

@@ -90,15 +90,13 @@ private String buildPrompt(List<NewsItem> items) {
         prompt.append("\nTASK:\n");
         prompt.append("1. Identify the TOP 3 most important updates (framework releases, major news)\n");
         prompt.append("2. List 3-5 quick mentions (interesting but less critical)\n");
-        prompt.append("3. Highlight 2-3 community discussions (Reddit/HN posts with high engagement)\n");
-        prompt.append("4. Provide a 1-sentence insight about trends you notice\n\n");
+        prompt.append("3. Highlight 2-3 community discussions (Reddit/HN posts with high engagement)\n\n");
 
         prompt.append("OUTPUT FORMAT (respond with ONLY this JSON, no markdown formatting):\n");
         prompt.append("{\n");
         prompt.append("  \"topUpdates\": [1, 3, 7],  // indices of top 3 items\n");
         prompt.append("  \"quickMentions\": [2, 5, 9, 12],  // indices of 3-5 items\n");
-        prompt.append("  \"communityBuzz\": [4, 8],  // indices of 2-3 discussion items\n");
-        prompt.append("  \"insight\": \"One sentence about what's trending\"\n");
+        prompt.append("  \"communityBuzz\": [4, 8]  // indices of 2-3 discussion items\n");
         prompt.append("}\n");
 
         return prompt.toString();
@@ -172,14 +170,12 @@ private Digest parseDigestResponse(String response, List<NewsItem> items) {
             List<NewsItem> topUpdates = extractItems(json, "topUpdates", items);
             List<NewsItem> quickMentions = extractItems(json, "quickMentions", items);
             List<NewsItem> communityBuzz = extractItems(json, "communityBuzz", items);
-            String insight = json.has("insight") ? json.get("insight").getAsString() : "";
 
             return Digest.builder()
                     .date(LocalDate.now().format(DateTimeFormatter.ofPattern("EEEE, MMMM d, yyyy")))
                     .topUpdates(topUpdates)
                     .quickMentions(quickMentions)
                     .communityBuzz(communityBuzz)
-                    .insight(insight)
                     .build();
 
         } catch (Exception e) {
@@ -212,7 +208,6 @@ private Digest createEmptyDigest() {
                 .topUpdates(new ArrayList<>())
                 .quickMentions(new ArrayList<>())
                 .communityBuzz(new ArrayList<>())
-                .insight("No news items collected today.")
                 .build();
     }
 
@@ -236,7 +231,6 @@ private Digest createFallbackDigest(List<NewsItem> items) {
                 .topUpdates(topUpdates)
                 .quickMentions(quickMentions)
                 .communityBuzz(communityBuzz)
-                .insight("Frontend development continues to evolve rapidly.")
                 .build();
     }
 }

src/main/java/com/codebrief/service/SlackService.java

@@ -79,12 +79,6 @@ private String formatDigest(Digest digest) {
             sb.append("\n");
         }
 
-        // Insight
-        if (digest.getInsight() != null && !digest.getInsight().isEmpty()) {
-            sb.append("💡 *Insight*\n");
-            sb.append("_").append(digest.getInsight()).append("_\n\n");
-        }
-
         sb.append("────────────────────────────────\n");
         sb.append("_Curated by Gemini AI • Running on GitHub Actions_");
 

src/test/java/com/codebrief/service/GeminiServiceTest.java

@@ -58,7 +58,6 @@ void testProcessNewsItems_EmptyList() {
         assertTrue(result.getTopUpdates().isEmpty());
         assertTrue(result.getQuickMentions().isEmpty());
         assertTrue(result.getCommunityBuzz().isEmpty());
-        assertEquals("No news items collected today.", result.getInsight());
     }
 
     @Test
@@ -67,7 +66,7 @@ void testProcessNewsItems_WithValidResponse() throws Exception {
         List<NewsItem> newsItems = createTestNewsItems();
 
         // This is what callGeminiAPI returns (the extracted text from the API response)
-        String extractedJsonText = "{\"topUpdates\": [1, 2], \"quickMentions\": [3], \"communityBuzz\": [4], \"insight\": \"Test insight\"}";
+        String extractedJsonText = "{\"topUpdates\": [1, 2], \"quickMentions\": [3], \"communityBuzz\": [4]}";
 
         when(retryTemplate.execute(any())).thenAnswer(invocation -> {
             return extractedJsonText;
@@ -82,7 +81,6 @@ void testProcessNewsItems_WithValidResponse() throws Exception {
         assertEquals(2, result.getTopUpdates().size());
         assertEquals(1, result.getQuickMentions().size());
         assertEquals(1, result.getCommunityBuzz().size());
-        assertEquals("Test insight", result.getInsight());
     }
 
     @Test
@@ -101,7 +99,6 @@ void testProcessNewsItems_APIFailure_UsesFallback() throws Exception {
         assertNotNull(result.getTopUpdates());
         assertNotNull(result.getQuickMentions());
         assertNotNull(result.getCommunityBuzz());
-        assertEquals("Frontend development continues to evolve rapidly.", result.getInsight());
     }
 
     @Test
@@ -120,7 +117,6 @@ void testProcessNewsItems_InvalidJSONResponse_UsesFallback() throws Exception {
         // Assert - Should use fallback logic
         assertNotNull(result);
         assertNotNull(result.getDate());
-        assertEquals("Frontend development continues to evolve rapidly.", result.getInsight());
     }
 
     @Test

src/test/java/com/codebrief/service/SlackServiceTest.java

@@ -94,7 +94,6 @@ void testPostDigest_EmptyDigest() throws Exception {
                 .topUpdates(List.of())
                 .quickMentions(List.of())
                 .communityBuzz(List.of())
-                .insight("")
                 .build();
 
         when(retryTemplate.execute(any())).thenAnswer(invocation -> {
@@ -140,7 +139,6 @@ void testPostDigest_FormatsMessageCorrectly() throws Exception {
                 .topUpdates(Arrays.asList(githubItem))
                 .quickMentions(List.of())
                 .communityBuzz(Arrays.asList(redditItem))
-                .insight("Server-side rendering is trending")
                 .build();
 
         ArgumentCaptor<String> messageCaptor = ArgumentCaptor.forClass(String.class);
@@ -171,7 +169,6 @@ void testPostDigest_HandlesItemsWithoutScores() throws Exception {
                 .topUpdates(Arrays.asList(itemWithoutScore))
                 .quickMentions(List.of())
                 .communityBuzz(List.of())
-                .insight("Test insight")
                 .build();
 
         when(retryTemplate.execute(any())).thenAnswer(invocation -> {
@@ -200,7 +197,6 @@ void testPostDigest_MultipleItemsInEachSection() throws Exception {
                         createNewsItem("Discussion 1", "Reddit", 450),
                         createNewsItem("Discussion 2", "HackerNews", 350)
                 ))
-                .insight("Multiple items test")
                 .build();
 
         when(retryTemplate.execute(any())).thenAnswer(invocation -> {
@@ -245,7 +241,6 @@ private Digest createTestDigest() {
                 .communityBuzz(Arrays.asList(
                         createNewsItem("Discussion", "Reddit", 450)
                 ))
-                .insight("Test insight about trends")
                 .build();
     }