From 8d303b2fb5306e4a830bbd3bb9099ad46f88ef94 Mon Sep 17 00:00:00 2001
From: MyoJun <168921419+hyoj-dev@users.noreply.github.com>
Date: Tue, 21 Jul 2026 16:37:54 +0900
Subject: [PATCH] =?UTF-8?q?fix:=20AI=20=ED=94=84=EB=A1=AC=ED=94=84?=
 =?UTF-8?q?=ED=8A=B8=20=EC=A1=B0=ED=9A=8C=20=EC=BB=A4=EB=84=A5=EC=85=98=20?=
 =?UTF-8?q?=EC=A1=B0=EA=B8=B0=20=EB=B0=98=ED=99=98?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../adapter/integration/ai/FeedbackPrompts.kt |  3 +
 .../ai/FeedbackPromptsTransactionTest.kt      | 86 +++++++++++++++++++
 2 files changed, 89 insertions(+)
 create mode 100644 src/test/kotlin/com/didit/adapter/integration/ai/FeedbackPromptsTransactionTest.kt

diff --git a/src/main/kotlin/com/didit/adapter/integration/ai/FeedbackPrompts.kt b/src/main/kotlin/com/didit/adapter/integration/ai/FeedbackPrompts.kt
index 9121e415..3c5d463c 100644
--- a/src/main/kotlin/com/didit/adapter/integration/ai/FeedbackPrompts.kt
+++ b/src/main/kotlin/com/didit/adapter/integration/ai/FeedbackPrompts.kt
@@ -7,6 +7,7 @@ import com.didit.domain.shared.Job
 import org.slf4j.LoggerFactory
 import org.springframework.core.io.ClassPathResource
 import org.springframework.stereotype.Component
+import org.springframework.transaction.annotation.Transactional
 
 @Component
 class FeedbackPrompts(
@@ -14,6 +15,7 @@ class FeedbackPrompts(
 ) {
     private val logger = LoggerFactory.getLogger(FeedbackPrompts::class.java)
 
+    @Transactional(readOnly = true)
     fun buildDeepQuestionPrompt(
         job: Job?,
         answers: List<String>,
@@ -25,6 +27,7 @@ class FeedbackPrompts(
             .replace("{{q3}}", answers.getOrElse(2) { "" })
     }
 
+    @Transactional(readOnly = true)
     fun buildSummaryPrompt(
         job: Job?,
         allAnswers: List<String>,
diff --git a/src/test/kotlin/com/didit/adapter/integration/ai/FeedbackPromptsTransactionTest.kt b/src/test/kotlin/com/didit/adapter/integration/ai/FeedbackPromptsTransactionTest.kt
new file mode 100644
index 00000000..0ea8d141
--- /dev/null
+++ b/src/test/kotlin/com/didit/adapter/integration/ai/FeedbackPromptsTransactionTest.kt
@@ -0,0 +1,86 @@
+package com.didit.adapter.integration.ai
+
+import com.didit.application.prompt.required.PromptRepository
+import com.didit.domain.prompt.Prompt
+import com.didit.domain.prompt.PromptJobType
+import com.didit.domain.prompt.PromptType
+import com.didit.domain.shared.Job
+import org.assertj.core.api.Assertions.assertThat
+import org.junit.jupiter.api.Test
+import org.mockito.kotlin.mock
+import org.mockito.kotlin.whenever
+import org.springframework.beans.factory.annotation.Autowired
+import org.springframework.context.annotation.Bean
+import org.springframework.context.annotation.Configuration
+import org.springframework.jdbc.datasource.DataSourceTransactionManager
+import org.springframework.jdbc.datasource.DriverManagerDataSource
+import org.springframework.test.context.junit.jupiter.SpringJUnitConfig
+import org.springframework.transaction.PlatformTransactionManager
+import org.springframework.transaction.annotation.EnableTransactionManagement
+import org.springframework.transaction.support.TransactionSynchronizationManager
+import javax.sql.DataSource
+
+@SpringJUnitConfig(FeedbackPromptsTransactionTest.Config::class)
+class FeedbackPromptsTransactionTest {
+    @Autowired
+    private lateinit var feedbackPrompts: FeedbackPrompts
+
+    @Autowired
+    private lateinit var promptRepository: PromptRepository
+
+    @Test
+    fun `buildSummaryPrompt - reads prompt in transaction and releases it before returning`() {
+        whenever(promptRepository.findByJobTypeAndPromptType(PromptJobType.DEVELOPER, PromptType.SUMMARY)).thenAnswer {
+            assertThat(TransactionSynchronizationManager.isActualTransactionActive()).isTrue()
+            Prompt(
+                jobType = PromptJobType.DEVELOPER,
+                promptType = PromptType.SUMMARY,
+                content = "{{q1}} {{q2}} {{q3}} {{q4}} {{deepQuestion}}",
+            )
+        }
+
+        val result = feedbackPrompts.buildSummaryPrompt(Job.DEVELOPER, listOf("1", "2", "3", "4"), "deep")
+
+        assertThat(result).isEqualTo("1 2 3 4 deep")
+        assertThat(TransactionSynchronizationManager.isActualTransactionActive()).isFalse()
+    }
+
+    @Test
+    fun `buildDeepQuestionPrompt - reads prompt in transaction and releases it before returning`() {
+        whenever(promptRepository.findByJobTypeAndPromptType(PromptJobType.DEVELOPER, PromptType.DEEP_QUESTION)).thenAnswer {
+            assertThat(TransactionSynchronizationManager.isActualTransactionActive()).isTrue()
+            Prompt(
+                jobType = PromptJobType.DEVELOPER,
+                promptType = PromptType.DEEP_QUESTION,
+                content = "{{q1}} {{q2}} {{q3}}",
+            )
+        }
+
+        val result = feedbackPrompts.buildDeepQuestionPrompt(Job.DEVELOPER, listOf("1", "2", "3"))
+
+        assertThat(result).isEqualTo("1 2 3")
+        assertThat(TransactionSynchronizationManager.isActualTransactionActive()).isFalse()
+    }
+
+    @Configuration
+    @EnableTransactionManagement
+    class Config {
+        @Bean
+        fun promptRepository(): PromptRepository = mock()
+
+        @Bean
+        fun feedbackPrompts(promptRepository: PromptRepository) = FeedbackPrompts(promptRepository)
+
+        @Bean
+        fun dataSource(): DataSource =
+            DriverManagerDataSource().apply {
+                setDriverClassName("org.h2.Driver")
+                url = "jdbc:h2:mem:feedback-prompts-transaction-test;DB_CLOSE_DELAY=-1"
+                username = "sa"
+                password = ""
+            }
+
+        @Bean
+        fun transactionManager(dataSource: DataSource): PlatformTransactionManager = DataSourceTransactionManager(dataSource)
+    }
+}