|
1 | 1 | ## Description |
2 | 2 |
|
3 | | -<!-- Provide a clear and concise description of the changes --> |
4 | | - |
5 | | -## Type of Change |
6 | | - |
7 | | -- [ ] 🐛 Bug fix (non-breaking change which fixes an issue) |
8 | | -- [ ] ✨ New feature (non-breaking change which adds functionality) |
9 | | -- [ ] 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected) |
10 | | -- [ ] 📚 Documentation update |
11 | | -- [ ] 🔧 Refactoring (no functional changes) |
12 | | -- [ ] ⚡ Performance improvement |
13 | | -- [ ] 🧪 Test addition or update |
14 | | -- [ ] 🔒 Security improvement |
15 | | -- [ ] 🏗️ Build system change |
16 | | -- [ ] 🔄 CI/CD change |
17 | | -- [ ] 🎨 Style change (formatting, missing semi colons, etc; no logic change) |
18 | | -- [ ] 🧹 Chore (updates to dependencies, etc) |
19 | | - |
20 | | -## Scope |
21 | | - |
22 | | -<!-- Select the scope of your changes --> |
23 | | - |
24 | | -- [ ] github-app - GitHub App listener and webhook handling |
25 | | -- [ ] temporal - Temporal.io workflow orchestration |
26 | | -- [ ] claude - Claude AI integration for diagnosis |
27 | | -- [ ] morph - Morph API integration for code patching |
28 | | -- [ ] freestyle - Freestyle testing framework |
29 | | -- [ ] lean - Lean 4 formal verification |
30 | | -- [ ] diagnosis - Root cause analysis engine |
31 | | -- [ ] patching - Automated code patching |
32 | | -- [ ] testing - Test framework and execution |
33 | | -- [ ] proofs - Formal proof generation |
34 | | -- [ ] merge - Auto-merge functionality |
35 | | -- [ ] observability - Metrics, tracing, and monitoring |
36 | | -- [ ] security - Security hardening and compliance |
37 | | -- [ ] docs - Documentation updates |
38 | | -- [ ] deps - Dependency updates |
| 3 | +<!-- Clear description of the change --> |
| 4 | + |
| 5 | +## Type of change |
| 6 | + |
| 7 | +- [ ] Bug fix (non-breaking) |
| 8 | +- [ ] New feature (non-breaking) |
| 9 | +- [ ] Breaking change |
| 10 | +- [ ] Documentation only |
| 11 | +- [ ] Refactor (no behavior change) |
| 12 | +- [ ] Performance |
| 13 | +- [ ] Tests |
| 14 | +- [ ] Security |
| 15 | +- [ ] Build / CI |
| 16 | +- [ ] Style (formatting only) |
| 17 | +- [ ] Chore (deps, tooling) |
| 18 | + |
| 19 | +## Scope (check all that apply) |
| 20 | + |
| 21 | +- [ ] `apps/github-app` |
| 22 | +- [ ] `apps/temporal-worker` |
| 23 | +- [ ] `services/claude` |
| 24 | +- [ ] `services/morph` |
| 25 | +- [ ] `services/freestyle` |
| 26 | +- [ ] `services/lean` |
| 27 | +- [ ] Docs |
| 28 | +- [ ] CI / GitHub Actions |
| 29 | +- [ ] Root tooling or config |
39 | 30 |
|
40 | 31 | ## Testing |
41 | 32 |
|
42 | | -<!-- Describe the tests that you ran to verify your changes --> |
| 33 | +<!-- How you verified the change --> |
43 | 34 |
|
44 | | -- [ ] Unit tests pass |
45 | | -- [ ] Integration tests pass |
46 | | -- [ ] End-to-end tests pass |
47 | | -- [ ] Type checking passes |
48 | | -- [ ] Linting passes |
49 | | -- [ ] Security scans pass |
50 | | -- [ ] Performance benchmarks pass |
| 35 | +- [ ] Unit tests pass (`pnpm test` where relevant) |
| 36 | +- [ ] Typecheck passes (`pnpm type-check` or package `tsc --noEmit`) |
| 37 | +- [ ] Lint passes (`pnpm lint`) |
| 38 | +- [ ] Ran full `pnpm validate` at repo root |
51 | 39 |
|
52 | 40 | ## Checklist |
53 | 41 |
|
54 | | -- [ ] My code follows the style guidelines of this project |
55 | | -- [ ] I have performed a self-review of my own code |
56 | | -- [ ] I have commented my code, particularly in hard-to-understand areas |
57 | | -- [ ] I have made corresponding changes to the documentation |
58 | | -- [ ] My changes generate no new warnings |
59 | | -- [ ] I have added tests that prove my fix is effective or that my feature works |
60 | | -- [ ] New and existing unit tests pass locally with my changes |
61 | | -- [ ] Any dependent changes have been merged and published in downstream modules |
62 | | -- [ ] I have updated the changelog if appropriate |
63 | | -- [ ] I have added/updated security documentation if applicable |
64 | | - |
65 | | -## Breaking Changes |
66 | | - |
67 | | -<!-- If this PR contains breaking changes, describe them here --> |
68 | | - |
69 | | -## Additional Notes |
| 42 | +- [ ] Self-review completed |
| 43 | +- [ ] Documentation updated (README / `docs/` if behavior or env vars changed) |
| 44 | +- [ ] No new avoidable warnings; tests added or updated where appropriate |
70 | 45 |
|
71 | | -<!-- Add any other context about the pull request here --> |
| 46 | +## Breaking changes |
72 | 47 |
|
73 | | -## Related Issues |
| 48 | +<!-- If any, describe migration --> |
74 | 49 |
|
75 | | -<!-- Link to any related issues --> |
| 50 | +## Related issues |
76 | 51 |
|
77 | | -Closes # |
| 52 | +<!-- Closes # --> |
0 commit comments