Skip to content

refactor(cleanupFolders): optimize item normalization logic and impro… - #62

Merged
clFaster merged 1 commit into
mainfrom
22-replace-expensive-jsonstringify-comparison-in-normalization-effect
Sep 9, 2026
Merged

refactor(cleanupFolders): optimize item normalization logic and impro…#62
clFaster merged 1 commit into
mainfrom
22-replace-expensive-jsonstringify-comparison-in-normalization-effect

Conversation

@clFaster

@clFaster clFaster commented Sep 9, 2026

Copy link
Copy Markdown
Owner

Implemented GitHub issue #22.

  • Replaced full-tree JSON.stringify comparisons with referential equality in BoardList.tsx (line 718).
  • Updated boardListDnd.ts (line 128) to preserve references when no cleanup occurs.
  • Retained duplicate removal and empty/single-item folder normalization.
    Verified with:
  • Production build
  • ESLint
  • Prettier
  • Direct normalization assertions covering unchanged, empty, singleton, and duplicate cases

@clFaster
clFaster enabled auto-merge September 9, 2026 19:27
@clFaster
clFaster disabled auto-merge September 9, 2026 19:34
@clFaster
clFaster merged commit bd8c492 into main Sep 9, 2026
11 checks passed
@clFaster
clFaster deleted the 22-replace-expensive-jsonstringify-comparison-in-normalization-effect branch September 9, 2026 19:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@codescene-delta-analysis codescene-delta-analysis[bot] codescene-delta-analysis[bot] approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@clFaster