Skip to content

Commit 785aff6

Browse files
committed
Update review areas to include test files.
1 parent 9853da8 commit 785aff6

1 file changed

Lines changed: 13 additions & 13 deletions

File tree

.claude/commands/review-areas.md

Lines changed: 13 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -46,19 +46,19 @@ Match each changed file against the table. A file may match more than one review
4646

4747
| Reviewer | Path patterns |
4848
|---|---|
49-
| `bson-reviewer` | `src/MongoDB.Bson/**`; any file elsewhere implementing `IBsonSerializer<T>` (grep for it if you suspect one) |
50-
| `transport-reviewer` | `src/MongoDB.Driver/Core/{Clusters,Servers,Connections,ConnectionPools,WireProtocol,Compression,Configuration,Misc}/**` |
51-
| `operations-reviewer` | `src/MongoDB.Driver/Core/Operations/**`; `src/MongoDB.Driver/Core/Bindings/**` (incl. `CoreSession*.cs`, `CoreTransaction*.cs`); `src/MongoDB.Driver/ClientSession*.cs` |
52-
| `auth-reviewer` | `src/MongoDB.Driver/Authentication/**`; `src/MongoDB.Driver.Authentication.AWS/**`; `src/MongoDB.Driver/MongoCredential.cs`; `src/MongoDB.Driver/Core/Connections/ConnectionInitializer.cs` |
53-
| `client-api-reviewer` | `src/MongoDB.Driver/{MongoClient,MongoDatabase,MongoClientSettings,MongoCredential,IMongoClient,IMongoDatabase,IMongoCollection}.cs`; `src/MongoDB.Driver/MongoCollectionImpl*.cs`; `src/MongoDB.Driver/MongoUrl*.cs`; `src/MongoDB.Driver/ServerApi*.cs` |
54-
| `builders-reviewer` | `src/MongoDB.Driver/Builders.cs`; `src/MongoDB.Driver/{FilterDefinition,UpdateDefinition,ProjectionDefinition,SortDefinition,IndexKeysDefinition,ArrayFilterDefinition,PipelineDefinition,PipelineStageDefinition,FieldDefinition,SetFieldDefinitions}*.cs` |
55-
| `aggregation-reviewer` | `src/MongoDB.Driver/{Aggregate,AggregateFluent,IAggregateFluent,ChangeStream,AggregateHelper,ChangeStreamHelper,AggregateExpressionDefinition}*.cs`; `src/MongoDB.Driver/Core/Operations/{AggregateOperation,AggregateToCollectionOperation,ChangeStreamOperation,ChangeStreamCursor}*.cs` |
56-
| `linq-reviewer` | `src/MongoDB.Driver/Linq/**` |
57-
| `gridfs-reviewer` | `src/MongoDB.Driver/GridFS/**` |
58-
| `search-reviewer` | `src/MongoDB.Driver/Search/**`; `src/MongoDB.Driver/{QueryVector,VectorSearchOptions,BinaryVectorExtensions,SearchIndexType,CreateSearchIndexModel,CreateVectorSearchIndexModel*,CreateAutoEmbeddingVectorSearchIndexModel}*.cs` |
59-
| `encryption-reviewer` | `src/MongoDB.Driver.Encryption/**`; `src/MongoDB.Driver/Encryption/**` |
60-
| `diagnostics-reviewer` | `src/MongoDB.Driver/Core/Events/**`; `src/MongoDB.Driver/Core/Logging/**` |
61-
| `geojson-reviewer` | `src/MongoDB.Driver/GeoJsonObjectModel/**` |
49+
| `bson-reviewer` | `src/MongoDB.Bson/**`; `tests/MongoDB.Bson.Tests/**`; any file elsewhere implementing `IBsonSerializer<T>` (grep for it if you suspect one) |
50+
| `transport-reviewer` | `src/MongoDB.Driver/Core/{Clusters,Servers,Connections,ConnectionPools,WireProtocol,Compression,Configuration,Misc}/**`; `tests/MongoDB.Driver.Tests/Core/{Clusters,Servers,Connections,ConnectionPools,WireProtocol,Compression,Configuration,Misc}/**` |
51+
| `operations-reviewer` | `src/MongoDB.Driver/Core/Operations/**`; `src/MongoDB.Driver/Core/Bindings/**` (incl. `CoreSession*.cs`, `CoreTransaction*.cs`); `src/MongoDB.Driver/ClientSession*.cs`; `tests/MongoDB.Driver.Tests/Core/Operations/**`; `tests/MongoDB.Driver.Tests/Core/Bindings/**` |
52+
| `auth-reviewer` | `src/MongoDB.Driver/Authentication/**`; `src/MongoDB.Driver.Authentication.AWS/**`; `src/MongoDB.Driver/MongoCredential.cs`; `src/MongoDB.Driver/Core/Connections/ConnectionInitializer.cs`; `tests/MongoDB.Driver.Tests/Authentication/**`; `tests/MongoDB.Driver.Tests/Communication/Security/**` |
53+
| `client-api-reviewer` | `src/MongoDB.Driver/{MongoClient,MongoDatabase,MongoClientSettings,MongoCredential,IMongoClient,IMongoDatabase,IMongoCollection}.cs`; `src/MongoDB.Driver/MongoCollectionImpl*.cs`; `src/MongoDB.Driver/MongoUrl*.cs`; `src/MongoDB.Driver/ServerApi*.cs`; `tests/MongoDB.Driver.Tests/MongoClient*.cs`; `tests/MongoDB.Driver.Tests/MongoDatabase*.cs`; `tests/MongoDB.Driver.Tests/MongoCollection*.cs`; `tests/MongoDB.Driver.Tests/IMongoClient*.cs`; `tests/MongoDB.Driver.Tests/IMongoDatabase*.cs`; `tests/MongoDB.Driver.Tests/IMongoCollection*.cs`; `tests/MongoDB.Driver.Tests/MongoUrl*.cs`; `tests/MongoDB.Driver.Tests/MongoCredential*.cs`; `tests/MongoDB.Driver.Tests/MongoIndex*.cs` |
54+
| `builders-reviewer` | `src/MongoDB.Driver/Builders.cs`; `src/MongoDB.Driver/{FilterDefinition,UpdateDefinition,ProjectionDefinition,SortDefinition,IndexKeysDefinition,ArrayFilterDefinition,PipelineDefinition,PipelineStageDefinition,FieldDefinition,SetFieldDefinitions}*.cs`; `tests/MongoDB.Driver.Tests/{FilterDefinition,UpdateDefinition,ProjectionDefinition,SortDefinition,IndexKeysDefinition,PipelineDefinition,PipelineStageDefinition,PipelineUpdateDefinition,RenderDollarForm,FindFluent,IFindFluent}*.cs` |
55+
| `aggregation-reviewer` | `src/MongoDB.Driver/{Aggregate,AggregateFluent,IAggregateFluent,ChangeStream,AggregateHelper,ChangeStreamHelper,AggregateExpressionDefinition}*.cs`; `src/MongoDB.Driver/Core/Operations/{AggregateOperation,AggregateToCollectionOperation,ChangeStreamOperation,ChangeStreamCursor}*.cs`; `tests/MongoDB.Driver.Tests/{Aggregate,AggregateFluent,IAggregateFluentExtensions,ChangeStream,NoPipelineInput}*.cs` |
56+
| `linq-reviewer` | `src/MongoDB.Driver/Linq/**`; `tests/MongoDB.Driver.Tests/Linq/**` |
57+
| `gridfs-reviewer` | `src/MongoDB.Driver/GridFS/**`; `tests/MongoDB.Driver.Tests/GridFS/**` |
58+
| `search-reviewer` | `src/MongoDB.Driver/Search/**`; `src/MongoDB.Driver/{QueryVector,VectorSearchOptions,BinaryVectorExtensions,SearchIndexType,CreateSearchIndexModel,CreateVectorSearchIndexModel*,CreateAutoEmbeddingVectorSearchIndexModel}*.cs`; `tests/MongoDB.Driver.Tests/Search/**`; `tests/MongoDB.Driver.Tests/{QueryVector,Rerank}*.cs` |
59+
| `encryption-reviewer` | `src/MongoDB.Driver.Encryption/**`; `src/MongoDB.Driver/Encryption/**`; `tests/MongoDB.Driver.Encryption.Tests/**`; `tests/MongoDB.Driver.Tests/Encryption/**` |
60+
| `diagnostics-reviewer` | `src/MongoDB.Driver/Core/Events/**`; `src/MongoDB.Driver/Core/Logging/**`; `tests/MongoDB.Driver.Tests/Core/Events/**`; `tests/MongoDB.Driver.Tests/Core/Logging/**` |
61+
| `geojson-reviewer` | `src/MongoDB.Driver/GeoJsonObjectModel/**`; `tests/MongoDB.Driver.Tests/GeoJsonObjectModel/**` |
6262
| `spec-conformance-reviewer` | `tests/MongoDB.Driver.Tests/Specifications/**`; `tests/MongoDB.Driver.TestHelpers/**`; `tests/MongoDB.Bson.TestHelpers/**`; `tests/MongoDB.TestHelpers/**` |
6363

6464
**Meta-mapping for reviewer definitions:** a change to `.claude/agents/<name>-reviewer.md` is reviewed by that same reviewer (e.g. `.claude/agents/bson-reviewer.md``bson-reviewer`, `.claude/agents/security-reviewer.md``security-reviewer`). The reviewer is best placed to judge whether its own brief still accurately characterizes the area. Cross-cutting reviewer definitions map to themselves the same way. Exception: `pr-summary-reviewer` only runs in external PR mode, so a local-range change to `.claude/agents/pr-summary-reviewer.md` won't be self-reviewed — that file's review happens when the PR is run through `/review-areas <PR#>`.

0 commit comments

Comments
 (0)