Skip to content

Conversation

@predatorx7
Copy link
Collaborator

@predatorx7 predatorx7 commented Jan 20, 2026

Summary by CodeRabbit

  • New Features

    • Added support for theme customization in app configuration.
    • Expanded verification error handling with new exception types.
  • Chores

    • Bumped SDK/version to 0.29.0 and updated prebuilt binaries.
    • Updated installation docs and added changelog sections for 0.29.0 and 0.27.0.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link

coderabbitai bot commented Jan 20, 2026

📝 Walkthrough

Walkthrough

This PR bumps the SDK from 0.25.0 to 0.29.0 across manifests and packaging assets (Package.swift, podspecs, download script, README, version file), renames the flutter_secure_storage binary target to flutter_secure_storage_darwin, and adds an optional theme field to app info models plus new verification exception variants.

Changes

Cohort / File(s) Summary
Version & Changelog
CHANGELOG.md, README.md, Sources/ReclaimInAppSdk/Resources/InAppSdk.version, Scripts/download_frameworks.sh
Version bumps from 0.25.0 → 0.29.0; changelog entries for 0.29.0/0.27.0 added; download URL/version constant updated.
Swift Package Manifests
Package.swift, Devel/Package.swift.local, Devel/Package.swift.prod
Updated many binaryTarget URLs/checksums to ios/0.29.0; renamed binary target flutter_secure_storageflutter_secure_storage_darwin (path/name updates).
CocoaPods / Podspecs
ReclaimInAppSdk.podspec, Devel/podspec.prod, Examples/SwiftUIWithPodExample/Podfile
Podspec version bumped to 0.29.0; xcframework download URLs and filenames updated (including flutter_secure_storage → flutter_secure_storage_darwin); example Podfile constraint updated.
Data Model: ReclaimAppInfo
Sources/ReclaimInAppSdk/ReclaimOverrides.swift
Added optional theme: String? property and updated initializer signature to accept theme.
API Messages / Pigeon Models
Sources/ReclaimInAppSdk/Messages.swift
Added theme handling to ClientReclaimAppInfoOverride (field, fromList/toList/equality). Added enum variants to ReclaimApiVerificationExceptionType (verificationDismissed, verificationFailed, verificationCancelled).
Verification Flow
Sources/ReclaimInAppSdk/ReclaimVerification.swift
Include theme: appInfo?.theme when constructing ReclaimAppInfoOverride.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

Suggested reviewers

  • Sajjad21990
  • kryptocodes
  • Karam19

Poem

🐰 The version hops from 0.25 to 0.29,
A tiny theme peeks out, optional and fine,
flutter_secure_storage_darwin takes its place,
New exceptions added with tidy grace,
Checksums and URLs refreshed — a release to chase! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 21.74% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: updating the inapp SDK module from 0.25.0 to 0.29.0, which is consistently reflected across all modified files.

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


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@predatorx7 predatorx7 mentioned this pull request Jan 20, 2026
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@CHANGELOG.md`:
- Line 12: Update the changelog entry text that currently reads "Fix feature
flag overrides causing interruption with some login pages and returning
incorrect values for fields not overriden" to correct the misspelling by
changing "overriden" to "overridden" so the line reads "Fix feature flag
overrides causing interruption with some login pages and returning incorrect
values for fields not overridden".


## 0.27.0

* Fix feature flag overrides causing interruption with some login pages and returning incorrect values for fields not overriden
Copy link

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Fix spelling error.

The word "overriden" should be spelled "overridden".

📝 Proposed fix
-* Fix feature flag overrides causing interruption with some login pages and returning incorrect values for fields not overriden
+* Fix feature flag overrides causing interruption with some login pages and returning incorrect values for fields not overridden
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
* Fix feature flag overrides causing interruption with some login pages and returning incorrect values for fields not overriden
* Fix feature flag overrides causing interruption with some login pages and returning incorrect values for fields not overridden
🧰 Tools
🪛 LanguageTool

[grammar] ~12-~12: Ensure spelling is correct
Context: ...turning incorrect values for fields not overriden * Update default feature flag values * Add...

(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)

🤖 Prompt for AI Agents
In `@CHANGELOG.md` at line 12, Update the changelog entry text that currently
reads "Fix feature flag overrides causing interruption with some login pages and
returning incorrect values for fields not overriden" to correct the misspelling
by changing "overriden" to "overridden" so the line reads "Fix feature flag
overrides causing interruption with some login pages and returning incorrect
values for fields not overridden".

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