Skip to content

Auto-merge green Dependabot updates and group the pm toolchain into one daily PR - #169

Merged
unbraind merged 3 commits into
mainfrom
ci/auto-merge-green-dependabot-updates
Oct 4, 2026
Merged

unbraind merged 3 commits into
mainfrom
ci/auto-merge-green-dependabot-updates

Conversation

@unbraind

@unbraind unbraind commented Oct 4, 2026 •

Copy link
Copy Markdown
Owner

What

  • .github/dependabot.yml: npm is checked daily; @unbrained/pm-cli and pm-* packages arrive as one pm-toolchain PR, other minor/patch updates as one dependencies PR. Other ecosystems are unchanged.
  • .github/workflows/dependabot-auto-merge.yml: for Dependabot PRs only (and only in unbraind), dependabot/fetch-metadata (SHA-pinned v3.1.0) classifies the update; the pm-toolchain group (calendar-versioned, so a year rollover reads as semver-major) and every other non-major update get gh pr merge --auto --squash. Workflow default permissions are {}; the job alone gets contents: write + pull-requests: write.
  • Repository setting: auto-merge allowed, merged branches deleted.

Why

Fleet rule ([companion pm-cli-website-6d05]): a pm CLI release must reach every package without a hand-written certification PR. Branch protection still requires test (22), test (26), so nothing merges red; a bump that fails CI stays open as a real defect. Major updates still need a person. Same change as the reviewed pilot unbraind/pm-presets#118.

pm item


Summary by cubic

Auto-merges green Dependabot updates and groups the pm toolchain into one daily PR, so version bumps land without a hand-written certification PR.

  • npm is now checked daily; @unbrained/pm-cli and pm-* packages arrive as one pm-toolchain PR, and all other dependencies as a disjoint dependencies PR.
  • A least-privilege workflow enables GitHub squash auto-merge; branch protection still requires the test checks, so a failing bump stays open as a defect.
  • The pm-toolchain group is calendar-versioned, so a year rollover reads as semver-major and still auto-merges; outside that group, only updates classified minor or patch auto-merge, so anything major or unclassified waits for a person.
  • Requires the repository setting that allows auto-merge; no manual rollout steps needed.

Written for commit d0553ac. Summary will update on new commits.

Review in cubic

Summary by Sourcery

Automate safe Dependabot updates and coordinate daily pm toolchain version bumps through grouped pull requests.

New Features:

  • Automatically enable squash auto-merge for eligible green Dependabot pull requests while leaving major or unclassified updates for manual review.

Enhancements:

  • Group the pm CLI toolchain into a single daily Dependabot pull request and consolidate other minor and patch npm updates separately.

Chores:

  • Allow repository auto-merge and configure merged branches for deletion.

…ne daily PR

Version bumps of @unbrained/pm-cli, pm-ops and pm-changelog are mechanical.
Dependabot now checks npm daily and groups the pm toolchain (and other
minor/patch updates) into single pull requests; a least-privilege workflow
enables squash auto-merge for every non-major Dependabot PR, so it lands as
soon as the required checks pass and a failing bump stays open as a defect.

pm item: pm-web-56no

@sourcery-ai sourcery-ai Bot 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.

Sorry @unbraind, you've used your own review budget of 250,000 diff characters for the last 7 days.

You can request another review in 1 day and 15 hours by commenting @sourcery-ai review. Upgrade to get a review now.

@coderabbitai

coderabbitai Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: unbraind/pm-web/.coderabbit.yaml
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a3056e2a-b089-4ac5-84b8-a0100992a3df
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@sourcery-ai

sourcery-ai Bot commented Oct 4, 2026

Copy link
Copy Markdown

Reviewer's Guide

This PR changes npm Dependabot to run daily and consolidate updates into a separately handled pm toolchain group and a minor/patch dependency group. A SHA-pinned, least-privilege workflow enables squash auto-merge for green Dependabot PRs, including calendar-versioned pm toolchain rollovers, while leaving other major updates for manual review; the repository is also configured to allow auto-merge and delete merged branches.

File-Level Changes

Change Details Files
Configure daily npm Dependabot updates with explicit grouping for the pm toolchain and non-major general dependencies.
  • Change npm checks from weekly to daily.
  • Group @unbrained/pm-cli and pm-* packages into pm-toolchain without update-type filtering.
  • Group all remaining minor and patch updates into dependencies while excluding pm packages.
.github/dependabot.yml
Add a least-privilege workflow that enables squash auto-merge for eligible Dependabot pull requests after required checks pass.
  • Restrict execution to Dependabot PRs in the unbraind organization.
  • Fetch pinned Dependabot metadata to identify dependency groups and update types.
  • Auto-merge the pm-toolchain group and all non-major updates, leaving unrelated major updates for manual review.
  • Set empty workflow-level permissions and grant write permissions only to the job.
