agentscope-ai / agentscope-ai/agentscope-java

[Bug]:MySQL AgentStateStore rejects Harness sandbox session IDs containing

Abierto
#2,583 3 comentarios 0 reacciones 0 asignados Ver en GitHub
area/core/memory area/core/tool area/extensions area/harness bug
Lenguaje dominante
Java
Estrellas
5.6k
Forks
1.3k
Merge medio
4 d 12 h
PR fusionados (30 d)
77

Descripción

```markdown
### Environment

- AgentScope Java: `2.0.1-sn-v2-SNAPSHOT`
- Java: 17
- Store: `MysqlAgentStateStore`
- Harness sandbox enabled

### Problem

When Harness persists sandbox state with session isolation, `SessionSandboxStateStore` generates a session ID like:

```text
sandbox/session/conv-a12d03e7-e213-4905-ad82-ad199ae1ea70
```

`MysqlAgentStateStore` then rejects it in `validateSessionId(...)`:

```text
java.lang.IllegalArgumentException:
AgentStateStore ID cannot contain path separators
```

Relevant call chain:

```text
SandboxManager.persistState(...)
-> SessionSandboxStateStore.save(...)
-> slotSessionId(...)
-> MysqlAgentStateStore.save(...)
-> validateSessionId(...)
```

### Expected behavior

Harness sandbox state should be persistable through `MysqlAgentStateStore`, or the sandbox store should encode/sanitize its internal scoped session IDs before delegating to `AgentStateStore`.

### Question

Should `SessionSandboxStateStore` use a storage-safe encoding for IDs such as `sandbox/session/`, or should `MysqlAgentStateStore` allow `/` in session IDs?

The database itself can store `/`; the failure comes from the application-level validation.

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.