Skip to content

CosmosDb ChangeFeed Support#454

Open
swapnil-nagar wants to merge 1 commit into
v4.xfrom
swapnil/CosmosDBUpdate
Open

CosmosDb ChangeFeed Support#454
swapnil-nagar wants to merge 1 commit into
v4.xfrom
swapnil/CosmosDBUpdate

Conversation

@swapnil-nagar

Copy link
Copy Markdown
Contributor

Adds Cosmos DB Change Feed mode support and validation so changeFeedMode is surfaced in types and preserved through function registration metadata.

Changes

  • Added CosmosDBv4ChangeFeedMode type with:
    • LatestVersion
    • AllVersionsAndDeletes
  • Added changeFeedMode to Cosmos DB v4 trigger options.
  • Re-exported CosmosDBChangeFeedMode from shared Cosmos DB types.
  • Updated Cosmos DB API docs to clarify default behavior and how to enable full change feed processing.
  • Added tests for:
    • Metadata conversion path (toCoreFunctionMetadata)
    • End-to-end app registration path (app.cosmosDB)

Version

  • Bumped package version to 4.16.1.

@swapnil-nagar swapnil-nagar requested a review from a team as a code owner June 19, 2026 06:18
@jviau

jviau commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

I am not familiar with E2E-test situation for this repo, but want to make sure the feature is validated either manually or by tests. A specific scenario to validate for is the incoming payload type when using AllVersionsAndDelete. With LatestVersion it will be T, for AVAD it will be ChangeFeedItem<T>. So the JSON representation of this CSharp class: https://learn.microsoft.com/en-us/dotnet/api/microsoft.azure.cosmos.changefeeditem-1?view=azure-dotnet-preview

@TsuyoshiUshio

TsuyoshiUshio commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

+1 for the Jacob's comment. Change looks good. E2E might be required.
For E2E testing with this change, If you've already done the manual E2E, I'd happy to approve. But I'm wondering if we can write E2E testing for nodejs-library/worker/extensions if one of them are changed, We can use https://github.com/Azure/azure-functions-nodejs-e2e-tests .

Comment thread package-lock.json
"integrity": "sha512-QMUezzXWII9EV5aTFXW1UBVUO77wYPpjqIF8/AviUCThNeSYZykpoTixUeaNNBwmCev0AMDWMAni+f8Hxb1IFw==",
"dev": true
},
"node_modules/fsevents": {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: This is not blocking issue. This library is dev dependency for MacOS. No harm but it is reviewing noise. :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants