2415 Fix PR review comments: Redis chat memory dependency version, output schema, auth validation, shared constant

- Remove pinned Spring AI version, use BOM-managed version
- Add output schema to addMessages action
- Use CONVERSATION_ID constant instead of string literal in return map
- Validate username+password are provided together (fail fast)
- Move DEFAULT_KEY_PREFIX to shared constants class

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: ivicac

server/libs/modules/components/ai/agent/chat-memory/chat-memory-redis/build.gradle.kts

@@ -1,5 +1,5 @@
 dependencies {
     implementation("redis.clients:jedis")
-    implementation("org.springframework.ai:spring-ai-model-chat-memory-repository-redis:2.0.0-M1")
+    implementation("org.springframework.ai:spring-ai-model-chat-memory-repository-redis")
     implementation(project(":server:libs:platform:platform-component:platform-component-service"))
 }

server/libs/modules/components/ai/agent/chat-memory/chat-memory-redis/src/main/java/com/bytechef/component/ai/agent/chat/memory/redis/RedisChatMemoryComponentHandler.java

@@ -16,6 +16,7 @@
 
 package com.bytechef.component.ai.agent.chat.memory.redis;
 
+import static com.bytechef.component.ai.agent.chat.memory.redis.constant.RedisChatMemoryConstants.DEFAULT_KEY_PREFIX;
 import static com.bytechef.component.ai.agent.chat.memory.redis.constant.RedisChatMemoryConstants.HOST;
 import static com.bytechef.component.ai.agent.chat.memory.redis.constant.RedisChatMemoryConstants.KEY_PREFIX;
 import static com.bytechef.component.ai.agent.chat.memory.redis.constant.RedisChatMemoryConstants.PASSWORD;
@@ -62,7 +63,7 @@ public class RedisChatMemoryComponentHandler implements ComponentHandler {
             string(KEY_PREFIX)
                 .label("Key Prefix")
                 .description("The prefix for Redis keys used to store chat messages.")
-                .defaultValue("bytechef-chat-memory:")
+                .defaultValue(DEFAULT_KEY_PREFIX)
                 .required(false),
             string(TIME_TO_LIVE)
                 .label("Time to Live")

server/libs/modules/components/ai/agent/chat-memory/chat-memory-redis/src/main/java/com/bytechef/component/ai/agent/chat/memory/redis/action/RedisChatMemoryAddMessagesAction.java

@@ -23,8 +23,10 @@
 import static com.bytechef.component.ai.agent.chat.memory.redis.util.RedisChatMemoryUtils.getChatMemoryRepository;
 import static com.bytechef.component.definition.ComponentDsl.action;
 import static com.bytechef.component.definition.ComponentDsl.array;
+import static com.bytechef.component.definition.ComponentDsl.integer;
 import static com.bytechef.component.definition.ComponentDsl.object;
 import static com.bytechef.component.definition.ComponentDsl.option;
+import static com.bytechef.component.definition.ComponentDsl.outputSchema;
 import static com.bytechef.component.definition.ComponentDsl.string;
 
 import com.bytechef.component.definition.ActionContext;
@@ -69,6 +71,12 @@ public class RedisChatMemoryAddMessagesAction {
                                 .label("Content")
                                 .description("The content of the message.")
                                 .required(true))))
+        .output(
+            outputSchema(
+                object()
+                    .properties(
+                        string(CONVERSATION_ID),
+                        integer("messageCount"))))
         .perform(RedisChatMemoryAddMessagesAction::perform);
 
     private RedisChatMemoryAddMessagesAction() {
@@ -96,7 +104,7 @@ protected static Object perform(
         repository.saveAll(conversationId, existingMessages);
 
         return Map.of(
-            "conversationId", conversationId,
+            CONVERSATION_ID, conversationId,
             "messageCount", existingMessages.size());
     }
 

server/libs/modules/components/ai/agent/chat-memory/chat-memory-redis/src/main/java/com/bytechef/component/ai/agent/chat/memory/redis/constant/RedisChatMemoryConstants.java

@@ -22,6 +22,7 @@
 public class RedisChatMemoryConstants {
 
     public static final String CONVERSATION_ID = "conversationId";
+    public static final String DEFAULT_KEY_PREFIX = "bytechef-chat-memory:";
     public static final String HOST = "host";
     public static final String KEY_PREFIX = "keyPrefix";
     public static final String MESSAGES = "messages";

server/libs/modules/components/ai/agent/chat-memory/chat-memory-redis/src/main/java/com/bytechef/component/ai/agent/chat/memory/redis/util/RedisChatMemoryUtils.java

@@ -16,6 +16,7 @@
 
 package com.bytechef.component.ai.agent.chat.memory.redis.util;
 
+import static com.bytechef.component.ai.agent.chat.memory.redis.constant.RedisChatMemoryConstants.DEFAULT_KEY_PREFIX;
 import static com.bytechef.component.ai.agent.chat.memory.redis.constant.RedisChatMemoryConstants.HOST;
 import static com.bytechef.component.ai.agent.chat.memory.redis.constant.RedisChatMemoryConstants.KEY_PREFIX;
 import static com.bytechef.component.ai.agent.chat.memory.redis.constant.RedisChatMemoryConstants.PASSWORD;
@@ -34,8 +35,6 @@
  */
 public class RedisChatMemoryUtils {
 
-    private static final String DEFAULT_KEY_PREFIX = "bytechef-chat-memory:";
-
     public static ChatMemoryRepository getChatMemoryRepository(Parameters connectionParameters) {
         JedisPooled jedisClient = getJedisClient(connectionParameters);
         String keyPrefix = connectionParameters.getString(KEY_PREFIX, DEFAULT_KEY_PREFIX);
@@ -59,7 +58,11 @@ public static JedisPooled getJedisClient(Parameters connectionParameters) {
         String username = connectionParameters.getString(USERNAME);
         String password = connectionParameters.getString(PASSWORD);
 
-        if (username != null && !username.isBlank() && password != null && !password.isBlank()) {
+        if (username != null && !username.isBlank()) {
+            if (password == null || password.isBlank()) {
+                throw new IllegalArgumentException("Password is required when username is provided");
+            }
+
             return new JedisPooled(host, port, username, password);
         } else if (password != null && !password.isBlank()) {
             return new JedisPooled(host, port, null, password);