From 71b627e5ba9ab82f139160a8b6b0b8559196444c Mon Sep 17 00:00:00 2001 From: Thorsten Hindermann Date: Thu, 18 Jun 2026 13:10:17 +0200 Subject: [PATCH] Fix Claude review for release bot PRs --- .github/workflows/claude-code-review.yml | 4 +++- docs/project-statistics.md | 3 ++- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/.github/workflows/claude-code-review.yml b/.github/workflows/claude-code-review.yml index b739b56..27a216f 100644 --- a/.github/workflows/claude-code-review.yml +++ b/.github/workflows/claude-code-review.yml @@ -39,6 +39,8 @@ jobs: plugin_marketplaces: 'https://github.com/anthropics/claude-code.git' plugins: 'code-review@claude-code-plugins' prompt: '/code-review:code-review ${{ github.repository }}/pull/${{ github.event.pull_request.number }}' + # Release-Please-PRs werden von github-actions[bot] erstellt; nur dieser Bot darf den Review-Lauf starten. + # Release Please PRs are created by github-actions[bot]; only this bot may start the review run. + allowed_bots: 'github-actions' # See https://github.com/anthropics/claude-code-action/blob/main/docs/usage.md # or https://code.claude.com/docs/en/cli-reference for available options - diff --git a/docs/project-statistics.md b/docs/project-statistics.md index 5ccebc2..d5e306f 100644 --- a/docs/project-statistics.md +++ b/docs/project-statistics.md @@ -1,6 +1,6 @@ # Projektstatistik InventarWorkerService -Stand: 2026-05-05 (aktualisiert inklusive Spec-Kit-Preset-Governance, Constitution v1.13.0 und fortgeschriebenem Fortschreibungsprotokoll sowie didaktische Inline-Code-Kommentar-Haertung fuer InventarWorkerService) +Stand: 2026-06-18 (aktualisiert inklusive Spec-Kit-Preset-Governance, Constitution v1.13.0, didaktischer Inline-Code-Kommentar-Haertung und Claude-Code-Review-Bot-Freigabe fuer Release-Please-PRs) ## Zweck und Pflege @@ -291,6 +291,7 @@ und auf explizite Anforderung fortgeschrieben. | 2026-04-19 | Zweiter Docs-Hotfix fuer Axe/Playwright-Kontext | Der erste Workflow-Hotfix beseitigte zwar das Paketauflösungsproblem, der folgende Lauf scheiterte aber erneut daran, dass `@axe-core/playwright` nicht mit `browser.newPage()` betrieben werden darf. Der Smoke-Test verwendet jetzt explizit `browser.newContext()` und erzeugt die Seite daraus, bevor `AxeBuilder` ausgeführt wird. Sichtbares Zusatzvolumen dieser Runde: ca. `+3` Produktions-/Konfigurationszeilen netto im Workflow plus `+1` Dokumentationszeile netto in diesem Ledger. Konservative Manualreferenz: grob `4 / 80 = 0.1` Arbeitstage bzw. `0.4` Stunden; Thorsten-Solo-Referenz: `4 / 100 = 0.0` Arbeitstage bzw. `0.3` Stunden; sichtbares Arbeitsfenster: fortgesetzte agentische Sitzung am 2026-04-19. | | 2026-05-05 | Spec-Kit-Preset-Governance auf Constitution v1.13.0 synchronisiert | Nach der Integration der sechs Spec-Kit-Presets (`a11y-governance`, `agent-parity-governance`, `architecture-governance`, `cross-platform-governance`, `isaqb-architecture-governance`, `security-governance`) wurden `constitution.md`, `.specify/memory/constitution.md`, die Spec-Kit-Plan-/Spec-/Tasks-/Command-Templates sowie die vier Agentenflaechen synchronisiert. Neu erfasst sind iSAQB/arc42-Architekturevidenz unter `docs/architecture/`, A11Y-Evidenz unter `docs/accessibility/`, sprachgetaggte Markdown-Codebloecke, CRA-/MSL-/Secure-Coding-Evidenz und Cross-Platform-/Agent-Parity-Templates. Sichtbares Zusatzvolumen vor dieser Ledger-Zeile: ca. `+1269` Dokumentations-/Template-Zeilen netto, `0` Produktionscode-Zeilen und `0` Testcode-Zeilen. Konservative Manualreferenz: `1269 / 80 = 15.9` Arbeitstage bzw. `123.8` Stunden; Thorsten-Solo-Referenz: `1269 / 100 = 12.7` Arbeitstage bzw. `99.0` Stunden; sichtbares Arbeitsfenster: 1 agentische Governance-Sitzung am 2026-05-05. | | 2026-06-05 | Didaktische Inline-Code-Kommentar-Haertung vorbereitet | `Lastenheft_Didactic-Inline-Code-Comment-Hardening.md` wurde als Specify-ready Intake fuer eine moderate Inline-Kommentar-Haertung angelegt. Der Lauf soll Service-, API-, Persistenz-, Cross-Platform-, TUI- und Test-Helfer-Flows pruefen, ohne Runtime-Verhalten, Datenbank-/API-/TUI-Funktionen oder Architektur zu veraendern. `AGENTS.md`, `CLAUDE.md`, `GEMINI.md` und `.github/copilot-instructions.md` halten nun fest, dass neue oder geaenderte nicht-triviale Logik auf didaktischen Kommentarbedarf geprueft wird und Kommentare Warum, Trade-off, Randbedingung, Plattformgrenze oder Proof-Grenze erklaeren muessen. Validierung: Doku-/Guidance-Suchcheck und `git diff --check`; keine Build-/Test-/DocFX-Ausfuehrung, weil nur Lastenheft, Guidance und Statistik geaendert wurden. | +| 2026-06-18 | Claude-Code-Review fuer Release-Please-PRs freigegeben | Der automatische `Claude Code Review`-Workflow blockierte Release-Please-PRs, weil `github-actions[bot]` als nicht-menschlicher Actor nicht in `allowed_bots` freigegeben war. `.github/workflows/claude-code-review.yml` erlaubt nun gezielt den Bot-Slug `github-actions`, ohne alle Bots per Wildcard zuzulassen. Sichtbares Zusatzvolumen: `0` Produktionscode-Zeilen, `0` Testcode-Zeilen, ca. `+3` Workflow-Konfigurationszeilen und diese Ledger-Fortschreibung. Konservative Manualreferenz: grob `4 / 80 = 0.1` Arbeitstage bzw. `0.4` Stunden; Thorsten-Solo-Referenz: `4 / 100 = 0.0` Arbeitstage bzw. `0.3` Stunden; sichtbares Arbeitsfenster: 1 kurze Agentensitzung am 2026-06-18. | ## Gesamtstatistik