a2aproject / a2aproject/a2a-js
[Bug]: toCoreMessage defaults optional taskId to "" instead of undefined
- Lingua principale
- TypeScript
- Stelle
- 613
- Fork
- 169
- Merge medio
- 1g 6h
- PR unite (30g)
- 21
Descrizione
### What happened?
When a v0.3 message does not have a `taskId`, `toCoreMessage` in the compat layer sets it to `""` instead of leaving it as `undefined`:
```
// dist/compat/v0_3/client/index.js, line 996
taskId: compatMsg.taskId ?? "",
```
`taskId` is optional in v0.3, so not every message has one. Setting it to `""` causes problems for consumers that check whether `taskId` is present, because `""` looks like a value but is not a valid task ID. In JavaScript, an optional field that is not set should be `undefined`, not `""`.
### Relevant log output
```shell
```
### Code of Conduct
- [x] I agree to follow this project's Code of Conduct
Guida per i contributori
Apri la guida per i contributori
Valutazione
Questa issue non è ancora stata valutata.