Skip to content

feat: add DRY and Event Storming anchors#569

Merged
rdmueller merged 1 commit into
LLM-Coding:mainfrom
raifdmueller:feat/dry-eventstorming-anchors
Jun 2, 2026
Merged

feat: add DRY and Event Storming anchors#569
rdmueller merged 1 commit into
LLM-Coding:mainfrom
raifdmueller:feat/dry-eventstorming-anchors

Conversation

@raifdmueller
Copy link
Copy Markdown
Contributor

@raifdmueller raifdmueller commented Jun 2, 2026

Adds two community-proposed anchors, both verified against the four quality criteria (Precise, Rich, Consistent, Attributable) and the recognition test.

DRY (Don't Repeat Yourself) — closes #560

Proposed by @GaboCapo. The Pragmatic Programmer (Hunt & Thomas, 1999). Core nuance captured: DRY targets duplicated knowledge/intent, not coincidental textual similarity; includes the Rule of Three (Fowler) and the wrong-abstraction caution (Sandi Metz) so the anchor doesn't read as a license to over-abstract. Category: Design Principles.

Event Storming according to Alberto Brandolini — closes #558

Proposed by @SidekickJohn. Collaborative domain-event modeling workshop (Brandolini, 2012/2013). Captures the colour notation, the three levels (Big Picture / Process Modeling / Design Level), pivotal events as bounded-context seams, hotspots, and Reverse Event Storming for legacy reconstruction. Categories: Requirements Engineering + Software Architecture.

Changes

  • docs/anchors/{dry,event-storming}.adoc + .de.adoc (EN + DE)
  • Regenerated website/public/data/*.json (additive — 156 → 158 anchors)
  • docs/changelog.adoc — 2026-06-02 section
  • skill/.../catalog.md (+ synced plugins/ copy)

Closes #560
Closes #558

Summary by CodeRabbit

Dokumentation

  • Zwei neue Semantic Anchors hinzugefügt: DRY (Don't Repeat Yourself) und Event Storming according to Alberto Brandolini
  • Umfassende Dokumentation in Englisch und Deutsch mit Kernkonzepten und Anwendungsrichtlinien verfügbar
  • Aktualisierte Kataloge, Kategorien und Rollenverknüpfungen für verbesserte Navigationserfahrung

)

DRY (Don't Repeat Yourself) — single authoritative representation of
knowledge; Rule of Three and wrong-abstraction caution as guardrails.
Proposed by @GaboCapo (LLM-Coding#560).

Event Storming according to Alberto Brandolini — collaborative
domain-event modeling workshop with colour notation and three levels.
Proposed by @SidekickJohn (LLM-Coding#558).

Each with EN + DE content, regenerated metadata, changelog and catalog
entries.

Closes LLM-Coding#560
Closes LLM-Coding#558
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Jun 2, 2026

Review Change Stack

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

Die PR fügt zwei neue Semantic Anchors in das Dokumentationssystem ein: DRY (Don't Repeat Yourself) als Design-Prinzip und Event Storming according to Alberto Brandolini als Requirements-Engineering-Methode. Beide werden mit English/Deutsch-Dokumentation, Katalogeinträgen und Website-Datenregistrierung registriert.

Changes

Semantic Anchors: DRY und Event Storming

Layer / File(s) Summary
DRY-Anker: Dokumentation, Katalog und Registrierung
docs/anchors/dry.adoc, docs/anchors/dry.de.adoc, skill/semantic-anchor-translator/references/catalog.md, plugins/semantic-anchors/skills/semantic-anchor-translator/references/catalog.md, website/public/data/anchors.json, website/public/data/categories.json, website/public/data/roles.json
DRY-Dokumentation wird in AsciiDoc (Englisch und Deutsch) mit Metadaten, Kernkonzepten und verwandten Ankern definiert, in beide Katalog-Dateien eingetragen und als JSON-Eintrag in die Website-Datenstruktur aufgenommen. Kategorien und Rollen werden entsprechend aktualisiert.
Event Storming-Anker: Dokumentation, Katalog und Registrierung
docs/anchors/event-storming.adoc, docs/anchors/event-storming.de.adoc, skill/semantic-anchor-translator/references/catalog.md, plugins/semantic-anchors/skills/semantic-anchor-translator/references/catalog.md, website/public/data/anchors.json, website/public/data/categories.json, website/public/data/roles.json
Event Storming-Dokumentation wird in AsciiDoc (Englisch und Deutsch) mit Workshop-Konzepten, Farb­notation und Pivot-Events definiert, in beide Katalog-Dateien eingetragen und als JSON-Eintrag registriert. Mehrere Kategorien und Rollen werden aktualisiert.
Changelog- und Metadaten-Aktualisierung
docs/changelog.adoc, website/public/data/metadata.json
Changelog wird um Einträge für beide neuen Anker erweitert. Metadaten-Datei wird aktualisiert (Anker-Zähler von 156 auf 158, Statistiken wie averageRolesPerAnchor und anchorsWithTags), und generatedAt-Zeitstempel wird aktualisiert.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • LLM-Coding/Semantic-Anchors#408: Aktualisiert ebenfalls die Semantic-Anchor-Katalog-Datei skill/semantic-anchor-translator/references/catalog.md durch Hinzufügen neuer Anchor-Einträge.
  • LLM-Coding/Semantic-Anchors#561: Ergänzt neue Semantic-Anchor-Inhalte und aktualisiert die gemeinsamen Website-Katalog- und Datendateien (website/public/data/anchors.json, categories.json, roles.json, metadata.json) durch Einfügen neuer Anchor-Einträge.
  • LLM-Coding/Semantic-Anchors#209: Modifiziert die gemeinsamen Website-Metadaten- und Anker-Registrierungen (website/public/data/anchors.json, categories.json, roles.json, metadata.json) zur Erweiterung und Neugenerierung von Anchor-Einträgen.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Der PR-Titel fasst präzise die Hauptänderungen zusammen: das Hinzufügen von zwei neuen Semantic Anchors (DRY und Event Storming).
Linked Issues check ✅ Passed Die Implementierung erfüllt alle Anforderungen aus den verknüpften Issues #560 und #558 vollständig.
Out of Scope Changes check ✅ Passed Alle Änderungen sind direkt auf die Anforderungen der beiden verknüpften Issues beschränkt und im Scope.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@rdmueller rdmueller merged commit 2e8dfc4 into LLM-Coding:main Jun 2, 2026
7 of 8 checks passed
rdmueller pushed a commit that referenced this pull request Jun 2, 2026
…ple -> dry)

The code-quality contract referenced anchor id 'dry-principle', which
does not exist — the DRY anchor's id is 'dry' (added in #569). The chip
linked to a 404. Point it at the real anchor.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Anchor Proposal]: DRY [Anchor Proposal]: Event Storming according to Alberto Brandolini

2 participants