-
Notifications
You must be signed in to change notification settings - Fork 411
Minor: transformState and setting error cleanup #6841
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (1)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including You can disable this status message by setting the 📝 WalkthroughWalkthroughThree files updated to refactor state management and add runtime checks. The settings store adds a guard for undefined settings before dispatching changes. The resize hook now directly calls Changes
Sequence DiagramsequenceDiagram
participant Component as LGraphNode
participant Hook as useNodeResize
participant Transform as useTransformState
rect rgb(200, 220, 255)
Note over Component,Transform: Previous Flow
Component->>Hook: useNodeResize(callback, {transformState})
end
rect rgb(220, 255, 220)
Note over Component,Transform: New Flow
Component->>Hook: useNodeResize(callback)
Hook->>Transform: useTransformState()
Transform-->>Hook: transformState
end
Hook->>Hook: Handle resize with transformState
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
🎨 Storybook Build Status✅ Build completed successfully! ⏰ Completed at: 11/23/2025, 08:19:56 AM UTC 🔗 Links🎉 Your Storybook is ready for review! |
🎭 Playwright Test Results❌ Some tests failed ⏰ Completed at: 11/23/2025, 08:29:52 AM UTC 📈 Summary
📊 Test Reports by Browser
🎉 Click on the links above to view detailed test results for each browser configuration. |
Bundle Size ReportSummary
Category Glance Per-category breakdownApp Entry Points — 3.18 MB (baseline 3.18 MB) • 🔴 +23 BMain entry bundles and manifests
Status: 3 added / 3 removed Graph Workspace — 940 kB (baseline 940 kB) • 🟢 -260 BGraph editor runtime, canvas, workflow orchestration
Status: 1 added / 1 removed Views & Navigation — 7.97 kB (baseline 7.97 kB) • ⚪ 0 BTop-level views, pages, and routed surfaces
Status: 1 added / 1 removed Panels & Settings — 306 kB (baseline 306 kB) • ⚪ 0 BConfiguration panels, inspectors, and settings screens
Status: 6 added / 6 removed UI Components — 141 kB (baseline 141 kB) • ⚪ 0 BReusable component library chunks
Status: 6 added / 6 removed Data & Services — 12.5 kB (baseline 12.5 kB) • ⚪ 0 BStores, services, APIs, and repositories
Status: 2 added / 2 removed Utilities & Hooks — 2.94 kB (baseline 2.94 kB) • ⚪ 0 BHelpers, composables, and utility bundles
Status: 1 added / 1 removed Vendor & Third-Party — 5.7 MB (baseline 5.7 MB) • ⚪ 0 BExternal libraries and shared vendor chunks
Other — 3.87 MB (baseline 3.87 MB) • ⚪ 0 BBundles that do not match a named category
Status: 18 added / 18 removed |
|
Updating Playwright Expectations |
|
@DrJKL Backport to Please manually cherry-pick commit Conflicting files
|
|
@DrJKL Backport to Please manually cherry-pick commit Conflicting files
|
Summary
Fixes the routing of TransformState through the node and the console error from a setting that ends up being undefined via useRenderModeSetting.
┆Issue is synchronized with this Notion page by Unito