Skip to content

Add Chronus CI workflow and update changelog docs to reference azpysdk changelog#46585

Draft
Copilot wants to merge 4 commits intomainfrom
copilot/update-changelog-for-azpysdk
Draft

Add Chronus CI workflow and update changelog docs to reference azpysdk changelog#46585
Copilot wants to merge 4 commits intomainfrom
copilot/update-changelog-for-azpysdk

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 28, 2026

Summary

This PR delivers what #45803 originally set out to do — CI enforcement of Chronus change descriptions — updated to reflect the azpysdk changelog command that has since been added to the repo.

What changed since PR #45803

Area #45803 This PR
Package location .github/package.json .github/chronus/package.json (already in repo)
Package manager pnpm npm
Lockfile .github/pnpm-lock.yaml .github/chronus/package-lock.json (already in repo)
User-facing tool npx chronus add azpysdk changelog add (with npx chronus as fallback)

Changes

.github/workflows/chronus-verify.yml (new)

  • Triggers on PRs to main that touch sdk/*/*/**
  • Skips runs from azure-sdk bot (automated release PRs)
  • Installs pinned Chronus from .github/chronus/package-lock.json via npm ci
  • Runs chronus verify from the repo root
  • On failure, prints a helpful error pointing to azpysdk changelog add and the docs

doc/dev/changelog_updates.md (updated)

  • Recommends azpysdk changelog (add / verify / status / create) as the primary tool
  • Keeps npx chronus commands as alternatives for users without azpysdk installed
  • Adds a note that the CI workflow enforces verification on every qualifying PR

On chronus-verify failure, post a sticky PR comment instructing the
contributor to comment '/chronus add [kind]' for a one-click fix.

Add chronus-fix.yml: an issue_comment-triggered workflow that
verifies commenter permissions, rejects fork PRs (security), parses
the requested kind, runs 'chronus add <package> --kind X --message
<PR title>' for each missing package, and pushes the result back to
the PR branch.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
…ghter perms

- Pin actions/checkout, setup-node, github-script to SHA + version comment.
- chronus-fix: restore .github/chronus from base branch before npm ci/run,
  so PR-head changes to the tooling cannot execute under the write token.
- persist-credentials: false on both checkouts; push uses an explicit
  x-access-token URL only at push time.
- Permissions: verify drops pull-requests:write for issues:write only;
  fix drops pull-requests:write to pull-requests:read.
- Slash command parsed by strict regex; rejects /chronus additive etc.
- Add concurrency groups on both workflows.
- Consolidate 4 small github-script steps into one auth+metadata step.
- Merge 4 terminal reply steps into one always() switch step.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
l0lawrence pushed a commit that referenced this pull request May 1, 2026
Touches sdk/template/azure-template without a chronus changelog entry to
confirm the new chronus-verify workflow posts the sticky one-click-fix
comment. This branch will be closed without merging.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.

2 participants