Improve the instructions for Github Copilot code review(GCCR)

[MaryGao]

Improve the instructions for Github code review with more fine-grained instructions.

GCCR agent could allow custom instructions under .github/instructions/**/*.instructions.md https://docs.github.com/en/copilot/how-tos/use-copilot-agents/request-a-code-review/use-code-review#customizing-copilots-reviews-with-custom-instructions and we could have improved instructions with this functionality.

• Go through 10+ existing reviewed PRs for mgmt SDKs
• Go throught 10+ PRs for data-plane SDKs
• Improvement area
  • List valid cases and invalid cases
  • List rules with files
  • List with comment section?

[MaryGao]

Step 1 - Go through 10+ existing reviewed PRs for mgmt SDKs
#37035 (comment)
Generation from TypeSpec

• [AutoPR @azure-arm-resourcesdeploymentstacks]-generated-from-SDK Generation - JS-5803515 #37171
• [AutoPR @azure-arm-edgemarketplace]-generated-from-SDK Generation - JS-5751507 #37035
• [AutoPR @azure-arm-managedops]-generated-from-SDK Generation - JS-5788708 #37142
• [AutoPR @azure-arm-containerregistry]-generated-from-SDK Generation - JS-5777148 #37083
• [AutoPR @azure-arm-resourceconnector]-generated-from-SDK Generation - JS-5737605 #37005
• [AutoPR @azure-arm-dell-storage]-generated-from-SDK Generation - JS-5707781 #36943

Generation from Swagger

• [mgmt] communication release #37062
• [mgmt] compute release #37051

Identify some issues:

• wrongly evaluating code consistency for baseUrl([AutoPR @azure-arm-resourcesdeploymentstacks]-generated-from-SDK Generation - JS-5803515 #37171 (review), https://github.com/Azure/azure-sdk-for-js/agents/pull/37171?session_id=44e15632-819c-4078-8bad-49958bd5fab9)
• correctly identify the un-existing files([AutoPR @azure-arm-edgeactions]-generated-from-SDK Generation - JS-5759847 #37048 (comment), https://github.com/Azure/azure-sdk-for-js/agents/pull/37048?session_id=b78f4509-28bd-43bf-b1f4-968ecf0b0879)
• wrong comment with api version enum value([AutoPR @azure-arm-edgemarketplace]-generated-from-SDK Generation - JS-5751507 #37035 (comment))
• correctly identify a typo issue([AutoPR @azure-arm-edgemarketplace]-generated-from-SDK Generation - JS-5751507 #37035 (comment), https://github.com/Azure/azure-sdk-for-js/agents/pull/37035?session_id=51d6fd09-8eee-4240-9ada-873162674094)
• codegen issue found([AutoPR @azure-arm-managedops]-generated-from-SDK Generation - JS-5788708 #37142 (comment))
• duplicated comments among samples-dev and samples: [AutoPR @azure-arm-managedops]-generated-from-SDK Generation - JS-5788708 #37142 (comment)

[MaryGao]

Findings with Copilot code review
Pros
Unstablitly/undeterministic - Some issues would be shared ones among packages but only found in one package

Missing comments - missing to report some issues due to no domain knowledge of Azure SDK domain area

Wrongly reported - wronly reported issues due to not understand underlying codesnippet

Unuseful comments - a lot of comments for low priority comments for mgmt samples

Logging - we don't know when and how CCR uses our customizated instructions so we don't know how to improve that

Cons
Could find real and general bugs