Skip to content

docs(adr): ADR-234 §11.3 — A3 revidiert, generate_project_facts archiviert#1007

Open
achimdehnert wants to merge 1 commit into
mainfrom
session/2026-07-08/achim-dehnert/adr234-a3-revision
Open

docs(adr): ADR-234 §11.3 — A3 revidiert, generate_project_facts archiviert#1007
achimdehnert wants to merge 1 commit into
mainfrom
session/2026-07-08/achim-dehnert/adr234-a3-revision

Conversation

@achimdehnert

Copy link
Copy Markdown
Owner

Summary

  • ADR-234 §11.3-Amendment ergaenzt: §11.2 A3 (generate_project_facts.py kanonisch,
    gen_project_facts.py Deprecation) wurde nie umgesetzt (keine PR, Uhr lief nie);
    Realitaet lief entgegengesetzt. §11.2 bleibt als Historie im Body erhalten (nicht geloescht).
  • scripts/generate_project_facts.py -> scripts/_ARCHIVED/generate_project_facts.py
    verschoben (dort lag bereits eine aeltere archivierte Fassung von 2025 — die aktuelle
    ersetzt sie, Git-Historie der alten Fassung bleibt ueber git log erhalten).
  • tools/repo_checker.py: hartcodierter Default /home/dehnert/github (CLI-Flag --repos-dir
    UND run_check()-Signatur) auf GITHUB_DIR-Konvention umgestellt (GITHUB_DIR >
    REPOS_DIR > ~/github), analog tools/sync_drift_meter.py.
  • .windsurf/workflows/onboarding-new-repo.md auf reinen Redirect-Stub zu /onboard-repo
    reduziert (Datei bleibt bestehen, keine toten Links).

Bekannter Rest-Scope (nicht Teil dieses PRs)