.github/workflows/dependabot-auto-merge.yml
Record the associated pm work item and history artifacts.
  • Add the pm-web-56no task definition.
  • Add the corresponding JSONL history entry.
.agents/pm/tasks/pm-web-56no.toon
.agents/pm/history/pm-web-56no.jsonl

Tips and commands

Interacting with Sourcery

  • Trigger a new review: Comment @sourcery-ai review on the pull request.
  • Continue discussions: Reply directly to Sourcery's review comments.
  • Generate a GitHub issue from a review comment: Ask Sourcery to create an
    issue from a review comment by replying to it. You can also reply to a
    review comment with @sourcery-ai issue to create an issue from it.
  • Generate a pull request title: Write @sourcery-ai anywhere in the pull
    request title to generate a title at any time. You can also comment
    @sourcery-ai title on the pull request to (re-)generate the title at any time.
  • Generate a pull request summary: Write @sourcery-ai summary anywhere in
    the pull request body to generate a PR summary at any time exactly where you
    want it. You can also comment @sourcery-ai summary on the pull request to
    (re-)generate the summary at any time.
  • Generate reviewer's guide: Comment @sourcery-ai guide on the pull
    request to (re-)generate the reviewer's guide at any time.
  • Resolve all Sourcery comments: Comment @sourcery-ai resolve on the
    pull request to resolve all Sourcery comments. Useful if you've already
    addressed all the comments and don't want to see them anymore.
  • Dismiss all Sourcery reviews: Comment @sourcery-ai dismiss on the pull
    request to dismiss all existing Sourcery reviews. Especially useful if you
    want to start fresh with a new review - don't forget to comment
    @sourcery-ai review to trigger a new review!

Customizing Your Experience

Access your dashboard to:

  • Enable or disable review features such as the Sourcery-generated pull request
    summary, the reviewer's guide, and others.
  • Change the review language.
  • Add, remove or edit custom review instructions.
  • Adjust other review settings.

Getting Help

@greptile-apps

greptile-apps Bot commented Oct 4, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[High risk] Enables automatic merging of dependency updates via CI workflow.

The changes since the previous review appear safe to merge.

What we checked:

  • Toolchain packages stay together: Both toolchain patterns appear unchanged in the other group's exclusions, so the catch-all cannot move matching packages there.

Summary

Adds daily npm groups and a Dependabot-only workflow that requests squash auto-merge.

  • pm-toolchain and dependencies have matching exclusions, keeping their packages separate.
  • The latest changes explicitly allow minor and patch updates outside pm-toolchain. Major and unclassified updates wait for a person.
  • unbraind explicitly intends calendar-versioned pm-toolchain updates to merge without manual approval, including year rollovers.
  • No new actionable issues were found. No previous review threads were supplied.
Diagram
%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Dependabot pull request] --> B{Author and owner match?}
  B -->|No| C[Skip job]
  B -->|Yes| D[Read update metadata]
  D --> E{pm-toolchain or minor or patch?}
  E -->|No| F[Leave for a person]
  E -->|Yes| G[Request squash auto-merge]
  G --> H[GitHub applies required merge checks]
Loading

Reviews (3) · Last reviewed commit: "Allow-list minor and patch updates and d..."

@unbraind

unbraind commented Oct 4, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai full review
@greptileai review

@coderabbitai

coderabbitai Bot commented Oct 4, 2026

Copy link
Copy Markdown

Rate Limit Exceeded

@unbraind have exceeded the limit for the number of chat messages per hour. Please wait 14 minutes and 22 seconds before sending another message.

…sions

Review feedback: an update that fetch-metadata cannot classify must not
auto-merge, so the condition now requires semver-minor or semver-patch
outside the calendar-versioned pm-toolchain group. The job's write scopes
are documented, the dependencies group selects every package explicitly,
and the item's acceptance criteria name the pm-toolchain exception.

pm item: pm-web-56no
@unbraind

unbraind commented Oct 4, 2026

Copy link
Copy Markdown
Owner Author

@coderabbitai full review
@greptileai review

@coderabbitai

coderabbitai Bot commented Oct 4, 2026

Copy link
Copy Markdown

Rate Limit Exceeded

@unbraind have exceeded the limit for the number of chat messages per hour. Please wait 54 minutes and 5 seconds before sending another message.

@unbraind
unbraind merged commit c66efe7 into main Oct 4, 2026
11 checks passed
@unbraind
unbraind deleted the ci/auto-merge-green-dependabot-updates branch October 4, 2026 18:02
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.

1 participant