Skip to content

Comments

feat: Create separate Swift target and move MPLog there#517

Merged
denischilik merged 21 commits intoworkstation/9.0-Releasefrom
feat/SDKE-823-Create-separate-swift-target-and-move-SceneDelegateHandler-there-2
Jan 26, 2026
Merged

feat: Create separate Swift target and move MPLog there#517
denischilik merged 21 commits intoworkstation/9.0-Releasefrom
feat/SDKE-823-Create-separate-swift-target-and-move-SceneDelegateHandler-there-2

Conversation

@denischilik
Copy link
Contributor

@denischilik denischilik commented Jan 21, 2026

Background

Creates a separate Swift target for SceneDelegateHandler to improve code organization and enable better Swift-only module distribution. This isolates Swift-specific scene delegate handling from the Objective-C core SDK.

What Has Changed

  • Create new mParticle-Apple-SDK-Swift target for Swift-only code
  • Create podspec for Swift target
  • Migrated MPLog and MPILogLevel to Swift target

Checklist

  • I have performed a self-review of my own code.
  • I have made corresponding changes to the documentation.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have tested this locally.

Reference Issue (For employees only. Ignore if you are an outside contributor)

@denischilik denischilik requested a review from a team as a code owner January 21, 2026 18:09
@denischilik denischilik force-pushed the feat/SDKE-823-Create-separate-swift-target-and-move-SceneDelegateHandler-there-2 branch 2 times, most recently from c34049f to 18fdb0a Compare January 21, 2026 18:42
@jamesnrokt jamesnrokt changed the title feat: SDKE-823 Create separate Swift target and move SceneDelegateHandler there feat: Create separate Swift target and move SceneDelegateHandler there Jan 21, 2026
@denischilik denischilik force-pushed the feat/SDKE-823-Create-separate-swift-target-and-move-SceneDelegateHandler-there-2 branch from ef7f2c8 to f6a952f Compare January 22, 2026 18:27
- Change default_subspec from 'mParticle' to 'mParticleNoLocation' in podspec
- Update Podfile platform to iOS 15.6
- Add post_install hook to ensure all pods use iOS 15.6 deployment target
- Update Media SDK pod to use NoLocation subspec
- Create mParticle-Apple-SDK-Swift target
- Add Swift source files and tests
- Update Xcode project configuration
- Move MPLogger.swift from mParticle-Apple-SDK to mParticle-Apple-SDK-Swift
- Update imports and references in Utils files
- Update Xcode project configuration
- Create mParticle-Apple-SDK-Swift.podspec for Swift module
- Add Swift podspec as dependency in main podspec (all subspecs)
- Add Swift podspec to Example Podfile with local path
…ndencies

- Copy Swift xcframework to temp_artifacts directory in build_framework()
- Add mParticle_Apple_SDK_Swift.xcframework dependency to Tuist project
- Update cleanup to remove both xcframeworks
@denischilik denischilik force-pushed the feat/SDKE-823-Create-separate-swift-target-and-move-SceneDelegateHandler-there-2 branch from f6a952f to d552808 Compare January 22, 2026 18:31
@github-actions
Copy link

github-actions bot commented Jan 22, 2026

📦 SDK Size Impact Report

Measures how much the SDK adds to an app's size (with-SDK minus without-SDK).

Metric Target Branch This PR Change
App Bundle Impact 1.59 MB 1.65 MB +64 KB
Executable Impact 896 bytes 896 bytes +N/A
XCFramework Size 8.39 MB 8.41 MB +20 KB

⚠️ This change increases SDK size impact.

Raw measurements

Target branch (workstation/9.0-Release):

{"baseline_app_size_kb":84,"baseline_executable_size_bytes":75464,"with_sdk_app_size_kb":1708,"with_sdk_executable_size_bytes":76360,"sdk_impact_kb":1624,"sdk_executable_impact_bytes":896,"xcframework_size_kb":8588}

This PR:

{"baseline_app_size_kb":84,"baseline_executable_size_bytes":75464,"with_sdk_app_size_kb":1772,"with_sdk_executable_size_bytes":76360,"sdk_impact_kb":1688,"sdk_executable_impact_bytes":896,"xcframework_size_kb":8608}

@denischilik denischilik changed the title feat: Create separate Swift target and move SceneDelegateHandler there feat: Create separate Swift target and move MPLog there Jan 22, 2026
BrandonStalnaker

This comment was marked as outdated.

@BrandonStalnaker BrandonStalnaker self-requested a review January 23, 2026 17:31
Copy link
Contributor

@nickolas-dimitrakas nickolas-dimitrakas left a comment

Choose a reason for hiding this comment

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

LGTM

@denischilik denischilik merged commit f745866 into workstation/9.0-Release Jan 26, 2026
13 of 15 checks passed
@denischilik denischilik deleted the feat/SDKE-823-Create-separate-swift-target-and-move-SceneDelegateHandler-there-2 branch January 26, 2026 13:20
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.

5 participants