perf: revisit angular adapter to improve memory usage and flexrender … - #6528
perf: revisit angular adapter to improve memory usage and flexrender …#6528KevinVandy wants to merge 2 commits into
Conversation
|
View your CI Pipeline Execution ↗ for commit 640df9e
☁️ Nx Cloud last updated this comment at |
🚀 Changeset Version PreviewNo changeset entries found. Merging this PR will not cause a version bump for any packages. |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (13)
📝 WalkthroughWalkthroughFlex-render now caches component metadata, supports keyed component reuse, synchronizes inputs and outputs explicitly, and uses one reactive rendering effect. Tests and benchmarks cover rendering updates, component replacement, and large-table scenarios. Table option and signal evaluation were also adjusted. ChangesFlex-render lifecycle and performance
Estimated code review effort: 4 (Complex) | ~45 minutes Sequence Diagram(s)sequenceDiagram
participant FlexViewRenderer
participant FlexRenderView
participant FlexRenderComponentRef
participant ComponentInstance
FlexViewRenderer->>FlexViewRenderer: resolve content and props
FlexViewRenderer->>FlexRenderView: create or update view
FlexRenderView->>FlexRenderComponentRef: compare component type and key
FlexRenderComponentRef->>ComponentInstance: synchronize inputs and outputs
ComponentInstance-->>FlexViewRenderer: render updated component
Suggested reviewers: ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
…dirty checking
🎯 Changes
✅ Checklist
pnpm run test:pr.🚀 Release Impact
Summary by CodeRabbit
New Features
Bug Fixes