Skip to content

Rename ExceptionCatcher to XCDocsExceptionCatcher across package - #26

Open
krzyzanowskim wants to merge 1 commit into
BitrigApp:mainfrom
krzyzanowskim:marcin/rename-excaption-catcher-target
Open

Rename ExceptionCatcher to XCDocsExceptionCatcher across package#26
krzyzanowskim wants to merge 1 commit into
BitrigApp:mainfrom
krzyzanowskim:marcin/rename-excaption-catcher-target

Conversation

@krzyzanowskim

@krzyzanowskim krzyzanowskim commented Apr 6, 2026

Copy link
Copy Markdown

The target name ExceptionCatcher is unfortunate and conflicts with https://github.com/sindresorhus/ExceptionCatcher package. The unfortunate part is that SwiftPM requires all targets unique in the graph 🤷

multiple packages ('exceptioncatcher', 'xcdocs') declare targets with a conflicting name: 'ExceptionCatcher’; target names need to be unique across the package graph

I propose rename ExceptionCatcher → XCDocsExceptionCatcher


Note

Low Risk
Low risk rename-only change, but it can break builds for any downstream consumers or internal references that still import ExceptionCatcher or expect the old target path/name.

Overview
Renames the SwiftPM target/module from ExceptionCatcher to XCDocsExceptionCatcher, including moving its target path to Sources/XCDocsExceptionCatcher and updating target dependencies in Package.swift.

Updates code and tests to import XCDocsExceptionCatcher (e.g., MADServiceObject and ExceptionCaptureTests) so the Objective-C exception catching helper continues to compile under the new module name.

Reviewed by Cursor Bugbot for commit b58bd09. Bugbot is set up for automated code reviews on this repo. Configure here.

- Update Package.swift: rename target, change path, adjust dependencies
  for XCDocsBridge and tests.
- Move Sources/ExceptionCatcher -> Sources/XCDocsExceptionCatcher.
- Update imports in MADServiceObject.swift and ExceptionCaptureTests.swift.
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