@thatix.io/context-first-agents-cli 0.1.0 โ 0.2.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/README.md +189 -7
- package/dist/commands/create-orchestrator.js +4 -1
- package/dist/commands/doctor.js +21 -5
- package/dist/commands/init.js +3 -1
- package/dist/templates/commands/en/engineer/plan.md +301 -0
- package/dist/templates/commands/en/engineer/pr.md +194 -0
- package/dist/templates/commands/en/engineer/pre-pr.md +325 -0
- package/dist/templates/commands/en/engineer/start.md +285 -0
- package/dist/templates/commands/en/engineer/work.md +256 -0
- package/dist/templates/commands/en/products/check.md +237 -0
- package/dist/templates/commands/en/products/collect.md +170 -0
- package/dist/templates/commands/en/products/refine.md +231 -0
- package/dist/templates/commands/en/products/spec.md +273 -0
- package/dist/templates/commands/en/quality/metrics.md +266 -0
- package/dist/templates/commands/en/quality/observe.md +172 -0
- package/dist/templates/commands/en/warm-up.md +59 -0
- package/dist/templates/commands/es/agents/CONTEXT-CONTRACT.md +63 -0
- package/dist/templates/commands/es/agents/implementer.md +27 -0
- package/dist/templates/commands/es/agents/integrator.md +24 -0
- package/dist/templates/commands/es/agents/reviewer.md +31 -0
- package/dist/templates/commands/es/agents/tester.md +22 -0
- package/dist/templates/commands/es/engineer/plan.md +335 -0
- package/dist/templates/commands/es/engineer/pr.md +228 -0
- package/dist/templates/commands/es/engineer/pre-pr.md +359 -0
- package/dist/templates/commands/es/engineer/start.md +318 -0
- package/dist/templates/commands/es/engineer/work.md +290 -0
- package/dist/templates/commands/es/orchestrate.md +125 -0
- package/dist/templates/commands/es/products/check.md +271 -0
- package/dist/templates/commands/es/products/collect.md +218 -0
- package/dist/templates/commands/es/products/refine.md +265 -0
- package/dist/templates/commands/es/products/spec.md +306 -0
- package/dist/templates/commands/es/quality/metrics.md +300 -0
- package/dist/templates/commands/es/quality/observe.md +205 -0
- package/dist/templates/commands/es/warm-up.md +59 -0
- package/dist/templates/commands/pt-BR/engineer/plan.md +335 -0
- package/dist/templates/commands/pt-BR/engineer/pr.md +228 -0
- package/dist/templates/commands/pt-BR/engineer/pre-pr.md +359 -0
- package/dist/templates/commands/pt-BR/engineer/start.md +319 -0
- package/dist/templates/commands/pt-BR/engineer/work.md +290 -0
- package/dist/templates/commands/pt-BR/products/check.md +271 -0
- package/dist/templates/commands/pt-BR/products/collect.md +219 -0
- package/dist/templates/commands/pt-BR/products/refine.md +265 -0
- package/dist/templates/commands/pt-BR/products/spec.md +307 -0
- package/dist/templates/commands/pt-BR/quality/metrics.md +300 -0
- package/dist/templates/commands/pt-BR/quality/observe.md +206 -0
- package/dist/templates/commands/pt-BR/warm-up.md +59 -0
- package/package.json +7 -3
- package/templates/commands/en/engineer/plan.md +301 -0
- package/templates/commands/en/engineer/pr.md +194 -0
- package/templates/commands/en/engineer/pre-pr.md +325 -0
- package/templates/commands/en/engineer/start.md +285 -0
- package/templates/commands/en/engineer/work.md +256 -0
- package/templates/commands/en/products/check.md +237 -0
- package/templates/commands/en/products/collect.md +170 -0
- package/templates/commands/en/products/refine.md +231 -0
- package/templates/commands/en/products/spec.md +273 -0
- package/templates/commands/en/quality/metrics.md +266 -0
- package/templates/commands/en/quality/observe.md +172 -0
- package/templates/commands/en/warm-up.md +59 -0
- package/templates/commands/es/agents/CONTEXT-CONTRACT.md +63 -0
- package/templates/commands/es/agents/implementer.md +27 -0
- package/templates/commands/es/agents/integrator.md +24 -0
- package/templates/commands/es/agents/reviewer.md +31 -0
- package/templates/commands/es/agents/tester.md +22 -0
- package/templates/commands/es/engineer/plan.md +335 -0
- package/templates/commands/es/engineer/pr.md +228 -0
- package/templates/commands/es/engineer/pre-pr.md +359 -0
- package/templates/commands/es/engineer/start.md +318 -0
- package/templates/commands/es/engineer/work.md +290 -0
- package/templates/commands/es/orchestrate.md +125 -0
- package/templates/commands/es/products/check.md +271 -0
- package/templates/commands/es/products/collect.md +218 -0
- package/templates/commands/es/products/refine.md +265 -0
- package/templates/commands/es/products/spec.md +306 -0
- package/templates/commands/es/quality/metrics.md +300 -0
- package/templates/commands/es/quality/observe.md +205 -0
- package/templates/commands/es/warm-up.md +59 -0
- package/templates/commands/pt-BR/engineer/plan.md +335 -0
- package/templates/commands/pt-BR/engineer/pr.md +228 -0
- package/templates/commands/pt-BR/engineer/pre-pr.md +359 -0
- package/templates/commands/pt-BR/engineer/start.md +319 -0
- package/templates/commands/pt-BR/engineer/work.md +290 -0
- package/templates/commands/pt-BR/products/check.md +271 -0
- package/templates/commands/pt-BR/products/collect.md +219 -0
- package/templates/commands/pt-BR/products/refine.md +265 -0
- package/templates/commands/pt-BR/products/spec.md +307 -0
- package/templates/commands/pt-BR/quality/metrics.md +300 -0
- package/templates/commands/pt-BR/quality/observe.md +206 -0
- package/templates/commands/pt-BR/warm-up.md +59 -0
|
@@ -0,0 +1,194 @@
|
|
|
1
|
+
# Pull Request Creation
|
|
2
|
+
|
|
3
|
+
This command creates Pull Requests for all modified repositories in the workspace.
|
|
4
|
+
|
|
5
|
+
## ๐ Prerequisites
|
|
6
|
+
|
|
7
|
+
Before creating PRs, make sure that:
|
|
8
|
+
- You have run `/pre-pr` and all validations passed
|
|
9
|
+
- All commits have been made
|
|
10
|
+
- All tests are passing
|
|
11
|
+
- Documentation is up to date
|
|
12
|
+
|
|
13
|
+
## Configuration
|
|
14
|
+
|
|
15
|
+
Read `context-manifest.json` and `ai.properties.md` from the orchestrator to get repositories, base_path, and task_management_system.
|
|
16
|
+
|
|
17
|
+
## ๐ CRITICAL: WHERE TO WORK
|
|
18
|
+
|
|
19
|
+
**โ ๏ธ ATTENTION: If you need to make last-minute adjustments, ALL CODE MUST BE CREATED INSIDE THE WORKTREE!**
|
|
20
|
+
|
|
21
|
+
**โ
CORRECT** - Work inside the worktree:
|
|
22
|
+
```
|
|
23
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/src/file.ts โ
|
|
24
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/README.md โ
|
|
25
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/CHANGELOG.md โ
|
|
26
|
+
```
|
|
27
|
+
|
|
28
|
+
**โ WRONG** - NEVER create code outside the worktree:
|
|
29
|
+
```
|
|
30
|
+
<orchestrator>/.sessions/file.ts โ
|
|
31
|
+
<orchestrator>/.sessions/<ISSUE-ID>/file.ts โ
|
|
32
|
+
{base_path}/<repo-name>/file.ts โ (main repository!)
|
|
33
|
+
```
|
|
34
|
+
|
|
35
|
+
**ABSOLUTE RULE**:
|
|
36
|
+
- ๐ **Any code adjustment** (docs, changelog, fixes) **MUST be in** `<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/`
|
|
37
|
+
- ๐ **NEVER modify** the main repository at `{base_path}/<repo-name>/`
|
|
38
|
+
- โ
**Work ONLY** inside the worktree of the specific repository
|
|
39
|
+
|
|
40
|
+
## ๐ฏ PR Creation Process
|
|
41
|
+
|
|
42
|
+
### 1. Identify Modified Repositories
|
|
43
|
+
|
|
44
|
+
For each repository in the workspace, check:
|
|
45
|
+
```bash
|
|
46
|
+
cd <repository>
|
|
47
|
+
git status
|
|
48
|
+
git log origin/main..HEAD # See unpushed commits
|
|
49
|
+
```
|
|
50
|
+
|
|
51
|
+
### 2. Push Branches
|
|
52
|
+
|
|
53
|
+
For each modified repository:
|
|
54
|
+
```bash
|
|
55
|
+
cd <repository>
|
|
56
|
+
git push origin <branch-name>
|
|
57
|
+
```
|
|
58
|
+
|
|
59
|
+
### 3. Create Pull Requests
|
|
60
|
+
|
|
61
|
+
For each repository, create a PR using GitHub CLI or web interface:
|
|
62
|
+
|
|
63
|
+
**Using GitHub CLI**:
|
|
64
|
+
```bash
|
|
65
|
+
cd <repository>
|
|
66
|
+
gh pr create --title "[ISSUE-ID] Feature Title" \
|
|
67
|
+
--body "$(cat ../.sessions/<ISSUE-ID>/pr-description.md)" \
|
|
68
|
+
--base main
|
|
69
|
+
```
|
|
70
|
+
|
|
71
|
+
**PR Description Template**:
|
|
72
|
+
|
|
73
|
+
```markdown
|
|
74
|
+
## ๐ฏ Objective
|
|
75
|
+
|
|
76
|
+
[Brief description of what this PR does]
|
|
77
|
+
|
|
78
|
+
## ๐ Changes
|
|
79
|
+
|
|
80
|
+
### Repository: <repo-name>
|
|
81
|
+
|
|
82
|
+
- [Change 1]
|
|
83
|
+
- [Change 2]
|
|
84
|
+
- [Change 3]
|
|
85
|
+
|
|
86
|
+
## ๐ Relationships
|
|
87
|
+
|
|
88
|
+
- **Issue**: <ISSUE-ID>
|
|
89
|
+
- **Related PRs**:
|
|
90
|
+
- <repo-1>#<PR-number>
|
|
91
|
+
- <repo-2>#<PR-number>
|
|
92
|
+
|
|
93
|
+
## โ
Checklist
|
|
94
|
+
|
|
95
|
+
- [ ] Code implemented and tested
|
|
96
|
+
- [ ] Unit tests added/updated
|
|
97
|
+
- [ ] Integration tests passing
|
|
98
|
+
- [ ] Documentation updated
|
|
99
|
+
- [ ] No breaking changes (or documented)
|
|
100
|
+
- [ ] Peer reviewed (after PR creation)
|
|
101
|
+
|
|
102
|
+
## ๐งช How to Test
|
|
103
|
+
|
|
104
|
+
1. [Step 1]
|
|
105
|
+
2. [Step 2]
|
|
106
|
+
3. [Expected result]
|
|
107
|
+
|
|
108
|
+
## ๐ธ Screenshots/Demos
|
|
109
|
+
|
|
110
|
+
[If applicable, add screenshots or links to demos]
|
|
111
|
+
|
|
112
|
+
## ๐ Notes for Reviewers
|
|
113
|
+
|
|
114
|
+
- [Point of attention 1]
|
|
115
|
+
- [Point of attention 2]
|
|
116
|
+
```
|
|
117
|
+
|
|
118
|
+
### 4. Link PRs
|
|
119
|
+
|
|
120
|
+
If there are multiple PRs (one per repository):
|
|
121
|
+
- Add cross-links between PRs
|
|
122
|
+
- Document the recommended merge order
|
|
123
|
+
- Indicate dependencies between PRs
|
|
124
|
+
|
|
125
|
+
### 5. Update Issue in Task Manager
|
|
126
|
+
|
|
127
|
+
If task manager is configured:
|
|
128
|
+
- Move the issue to "In Review" or "PR Open"
|
|
129
|
+
- Add PR links to the issue
|
|
130
|
+
- Add a comment summarizing the changes
|
|
131
|
+
|
|
132
|
+
### 6. Session Documentation
|
|
133
|
+
|
|
134
|
+
Update `./.sessions/<ISSUE-ID>/pr.md`:
|
|
135
|
+
|
|
136
|
+
```markdown
|
|
137
|
+
# [Feature Title] - Pull Requests
|
|
138
|
+
|
|
139
|
+
## Created PRs
|
|
140
|
+
|
|
141
|
+
### <repo-1>
|
|
142
|
+
- **Link**: <PR URL>
|
|
143
|
+
- **Status**: Open
|
|
144
|
+
- **Commits**: X commits
|
|
145
|
+
|
|
146
|
+
### <repo-2>
|
|
147
|
+
- **Link**: <PR URL>
|
|
148
|
+
- **Status**: Open
|
|
149
|
+
- **Commits**: Y commits
|
|
150
|
+
|
|
151
|
+
## Recommended Merge Order
|
|
152
|
+
|
|
153
|
+
1. <repo-1> - [Justification]
|
|
154
|
+
2. <repo-2> - [Justification]
|
|
155
|
+
|
|
156
|
+
## Merge Notes
|
|
157
|
+
|
|
158
|
+
- [Important note 1]
|
|
159
|
+
- [Important note 2]
|
|
160
|
+
```
|
|
161
|
+
|
|
162
|
+
## ๐ Final Checklist
|
|
163
|
+
|
|
164
|
+
Before requesting review:
|
|
165
|
+
- [ ] All PRs created
|
|
166
|
+
- [ ] Complete and clear descriptions
|
|
167
|
+
- [ ] PRs linked to each other
|
|
168
|
+
- [ ] Issue updated in task manager
|
|
169
|
+
- [ ] Tests passing in CI/CD
|
|
170
|
+
- [ ] Complete session documentation
|
|
171
|
+
|
|
172
|
+
## ๐ข Communication
|
|
173
|
+
|
|
174
|
+
Notify the team about the PRs:
|
|
175
|
+
- Mention relevant reviewers
|
|
176
|
+
- Highlight critical or breaking changes
|
|
177
|
+
- Indicate urgency if applicable
|
|
178
|
+
|
|
179
|
+
---
|
|
180
|
+
|
|
181
|
+
**Provided arguments**:
|
|
182
|
+
|
|
183
|
+
```
|
|
184
|
+
#$ARGUMENTS
|
|
185
|
+
```
|
|
186
|
+
|
|
187
|
+
---
|
|
188
|
+
|
|
189
|
+
## ๐ฏ Next Steps
|
|
190
|
+
|
|
191
|
+
1. Await PR reviews
|
|
192
|
+
2. Respond to comments and make adjustments
|
|
193
|
+
3. After approval, merge in the recommended order
|
|
194
|
+
4. Run `context-cli feature:end <ISSUE-ID>` to clean the workspace
|
|
@@ -0,0 +1,325 @@
|
|
|
1
|
+
# Pull Request Preparation
|
|
2
|
+
|
|
3
|
+
This command validates that everything is ready to create Pull Requests.
|
|
4
|
+
|
|
5
|
+
## ๐ Prerequisites
|
|
6
|
+
|
|
7
|
+
- Complete implementation (all tasks from `/plan` executed)
|
|
8
|
+
- All commits made
|
|
9
|
+
- Clean and organized workspace
|
|
10
|
+
|
|
11
|
+
## Configuration
|
|
12
|
+
|
|
13
|
+
Read `context-manifest.json` and `ai.properties.md` from the orchestrator to get repositories, base_path, and task_management_system.
|
|
14
|
+
|
|
15
|
+
## ๐ฏ Objective
|
|
16
|
+
|
|
17
|
+
Ensure that the implementation is complete, tested, and ready for review before creating PRs.
|
|
18
|
+
|
|
19
|
+
## ๐ CRITICAL: WHERE TO WORK
|
|
20
|
+
|
|
21
|
+
**โ ๏ธ ATTENTION: ALL CODE (tests, fixes, adjustments) MUST BE CREATED INSIDE THE WORKTREE!**
|
|
22
|
+
|
|
23
|
+
**โ
CORRECT** - Work inside the worktree:
|
|
24
|
+
```
|
|
25
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/src/file.ts โ
|
|
26
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/tests/test.ts โ
|
|
27
|
+
<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/.eslintrc.js โ
|
|
28
|
+
```
|
|
29
|
+
|
|
30
|
+
**โ WRONG** - NEVER create code outside the worktree:
|
|
31
|
+
```
|
|
32
|
+
<orchestrator>/.sessions/test.ts โ
|
|
33
|
+
<orchestrator>/.sessions/<ISSUE-ID>/test.ts โ
|
|
34
|
+
{base_path}/<repo-name>/test.ts โ (main repository!)
|
|
35
|
+
```
|
|
36
|
+
|
|
37
|
+
**ABSOLUTE RULE**:
|
|
38
|
+
- ๐ **ALL code** (tests, fixes, configurations) **MUST be in** `<orchestrator>/.sessions/<ISSUE-ID>/<repo-name>/`
|
|
39
|
+
- ๐ **NEVER modify** the main repository in `{base_path}/<repo-name>/`
|
|
40
|
+
- โ
**Work ONLY** inside the worktree of the specific repository
|
|
41
|
+
|
|
42
|
+
## โ
Validation Checklist
|
|
43
|
+
|
|
44
|
+
### 1. Implementation Completeness
|
|
45
|
+
|
|
46
|
+
```markdown
|
|
47
|
+
## Completeness Check
|
|
48
|
+
|
|
49
|
+
- [ ] All plan tasks have been executed
|
|
50
|
+
- [ ] All functional requirements from the PRD have been implemented
|
|
51
|
+
- [ ] All acceptance criteria have been met
|
|
52
|
+
- [ ] No functionality is half-done
|
|
53
|
+
```
|
|
54
|
+
|
|
55
|
+
### 2. Code Quality
|
|
56
|
+
|
|
57
|
+
For each modified repository:
|
|
58
|
+
|
|
59
|
+
```bash
|
|
60
|
+
cd <repository>
|
|
61
|
+
|
|
62
|
+
# Check status
|
|
63
|
+
git status
|
|
64
|
+
|
|
65
|
+
# Check linting (examples by stack):
|
|
66
|
+
# Node.js: npm run lint / yarn lint / pnpm lint
|
|
67
|
+
# Python: flake8 . / pylint src/ / black --check .
|
|
68
|
+
# Java: mvn checkstyle:check / gradle check
|
|
69
|
+
# Go: golangci-lint run / go vet ./...
|
|
70
|
+
# Ruby: rubocop
|
|
71
|
+
# Rust: cargo clippy
|
|
72
|
+
# PHP: ./vendor/bin/phpcs
|
|
73
|
+
# C#: dotnet format --verify-no-changes
|
|
74
|
+
|
|
75
|
+
# Check formatting (examples by stack):
|
|
76
|
+
# Node.js: npm run format:check / prettier --check .
|
|
77
|
+
# Python: black --check . / autopep8 --diff .
|
|
78
|
+
# Java: mvn formatter:validate
|
|
79
|
+
# Go: gofmt -l . / go fmt ./...
|
|
80
|
+
# Ruby: rubocop --format-only
|
|
81
|
+
# Rust: cargo fmt --check
|
|
82
|
+
|
|
83
|
+
# Check build (examples by stack):
|
|
84
|
+
# Node.js: npm run build / yarn build
|
|
85
|
+
# Python: python setup.py build
|
|
86
|
+
# Java: mvn compile / gradle build
|
|
87
|
+
# Go: go build ./...
|
|
88
|
+
# Ruby: rake build
|
|
89
|
+
# Rust: cargo build
|
|
90
|
+
```
|
|
91
|
+
|
|
92
|
+
Checklist:
|
|
93
|
+
```markdown
|
|
94
|
+
## Code Quality
|
|
95
|
+
|
|
96
|
+
### <repo-1>
|
|
97
|
+
- [ ] Linting without errors
|
|
98
|
+
- [ ] Correct formatting
|
|
99
|
+
- [ ] Build without errors
|
|
100
|
+
- [ ] No critical warnings
|
|
101
|
+
|
|
102
|
+
### <repo-2>
|
|
103
|
+
- [ ] Linting without errors
|
|
104
|
+
- [ ] Correct formatting
|
|
105
|
+
- [ ] Build without errors
|
|
106
|
+
- [ ] No critical warnings
|
|
107
|
+
```
|
|
108
|
+
|
|
109
|
+
### 3. Tests
|
|
110
|
+
|
|
111
|
+
For each repository:
|
|
112
|
+
|
|
113
|
+
```bash
|
|
114
|
+
cd <repository>
|
|
115
|
+
|
|
116
|
+
# Run unit tests (examples by stack):
|
|
117
|
+
# Node.js: npm run test:unit / jest / vitest
|
|
118
|
+
# Python: pytest tests/unit / python -m unittest
|
|
119
|
+
# Java: mvn test / gradle test
|
|
120
|
+
# Go: go test ./... -short
|
|
121
|
+
# Ruby: rspec spec/unit / rake test:unit
|
|
122
|
+
# Rust: cargo test --lib
|
|
123
|
+
# PHP: ./vendor/bin/phpunit --testsuite=unit
|
|
124
|
+
# C#: dotnet test --filter Category=Unit
|
|
125
|
+
|
|
126
|
+
# Run integration tests (examples by stack):
|
|
127
|
+
# Node.js: npm run test:integration
|
|
128
|
+
# Python: pytest tests/integration
|
|
129
|
+
# Java: mvn verify / gradle integrationTest
|
|
130
|
+
# Go: go test ./... -run Integration
|
|
131
|
+
# Ruby: rspec spec/integration
|
|
132
|
+
# Rust: cargo test --test '*'
|
|
133
|
+
# PHP: ./vendor/bin/phpunit --testsuite=integration
|
|
134
|
+
|
|
135
|
+
# Check coverage (examples by stack):
|
|
136
|
+
# Node.js: npm run test:coverage / jest --coverage
|
|
137
|
+
# Python: pytest --cov=src tests/
|
|
138
|
+
# Java: mvn jacoco:report / gradle jacocoTestReport
|
|
139
|
+
# Go: go test -cover ./...
|
|
140
|
+
# Ruby: rspec --coverage
|
|
141
|
+
# Rust: cargo tarpaulin
|
|
142
|
+
# PHP: ./vendor/bin/phpunit --coverage-html coverage/
|
|
143
|
+
```
|
|
144
|
+
|
|
145
|
+
Checklist:
|
|
146
|
+
```markdown
|
|
147
|
+
## Tests
|
|
148
|
+
|
|
149
|
+
### <repo-1>
|
|
150
|
+
- [ ] All unit tests passing
|
|
151
|
+
- [ ] All integration tests passing
|
|
152
|
+
- [ ] Adequate test coverage (>= X%)
|
|
153
|
+
- [ ] New tests added for new features
|
|
154
|
+
|
|
155
|
+
### <repo-2>
|
|
156
|
+
- [ ] All unit tests passing
|
|
157
|
+
- [ ] All integration tests passing
|
|
158
|
+
- [ ] Adequate test coverage (>= X%)
|
|
159
|
+
- [ ] New tests added for new features
|
|
160
|
+
```
|
|
161
|
+
|
|
162
|
+
### 4. Documentation
|
|
163
|
+
|
|
164
|
+
```markdown
|
|
165
|
+
## Documentation
|
|
166
|
+
|
|
167
|
+
- [ ] README updated (if necessary)
|
|
168
|
+
- [ ] Appropriate code comments
|
|
169
|
+
- [ ] API documentation updated (if there are changes)
|
|
170
|
+
- [ ] Changelog updated
|
|
171
|
+
- [ ] Technical documentation updated in metaspecs (if applicable)
|
|
172
|
+
```
|
|
173
|
+
|
|
174
|
+
### 5. Commits
|
|
175
|
+
|
|
176
|
+
```markdown
|
|
177
|
+
## Commits
|
|
178
|
+
|
|
179
|
+
- [ ] All commits have clear and descriptive messages
|
|
180
|
+
- [ ] Commits follow the project standard (conventional commits, etc.)
|
|
181
|
+
- [ ] No commits with generic messages ("fix", "update", etc.)
|
|
182
|
+
- [ ] Commits are logically organized
|
|
183
|
+
- [ ] No debug or temporary commits
|
|
184
|
+
```
|
|
185
|
+
|
|
186
|
+
### 6. Synchronization
|
|
187
|
+
|
|
188
|
+
```markdown
|
|
189
|
+
## Synchronization
|
|
190
|
+
|
|
191
|
+
- [ ] Branches are up to date with the base branch (main/develop)
|
|
192
|
+
- [ ] No merge conflicts
|
|
193
|
+
- [ ] Changes between repositories are synchronized
|
|
194
|
+
- [ ] Dependencies between repos have been tested
|
|
195
|
+
```
|
|
196
|
+
|
|
197
|
+
### 7. Security
|
|
198
|
+
|
|
199
|
+
```markdown
|
|
200
|
+
## Security
|
|
201
|
+
|
|
202
|
+
- [ ] No credentials or secrets in the code
|
|
203
|
+
- [ ] No sensitive data in logs
|
|
204
|
+
- [ ] Security dependencies have been checked
|
|
205
|
+
- [ ] No known vulnerabilities introduced
|
|
206
|
+
```
|
|
207
|
+
|
|
208
|
+
### 8. Performance
|
|
209
|
+
|
|
210
|
+
```markdown
|
|
211
|
+
## Performance
|
|
212
|
+
|
|
213
|
+
- [ ] No obvious performance regressions
|
|
214
|
+
- [ ] Costly queries/operations have been optimized
|
|
215
|
+
- [ ] No memory leaks introduced
|
|
216
|
+
- [ ] PRD performance requirements have been met
|
|
217
|
+
```
|
|
218
|
+
|
|
219
|
+
## ๐ Cross Validation
|
|
220
|
+
|
|
221
|
+
If multiple repositories were modified:
|
|
222
|
+
|
|
223
|
+
```markdown
|
|
224
|
+
## Cross Validation
|
|
225
|
+
|
|
226
|
+
- [ ] Tested integration between repositories locally
|
|
227
|
+
- [ ] APIs/contracts between repos are consistent
|
|
228
|
+
- [ ] No undocumented breaking changes
|
|
229
|
+
- [ ] Deployment/merge order is clear
|
|
230
|
+
```
|
|
231
|
+
|
|
232
|
+
## ๐ PR Description Preparation
|
|
233
|
+
|
|
234
|
+
Create `./.sessions/<ISSUE-ID>/pr-description.md`:
|
|
235
|
+
|
|
236
|
+
```markdown
|
|
237
|
+
## ๐ฏ Objective
|
|
238
|
+
[Brief description of what this feature does]
|
|
239
|
+
|
|
240
|
+
## ๐ Main Changes
|
|
241
|
+
- [Change 1]
|
|
242
|
+
- [Change 2]
|
|
243
|
+
- [Change 3]
|
|
244
|
+
|
|
245
|
+
## ๐ Links
|
|
246
|
+
- **Issue**: [ISSUE-ID]
|
|
247
|
+
- **PRD**: [link or path]
|
|
248
|
+
- **Technical Plan**: [link or path]
|
|
249
|
+
|
|
250
|
+
## โ
Checklist
|
|
251
|
+
- [x] Code implemented and tested
|
|
252
|
+
- [x] Unit tests added/updated
|
|
253
|
+
- [x] Integration tests passing
|
|
254
|
+
- [x] Documentation updated
|
|
255
|
+
- [x] Linting and formatting OK
|
|
256
|
+
- [x] Build without errors
|
|
257
|
+
|
|
258
|
+
## ๐งช How to Test
|
|
259
|
+
1. [Step 1]
|
|
260
|
+
2. [Step 2]
|
|
261
|
+
3. [Expected result]
|
|
262
|
+
|
|
263
|
+
## ๐ Notes for Reviewers
|
|
264
|
+
- [Point of attention 1]
|
|
265
|
+
- [Point of attention 2]
|
|
266
|
+
```
|
|
267
|
+
|
|
268
|
+
## ๐จ Issues Found
|
|
269
|
+
|
|
270
|
+
If any validation fails:
|
|
271
|
+
1. ๐ **STOP** the PR creation process
|
|
272
|
+
2. ๐ **DOCUMENT** the problem
|
|
273
|
+
3. ๐ง **FIX** the problem
|
|
274
|
+
4. ๐ **RUN** `/pre-pr` again
|
|
275
|
+
|
|
276
|
+
## ๐ Validation Report
|
|
277
|
+
|
|
278
|
+
Create `./.sessions/<ISSUE-ID>/pre-pr-report.md`:
|
|
279
|
+
|
|
280
|
+
```markdown
|
|
281
|
+
# Pre-PR Validation Report
|
|
282
|
+
|
|
283
|
+
**Date**: [date/time]
|
|
284
|
+
**Issue**: [ISSUE-ID]
|
|
285
|
+
|
|
286
|
+
## Overall Status
|
|
287
|
+
โ
Ready for PR / โ ๏ธ Pending / โ Blocked
|
|
288
|
+
|
|
289
|
+
## Validated Repositories
|
|
290
|
+
- **<repo-1>**: โ
OK
|
|
291
|
+
- **<repo-2>**: โ
OK
|
|
292
|
+
|
|
293
|
+
## Test Summary
|
|
294
|
+
- **Unit Tests**: X/X passing
|
|
295
|
+
- **Integration Tests**: Y/Y passing
|
|
296
|
+
- **Coverage**: Z%
|
|
297
|
+
|
|
298
|
+
## Pending Items (if any)
|
|
299
|
+
- [Pending item 1]
|
|
300
|
+
- [Pending item 2]
|
|
301
|
+
|
|
302
|
+
## Next Steps
|
|
303
|
+
- [x] All validations passed
|
|
304
|
+
- [ ] Run `/pr` to create Pull Requests
|
|
305
|
+
```
|
|
306
|
+
|
|
307
|
+
---
|
|
308
|
+
|
|
309
|
+
**Provided arguments**:
|
|
310
|
+
|
|
311
|
+
```
|
|
312
|
+
#$ARGUMENTS
|
|
313
|
+
```
|
|
314
|
+
|
|
315
|
+
---
|
|
316
|
+
|
|
317
|
+
## ๐ฏ Next Step
|
|
318
|
+
|
|
319
|
+
If all validations passed:
|
|
320
|
+
|
|
321
|
+
```bash
|
|
322
|
+
/pr
|
|
323
|
+
```
|
|
324
|
+
|
|
325
|
+
This command will create Pull Requests for all modified repositories.
|