agentscope-ai / agentscope-ai/agentscope-java

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

Aberta
#2,583 3 comentários 0 reações 0 responsáveis Ver no GitHub
area/core/memory area/core/tool area/extensions area/harness bug
Linguagem predominante
Java
Estrelas
5.6k
Forks
1.3k
Merge médio
4d 12h
PRs com merge (30d)
77

Descrição

```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.

Guia de contribuição

Abrir o guia de contribuição

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.