Skip to content

[ZEPPELIN-6741] Define shared AI Assistant UI contracts and message history behavior - #5561

Open
voidmatcha wants to merge 4 commits into
apache:branch-voidmatcha-assistant-prerequisites-tempfrom
voidmatcha:voidmatcha/assistant-v2-3-contracts
Open

voidmatcha wants to merge 4 commits into
apache:branch-voidmatcha-assistant-prerequisites-tempfrom
voidmatcha:voidmatcha/assistant-v2-3-contracts

Conversation

@voidmatcha

@voidmatcha voidmatcha commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

What is this PR for?

Define the shared types and message history behavior used by the Assistant UI and its host. This gives subsequent UI work a common contract to build against.

Depends on #5558.

What type of PR is it?

Improvement

Todos

What is the Jira issue?

ZEPPELIN-6741

How should this be tested?

Run from zeppelin-web-angular/:

npm run build-project:sdk
npm run typecheck:react
npm --prefix projects/zeppelin-react test -- src/entities/assistant/model/messageHistory.spec.ts

Screenshots (if appropriate)

N/A

Questions:

  • Does the license files need to update? No
  • Is there breaking changes for older versions? No
  • Does this needs documentation? No

@voidmatcha
voidmatcha changed the base branch from temp/voidmatcha/assistant-prerequisites to branch-voidmatcha-assistant-prerequisites-temp October 9, 2026 01:35
@voidmatcha voidmatcha closed this Oct 9, 2026
@voidmatcha voidmatcha reopened this Oct 9, 2026
@voidmatcha

voidmatcha commented Oct 9, 2026 •

Copy link
Copy Markdown
Member Author

This PR defines the types shared by the Assistant UI and its host, and adds the conversion from stored message history to visible UI messages. The current diff is +211/-0 across five files. Excluding 63 lines in *.spec.* tests, the review diff is +148/-0. There is no lockfile change.

This depends on #5558. Review and approval can happen now, but I will wait for #5558 to merge, then rebase this branch onto the latest master and retarget the PR to master before merging it.

@voidmatcha
voidmatcha force-pushed the branch-voidmatcha-assistant-prerequisites-temp branch from 4235999 to 95bd1b7 Compare October 10, 2026 13:40
@voidmatcha
voidmatcha force-pushed the voidmatcha/assistant-v2-3-contracts branch from 89a0192 to 80efed9 Compare October 10, 2026 13:40
@voidmatcha voidmatcha changed the title [ZEPPELIN-6741] Define shared Assistant UI contracts and message history behavior [ZEPPELIN-6741] Define shared AI Assistant UI contracts and message history behavior Oct 10, 2026
Give the sidebar shell and transport one SDK type to consume without duplicating its declaration in their separate PRs.

Confidence: high
Scope-risk: requires dependent PR branches to synchronize
Not-tested: standalone contracts build in this temporary worktree
@voidmatcha
voidmatcha force-pushed the branch-voidmatcha-assistant-prerequisites-temp branch from 95bd1b7 to 397a5df Compare October 11, 2026 11:30
@voidmatcha
voidmatcha force-pushed the voidmatcha/assistant-v2-3-contracts branch from 699239b to 3e95adf Compare October 11, 2026 11:30

This branch has not been deployed

No deployments
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.

1 participant