Skip to content

build(deps): pin cloud-commitments-go to main (6168f8b) - #25

Merged
cristim merged 1 commit into
mainfrom
build/pin-cloud-commitments-go-6168f8b
Sep 28, 2026
Merged

cristim merged 1 commit into
mainfrom
build/pin-cloud-commitments-go-6168f8b

Conversation

@cristim

@cristim cristim commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

Summary

Re-pins github.com/LeanerCloud/cloud-commitments-go/{pkg,providers/aws,providers/azure,providers/gcp} from the abandoned split-PR pseudo-versions to the commit that actually landed on go main.

Why

cloud-commitments-go PR #1 was squash-merged to main as 6168f8b5360d5189c140cd8f4de614531cffa998. This repo's previous pins were pseudo-versions taken from the unmerged split branch before the squash:

Module Old pin New pin
pkg v0.0.0-20260925082912-43ab778da7ac v0.0.0-20260928074610-6168f8b5360d
providers/aws v0.0.0-20260926232454-692cacc627d1 v0.0.0-20260928074610-6168f8b5360d
providers/azure v0.0.0-20260926232454-692cacc627d1 v0.0.0-20260928074610-6168f8b5360d
providers/gcp v0.0.0-20260926232454-692cacc627d1 v0.0.0-20260928074610-6168f8b5360d

Those commits (43ab778da7ac, 692cacc627d1) are no longer reachable from go main after the squash-merge, so this repo's go.mod pointed at history that no longer exists there.

Change

GOWORK=off GOTOOLCHAIN=go1.26.6 go get \
  github.com/LeanerCloud/cloud-commitments-go/pkg@6168f8b5360d5189c140cd8f4de614531cffa998 \
  github.com/LeanerCloud/cloud-commitments-go/providers/aws@6168f8b5360d5189c140cd8f4de614531cffa998 \
  github.com/LeanerCloud/cloud-commitments-go/providers/azure@6168f8b5360d5189c140cd8f4de614531cffa998 \
  github.com/LeanerCloud/cloud-commitments-go/providers/gcp@6168f8b5360d5189c140cd8f4de614531cffa998
go mod tidy

All four modules resolved to v0.0.0-20260928074610-6168f8b5360d (MVS resolved cleanly). go.work still only contains use ., unchanged.

The renamed exported identifiers in cloud-commitments-go (AWSProvider->Provider, XxxAPI->API, etc.) required no call-site changes here; go build/go vet pass with no errors.

Verification

  • go build -o /dev/null ./...: pass
  • go vet ./...: pass (no findings)
  • go test -race -short ./...: pass, all 3 packages ok (root, cmd/cudly-mcp, tools), no FAIL
  • go mod tidy -diff: empty (clean)
  • golangci-lint v2.10.1 (CI-pinned version, golangci-lint run --timeout=10m, exact CI invocation): 0 issues

Housekeeping re-pin, not tied to a specific issue.

Summary by CodeRabbit

  • Chores
    • Updated internal module versions. These maintenance changes do not alter application features, workflows, or behavior. No user-facing functionality was added or removed, and existing capabilities remain unchanged. There are no new settings or changes to how the application is used in this release.

The previous pseudo-version pins (pkg@43ab778da, providers/{aws,azure,gcp}@692cacc627d1)
were on the unmerged cloud-commitments-go PR #1 split branch, which no longer exists
on main after the squash-merge to 6168f8b5360d5189c140cd8f4de614531cffa998. Re-pin all
four modules to that commit.
@coderabbitai

coderabbitai Bot commented Sep 28, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository: LeanerCloud/cloud-commitments-mcp/.coderabbit.yaml

Review profile: CHILL

Plan: Essentials

Run ID: d95b1a8b-900f-412d-af85-bc7b5fd1a9cc

📥 Commits

Reviewing files that changed from the base of the PR and between ff9cace and 50b1f72.

⛔ Files ignored due to path filters (1)
  • go.sum is excluded by !**/*.sum
📒 Files selected for processing (1)
  • go.mod

Included review availability: This review used your included allowance. 0 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 3 reviews per hour.


📝 Walkthrough

Walkthrough

Four cloud-commitments-go module dependencies now use the pseudo-version v0.0.0-20260928074610-6168f8b5360d.

Changes

Go module versions

Layer / File(s) Summary
Update module pins
go.mod
The pkg, AWS, Azure, and GCP module dependencies now use version v0.0.0-20260928074610-6168f8b5360d.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 50b1f

All four updated dependencies resolve to the intended revision and checksums; no material merge risk is evident from this change.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the dependency update and specifies the cloud-commitments-go commit being pinned. It accurately summarizes the main change.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

@cristim cristim added type/chore Maintenance / non-user-visible severity/low Minor harm urgency/this-sprint Within the current sprint impact/internal Team-internal only effort/xs Trivial / one-liner priority/p2 Backlog-worthy triaged Item has been triaged labels Sep 28, 2026
@cristim

cristim commented Sep 28, 2026

Copy link
Copy Markdown
Member Author

Independent review + local verification: MERGE at 50b1f72. Re-pins cloud-commitments-go to main (6168f8b5); go.sum changes limited to the four modules; build/vet/race tests pass; MCP stdio probe returns the same 11 tools with nothing on stdout but protocol frames; 13/13 checks.

@cristim
cristim merged commit fdf3b18 into main Sep 28, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

effort/xs Trivial / one-liner impact/internal Team-internal only priority/p2 Backlog-worthy severity/low Minor harm triaged Item has been triaged type/chore Maintenance / non-user-visible urgency/this-sprint Within the current sprint

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant