Repository navigation
[ZEPPELIN-6743] Mount an opt-in AI Assistant shell in the notebook sidebar - #5565
Open
voidmatcha wants to merge 5 commits into
Conversation
Member
Author
|
This PR adds the opt-in Assistant sidebar shell and its Angular/React mount point. Conversation UI and server transport are outside this diff. The current diff is +924/-7 across 21 files. Excluding 299 lines in The SDK build, React typecheck, focused React and Angular tests, lint, and React production build pass locally. Webpack reports an asset-size warning. This depends on #5558 and #5561. The temporary base contains those prerequisites so reviews can proceed now. After they merge, I will rebase this branch onto the latest |
voidmatcha
force-pushed
the
voidmatcha/assistant-sidebar-shell
branch
2 times, most recently
from
October 11, 2026 04:28
aee705e to
654d828
Compare
The opt-in Assistant host handles panel visibility and error states. Include it in the page coverage registry so the reporter tracks its currently untested browser flow. Confidence: high Scope-risk: none Not-tested: Full browser E2E and GitHub CI
Cover the flagged notebook shell through its visible sidebar controls. Create and remove a disposable notebook for each test run. Confidence: high Scope-risk: none Not-tested: full E2E suite and runtime fault injection
voidmatcha
force-pushed
the
branch-voidmatcha-assistant-contracts-temp
branch
from
October 11, 2026 11:30
699239b to
3e95adf
Compare
voidmatcha
force-pushed
the
voidmatcha/assistant-sidebar-shell
branch
from
October 11, 2026 11:31
654d828 to
d0763c9
Compare
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What is this PR for?
Add an opt-in AI Assistant button and empty panel shell to the notebook sidebar. With
?reactAssistant=true, the Angular notebook mounts the React shell, coordinates it with the file tree and table of contents, and shares the sidebar width for resizing. The panel closes when the active note changes and shows a fallback if the React remote fails to load.This establishes the notebook mount point for later conversation UI work. Conversation components, server transport, and message handling are outside this PR.
This PR depends on #5558 and #5561. Its temporary base contains those changes so the shell can be reviewed separately.
What type of PR is it?
Feature
Todos
What is the Jira issue?
ZEPPELIN-6743
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/pages/AssistantWorkspace.spec.tsx npm run test:shell -- src/app/pages/workspace/notebook/assistant src/app/pages/workspace/notebook/sidebar/sidebar.component.spec.ts src/app/services/react-feature.service.spec.ts npm run lint npm run build:reactIn an authenticated notebook, open
?reactAssistant=trueand check opening, closing, resizing, switching sidebar sections, and switching notes. Confirm that the button is absent without the flag and in view-only and revision views.Screenshots (if appropriate)
N/A
Questions: