feat: add changelog page to website#154
Merged
Merged
Conversation
Adds a chronological changelog (docs/changelog.adoc) listing all semantic anchors added to the catalog with dates and community contributor credits. Integrates it into the website as a new nav item with EN/DE i18n support. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
WalkthroughFügt eine neue Changelog-Seite zur Dokumentationswebsite hinzu: Änderungen
SequenzdiagrammsequenceDiagram
participant User
participant Nav as Header Navigation
participant Router as Router
participant PageRender as renderChangelogPage
participant DocRender as renderDocPage
participant DocLoader as docs/changelog.adoc
User->>Nav: Klick auf "Changelog"
Nav->>Router: Navigate to /changelog
Router->>PageRender: Aufruf renderChangelogPage
PageRender->>DocRender: initialisiere renderDocPage
DocRender->>Nav: aktiviere Nav-Link
DocRender->>DocLoader: lade docs/changelog.adoc
DocLoader-->>PageRender: liefere Inhalt
PageRender-->>User: zeige Changelog-Seite
Geschätzter Code-Review-Aufwand🎯 3 (Moderate) | ⏱️ ~20 minutes Möglicherweise verwandte PRs
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@website/src/main.js`:
- Around line 182-189: The changelog HTML isn't generated because the build
script (render-docs.js) only calls renderFile for about.adoc, CONTRIBUTING.adoc
and all-anchors.adoc; add two renderFile calls to render the changelog sources
so loadDocContent('docs/changelog.adoc') used by renderChangelogPage can fetch
the generated file: add renderFile(path.join(ROOT, 'docs/changelog.adoc'),
path.join(WEB_DOCS, 'changelog.html')) and renderFile(path.join(ROOT,
'docs/changelog.de.adoc'), path.join(WEB_DOCS, 'changelog.de.html')) to the list
of renderFile invocations in render-docs.js (and ensure the build task that runs
render-docs.js includes these new outputs).
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yml
Review profile: CHILL
Plan: Pro
Run ID: cb89a07e-cdbe-46a2-99a8-d0f940ca1dad
📒 Files selected for processing (5)
docs/changelog.adocwebsite/src/components/header.jswebsite/src/main.jswebsite/src/translations/de.jsonwebsite/src/translations/en.json
Comment on lines
+182
to
+189
| function renderChangelogPage() { | ||
| const pageContent = document.getElementById('page-content') | ||
| if (!pageContent) return | ||
|
|
||
| pageContent.innerHTML = renderDocPage() | ||
| updateActiveNavLink() | ||
| loadDocContent('docs/changelog.adoc') | ||
| } |
Contributor
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Prüfen, ob changelog.adoc im render-docs.js enthalten ist
echo "=== Suche nach changelog in render-docs.js ==="
rg -n "changelog" scripts/render-docs.js || echo "changelog nicht gefunden in render-docs.js"
echo ""
echo "=== Aktuelle renderFile Aufrufe ==="
rg -n "renderFile\(" scripts/render-docs.jsRepository: LLM-Coding/Semantic-Anchors
Length of output: 712
🏁 Script executed:
#!/bin/bash
echo "=== Suche nach renderChangelogPage ==="
rg -n "renderChangelogPage" website/src/main.js | head -20
echo ""
echo "=== Suche nach loadDocContent Implementierung ==="
rg -n "function loadDocContent" website/src/ -A 5
echo ""
echo "=== Prüfen auf docs/changelog.adoc ==="
find . -name "*changelog*" -type f | grep -E "\.(adoc|html)$" || echo "Keine changelog-Dateien gefunden"
echo ""
echo "=== Prüfen auf addRoute('/changelog') ==="
rg -n "addRoute.*changelog" website/src/main.jsRepository: LLM-Coding/Semantic-Anchors
Length of output: 859
Kritisch: docs/changelog.html wird nicht im Build-Prozess generiert.
Die Funktion renderChangelogPage (Zeile 182) ruft loadDocContent('docs/changelog.adoc') auf, die dies in docs/changelog.html konvertiert und abruft. Die Route ist registriert (Zeile 105), und die Quelldatei docs/changelog.adoc existiert. Allerdings fehlt in scripts/render-docs.js der entsprechende renderFile-Aufruf – nur about.adoc, CONTRIBUTING.adoc und all-anchors.adoc werden gerendert. Dies führt zu einem 404-Fehler beim Aufrufen der /changelog-Route.
Fügen Sie in scripts/render-docs.js die fehlenden renderFile-Aufrufe hinzu:
renderFile(path.join(ROOT, 'docs/changelog.adoc'), path.join(WEB_DOCS, 'changelog.html'))
renderFile(path.join(ROOT, 'docs/changelog.de.adoc'), path.join(WEB_DOCS, 'changelog.de.html'))🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@website/src/main.js` around lines 182 - 189, The changelog HTML isn't
generated because the build script (render-docs.js) only calls renderFile for
about.adoc, CONTRIBUTING.adoc and all-anchors.adoc; add two renderFile calls to
render the changelog sources so loadDocContent('docs/changelog.adoc') used by
renderChangelogPage can fetch the generated file: add renderFile(path.join(ROOT,
'docs/changelog.adoc'), path.join(WEB_DOCS, 'changelog.html')) and
renderFile(path.join(ROOT, 'docs/changelog.de.adoc'), path.join(WEB_DOCS,
'changelog.de.html')) to the list of renderFile invocations in render-docs.js
(and ensure the build task that runs render-docs.js includes these new outputs).
- Dismiss onboarding modal via localStorage before tests - Use #header-search-input and #header-role-filter (desktop) instead of mobile-only #search-input and #role-filter - Update h1 expectations to h2 (heading level changed in layout redesign) - Use .first() for nav link clicks to target desktop nav - Fix heading hierarchy test for new structure Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This was referenced Mar 9, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
docs/changelog.adoc) listing all semantic anchors added to the catalog/changelogloads and renders the AsciiDoc contentCloses #153
Test plan
#/changelog🤖 Generated with Claude Code
Summary by CodeRabbit
Neue Funktionen
Dokumentation
Tests