[cristim]

Summary

• The OpenSearch purchase path now calls the shared purchaseguard.Guard and purchaseguard.Recover instead of its own copy of the issue #641 guard, mirroring ElastiCache (refactor(aws): share the purchase idempotency guard, starting with ElastiCache #507, providers/aws/services/elasticache/client.go idempotencyGuard and recoverAlreadyExists).
• The guard still runs before the purchase call. Valid purchases behave as before.
• New purchase_guard_test.go covers each guard rule at this path with bounded mocks.

Verification

• providers/aws build, vet, go test -timeout 300s -count=1 ./..., golangci-lint (0 issues) and the git-secrets script, with and without GOWORK=off.
• Mutation probes (reverted): skipping the guard fails the short-circuit, lookup-order, lookup-error and no-recovery tests; skipping Recover fails the AlreadyExists recovery test.
• The new tests are characterization tests of an existing behavior, so they pass before and after this refactor; the probes show they fail when the guard is removed.
• None of the files touched here are touched by fix(ladder): stop subtracting existing commitment from residual baselines #441, fix(gcp): fail closed on region-less storage SKUs and ambiguous SKU matches #471, fix(azure): count savings plans billed to another subscription in the duplicate check #491 or fix(azure/compute): skip scope checks on terminal VM reservations #439.

Closes #10 (ElastiCache, RDS and MemoryDB landed in the sibling PRs)

🤖 Generated with Claude Code

[coderabbitai]

Warning

Review limit reached

• Run on-demand review

_{This review includes 3 billable files and costs up to $0.75.}

• Ask an admin to make reviews automatic

Open in CodeRabbit

Reviews can continue after your included limit without a manual trigger. An admin must approve usage-based billing.

Or wait 43 minutes for your next included review.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available. Your 89 included PR review attempts over the past 7 days set your current allowance at 1 review per hour.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

• Configuration used: Repository: LeanerCloud/cloud-commitments-go/.coderabbit.yaml
• Review profile: CHILL
• Plan: Essentials
• Run ID: 4303bc17-f974-4505-b72d-05214c92ec1b

📥 Commits

Reviewing files that changed from the base of the PR and between 00ff5cf and e7e57b8.

📒 Files selected for processing: 3

• CHANGELOG.md
• providers/aws/services/opensearch/client.go
• providers/aws/services/opensearch/purchase_guard_test.go

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

[cristim]

Gate note (claude-cc-libs)

Rebased onto main after the overlapping PRs merged. The only conflict was CHANGELOG.md, resolved by keeping both entries. Excluding CHANGELOG.md, git diff origin/main e7e57b81b223be9320130fbde72c3a87e49bf547 is identical to the reviewed 2e46838ffb9bb864c28d922e19a798abe687149a diff apart from hunk line numbers. Re-verified on the combined tree: vet and go test -count=1 pass.

[cristim]

Merge gate verdict (claude-cc-libs)

Every gate holds at head e7e57b8:

• Two independent reviews approve the reviewed head e7e57b8. The branch was then updated from main, and the PR diff is byte-identical to the reviewed diff (compared mechanically).
• CI is green on e7e57b8, which is the reviewed change on top of current main.
• 0 unresolved threads; MERGEABLE/CLEAN.

CodeRabbit is waived as a merge gate under the 2026-10-19 decision. Squash-merging.