chore(network-graph): preserve duplicate generated optimization lineage - #1580
chore(network-graph): preserve duplicate generated optimization lineage#1580seonghobae wants to merge 6 commits into
Conversation
NetworkGraph.tsx 내부의 useMemo Hook들에서 `Array.from(map.values()).slice(0, 5)`나 `nodes.map(...).filter(...).slice(0, 5)` 등의 함수 체이닝이 거대한 중간 객체 배열을 O(N)으로 전체 생성한 후 아주 작은 N(5~8) 크기만큼만 취하고 나머지는 GC에 버리는 성능 비효율이 있었습니다. 이를 5개, 8개 등의 뷰 최대 표현 요구치에 도달하면 즉시 break로 반복을 탈출하는 Imperative `for...of` 구조로 교체해, 최악의 경우 O(N)이던 Map→Array 캐스팅과 배열 순회 비용을 O(K) 수준으로 최적화했습니다.
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueThanks 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 |
NetworkGraph.tsx 내부의 useMemo Hook들에서 `Array.from(map.values()).slice(0, 5)`나 `nodes.map(...).filter(...).slice(0, 5)` 등의 함수 체이닝이 거대한 중간 객체 배열을 O(N)으로 전체 생성한 후 아주 작은 N(5~8) 크기만큼만 취하고 나머지는 GC에 버리는 성능 비효율이 있었습니다. 이를 5개, 8개 등의 뷰 최대 표현 요구치에 도달하면 즉시 break로 반복을 탈출하는 Imperative `for...of` 구조로 교체해, 최악의 경우 O(N)이던 Map→Array 캐스팅과 배열 순회 비용을 O(K) 수준으로 최적화했습니다.
seonghobae
changed the title
NetworkGraph.tsx 내부의 useMemo Hook들에서 `Array.from(map.values()).slice(0, 5)`나 `nodes.map(...).filter(...).slice(0, 5)` 등의 함수 체이닝이 거대한 중간 객체 배열을 O(N)으로 전체 생성한 후 아주 작은 N(5~8) 크기만큼만 취하고 나머지는 GC에 버리는 성능 비효율이 있었습니다. 이를 5개, 8개 등의 뷰 최대 표현 요구치에 도달하면 즉시 break로 반복을 탈출하는 Imperative `for...of` 구조로 교체해, 최악의 경우 O(N)이던 Map→Array 캐스팅과 배열 순회 비용을 O(K) 수준으로 최적화했습니다.
seonghobae
added
priority: medium
ChatGPT Codex Connector
NetworkGraph.tsx 내부의 useMemo Hook들에서 `Array.from(map.values()).slice(0, 5)`나 `nodes.map(...).filter(...).slice(0, 5)` 등의 함수 체이닝이 거대한 중간 객체 배열을 O(N)으로 전체 생성한 후 아주 작은 N(5~8) 크기만큼만 취하고 나머지는 GC에 버리는 성능 비효율이 있었습니다. 이를 5개, 8개 등의 뷰 최대 표현 요구치에 도달하면 즉시 break로 반복을 탈출하는 Imperative `for...of` 구조로 교체해, 최악의 경우 O(N)이던 Map→Array 캐스팅과 배열 순회 비용을 O(K) 수준으로 최적화했습니다.
NetworkGraph.tsx 내부의 useMemo Hook들에서 `Array.from(map.values()).slice(0, 5)`나 `nodes.map(...).filter(...).slice(0, 5)` 등의 함수 체이닝이 거대한 중간 객체 배열을 O(N)으로 전체 생성한 후 아주 작은 N(5~8) 크기만큼만 취하고 나머지는 GC에 버리는 성능 비효율이 있었습니다. 이를 5개, 8개 등의 뷰 최대 표현 요구치에 도달하면 즉시 break로 반복을 탈출하는 Imperative `for...of` 구조로 교체해, 최악의 경우 O(N)이던 Map→Array 캐스팅과 배열 순회 비용을 O(K) 수준으로 최적화했습니다.
1 participant
Lifecycle repair
This generated direct-
developPR duplicates an already-owned NetworkGraph performance contract. It is preserved as provenance but is no longer an active source writer.develop@042b0c70531b229af3acbd0421a2f23098d848b333d3af25bea8dfb07a8a22a31023cf5e2953f853b8637567b97a8548619f5ded703b474621ca79a7developtree8fde14381aaa430eeaaf61151dab6f6800127cd3; GitHub reports an empty PR diff.Canonical owner path
Draft #1522
perf(network-graph): bound relationship and node option materializationalready owns the useful optimization on exact headfd938c1c6ae1328fafba1959fd55c3b9037e4346. Its product source uses the same bounded early-exit iteration for relationship options, node options, and the first five non-empty node labels, and it carries a dedicated executableNetworkGraph.option-limits.test.tsxregression proving the visible limits and ordering semantics. #1560 separately owns the memoized parent-rerender boundary.The generated #1580 changed only
NetworkGraph.tsx,CHANGELOG.md, and a Jules learning note. It duplicated #1522's source intent without the dedicated option-limit regression and described rendering/memory improvement without current browser profiling or buyer-path p95 evidence. Those claims are not accepted as delivery evidence and do not justify a parallel direct-developwriter.No generated source/CHANGELOG/guidance delta is current product authority. The historical commit remains in ancestry; no force-push, destructive rebase, self-approval, dummy/no-op requeue, or gate weakening was used.
Succession boundary
Do not close this PR merely to reduce the queue. Keep it open Draft until #1522 is protected-integrated and a fresh exact-tree/equivalence audit proves every valid source, test, fixture, contract, and evidence contribution from the generated commit is either completely inherited by #1522/#1560 or deliberately rejected with recorded rationale. Only verified complete succession may justify closure.