Skip to content

Migrate demo apps from CocoaPods to SPM#1

Open
chante-gobel wants to merge 1 commit into
mainfrom
feature/cocoapods-to-spm-migration
Open

Migrate demo apps from CocoaPods to SPM#1
chante-gobel wants to merge 1 commit into
mainfrom
feature/cocoapods-to-spm-migration

Conversation

@chante-gobel

@chante-gobel chante-gobel commented Apr 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Migrates both demo apps (ComapiFoundationSample ObjC and ComapiFoundationSample-Swift) from CocoaPods to Swift Package Manager.

Changes

Dependencies

  • Removed Podfile and Podfile.lock
  • Added CMPComapiFoundation (local SPM package) and SnapKit 5.7.1 (remote SPM) in project.pbxproj
  • Removed all CocoaPods build phases ([CP] Check Pods Manifest.lock, [CP] Embed Pods Frameworks)

Authentication

  • Replaced JWT + Base64 CocoaPods with a native CommonCrypto-based HS256 JWT implementation in both:
    • CMPAuthenticationManager.m (ObjC)
    • AuthenticationManager.swift (Swift)
  • Removed JWT and Base64 imports from Swift bridging header

Import fixes

  • Added missing import UIKit, import CMPComapiFoundation, import SnapKit across Swift demo files:
    • AppConfigurator.swift
    • ChatViewModel.swift
    • ChatTextMessageCell.swift
    • MessageStatusView.swift
    • CreateConversationViewModel.swift
    • ConversationViewModel.swift

Testing

  • ObjC demo app builds and runs ✅
  • Swift demo app builds and runs ✅
  • Push notifications working on both ✅

- Remove Podfile and Podfile.lock
- Add CMPComapiFoundation and SnapKit as SPM dependencies in project.pbxproj
- Replace JWT/Base64 CocoaPods with native CommonCrypto JWT implementation in both ObjC and Swift auth managers
- Remove CocoaPods bridging header imports (JWT, Base64) from Swift bridging header
- Add missing UIKit, CMPComapiFoundation, SnapKit imports across Swift demo app files
- Fix Info.plist credentials for both apps (SERVER_SCHEME/SERVER_HOST were swapped)
- Update Package.resolved with SnapKit 5.7.1
@chante-gobel chante-gobel marked this pull request as ready for review April 9, 2026 15:18
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