release: v0.8 — UX & Charts

.claude/agents/project-architect.md

@@ -3,11 +3,13 @@ name: project-architect
 description: Analyze feature requests and produce implementation plans with file impact analysis, dependency mapping, and risk assessment. Use before starting complex features.
 model: opus
 ---
+
 You are a software architect for the NeoBoard monorepo — an open-source dashboarding tool for hybrid database architectures (for now Neo4j + PostgreSQL, in the future many more).
 
 ## Context
 
 Read these files for project rules and architecture:
+
 - `CLAUDE.md` — Working rules, architecture boundaries, query safety, credentials
 - `claude_code_docs/` — Detailed docs on testing, widget architecture, performance
 
@@ -21,22 +23,39 @@ Next.js 15 (App Router), React 19, TypeScript, shadcn/ui, Tailwind CSS, ECharts,
 - `component/` — React UI library. NO business logic, NO API calls, NO stores.
 - `connection/` — DB connector library. NO UI, NO React.
 
+## Input
+
+You may receive:
+
+- An issue number to fetch
+- A `REQUIREMENTS BRIEF` from a `/grill` session — if provided, this is your primary source of truth for what the user wants. It contains answers to detailed clarifying questions about scope, UX, data model, security, edge cases, and testing.
+
 ## Steps
 
 1. If given an issue number, fetch it: `gh issue view <number>`
-2. Read `CLAUDE.md` and relevant docs in `claude_code_docs/`.
-3. Search the codebase to understand existing patterns related to the feature.
-4. Produce a structured implementation plan.
+2. If a `REQUIREMENTS BRIEF` is provided, read it carefully — it supersedes the issue body for specifics.
+3. Read `CLAUDE.md` and relevant docs in `claude_code_docs/`.
+4. Search the codebase thoroughly to understand existing patterns related to the feature:
+   - Find files that will need modification
+   - Identify interfaces and types to extend
+   - Find similar features already implemented to reuse patterns
+   - Check for potential conflicts with ongoing work
+5. Produce a structured implementation plan.
+6. Save the plan to `claude_code_docs/plans/issue-<number>.md`.
 
 ## Output Format
 
 ```
 # Implementation Plan: <Feature Name>
 
+## Requirements Summary
+<2-3 sentences summarizing what was agreed during the grilling session — scope, MVP, key decisions>
+
 ## Impact Analysis
 - Packages affected: [app, component, connection]
 - Files to modify: [path — what changes]
 - Files to create: [path — purpose]
+- Estimated size: S / M / L / XL
 
 ## Existing Patterns to Reuse
 - `path/to/file.ts:line` — Pattern description
@@ -62,13 +81,18 @@ Next.js 15 (App Router), React 19, TypeScript, shadcn/ui, Tailwind CSS, ECharts,
 1. **[Step name]** (S/M/L) — Description
    - Files: [paths]
    - Tests: [what to test]
+   - Acceptance: [how to verify this step is done]
 ...
 
 ## Testing Strategy
-- Unit tests: [what to cover]
+- Unit tests: [what to cover, which files]
 - Integration tests: [what to cover]
-- E2E tests: [what to cover]
+- E2E tests: [critical user flows to cover]
+- Edge cases from brief: [list specific edge cases identified during grilling]
 
 ## Risks
 - [Risk] — Mitigation
+
+## Open Questions
+- [Any remaining ambiguity not resolved during grilling]
 ```

.claude/settings.json

