Skip to content

Update a2ui-core blueprint#1991

Open
nan-yu wants to merge 1 commit into
a2ui-project:mainfrom
nan-yu:a2ui-core-api
Open

Update a2ui-core blueprint#1991
nan-yu wants to merge 1 commit into
a2ui-project:mainfrom
nan-yu:a2ui-core-api

Conversation

@nan-yu

@nan-yu nan-yu commented Jul 12, 2026

Copy link
Copy Markdown
Collaborator

Description

Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.

List which issues are fixed by this PR. For larger changes, raising an issue first helps reduce redundant work.

Pre-launch Checklist

One time:

For this PR:

  • I have added updates to the CHANGELOG.
  • I updated/added relevant documentation.
  • My code changes (if any) have tests.
  • If my branch is on fork, I have verified that scripts/e2e_test.sh passes.

If you need help, consider asking for advice on the discussion board.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the A2UI Core blueprint to document the Node Graph Layer, multi-version protocol branching, the validation layer (A2uiValidator), and a structured exception hierarchy. The review feedback correctly identifies that the v0_9_1 directory is missing from the documented file structures for both schema/ and basic_catalog/ despite being listed as a supported version. Additionally, it is recommended to add the targetVersion property to the ValidationConfig interface to match the SDK's capabilities.

Comment thread blueprints/modules/a2ui_core.blueprint.md
Comment thread blueprints/modules/a2ui_core.blueprint.md
Comment thread blueprints/modules/a2ui_core.blueprint.md
- **`NodeGraph`**: Binds directly to a `SurfaceModel` to construct and manage the root node hierarchy (`rootNode`), handling dynamic updates, node instantiation, and cleanup lifecycle callbacks.

```typescript
class ComponentNode {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Want to leave this out for now so that @andrewkolos can add it later? Or else if this is exactly the same as Andrew's design, let's just submit.

Up to what you prefer, @andrewkolos !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants