feat: Current Status for remaining 20 anchors, LASR rewrite — triage complete (batch 4 of #603)#607
Conversation
…LM-Coding#603) Completes the LLM-Coding#603 triage. Fourteen versioned/living anchors and five thin-prior anchors get Current Status sections (EN + DE) with fetch-verified sources; the About page states the lexicon positioning (inclusion means precision, not endorsement). LASR is rewritten from scratch: verification revealed the previous entry was a silent substitution on a thin prior — it described a fabricated "lightweight architecture documentation template" with an invented acronym expansion (the AgentSkill catalog carried a second, different invented expansion: "Layers-Aspects-Solution Strategy-Rationale"). The real LASR is the Lightweight Approach for Software Reviews (Stefan Toth & Stefan Zörner, lasr-reviews.org; "Reviewing Software Systems", Leanpub 2023), a streamlined ATAM alternative with four steps (Lean Mission Statement, Evaluation Criteria, Risk-based Review, Quality-focused Analysis). The corrected anchor documents its own failure case as evidence for the thin-prior warning. The catalog.md expansion is fixed accordingly. Other notable verified facts: arc42 v9.0 (July 2025), GitHub Flow's deployment essence is gone from GitHub's current docs, LINDDUN is now a GO/PRO/MAESTRO family, Papers with Code no longer exists, IEC 61508 Ed 3 forecast ~Oct 2026, IOSP's canonical source is Westphal's Substack (clean-code-developer.de has no IOSP page). Refs LLM-Coding#603 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
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 (2)
✅ Files skipped from review due to trivial changes (2)
WalkthroughDie PR ergänzt in zahlreichen Anchor‑Dokumenten EN/DE jeweils eine "Current Status" Sektion, restrukturiert den LASR‑Anchor zu einer Software‑Review‑Methode (inkl. Katalog‑Updates) und fügt eine Präzisierung im About‑Lexikon hinzu. ChangesKatalog‑Prinzip und LASR‑Restrukturierung
Umfassende Current‑Status‑Dokumentation (EN & DE, Batch 4)
Dokumentation des Batches
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related issues
Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 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 |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In
`@plugins/semantic-anchors/skills/semantic-anchor-translator/references/catalog.md`:
- Around line 164-167: Update the "Core" field for the LASR entry (heading "LASR
by Toth/Zörner") in both catalog files to replace the fabricated "Lightweight
architecture description framework" text with the correct LASR method
description: state that LASR is a structured review method featuring a Lean
Mission Statement, Evaluation Criteria, Risk-based Review, Quality-focused
Analysis, LASR Result Diagram, and Workshop-format delivery; ensure the "Also
known as" line remains "Lightweight Approach for Software Reviews" and that the
corrected "Core" text appears in both
plugins/semantic-anchors/skills/semantic-anchor-translator/references/catalog.md
and skill/semantic-anchor-translator/references/catalog.md so agents consuming
references/catalog.md get the accurate LASR definition.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yml
Review profile: CHILL
Plan: Pro
Run ID: 5cd557bd-549c-4710-b04e-9268a921fda6
📒 Files selected for processing (43)
docs/about.adocdocs/about.de.adocdocs/anchors/arc42.adocdocs/anchors/arc42.de.adocdocs/anchors/bem-methodology.adocdocs/anchors/bem-methodology.de.adocdocs/anchors/c4-diagrams.adocdocs/anchors/c4-diagrams.de.adocdocs/anchors/chain-of-thought.adocdocs/anchors/chain-of-thought.de.adocdocs/anchors/crc-cards.adocdocs/anchors/crc-cards.de.adocdocs/anchors/diataxis-framework.adocdocs/anchors/diataxis-framework.de.adocdocs/anchors/fagan-inspection.adocdocs/anchors/fagan-inspection.de.adocdocs/anchors/fowler-patterns.adocdocs/anchors/fowler-patterns.de.adocdocs/anchors/github-flow.adocdocs/anchors/github-flow.de.adocdocs/anchors/gom.adocdocs/anchors/gom.de.adocdocs/anchors/iec-61508-sil-levels.adocdocs/anchors/iec-61508-sil-levels.de.adocdocs/anchors/iosp.adocdocs/anchors/iosp.de.adocdocs/anchors/lasr.adocdocs/anchors/lasr.de.adocdocs/anchors/linddun.adocdocs/anchors/linddun.de.adocdocs/anchors/llm-evaluations.adocdocs/anchors/llm-evaluations.de.adocdocs/anchors/meaningful-human-control.adocdocs/anchors/meaningful-human-control.de.adocdocs/anchors/site-reliability-engineering.adocdocs/anchors/site-reliability-engineering.de.adocdocs/anchors/sota.adocdocs/anchors/sota.de.adocdocs/anchors/todotxt-flavoured-markdown.adocdocs/anchors/todotxt-flavoured-markdown.de.adocdocs/changelog.adocplugins/semantic-anchors/skills/semantic-anchor-translator/references/catalog.mdskill/semantic-anchor-translator/references/catalog.md
The previous commit fixed the "Also known as" line but left the Core field describing the invented "lightweight architecture description framework" with the fabricated Layers/Aspects/Solution Strategy/Rationale expansion. The Core field now describes the real review method (Lean Mission Statement, Evaluation Criteria, Risk-based Review, Quality-focused Analysis, Result Diagram, workshop format). Found by CodeRabbit review. Refs LLM-Coding#603 Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Summary
Final batch of the #603 triage. With this PR, every flagged anchor from the full-catalog triage carries its sourced section: 23 Criticism + 26 Current Status (incl. thin-prior notes), all EN + DE, all sources fetch-verified by parallel research agents.
The headline finding: LASR was a silent substitution — in our own catalog
Verification revealed the LASR anchor was fabricated: it described a "lightweight architecture documentation template" with an invented acronym expansion (Lösungsstrategie–Architekturüberblick–Schnittstellen–Risiken), and the AgentSkill catalog carried a second, different invented expansion (Layers–Aspects–Solution Strategy–Rationale). The real LASR is the Lightweight Approach for Software Reviews (Stefan Toth & Stefan Zörner, lasr-reviews.org; Reviewing Software Systems, Leanpub 2023) — a streamlined ATAM alternative with four steps (Lean Mission Statement, Evaluation Criteria, Risk-based Review, Quality-focused Analysis).
This is exactly the Use-Case-3.0 failure mode the training-data article describes — silent substitution on a thin prior — occurring during this catalog's own creation, twice, with two different hallucinations. The rewritten anchor documents its own failure case as evidence for the thin-prior warning. catalog.md is fixed too.
Current Status sections added (14 versioned/living)
arc42c4-diagramsdiataxis-frameworkgithub-flowfowler-patternsbem-methodologyiec-61508-sil-levelslinddunsite-reliability-engineeringsotachain-of-thoughtllm-evaluationsfagan-inspectioncrc-cardsThin-prior notes (5) + corrections
gom(German-only academic, no DOI — dblp + Uni Münster records),iosp(correction: canonical source is Westphal's Substack; clean-code-developer.de has no IOSP page),meaningful-human-control(single Frontiers 2018 paper),todotxt-flavoured-markdown(doubly niche),lasr(rewritten, see above). Each advises supplying the definition in the prompt.docs-as-codeneeded no action (the "according to" qualifier already pins it).Also
Verification
node scripts/render-docs.js: all 317 fragments render clean; sections confirmed in EN + DE HTML outputCloses the work plan of #603 (the issue can be closed after merge — future anchors are covered by the contribution convention shipped in #605).
🤖 Generated with Claude Code
Summary by CodeRabbit