.github/pull_request_template.md

1## Summary
2
3_What does this PR do? One or two sentences._
4
5## Motivation
6
7_Why is this change needed? Link to the relevant issue if applicable (e.g. Closes #123)._
8
9## Changes
10
11-
12-
13
14## Checklist
15
16### Build & tests
17- [ ] `swift build` passes with no errors or warnings
18- [ ] Existing tests pass (`swift test` or ⌘U in Xcode)
19- [ ] New tests added for any new behaviour (especially crypto changes)
20
21### Protocol & compatibility
22- [ ] Wire format is backward-compatible, or a versioning mechanism is included
23- [ ] No new server dependencies introduced
24- [ ] Android interoperability preserved (if touching the wire protocol)
25
26### Security
27- [ ] Crypto changes reviewed against the relevant spec (Signal, RFC 9420, etc.)
28- [ ] No private key material logged or exposed in error messages
29- [ ] New Keychain entries use `kSecAttrAccessibleWhenUnlockedThisDeviceOnly`
30- [ ] No new external dependencies added without justification
31
32### Code quality
33- [ ] All Swift code compiles under strict concurrency (`-strict-concurrency=complete`)
34- [ ] No force-unwraps (`!`) on untrusted external data
35- [ ] `private` access used where the symbol does not need cross-file visibility
36
37## Testing notes
38
39_How did you test this? Physical devices used, peer count, any edge cases exercised?_
40
41## Screenshots / recording (if UI change)
42
43_Paste screenshots or a short screen recording here._