Restructure CI reviews: 4 composite actions with requirements gate (#11)

* Session work\n\nAuto-commit at session end to preserve work in progress.\n\nSession-ID: 667d5a5c-c36b-4543-bfec-65936b7a6417\nSession-Timestamp: 2026-01-03T18:33:59.266Z\nBranch: claude-upbeat-bandicoot-ul46xpp5\n\n🤖 Generated with [Claude Code](https://claude.com/claude-code)\n\nCo-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

* Restructure CI to exactly 3 checks: Static Analysis, Tests, Reviews

- Delete old workflows (basic-ci.yml, ci.yml, e2e-tests.yml, deployment-ci.yml)
- Delete old fragmented composite actions
- Create 3 clean composite actions:
  - static-analysis: Lint (ESLint) + Types (TypeScript)
  - tests: Unit (Vitest) + E2E (Playwright, only if has_app)
  - reviews: Requirements (gate), Code Quality, Context, UI
- Rewrite ci-pipeline.yml with only 3 jobs
- Context Review now only checks CLAUDE.md files that are parents of changed files
- Reviews post single consolidated comment

Dependency chain:
Static Analysis → Tests → Reviews

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Restructure CI with proper check naming

Checks now appear as:
- CI / Static Analysis / Lint (ESLint)
- CI / Static Analysis / Types (TypeScript)
- CI / Tests / Unit (Vitest)
- CI / Tests / E2E (Playwright) [disabled for non-app repos]
- CI / Reviews / Requirements
- CI / Reviews / Code Quality
- CI / Reviews / Context
- CI / Reviews / UI [disabled for non-app repos]
- CI / Reviews / Summary

Dependencies:
- Unit tests require Static Analysis to pass
- E2E requires Unit to pass
- Reviews require Unit to pass
- Code Quality & Context require Requirements to pass

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Add dynamic app detection and update review dependencies

- Add app detection step to lint job (checks for next.config, vite.config, playwright.config, apps/, src/app/)
- E2E tests now only run when app is detected
- Requirements review now depends on both unit and e2e tests
- UI review now only runs when app is detected

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* PR-only CI with auto package manager detection

- Remove push trigger to avoid duplicate checks
- Auto-detect package manager (bun/npm/pnpm/yarn) from lockfile
- Each review job incrementally builds a single PR comment
- Remove separate summary job
- Add minimal bun setup with typescript, eslint, vitest

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Fix bun lockfile detection for text format

Bun now generates bun.lock (text) instead of bun.lockb (binary).
Check for both formats.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Include root ts files in tsconfig

Fix ESLint error: vitest.config.ts was not in project

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

* Pass unit tests when no test files exist

Add --passWithNoTests flag to vitest

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>

Author: sylvansys

.github/actions/consolidate-comment/action.yml

@@ -1,38 +1,26 @@
 # CI Configuration
-# Controls which CI workflows run in this repository
+# Template for Claude Code repos
 
 ci:
-  # Basic CI (always enabled)
-  basic:
-    enabled: true
+  # Set to true if this repo has an app (enables E2E + UI review)
+  has_app: false
+
+  # Static Analysis: Lint (ESLint) + Types (TypeScript)
+  static_analysis:
     lint: true
     typecheck: true
-    vitest: true
 
-  # E2E Testing
-  e2e:
-    enabled: true
-    framework: playwright
+  # Tests: Unit (Vitest) + E2E (Playwright)
+  # E2E only runs if has_app: true
+  tests:
+    unit: true
+    e2e: true
 
-  # Code Reviews (runs on every commit)
+  # Reviews: Requirements, Code Quality, Context, UI
+  # Requirements must pass before others run
+  # UI only runs if has_app: true and screenshots exist
   reviews:
-    enabled: true
     requirements: true
-    rules: true
-    project_memory: true
-    agents: true
-    skills: true
-    playwright_ui: true
-
-  # Deployment CI (PR only)
-  deployment:
-    enabled: false
-
-    vercel:
-      enabled: false
-
-    supabase:
-      enabled: false
-
-    cloudflare:
-      enabled: false
+    code_quality: true
+    context: true
+    ui: true