refactor(aidd-dev): lean the review skill for small models
Les mainteneurs répondent en général sous 1 jour
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 48/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- typescript
- Domaine
- developer-experience
Piste de recherche
Read the existing aidd-dev:05-review skill, then compare the brainstorm and plan skills and issues #927, #896, and #985 for the contracts this review must preserve. Check the router with opencode using mistral-small-2603 and mistral-medium-2604, and run the repository’s pre-commit gates. Done means the skill has fewer words, preserves the stated review rules, and stays read-only; the payload does not name the skill’s file path or specific gate commands.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Problem
aidd-dev:05-review is 10 files and 1589 words: five actions (prepare, review-code, review-functional, review-relevancy, finalize), the largest of the core development skills. Under opencode with Mistral small and medium models, each extra action and reference re-sends the whole context. The lean plan skill also changes what a review can trace: an Impact section with documentation impact (#927) and a safety-net baseline (#896).
Scope
- Same method as the brainstorm and plan skills: mermaid router with action paths, recipe steps, one
Outputper action, rules only where the action owns them, no duplicate between actions, references and the report template. - Carry the plan-side contract: verify the declared documentation impact was addressed (#927), check the safety-net baseline is still green and flag any modified or deleted baseline test (#896).
- Keep the
reviewedstatus rule that now lives in the plan template: only the review sets it, and a rejected review leaves the planimplemented. - Check the router on opencode with
mistral-small-2603andmistral-medium-2604.
Acceptance criteria
- The pre-commit gates pass, with fewer words than today.
- On the same diffs, Claude
haikuandsonnetandmistral-medium-2604report the same findings as the original or more, at lower or equal cost. - The review stays read-only: no edit to the reviewed code.
Prior art in this repo
- The brainstorm skill issue #1071 and the plan skill issue.
- #927, #896, #985.
Out of scope
plan,implement,assert: separate issues.- Auditing a whole codebase (
04-audit).
- Langage dominant
- TypeScript
- Étoiles
- 481
- Forks
- 45
- Merge moyen
- 1 j 2 h
- PR mergées (30 j)
- 77
Préparer son environnement
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ai-driven-dev/framework
-
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
ai-driven-dev/framework#1002 ·
Les mainteneurs répondent en général sous 1 jour
-
feat(aidd-dev/test): enforce test effectiveness and test-double disciplinePeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
ai-driven-dev/framework#952 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
refactor(aidd-orchestrator): the check zone says when to stop, and reviews its axes in one roundOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
ai-driven-dev/framework#887 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 4/5 1-2 jours Accessibilité débutants 64/100
ai-driven-dev/framework#1084 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
ai-driven-dev/framework#1081 ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de ai-driven-dev/framework
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 68/100
awesome-dsh-plugin/awesome-dsh-plugin#9184 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
farion1231/cc-switch#8829 ·
Les mainteneurs répondent en général sous 1 jour
-
[core-client] Remove unused hand-maintained transport helpers after the definition-table cutoverOuvertedoc-audit enhancement
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100