GeneralBots/botserver
2
1
Fork 0
Code Issues Pull requests Projects Releases 323 Packages Wiki Activity Actions

Fix: Use bot_id instead of user_id in suggestion Redis keys
All checks were successful
BotServer CI / build (push) Successful in 12m46s
Details

Browse source 
- Changed all suggestion key formats from suggestions:user_id:session_id to suggestions:bot_id:session_id
- Fixes bug where suggestions were stored under wrong key, preventing frontend from retrieving them
- Affects: CLEAR SUGGESTIONS, ADD SUGGESTION, ADD SUGGESTION TEXT, ADD_SUGGESTION_TOOL
- Impact: Suggestions now correctly associated with bot, not user
This commit is contained in:
Rodrigo Rodriguez (Pragmatismo) 2026-03-18 10:39:27 -03:00
parent 346c83871a
commit ec4fcc094a
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
src/basic/keywords/add_suggestion.rs
View file

@ -25,7 +25,7 @@ pub fn clear_suggestions_keyword(
engine engine
.register_custom_syntax(["CLEAR", "SUGGESTIONS"], true, move |_context, _inputs| { .register_custom_syntax(["CLEAR", "SUGGESTIONS"], true, move |_context, _inputs| {
if let Some(cache_client) = &cache { if let Some(cache_client) = &cache {
let redis_key = format!("suggestions:{}:{}", user_session.user_id, user_session.id); let redis_key = format!("suggestions:{}:{}", user_session.bot_id, user_session.id);
let mut conn: redis::Connection = match cache_client.get_connection() { let mut conn: redis::Connection = match cache_client.get_connection() {
Ok(conn) => conn, Ok(conn) => conn,
Err(e) => { Err(e) => {
@ -200,7 +200,7 @@ fn add_context_suggestion(
button_text: &str, button_text: &str,
) -> Result<(), Box<rhai::EvalAltResult>> { ) -> Result<(), Box<rhai::EvalAltResult>> {
if let Some(cache_client) = cache { if let Some(cache_client) = cache {
let redis_key = format!("suggestions:{}:{}", user_session.user_id, user_session.id); let redis_key = format!("suggestions:{}:{}", user_session.bot_id, user_session.id);
let suggestion = json!({ let suggestion = json!({
"type": "context", "type": "context",
@ -236,7 +236,7 @@ fn add_context_suggestion(
let active_key = format!( let active_key = format!(
"active_context:{}:{}", "active_context:{}:{}",
user_session.user_id, user_session.id user_session.bot_id, user_session.id
); );
let _: Result<i64, redis::RedisError> = redis::cmd("HSET") let _: Result<i64, redis::RedisError> = redis::cmd("HSET")
@ -261,7 +261,7 @@ fn add_text_suggestion(
button_text: &str, button_text: &str,
) -> Result<(), Box<rhai::EvalAltResult>> { ) -> Result<(), Box<rhai::EvalAltResult>> {
if let Some(cache_client) = cache { if let Some(cache_client) = cache {
let redis_key = format!("suggestions:{}:{}", user_session.user_id, user_session.id); let redis_key = format!("suggestions:{}:{}", user_session.bot_id, user_session.id);
let suggestion = json!({ let suggestion = json!({
"type": "text_value", "type": "text_value",
@ -316,7 +316,7 @@ fn add_tool_suggestion(
tool_name, button_text tool_name, button_text
); );
if let Some(cache_client) = cache { if let Some(cache_client) = cache {
let redis_key = format!("suggestions:{}:{}", user_session.user_id, user_session.id); let redis_key = format!("suggestions:{}:{}", user_session.bot_id, user_session.id);
info!("Adding suggestion to Redis key: {}", redis_key); info!("Adding suggestion to Redis key: {}", redis_key);
// Create action object and serialize it to JSON string // Create action object and serialize it to JSON string