lukas/nixos-config
Archived
1
0
Fork 1
Code Issues Pull requests Releases Wiki Activity
This repository has been archived on 2026-09-17. You can view files and clone it, but you cannot make any changes to its state, such as pushing and creating new issues, pull requests or comments.
nixos-config/docs/review/board-show-commit-on-card.md
pijukka 05575df408 docs: commit backlog of already-resolved intent/review corpus files 
board-agent's own README documents docs/intent/ and docs/research-intent/
as tracked corpus directories, and docs/review/<slug>.md as the reviewer's
persisted report per card - but the pipeline only writes these files to
the checkout, it never commits them itself. This had built up a backlog
of intent/review docs for cards already marked deployed or archiviert in
the live board state, which were sitting uncommitted in the working tree.

Adds that backlog as-is: the originating intents, their reviewer reports,
the research-intent/research pair for the board UX evaluation, and one
standalone process-retrospective report (board-review-pipeline-context-
overhead.md, same genre as the two already-tracked audit reports in
docs/review/). No code changes.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01T7UVrKsttMP9a6FiGFna9b
2026-09-10 12:06:53 +02:00

2 KiB
Raw Permalink Blame History

Geprüfter Commit: 6acea5a ("board-agent: Commit-Hash auf Karte im Board-UI anzeigen") Scope: hosts/blunix/localServices/board-agent/board.html, trigger.py, worker.py (3 Dateien, +14/-0 Zeilen laut Diff)

Prüfmethoden:

  • git show 6acea5a (vollständiger Diff) gelesen
  • grep -rn commit_hash über board-agent/, um alle Berührungspunkte der neuen Kette zu finden
  • Datenfluss end-to-end nachvollzogen: completion_watcher.py (Quelle: commits[0]["hash"], unverändert in diesem Commit) → Review-Job-Payload → worker.py update_state(..., commit_hash=...) beim Abschluss des Review-Jobs → trigger.py merged_board() (zwei Stellen, für Cards mit vorhandenem und mit fehlendem Intent-Dokument) → board.html renderCard() (Anzeige der ersten 7 Zeichen)
  • worker.py process_job/update_state gelesen, um Verhalten bei Fehlerpfaden zu prüfen (dort wird commit_hash nicht übergeben; da update_state per card.update(extra) nur vorhandene Keys mergt, bleibt ein zuvor gespeicherter Wert erhalten statt gelöscht zu werden – kein Bug)
  • board.html: Anzeige nutzt textContent (keine XSS-Gefahr) und ist durch if (card.commit_hash) gegen fehlende/leere Werte abgesichert
  • git diff 6acea5a~1 6acea5a -- test_board_agent.py geprüft: keine Teständerungen in diesem Commit enthalten

Befunde:

  • Keine funktionalen oder sicherheitsrelevanten Probleme gefunden. Änderung ist rein additiv (neues optionales Feld, defensiv konsumiert), kein bestehender Pfad wird verändert.
  • Kleinerer Hinweis (nicht blockierend): Für diese Erweiterung wurden keine neuen Tests ergänzt (z. B. dass commit_hash korrekt in merged_board()/update_state landet). Bei einer so kleinen, gut nachvollziehbaren Änderung mit klarer Datenkette ist das vertretbar, wäre aber die einzige Verbesserungsmöglichkeit.

Unsicherheiten:

  • Das UI-Rendering (board.html) wurde nur durch Code-Lektüre, nicht im Browser geprüft (kein Browserzugriff in diesem read-only Review-Lauf).

Entscheidung: bereit für Deployment