fix: prevent duplicate task IDs in Kanban columns

[ArjinAlbay]

Root Cause:

• Tasks were being added to column.taskIds arrays without checking for duplicates
• Multiple sync operations or edge cases could result in the same task ID appearing multiple times
• React requires unique keys, causing "Encountered two children with the same key" errors

Changes:

1. syncFromGitHub: Check if task.id already exists before adding to column
2. addTask: Check for duplicates when creating new tasks
3. addTaskFromActionItem: Check for duplicates when adding from action items
4. restoreTask: Check for duplicates when restoring from archive
5. bulkMove: Filter out duplicates when moving multiple tasks
6. onRehydrateStorage: Automatic deduplication on localStorage hydration

Impact:

• Eliminates React duplicate key warnings
• Prevents UI rendering issues
• Maintains data integrity across sync operations
• Automatic cleanup of existing duplicates on app load

Summary by CodeRabbit

Release Notes

• New Features

  • Added a floating Quick Wins notification widget to the dashboard displaying available opportunities with a badge count.
  • Toast notifications alert users when new Good First Issues or Easy Fixes become available.
  • Quick Wins panel displays categorized opportunities with dedicated sections and action counts.

• Bug Fixes

  • Resolved duplicate task IDs appearing in kanban columns during add or move operations.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
githubmon	Ready	Preview	Comment	Nov 23, 2025 3:24pm

[coderabbitai]

Caution

Review failed

The pull request is closed.

Walkthrough

This PR introduces a new QuickWinsNotifier component to the dashboard, replacing QuickWinsCounters, and adds deduplication logic to the kanban store to prevent duplicate task IDs when adding or moving tasks into columns.

Changes

Cohort / File(s)	Summary
Dashboard UI Component Replacement src/app/dashboard/page.tsx	Replaced QuickWinsCounters import and component with QuickWinsNotifier; layout and TodoDashboard rendering remain unchanged.
New Quick Wins Notifier Widget src/components/widget/QuickWinsNotifier.tsx	New client-side React component that fetches good issues and easy fixes on mount, displays a floating button with count badge, shows a toast notification when new items arrive, expands an interactive panel with per-section loading/error states, and includes a link to the full quick-wins page.
Kanban Store Deduplication Logic src/stores/kanban.ts	Added checks to prevent duplicate task IDs when adding or moving tasks across columns (GitHub sync, manual add, action items, restoration, bulk operations). Includes deduplication on store rehydration with warning logs.

Sequence Diagram

sequenceDiagram
    participant User
    participant Dashboard as Dashboard Page
    participant Notifier as QuickWinsNotifier
    participant Store as Quick Wins Store
    
    Dashboard->>Notifier: Mount component
    Notifier->>Store: fetchGoodIssues()
    Notifier->>Store: fetchEasyFixes()
    Store-->>Notifier: Loading state
    Note over Notifier: Fetch in progress
    Store-->>Notifier: Data loaded
    
    alt New items detected
        Notifier->>User: Show toast notification
        Note over Notifier: totalCount increased
        rect rgb(200, 220, 255)
            Notifier->>Notifier: Wait 500ms
            Notifier->>Notifier: Expand panel
            Notifier->>Notifier: Start pulse animation (3s)
        end
    end
    
    User->>Notifier: Click to expand/dismiss
    alt Panel expanded
        Notifier->>Notifier: Display Good First Issues section
        Notifier->>Notifier: Display Easy Fixes section
        Note over Notifier: Show per-section loading/error states
    end
    
    User->>Notifier: Click "View All Quick Wins"
    Notifier->>Dashboard: Navigate to /quick-wins
    
    User->>Notifier: Click overlay
    Notifier->>Notifier: Close panel

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

• QuickWinsNotifier complexity: New component with multiple state variables (expansion, pulse, notification dismissal), store integration, async fetch logic, and conditional rendering based on loading/error states requires careful verification
• Kanban deduplication patterns: Multiple similar deduplication checks applied across different operations (GitHub sync, add, move, restoration) require consistent validation logic across all code paths
• Integration points: Verify that component properly handles edge cases (loading states, error states, no items scenario) and that store deduplication doesn't affect existing kanban functionality

Possibly related PRs

• PR #2: Directly replaces QuickWinsCounters with QuickWinsNotifier on the dashboard, making it a predecessor change to this component introduction.
• PR #5: Also modifies kanban store task operations and deduplication patterns, sharing overlapping logic for handling task ID management.
• PR #4: Adds tracking and APIs for added items in kanban store, related to the same task add/move logic being enhanced with deduplication.

Poem

🐰 A notifier springs up with glee,
Quick wins to display for all to see!
No duplicate tasks shall clutter the way,
Fresh notifications brighten the day!
The dashboard hops forward, organized and bright! 🌟

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch claude/analyze-kanban-board-01SpdbuZU5vqEWhQ78MZTsSa

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 355d4a1 and 12b26ef.

📒 Files selected for processing (3)

• src/app/dashboard/page.tsx (2 hunks)
• src/components/widget/QuickWinsNotifier.tsx (1 hunks)
• src/stores/kanban.ts (7 hunks)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}