Skip to content

fix(dashboard): skip stale canvas deletes - #7334

Merged
houko merged 3 commits into
mainfrom
audit/dashboard-canvas-noop
Aug 15, 2026
Merged

houko merged 3 commits into
mainfrom
audit/dashboard-canvas-noop

Conversation

@houko

@houko houko commented Aug 14, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • preserve canvas node and edge array references when cascade deletion finds no matching node
  • skip undo history and state updates when a context-menu node or connection target has gone stale
  • cover stale node and connection deletion in the canvas helper tests

Verification

  • corepack pnpm exec vitest run src/lib/canvas.test.ts (1 file, 9 tests)
  • corepack pnpm test (101 files, 1061 tests)
  • corepack pnpm lint
  • corepack pnpm typecheck
  • corepack pnpm build

Out of scope

  • none

@github-actions github-actions Bot added no-rust-required This task does not require Rust knowledge size/S 10-49 lines changed labels Aug 14, 2026
@github-actions github-actions Bot added size/M 50-249 lines changed and removed size/S 10-49 lines changed labels Aug 15, 2026
@houko
houko enabled auto-merge (squash) August 15, 2026 02:21
@houko
houko merged commit 8de6c5c into main Aug 15, 2026
39 checks passed
@houko
houko deleted the audit/dashboard-canvas-noop branch August 15, 2026 02:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-rust-required This task does not require Rust knowledge size/M 50-249 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant