From a1d2fbd96bb6713c3172198d71e94274af409aee Mon Sep 17 00:00:00 2001
From: naoNao89 <90588855+naoNao89@users.noreply.github.com>
Date: Thu, 31 Jul 2025 05:29:42 +0700
Subject: [Editor] Prevent excessive project creation from invalid routes

Race conditions in useEffect hook caused navigation to invalid project IDs
to create multiple 'Untitled Project' entries. Users reported accumulating
885+ projects due to this bug.

This change adds global invalid project ID tracking across component remounts
and prevents race conditions through:
- Global invalidProjectIds Set in project store with tracking functions
- isInitializingRef to prevent duplicate initialization attempts
- Proper useEffect cleanup with cancellation tokens
- Specific error handling for 'not found' vs recoverable errors
- Removed problematic activeProject?.id dependency

Testing:
- Add comprehensive test suite (15 tests) with Vitest + React Testing Library
- Include integration tests demonstrating bug and validating fix
- Update CI pipeline to run automated tests

Fixes #182
---
 .github/workflows/bun-ci.yml | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

(limited to '.github/workflows/bun-ci.yml')

diff --git a/.github/workflows/bun-ci.yml b/.github/workflows/bun-ci.yml
index f01b9878..1f33096a 100644
--- a/.github/workflows/bun-ci.yml
+++ b/.github/workflows/bun-ci.yml
@@ -53,5 +53,8 @@ jobs:
 
       - name: Run tests
         working-directory: apps/web
-        run: echo "No tests implemented yet"
-        continue-on-error: true
+        run: |
+          echo "Running test suite..."
+          bun run test:run
+          echo "Running excessive projects bug tests..."
+          bun run test:excessive-projects
-- 
cgit v1.3.1