[devops] Vendor Maestro changelog tool - #26461
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull request overview
This PR vendors the Maestro changelog generator into the repo under scripts/changelog and updates the maestro-changelog GitHub Actions workflow to run the local tool instead of cloning spouliot/dotnet-tools.
Changes:
- Add a new
scripts/changelog.NET console tool (README, csproj, and Makefile fragment). - Update
.github/workflows/maestro-changelog.ymltocheckoutthe repo and run the vendored tool viadotnet run. - Integrate the tool with the repo’s script templating mechanism via
fragment.mk.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/changelog/README.md | Documents the new vendored changelog tool and its invocation. |
| scripts/changelog/Program.cs | Implements the changelog generator logic (diff parsing + nested dependency traversal). |
| scripts/changelog/fragment.mk | Adds Makefile template integration for the script. |
| scripts/changelog/changelog.csproj | Defines the tool project targeting net$(BundledNETCoreAppTargetFrameworkVersion). |
| .github/workflows/maestro-changelog.yml | Switches workflow to use the vendored tool instead of cloning an external repo. |
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Suppressed comments (1)
.github/workflows/maestro-changelog.yml:26
⚠️ This workflow installs .NET 9 explicitly, but the vendored tool targetsnet$(BundledNETCoreAppTargetFrameworkVersion). In this repoglobal.jsonpins a .NET 10 preview SDK, so this job is likely to fail to build/run the tool. Useglobal-json-file: ./global.json(as other workflows do) so the action uses the repo-pinned SDK.
- uses: actions/setup-dotnet@a98b56852c35b8e3190ac28c8c2271da59106c68 # v6.0.0
with:
dotnet-version: '9'
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.
Suppressed comments (1)
Previously missed (1) — in code that hasn't changed since the last review.
scripts/changelog/Program.cs:14
- 🤖 💡 suggestion Code organization — These lists are initialized once and never reassigned; marking them
readonlyprevents accidental reassignment and makes the intent clearer.
Rule: Prefer immutable fields when possible
static List<string> list = new ();
static List<string> filters = new ();
| cd dotnet-tools/changelog | ||
| dotnet build | ||
| ./bin/Debug/net9.0/changelog https://github.com/$GITHUB_REPOSITORY/pull/${GITHUB_REF_NAME/\/*/} > /tmp/changelog.txt 2>&1 | ||
| dotnet run --project scripts/changelog/changelog.csproj --verbosity quiet -- https://github.com/$GITHUB_REPOSITORY/pull/${GITHUB_REF_NAME/\/*/} /tmp/changelog.txt |
This comment has been minimized.
This comment has been minimized.
✅ API diff for current PR / commitNET (empty diffs)✅ API diff vs stableNET (empty diffs)ℹ️ Generator diffGenerator Diff: vsdrops (html) vsdrops (raw diff) gist (raw diff) - Please review changes) Pipeline on Agent |
This comment has been minimized.
This comment has been minimized.
🚀 [CI Build #80c6b5c] Test results 🚀Test results✅ All tests passed on VSTS: test results. 🎉 All 205 tests passed 🎉 Tests counts✅ assembly-processing: All 1 tests passed. Html Report (VSDrops) Download macOS tests✅ Tests on macOS Monterey (12): All 5 tests passed. Html Report (VSDrops) Download Linux Build VerificationPipeline on Agent |
Replace the external dotnet-tools clone in the Maestro changelog workflow with a vendored scripts/changelog project.
The workflow now checks out the repository and runs the local tool instead of cloning spouliot/dotnet-tools.
🤖 Pull request created by Copilot