Skip to content

chore(deps): Change Sentry to branch main#93

Closed
philprime wants to merge 4 commits intophilprime/sentry-spmfrom
philprime/sentry-spm-main
Closed

chore(deps): Change Sentry to branch main#93
philprime wants to merge 4 commits intophilprime/sentry-spmfrom
philprime/sentry-spm-main

Conversation

@philprime
Copy link
Member

@philprime philprime commented Feb 11, 2026

No description provided.

Copilot AI review requested due to automatic review settings February 11, 2026 16:44
@philprime philprime changed the base branch from main to philprime/sentry-spm February 11, 2026 16:48
@philprime philprime changed the title feat: Switch from Sentry xcframework to SentrySPM compile-from-source (main) chore(deps): Change Sentry to branch main Feb 11, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Migrates the app and share extension to use the SentrySPM Swift Package product (compile-from-source) instead of importing Sentry / SentrySwiftUI directly, updating Xcode/SwiftPM metadata and the generated license listings accordingly.

Changes:

  • Replace import Sentry / import SentrySwiftUI with import SentrySPM across app + share extension sources.
  • Update the Xcode project’s Swift Package product dependencies to use SentrySPM.
  • Update Package.resolved and the Settings.bundle license metadata to reflect the new Sentry dependency source/versioning.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
Targets/ShareExtension/Sources/ShareViewController.swift Switch import to SentrySPM for the share extension’s Sentry usage.
Targets/App/Sources/Utils/SentryMetricsHelper.swift Switch import to SentrySPM for metrics helper usage.
Targets/App/Sources/UI/MainContainerView.swift Switch import to SentrySPM for SwiftUI tracing integration.
Targets/App/Sources/UI/LinkLists/LinkListsContainerView.swift Switch imports to a single SentrySPM import.
Targets/App/Sources/UI/LinkListInfo/LinkListInfoContainerView.swift Switch imports to a single SentrySPM import.
Targets/App/Sources/UI/LinkListDetail/LinkListDetailContainerView.swift Switch imports to a single SentrySPM import.
Targets/App/Sources/UI/LinkInfo/LinkInfoContainerView.swift Switch imports to a single SentrySPM import.
Targets/App/Sources/UI/LinkDetailNFCSharing/LinkDetailNFCSharingViewModel.swift Switch import to SentrySPM for Sentry metrics/capture usage.
Targets/App/Sources/UI/LinkDetailNFCSharing/LinkDetailNFCSharingContainerView.swift Switch imports to a single SentrySPM import.
Targets/App/Sources/UI/LinkDetail/LinkDetailRenderView.swift Switch import to SentrySPM.
Targets/App/Sources/UI/LinkDetail/LinkDetailContainerView.swift Switch imports to a single SentrySPM import.
Targets/App/Sources/UI/CreateListEditor/CreateLinkListEditorContainerView.swift Switch imports to a single SentrySPM import.
Targets/App/Sources/UI/CreateLinkWithoutListEditor/LinkListPickerContainerView.swift Switch imports to a single SentrySPM import.
Targets/App/Sources/UI/CreateLinkWithoutListEditor/CreateLinkWithListPickerEditorContainerView.swift Switch imports to a single SentrySPM import.
Targets/App/Sources/UI/CreateLinkEditor/CreateLinkEditorContainerView.swift Switch imports to a single SentrySPM import.
Targets/App/Sources/Services/QRCodeCache.swift Switch import to SentrySPM for capture/breadcrumb usage.
Targets/App/Sources/Services/DataSeedingService.swift Switch import to SentrySPM for capture/breadcrumb usage.
Targets/App/Sources/Services/AppHealthObserver.swift Switch import to SentrySPM for metrics/breadcrumb usage.
Targets/App/Sources/Resources/Settings.bundle/Licenses.plist Update displayed Sentry license title metadata.
Targets/App/Sources/Resources/Settings.bundle/Licenses.latest_result.txt Update generated license-plist result metadata for Sentry.
Targets/App/Sources/Main/FlinkyApp.swift Switch import to SentrySPM for app startup configuration.
Flinky.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved Update sentry-cocoa resolution to main branch + revision.
Flinky.xcodeproj/project.pbxproj Replace Sentry/SentrySwiftUI package products with SentrySPM and update package reference requirements.
Comments suppressed due to low confidence (2)

Flinky.xcodeproj/project.pbxproj:690

  • The Flinky app target’s SentrySPM package reference looks inconsistent: the target depends on a PBXTargetDependency whose productRef is D4C71A17, but packageProductDependencies lists D410FC5C instead. These should reference the same XCSwiftPackageProductDependency to avoid Xcode/SPM resolution and linking issues.
			packageProductDependencies = (
				D4CFD1782E1E38F2001447E9 /* SFSafeSymbols */,
				D44A62A52E33B2800018D789 /* OnLaunch */,
				D410FC5C2F3C8AAB0028B18D /* SentrySPM */,
			);

Flinky.xcodeproj/project.pbxproj:716

  • The ShareExtension target has a similar SentrySPM mismatch: dependencies uses a PBXTargetDependency pointing at productRef D4C71A15, but packageProductDependencies lists D4C71A13. Align these to a single SentrySPM product dependency entry so the target consistently resolves/links the package product.
			name = ShareExtension;
			packageProductDependencies = (
				D4C71A132F3C8B05006E688D /* SentrySPM */,
			);

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@philprime
Copy link
Member Author

Closed in favor of #92

@philprime philprime closed this Feb 12, 2026
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.

1 participant