Repository navigation
chore: fix Dependabot version-bump gate and stop tightening squawk-cli - #14
Conversation
…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>
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ 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.
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.
There was a problem hiding this comment.
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.
Sent by Cursor Automation: Assign PR reviewers & Assess Risk & Approve if Safe
There was a problem hiding this comment.
PR Risk Assessment (re-evaluated after synchronize)
Risk level: Very Low
Review required: No
Reviewers assigned: None (no review required)
Action taken: No approval action — an active approval already exists on the current head, so no new approval was submitted.
Re-evaluation: The new commit (86800bd) is a merge of main into the branch, bringing in the poetry 2.5.1 / python 3.13 tooling updates from #17. This is branch healing with no change to the PR's own logic: the substantive diff remains confined to .github/dependabot.yml (ignore squawk-cli so Dependabot cannot 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). No production/runtime code changes — the test-coverage gate does not apply. No importlinter.toml changes.
risk_level: Very Low
review_required: false
reviewers: []
action_taken: no_action
rationale: Diff touches only .github/dependabot.yml and .github/workflows/ci.yml (CI gating + Dependabot config, no production code); the new commit is a merge of main (branch healing, no logic change). PR already has an active approval on head 86800bd, so no re-approval was needed.
Sent by Cursor Automation: Assign PR reviewers & Assess Risk & Approve if Safe



Summary
Follow-ups surfaced by the first Dependabot PRs (#12, #13):
ci.yml): the "version must differ from main" check fails on any Dependabot pip PR, because Dependabot editspyproject.toml(dependency constraints) but never bumps the package version. Skipped fordependabot[bot]; dependency changes roll into the next release.squawk-cli(dependabot.yml):versioning-strategy: increasewas raising the intentional>=2.0floor (e.g. to>=2.61.0), which would force consumers off their ownadditional_dependenciespins. Now ignored.Also created the missing
dependencieslabel so Dependabot can apply it.Notes
Note
Low Risk
Only GitHub Actions and Dependabot configuration change; no application or release runtime behavior.
Overview
Fixes two issues that were blocking or breaking Dependabot pip PRs.
Dependabot now ignores
squawk-cliupdates soversioning-strategy: increasecannot raise the deliberate>=2.0constraint (which would override consumers’additional_dependenciespins).CI still requires
pyproject.tomland__init__.pyversions to match, but skips the “package version must differ frommain” gate when the PR author isdependabot[bot](usingpull_request.user.login, notgithub.actor, so Update-branch pushes do not re-enable the check).Reviewed by Cursor Bugbot for commit 86800bd. Bugbot is set up for automated code reviews on this repo. Configure here.