Skip to content

chore: update baton-sdk to v0.7.16#84

Open
gontzess wants to merge 3 commits into
mainfrom
task-sdk-upgrade-batch-10
Open

chore: update baton-sdk to v0.7.16#84
gontzess wants to merge 3 commits into
mainfrom
task-sdk-upgrade-batch-10

Conversation

@gontzess

@gontzess gontzess commented Jan 27, 2026

Copy link
Copy Markdown

Summary

  • Upgrade baton-sdk to v0.7.16
  • Upgrade Go to 1.25.2
  • Update CI workflow to use go-version-file instead of hardcoded Go versions
  • Update CredentialOptions_RandomPassword to LocalCredentialOptions_RandomPassword (SDK API change)
  • Add WithDefaultCapabilitiesConnectorBuilder option

CI Status

  • go-lint: PASS
  • go-test: PASS
  • test-cloud-version: FAIL (pre-existing issue - "old account manager interface implemented for user" error)

Build Status

  • Local build: PASS (verified go build ./...)

Test plan

  • go build ./...
  • go test ./...
  • go-lint passes
  • go-test passes

@gontzess
gontzess requested a review from a team January 27, 2026 16:04
@coderabbitai

coderabbitai Bot commented Jan 27, 2026

Copy link
Copy Markdown

Walkthrough

Go tooling upgraded from 1.23.4 to 1.25.2 with baton-sdk bumped to v0.7.4 and multiple indirect dependencies updated. Method receiver syntax refined in config code, GetString logic refactored for type safety, and credential options type reference changed.

Changes

Cohort / File(s) Summary
Dependency and Toolchain Updates
go.mod
Go version bumped to 1.25.2; removed toolchain directive. Baton-SDK upgraded to v0.7.4. Multiple indirect dependencies updated (gopsutil v3→v4, klauspost/compress, otter, tklauser/*, golang.org/x/sys). Added Masterminds/semver/v3 and ebbitengine/purego; removed dolthub/maphash and gammazero/deque.
Config Generation Refactoring
pkg/config/conf.gen.go
Method receiver syntax corrected for Gitlab.findFieldByTag. GetString logic enhanced with type-safe pattern: accepts string or []byte values, panics on type mismatch, returns empty string if field not found.
Connector Type Reference Update
pkg/connector/users.go
RandomPassword option type changed from v2.CredentialOptions\_RandomPassword to v2.LocalCredentialOptions\_RandomPassword in crypto.GenerateRandomPassword call.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 A hop, skip, and Go update,
Dependencies dance to version fate,
Types align in safety's glow,
Version jumps from one to two-oh!
Credentials dressed in finer clothes,
As the rabbit's code still flows! 🥕

🚥 Pre-merge checks | ✅ 1 | ❌ 2
❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check ⚠️ Warning The title states 'update baton-sdk to v0.7.16' but the PR objectives and go.mod changes show baton-sdk was upgraded to v0.7.4, not v0.7.16. Additionally, the PR also includes a significant Go version upgrade from 1.23.4 to 1.25.2, which is omitted from the title. Update the title to accurately reflect the actual changes, such as: 'chore: upgrade baton-sdk to v0.7.4 and Go to 1.25.2' or similar to represent the main changes.
✅ Passed checks (1 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch task-sdk-upgrade-batch-10

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

@gontzess gontzess changed the title Upgrade baton-sdk to v0.7.x and Go to 1.25.2 Upgrade baton-sdk to v0.7.4 and Go to 1.25.2 Jan 27, 2026
@gontzess
gontzess marked this pull request as draft January 27, 2026 18:59
@gontzess
gontzess force-pushed the task-sdk-upgrade-batch-10 branch from 01b083e to 10a53b9 Compare January 27, 2026 20:22
@gontzess
gontzess force-pushed the task-sdk-upgrade-batch-10 branch from 10a53b9 to e211acd Compare January 27, 2026 21:06
@gontzess
gontzess marked this pull request as ready for review January 27, 2026 22:03
gontzess and others added 2 commits January 28, 2026 11:47
@gontzess gontzess changed the title Upgrade baton-sdk to v0.7.4 and Go to 1.25.2 chore: update baton-sdk to v0.7.16 Feb 6, 2026
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