[Cosmos] Add .pragentrules for PR review automation#48777
Closed
jeet1995 wants to merge 1 commit intoAzure:mainfrom
Closed
[Cosmos] Add .pragentrules for PR review automation#48777jeet1995 wants to merge 1 commit intoAzure:mainfrom
jeet1995 wants to merge 1 commit intoAzure:mainfrom
Conversation
Add review rules for sdk/cosmos/ that the PR Deep Reviewer agent consumes as mandatory constraints. These encode Cosmos-specific knowledge the agent cannot infer on its own: - @beta annotation gating for preview APIs (addresses PR Azure#47566 miss) - Public enum extensibility scrutiny - Azure SDK Java central guidelines check (new client surface only) - Java 8 baseline compatibility - Suppressions: CHANGELOG per-PR, API contract artifacts, CheckStyle disabling - Domain emphasis: multi-region failover, retry/timeout criticality Modeled after the .NET Cosmos SDK precedent (Azure/azure-cosmos-dotnet-v3). Tested with pragentrules-resolver against PR Azure#47566 changed files. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Member
Author
|
Closing — moving .pragentrules to the cosmos-sdk-copilot-toolkit repo instead of placing it in each SDK repo. The PR Deep Reviewer agent will load rules from the toolkit as a fallback. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds
sdk/cosmos/.pragentrules— review rules consumed by the PR Deep Reviewer agent as mandatory constraints. These encode Cosmos-specific knowledge the agent cannot infer on its own.Why
PR #47566 merged a new public enum (
QuantizerType) and public methods onCosmosVectorIndexSpecwithout@Betaannotation, even though vector search is a preview feature. No existing automation caught this — not CheckStyle, not Revapi, not the PR review agent.These rules close that gap. Modeled after the .NET Cosmos SDK precedent.
Rules Added
@Beta— asks if service feature is GATesting
Ran
pragentrules-resolveragainst PR #47566 changed files:QuantizerType.java,CosmosVectorIndexSpec.java,CHANGELOG.md,VectorIndexTest.javasdk/cosmos/.pragentrulesSafety
.pragentrulesare consumed only by the PR Deep Reviewer agent — zero CI impact