Remove redudant session_id from document path #32

A8072846 · 2026-03-11T17:36:48Z

A8072846 commented

2026-03-11 17:36:48 +00:00

Session IDs (#28)

Solución

Modificado el método _session_ref() en session.py para usar únicamente user_id en el path del documento, eliminando la redundancia del session_id.

Cambio realizado:

# Antes
f"{app_name}__{user_id}__{session_id}"

# Después  
f"{app_name}__{user_id}"
Resultado
Antes: adk_sessions/va_agent__555-1000__555-1000
Ahora: adk_sessions/va_agent__555-1000
Impacto
 -Elimina redundancia en document IDs
 -Clarifica que cada usuario tiene una única sesión continua
 -Paths más limpios y cortos en Firestore
⚠️ Sesiones existentes con formato antiguo no serán encontradas (usuarios crearán nuevas sesiones automáticamente)
Archivos Modificados
src/va_agent/session.py (línea 100): Cambio en formato de document ID
src/va_agent/session.py (línea 45-47): Actualización de docstring
Detalles Técnicos
El cambio afecta únicamente al método _session_ref(). Todos los demás métodos (_events_col, create_session, get_session, delete_session, append_event) funcionan sin modificaciones ya que internamente llaman a _session_ref().

-Screenshot de como se guarda en Firestore:

## Session IDs (#28) ### Solución Modificado el método `_session_ref()` en `session.py` para usar únicamente `user_id` en el path del documento, eliminando la redundancia del `session_id`. **Cambio realizado:** ```python # Antes f"{app_name}__{user_id}__{session_id}" # Después f"{app_name}__{user_id}" Resultado Antes: adk_sessions/va_agent__555-1000__555-1000 Ahora: adk_sessions/va_agent__555-1000 Impacto -Elimina redundancia en document IDs -Clarifica que cada usuario tiene una única sesión continua -Paths más limpios y cortos en Firestore ⚠️ Sesiones existentes con formato antiguo no serán encontradas (usuarios crearán nuevas sesiones automáticamente) Archivos Modificados src/va_agent/session.py (línea 100): Cambio en formato de document ID src/va_agent/session.py (línea 45-47): Actualización de docstring Detalles Técnicos El cambio afecta únicamente al método _session_ref(). Todos los demás métodos (_events_col, create_session, get_session, delete_session, append_event) funcionan sin modificaciones ya que internamente llaman a _session_ref(). -Screenshot de como se guarda en Firestore:

2e2967a4-4003-42aa-981f-0f0d12f3e86b.jpg

54 KiB

A8072846 added 1 commit 2026-03-11 17:36:49 +00:00

Remove redudant session_id from document path

CI / ci (pull_request) Failing after 12s

Details

8826d84e59

A8065384 added 1 commit 2026-03-11 23:10:01 +00:00

Merge branch 'main' into issue/session

CI / ci (pull_request) Failing after 12s

Details

f3afdff515

CI / ci (pull_request) Failing after 12s

Details

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin issue/session:issue/session

git checkout issue/session

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: va/agent#32