Skip to content

fix(studio): restore shared editor controls and root provider wiring - #75

Draft
zedarvates wants to merge 1 commit into
mainfrom
fix/studio-contract-recovery
Draft

zedarvates wants to merge 1 commit into
mainfrom
fix/studio-contract-recovery

Conversation

@zedarvates

@zedarvates zedarvates commented Oct 3, 2026 •

Copy link
Copy Markdown
Owner

The sequence editor imports missing Zustand stores/editor/* modules although its panel/tool/chat reducers already exist in Redux. Wire the editor, toolbar and compact assistant to those existing slices and register the previously omitted chat reducer so consumers share panel visibility, tool selection and messages.

Remove the root ComfyUIProvider import/wrapper, which has no versioned implementation or context consumer; current ComfyUI hooks continue to access the existing servers service directly.

Validation: six local Node 24.19.0 integration tests pass for chat registration, independent consumer synchronization, production layout transitions, timeline tool state, shared chat visibility/messages and stable action callbacks. git diff --check passes. The focused CI run succeeded on current head 02990d8e0211a5a848bedd499d25f07788c4f3fa, including dependency installation and these six tests on Node 22.12.0 and 24.19.0. SonarCloud Code Analysis is successful on the same SHA; this does not establish full build/runtime behavior or closure of all security alerts.

Partial repair for #74. The full Vite bundle now advances beyond these missing entry modules but still fails on addons/demo-addon in AddonManager. Full strict typechecking and renderer startup are not established. The compact assistant's existing response is simulated; no live provider, workflow execution or video/comics/manga generation is claimed. See scope and remaining acceptance. This PR remains draft and closes neither #74 nor the dependency reviews.

@sonarqubecloud

sonarqubecloud Bot commented Oct 3, 2026

Copy link
Copy Markdown

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