Skip to content

Commit 97dbf1e

Browse files
committed
Fix the Q Developer rules
1 parent 8fff29a commit 97dbf1e

File tree

1 file changed

+4
-14
lines changed

1 file changed

+4
-14
lines changed

.amazonq/rules/review-rules.md

Lines changed: 4 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -3,22 +3,12 @@
33
- Comments should be provided in Japanese by default, while maintaining English-based technical analysis to ensure review quality.
44
- When you find a problem, suggest a solution.
55
- If you find a problem that is not in the code, point it out.
6+
7+
## Security review
8+
69
- Always evaluate security implications of code changes.
710
- Check for potential vulnerabilities and secure coding practices.
8-
- Check for hardcoded credentials, API keys, and sensitive data
911
- Verify no hardcoded credentials or secrets
1012
- Ensure proper input validation and sanitization
1113
- Check dependencies for known vulnerabilities
12-
- Verify proper input validation and sanitization
13-
- Ensure secure communication protocols (HTTPS, TLS)
14-
- Review permission and access control implementations
15-
- Check for proper error handling of security-related operations
16-
- Validate secure storage of sensitive data
17-
- Check for proper WebView configuration.
18-
- Secure data storage using Android Keystore, and safe handling of Intent data.
19-
- Since this is a Device Flow client, please verify OAuth 2.0 Device Flow-specific security considerations.
20-
- Ensure that proper PKCE implementation is in place
21-
- Ensure that secure token handling is implemented
22-
- Ensure that state verification is performed
23-
- Verify secure random number generation for state and PKCE values
24-
- Check for proper timing attack mitigations in token comparisons
14+
- Use GitHub Dependabot alerts and `pnpm audit`

0 commit comments

Comments
 (0)