Skip to content

fix: align braze-38 SDK range with its directory major#730

Merged
nickolas-dimitrakas merged 1 commit into
workstation/6.0-Releasefrom
fix/braze-ga-sdk-major-alignment-workstation-6
Jul 15, 2026
Merged

fix: align braze-38 SDK range with its directory major#730
nickolas-dimitrakas merged 1 commit into
workstation/6.0-Releasefrom
fix/braze-ga-sdk-major-alignment-workstation-6

Conversation

@nickolas-dimitrakas

@nickolas-dimitrakas nickolas-dimitrakas commented Jul 15, 2026

Copy link
Copy Markdown

Summary

The kit directory suffix N (e.g. kits/braze/braze-38) encodes the intended major version of the wrapped vendor SDK. braze/braze-38 pinned a range that resolved to the wrong major:

Kit Was Now
braze/braze-38 com.braze:android-sdk-ui:[37.0.0,38.0.0) (→ 37.x) [38.0.0,39.0.0)

Siblings braze-39/40/41 already align, and there is no braze-37 directory, so the range was the defect.

Verification

  • CI build-kits / Build braze-38 passes with the new range.
  • Confirmed com.braze:android-sdk-ui:38.0.0 is published, so the range resolves.
  • README does not hardcode the wrapped SDK version — no doc changes.

Scope change

This PR originally also bumped the GA kits (ga-23/ga4-23) to firebase-analytics major 23. That is removed here and moved to a dedicated PR, because firebase-analytics 23.x ships Kotlin 2.1/2.2 metadata that the repo's current Kotlin 2.0.20 compiler cannot read — so GA-23 must be coupled with a repo-wide Kotlin toolchain upgrade.

The pre-existing cross-platform-tests failure (fails on :android:buildSrc:compileKotlin + emulator infra, unrelated to this change) is not addressed here.

🤖 Generated with Claude Code

@nickolas-dimitrakas
nickolas-dimitrakas requested a review from a team as a code owner July 15, 2026 15:25
@cursor

cursor Bot commented Jul 15, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Single dependency coordinate change in a kit wrapper; consumers may resolve a newer Braze UI SDK major but no auth, data, or application code paths change in this diff.

Overview
Updates kits/braze/braze-38 so the wrapped vendor SDK matches the braze-38 directory naming: com.braze:android-sdk-ui moves from [37.0.0,38.0.0) (resolving to 37.x) to [38.0.0,39.0.0) (38.x), consistent with sibling kits like braze-39 / braze-40.

This is a Gradle dependency range-only change; no kit source or integration logic is modified in the diff.

Reviewed by Cursor Bugbot for commit ed07111. Bugbot is set up for automated code reviews on this repo. Configure here.

@nickolas-dimitrakas nickolas-dimitrakas self-assigned this Jul 15, 2026
The kit directory suffix N encodes the wrapped vendor SDK's intended major
version, but braze/braze-38 pinned com.braze:android-sdk-ui:[37.0.0,38.0.0)
(major 37). Bumped to [38.0.0,39.0.0) to match the braze-38 directory; siblings
braze-39/40/41 already align and there is no braze-37 directory.

The GA kit (ga-23/ga4-23) major-23 alignment is deferred to a separate PR that
also bumps the repo Kotlin toolchain, because firebase-analytics 23.x ships
Kotlin 2.1/2.2 metadata that the current Kotlin 2.0.20 compiler cannot read.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@nickolas-dimitrakas
nickolas-dimitrakas force-pushed the fix/braze-ga-sdk-major-alignment-workstation-6 branch from 608bb3f to ed07111 Compare July 15, 2026 15:50
@nickolas-dimitrakas nickolas-dimitrakas changed the title fix: align braze-38 and GA kit SDK ranges with their directory major fix: align braze-38 SDK range with its directory major Jul 15, 2026
@sonarqubecloud

Copy link
Copy Markdown

@nickolas-dimitrakas
nickolas-dimitrakas merged commit 319bae2 into workstation/6.0-Release Jul 15, 2026
90 of 92 checks passed
@nickolas-dimitrakas
nickolas-dimitrakas deleted the fix/braze-ga-sdk-major-alignment-workstation-6 branch July 15, 2026 17:34
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.

2 participants