chore: device attributes update

[Shahroz16]

closes: https://linear.app/customerio/issue/MBL-168/device-attributes-in-journeys

It does the following:

• Remove CIO device attributes being sent into context.device (it was unnecessary)
• if autoTrackDeviceAttributes is false, remove all device attributes from going into properties, context attributes stays unaffected
• Selected context attributes are now part of device attributes by adding an event plugin
• Remove library from context as part of https://linear.app/customerio/issue/MBL-173/remove-library-value

[github-actions]

Sample app builds 📱

Below you will find the list of the latest versions of the sample apps. It's recommended to always download the latest builds of the sample apps to accurately test the pull request.

---

• APN-UIKit: shahroz-mbl-168 (1710501018)
• CocoaPods-FCM: shahroz-mbl-168 (1710501074)

[Shahroz16]

we are not sending CIO device attributes in context, it was providing no value

[codecov]

Codecov Report

Attention: Patch coverage is 96.87500% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 54.51%. Comparing base (9996872) to head (ad49cbc).
Report is 114 commits behind head on main-replica-for-cdp.

❗ Current head ad49cbc differs from pull request most recent head 06571fc. Consider uploading reports for the commit 06571fc to get more accurate results

Files	Patch %	Lines
...taPipeline/Plugins/DeviceContexualAttributes.swift	95.65%	1 Missing ⚠️

Additional details and impacted files

@@                   Coverage Diff                    @@
##           main-replica-for-cdp     #640      +/-   ##
========================================================
- Coverage                 57.35%   54.51%   -2.84%     
========================================================
  Files                       132      139       +7     
  Lines                      3780     3907     +127     
========================================================
- Hits                       2168     2130      -38     
- Misses                     1612     1777     +165     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ami-aman]

Remove CIO device attributes being sent into context.device (it was unnecessary)

@Shahroz16 In tracking migration, we are doing the same i.e. sending the attributes in context.device. I believe this change will affect it too ?

[Shahroz16]

Remove CIO device attributes being sent into context.device (it was unnecessary)

@Shahroz16 In tracking migration, we are doing the same i.e. sending the attributes in context.device. I believe this change will affect it too ?

@ami-aman nope, token etc is not a device attribute, its model, OS version etc. Context.device already collects them, we were just adding ours alongside Segments, which was causing duplications.

[ami-aman]

nope, token etc is not a device attribute, its model, OS version etc. Context.device already collects them, we were just adding ours alongside Segments, which was causing duplications.

Hmm..Device attributes does not only contain token it also contain model, OS version etc as collected by our current Journey module. When we do migration, we are sending the data as is to CDP which is kinda duplicate ! Just thinking out loud.

[mrehan27]

LGTM 👍🏻