grep -rn generate_project_facts . zeigt zusaetzlich ~78 Treffer in
.windsurf/project-facts/*.md — Footer-Zeile „Neu generieren:
python3 platform/scripts/generate_project_facts.py ", geschrieben vom alten
(jetzt archivierten) Generator. Das sind Daten-Artefakte, keine aktiven Code-Pfade;
Fix waere ein gen_project_facts.py --force-Regenerierungslauf ueber alle Repos
(GitHub-API-lastig, groesserer Diff) — bewusst nicht in diesem PR gebuendelt, da
ausserhalb der 4 vereinbarten Schritte. Vorschlag: eigenes Follow-up-Issue.
Ausserdem 2 Treffer in docs/adr/ADR-132-...md (historische Beschreibung, unveraendert
gelassen wie ADR-234 §11.2).

Test plan

  • make test -> 378 passed, 1 warning (pre-existing, unabhaengig von diesem Diff)
  • iil-adrfw validate docs/adr/ -> exit 0; ADR-234 zeigt nur die gleichen
    Alias-Warnungen (decision-makers/relates_to/date) wie ~30 andere ADRs
    im Repo — keine neuen Fehler
  • git mv Historie fuer archivierte Datei erhalten
  • Pre-push Platform-Gates (Registry-View-Reader-Guard, ADR-226 publish-gate,
    check_publish_gate) liefen automatisch beim Push und sind gruen

Ref #994 (siehe „Bekannter Rest-Scope" oben — nicht alle Akzeptanzkriterien
im striktesten Sinn erfuellt, daher kein „Closes").

🤖 Generated with Claude Code

…viert (#994)

§11.2 A3 (2026-06-12) wurde nie umgesetzt (keine PR, Status blieb proposed).
Realitaet lief entgegengesetzt: gen_project_facts.py ist aktiv gepflegt
(bootstrap.sh, 6 Skill-Referenzen, eigene Tests), generate_project_facts.py
hat seit 06-12 keinen Commit mehr.

- ADR-234: §11.3-Amendment ergaenzt, §11.2 bleibt als Historie erhalten
- scripts/generate_project_facts.py -> scripts/_ARCHIVED/ verschoben
- tools/repo_checker.py: REPOS_DIR/hartcodierter Pfad-Default auf
  GITHUB_DIR-Konvention umgestellt (Zeilen ~958, ~1001)
- .windsurf/workflows/onboarding-new-repo.md auf reinen Redirect zu
  /onboard-repo reduziert

Ref: platform#994, repo-optimize 2026-07-08-runB (Befunde L-13/X-15).
@achimdehnert achimdehnert requested a review from wirdigital as a code owner July 8, 2026 12:04
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

⚠️ Architecture Guardian — Violations Found

Checked 5 files, +562/-1093 lines.

?? Auto-Warning (Gate 1)

  • G-004: PR hat 1655 geänderte Zeilen (Limit: 600) ((gesamt))
    • ?? Große PRs sind schwerer zu reviewen. Erwäge Aufteilen in kleinere PRs

Architecture Guardian v0.1.0

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

Context Reviewer

Checked 5 files.

Betroffene ADRs: ADR-009, ADR-021, ADR-028, ADR-035, ADR-040, ADR-042, ADR-043, ADR-045, ADR-048, ADR-049, ADR-050, ADR-052, ADR-054
Betroffene Prinzipien: P-002 Zero Breaking Changes, P-003 Tenant-Isolation, P-004 Minimal Diff
Betroffene Projekte: platform

Kontext-Hinweise

  • ADR → [ADR-009]: Diese Änderung betrifft Bereiche die in ADR-009 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-021]: Diese Änderung betrifft Bereiche die in ADR-021 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-028]: Diese Änderung betrifft Bereiche die in ADR-028 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-035]: Diese Änderung betrifft Bereiche die in ADR-035 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-040]: Diese Änderung betrifft Bereiche die in ADR-040 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-042]: Diese Änderung betrifft Bereiche die in ADR-042 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-043]: Diese Änderung betrifft Bereiche die in ADR-043 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-045]: Diese Änderung betrifft Bereiche die in ADR-045 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-048]: Diese Änderung betrifft Bereiche die in ADR-048 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-049]: Diese Änderung betrifft Bereiche die in ADR-049 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-050]: Diese Änderung betrifft Bereiche die in ADR-050 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-052]: Diese Änderung betrifft Bereiche die in ADR-052 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • ADR → [ADR-054]: Diese Änderung betrifft Bereiche die in ADR-054 geregelt sind. Bitte prüfen ob die Änderung konform ist.
  • Prinzip → [P-002 Zero Breaking Changes]: Änderung berührt P-002 Zero Breaking Changes. FYI für den Reviewer.
  • Prinzip → [P-003 Tenant-Isolation]: Änderung berührt P-003 Tenant-Isolation. FYI für den Reviewer.
  • Prinzip → [P-004 Minimal Diff]: PR hat 1655 geänderte Zeilen. Erwäge Aufteilen wenn möglich.

Context Reviewer v0.1.0 — informativ, nie blockierend


Context Reviewer v0.1.0 — informativ, nie blockierend

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

🤖 ADR Dual-Tool Review

ADR Structural AI Verdict Cost
ADR-234 ⏭️ skipped

Total cost this run: $0.0000
Model: claude-3-5-haiku-20241022 · Schema v4 ai_sparring_by patched

ADR-234 — ⏭️ Skipped

No review needed


Tool 1: iil-adrfw · Tool 2: Claude API (non-accountable — see ai_sparring_by)

@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

🤖 ADR Review — Score 8/10 (adr-review-passed)

SCORE: 8


Verdikt: Der ADR-234 ist eine umfassende und detaillierte Entscheidung, die die Einführung einer sauberen Repo-Zustand-Invariante als erzwungene Invariante statt eines laufenden Reparatur-Tasks zum Ziel hat.

Stärken:

  • Der ADR-234 adressiert ein wichtiges Problem, nämlich die Notwendigkeit, den Zustand von Repositories zu überwachen und sicherzustellen, dass sie sauber und konsistent sind.
  • Die Entscheidung ist gut begründet und basiert auf einer sorgfältigen Analyse der aktuellen Situation und der möglichen Alternativen.
  • Der ADR-234 enthält eine klare und detaillierte Beschreibung der Implementierungsschritte und der erforderlichen Änderungen.

Schwächen/Risiken:

  • Der ADR-234 ist sehr umfassend und komplex, was die Implementierung und das Verständnis erschweren kann.
  • Es gibt einige offene Fragen und Unklarheiten, wie z.B. die genaue Implementierung des R2-Backends und die Entscheidung über die Verwendung von GitHub-Rulesets oder eines imperativen Reconciler-Bots.
  • Der ADR-234 enthält einige technische Details, die möglicherweise nicht für alle Leser verständlich sind.
  • Es gibt einige potenzielle Risiken und Herausforderungen, wie z.B. die Möglichkeit, dass die Implementierung des R2-Backends nicht wie erwartet funktioniert oder dass die Entscheidung über die Verwendung von GitHub-Rulesets oder eines imperativen Reconciler-Bots nicht optimal ist.
  • Der ADR-234 enthält einige Abhängigkeiten von anderen ADRs und Entscheidungen, die möglicherweise nicht klar sind.

Konkrete Nachbesserung:

  • Es wäre hilfreich, den ADR-234 zu vereinfachen und zu strukturieren, um ihn besser verständlich zu machen.
  • Es wäre auch hilfreich, die offenen Fragen und Unklarheiten zu klären und die Implementierungsschritte genauer zu beschreiben.
  • Es wäre ratsam, die potenziellen Risiken und Herausforderungen genauer zu analysieren und Strategien zu entwickeln, um sie zu minimieren.

1 ADR-Datei(en) · via groq/llama-3.3-70b-versatile · Flatrate · informativ, nicht blockierend

@github-actions github-actions Bot added the adr-review-passed ADR Review passed (Score ≥7) label Jul 8, 2026
@achimdehnert

Copy link
Copy Markdown
Owner Author

Orchestrator-Abnahme: Alle 4 Schritte unabhängig verifiziert — repo_checker.py (2 Stellen, GITHUB_DIR>REPOS_DIR>~/github-Konvention korrekt) · onboarding-new-repo.md sauberer Redirect-Stub · ADR-234 §11.3-Text akkurat, §11.2 bewusst erhalten (Historie sichtbar) · 76-Stale-Footer-Fund an 2 Stichproben (137-hub, dev-hub) bestätigt real. 13/13 CI-Checks grün.

Ein Punkt geprüft, kein Blocker: git mv hat unter scripts/_ARCHIVED/generate_project_facts.py eine ANDERE, ältere archivierte Skript-Version überschrieben (386 vs. 552 Zeilen, unterschiedlicher Inhalt/Zweck aus einem früheren Cleanup). Verifiziert: kein Datenverlust — beide Versionen bleiben über git log --all -- scripts/_ARCHIVED/generate_project_facts.py abrufbar. Sauberer wäre ein unterscheidbarer Dateiname gewesen, aber nicht merge-blockierend.

„Ref #994" statt „Closes #994" korrekt — Akzeptanzkriterium „grep → nur _ARCHIVED/ADR-Historie" nicht 100% erfüllt (76 stale Footer-Referenzen in .windsurf/project-facts/*.md, GitHub-API-heavy Regen-Sweep über ~76 Repos wäre nötig — zu Recht als Folgearbeit ausgescoped statt hier improvisiert). ✅ merge-bereit — Merge beim Owner. #994 bleibt offen bis Owner über den Footer-Regen-Sweep entscheidet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adr-review-passed ADR Review passed (Score ≥7)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant