feat(nexus-dev): nexus-stack-Skill gegen realen Code-Stand (K-101/103/104/106) #2

Merged
l.kirchner merged 2 commits from feat/nexus-stack-skill-real into main 2026-06-13 13:44:00 +02:00
Collaborator

nexus-stack-Skill (Plugin nexus-dev) gegen den realen, gemergten Code-Stand auf nexus-hub main neu geschrieben — der bisherige Inhalt war ein K-101-Platzhalter, inzwischen existieren K-101/103/104/106 als gemergter Code.

Knappe, präskriptive Muster, jede Behauptung am Code verifiziert:

  • create_app-Factory + configure_logging-Reihenfolge, Settings/Secret-Repr, fail-closed-Defaults
  • visibility.py als EINZIGER Lese-Query-Pfad (SICHT-1/2/3) + der Architektur-Test, der direkte Selects/Raw-SQL außerhalb verbietet; SICHT-4-Suite-Pflicht
  • router.run als einzige LLM-Schnittstelle: Hard-Guard + LOCAL_ENDPOINT_ALLOWLIST (gepinnte luki-ai host:port), kein externer Fallback, AST-Import-Guard
  • Session-Auth via get_current_user/require_admin (404 für member), kein Dev-/Header-Mechanismus mehr; OIDC-State-Regel
  • audit.record-Whitelist (append-only), PRIV-5-Redaction-CONTENT_KEYS
  • Alembic-Autogenerate-Fluss (Constraints benennen, Trigger/Extensions von Hand), Wegwerf-pgvector/pg16-Container-Strategie der CI

Plugin 0.2.0 → 0.3.0. Review: codex-bot.

nexus-stack-Skill (Plugin nexus-dev) gegen den realen, gemergten Code-Stand auf nexus-hub main neu geschrieben — der bisherige Inhalt war ein K-101-Platzhalter, inzwischen existieren K-101/103/104/106 als gemergter Code. Knappe, präskriptive Muster, jede Behauptung am Code verifiziert: - `create_app`-Factory + `configure_logging`-Reihenfolge, Settings/Secret-Repr, fail-closed-Defaults - `visibility.py` als EINZIGER Lese-Query-Pfad (SICHT-1/2/3) + der Architektur-Test, der direkte Selects/Raw-SQL außerhalb verbietet; SICHT-4-Suite-Pflicht - `router.run` als einzige LLM-Schnittstelle: Hard-Guard + `LOCAL_ENDPOINT_ALLOWLIST` (gepinnte luki-ai host:port), kein externer Fallback, AST-Import-Guard - Session-Auth via `get_current_user`/`require_admin` (404 für member), kein Dev-/Header-Mechanismus mehr; OIDC-State-Regel - `audit.record`-Whitelist (append-only), PRIV-5-Redaction-CONTENT_KEYS - Alembic-Autogenerate-Fluss (Constraints benennen, Trigger/Extensions von Hand), Wegwerf-pgvector/pg16-Container-Strategie der CI Plugin 0.2.0 → 0.3.0. Review: codex-bot.
claude-bot added 1 commit 2026-06-13 13:37:50 +02:00
Replaces the K-101 placeholder with prescriptive patterns verified
against nexus-hub main: create_app factory + configure_logging order,
Settings/secret-repr, the visibility.py single-query-path with the
architecture test that bans direct selects, router.run as the only LLM
interface (hard guard + LOCAL_ENDPOINT_ALLOWLIST), session auth via
get_current_user/require_admin (404 for member), audit.record whitelist,
PRIV-5 redaction CONTENT_KEYS, alembic autogenerate flow (name
constraints, hand-add triggers/extensions), and the disposable
pgvector/pg16 CI container strategy. Plugin 0.2.0 -> 0.3.0.
l.kirchner added 1 commit 2026-06-13 13:40:45 +02:00
codex-bot approved these changes 2026-06-13 13:41:11 +02:00
codex-bot left a comment
Collaborator

Codex-Review (Faktencheck gegen nexus-hub main): 3 Findings gemeldet (überzeichnete Audit-on-block-Garantie, zu breit beschriebene Architektur-Test-Patterns, fail-closed nicht universell) — alle in 17e6ad0→HEAD präzisiert. Restliche Behauptungen deckungsgleich mit dem gemergten Code (App-Factory, visibility-Pfad + ALLOWED_FILES, router.run-Reihenfolge/Hard-Guard/Allowlist, Session-Auth 401/404, audit-Whitelist+TRUNCATE-Trigger, CONTENT_KEYS, Alembic, CI-pgvector-Ports/SICHT-4). Approve — Merge via Browser-Claude.

Codex-Review (Faktencheck gegen nexus-hub main): 3 Findings gemeldet (überzeichnete Audit-on-block-Garantie, zu breit beschriebene Architektur-Test-Patterns, fail-closed nicht universell) — alle in 17e6ad0→HEAD präzisiert. Restliche Behauptungen deckungsgleich mit dem gemergten Code (App-Factory, visibility-Pfad + ALLOWED_FILES, router.run-Reihenfolge/Hard-Guard/Allowlist, Session-Auth 401/404, audit-Whitelist+TRUNCATE-Trigger, CONTENT_KEYS, Alembic, CI-pgvector-Ports/SICHT-4). Approve — Merge via Browser-Claude.
l.kirchner merged commit 463bd5fbba into main 2026-06-13 13:44:00 +02:00
Sign in to join this conversation.
No Reviewers
No labels
3 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: l.kirchner/claude-marketplace#2