chore(tests): remove orphaned v1 tests for removed /api/projects endpoint

[frankbria]

What

Deletes two orphaned test files targeting the v1 POST /api/projects endpoint, removed in 66241c2 (refactor: remove orphaned v1 web layer):

• tests/api/test_project_creation_api.py (task cf-11)
• tests/api/test_workspace_cleanup.py (Issue Improve workspace cleanup in project creation rollback #7)

Why

The route no longer exists, so these fail with 404 on a full local run. They carry no v2 marker, so CI's -m v2 suite never ran them — silently dead since the v1 web layer was removed. v2-equivalent coverage lives under tests/ui/.

tests/cli/test_api_client.py is intentionally kept (mocked CLI client-helper tests; still passes).

Scope note

Narrow cleanup. The broader CI blind spot — -m v2 deselects ~535 tests (~40% of files lack the marker) — is tracked in a separate issue.

Deletions only; no production code changed.

[coderabbitai]

Warning

Review limit reached

@frankbria, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 1 hour, 42 minutes, and 34 seconds. Learn how PR review limits work.

Your organization has used up its prepaid credits, and credit purchases are no longer available. Enable the review add-on in the billing tab to keep reviews running — you're only billed for reviews past your plan's rate limits ($0.25/file).

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f405766d-82ab-4817-bb05-19e098b16a51

📥 Commits

Reviewing files that changed from the base of the PR and between d65b313 and 594bceb.

📒 Files selected for processing (2)

• tests/api/test_project_creation_api.py
• tests/api/test_workspace_cleanup.py

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch chore/remove-dead-v1-tests

---

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

[claude]

Code Review

Verdict: Approve — clean, well-justified deletion.

---

What this PR does

Removes 496 lines across two dead test files that targeted the v1 POST /api/projects endpoint removed in 66241c2. Both files carry only @pytest.mark.unit / @pytest.mark.integration markers — never @pytest.mark.v2 — so the CI -m v2 suite skipped them, leaving them silently broken since the v1 web layer was removed.

---

Verification

• tests/api/ is not emptied: test_session_chat_ws.py, test_health_endpoint.py, and conftest.py remain — no orphaned directory.
• The deleted tests are truly dead: both files patch v1-specific internals (app.state.workspace_manager, app.state.db) that no longer exist in v2. Even with the fixture, they'd 404 or crash on import.
• No v2 coverage gap: tests/ui/ has no counterpart for the workspace-cleanup failure scenarios in test_workspace_cleanup.py, but v2 replaced that architecture entirely (WorkspaceManager → create_or_load_workspace), so those scenarios are architecture-specific and don't represent a gap.

---

One minor note

The PR description mentions the broader CI blind spot (~535 tests without a v2 marker, ~40% of files) as a tracked follow-on. Worth confirming that issue exists and is linked here for traceability — but that's out of scope for this PR and not a blocker.

---

No production code is touched. Deletions only. LGTM.