Sync from fork: host Brownfield Experiment & Fair Comparison reports on-site#464
Merged
Conversation
Previously linked externally to a personalAssistant repo. The report documents a controlled experiment: delete documentation from a greenfield project (Bausteinsicht), regenerate from code via the Semantic-Anchor-driven prompt, and compare. Findings include the Brownfield Preparation Checklist (six items code cannot reveal) and v2 of the prompt with role assignments on Open Questions. Wires it up as a first-class route /brownfield-experiment-report so brownfield-workflow.adoc can link to it locally instead of the external repo. The Fair Comparison link is left unchanged for now — not yet hosted.
The brownfield-workflow page references a second report — the Fair Comparison of Direct vs. Socratic vs. Two-Phase reverse-engineering approaches with identical team answers. Like the Experiment 1a report, host it locally instead of pointing at the personalAssistant repo. - New file docs/brownfield-fair-comparison.adoc - New route /brownfield-fair-comparison wired the same way as the Experiment 1a report (router title, page handler, fallback handler, render-docs entry, prerender-routes entry — 13 routes pre-rendered) - brownfield-workflow.adoc now links to the internal route
…eport feat: host Brownfield Experiment 1a Report on the site
|
ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (7)
WalkthroughDer PR fügt zwei neue AsciiDoc-Dokumentationsberichte zur Brownfield-Dokumentationswiederherstellung hinzu und integriert sie in die Website. Die Dokumentations-Rendering-Pipeline, Prerendering-Konfiguration und SPA-Routing werden aktualisiert, um die neuen Routen zu unterstützen. Eine bestehende Workflow-Dokumentation wird mit internen Ankern aktualisiert. ChangesBrownfield-Dokumentation und Routing-Integration
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related 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 |
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
Brings fork PR #27 to upstream.
The Brownfield Workflow page used to link the two experiment reports out to a
personalAssistantrepo. Both are now hosted on the site itself with their own routes:/brownfield-experiment-report— Experiment 1a methodology and findings (subject project: Bausteinsicht), arc42 chapter-by-chapter assessment, Brownfield Preparation Checklist, prompt v2./brownfield-fair-comparison— Direct vs. Socratic vs. Two-Phase reverse-engineering with identical team answers; measures the structural value of the Question Tree.Both routes are wired up end-to-end (page handlers, fallback content loader, route titles,
render-docs.jsAsciiDoc-to-HTML,prerender-routes.jsSEO prerender — 13 routes total, was 11). The brownfield-workflow page now links to them locally.Reference
Fork PR: raifdmueller#27
Summary by CodeRabbit
Dokumentation
Chores