Skip to content

Refactor release flow to PR-time Dependabot changesets - #56

Open
TylerJDev with Copilot wants to merge 18 commits into
mainfrom
copilot/copilotimplement-changesets-release-flow
Open

Refactor release flow to PR-time Dependabot changesets#56
TylerJDev with Copilot wants to merge 18 commits into
mainfrom
copilot/copilotimplement-changesets-release-flow

Conversation

Copilot AI commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

This PR automatically adds patch changesets to qualifying Dependabot updates, allowing our usual @changesets release flow to follow our other repositories, such as primer/react.

  • Dependabot changeset automation

    • Added a pull_request_target workflow scoped to dependabot[bot].

    • Uses pinned dependabot/fetch-metadata with alert-lookup: true.

    • Compares base/head package.json files without executing PR code.

    • Creates, updates, or removes one deterministic changeset:

      .changeset/dependabot-<pr-number>.md
    • Adds a patch changeset when a Dependabot PR resolves a security alert or changes a direct production dependency range.

  • Human PR changeset policy

    • Added a PR check requiring .changeset/*.md for non-Dependabot PRs.
    • Allows the skip changeset label when no release is required.
    • Exempts Dependabot PRs so the dedicated workflow owns dependency release decisions.

Copilot AI and others added 3 commits August 24, 2026 18:18
Co-authored-by: TylerJDev <26746305+TylerJDev@users.noreply.github.com>
Co-authored-by: TylerJDev <26746305+TylerJDev@users.noreply.github.com>
Co-authored-by: TylerJDev <26746305+TylerJDev@users.noreply.github.com>
Copilot AI changed the title [WIP] Refactor Changesets release workflow for Dependabot Refactor release flow to PR-time Dependabot changesets Aug 24, 2026
Copilot AI requested a review from TylerJDev August 24, 2026 18:24
@TylerJDev
TylerJDev changed the base branch from copilot/implement-changesets-release-flow to main August 25, 2026 14:21
@TylerJDev
TylerJDev requested a balanced review from Copilot August 25, 2026 14:36
@TylerJDev
TylerJDev marked this pull request as ready for review August 25, 2026 14:44
@TylerJDev
TylerJDev requested a review from a team as a code owner August 25, 2026 14:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Refactors releases around Changesets and automates changeset policy for human and Dependabot PRs.

Changes:

  • Adds Changesets release configuration and automation.
  • Generates patch changesets for qualifying Dependabot updates.
  • Enforces changesets or a skip label on human PRs.
Show a summary per file
File Description
.changeset/README.md Documents Changesets.
.changeset/config.json Configures package releases.
.github/workflows/check-for-changeset.yml Enforces human PR policy.
.github/workflows/dependabot-changeset.yml Automates Dependabot changesets.
.github/workflows/pr.yml Runs policy tests.
.github/workflows/publish.yml Removes legacy publishing.
.github/workflows/release.yml Adds Changesets releases.
README.md Documents release procedures.
package-lock.json Locks Changesets dependencies.
package.json Adds release tooling and metadata.
scripts/dependabot-changeset.mjs Implements Dependabot policy.
scripts/dependabot-changeset.test.mjs Tests policy evaluation.

Review details

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

  • Files reviewed: 11/12 changed files
  • Comments generated: 7
  • Review effort level: Balanced

Comment thread .github/workflows/dependabot-changeset.yml
Comment thread .github/workflows/release.yml Outdated
Comment thread .github/workflows/release.yml
Comment thread package.json Outdated
Comment thread .github/workflows/check-for-changeset.yml Outdated
Comment thread scripts/dependabot-changeset.mjs Outdated
)
})

test('a previously generated changeset is removed when the PR no longer qualifies', () => {
Copilot AI and others added 2 commits August 25, 2026 17:10
…e-publish, removed files, dismissed alerts, and test coverage

Co-authored-by: TylerJDev <26746305+TylerJDev@users.noreply.github.com>
Co-authored-by: TylerJDev <26746305+TylerJDev@users.noreply.github.com>

@francinelucca francinelucca left a comment

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.

seeing some high level callouts from copilot, please re-request review when those have been addressed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Review details

Suppressed comments (1)

Previously missed (1) — in code that hasn't changed since the last review.

scripts/dependabot-changeset.mjs:103

  • The 404 shortcut also applies to PUT and DELETE, so a failed write (for example, a missing branch or inaccessible resource) is treated as success and applyDependabotChangeset reports created/removed even though no commit was made. Only a GET uses 404 to mean “no existing changeset”; mutation responses should flow through the error path.
  if (response.status === 404) return undefined
  • Files reviewed: 11/12 changed files
  • Comments generated: 2
  • Review effort level: Balanced

Comment thread scripts/dependabot-changeset.mjs
Comment thread .github/workflows/dependabot-changeset.yml
TylerJDev and others added 2 commits August 26, 2026 10:47
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants