05575df408
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
23 lines
2.9 KiB
Markdown
23 lines
2.9 KiB
Markdown
Geprüfter Commit: 04ceda4681308e099b7e27b095fa31837cbaf0bc ("board-agent: drei getrennte Klickbereiche (Intent/Commit/Review) statt überlagerter Karten-Handler")
|
|
|
|
Scope: `git show 04ceda4` zeigt genau eine geänderte Datei: `hosts/blunix/localServices/board-agent/board.html`. Das entspricht der Vorgabe aus `docs/intent/board-card-three-links.md` ("Genau eine Datei, genau diese Änderung").
|
|
|
|
Prüfmethoden:
|
|
- Vollständiger Diff via `git show 04ceda4` gelesen.
|
|
- Intent-Dokument `docs/intent/board-card-three-links.md` als Referenz herangezogen und Punkt für Punkt mit dem Diff abgeglichen.
|
|
- Aktuellen Dateizustand (`renderCard()` und CSS-Regeln) gelesen, um sicherzustellen, dass keine toten/überlappenden Handler auf dem Karten-Element `el` verbleiben.
|
|
- Gezielt nach verbliebenen `el.tabIndex`, `el.setAttribute`, `el.addEventListener`-Aufrufen und nach Verwendung der CSS-Klasse `.has-review` gesucht.
|
|
|
|
Befunde:
|
|
1. (informativ, keine Handlung nötig) Die Klasse `.has-review` wird weiterhin per `el.classList.add("has-review")` gesetzt (Zeile 330), hat aber keine zugehörige CSS-Regel mehr (die alte `.card.has-review { cursor: pointer; }`-Regel wurde im Zuge des Umbaus entfernt). Toter Code ohne funktionalen Effekt, keine Fehlfunktion.
|
|
2. Intent-Element: wird bei jeder `card.source === "intent"` erzeugt, mit eigenem `tabIndex`, `role="button"`, `aria-haspopup`, `aria-label`, Klick- und Enter/Space-Keydown-Handler auf `showIntent(card)` — entspricht der Vorgabe.
|
|
3. Commit-Anzeige: reiner `<span class="commit-hash">` mit gekürztem Hash (`slice(0,7)`), kein Klick-Handler, kein `href` — funktional identisch zur bisherigen Anzeige, jetzt aber als eigenständiges Element ohne überlagerten Klick-Bereich. Entspricht der Vorgabe.
|
|
4. Review-Element: nur bei vorhandenem `card.review_report`, mit eigenem `tabIndex`, `role="button"`, `aria-haspopup`, `aria-label`, Klick- und Keydown-Handler auf `showReview(card)`. Entspricht der Vorgabe.
|
|
5. Die alten, überlagerten Handler auf dem gesamten Karten-Element `el` (Klick/Keydown für Intent bzw. Review, gemeinsames `tabIndex`/`role`/`aria-*`) wurden vollständig entfernt; `el` behält nur noch den ursprünglichen `dragstart`-Handler.
|
|
6. Kein Forgejo-/Repo-Link eingeführt, wie im Intent explizit ausgeschlossen.
|
|
7. Keine anderen Dateien im Commit verändert (README.md, test_board_agent.py, trigger.py zeigen als geänderte Dateien im Arbeitsbaum, sind aber nicht Teil dieses Commits — außerhalb des vorgegebenen Scopes und daher hier nicht bewertet).
|
|
|
|
Unsicherheiten:
|
|
- Keine automatisierten Tests für board.html vorhanden (reines Frontend-HTML/JS ohne Test-Harness im Repo); Prüfung erfolgte ausschließlich per Code-Lesung, kein Browser-Rendering durchgeführt. Das Verhalten (drei unabhängig bedienbare Elemente ohne gegenseitige Auslösung) ist aus dem Code klar ableitbar, aber nicht per UI-Test verifiziert.
|
|
|
|
Entscheidung: bereit für Deployment
|