Update copilot-instructions.md

Author: zhoxing-ms

.github/copilot-instructions.md

@@ -176,6 +176,37 @@ The repository enforces several checks before code integration:
 - Recording issues: Ensure proper test isolation
 - Coverage issues: Use `azdev cmdcov` to identify gaps
 
+## Specific Coding Guidelines for PR review
+
+### 1. Rule of help messages
+---
+applyTo: "src/azure-cli/azure/cli/command_modules/**/_params.py"
+---
+Firstly, find the help messages, their format is `...help='{help_message}'...` or `....help="{help_message}"...`, {help_message} is the placeholder
+Next, you need to review the format of the found help messages according to the following rules:
+1. If the help message does not start with a verb, skip this check. Otherwise, check the verb at the beginning of help message
+2. Confirm if the verb is in the first person voice, such as the third person voice of 'Enables......' is not compliant
+3. Confirm if the verb is in the active voice, such as the passive voice 'Enabled......' is not compliant
+
+### 2. Confirmation for dangerous commands
+---
+applyTo: "src/azure-cli/azure/cli/command_modules/**/commands.py"
+---
+Firstly, find the command signature, their format is `... g.custom_command('{command_signature}', ...)...` or `... g.command('{command_signature}', ...)...`, {command_signature} is the command signature
+Next, you need to determine whether the command signature is a dangerous operation, such as `delete`, `remove`, `stop` operation which will break or remove resources. If so, You need to suggest adding the flag `confirmation=True`, such as `g.custom_command('{command_signature}', ..., )...`
+
+### 3. Common code best practices
+- ​​Use 4 spaces for indentation​​, never mix tabs and spaces
+​​- Naming:
+    (1) Variables/functions: snake_case
+    (2) Classes: CamelCase
+    (3) Constants: UPPER_CASE_WITH_UNDERSCORES
+- ​​Surround operators with spaces​​ (x = y + 1), except in function parameters
+- ​​Avoid extraneous whitespace​​ in brackets, before commas/colons
+- ​​Handle exceptions explicitly​​ using try-except blocks
+- ​​Break long expressions​​ using parentheses alignment or hanging indents
+​​- Use blank lines​​: two between top-level definitions, one between methods
+
 ## Important Notes for Agents
 
 1. **ALWAYS use azdev**: Never use the deprecated `dev_setup.py` script
@@ -186,4 +217,4 @@ The repository enforces several checks before code integration:
 6. **Incremental testing**: Use module-specific tests during development, not full suite
 7. **Network resilience**: Be prepared for pip timeout issues and retry with longer timeouts
 
-This repository has complex build requirements but following these instructions will minimize exploration time and avoid common pitfalls. The development workflow is well-established through azdev tooling.
+This repository has complex build requirements but following these instructions will minimize exploration time and avoid common pitfalls. The development workflow is well-established through azdev tooling.