diff --git a/.github/pull_request_template_v2.md b/.github/pull_request_template_v2.md new file mode 100644 index 0000000000..86953dda0a --- /dev/null +++ b/.github/pull_request_template_v2.md @@ -0,0 +1,128 @@ +# Context + +### Spec PR + +Zipstack/unstract-review-context#____ + +### Summary + + +### Implementation Ticket + + +--- + +# Implementation Plan + +## Exit Criteria (Definition of Done) + + +- [ ] All CI checks pass +- [ ] Acceptance criteria from spec PR verified +- [ ] Documentation updated (if applicable) +- [ ] No regressions introduced + +## Implementation Constraints + + +## Sequencing & Dependencies + + +- Depends on: #____ +- Blocks: #____ + +## Verification Plan + + +- [ ] Unit tests added/updated +- [ ] Integration tests covering: ... +- [ ] Performance tests (if applicable): ... +- [ ] Manual verification steps: ... + +## Rollout Strategy + + +- [ ] Feature flag: `flag_name` +- [ ] Migration required: [ ] reversible [ ] irreversible +- [ ] Canary / staged rollout plan: ... +- [ ] Monitoring / alerting in place for: ... + +--- + +# Testing Evidence + +## Test Results + + +## Risk Assessment + + +| Risk | Likelihood | Impact | Mitigation | +|------|------------|--------|------------| +| | | | | + +--- + + + + + + + + +# Spec Compliance Report + +## Verdict + + +## Spec PR Linkage + + +## Compliance Checks + +### Product Requirements + +#### Acceptance Criteria + + +#### Scope Definition + + +#### Non-Functional Requirements + + +#### Dependencies + + +#### Assumptions & Constraints + + +### Specification + +#### Architecture Decision Records (ADRs) + + +#### API / Interface Contracts + + +#### Data Model & State Ownership + + +#### Failure Modes & Resilience + + +#### Security Considerations + + +#### Implementation Constraints + + +## Conflicts (if any) + +| # | Artifact | File:Line | Violation | Severity | +|---|----------|-----------|-----------|----------| +| | | | | | + +## Recommendation + +