a2ui-project / a2ui-project/a2ui

`A2uiMessage.fromJson` accepts objects carrying more than one of `createSurface`/`updateComponents`/`updateDataModel`/`deleteSurface`

オープン
#1,895 コメント 1 件 リアクション 0 件 担当者 1 名 @andrewkolos が担当を希望しています GitHub で見る
component: genui P3 status: first-line-handled type: bug
主要言語
TypeScript
スター
16.4k
フォーク
1.3k
平均マージ
3日 15時間
マージ済み PR(30日)
134

説明

_↴ Ported from [flutter/genui#934](https://github.com/flutter/genui/issues/934) — originally opened by [andrewkolos](https://github.com/andrewkolos) on 2026-05-26._
_Original labels: front-line-handled_
_Original assignees: [andrewkolos](https://github.com/andrewkolos)_

---

[`A2uiMessage.fromJson`](https://github.com/flutter/genui/blob/ae53cc52b35b72312f01b80eeb565163e3c869d7/packages/genui/lib/src/model/a2ui_message.dart#L20). The message type is parsed in the order of 1) `createSurface`, 2) `updateComponents`, 3) `updateDataModel`, and then 4) `deleteSurface`). The function returns on the first match.

A payload that matches multiple of these checks will silently parse as whatever happens to come first instead of failing validation.

```dart
final message = A2uiMessage.fromJson({
'version': 'v0.9',
'createSurface': {'surfaceId': 's1', 'catalogId': 'c1'},
'updateComponents': {'surfaceId': 's1', 'components': []},
});
// Expected: A2uiValidationException
// Actual: parses as CreateSurface, drops updateComponents silently
```

I would normally consider this a corner case and not bother with fixing it, as I think an LLM is very unlikely to generate a message that matches multiple types. However, 1) I'd like `package:genui` to have as much parity with other renderers as possible ([`web_core` does include logic to catch this case](https://github.com/google/A2UI/blob/0e0375749bf8a940d57969d612e74f7221ad246e/renderers/web_core/src/v0_9/processing/message-processor.ts#L229-L262)) and 2) the 0.9 spec does explicitly specify `oneOf` and `additionalProperties: false` for each message type, so fixing this would also make the genui package follow the spec more closely.

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

この issue は、ファイル `packages/genui/lib/src/model/a2ui_message.dart` にある Dart パッケージ `genui` のものです。`A2uiMessage.fromJson` メソッドとその解析順序を確認してください。web_core renderer の検証ロジック(`renderers/web_core/src/v0_9/processing/message-processor.ts` の 229-262 行目)と比較してください。修正では、JSON ペイロードに指定されたメッセージフィールドのうちちょうど 1 つだけが含まれることを保証し、複数一致した場合は `A2uiValidationException` をスローする必要があります。無効なケースのテストを書いてください。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
dart, typescript
領域
backend-api-design
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
静か
明瞭さ
明確に書かれている
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。