@@ -1,17 +1,28 @@
 {
   "permissions": {
     "allow": [
-      "Bash(npm *)", "Bash(npx *)", "Bash(gh *)", "Bash(git *)",
-      "Bash(node *)", "Bash(cat *)", "Bash(ls *)", "Bash(find *)",
-      "Bash(grep *)", "Bash(head *)", "Bash(tail *)", "Bash(wc *)",
-      "Bash(echo *)", "Bash(mkdir *)", "Bash(cp *)", "Bash(mv *)",
+      "Bash(npm *)",
+      "Bash(npx *)",
+      "Bash(gh *)",
+      "Bash(git *)",
+      "Bash(node *)",
+      "Bash(cat *)",
+      "Bash(ls *)",
+      "Bash(find *)",
+      "Bash(grep *)",
+      "Bash(head *)",
+      "Bash(tail *)",
+      "Bash(wc *)",
+      "Bash(echo *)",
+      "Bash(mkdir *)",
+      "Bash(cp *)",
+      "Bash(mv *)",
       "Bash(docker compose *)",
-      "Read(*)", "Edit(*)", "Write(*)"
+      "Read(*)",
+      "Edit(*)",
+      "Write(*)"
     ],
-    "deny": [
-      "Bash(rm -rf /)", "Bash(rm -rf ~)",
-      "Edit(.env*)", "Write(.env*)"
-    ]
+    "deny": ["Bash(rm -rf /)", "Bash(rm -rf ~)", "Edit(.env*)", "Write(.env*)"]
   },
   "hooks": {
     "PreToolUse": [
@@ -60,7 +71,7 @@
         "hooks": [
           {
             "type": "command",
-            "command": "printf 'CRITICAL RULES (re-injected after compaction):\\n- TDD mandatory: write failing test FIRST, then implement\\n- Package boundaries: component/ has NO business logic/API/stores; connection/ has NO React/UI\\n- Query safety: NEVER interpolate user input, ALWAYS parameterized queries\\n- Run cd app && npx next lint --fix after app/ changes\\n- Run npm run build before committing\\n- PRs target dev branch, not main\\n- Coverage target: 80%% per package'"
+            "command": "printf 'CRITICAL RULES (re-injected after compaction):\\n- TDD mandatory: write failing test FIRST, then implement\\n- Package boundaries: component/ has NO business logic/API/stores; connection/ has NO React/UI\\n- Query safety: NEVER interpolate user input, ALWAYS parameterized queries\\n- Run cd app && npx next lint --fix after app/ changes\\n- Run npm run build before committing\\n- PRs target dev branch, not main\\n- Coverage target: 80%% per package\\n- WORKTREE AGENTS: tests are safe to run locally (dynamic ports). CI is the source of truth.\\n- ORCHESTRATOR: max 3 concurrent workers. Never auto-merge. Track CONFLICT_FILES across workers.'"
           }
         ]
       }

.github/workflows/ci.yml

@@ -24,13 +24,12 @@ concurrency:
   cancel-in-progress: true
 
 jobs:
-  # ── Job 1: Fast quality gate ────────────────────────────────────────────────
-  # Lint + type-check all packages + component build. Fails fast before
-  # expensive test jobs start.
-  lint-typecheck:
-    name: Lint & Type Check
+  # ── Job 0: Type-check ───────────────────────────────────────────────────────
+  # Pre-hook gates are bypassable; enforce type safety in CI as well.
+  typecheck:
+    name: TypeScript type-check
     runs-on: ubuntu-latest
-    timeout-minutes: 5
+    timeout-minutes: 10
 
     steps:
       - name: Checkout
@@ -42,41 +41,31 @@ jobs:
           node-version: '20'
           cache: 'npm'
           cache-dependency-path: |
-            package-lock.json
             app/package-lock.json
             component/package-lock.json
             connection/package-lock.json
 
       - name: Install dependencies
         run: |
-          npm ci &
-          npm ci --prefix component &
           npm ci --prefix app &
+          npm ci --prefix component &
           npm ci --prefix connection &
           wait
 
-      - name: Lint (all packages)
-        run: npm run lint
-
-      - name: Type-check app
-        working-directory: app
-        run: npx tsc --noEmit
-
       - name: Type-check component
         working-directory: component
         run: npx tsc --noEmit
 
-      - name: Build component library
-        working-directory: component
-        run: npm run build
+      - name: Type-check app
+        working-directory: app
+        run: npx tsc --noEmit
 
-  # ── Job 2: Unit & integration tests with coverage ──────────────────────────
-  # Runs all three packages in a single job to avoid redundant npm ci.
+  # ── Job 1: Unit & integration tests with coverage ──────────────────────────
+  # All three packages run in parallel within a single job.
   # Connection integration tests use GitHub service containers.
   unit-tests:
     name: Unit & Integration Tests
     runs-on: ubuntu-latest
-    needs: lint-typecheck
     timeout-minutes: 15
 
     services:
@@ -129,19 +118,20 @@ jobs:
           npm ci --prefix connection &
           wait
 
-      - name: App unit tests (with coverage)
-        working-directory: app
-        run: npm run test:coverage
-        env:
-          NODE_ENV: test
-
-      - name: Component unit tests (with coverage)
-        working-directory: component
-        run: npm run test:coverage
-
-      - name: Connection integration tests (with coverage)
-        working-directory: connection
-        run: npm run test:coverage
+      - name: Run all tests in parallel
+        run: |
+          cd app && NODE_ENV=test npm run test:coverage &
+          APP_PID=$!
+          cd component && npm run test:coverage &
+          COMP_PID=$!
+          cd connection && npm run test:coverage &
+          CONN_PID=$!
+
+          FAIL=0
+          wait $APP_PID  || FAIL=1
+          wait $COMP_PID || FAIL=1
+          wait $CONN_PID || FAIL=1
+          exit $FAIL
         env:
           POSTGRES_HOST: localhost
           POSTGRES_PORT: '5432'
@@ -163,12 +153,11 @@ jobs:
             component/coverage/lcov.info
             connection/coverage/lcov.info
 
-  # ── Job 3: E2E tests with coverage ─────────────────────────────────────────
+  # ── Job 2: E2E tests with coverage ─────────────────────────────────────────
   # Playwright + Testcontainers. Runs in parallel with unit-tests.
   e2e:
     name: E2E Tests (Playwright)
     runs-on: ubuntu-latest
-    needs: lint-typecheck
     timeout-minutes: 25
 
     steps:
@@ -252,12 +241,12 @@ jobs:
           path: app/coverage-e2e/lcov.info
           if-no-files-found: warn
 
-  # ── Job 4: SonarCloud scan (no test execution) ─────────────────────────────
+  # ── Job 3: SonarCloud scan (no test execution) ─────────────────────────────
   # Downloads coverage artifacts from unit-tests + e2e and runs the scan.
   sonar:
     name: SonarCloud Scan
     runs-on: ubuntu-latest
-    needs: [unit-tests, e2e]
+    needs: [typecheck, unit-tests, e2e]
     if: ${{ !cancelled() }}
     timeout-minutes: 10