context-first-cli 2.1.4 → 2.1.6
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/dist/templates/commands/en/engineer/pr.md +38 -15
- package/dist/templates/commands/en/engineer/pre-pr.md +38 -15
- package/dist/templates/commands/en/engineer/work.md +34 -10
- package/dist/templates/commands/es/engineer/pr.md +35 -12
- package/dist/templates/commands/es/engineer/pre-pr.md +58 -35
- package/dist/templates/commands/es/engineer/work.md +79 -55
- package/dist/templates/commands/pt-BR/engineer/pr.md +23 -0
- package/dist/templates/commands/pt-BR/engineer/pre-pr.md +23 -0
- package/dist/templates/commands/pt-BR/engineer/work.md +26 -2
- package/package.json +1 -1
- package/templates/commands/en/engineer/pr.md +38 -15
- package/templates/commands/en/engineer/pre-pr.md +38 -15
- package/templates/commands/en/engineer/work.md +34 -10
- package/templates/commands/es/engineer/pr.md +35 -12
- package/templates/commands/es/engineer/pre-pr.md +58 -35
- package/templates/commands/es/engineer/work.md +79 -55
- package/templates/commands/pt-BR/engineer/pr.md +23 -0
- package/templates/commands/pt-BR/engineer/pre-pr.md +23 -0
- package/templates/commands/pt-BR/engineer/work.md +26 -2
|
@@ -10,32 +10,55 @@ Before creating PRs, make sure that:
|
|
|
10
10
|
- All tests are passing
|
|
11
11
|
- Documentation is up to date
|
|
12
12
|
|
|
13
|
+
## 🛑 CRITICAL: WHERE TO WORK
|
|
14
|
+
|
|
15
|
+
**⚠️ ATTENTION: If you need to make last-minute adjustments, ALL CODE MUST BE CREATED INSIDE THE WORKTREE!**
|
|
16
|
+
|
|
17
|
+
**✅ CORRECT** - Work inside the worktree:
|
|
18
|
+
```
|
|
19
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/src/file.ts ✅
|
|
20
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/README.md ✅
|
|
21
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/CHANGELOG.md ✅
|
|
22
|
+
```
|
|
23
|
+
|
|
24
|
+
**❌ WRONG** - NEVER create code outside the worktree:
|
|
25
|
+
```
|
|
26
|
+
<orchestrator>/.sessions/file.ts ❌
|
|
27
|
+
<orchestrator>/.sessions/<ISSUE-ID>/file.ts ❌
|
|
28
|
+
{base_path}/<repo-name>/file.ts ❌ (main repository!)
|
|
29
|
+
```
|
|
30
|
+
|
|
31
|
+
**ABSOLUTE RULE**:
|
|
32
|
+
- 🛑 **Any code adjustment** (docs, changelog, fixes) **MUST be in** `<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/`
|
|
33
|
+
- 🛑 **NEVER modify** the main repository in `{base_path}/<repo-name>/`
|
|
34
|
+
- ✅ **Work ONLY** inside the specific repository’s worktree
|
|
35
|
+
|
|
13
36
|
## 🎯 PR Creation Process
|
|
14
37
|
|
|
15
38
|
### 1. Identify Modified Repositories
|
|
16
39
|
|
|
17
40
|
For each repository in the workspace, check:
|
|
18
41
|
```bash
|
|
19
|
-
cd <
|
|
42
|
+
cd <repository>
|
|
20
43
|
git status
|
|
21
|
-
git log origin/main..HEAD #
|
|
44
|
+
git log origin/main..HEAD # See unpushed commits
|
|
22
45
|
```
|
|
23
46
|
|
|
24
47
|
### 2. Push Branches
|
|
25
48
|
|
|
26
49
|
For each modified repository:
|
|
27
50
|
```bash
|
|
28
|
-
cd <
|
|
51
|
+
cd <repository>
|
|
29
52
|
git push origin <branch-name>
|
|
30
53
|
```
|
|
31
54
|
|
|
32
55
|
### 3. Create Pull Requests
|
|
33
56
|
|
|
34
|
-
For each repository, create a PR using GitHub CLI or
|
|
57
|
+
For each repository, create a PR using GitHub CLI or web interface:
|
|
35
58
|
|
|
36
59
|
**Using GitHub CLI**:
|
|
37
60
|
```bash
|
|
38
|
-
cd <
|
|
61
|
+
cd <repository>
|
|
39
62
|
gh pr create --title "[ISSUE-ID] Feature Title" \
|
|
40
63
|
--body "$(cat ../.sessions/<ISSUE-ID>/pr-description.md)" \
|
|
41
64
|
--base main
|
|
@@ -50,7 +73,7 @@ gh pr create --title "[ISSUE-ID] Feature Title" \
|
|
|
50
73
|
|
|
51
74
|
## 📝 Changes
|
|
52
75
|
|
|
53
|
-
### Repository: <
|
|
76
|
+
### Repository: <repo-name>
|
|
54
77
|
|
|
55
78
|
- [Change 1]
|
|
56
79
|
- [Change 2]
|
|
@@ -84,22 +107,22 @@ gh pr create --title "[ISSUE-ID] Feature Title" \
|
|
|
84
107
|
|
|
85
108
|
## 🔍 Notes for Reviewers
|
|
86
109
|
|
|
87
|
-
- [
|
|
88
|
-
- [
|
|
110
|
+
- [Attention point 1]
|
|
111
|
+
- [Attention point 2]
|
|
89
112
|
```
|
|
90
113
|
|
|
91
114
|
### 4. Link PRs
|
|
92
115
|
|
|
93
116
|
If there are multiple PRs (one per repository):
|
|
94
|
-
- Add cross-links between
|
|
95
|
-
- Document
|
|
117
|
+
- Add cross-links between PRs
|
|
118
|
+
- Document recommended merge order
|
|
96
119
|
- Indicate dependencies between PRs
|
|
97
120
|
|
|
98
121
|
### 5. Update Issue in Task Manager
|
|
99
122
|
|
|
100
|
-
If
|
|
123
|
+
If a task manager is configured:
|
|
101
124
|
- Move the issue to "In Review" or "PR Open"
|
|
102
|
-
- Add PR links
|
|
125
|
+
- Add PR links to the issue
|
|
103
126
|
- Add a comment summarizing the changes
|
|
104
127
|
|
|
105
128
|
### 6. Session Documentation
|
|
@@ -126,7 +149,7 @@ Update `./.sessions/<ISSUE-ID>/pr.md`:
|
|
|
126
149
|
1. <repo-1> - [Justification]
|
|
127
150
|
2. <repo-2> - [Justification]
|
|
128
151
|
|
|
129
|
-
## Merge
|
|
152
|
+
## Notes for Merge
|
|
130
153
|
|
|
131
154
|
- [Important note 1]
|
|
132
155
|
- [Important note 2]
|
|
@@ -146,7 +169,7 @@ Before requesting review:
|
|
|
146
169
|
|
|
147
170
|
Notify the team about the PRs:
|
|
148
171
|
- Mention relevant reviewers
|
|
149
|
-
- Highlight critical
|
|
172
|
+
- Highlight critical or breaking changes
|
|
150
173
|
- Indicate urgency if applicable
|
|
151
174
|
|
|
152
175
|
---
|
|
@@ -161,7 +184,7 @@ Notify the team about the PRs:
|
|
|
161
184
|
|
|
162
185
|
## 🎯 Next Steps
|
|
163
186
|
|
|
164
|
-
1. Await PR
|
|
187
|
+
1. Await PR reviews
|
|
165
188
|
2. Respond to comments and make adjustments
|
|
166
189
|
3. After approval, merge in the recommended order
|
|
167
190
|
4. Run `context-cli feature:end <ISSUE-ID>` to clean the workspace
|
|
@@ -4,7 +4,7 @@ This command validates that everything is ready to create Pull Requests.
|
|
|
4
4
|
|
|
5
5
|
## 📋 Prerequisites
|
|
6
6
|
|
|
7
|
-
- Complete implementation (all `/plan`
|
|
7
|
+
- Complete implementation (all tasks from `/plan` executed)
|
|
8
8
|
- All commits made
|
|
9
9
|
- Clean and organized workspace
|
|
10
10
|
|
|
@@ -12,6 +12,29 @@ This command validates that everything is ready to create Pull Requests.
|
|
|
12
12
|
|
|
13
13
|
Ensure that the implementation is complete, tested, and ready for review before creating the PRs.
|
|
14
14
|
|
|
15
|
+
## 🛑 CRITICAL: WHERE TO WORK
|
|
16
|
+
|
|
17
|
+
**⚠️ ATTENTION: ALL CODE (tests, fixes, adjustments) MUST BE CREATED INSIDE THE WORKTREE!**
|
|
18
|
+
|
|
19
|
+
**✅ CORRECT** - Work inside the worktree:
|
|
20
|
+
```
|
|
21
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/src/file.ts ✅
|
|
22
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/tests/test.ts ✅
|
|
23
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/.eslintrc.js ✅
|
|
24
|
+
```
|
|
25
|
+
|
|
26
|
+
**❌ WRONG** - NEVER create code outside the worktree:
|
|
27
|
+
```
|
|
28
|
+
<orchestrator>/.sessions/test.ts ❌
|
|
29
|
+
<orchestrator>/.sessions/<ISSUE-ID>/test.ts ❌
|
|
30
|
+
{base_path}/<repo-name>/test.ts ❌ (main repository!)
|
|
31
|
+
```
|
|
32
|
+
|
|
33
|
+
**ABSOLUTE RULE**:
|
|
34
|
+
- 🛑 **ALL code** (tests, fixes, configurations) **MUST be in** `<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/`
|
|
35
|
+
- 🛑 **NEVER modify** the main repository at `{base_path}/<repo-name>/`
|
|
36
|
+
- ✅ **Work ONLY** inside the specific repository’s worktree
|
|
37
|
+
|
|
15
38
|
## ✅ Validation Checklist
|
|
16
39
|
|
|
17
40
|
### 1. Implementation Completeness
|
|
@@ -19,10 +42,10 @@ Ensure that the implementation is complete, tested, and ready for review before
|
|
|
19
42
|
```markdown
|
|
20
43
|
## Completeness Check
|
|
21
44
|
|
|
22
|
-
- [ ] All plan
|
|
45
|
+
- [ ] All tasks from the plan have been executed
|
|
23
46
|
- [ ] All functional requirements from the PRD have been implemented
|
|
24
47
|
- [ ] All acceptance criteria have been met
|
|
25
|
-
- [ ] No functionality is
|
|
48
|
+
- [ ] No functionality is half-done
|
|
26
49
|
```
|
|
27
50
|
|
|
28
51
|
### 2. Code Quality
|
|
@@ -163,7 +186,7 @@ Checklist:
|
|
|
163
186
|
|
|
164
187
|
- [ ] Branches are up to date with the base branch (main/develop)
|
|
165
188
|
- [ ] No merge conflicts
|
|
166
|
-
- [ ] Changes
|
|
189
|
+
- [ ] Changes across repositories are synchronized
|
|
167
190
|
- [ ] Dependencies between repos have been tested
|
|
168
191
|
```
|
|
169
192
|
|
|
@@ -184,7 +207,7 @@ Checklist:
|
|
|
184
207
|
## Performance
|
|
185
208
|
|
|
186
209
|
- [ ] No obvious performance regressions
|
|
187
|
-
- [ ]
|
|
210
|
+
- [ ] Expensive queries/operations have been optimized
|
|
188
211
|
- [ ] No memory leaks introduced
|
|
189
212
|
- [ ] PRD performance requirements have been met
|
|
190
213
|
```
|
|
@@ -234,16 +257,16 @@ Create `./.sessions/<ISSUE-ID>/pr-description.md`:
|
|
|
234
257
|
3. [Expected result]
|
|
235
258
|
|
|
236
259
|
## 🔍 Notes for Reviewers
|
|
237
|
-
- [
|
|
238
|
-
- [
|
|
260
|
+
- [Point of attention 1]
|
|
261
|
+
- [Point of attention 2]
|
|
239
262
|
```
|
|
240
263
|
|
|
241
264
|
## 🚨 Issues Found
|
|
242
265
|
|
|
243
266
|
If any validation fails:
|
|
244
267
|
1. 🛑 **STOP** the PR creation process
|
|
245
|
-
2. 📝 **DOCUMENT** the
|
|
246
|
-
3. 🔧 **FIX** the
|
|
268
|
+
2. 📝 **DOCUMENT** the problem
|
|
269
|
+
3. 🔧 **FIX** the problem
|
|
247
270
|
4. 🔄 **RUN** `/pre-pr` again
|
|
248
271
|
|
|
249
272
|
## 📊 Validation Report
|
|
@@ -253,11 +276,11 @@ Create `./.sessions/<ISSUE-ID>/pre-pr-report.md`:
|
|
|
253
276
|
```markdown
|
|
254
277
|
# Pre-PR Validation Report
|
|
255
278
|
|
|
256
|
-
**Date**: [date/time]
|
|
279
|
+
**Date**: [date/time]
|
|
257
280
|
**Issue**: [ISSUE-ID]
|
|
258
281
|
|
|
259
282
|
## Overall Status
|
|
260
|
-
✅ Ready for PR / ⚠️ Pending / ❌ Blocked
|
|
283
|
+
✅ Ready for PR / ⚠️ Pending Issues / ❌ Blocked
|
|
261
284
|
|
|
262
285
|
## Validated Repositories
|
|
263
286
|
- **<repo-1>**: ✅ OK
|
|
@@ -268,9 +291,9 @@ Create `./.sessions/<ISSUE-ID>/pre-pr-report.md`:
|
|
|
268
291
|
- **Integration Tests**: Y/Y passing
|
|
269
292
|
- **Coverage**: Z%
|
|
270
293
|
|
|
271
|
-
## Pending
|
|
272
|
-
- [Pending
|
|
273
|
-
- [Pending
|
|
294
|
+
## Pending Issues (if any)
|
|
295
|
+
- [Pending issue 1]
|
|
296
|
+
- [Pending issue 2]
|
|
274
297
|
|
|
275
298
|
## Next Steps
|
|
276
299
|
- [x] All validations passed
|
|
@@ -295,4 +318,4 @@ If all validations passed:
|
|
|
295
318
|
/pr
|
|
296
319
|
```
|
|
297
320
|
|
|
298
|
-
This command will create
|
|
321
|
+
This command will create Pull Requests for all modified repositories.
|
|
@@ -36,16 +36,40 @@ Before executing, make sure that:
|
|
|
36
36
|
- ❌ NEVER checkout branches in the main repositories
|
|
37
37
|
- ❌ NEVER navigate to `{base_path}/{repo-id}/`
|
|
38
38
|
|
|
39
|
+
## 🛑 CRITICAL: WHERE TO CREATE CODE
|
|
40
|
+
|
|
41
|
+
**⚠️ ATTENTION: ALL CODE MUST BE CREATED INSIDE THE REPOSITORY WORKTREE!**
|
|
42
|
+
|
|
43
|
+
**✅ CORRECT** - Create code inside the worktree:
|
|
44
|
+
```
|
|
45
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/src/file.ts ✅
|
|
46
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/tests/test.ts ✅
|
|
47
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/package.json ✅
|
|
48
|
+
```
|
|
49
|
+
|
|
50
|
+
**❌ WRONG** - NEVER create code directly in .sessions:
|
|
51
|
+
```
|
|
52
|
+
<orchestrator>/.sessions/src/file.ts ❌
|
|
53
|
+
<orchestrator>/.sessions/<ISSUE-ID>/src/file.ts ❌
|
|
54
|
+
<orchestrator>/.sessions/<ISSUE-ID>/file.ts ❌
|
|
55
|
+
```
|
|
56
|
+
|
|
57
|
+
**ABSOLUTE RULE**:
|
|
58
|
+
- 🛑 **EVERY code file** (`.ts`, `.js`, `.py`, `.java`, etc.) **MUST be inside** `<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/`
|
|
59
|
+
- 🛑 **NEVER create code** directly in `<orchestrator>/.sessions/` or `<orchestrator>/.sessions/<ISSUE-ID>/`
|
|
60
|
+
- ✅ **Only valid place**: Inside the specific repository worktree
|
|
61
|
+
|
|
39
62
|
## ⚠️ IMPORTANT: Immutable Files
|
|
40
63
|
|
|
41
64
|
**This command must READ but NOT MODIFY:**
|
|
42
65
|
- ✅ **READ** `.sessions/<ISSUE-ID>/context.md` (immutable)
|
|
43
66
|
- ✅ **READ** `.sessions/<ISSUE-ID>/architecture.md` (immutable)
|
|
44
67
|
- ✅ **UPDATE** `.sessions/<ISSUE-ID>/plan.md` (mark progress)
|
|
45
|
-
- ✅ **IMPLEMENT** code
|
|
46
|
-
- ✅ **MAKE COMMITS** in the
|
|
68
|
+
- ✅ **IMPLEMENT** code **INSIDE THE WORKTREE**: `.sessions/<ISSUE-ID>/<repo-name>/`
|
|
69
|
+
- ✅ **MAKE COMMITS** in the worktrees: `.sessions/<ISSUE-ID>/<repo-name>/`
|
|
47
70
|
- ❌ **DO NOT modify `context.md` or `architecture.md`**
|
|
48
|
-
- ❌ **DO NOT checkout branches in the main repositories (outside workspace)**
|
|
71
|
+
- ❌ **DO NOT checkout branches in the main repositories (outside the workspace)**
|
|
72
|
+
- 🛑 **NEVER create code directly in `.sessions/` or `.sessions/<ISSUE-ID>/`**
|
|
49
73
|
|
|
50
74
|
## 📚 Load MetaSpecs
|
|
51
75
|
|
|
@@ -53,8 +77,8 @@ Before executing, make sure that:
|
|
|
53
77
|
1. Read `context-manifest.json` from the orchestrator
|
|
54
78
|
2. Find the repository with `"role": "metaspecs"`
|
|
55
79
|
3. Read `ai.properties.md` to get the `base_path`
|
|
56
|
-
4. The metaspecs are
|
|
57
|
-
5. Read
|
|
80
|
+
4. The metaspecs are at: `{base_path}/{metaspecs-repo-id}/`
|
|
81
|
+
5. Read relevant `index.md` files during implementation to:
|
|
58
82
|
- Follow coding standards
|
|
59
83
|
- Respect defined architecture
|
|
60
84
|
- Use correct conventions
|
|
@@ -71,7 +95,7 @@ Implement a specific unit of work from the plan, which may involve:
|
|
|
71
95
|
|
|
72
96
|
**⚠️ IMPORTANT: PROGRESS CONTROL**
|
|
73
97
|
|
|
74
|
-
This command executes work in **incremental phases**. After completing each **MAIN PHASE** (e.g., Phase 1 → Phase 2):
|
|
98
|
+
This command executes the work in **incremental phases**. After completing each **MAIN PHASE** (e.g., Phase 1 → Phase 2):
|
|
75
99
|
|
|
76
100
|
1. 🛑 **STOP** execution
|
|
77
101
|
2. 📊 **PRESENT** a summary of what was done
|
|
@@ -84,7 +108,7 @@ This command executes work in **incremental phases**. After completing each **MA
|
|
|
84
108
|
- ✅ **PAUSE** between main phases (Phase 1 → Phase 2 → Phase 3)
|
|
85
109
|
- ❌ **DO NOT pause** between subphases (Phase 1.1 → Phase 1.2 → Phase 1.3)
|
|
86
110
|
|
|
87
|
-
**DO NOT implement everything at once**. Work main phase by main phase,
|
|
111
|
+
**DO NOT implement everything at once**. Work main phase by main phase, awaiting developer confirmation.
|
|
88
112
|
|
|
89
113
|
---
|
|
90
114
|
|
|
@@ -92,7 +116,7 @@ This command executes work in **incremental phases**. After completing each **MA
|
|
|
92
116
|
|
|
93
117
|
Based on the technical plan (`./.sessions/<ISSUE-ID>/plan.md`), identify:
|
|
94
118
|
- Which specific task will be implemented now
|
|
95
|
-
- In which
|
|
119
|
+
- In which repository(ies) of the workspace
|
|
96
120
|
- Which files will be created/modified
|
|
97
121
|
- Dependencies with other tasks
|
|
98
122
|
|
|
@@ -100,7 +124,7 @@ Based on the technical plan (`./.sessions/<ISSUE-ID>/plan.md`), identify:
|
|
|
100
124
|
|
|
101
125
|
|
|
102
126
|
|
|
103
|
-
**IMPORTANT**: Work ONLY inside the workspace
|
|
127
|
+
**IMPORTANT**: Work ONLY inside the workspace in `.sessions/<ISSUE-ID>/`
|
|
104
128
|
|
|
105
129
|
For each repository in the workspace:
|
|
106
130
|
|
|
@@ -133,7 +157,7 @@ Before committing:
|
|
|
133
157
|
|
|
134
158
|
### 4. Commit
|
|
135
159
|
|
|
136
|
-
For each modified
|
|
160
|
+
For each repository modified **inside the workspace**:
|
|
137
161
|
|
|
138
162
|
```bash
|
|
139
163
|
# Navigate to the worktree inside the workspace
|
|
@@ -5,18 +5,41 @@ Este comando crea Pull Requests para todos los repositorios modificados en el wo
|
|
|
5
5
|
## 📋 Requisitos previos
|
|
6
6
|
|
|
7
7
|
Antes de crear PRs, asegúrate de que:
|
|
8
|
-
-
|
|
8
|
+
- Ejecutaste `/pre-pr` y todas las validaciones pasaron
|
|
9
9
|
- Todos los commits fueron realizados
|
|
10
10
|
- Todas las pruebas están pasando
|
|
11
11
|
- La documentación está actualizada
|
|
12
12
|
|
|
13
|
+
## 🛑 CRÍTICO: DÓNDE TRABAJAR
|
|
14
|
+
|
|
15
|
+
**⚠️ ATENCIÓN: Si necesitas hacer ajustes de última hora, ¡TODO EL CÓDIGO DEBE SER CREADO DENTRO DEL WORKTREE!**
|
|
16
|
+
|
|
17
|
+
**✅ CORRECTO** - Trabajar dentro del worktree:
|
|
18
|
+
```
|
|
19
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/src/file.ts ✅
|
|
20
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/README.md ✅
|
|
21
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/CHANGELOG.md ✅
|
|
22
|
+
```
|
|
23
|
+
|
|
24
|
+
**❌ INCORRECTO** - NUNCA crear código fuera del worktree:
|
|
25
|
+
```
|
|
26
|
+
<orchestrator>/.sessions/file.ts ❌
|
|
27
|
+
<orchestrator>/.sessions/<ISSUE-ID>/file.ts ❌
|
|
28
|
+
{base_path}/<repo-name>/file.ts ❌ (¡repositorio principal!)
|
|
29
|
+
```
|
|
30
|
+
|
|
31
|
+
**REGLA ABSOLUTA**:
|
|
32
|
+
- 🛑 **Cualquier ajuste de código** (docs, changelog, fixes) **DEBE estar en** `<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/`
|
|
33
|
+
- 🛑 **NUNCA modifiques** el repositorio principal en `{base_path}/<repo-name>/`
|
|
34
|
+
- ✅ **Trabaja SÓLO** dentro del worktree del repositorio específico
|
|
35
|
+
|
|
13
36
|
## 🎯 Proceso de Creación de PRs
|
|
14
37
|
|
|
15
38
|
### 1. Identificar Repositorios Modificados
|
|
16
39
|
|
|
17
40
|
Para cada repositorio en el workspace, verifica:
|
|
18
41
|
```bash
|
|
19
|
-
cd <
|
|
42
|
+
cd <repositorio>
|
|
20
43
|
git status
|
|
21
44
|
git log origin/main..HEAD # Ver commits no pushados
|
|
22
45
|
```
|
|
@@ -25,7 +48,7 @@ git log origin/main..HEAD # Ver commits no pushados
|
|
|
25
48
|
|
|
26
49
|
Para cada repositorio modificado:
|
|
27
50
|
```bash
|
|
28
|
-
cd <
|
|
51
|
+
cd <repositorio>
|
|
29
52
|
git push origin <branch-name>
|
|
30
53
|
```
|
|
31
54
|
|
|
@@ -35,7 +58,7 @@ Para cada repositorio, crea un PR usando el GitHub CLI o la interfaz web:
|
|
|
35
58
|
|
|
36
59
|
**Usando GitHub CLI**:
|
|
37
60
|
```bash
|
|
38
|
-
cd <
|
|
61
|
+
cd <repositorio>
|
|
39
62
|
gh pr create --title "[ISSUE-ID] Título de la Feature" \
|
|
40
63
|
--body "$(cat ../.sessions/<ISSUE-ID>/pr-description.md)" \
|
|
41
64
|
--base main
|
|
@@ -50,7 +73,7 @@ gh pr create --title "[ISSUE-ID] Título de la Feature" \
|
|
|
50
73
|
|
|
51
74
|
## 📝 Cambios
|
|
52
75
|
|
|
53
|
-
### Repositorio: <
|
|
76
|
+
### Repositorio: <nombre-del-repo>
|
|
54
77
|
|
|
55
78
|
- [Cambio 1]
|
|
56
79
|
- [Cambio 2]
|
|
@@ -70,17 +93,17 @@ gh pr create --title "[ISSUE-ID] Título de la Feature" \
|
|
|
70
93
|
- [ ] Pruebas de integración pasando
|
|
71
94
|
- [ ] Documentación actualizada
|
|
72
95
|
- [ ] Sin breaking changes (o documentados)
|
|
73
|
-
- [ ] Revisado por pares (
|
|
96
|
+
- [ ] Revisado por pares (tras creación del PR)
|
|
74
97
|
|
|
75
|
-
## 🧪 Cómo
|
|
98
|
+
## 🧪 Cómo Probar
|
|
76
99
|
|
|
77
100
|
1. [Paso 1]
|
|
78
101
|
2. [Paso 2]
|
|
79
102
|
3. [Resultado esperado]
|
|
80
103
|
|
|
81
|
-
## 📸
|
|
104
|
+
## 📸 Capturas/Demos
|
|
82
105
|
|
|
83
|
-
[Si aplica, añade capturas
|
|
106
|
+
[Si aplica, añade capturas o enlaces a demos]
|
|
84
107
|
|
|
85
108
|
## 🔍 Notas para Revisores
|
|
86
109
|
|
|
@@ -112,12 +135,12 @@ Actualiza `./.sessions/<ISSUE-ID>/pr.md`:
|
|
|
112
135
|
## PRs Creados
|
|
113
136
|
|
|
114
137
|
### <repo-1>
|
|
115
|
-
- **
|
|
138
|
+
- **Enlace**: <URL del PR>
|
|
116
139
|
- **Estado**: Abierto
|
|
117
140
|
- **Commits**: X commits
|
|
118
141
|
|
|
119
142
|
### <repo-2>
|
|
120
|
-
- **
|
|
143
|
+
- **Enlace**: <URL del PR>
|
|
121
144
|
- **Estado**: Abierto
|
|
122
145
|
- **Commits**: Y commits
|
|
123
146
|
|
|
@@ -163,5 +186,5 @@ Notifica al equipo sobre los PRs:
|
|
|
163
186
|
|
|
164
187
|
1. Esperar revisión de los PRs
|
|
165
188
|
2. Responder comentarios y hacer ajustes
|
|
166
|
-
3. Tras
|
|
189
|
+
3. Tras aprobación, hacer merge en el orden recomendado
|
|
167
190
|
4. Ejecutar `context-cli feature:end <ISSUE-ID>` para limpiar el workspace
|
|
@@ -12,6 +12,29 @@ Este comando valida que todo está listo para crear Pull Requests.
|
|
|
12
12
|
|
|
13
13
|
Garantizar que la implementación está completa, probada y lista para revisión antes de crear los PRs.
|
|
14
14
|
|
|
15
|
+
## 🛑 CRÍTICO: DÓNDE TRABAJAR
|
|
16
|
+
|
|
17
|
+
**⚠️ ATENCIÓN: TODO CÓDIGO (tests, fixes, ajustes) DEBE SER CREADO DENTRO DEL WORKTREE!**
|
|
18
|
+
|
|
19
|
+
**✅ CORRECTO** - Trabajar dentro del worktree:
|
|
20
|
+
```
|
|
21
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/src/file.ts ✅
|
|
22
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/tests/test.ts ✅
|
|
23
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/.eslintrc.js ✅
|
|
24
|
+
```
|
|
25
|
+
|
|
26
|
+
**❌ INCORRECTO** - NUNCA crear código fuera del worktree:
|
|
27
|
+
```
|
|
28
|
+
<orchestrator>/.sessions/test.ts ❌
|
|
29
|
+
<orchestrator>/.sessions/<ISSUE-ID>/test.ts ❌
|
|
30
|
+
{base_path}/<repo-name>/test.ts ❌ (¡repositorio principal!)
|
|
31
|
+
```
|
|
32
|
+
|
|
33
|
+
**REGLA ABSOLUTA**:
|
|
34
|
+
- 🛑 **TODO código** (tests, fixes, configuraciones) **DEBE estar en** `<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/`
|
|
35
|
+
- 🛑 **NUNCA modifiques** el repositorio principal en `{base_path}/<repo-name>/`
|
|
36
|
+
- ✅ **Trabaja SOLO** dentro del worktree del repositorio específico
|
|
37
|
+
|
|
15
38
|
## ✅ Checklist de Validación
|
|
16
39
|
|
|
17
40
|
### 1. Completitud de la Implementación
|
|
@@ -79,14 +102,14 @@ Checklist:
|
|
|
79
102
|
- [ ] Sin warnings críticos
|
|
80
103
|
```
|
|
81
104
|
|
|
82
|
-
### 3.
|
|
105
|
+
### 3. Tests
|
|
83
106
|
|
|
84
107
|
Para cada repositorio:
|
|
85
108
|
|
|
86
109
|
```bash
|
|
87
110
|
cd <repositorio>
|
|
88
111
|
|
|
89
|
-
# Ejecutar
|
|
112
|
+
# Ejecutar tests unitarios (ejemplos por stack):
|
|
90
113
|
# Node.js: npm run test:unit / jest / vitest
|
|
91
114
|
# Python: pytest tests/unit / python -m unittest
|
|
92
115
|
# Java: mvn test / gradle test
|
|
@@ -96,7 +119,7 @@ cd <repositorio>
|
|
|
96
119
|
# PHP: ./vendor/bin/phpunit --testsuite=unit
|
|
97
120
|
# C#: dotnet test --filter Category=Unit
|
|
98
121
|
|
|
99
|
-
# Ejecutar
|
|
122
|
+
# Ejecutar tests de integración (ejemplos por stack):
|
|
100
123
|
# Node.js: npm run test:integration
|
|
101
124
|
# Python: pytest tests/integration
|
|
102
125
|
# Java: mvn verify / gradle integrationTest
|
|
@@ -117,19 +140,19 @@ cd <repositorio>
|
|
|
117
140
|
|
|
118
141
|
Checklist:
|
|
119
142
|
```markdown
|
|
120
|
-
##
|
|
143
|
+
## Tests
|
|
121
144
|
|
|
122
145
|
### <repo-1>
|
|
123
|
-
- [ ]
|
|
124
|
-
- [ ]
|
|
125
|
-
- [ ] Cobertura de
|
|
126
|
-
- [ ]
|
|
146
|
+
- [ ] Todos los tests unitarios pasan
|
|
147
|
+
- [ ] Todos los tests de integración pasan
|
|
148
|
+
- [ ] Cobertura de tests adecuada (>= X%)
|
|
149
|
+
- [ ] Nuevos tests añadidos para nuevas funcionalidades
|
|
127
150
|
|
|
128
151
|
### <repo-2>
|
|
129
|
-
- [ ]
|
|
130
|
-
- [ ]
|
|
131
|
-
- [ ] Cobertura de
|
|
132
|
-
- [ ]
|
|
152
|
+
- [ ] Todos los tests unitarios pasan
|
|
153
|
+
- [ ] Todos los tests de integración pasan
|
|
154
|
+
- [ ] Cobertura de tests adecuada (>= X%)
|
|
155
|
+
- [ ] Nuevos tests añadidos para nuevas funcionalidades
|
|
133
156
|
```
|
|
134
157
|
|
|
135
158
|
### 4. Documentación
|
|
@@ -141,7 +164,7 @@ Checklist:
|
|
|
141
164
|
- [ ] Comentarios de código adecuados
|
|
142
165
|
- [ ] Documentación de APIs actualizada (si hay cambios)
|
|
143
166
|
- [ ] Changelog actualizado
|
|
144
|
-
- [ ] Documentación técnica actualizada en
|
|
167
|
+
- [ ] Documentación técnica actualizada en metaspecs (si aplica)
|
|
145
168
|
```
|
|
146
169
|
|
|
147
170
|
### 5. Commits
|
|
@@ -150,9 +173,9 @@ Checklist:
|
|
|
150
173
|
## Commits
|
|
151
174
|
|
|
152
175
|
- [ ] Todos los commits tienen mensajes claros y descriptivos
|
|
153
|
-
- [ ]
|
|
176
|
+
- [ ] Commits siguen el estándar del proyecto (conventional commits, etc.)
|
|
154
177
|
- [ ] No hay commits con mensajes genéricos ("fix", "update", etc.)
|
|
155
|
-
- [ ]
|
|
178
|
+
- [ ] Commits están organizados lógicamente
|
|
156
179
|
- [ ] No hay commits de debug o temporales
|
|
157
180
|
```
|
|
158
181
|
|
|
@@ -161,7 +184,7 @@ Checklist:
|
|
|
161
184
|
```markdown
|
|
162
185
|
## Sincronización
|
|
163
186
|
|
|
164
|
-
- [ ]
|
|
187
|
+
- [ ] Branches están actualizadas con la branch base (main/develop)
|
|
165
188
|
- [ ] No hay conflictos de merge
|
|
166
189
|
- [ ] Cambios entre repositorios están sincronizados
|
|
167
190
|
- [ ] Dependencias entre repos fueron probadas
|
|
@@ -172,7 +195,7 @@ Checklist:
|
|
|
172
195
|
```markdown
|
|
173
196
|
## Seguridad
|
|
174
197
|
|
|
175
|
-
- [ ] No hay credenciales
|
|
198
|
+
- [ ] No hay credenciales o secretos en el código
|
|
176
199
|
- [ ] No hay datos sensibles en logs
|
|
177
200
|
- [ ] Dependencias de seguridad fueron verificadas
|
|
178
201
|
- [ ] No hay vulnerabilidades conocidas introducidas
|
|
@@ -183,7 +206,7 @@ Checklist:
|
|
|
183
206
|
```markdown
|
|
184
207
|
## Performance
|
|
185
208
|
|
|
186
|
-
- [ ] No hay regresiones de performance
|
|
209
|
+
- [ ] No hay regresiones de performance obvias
|
|
187
210
|
- [ ] Queries/operaciones costosas fueron optimizadas
|
|
188
211
|
- [ ] No hay memory leaks introducidos
|
|
189
212
|
- [ ] Requisitos de performance del PRD fueron cumplidos
|
|
@@ -191,7 +214,7 @@ Checklist:
|
|
|
191
214
|
|
|
192
215
|
## 🔍 Validación Cruzada
|
|
193
216
|
|
|
194
|
-
Si
|
|
217
|
+
Si múltiples repositorios fueron modificados:
|
|
195
218
|
|
|
196
219
|
```markdown
|
|
197
220
|
## Validación Cruzada
|
|
@@ -199,12 +222,12 @@ Si se modificaron múltiples repositorios:
|
|
|
199
222
|
- [ ] Probé la integración entre los repositorios localmente
|
|
200
223
|
- [ ] APIs/contratos entre repos están consistentes
|
|
201
224
|
- [ ] No hay breaking changes no documentados
|
|
202
|
-
- [ ]
|
|
225
|
+
- [ ] Orden de deploy/merge está claro
|
|
203
226
|
```
|
|
204
227
|
|
|
205
228
|
## 📄 Preparación de la Descripción del PR
|
|
206
229
|
|
|
207
|
-
|
|
230
|
+
Crea `./.sessions/<ISSUE-ID>/pr-description.md`:
|
|
208
231
|
|
|
209
232
|
```markdown
|
|
210
233
|
## 🎯 Objetivo
|
|
@@ -215,15 +238,15 @@ Cree `./.sessions/<ISSUE-ID>/pr-description.md`:
|
|
|
215
238
|
- [Cambio 2]
|
|
216
239
|
- [Cambio 3]
|
|
217
240
|
|
|
218
|
-
## 🔗
|
|
241
|
+
## 🔗 Links
|
|
219
242
|
- **Issue**: [ISSUE-ID]
|
|
220
243
|
- **PRD**: [link o ruta]
|
|
221
244
|
- **Plan Técnico**: [link o ruta]
|
|
222
245
|
|
|
223
246
|
## ✅ Checklist
|
|
224
247
|
- [x] Código implementado y probado
|
|
225
|
-
- [x]
|
|
226
|
-
- [x]
|
|
248
|
+
- [x] Tests unitarios añadidos/actualizados
|
|
249
|
+
- [x] Tests de integración pasando
|
|
227
250
|
- [x] Documentación actualizada
|
|
228
251
|
- [x] Linting y formateo OK
|
|
229
252
|
- [x] Build sin errores
|
|
@@ -241,17 +264,17 @@ Cree `./.sessions/<ISSUE-ID>/pr-description.md`:
|
|
|
241
264
|
## 🚨 Problemas Encontrados
|
|
242
265
|
|
|
243
266
|
Si alguna validación falla:
|
|
244
|
-
1. 🛑 **
|
|
245
|
-
2. 📝 **
|
|
246
|
-
3. 🔧 **
|
|
247
|
-
4. 🔄 **
|
|
267
|
+
1. 🛑 **DETÉN** el proceso de creación de PR
|
|
268
|
+
2. 📝 **DOCUMENTA** el problema
|
|
269
|
+
3. 🔧 **CORRIGE** el problema
|
|
270
|
+
4. 🔄 **EJECUTA** `/pre-pr` nuevamente
|
|
248
271
|
|
|
249
|
-
## 📊
|
|
272
|
+
## 📊 Reporte de Validación
|
|
250
273
|
|
|
251
|
-
|
|
274
|
+
Crea `./.sessions/<ISSUE-ID>/pre-pr-report.md`:
|
|
252
275
|
|
|
253
276
|
```markdown
|
|
254
|
-
#
|
|
277
|
+
# Reporte de Validación Pre-PR
|
|
255
278
|
|
|
256
279
|
**Fecha**: [fecha/hora]
|
|
257
280
|
**Issue**: [ISSUE-ID]
|
|
@@ -263,12 +286,12 @@ Cree `./.sessions/<ISSUE-ID>/pre-pr-report.md`:
|
|
|
263
286
|
- **<repo-1>**: ✅ OK
|
|
264
287
|
- **<repo-2>**: ✅ OK
|
|
265
288
|
|
|
266
|
-
## Resumen de
|
|
267
|
-
- **
|
|
268
|
-
- **
|
|
289
|
+
## Resumen de Tests
|
|
290
|
+
- **Tests Unitarios**: X/X pasando
|
|
291
|
+
- **Tests de Integración**: Y/Y pasando
|
|
269
292
|
- **Cobertura**: Z%
|
|
270
293
|
|
|
271
|
-
## Pendientes (si
|
|
294
|
+
## Pendientes (si hay)
|
|
272
295
|
- [Pendiente 1]
|
|
273
296
|
- [Pendiente 2]
|
|
274
297
|
|