Skip to content

fix(backlog): post-merge review follow-ups (snapshot-merge union, canvas concurrency, 4 nits)#30

Merged
New1Direction merged 1 commit into
mainfrom
fix/post-merge-backlog
Jun 16, 2026
Merged

fix(backlog): post-merge review follow-ups (snapshot-merge union, canvas concurrency, 4 nits)#30
New1Direction merged 1 commit into
mainfrom
fix/post-merge-backlog

Conversation

@New1Direction

Copy link
Copy Markdown
Owner

Summary

Clears the deferred findings from the stack pre-merge review. All on top of the merged stack; reviewed (APPROVE, 0 blocking) and unit-tested.

  • importStores merge now unions snapshot history by timestamp (dedupe, re-clamp to cap) instead of overwriting — no data loss on backup restore. replace mode unchanged.
  • renderCanvas mount-guard claimed synchronously + reset on failure — fixes a double-mount race between tab-switch and storage events.
  • Canvas tab now refreshes from the CTA to a real Blueprint when Deep Dive finishes after the tab is open.
  • 4 LOW nits: drop always-null snapshot version; no (+0) sparkline caption; count dropped invalid scenes into backup warnings; toCanvasSvg bounds via reduce (avoids RangeError on huge scenes).

Test plan

  • 832 tests / 84 files green (+9 new regression tests, RED→GREEN)
  • eslint 0 errors
  • Adversarial review: APPROVE

@New1Direction
New1Direction merged commit 85a569b into main Jun 16, 2026
1 check passed
@New1Direction
New1Direction deleted the fix/post-merge-backlog branch June 16, 2026 19:16
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