Skip to content

chore: fix Dependabot version-bump gate and stop tightening squawk-cli - #14

Open
stephen-kintsugi wants to merge 2 commits into
mainfrom
chore/dependabot-config-and-version-gate
Open

chore: fix Dependabot version-bump gate and stop tightening squawk-cli#14
stephen-kintsugi wants to merge 2 commits into
mainfrom
chore/dependabot-config-and-version-gate

Conversation

@stephen-kintsugi

@stephen-kintsugi stephen-kintsugi commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

Summary

Follow-ups surfaced by the first Dependabot PRs (#12, #13):

  • Exempt Dependabot from the version-bump gate (ci.yml): the "version must differ from main" check fails on any Dependabot pip PR, because Dependabot edits pyproject.toml (dependency constraints) but never bumps the package version. Skipped for dependabot[bot]; dependency changes roll into the next release.
  • Stop tightening squawk-cli (dependabot.yml): versioning-strategy: increase was raising the intentional >=2.0 floor (e.g. to >=2.61.0), which would force consumers off their own additional_dependencies pins. Now ignored.

Also created the missing dependencies label so Dependabot can apply it.

Notes


Note

Low Risk
Workflow and Dependabot config only; no runtime package or security logic changes.

Overview
Fixes CI and Dependabot behavior so dependency-only bot PRs can pass without manual version bumps or unwanted constraint changes.

CI still requires pyproject.toml and __init__.py versions to match, but the “version must differ from main” step now runs only when the PR author is not dependabot[bot] (using pull_request.user.login, not github.actor, so Update-branch pushes don’t accidentally re-apply the gate).

Dependabot adds an explicit ignore for squawk-cli so versioning-strategy: increase won’t tighten the intentional broad >=2.0 floor that lets hook consumers pin via additional_dependencies.

Reviewed by Cursor Bugbot for commit 769aff3. Bugbot is set up for automated code reviews on this repo. Configure here.

…ng squawk-cli

- ci.yml: skip the "version differs from main" check for dependabot[bot]. Dependabot
  edits pyproject.toml (dependency constraints) but never bumps the package version,
  so its dependency PRs would otherwise always fail the version-consistency gate.
- dependabot.yml: ignore squawk-cli so versioning-strategy: increase does not tighten
  its intentionally broad >=2.0 floor (consumers pin their own via additional_dependencies).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

@cursor cursor 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.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 1c6845a. Configure here.

Comment thread .github/workflows/ci.yml Outdated
github.actor is the user who triggered the current run, so an Update-branch or
manual push on a Dependabot PR would set it to a human and wrongly re-enable the
version-bump check. Use github.event.pull_request.user.login, which is stable.

@cursor cursor 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.

PR Risk Assessment

Risk level: Very Low
Review required: No
Reviewers assigned: None (jaykintsugi already requested via CODEOWNERS; no additional assignment needed)
Action taken: Approved

Rationale: The diff only touches .github/dependabot.yml (ignore squawk-cli so Dependabot does not tighten the intentional >=2.0 floor) and .github/workflows/ci.yml (exempt dependabot[bot] PRs from the version-bump gate, keyed on pull_request.user.login rather than github.actor). No production/runtime code paths change; blast radius is limited to CI gating and Dependabot behavior in this repo, and both edits are trivially reversible.

No production code changes — test coverage gate does not apply.

Re-evaluated after the follow-up commit fixing the Dependabot exemption to use PR author instead of github.actor; incremental change is a correctness fix with no material risk increase.


risk_level: Very Low
review_required: false
reviewers: []
action_taken: approved
rationale: CI workflow and Dependabot config only (.github/workflows/ci.yml, .github/dependabot.yml); exempts bot PRs from version-bump gate and stops squawk-cli constraint tightening — no production code or runtime behavior change.
Open in Web View Automation 

Sent by Cursor Automation: Assign PR reviewers & Assess Risk & Approve if Safe

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