Skip to content

MOEN-45786: Dependency Update 2026-07-01 - #128

Merged
umangmoe merged 4 commits into
developmentfrom
MOEN-45786/dependency_update
Jul 2, 2026
Merged

MOEN-45786: Dependency Update 2026-07-01#128
umangmoe merged 4 commits into
developmentfrom
MOEN-45786/dependency_update

Conversation

@sdk-bot-user

Copy link
Copy Markdown
Collaborator

Summary

Automated dependency update (target: both).

Platform status

  • Android: success
  • iOS: success

@moe-hodor

moe-hodor Bot commented Jul 1, 2026

Copy link
Copy Markdown

🚪 Hodor is reviewing this PR... 👀

@moe-hodor moe-hodor Bot added branch:unknown Branch prefix did not match policy cfr:unknown Change Failure Rate: indeterminate labels Jul 1, 2026
@moe-hodor

moe-hodor Bot commented Jul 1, 2026

Copy link
Copy Markdown

⚠️ Branching Strategy — policy violation

Branch: MOEN-45786/dependency_update — category: unknown, impact: ❓ Unknown CFR

Branching Strategy — unrecognized branch prefix

Branch MOEN-45786/dependency_update doesn't use a recognized prefix. Per the Branching Strategy, use:

  • feature/ — planned work (neutral CFR)
  • patch/ — weekly maintenance (neutral CFR)
  • hotfix/ — emergency fix (⚠ increases CFR)
  • revert/ — rollback (⚠ increases CFR)

Posted by Hodor's branching-strategy gate. Per the Branching Strategy, branches must use feature/, patch/, hotfix/, or revert/ prefix. This comment is posted directly by the gate (not by the agent) to guarantee policy visibility.

ext {
moengageNativeBOMVersion = "2.2.1"
moengagePluginBaseBOMVersion = "3.0.1"
moengagePluginBaseBOMVersion = "3.1.0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔴 Critical — Version inconsistency: moengagePluginBaseBOMVersion was updated to 3.1.0 but moengageNativeBOMVersion is still at 2.2.1 while all other modules (cards, core, personalize, geofence) are at 2.3.0.

Impact: When an app uses both Inbox and other modules, Gradle will resolve dependencies from different BOM versions (2.2.1 vs 2.3.0), potentially pulling incompatible artifacts or causing runtime conflicts.

Fix: Update moengageNativeBOMVersion = "2.2.1" to "2.3.0" to match the other modules.

Comment thread sdk/geofence/android/build.gradle Outdated
ext {
moengageNativeBOMVersion = "2.2.2"
moengageNativeBOMVersion = "2.3.0"
moengagePluginBaseBOMVersion = "3.0.1"

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔴 Critical — Version inconsistency: moengageNativeBOMVersion was updated to 2.3.0 but moengagePluginBaseBOMVersion is still at 3.0.1 while all other modules (cards, core, inbox, personalize) are at 3.1.0.

Impact: The Geofence module will pull plugin dependencies from BOM 3.0.1 while other modules use 3.1.0, causing version drift and potential incompatibilities in shared plugin-base classes.

Fix: Update moengagePluginBaseBOMVersion = "3.0.1" to "3.1.0" to match the other modules.

@moe-hodor moe-hodor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review: MOEN-45786 Dependency Update 2026-07-01

Branching & CFR: MOEN-45786/dependency_update — category: unknown, impact: ❓ Unknown CFR

⚠️ Branching Strategy — unrecognized branch prefix — Branch MOEN-45786/dependency_update doesn't use a recognized prefix. Per the Branching Strategy, use:

  • feature/ — planned work (neutral CFR)
  • patch/ — weekly maintenance (neutral CFR)
  • hotfix/ — emergency fix (⚠ increases CFR)
  • revert/ — rollback (⚠ increases CFR)

Summary

Automated dependency update for MoEngage React Native SDK:

  • Android BOM: 2.2.x2.3.0
  • Plugin BOM: 3.0.13.1.0
  • iOS plugins: minor version bumps across Cards, Core, Geofence, Inbox, Personalize

Quality Gates

PR Size: 62 lines changed across 15 files — Good size.
Tests: 5 test file(s) modified (CHANGELOGs).
⚠️ Duplication: Similar changes detected in multiple files (expected for multi-module version bump).

Jira Alignment

MOEN-45786: "React Native Version update for personalize SDK update"

  • ✅ PR updates Personalize SDK versions (iOS: 1.0.01.1.0, Android BOM bump)
  • ⚠️ No Jira description to validate full scope

Issues Found

  • 🔴 Critical: 2 version inconsistency issues
  • ⚠️ Warning: 1 branching strategy violation (from pipeline)

Key Concerns

  1. Incomplete Version Updates (🔴 Critical)

    • sdk/inbox/android/build.gradle: moengageNativeBOMVersion still at 2.2.1 while all other modules upgraded to 2.3.0
    • sdk/geofence/android/build.gradle: moengagePluginBaseBOMVersion still at 3.0.1 while all other modules upgraded to 3.1.0
    • This creates version drift across modules, which can cause runtime conflicts when multiple SDKs are used together
  2. Branching Prefix (⚠️ Warning)

    • Should use patch/MOEN-45786-dependency-update for maintenance work

Regression Risk

Low — This is a dependency version bump with no code changes. However, the version inconsistency issues must be fixed to avoid:

  • Module A depending on BOM 2.2.1 and Module B depending on BOM 2.3.0 could cause classpath conflicts
  • Plugin dependencies may resolve to incompatible versions

What's Good

✅ CHANGELOG updates are comprehensive and well-documented
✅ iOS .podspec versions are consistently updated across all modules
✅ Android version bumps are applied to most modules correctly

Verdict

REQUEST_CHANGES — Fix the two version inconsistency issues in sdk/inbox and sdk/geofence to ensure all modules use the same BOM versions (2.3.0 for native BOM, 3.1.0 for plugin BOM).

@umangmoe
umangmoe merged commit 0bbf25a into development Jul 2, 2026
9 checks passed
@umangmoe
umangmoe deleted the MOEN-45786/dependency_update branch July 2, 2026 06:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch:unknown Branch prefix did not match policy cfr:unknown Change Failure Rate: indeterminate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants