Rename CosmosMongoDB to AzureDocumentDB - #45
Conversation
Co-authored-by: adamsitnik <6011991+adamsitnik@users.noreply.github.com>
Co-authored-by: adamsitnik <6011991+adamsitnik@users.noreply.github.com>
Co-authored-by: adamsitnik <6011991+adamsitnik@users.noreply.github.com>
Co-authored-by: adamsitnik <6011991+adamsitnik@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Renames the MEVD Cosmos MongoDB provider (packages, namespaces, and tests) to AzureDocumentDB to reflect updated product naming, along with corresponding documentation and CI updates.
Changes:
- Renamed provider surface area from
CosmosMongoDB→AzureDocumentDBacross source, unit tests, and conformance tests. - Updated solution filters/solution manifests to reference the new provider projects.
- Updated documentation and CI workflow matrices/exclusions to use
AzureDocumentDB.
Reviewed changes
Copilot reviewed 52 out of 55 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| README.md | Updates package list/links and live-test note to AzureDocumentDB. |
| MEVD/test/CosmosMongoDB.ConformanceTests/ModelTests/CosmosMongoNoVectorModelTests.cs | Removes old CosmosMongoDB conformance model test (moved/renamed). |
| MEVD/test/CosmosMongoDB.ConformanceTests/ModelTests/CosmosMongoNoDataModelTests.cs | Removes old CosmosMongoDB conformance model test (moved/renamed). |
| MEVD/test/CosmosMongoDB.ConformanceTests/ModelTests/CosmosMongoMultiVectorModelTests.cs | Removes old CosmosMongoDB conformance model test (moved/renamed). |
| MEVD/test/CosmosMongoDB.ConformanceTests/ModelTests/CosmosMongoBasicModelTests.cs | Removes old CosmosMongoDB conformance model test (moved/renamed). |
| MEVD/test/AzureDocumentDB.UnitTests/AzureDocumentDBVectorStoreTests.cs | Renames unit tests/types/usings to AzureDocumentDB provider. |
| MEVD/test/AzureDocumentDB.UnitTests/AzureDocumentDBHotelModel.cs | Renames unit test model type/namespace to AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.UnitTests/AzureDocumentDBCollectionTests.cs | Renames collection unit tests to AzureDocumentDB provider types. |
| MEVD/test/AzureDocumentDB.UnitTests/AzureDocumentDB.UnitTests.csproj | Updates test assembly/root namespace and project reference to AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.UnitTests/.editorconfig | Adds test-project analyzer suppressions for AzureDocumentDB.UnitTests. |
| MEVD/test/AzureDocumentDB.ConformanceTests/TypeTests/AzureDocumentDBKeyTypeTests.cs | Renames key type conformance tests to AzureDocumentDB + updated TestStore. |
| MEVD/test/AzureDocumentDB.ConformanceTests/TypeTests/AzureDocumentDBEmbeddingTypeTests.cs | Renames embedding type conformance tests to AzureDocumentDB + updated TestStore. |
| MEVD/test/AzureDocumentDB.ConformanceTests/TypeTests/AzureDocumentDBDataTypeTests.cs | Renames data type conformance tests to AzureDocumentDB + updated TestStore. |
| MEVD/test/AzureDocumentDB.ConformanceTests/Support/AzureDocumentDBTestStore.cs | Renames conformance TestStore to AzureDocumentDB and updates env var key/error message. |
| MEVD/test/AzureDocumentDB.ConformanceTests/Support/AzureDocumentDBTestEnvironment.cs | Renames test environment helper + configuration keys to AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/Support/AzureDocumentDBFixture.cs | Renames fixture to AzureDocumentDB and wires it to AzureDocumentDBTestStore. |
| MEVD/test/AzureDocumentDB.ConformanceTests/Properties/AssemblyAttributes.cs | Disables test parallelization for AzureDocumentDB conformance tests. |
| MEVD/test/AzureDocumentDB.ConformanceTests/ModelTests/AzureDocumentDBNoVectorModelTests.cs | Adds renamed NoVector model tests for AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/ModelTests/AzureDocumentDBNoDataModelTests.cs | Adds renamed NoData model tests for AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/ModelTests/AzureDocumentDBMultiVectorModelTests.cs | Adds renamed MultiVector model tests for AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/ModelTests/AzureDocumentDBBasicModelTests.cs | Adds renamed Basic model tests for AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/AzureDocumentDBTestSuiteImplementationTests.cs | Renames suite implementation tests class/namespace to AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/AzureDocumentDBIndexKindTests.cs | Renames index-kind tests and updates provider-specific notes to AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/AzureDocumentDBFilterTests.cs | Renames filter tests and updates provider-specific notes to AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/AzureDocumentDBEmbeddingGenerationTests.cs | Renames embedding generation tests and DI registrations to AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/AzureDocumentDBDistanceFunctionTests.cs | Renames distance function tests and provider-specific note to AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/AzureDocumentDBDependencyInjectionTests.cs | Renames DI tests and updates config keys/registration APIs to AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/AzureDocumentDBCollectionManagementTests.cs | Renames collection management tests to AzureDocumentDB fixture. |
| MEVD/test/AzureDocumentDB.ConformanceTests/AzureDocumentDBBsonMappingTests.cs | Renames BSON mapping tests and updates collection/test store usage to AzureDocumentDB. |
| MEVD/test/AzureDocumentDB.ConformanceTests/AzureDocumentDB.ConformanceTests.csproj | Updates conformance project root namespace and project reference to AzureDocumentDB. |
| MEVD/src/CosmosMongoDB/MongoShared/README.md | Removes CosmosMongoDB MongoShared README (relocated/updated). |
| MEVD/src/AzureDocumentDB/README.md | Updates provider README title and install command to AzureDocumentDB package. |
| MEVD/src/AzureDocumentDB/MongoShared/README.md | Adds AzureDocumentDB MongoShared provenance/sync guidance. |
| MEVD/src/AzureDocumentDB/MongoShared/MongoModelBuilder.cs | Updates namespace to AzureDocumentDB for shared Mongo utilities. |
| MEVD/src/AzureDocumentDB/MongoShared/MongoMapper.cs | Updates namespace to AzureDocumentDB for shared Mongo utilities. |
| MEVD/src/AzureDocumentDB/MongoShared/MongoDynamicMapper.cs | Updates namespace to AzureDocumentDB for shared Mongo utilities. |
| MEVD/src/AzureDocumentDB/MongoShared/MongoConstants.cs | Updates namespace to AzureDocumentDB for shared Mongo constants. |
| MEVD/src/AzureDocumentDB/MongoShared/IMongoMapper.cs | Updates namespace to AzureDocumentDB for shared Mongo interfaces. |
| MEVD/src/AzureDocumentDB/MongoShared/ErrorHandlingAsyncCursor.cs | Updates namespace to AzureDocumentDB for shared error-handling cursor wrapper. |
| MEVD/src/AzureDocumentDB/MongoShared/BsonValueFactory.cs | Updates namespace to AzureDocumentDB for BSON factory helper. |
| MEVD/src/AzureDocumentDB/AzureDocumentDBVectorStoreOptions.cs | Renames vector store options type to AzureDocumentDB. |
| MEVD/src/AzureDocumentDB/AzureDocumentDBVectorStore.cs | Renames vector store implementation type and updates internal references. |
| MEVD/src/AzureDocumentDB/AzureDocumentDBServiceCollectionExtensions.cs | Renames DI extension class/methods to AzureDocumentDB and updates registrations. |
| MEVD/src/AzureDocumentDB/AzureDocumentDBFilterTranslator.cs | Renames filter translator to AzureDocumentDB and updates exception messages. |
| MEVD/src/AzureDocumentDB/AzureDocumentDBDynamicCollection.cs | Renames dynamic collection type to AzureDocumentDB. |
| MEVD/src/AzureDocumentDB/AzureDocumentDBConstants.cs | Renames constants type to AzureDocumentDB. |
| MEVD/src/AzureDocumentDB/AzureDocumentDBCollectionSearchMapping.cs | Renames search mapping helpers to AzureDocumentDB. |
| MEVD/src/AzureDocumentDB/AzureDocumentDBCollectionOptions.cs | Renames collection options type to AzureDocumentDB. |
| MEVD/src/AzureDocumentDB/AzureDocumentDBCollectionCreateMapping.cs | Renames create-mapping helpers and updates messages/links to AzureDocumentDB. |
| MEVD/src/AzureDocumentDB/AzureDocumentDBCollection.cs | Renames main collection implementation type to AzureDocumentDB and updates internal references. |
| MEVD/src/AzureDocumentDB/AzureDocumentDB.csproj | Updates assembly/package identity and InternalsVisibleTo target to AzureDocumentDB. |
| MEVD/src/AzureDocumentDB/AssemblyInfo.cs | Adds license header placeholder file for AzureDocumentDB project. |
| MEVD/MEVD.slnf | Replaces CosmosMongoDB projects with AzureDocumentDB projects in solution filter. |
| CommunityToolkit.AI.slnx | Replaces CosmosMongoDB projects with AzureDocumentDB projects in solution manifest. |
| .github/workflows/mevd.yml | Updates CI exclusion matrix/comment to AzureDocumentDB provider name. |
Suppressed comments (8)
MEVD/src/AzureDocumentDB/AzureDocumentDBServiceCollectionExtensions.cs:28
- The NativeAOT/trimming attribute messages are swapped here:
RequiresUnreferencedCodeshould use the trimming message, andRequiresDynamicCodeshould use the NativeAOT message. As-is, tooling will emit misleading warnings.
This issue also appears in the following locations of the same file:
- line 44
- line 73
- line 94
- line 128
- line 149
MEVD/src/AzureDocumentDB/AzureDocumentDBServiceCollectionExtensions.cs:45
- The NativeAOT/trimming attribute messages are swapped here:
RequiresUnreferencedCodeshould use the trimming message, andRequiresDynamicCodeshould use the NativeAOT message. As-is, tooling will emit misleading warnings.
MEVD/src/AzureDocumentDB/AzureDocumentDBServiceCollectionExtensions.cs:74 - The NativeAOT/trimming attribute messages are swapped here:
RequiresUnreferencedCodeshould use the trimming message, andRequiresDynamicCodeshould use the NativeAOT message. As-is, tooling will emit misleading warnings.
MEVD/src/AzureDocumentDB/AzureDocumentDBServiceCollectionExtensions.cs:95 - The NativeAOT/trimming attribute messages are swapped here:
RequiresUnreferencedCodeshould use the trimming message, andRequiresDynamicCodeshould use the NativeAOT message. As-is, tooling will emit misleading warnings.
MEVD/src/AzureDocumentDB/AzureDocumentDBServiceCollectionExtensions.cs:129 - The NativeAOT/trimming attribute messages are swapped here:
RequiresUnreferencedCodeshould use the trimming message, andRequiresDynamicCodeshould use the NativeAOT message. As-is, tooling will emit misleading warnings.
MEVD/src/AzureDocumentDB/AzureDocumentDBServiceCollectionExtensions.cs:150 - The NativeAOT/trimming attribute messages are swapped here:
RequiresUnreferencedCodeshould use the trimming message, andRequiresDynamicCodeshould use the NativeAOT message. As-is, tooling will emit misleading warnings.
MEVD/src/AzureDocumentDB/AzureDocumentDBVectorStore.cs:16 - Grammar: use “an Azure …” (not “a Azure …”).
MEVD/src/AzureDocumentDB/AzureDocumentDBCollection.cs:23 - The XML docs for
TKeysay onlystringis supported, but the implementation supports multiple key types (sees_validKeyTypes). Updating the docs will prevent confusion for API consumers.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Co-authored-by: adamsitnik <6011991+adamsitnik@users.noreply.github.com>
Co-authored-by: adamsitnik <6011991+adamsitnik@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 52 out of 55 changed files in this pull request and generated no new comments.
Suppressed comments (4)
MEVD/src/AzureDocumentDB/DocumentDBVectorStore.cs:16
- XML doc grammar: use “an Azure …” (vowel sound) instead of “a Azure …”.
MEVD/src/AzureDocumentDB/DocumentDBCollection.cs:23 - The TKey type parameter documentation is inaccurate: this collection supports multiple key types (see s_validKeyTypes), not just string.
MEVD/src/AzureDocumentDB/DocumentDBServiceCollectionExtensions.cs:28 - The Requires* attributes are currently wired to the wrong message constants: RequiresUnreferencedCode should reference the trimming message, and RequiresDynamicCode should reference the NativeAOT/dynamic-code message. This makes the warnings misleading (and the same swap likely needs to be applied consistently throughout this file).
MEVD/src/AzureDocumentDB/DocumentDBConstants.cs:9 - DocumentDBConstants.VectorStoreSystemName still reports the old CosmosDB MongoDB identifier. Since this PR renames the provider to AzureDocumentDB, the system name will be misleading in metadata/error reports; consider updating it to a DocumentDB-specific identifier (compare CosmosNoSqlConstants.VectorStoreSystemName = "azure.cosmosdbnosql").
Co-authored-by: adamsitnik <6011991+adamsitnik@users.noreply.github.com>
Reflects most recent product name changes. We can change the package and type names as we have not officially announced this packages yet.