Skip to content

fix(studio): restore corrupted import bindings and paths - #77

Draft
zedarvates wants to merge 2 commits into
fix/studio-renderer-contractsfrom
fix/studio-import-bindings
Draft

zedarvates wants to merge 2 commits into
fix/studio-renderer-contractsfrom
fix/studio-import-bindings

Conversation

@zedarvates

@zedarvates zedarvates commented Oct 3, 2026 •

Copy link
Copy Markdown
Owner

Creative Studio imported nonexistent names such as _Pause, _Character and useSta_te, and several module paths/export names were corrupted. Restore the actual exported names and existing module paths without renaming unrelated local bindings or inventing missing services. Preserve the ImageIcon alias and PropertiesPanel's original encoding. Package versions, lockfiles and full compiler settings/inclusions are unchanged.

Partial follow-up to #74, draft stacked on #76 at 09bbd0503eb82de4c84f5a7a60801d0d390b40ee (itself stacked on #75).

The renderer workflow audits named imports/re-exports from React, React DOM, Lucide, Material UI, Framer Motion and Three against installed declarations, including unused imports that a bundle can remove. A separate Full application types (unfinished) job runs the complete existing TypeScript build, propagates its exit status through tee using explicit Bash/pipefail, and uploads the diagnostics.

Current-head evidence — b0018678cc7ca67e229264d0b6bb6dc1eb6e3a3a:

  • Renderer Node 22.12.0 and Node 24.19.0 succeed: isolated installation, the shared-library name audit (5,713 bindings in 966 files), two strict bridge contract programs, ten focused tests and production bundles.
  • Node 24 also passes the bounded fresh Chromium launcher check with providers/backends blocked. Browser steps on Node 22 are skipped by matrix design.
  • Full application compiler fails, exit 1, with 1,317 diagnostics in isolated CI. The run therefore concludes failure. Its studio-full-types-diagnostics artifact is retained.
  • Shared controls CI succeeds on the same head.
  • No SonarCloud check/report or commit-status context is exposed through GitHub on this head. fix(studio): restore shared editor controls and root provider wiring #75's older passed gate is separate evidence; no security-alert closure is inferred.

Local comparison: full tsc -b still fails with 1,254 diagnostics, down from 1,623 on #76 in the same repository setup, with unchanged compiler settings/source inclusions. The isolated CI count is a separate measurement; it must not be described as a measured CI regression or compared as if the installations were identical. A temporary negative probe rejects a nonexistent unused export while valid aliases/re-exports pass.

Review/blockers: review the import/path repairs and the shared-library audit's explicit scope. Other missing modules/exports, corrupted local bindings, unvalidated domain payloads and optional Electron capabilities remain unresolved. These checks do not exercise project creation, editing, playback, generation or the primary Electron launcher; the #70/#63/#73 dependency combination has not been retested with this batch. Keep #74 open and this PR in draft. No merge approval or commercial availability is claimed. Scope and commands.

@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