Hartmut/Nexus
1
0
Fork 0
Code Issues 11 Pull Requests 1 Actions Packages Projects Releases Wiki Activity
Files
4a5edeef3e42c3f837985daa9b14f1f1cde11acd
Nexus/docs/old-markdowns/review-report-2026-03-15-computation-graph.md
T
Hartmut 4a5edeef3e
CI / Unit Tests (pull_request) Successful in 5m46s
Details
CI / Lint (pull_request) Failing after 3m49s
Details
CI / E2E Tests (pull_request) Has been skipped
Details
CI / Fresh-Linux Docker Deploy (pull_request) Has been skipped
Details
CI / Assistant Split Regression (pull_request) Failing after 35s
Details
CI / Architecture Guardrails (pull_request) Failing after 2m14s
Details
CI / Typecheck (pull_request) Successful in 4m22s
Details
CI / Build (pull_request) Has been skipped
Details
CI / Release Images (pull_request) Has been skipped
Details
rename(phase 1): CapaKraken → Nexus across code, UI, docs, CI 
- @capakraken/* → @nexus/* across 12 packages (root + 11 workspaces),
  1551 import lines migrated via codemod
- User-visible brand strings renamed (emails, page titles, PWA
  manifest, mobile header, MFA backup-codes header, tooltips, signin
  page, invite page, weekly digest, install prompt)
- TOTP issuer "CapaKraken" → "Nexus" (existing secrets still valid;
  re-enrollment relabels them in users' authenticator apps)
- Function rename: assertCapaKrakenDbTarget → assertNexusDbTarget
- LocalStorage migration shim in apps/web/src/app/layout.tsx copies
  capakraken_* → nexus_* on first load (guarded by nexus_migrated_v1
  sentinel; runs once per browser, then never again)
- Service-worker cache name capakraken-v2 → nexus-v2 with one-time
  caches.delete('capakraken-v2') from the same shim
- Email-domain fixtures @capakraken.{dev,app} → @nexus.{dev,app} in
  seed data, e2e specs, SMTP default fallback
- Dockerfile.dev / Dockerfile.prod / all .github/workflows/*.yml
  pnpm --filter @capakraken/* → @nexus/*
- README, CLAUDE.md, LEARNINGS.md, all docs/*.md, .env.example,
  tooling/deploy/.env.production.example brand sweep

Phase 1 deliberately leaves untouched (handled in Phase 3 cutover):
- PostgreSQL DB name "capakraken" and POSTGRES_USER "capakraken"
- Volume names capakraken_pgdata etc.
- Compose project name "capakraken" / "capakraken-prod"
- db-target-guard default expectedDatabase
- env-var CAPAKRAKEN_EXPECTED_DB_NAME
- Container DNS names in docker-compose.ci.yml

Quality gates green: pnpm typecheck (7/7), pnpm test:unit (7/7),
pnpm lint (0 errors), check:exports/imports/architecture all pass.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
2026-05-21 15:10:44 +02:00

4.3 KiB
Raw Blame History

Review-Report — 2026-03-15 (3D Computation Graph)

Ergebnis: ✅ Bestanden

Alle Quality Gates bestanden. Keine kritischen Probleme. Zwei Minor-Empfehlungen.

Quality Gates

Gate Status Details
Engine Tests ✅ 283/283 (19 files)
Staffing Tests ✅ 37/37 (3 files)
API Tests ✅ 209/209 (21 files)
Application Tests ✅ 67/67 (15 files)
TypeScript (web) ✅ 0 errors (excl. BlueprintFieldEditor TS2589)
TypeScript (api) ✅ 0 errors

Code-Review-Checkliste

Architektur

  • Keine zirkulaeren Abhaengigkeiten — api → engine/shared/db (erlaubt)
  • engine und staffing unveraendert, keine DB-Imports
  • Neuer Router computationGraph in index.ts registriert
  • Keine SSE-Events noetig (read-only Feature, keine Mutations)

TypeScript & Typsicherheit

  • any-Types nur an react-force-graph-3d-Grenzen mit eslint-disable Kommentar (6 Stellen)
  • Prisma-Enums gecastet: pa.status as unknown as string + as Parameters<typeof computeBudgetStatus>[2]
  • JSONB-Feld gecastet: commercialTerms as { contingencyPercent?: number; ... } | null
  • scheduleRules as SpainScheduleRule | null korrekt
  • exactOptionalPropertyTypes beachtet: ...(formula ? { formula } : {}) Pattern

Datenbank & Prisma

  • Keine Schema-Aenderungen — rein lesende Queries
  • Geldbetraege in Integer-Cents: lcrCents, dailyCostCents, budgetCents etc.
  • Kein Seed noetig (kein neues Modell)

UI & Komponenten

  • "use client" Direktive gesetzt
  • Three.js via dynamic(() => import(...), { ssr: false }) — kein SSR-Problem
  • Neue Seite in AppShell-Navigation ergaenzt ("Computation Graph" unter Analytics)
  • Opake Hintergruende: bg-zinc-50, bg-zinc-900/95 (95% ist akzeptabel fuer Tooltip)

Sicherheit

  • Beide Procedures nutzen controllerProcedure (ADMIN + MANAGER + CONTROLLER)
  • Keine Raw-Queries — nur Prisma findMany/findUniqueOrThrow
  • Keine sensiblen Daten im Response — nur berechnete Werte und Formeln

Gefundene Probleme

Kritisch

Keine.

Minor

  1. Duplizierte Types — GraphNode, GraphLink, Domain sind sowohl in packages/api/.../computation-graph.ts als auch apps/web/.../domain-colors.ts definiert. Funktioniert (tRPC inferiert die Typen), aber bei Aenderungen muss man beide Stellen anpassen. Empfehlung: Types nach @nexus/shared verschieben wenn sie stabil sind.

  2. project.list Query — Der Client castet das Ergebnis via (projectData as any)?.projects ?? (projectData as any). Das deutet auf Unsicherheit ueber das Return-Format hin. Sollte nach dem Merge geprueft werden, ob .projects oder direkt das Array zurueckkommt.

Empfehlungen

  1. Bundle Size Monitoring — three und react-force-graph-3d fuegen ~700KB (gzipped) hinzu. Dank dynamic import + { ssr: false } trifft das nur die Computation Graph Seite. Trotzdem: bei der naechsten Bundle-Analyse verifizieren.

  2. E2E-Test — Aktuell kein Test fuer die neue Seite. Ein Playwright-Smoke-Test (navigate to /analytics/computation-graph, expect canvas element) waere sinnvoll.

  3. Link Formula Labels — Plan sah Three.js Text-Sprites auf Kanten vor (E3). Die Formeln sind in den Link-Daten vorhanden (formula Feld), werden aber aktuell nur bei Hover (indirekt ueber Node-Tooltip) sichtbar. Kann als Follow-up ergaenzt werden.

Learnings-Vorschlag fuer LEARNINGS.md

### react-force-graph-3d in Next.js 15

- Muss als `dynamic(() => import("react-force-graph-3d"), { ssr: false })` geladen werden
- React 19 Kompatibilitaet: funktioniert, aber TypeScript-Generics sind loose — `as any` Cast + eslint-disable noetig bei Callbacks (`onNodeClick`, `nodeThreeObject`, `linkColor`)
- Node-Rendering via Canvas-basierte Three.js Sprites (nicht HTML-Overlays) — performanter bei 50+ Knoten
- `warmupTicks={50}` + `cooldownTicks={0}` verhindert die Kraft-Simulation und nutzt stattdessen die fixen `fx/fy/fz` Positionen
Reference in New Issue View Git Blame Copy Permalink