Second round SDK CI/CD sync + exceed by jeremy · Pull Request #167 · basecamp/basecamp-cli

AI review requested due to automatic review settings

February 28, 2026 10:48

chatgpt-codex-connector[bot]

Copilot AI review requested due to automatic review settings

February 28, 2026 10:55
CI is the safety gate — auto-merging action updates is circular since
a broken update approves itself. Invert the policy to auto-merge
patch+minor for all ecosystems except github_actions.

The ecosystem name uses underscore (github_actions) because
fetch-metadata derives it from the branch name, not dependabot.yml.
Pin schedule to 06:00 America/Chicago for predictable update timing.
Set open-pull-requests-limit to 10 for both gomod and github-actions.
go-mod-tidy runs on commit to catch untidy modules before CI.
go-test-short runs on push only to avoid slowing down commits.

Uses files pattern instead of types: [go] so go.mod/go.sum-only
changes also trigger the hooks. Adds default_install_hook_types
so pre-commit install sets up both pre-commit and pre-push hooks.
Disable errorlint asserts/comparison checks — the CLI uses sentinel
errors and type assertions like the SDK. Set max-issues-per-linter
and max-same-issues to 0 so lint output is never silently truncated.

Copilot AI review requested due to automatic review settings

February 28, 2026 11:01

@jeremy

Add non-mutating tidy-check Makefile target that backs up go.mod/go.sum,
runs go mod tidy, then verifies no diff — restoring originals on failure
so make check never leaves the working tree dirty.

Add both provenance-check and tidy-check to the check gate. Add
tidy-check step to test.yml before unit tests. Fix pre-existing untidy
go.mod (pflag was indirect but is directly imported).

@jeremy jeremy deleted the ci/sdk-sync-round2 branch

February 28, 2026 11:09