chore: Agent-Skills-Konfiguration unter docs/agents ablegen - #47
Conversation
Die Engineering-Skills erwarten pro Repo zwei Angaben: wo Issues leben und wo die Domänendokumentation liegt. Beides steht jetzt in docs/agents/ statt im Kopf des jeweiligen Agenten. Der ADR-Pfad weicht vom Standard der Skills ab: dieses Repo legt seine Entscheidungen unter docs/decisions/ ab, nicht unter docs/adr/. Das Template wurde entsprechend angepasst, statt die bestehenden Links in CLAUDE.md und ADR-0005 zu brechen. Triage-Labels fehlen bewusst — der triage-Skill ist hier nicht installiert. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
Warning Review limit reached
Next review available in: 51 minutes You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughDie Änderungen ergänzen Dokumentation für Agent-Skills. Sie beschreiben Domänenkontext, ADRs, GitHub-Issues, Pull Requests und ChangesAgent-Skills
Estimated code review effort: 1 (Trivial) | ~5 Minuten 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ 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 |
Der triage-Skill ist doch installiert — er trägt nur disable-model-invocation: true und taucht deshalb nicht in der Skill-Liste des Modells auf. Damit gehört das Label-Mapping dazu. Statt der englischen Standardnamen die vier fehlenden Rollen im vorhandenen Vokabular des Repos: status: triage, status: info-fehlt, status: agent-ready, status: mensch. wontfix gab es bereits. Die vier Labels sind auf GitHub angelegt. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d0d360d770
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - **Map**: a single issue labelled `wayfinder:map`, holding the Notes / Decisions-so-far / Fog body. `gh issue create --label wayfinder:map`. | ||
| - **Child ticket**: an issue linked to the map as a GitHub sub-issue (`gh api` on the sub-issues endpoint). Where sub-issues aren't enabled, add the child to a task list in the map body and put `Part of #<map>` at the top of the child body. Labels: `wayfinder:<type>` (`research`/`prototype`/`grilling`/`task`). Once claimed, the ticket is assigned to the driving dev. | ||
| - **Blocking**: GitHub's **native issue dependencies** — the canonical, UI-visible representation. Add an edge with `gh api --method POST repos/<owner>/<repo>/issues/<child>/dependencies/blocked_by -F issue_id=<blocker-db-id>`, where `<blocker-db-id>` is the blocker's numeric **database id** (`gh api repos/<owner>/<repo>/issues/<n> --jq .id`, _not_ the `#number` or `node_id`). GitHub reports `issue_dependencies_summary.blocked_by` (open blockers only — the live gate). Where dependencies aren't available, fall back to a `Blocked by: #<n>, #<n>` line at the top of the child body. A ticket is unblocked when every blocker is closed. | ||
| - **Frontier query**: list the map's open children (`gh issue list --state open`, scoped to the map's sub-issues / task list), drop any with an open blocker (`issue_dependencies_summary.blocked_by > 0`, or an open issue in the `Blocked by` line) or an assignee; first in map order wins. |
There was a problem hiding this comment.
For maps with native issue dependencies, this query names a field that the GitHub CLI does not expose: I checked gh issue list --help/field validation, and the available JSON fields are blockedBy/blocking rather than issue_dependencies_summary, which is rejected as unknown. A wayfinder run following this would not reliably drop blocked child tickets and can claim work that is still blocked; switch the frontier check to the supported dependency fields or to the REST dependency endpoints.
Useful? React with 👍 / 👎.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 5
🤖 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 `@CLAUDE.md`:
- Around line 100-111: Vor dem Commit das Pflicht-Gate ausführen: bun run
typecheck, bun run lint, bun run test und bun run build in dieser Reihenfolge
erfolgreich abschließen. Keine Markdown-Ausnahme und keinen Auto-Fix-Loop
verwenden.
- Around line 109-110: Aktualisiere den Single-context-Hinweis in CLAUDE.md:
Verweise auf CONTEXT.md im Wurzelverzeichnis nur bedingt, falls die Datei
vorhanden ist, oder entferne den Zusatz „(noch nicht angelegt)“. Behalte den
bestehenden Hinweis auf docs/decisions/ für ADRs unverändert.
In `@docs/agents/domain.md`:
- Around line 20-27: Update the repository tree example in the documentation
around the ADR entries so it no longer presents only ADR-0001 and ADR-0002 as
the complete inventory. Either remove the concrete ADR file list or explicitly
label it as an incomplete example, while preserving the later reference to
ADR-0003 and the remaining structure.
In `@docs/agents/issue-tracker.md`:
- Around line 41-42: Update the Map and Child ticket instructions in
issue-tracker.md to document that wayfinder:map and wayfinder:<type> labels may
be unavailable, and provide a concrete fallback that creates or updates issues
without those labels while preserving the map/child relationship and assignment
steps.
- Line 44: Aktualisiere die Frontier-Query-Dokumentation: Verwende bei gh issue
list das JSON-Feld blockedBy statt issue_dependencies_summary.blocked_by und
behandle einen nicht leeren blockedBy-Block als offenen Blocker. Ergänze
außerdem ein explizites --limit oder beschreibe Pagination, damit nicht nur das
Standardlimit von 30 Issues berücksichtigt wird.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 791e4694-f0ea-4574-b3ee-5c6848e07c15
📒 Files selected for processing (3)
CLAUDE.mddocs/agents/domain.mddocs/agents/issue-tracker.md
| ## Agent skills | ||
|
|
||
| ### Issue tracker | ||
|
|
||
| Issues leben als GitHub Issues in `arn0ld87/sitzplan-studio`, bedient über die | ||
| `gh`-CLI. Siehe [`docs/agents/issue-tracker.md`](docs/agents/issue-tracker.md). | ||
|
|
||
| ### Domain docs | ||
|
|
||
| Single-context: `CONTEXT.md` im Wurzelverzeichnis (noch nicht angelegt), ADRs | ||
| unter [`docs/decisions/`](docs/decisions/) — nicht unter `docs/adr/`. Siehe | ||
| [`docs/agents/domain.md`](docs/agents/domain.md). |
There was a problem hiding this comment.
📐 Maintainability & Code Quality | 🟠 Major | ⚡ Quick win
Das Pflicht-Gate vor dem Commit ausführen.
Die PR-Ziele bestätigen, dass bun run typecheck, bun run lint, bun run test und bun run build wegen der Markdown-Änderungen nicht ausgeführt wurden. Die Regel für **/* enthält keine Ausnahme für Markdown. Führt die vier Befehle vor dem Commit in dieser Reihenfolge erfolgreich aus. Verwendet keinen Auto-Fix-Loop.
As per coding guidelines: **/* erfordert die sequentielle Ausführung aller vier Befehle ohne Auto-Fix-Loop.
🤖 Prompt for 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.
In `@CLAUDE.md` around lines 100 - 111, Vor dem Commit das Pflicht-Gate ausführen:
bun run typecheck, bun run lint, bun run test und bun run build in dieser
Reihenfolge erfolgreich abschließen. Keine Markdown-Ausnahme und keinen
Auto-Fix-Loop verwenden.
Source: Coding guidelines
Codex und CodeRabbit haben unabhängig dieselbe Stelle getroffen: die Wayfinder-Frontier-Query nannte issue_dependencies_summary.blocked_by als JSON-Feld von gh issue list. Das Feld gibt es dort nicht — gh 2.96.0 kennt blockedBy. Das Summary-Objekt stammt aus der REST-API und ist jetzt als solches gekennzeichnet. Dazu ein explizites --limit, weil gh sonst nach 30 Issues abschneidet und die Frontier still unvollständig bleibt. Außerdem: Hinweis, dass die wayfinder:*-Labels hier nicht existieren und gh sie nicht von selbst anlegt; ADR-Baum nennt keine zwei Dateien mehr, wo sieben liegen; der Status von CONTEXT.md wird nicht mehr festgeschrieben. Das Pre-Push-Gate wurde auf Anweisung übersprungen — reine Markdown-Änderung. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 participant
Worum es geht
Die Engineering-Skills (
to-tickets,to-spec,triage,code-review,domain-modeling,wayfinder) erwarten pro Repo drei Angaben: wo Issues leben, wie die Triage-Labels heißen und wo die Domänendokumentation liegt. Bisher musste das jeder Agent raten. Jetzt steht es indocs/agents/.Was drin ist
docs/agents/issue-tracker.md— GitHub Issues in diesem Repo, bedient über diegh-CLI. Das Flag „PRs als Anfrage-Surface" steht aufno.docs/agents/triage-labels.md— die fünf kanonischen Triage-Rollen, abgebildet auf das bestehende deutsche Präfix-Schema statt auf englische Duplikate:needs-triagestatus: triageneeds-infostatus: info-fehltready-for-agentstatus: agent-readyready-for-humanstatus: menschwontfixwontfix(existierte bereits)Die vier fehlenden Labels sind im Repo angelegt, passend zu
status: blockiert.docs/agents/domain.md— single-context. Der ADR-Pfad weicht vom Standard der Skills ab: dieses Repo legt seine Entscheidungen unterdocs/decisions/ab, nicht unterdocs/adr/. Das Template wurde angepasst, statt die bestehenden Links inCLAUDE.mdund ADR-0005 zu brechen.CLAUDE.md— neuer Abschnitt## Agent skillsmit Verweisen auf alle drei Dateien.Was bewusst fehlt
CONTEXT.md. Das legt derdomain-modeling-Skill an, sobald das erste Mal wirklich Begriffe geklärt werden — nicht auf Vorrat.Prüfung
Reine Markdown-Änderung, kein Quellcode berührt. Das Pre-Push-Gate (
typecheck/lint/test/build) wurde daher nicht ausgeführt. Die angelegten Labels sind mitgh label listverifiziert.🤖 Generated with Claude Code