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
Changes only affect Dependabot configuration and CI gating for bot PRs; no runtime or application logic is modified.

Overview
Fixes CI and Dependabot behavior that broke the first dependency update PRs.

CI version gate: The check that requires the package version in pyproject.toml to differ from main now runs only when the PR author is not dependabot[bot]. Dependabot edits pyproject.toml for dependency bumps but never bumps the library version, so those PRs would always fail. The exemption uses pull_request.user.login rather than github.actor so human actions on a bot branch do not accidentally re-enable the check.

Dependabot: squawk-cli is added to the pip ignore list so Dependabot cannot raise the deliberate broad floor (>=2.0 in pyproject.toml) via versioning-strategy: increase, which would interfere with consumers pinning squawk 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