Cherry-Pick: Add changes to create Nuget Core Package#3602
Open
RubenCerna2079 wants to merge 4 commits into
Open
Cherry-Pick: Add changes to create Nuget Core Package#3602RubenCerna2079 wants to merge 4 commits into
RubenCerna2079 wants to merge 4 commits into
Conversation
## Why make this change? - Resolves issue #3469 ## What is this change? - `Cli.csproj`: Changes package description and tags to include new MCP endpoint. - `Core.csproj`: Adds authoring requirements based on the following link https://devdiv.visualstudio.com/DevDiv/_wiki/wikis/DevDiv.wiki/11557/Publish-as-Microsoft?anchor=publishing-microsoft-packages-on-nuget.org ## How was this tested? - [ ] Integration Tests - [ ] Unit Tests No tests are necessary for this change ## Sample Request(s) N/A
…ent (#3475) ## Why make this change? `nuget/README.md` (the README shown on the NuGet package page) was minimal and stale — it described only REST and GraphQL, omitted the MCP endpoint entirely and other content present in the root `README.md`. ## What is this change? Rewrote `nuget/README.md` to match the quality and completeness of the root `README.md`, adapted for the NuGet context: - **Badges** — docs link, license - **About** — updated to cover REST, GraphQL, and MCP endpoints - **Features** — added MCP section: DML tools, custom tools via stored procedures, configurable path (`/mcp` default), AI client compatibility (GitHub Copilot, Azure AI Foundry) - **Resources** — aligned with root README; links use absolute GitHub URLs ## How was this tested? - [ ] Integration Tests - [ ] Unit Tests Documentation-only change; no code logic modified. ## Sample Request(s) N/A --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: RubenCerna2079 <32799214+RubenCerna2079@users.noreply.github.com> Co-authored-by: Ruben Cerna <rcernaserna@microsoft.com> Co-authored-by: Aniruddh Munde <anmunde@microsoft.com>
## Why make this change? - Solves issue #3479 ## What is this change? - Changes `Cli.proj` and `Core.proj` files to retrieve their unique NOTICE files from their respective new path. - Changes `notice-generation.ps1` script to use the NOTICE file path from the pipeline as an argument instead of having it hard coded. - Changes `create-manifest-file.ps1` script to add the new nuget Core package that will be created in the pipeline. ## How was this tested? This is tested alongside the changes from the [pipeline](https://msdata.visualstudio.com/CosmosDB/_git/data-api-builder/pullrequest/2097650) ## Sample Request(s) N/A --------- Co-authored-by: Aniruddh Munde <anmunde@microsoft.com>
RubenCerna2079
changed the title
RubenCerna2079
requested review from
Alekhya-Polavarapu,
Aniruddh25,
JerryNixon,
aaronburtle,
anushakolan,
rusamant,
sourabh1007,
souvikghosh04,
stuartpa and
vadeveka
as code owners
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Cherry-picks prior changes to enable building and publishing a new “Core” NuGet package alongside the existing CLI/package artifacts.
Changes:
- Add NuGet package metadata and packing assets for
Microsoft.DataApiBuilder.Core. - Update CLI NuGet metadata (tags/description) and adjust NOTICE file packing location.
- Update release/packaging scripts to support multiple NuGet artifacts and external NOTICE generation; add/update NuGet READMEs.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/Core/Azure.DataApiBuilder.Core.csproj | Adds NuGet packaging metadata and packs README/icon/license/notice for the new Core package. |
| src/Cli/Cli.csproj | Updates package tags/description for MCP and adjusts NOTICE file include path for packing. |
| scripts/notice-generation.ps1 | Makes notice output path a required parameter (caller-controlled) and adjusts content rewriting. |
| scripts/create-manifest-file.ps1 | Adds hash + URL metadata for separate CLI and Core NuGet packages in the release manifest. |
| nuget/nuget_core/README.md | Introduces README content for the new Core NuGet package. |
| nuget/README.md | Updates NuGet README to mention MCP, adds docs links and “getting started” info. |
Comments suppressed due to low confidence (1)
nuget/nuget_core/README.md:1
- The
�characters indicate an encoding/copy issue and will render incorrectly on NuGet/GitHub. Replace them with the intended punctuation (commonly an em dash—or a hyphen-) and ensure the file is saved as UTF-8.
# Data API builder Core Library for Azure Databases
Aniruddh25
approved these changes
May 19, 2026
RubenCerna2079
changed the title
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Aniruddh25
approved these changes
May 20, 2026
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.
Why make this change?
This change cherry-picks the PRs that allow us to create the new Nuget Core Package
What is this change?
Cherry-picked PRs
patterns.namedefault value #3472How was this tested?
Existing tests in cherry-pick cover new changes.
Sample Request(s)
N/A