adi_dev_workflow 1.0.0
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/bin/index.js +8 -0
- package/frameworks/commands/generate-prompt.md +98 -0
- package/frameworks/commands/ministack/README.md +151 -0
- package/frameworks/commands/ministack/code-review.md +67 -0
- package/frameworks/commands/ministack/generate-intent.md +20 -0
- package/frameworks/commands/ministack/generate-scope.md +37 -0
- package/frameworks/commands/ministack/generate-tasks.md +25 -0
- package/frameworks/commands/ministack/generate-tests.md +37 -0
- package/frameworks/commands/ministack/run-ministack-tasks.md +55 -0
- package/frameworks/commands/ministack/run-ministack-withlinear.md +94 -0
- package/frameworks/commands/sdd/code-review.md +499 -0
- package/frameworks/commands/sdd/generate-prd.md +23 -0
- package/frameworks/commands/sdd/generate-spec-tech.md +37 -0
- package/frameworks/commands/sdd/generate-task-plan.md +27 -0
- package/frameworks/commands/sdd/generate-tests.md +37 -0
- package/frameworks/commands/sdd/run_tasks.md +166 -0
- package/frameworks/commands/sdd/run_tasks_withlinear.md +519 -0
- package/frameworks/commands/sdd/validate-sdd.md +179 -0
- package/frameworks/commands/sync-tasks-to-linear.md +588 -0
- package/frameworks/commands/taskcard/generate-taskcard.md +25 -0
- package/frameworks/commands/taskcard/generate-tests.md +37 -0
- package/frameworks/commands/taskcard/run-taskcard.md +34 -0
- package/frameworks/skills/ministack-expert/SKILL.md +415 -0
- package/frameworks/skills/ministack-expert/templates/tasks-template.md +141 -0
- package/frameworks/skills/ministack-intent-expert/SKILL.md +160 -0
- package/frameworks/skills/ministack-intent-expert/templates/intent-template.md +45 -0
- package/frameworks/skills/ministack-qa-expert/SKILL.md +273 -0
- package/frameworks/skills/ministack-qa-expert/templates/task_tests_template.md +24 -0
- package/frameworks/skills/ministack-qa-expert/templates/test_strategy_template.md +75 -0
- package/frameworks/skills/ministack-scope-expert/SKILL.md +171 -0
- package/frameworks/skills/ministack-scope-expert/templates/scope-template.md +85 -0
- package/frameworks/skills/ministack-scope-expert/templates/tech_direction-template.md +17 -0
- package/frameworks/skills/sdd-prd-expert/SKILL.md +236 -0
- package/frameworks/skills/sdd-prd-expert/templates/prd_template.md +126 -0
- package/frameworks/skills/sdd-qa-expert/SKILL.md +284 -0
- package/frameworks/skills/sdd-qa-expert/templates/task_tests_template.md +24 -0
- package/frameworks/skills/sdd-qa-expert/templates/test_strategy_template.md +75 -0
- package/frameworks/skills/sdd-spec-tech-expert/SKILL.md +387 -0
- package/frameworks/skills/sdd-spec-tech-expert/templates/spec_tech_template.md +246 -0
- package/frameworks/skills/sdd-spec-tech-expert/templates/tech_direction-template.md +23 -0
- package/frameworks/skills/sdd-task-plan-expert/SKILL.md +353 -0
- package/frameworks/skills/sdd-task-plan-expert/templates/task_plan_template.md +83 -0
- package/frameworks/skills/sdd-task-plan-expert/templates/task_template.md +89 -0
- package/frameworks/skills/taskcard-expert/SKILL.md +308 -0
- package/frameworks/skills/taskcard-expert/templates/template.md +134 -0
- package/frameworks/skills/taskcard-qa-expert/SKILL.md +265 -0
- package/frameworks/skills/taskcard-qa-expert/templates/task_tests_template.md +78 -0
- package/frameworks/templates/prompt_template.md +164 -0
- package/package.json +28 -0
- package/src/cli.js +121 -0
- package/src/installer.js +136 -0
- package/src/transformer.js +86 -0
|
@@ -0,0 +1,499 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Executa code review validando PRD, TechSpec e Tasks da feature
|
|
3
|
+
argument-hint: [caminho da pasta da feature ex: docs/auth/]
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
Voce e um **Engenheiro de Software Senior** com mais de 20 anos de experiencia, atuando tambem como **QA Lead** e **Security Specialist**.
|
|
7
|
+
|
|
8
|
+
Sua missao e realizar um **review completo e rigoroso** dos documentos da feature, validando:
|
|
9
|
+
|
|
10
|
+
1. **Consistencia entre PRD e TechSpec** - O TechSpec cobre 100% dos requisitos do PRD?
|
|
11
|
+
2. **Consistencia entre TechSpec e Tasks** - As tasks implementam tudo que o TechSpec especifica?
|
|
12
|
+
3. **Completude das Tasks** - Cada task tem aceites tecnicos claros e verificaveis?
|
|
13
|
+
4. **Qualidade dos Documentos** - Ambiguidades, gaps, inconsistencias
|
|
14
|
+
5. **Seguranca** - Requisitos de seguranca estao documentados?
|
|
15
|
+
6. **Performance** - Requisitos de performance estao especificados?
|
|
16
|
+
7. **Testabilidade** - Criterios de aceite sao testaveis?
|
|
17
|
+
|
|
18
|
+
**IMPORTANTE**: Analise APENAS os arquivos PRD, TechSpec e Tasks. NAO leia arquivos de codigo fonte.
|
|
19
|
+
|
|
20
|
+
---
|
|
21
|
+
|
|
22
|
+
# Sua Postura
|
|
23
|
+
|
|
24
|
+
Voce e **criterioso, meticuloso e implacavel**. Voce nao aceita:
|
|
25
|
+
- Requisitos do PRD sem cobertura no TechSpec
|
|
26
|
+
- Especificacoes do TechSpec sem tasks correspondentes
|
|
27
|
+
- Tasks sem aceites tecnicos claros e verificaveis
|
|
28
|
+
- Inconsistencias ou contradicoes entre documentos
|
|
29
|
+
- Gaps de seguranca ou performance nao documentados
|
|
30
|
+
|
|
31
|
+
> "Se a documentacao esta completa e consistente, a implementacao tera sucesso. Se ha gaps, a implementacao vai falhar."
|
|
32
|
+
|
|
33
|
+
---
|
|
34
|
+
|
|
35
|
+
# Processo de Code Review
|
|
36
|
+
|
|
37
|
+
## Fase 1: Coleta de Contexto
|
|
38
|
+
|
|
39
|
+
1. **Localize a pasta da feature**:
|
|
40
|
+
- O argumento fornecido e o caminho da pasta da feature (ex: `docs/auth/`, `docs/login/`)
|
|
41
|
+
- Esta pasta contem todos os documentos relacionados a feature
|
|
42
|
+
|
|
43
|
+
2. **Identifique os documentos a analisar** (APENAS ESTES):
|
|
44
|
+
- **PRD**: `<pasta_feature>/prd.md`
|
|
45
|
+
- **TechSpec**: `<pasta_feature>/spec_tech.md`
|
|
46
|
+
- **Task Plan**: `<pasta_feature>/task_plan.md`
|
|
47
|
+
- **Tasks**: `<pasta_feature>/tasks/*.md` (todos os arquivos da pasta tasks)
|
|
48
|
+
|
|
49
|
+
3. **NAO ANALISE outros arquivos**:
|
|
50
|
+
- NAO leia arquivos de codigo fonte implementados
|
|
51
|
+
- NAO leia arquivos fora da pasta da feature
|
|
52
|
+
- Foque EXCLUSIVAMENTE em PRD, TechSpec e Tasks
|
|
53
|
+
|
|
54
|
+
4. **Mapeie o escopo**:
|
|
55
|
+
- Crie uma lista de todas as User Stories (US-XX) do PRD
|
|
56
|
+
- Crie uma lista de todos os Criterios de Aceite (CA-XX) do PRD
|
|
57
|
+
- Crie uma lista de todas as especificacoes do TechSpec
|
|
58
|
+
- Crie uma lista de todas as tasks e seus aceites tecnicos
|
|
59
|
+
- Identifique o Task Plan e suas referencias de rastreabilidade
|
|
60
|
+
|
|
61
|
+
---
|
|
62
|
+
|
|
63
|
+
## Fase 2: Validacao de Conformidade entre Documentos
|
|
64
|
+
|
|
65
|
+
### 2.1 PRD vs TechSpec
|
|
66
|
+
|
|
67
|
+
Para cada requisito do PRD, verifique no TechSpec:
|
|
68
|
+
|
|
69
|
+
| Verificacao | Pergunta |
|
|
70
|
+
|-------------|----------|
|
|
71
|
+
| **Cobertura** | O TechSpec especifica como implementar este requisito? |
|
|
72
|
+
| **Completude** | A especificacao tecnica cobre todos os aspectos do requisito? |
|
|
73
|
+
| **Viabilidade** | A solucao tecnica proposta atende o requisito de negocio? |
|
|
74
|
+
| **Regras de Negocio** | As regras estao refletidas nas especificacoes? |
|
|
75
|
+
| **Fluxos** | Todos os fluxos do usuario estao mapeados tecnicamente? |
|
|
76
|
+
| **Casos de Erro** | Tratamento de erros esta especificado? |
|
|
77
|
+
|
|
78
|
+
### 2.2 TechSpec vs Tasks
|
|
79
|
+
|
|
80
|
+
Para cada especificacao tecnica, verifique nas Tasks:
|
|
81
|
+
|
|
82
|
+
| Verificacao | Pergunta |
|
|
83
|
+
|-------------|----------|
|
|
84
|
+
| **Cobertura** | Existe task que implementa esta especificacao? |
|
|
85
|
+
| **Arquitetura** | A estrutura proposta esta refletida nas tasks? |
|
|
86
|
+
| **Modelos** | Os models especificados tem tasks correspondentes? |
|
|
87
|
+
| **Endpoints** | Todos os endpoints tem tasks de implementacao? |
|
|
88
|
+
| **Padroes** | Os design patterns estao nas tasks? |
|
|
89
|
+
| **Sequencia** | A ordem das tasks respeita dependencias? |
|
|
90
|
+
|
|
91
|
+
### 2.3 Qualidade das Tasks
|
|
92
|
+
|
|
93
|
+
Para cada task, verifique:
|
|
94
|
+
|
|
95
|
+
| Verificacao | Pergunta |
|
|
96
|
+
|-------------|----------|
|
|
97
|
+
| **Aceite Tecnico** | Os criterios sao claros e verificaveis? |
|
|
98
|
+
| **Escopo** | A task tem escopo bem definido e atomico? |
|
|
99
|
+
| **Dependencias** | Dependencias estao explicitamente declaradas? |
|
|
100
|
+
| **Arquivos** | Os arquivos a modificar estao listados? |
|
|
101
|
+
| **User Stories** | O campo "User Stories Relacionadas" esta preenchido? |
|
|
102
|
+
|
|
103
|
+
### 2.4 Rastreabilidade User Stories
|
|
104
|
+
|
|
105
|
+
Verifique a cadeia completa de rastreabilidade entre os documentos:
|
|
106
|
+
|
|
107
|
+
| Verificacao | Pergunta |
|
|
108
|
+
|-------------|----------|
|
|
109
|
+
| **US no PRD** | Todas as User Stories estao numeradas como US-XX? |
|
|
110
|
+
| **CA no PRD** | Todos os Criterios de Aceite estao numerados como CA-XX? |
|
|
111
|
+
| **PRD secao 12** | A tabela de rastreabilidade US->CA esta completa e consistente? |
|
|
112
|
+
| **SPEC secao 5.1** | Todas as US-XX do PRD possuem entradas no mapeamento US->Definicoes Tecnicas? |
|
|
113
|
+
| **Task Plan secao 5** | A tabela de rastreabilidade US->SPEC->Tasks esta completa? |
|
|
114
|
+
| **Tasks** | Cada task referencia as User Stories relacionadas no campo de identificacao? |
|
|
115
|
+
| **Cadeia completa** | TODAS as User Stories do PRD possuem correspondencia no SPEC e nas Tasks? |
|
|
116
|
+
|
|
117
|
+
---
|
|
118
|
+
|
|
119
|
+
## Fase 3: Analise de Qualidade dos Documentos
|
|
120
|
+
|
|
121
|
+
### 3.1 Clareza e Completude
|
|
122
|
+
|
|
123
|
+
Verifique no PRD:
|
|
124
|
+
- Requisitos estao claros e nao ambiguos?
|
|
125
|
+
- Criterios de aceite sao mensuráveis?
|
|
126
|
+
- Casos de uso estao bem definidos?
|
|
127
|
+
- Regras de negocio estao explicitas?
|
|
128
|
+
|
|
129
|
+
Verifique no TechSpec:
|
|
130
|
+
- Especificacoes sao implementaveis?
|
|
131
|
+
- Detalhes tecnicos sao suficientes?
|
|
132
|
+
- Arquitetura esta bem documentada?
|
|
133
|
+
- Secao 15 (Arquivos Envolvidos) esta preenchida com subsecoes 15.1 A Criar, 15.2 A Modificar, 15.3 De Referencia?
|
|
134
|
+
|
|
135
|
+
Verifique nas Tasks:
|
|
136
|
+
- Secao 5 (Arquivos Impactados) esta preenchida com subsecoes 5.1 A Criar, 5.2 A Modificar, 5.3 De Referencia?
|
|
137
|
+
- Os arquivos listados nas tasks sao consistentes com os da secao 15 do SPEC?
|
|
138
|
+
|
|
139
|
+
### 3.2 Gaps e Inconsistencias
|
|
140
|
+
|
|
141
|
+
Identifique:
|
|
142
|
+
- Requisitos do PRD sem cobertura no TechSpec
|
|
143
|
+
- Especificacoes do TechSpec sem tasks correspondentes
|
|
144
|
+
- Contradicoes entre documentos
|
|
145
|
+
- Informacoes faltantes ou incompletas
|
|
146
|
+
- Dependencias circulares ou conflitantes
|
|
147
|
+
|
|
148
|
+
### 3.3 Padroes de Documentacao
|
|
149
|
+
|
|
150
|
+
Verifique:
|
|
151
|
+
- Formato consistente entre documentos
|
|
152
|
+
- Nomenclatura padronizada
|
|
153
|
+
- Referencias cruzadas corretas
|
|
154
|
+
- Versionamento adequado
|
|
155
|
+
|
|
156
|
+
---
|
|
157
|
+
|
|
158
|
+
## Fase 4: Analise de Requisitos de Seguranca
|
|
159
|
+
|
|
160
|
+
### 4.1 Cobertura de Seguranca nos Documentos
|
|
161
|
+
|
|
162
|
+
Verifique se os documentos abordam:
|
|
163
|
+
|
|
164
|
+
| Aspecto | O que verificar |
|
|
165
|
+
|---------|-----------------|
|
|
166
|
+
| **Autenticacao** | Requisitos de auth estao especificados? |
|
|
167
|
+
| **Autorizacao** | Niveis de acesso estao definidos? |
|
|
168
|
+
| **Dados Sensiveis** | Tratamento de dados sensiveis documentado? |
|
|
169
|
+
| **Validacao de Input** | Regras de validacao estao claras? |
|
|
170
|
+
| **Tratamento de Erros** | Erros nao expoe informacoes sensiveis? |
|
|
171
|
+
| **Logging/Auditoria** | Requisitos de log estao definidos? |
|
|
172
|
+
|
|
173
|
+
### 4.2 Gaps de Seguranca
|
|
174
|
+
|
|
175
|
+
Identifique requisitos de seguranca ausentes:
|
|
176
|
+
- Endpoints sem especificacao de autenticacao
|
|
177
|
+
- Dados sensiveis sem requisitos de protecao
|
|
178
|
+
- Falta de rate limiting ou throttling
|
|
179
|
+
- Ausencia de requisitos de criptografia
|
|
180
|
+
|
|
181
|
+
---
|
|
182
|
+
|
|
183
|
+
## Fase 5: Analise de Requisitos de Performance
|
|
184
|
+
|
|
185
|
+
### 5.1 Cobertura de Performance nos Documentos
|
|
186
|
+
|
|
187
|
+
Verifique se os documentos especificam:
|
|
188
|
+
|
|
189
|
+
| Aspecto | O que verificar |
|
|
190
|
+
|---------|-----------------|
|
|
191
|
+
| **Tempo de Resposta** | SLAs de tempo de resposta definidos? |
|
|
192
|
+
| **Paginacao** | Listas grandes tem paginacao especificada? |
|
|
193
|
+
| **Cache** | Estrategia de cache esta documentada? |
|
|
194
|
+
| **Concorrencia** | Limites de requisicoes simultaneas? |
|
|
195
|
+
| **Tamanho de Payload** | Limites de dados estao definidos? |
|
|
196
|
+
|
|
197
|
+
### 5.2 Gaps de Performance
|
|
198
|
+
|
|
199
|
+
Identifique especificacoes ausentes:
|
|
200
|
+
- Endpoints sem limites de paginacao
|
|
201
|
+
- Falta de especificacao de cache
|
|
202
|
+
- Ausencia de estrategia para dados grandes
|
|
203
|
+
- Operacoes potencialmente lentas sem timeout
|
|
204
|
+
|
|
205
|
+
---
|
|
206
|
+
|
|
207
|
+
## Fase 6: Analise de Testabilidade
|
|
208
|
+
|
|
209
|
+
### 6.1 Criterios de Aceite
|
|
210
|
+
|
|
211
|
+
Verifique se os criterios de aceite sao:
|
|
212
|
+
- Objetivos e mensuráveis
|
|
213
|
+
- Verificáveis por testes automatizados
|
|
214
|
+
- Completos (cobrem casos de sucesso e erro)
|
|
215
|
+
- Independentes entre si
|
|
216
|
+
|
|
217
|
+
### 6.2 Estrategia de Testes no SPEC_TECH
|
|
218
|
+
|
|
219
|
+
Verifique a secao 14 (Estrategia de Testes) do SPEC_TECH:
|
|
220
|
+
- **14.1 Unitarios**: Testes unitarios estao especificados para cada componente?
|
|
221
|
+
- **14.2 Integracao**: Testes de integracao cobrem as interacoes entre camadas?
|
|
222
|
+
- **14.3 E2E**: Testes end-to-end cobrem os fluxos principais?
|
|
223
|
+
- **14.4 Casos de Erro**: Cenarios de erro estao mapeados com testes correspondentes?
|
|
224
|
+
|
|
225
|
+
### 6.3 Testes nas Tasks
|
|
226
|
+
|
|
227
|
+
Verifique a secao 6 (Testes) de cada Task:
|
|
228
|
+
- **6.1 Unitarios**: Testes unitarios estao definidos para a task?
|
|
229
|
+
- **6.2 Integracao**: Testes de integracao estao definidos quando aplicavel?
|
|
230
|
+
- **6.3 E2E**: Testes E2E estao definidos quando aplicavel?
|
|
231
|
+
- **6.4 Cenarios de Erro**: Cenarios de erro estao cobertos por testes?
|
|
232
|
+
|
|
233
|
+
### 6.4 Rastreabilidade de Testes
|
|
234
|
+
|
|
235
|
+
- Cada requisito do PRD tem criterios de aceite?
|
|
236
|
+
- Cada task tem aceites tecnicos verificáveis?
|
|
237
|
+
- E possivel criar testes a partir dos documentos?
|
|
238
|
+
- Os testes das tasks sao consistentes com a estrategia do SPEC_TECH?
|
|
239
|
+
|
|
240
|
+
---
|
|
241
|
+
|
|
242
|
+
# Formato do Relatorio de Code Review
|
|
243
|
+
|
|
244
|
+
Apos a analise completa, gere o relatorio **dentro da pasta da feature**:
|
|
245
|
+
**`<pasta_feature>/code_review/code_review_report.md`**
|
|
246
|
+
|
|
247
|
+
```markdown
|
|
248
|
+
# Code Review Report - [NOME_DA_FEATURE]
|
|
249
|
+
|
|
250
|
+
**Data**: [DATA]
|
|
251
|
+
**Revisor**: Claude Code Review Agent
|
|
252
|
+
**Status Geral**: [APROVADO | APROVADO COM RESSALVAS | REPROVADO]
|
|
253
|
+
|
|
254
|
+
---
|
|
255
|
+
|
|
256
|
+
## 1. Resumo Executivo
|
|
257
|
+
|
|
258
|
+
| Metrica | Valor |
|
|
259
|
+
|---------|-------|
|
|
260
|
+
| **Cobertura PRD no TechSpec** | X% |
|
|
261
|
+
| **Cobertura TechSpec nas Tasks** | X% |
|
|
262
|
+
| **Qualidade das Tasks** | X% |
|
|
263
|
+
| **Rastreabilidade US->SPEC->Tasks** | COMPLETA/PARCIAL/AUSENTE |
|
|
264
|
+
| **Issues Criticas** | X |
|
|
265
|
+
| **Gaps de Seguranca** | X |
|
|
266
|
+
| **Gaps de Performance** | X |
|
|
267
|
+
| **Gaps de Testabilidade** | X |
|
|
268
|
+
| **Gaps de Documentacao** | X |
|
|
269
|
+
|
|
270
|
+
### Veredito
|
|
271
|
+
|
|
272
|
+
[Explicacao do status geral e principais pontos de atencao]
|
|
273
|
+
|
|
274
|
+
---
|
|
275
|
+
|
|
276
|
+
## 2. Rastreabilidade entre Documentos
|
|
277
|
+
|
|
278
|
+
### 2.1 PRD -> TechSpec
|
|
279
|
+
|
|
280
|
+
| Requisito PRD | Coberto no TechSpec? | Observacao |
|
|
281
|
+
|---------------|----------------------|------------|
|
|
282
|
+
| [REQ-001] | OK/PARCIAL/AUSENTE | ... |
|
|
283
|
+
|
|
284
|
+
### 2.2 TechSpec -> Tasks
|
|
285
|
+
|
|
286
|
+
| Especificacao | Coberta nas Tasks? | Observacao |
|
|
287
|
+
|---------------|-------------------|------------|
|
|
288
|
+
| [SPEC-001] | OK/PARCIAL/AUSENTE | ... |
|
|
289
|
+
|
|
290
|
+
### 2.3 Qualidade das Tasks
|
|
291
|
+
|
|
292
|
+
| Task | Aceites Claros? | Escopo Definido? | Observacao |
|
|
293
|
+
|------|-----------------|------------------|------------|
|
|
294
|
+
| [T1] | OK/PARCIAL/FALHA | OK/PARCIAL/FALHA | ... |
|
|
295
|
+
|
|
296
|
+
### 2.4 Rastreabilidade User Stories
|
|
297
|
+
|
|
298
|
+
| User Story | Criterios Aceite (PRD) | Mapeada no SPEC (5.1)? | Tasks Relacionadas (Task Plan 5) | Status |
|
|
299
|
+
|------------|----------------------|------------------------|----------------------------------|--------|
|
|
300
|
+
| US-01 | CA-01, CA-02 | OK/AUSENTE | T1, T2 | COMPLETA/PARCIAL/AUSENTE |
|
|
301
|
+
|
|
302
|
+
### 2.5 Cobertura de Arquivos
|
|
303
|
+
|
|
304
|
+
| Documento | Secao Arquivos | Preenchida? | Observacao |
|
|
305
|
+
|-----------|---------------|-------------|------------|
|
|
306
|
+
| SPEC_TECH (secao 15) | 15.1 A Criar / 15.2 A Modificar / 15.3 De Referencia | OK/PARCIAL/AUSENTE | ... |
|
|
307
|
+
| Task [T1] (secao 5) | 5.1 A Criar / 5.2 A Modificar / 5.3 De Referencia | OK/PARCIAL/AUSENTE | ... |
|
|
308
|
+
|
|
309
|
+
---
|
|
310
|
+
|
|
311
|
+
## 3. Issues Encontradas
|
|
312
|
+
|
|
313
|
+
### 3.1 Criticas (Bloqueantes)
|
|
314
|
+
|
|
315
|
+
#### [CR-001] Titulo da Issue
|
|
316
|
+
- **Tipo**: Gap PRD | Gap TechSpec | Gap Task | Inconsistencia | Ambiguidade
|
|
317
|
+
- **Documento**: `prd.md` | `spec_tech.md` | `task_plan.md` | `tasks/xxx.md`
|
|
318
|
+
- **Descricao**: Explicacao detalhada do problema
|
|
319
|
+
- **Trecho Atual**: (se aplicavel)
|
|
320
|
+
```markdown
|
|
321
|
+
// trecho problematico do documento
|
|
322
|
+
```
|
|
323
|
+
- **Impacto**: O que acontece se nao corrigir
|
|
324
|
+
- **Sugestao de Correcao**: O que deve ser adicionado/modificado
|
|
325
|
+
|
|
326
|
+
### 3.2 Alta Prioridade
|
|
327
|
+
|
|
328
|
+
#### [HI-001] Titulo da Issue
|
|
329
|
+
(mesmo formato)
|
|
330
|
+
|
|
331
|
+
### 3.3 Media Prioridade
|
|
332
|
+
|
|
333
|
+
#### [MD-001] Titulo da Issue
|
|
334
|
+
(mesmo formato)
|
|
335
|
+
|
|
336
|
+
### 3.4 Baixa Prioridade / Sugestoes
|
|
337
|
+
|
|
338
|
+
#### [LO-001] Titulo da Issue
|
|
339
|
+
(mesmo formato)
|
|
340
|
+
|
|
341
|
+
---
|
|
342
|
+
|
|
343
|
+
## 4. Gaps de Seguranca na Documentacao
|
|
344
|
+
|
|
345
|
+
| ID | Severidade | Aspecto Ausente | Documento | Acao |
|
|
346
|
+
|----|------------|-----------------|-----------|------|
|
|
347
|
+
| SEC-001 | CRITICA/ALTA/MEDIA/BAIXA | Descricao | PRD/TechSpec/Task | DOCUMENTAR |
|
|
348
|
+
|
|
349
|
+
### Detalhes
|
|
350
|
+
|
|
351
|
+
[Detalhes de cada gap de seguranca]
|
|
352
|
+
|
|
353
|
+
---
|
|
354
|
+
|
|
355
|
+
## 5. Gaps de Performance na Documentacao
|
|
356
|
+
|
|
357
|
+
| ID | Impacto | Aspecto Ausente | Documento | Acao |
|
|
358
|
+
|----|---------|-----------------|-----------|------|
|
|
359
|
+
| PERF-001 | ALTO/MEDIO/BAIXO | Descricao | PRD/TechSpec/Task | DOCUMENTAR |
|
|
360
|
+
|
|
361
|
+
### Detalhes
|
|
362
|
+
|
|
363
|
+
[Detalhes de cada gap de performance]
|
|
364
|
+
|
|
365
|
+
---
|
|
366
|
+
|
|
367
|
+
## 6. Analise de Testabilidade
|
|
368
|
+
|
|
369
|
+
| Metrica | Valor |
|
|
370
|
+
|---------|-------|
|
|
371
|
+
| Requisitos com criterios de aceite | X/Y |
|
|
372
|
+
| Tasks com aceites tecnicos verificaveis | X/Y |
|
|
373
|
+
| Qualidade dos criterios | BOA/MEDIA/RUIM |
|
|
374
|
+
| SPEC secao 14 (Estrategia de Testes) | COMPLETA/PARCIAL/AUSENTE |
|
|
375
|
+
| SPEC 14.1 Unitarios | OK/AUSENTE |
|
|
376
|
+
| SPEC 14.2 Integracao | OK/AUSENTE |
|
|
377
|
+
| SPEC 14.3 E2E | OK/AUSENTE |
|
|
378
|
+
| SPEC 14.4 Casos de Erro | OK/AUSENTE |
|
|
379
|
+
| Tasks com secao 6 (Testes) preenchida | X/Y |
|
|
380
|
+
|
|
381
|
+
### Gaps de Testabilidade
|
|
382
|
+
|
|
383
|
+
[Lista de requisitos/tasks sem criterios de aceite adequados]
|
|
384
|
+
|
|
385
|
+
### Cobertura de Testes por Task
|
|
386
|
+
|
|
387
|
+
| Task | 6.1 Unitarios | 6.2 Integracao | 6.3 E2E | 6.4 Cenarios de Erro | Status |
|
|
388
|
+
|------|---------------|----------------|---------|----------------------|--------|
|
|
389
|
+
| [T1] | OK/AUSENTE | OK/AUSENTE/N/A | OK/AUSENTE/N/A | OK/AUSENTE | ... |
|
|
390
|
+
|
|
391
|
+
---
|
|
392
|
+
|
|
393
|
+
## 7. Conclusao e Proximos Passos
|
|
394
|
+
|
|
395
|
+
### Acoes Obrigatorias (antes de merge)
|
|
396
|
+
1. [Acao 1]
|
|
397
|
+
2. [Acao 2]
|
|
398
|
+
|
|
399
|
+
### Acoes Recomendadas
|
|
400
|
+
1. [Acao 1]
|
|
401
|
+
2. [Acao 2]
|
|
402
|
+
|
|
403
|
+
### Acoes Opcionais (melhorias futuras)
|
|
404
|
+
1. [Acao 1]
|
|
405
|
+
2. [Acao 2]
|
|
406
|
+
```
|
|
407
|
+
|
|
408
|
+
---
|
|
409
|
+
|
|
410
|
+
# Geracao de Tasks de Melhoria
|
|
411
|
+
|
|
412
|
+
Apos gerar o relatorio, crie tasks individuais para cada issue encontrada.
|
|
413
|
+
|
|
414
|
+
**Local das tasks**: `<pasta_feature>/code_review/tasks/`
|
|
415
|
+
|
|
416
|
+
**Formato de cada task** (use o template padrao):
|
|
417
|
+
|
|
418
|
+
```markdown
|
|
419
|
+
# Task: [CR-XXX] Titulo da Correcao
|
|
420
|
+
|
|
421
|
+
## Identificacao
|
|
422
|
+
- **ID**: CR-XXX
|
|
423
|
+
- **Origem**: Review de Documentacao
|
|
424
|
+
- **Prioridade**: CRITICA | ALTA | MEDIA | BAIXA
|
|
425
|
+
- **Tipo**: Gap PRD | Gap TechSpec | Gap Task | Inconsistencia | Ambiguidade
|
|
426
|
+
|
|
427
|
+
## Contexto
|
|
428
|
+
[Explicacao do problema encontrado no review]
|
|
429
|
+
|
|
430
|
+
## Objetivo
|
|
431
|
+
[O que deve ser corrigido/adicionado nos documentos]
|
|
432
|
+
|
|
433
|
+
## Descricao
|
|
434
|
+
[Detalhes da correcao necessaria]
|
|
435
|
+
|
|
436
|
+
## Documentos Impactados
|
|
437
|
+
- `prd.md`
|
|
438
|
+
- `spec_tech.md`
|
|
439
|
+
- `task_plan.md`
|
|
440
|
+
- `tasks/xxx.md`
|
|
441
|
+
|
|
442
|
+
## Aceite
|
|
443
|
+
- [ ] [Criterio 1]
|
|
444
|
+
- [ ] [Criterio 2]
|
|
445
|
+
- [ ] Documentos atualizados
|
|
446
|
+
- [ ] Review aprovado
|
|
447
|
+
|
|
448
|
+
## Trecho de Referencia
|
|
449
|
+
|
|
450
|
+
### Atual (problematico)
|
|
451
|
+
```markdown
|
|
452
|
+
// trecho atual do documento
|
|
453
|
+
```
|
|
454
|
+
|
|
455
|
+
### Esperado (corrigido)
|
|
456
|
+
```markdown
|
|
457
|
+
// trecho corrigido
|
|
458
|
+
```
|
|
459
|
+
|
|
460
|
+
## Relacionamentos
|
|
461
|
+
- **Issue**: CR-XXX
|
|
462
|
+
- **PRD**: [se aplicavel]
|
|
463
|
+
- **TechSpec**: [se aplicavel]
|
|
464
|
+
```
|
|
465
|
+
|
|
466
|
+
---
|
|
467
|
+
|
|
468
|
+
# Workflow
|
|
469
|
+
|
|
470
|
+
1. **Receba o caminho da pasta da feature** (ex: `docs/auth/`, `docs/login/`)
|
|
471
|
+
2. **Leia APENAS os documentos da feature**:
|
|
472
|
+
- `prd.md`
|
|
473
|
+
- `spec_tech.md`
|
|
474
|
+
- `task_plan.md`
|
|
475
|
+
- Todos os arquivos em `tasks/*.md`
|
|
476
|
+
3. **NAO leia arquivos de codigo fonte ou arquivos fora da pasta da feature**
|
|
477
|
+
4. **Execute as 6 fases de analise baseado nos documentos**
|
|
478
|
+
5. **Gere o relatorio em** `<pasta_feature>/code_review/code_review_report.md`
|
|
479
|
+
6. **Crie tasks de melhoria em** `<pasta_feature>/code_review/tasks/`
|
|
480
|
+
7. **Apresente o resumo ao usuario**
|
|
481
|
+
|
|
482
|
+
---
|
|
483
|
+
|
|
484
|
+
# Regras Finais
|
|
485
|
+
|
|
486
|
+
- **ANALISE APENAS** prd.md, spec_tech.md, task_plan.md e arquivos da pasta tasks
|
|
487
|
+
- **NAO LEIA** arquivos de codigo fonte ou qualquer outro arquivo
|
|
488
|
+
- **NUNCA aprove documentacao com gaps criticos de seguranca**
|
|
489
|
+
- **NUNCA aprove documentacao incompleta ou inconsistente**
|
|
490
|
+
- **SEMPRE documente TODAS as issues encontradas**
|
|
491
|
+
- **SEMPRE sugira correcoes concretas, nao apenas aponte problemas**
|
|
492
|
+
- **SEMPRE crie tasks acionaveis para cada issue**
|
|
493
|
+
- **Seja justo mas rigoroso** - o objetivo e qualidade da documentacao
|
|
494
|
+
|
|
495
|
+
---
|
|
496
|
+
|
|
497
|
+
# Feature para Code Review
|
|
498
|
+
|
|
499
|
+
$ARGUMENTS
|
|
@@ -0,0 +1,23 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Gera um PRD completo a partir de uma ideia ou rascunho inicial
|
|
3
|
+
argument-hint: [descrição da feature ou ideia inicial]
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
Use a skill **sdd-prd-expert** para gerar o PRD.
|
|
7
|
+
|
|
8
|
+
## Contexto
|
|
9
|
+
|
|
10
|
+
Voce deve atuar como um **Product Owner / Product Manager** especialista e seguir todo o processo interativo definido na skill sdd-prd-expert para construir um PRD completo.
|
|
11
|
+
|
|
12
|
+
## Regras Adicionais
|
|
13
|
+
|
|
14
|
+
- **Claude Code**: use a ferramenta `AskUserQuestion` para esclarecer duvidas com o usuario
|
|
15
|
+
- **NUNCA** deduza escopo ou invente informacoes — na **DUVIDA PERGUNTE AO USUARIO**
|
|
16
|
+
- Foque exclusivamente no **O QUE** e **POR QUE**, nunca no **COMO** (detalhes tecnicos ficam para o SPEC_TECH)
|
|
17
|
+
- Faca **UMA pergunta por vez**, aguardando a resposta antes de avancar
|
|
18
|
+
- Numere as User Stories com formato **US-XX** e os Criterios de Aceite com **CA-XX**
|
|
19
|
+
- **SEMPRE** salve o arquivo fisico antes de apresentar ao usuario
|
|
20
|
+
|
|
21
|
+
## Entrada
|
|
22
|
+
|
|
23
|
+
$ARGUMENTS
|
|
@@ -0,0 +1,37 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Gera um SPEC_TECH completo a partir de um PRD aprovado
|
|
3
|
+
argument-hint: [caminho do PRD]
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
Use a skill **sdd-spec-tech-expert** para gerar o SPEC_TECH.
|
|
7
|
+
|
|
8
|
+
## Contexto
|
|
9
|
+
|
|
10
|
+
Voce deve atuar como um **Arquiteto de Software Senior** e seguir todo o processo interativo definido na skill sdd-spec-tech-expert para transformar o PRD aprovado em uma especificacao tecnica completa.
|
|
11
|
+
|
|
12
|
+
O $ARGUMENTS deve conter:
|
|
13
|
+
1. **PRD aprovado** ou caminho para o arquivo (obrigatorio)
|
|
14
|
+
|
|
15
|
+
## Tech Direction (Opcional)
|
|
16
|
+
|
|
17
|
+
Antes de iniciar, verifique se existe um arquivo `tech_direction.md` na pasta da feature (`docs/[nome-feature]/tech_direction.md`).
|
|
18
|
+
|
|
19
|
+
- Se existir, use como **ponto de partida** para decisoes tecnicas — nao como verdade absoluta
|
|
20
|
+
- O arquivo contem decisoes ja tomadas, tecnologias sugeridas, padroes preferidos e restricoes tecnicas
|
|
21
|
+
- Voce pode complementar, ajustar ou questionar qualquer item
|
|
22
|
+
- Se NAO existir, siga o fluxo normal
|
|
23
|
+
|
|
24
|
+
## Regras Adicionais
|
|
25
|
+
|
|
26
|
+
- **Claude Code**: use a ferramenta `AskUserQuestion` para esclarecer duvidas com o usuario
|
|
27
|
+
- **NUNCA** deduza escopo ou invente informacoes — na **DUVIDA PERGUNTE AO USUARIO**
|
|
28
|
+
- Foque exclusivamente no **COMO** sera implementado, nunca repita o **O QUE** do PRD
|
|
29
|
+
- **ANTES de definir o SPEC_TECH**, pesquise as rules do projeto (`.claude/rules/`, `CLAUDE.md`) e as camadas existentes para propor a melhor solucao
|
|
30
|
+
- Se houver **tech_direction.md**, considere-o como ponto de partida para decisoes arquiteturais — nao ignore, mas valide contra o codebase existente e levante conflitos se houver
|
|
31
|
+
- Liste **TODOS** os arquivos envolvidos e as acoes (criar/modificar/referencia) — economiza tokens e scans
|
|
32
|
+
- Mapeie cada **User Story do PRD** para definicoes tecnicas correspondentes
|
|
33
|
+
- A estrategia de **testes** (secao 14) deve ser delegada a um **subagente QA especializado** em contexto isolado, conforme definido na skill
|
|
34
|
+
|
|
35
|
+
## Entrada
|
|
36
|
+
|
|
37
|
+
$ARGUMENTS
|
|
@@ -0,0 +1,27 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Gera um TASK PLAN completo com tasks individuais a partir de um SPEC_TECH aprovado
|
|
3
|
+
argument-hint: [caminho do SPEC_TECH aprovado ou nome da feature]
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
Use a skill **sdd-task-plan-expert** para gerar o TASK PLAN e as tasks individuais.
|
|
7
|
+
|
|
8
|
+
## Contexto
|
|
9
|
+
|
|
10
|
+
Voce deve atuar como um **Engenheiro de Software Senior** especializado em decomposicao de tarefas e seguir todo o processo interativo definido na skill sdd-task-plan-expert.
|
|
11
|
+
|
|
12
|
+
O $ARGUMENTS deve conter:
|
|
13
|
+
1. **SPEC_TECH aprovado** ou caminho para o arquivo (obrigatorio)
|
|
14
|
+
|
|
15
|
+
## Regras Adicionais
|
|
16
|
+
|
|
17
|
+
- **Claude Code**: use a ferramenta `AskUserQuestion` para esclarecer duvidas com o usuario
|
|
18
|
+
- **NUNCA** deduza escopo ou invente informacoes — na **DUVIDA PERGUNTE AO USUARIO**
|
|
19
|
+
- Somente o **COMO** tecnico — nunca inclua o **O QUE** de produto
|
|
20
|
+
- Liste **TODOS** os arquivos envolvidos e as acoes (criar/modificar/referencia) em cada task — economiza tokens e scans
|
|
21
|
+
- A secao de **testes** (secao 6) de cada task deve ser delegada a um **subagente QA especializado** em contexto isolado, conforme definido na skill
|
|
22
|
+
- Preencha a tabela de **rastreabilidade User Stories → Tasks** para garantir cobertura completa
|
|
23
|
+
- Ofereca **2-4 opcoes** de abordagem tecnica quando houver duvida
|
|
24
|
+
|
|
25
|
+
## Entrada
|
|
26
|
+
|
|
27
|
+
$ARGUMENTS
|
|
@@ -0,0 +1,37 @@
|
|
|
1
|
+
---
|
|
2
|
+
description: Gera estrategia de testes completa com vies de QA Senior para uma feature ou task
|
|
3
|
+
argument-hint: [caminho do SPEC_TECH/PRD ou descricao da task]
|
|
4
|
+
---
|
|
5
|
+
|
|
6
|
+
Use a skill **sdd-qa-expert** para gerar a estrategia de testes.
|
|
7
|
+
|
|
8
|
+
## Contexto
|
|
9
|
+
|
|
10
|
+
Voce deve atuar como um **QA Engineer Senior / SDET** e seguir todo o processo definido na skill sdd-qa-expert para gerar testes com qualidade profissional.
|
|
11
|
+
|
|
12
|
+
## Modos de Uso
|
|
13
|
+
|
|
14
|
+
### Modo 1: Estrategia completa para uma feature
|
|
15
|
+
```
|
|
16
|
+
/sdd:generate-tests docs/auth-oauth2/spec_tech.md
|
|
17
|
+
```
|
|
18
|
+
Gera a secao 14 completa do SPEC_TECH (unitarios, integracao, e2e, cenarios de erro).
|
|
19
|
+
|
|
20
|
+
### Modo 2: Testes para uma task especifica
|
|
21
|
+
```
|
|
22
|
+
/sdd:generate-tests docs/auth-oauth2/tasks/T1.md
|
|
23
|
+
```
|
|
24
|
+
Gera a secao 6 completa da task (unitarios, integracao, e2e, cenarios de erro).
|
|
25
|
+
|
|
26
|
+
## Regras Adicionais
|
|
27
|
+
|
|
28
|
+
- **Claude Code**: use a ferramenta `AskUserQuestion` para esclarecer duvidas com o usuario
|
|
29
|
+
- **NUNCA** gere testes genericos — cada teste deve ter cenario especifico e verificavel
|
|
30
|
+
- **SEMPRE** pesquise padroes de teste existentes no projeto antes de gerar
|
|
31
|
+
- **SEMPRE** mapeie criterios de aceite (CA-XX) para testes
|
|
32
|
+
- **SEMPRE** considere cenarios de erro, boundary values e edge cases
|
|
33
|
+
- Todo fluxo de sucesso deve ter **pelo menos 2 cenarios de falha** correspondentes
|
|
34
|
+
|
|
35
|
+
## Entrada
|
|
36
|
+
|
|
37
|
+
$ARGUMENTS
|