Repository navigation
Preserve animated zoom across semantic LOD transitions - #207
Conversation
🧪 Comprehensive Test Suite
Full-stack smoke gate runs in the CI workflow. |
|
Follow-up independent review found a missed selector mismatch in A fixed-camera Overview→Items control now observes the real visible title getBBox measurement, retains geometry checks, and requires zero camera publication with the clock paused. The incorrect selector fails with zero visible measurements; a geometry-only draft had passed and is explicitly not counted as a negative control. The selector correction and strengthened desktop/phone acceptance are running locally. The updated exact head will receive renewed review and CI before integration. |
🧪 Comprehensive Test Suite
Full-stack smoke gate runs in the CI workflow. |
|
Final preflight for native dev-only integration: source Final CI and Comprehensive suite succeeded. Every applicable job/step passed, including Node22/24, installer, installed dependencies, build/types/lint, security, real Neo4j and showcase. The main/nightly-only production job is correctly inapplicable to this dev PR. Actual downloaded strict evidence: smoke6, new-user42, semanticLOD2, routes7, mobile27, performance3; every case passed, no warnings/skips/evidence errors/timeouts or nonzero executions. The living-effects browser step also passed. Prior local negative controls and the late selector correction are retained and independently reviewed at this exact head. This uses the owner's standing dev administrator integration authorization; independent agent review is not a GitHub non-author approval. The current dev post-merge failure40597120363 is the known pre-correction LOD interruption, so #203/#205 closeout remains pending corrective post-merge CI. No main, release, deployment, identity or protection changes are included. |
|
Native dev admin integration completed: Post-merge CI40604945011 and Comprehensive40604945017 are queued/running. #203/#205 remain open until corrective dev validation completes. Prior failure40597120363 remains retained. No claim of main promotion, publication or deployment. |
Animated zoom could stop just above the overview exit threshold instead of reaching its target. Title remeasurement called D3's camera transform and interrupted the active animation. It now refreshes source-title geometry at the current scale without issuing camera commands.
Fixes #205. The correction is based on fixture-maintenance PR #204, now merged to dev. No shared CodeGraph library or dependency change.
Validation at
b12d08357b70f5f9b83c9eb0b84d6d411ccdab3c:.main-group. Its earlier geometry-only pass and review clearance were withdrawn. The final patch targets the actual.main-graph-group.getBBoxon the real title after visibility changes. It requires a visible measurement, bounded geometry, exactly unchanged camera and zero camera publications. Wrong-selector negative fails at zero visible measurements; corrected desktop/phone LOD journeys pass 2/2 in 4.2 minutes, one worker and zero retries. Earlier failures and traces are retained.Current dev post-merge CI 40597120363 failed this same pre-correction LOD transition. Corrective integration and successful post-merge CI are required before #203/#205 closeout. Independent agent review is not a GitHub non-author approval.
This fixes transition ownership and title refresh; multilevel grouping, bounded 1k rendering, progressive 10k loading and hosted acceptance remain unfinished. No deployment is claimed.