Skip to content

feat: Replace PLCrashReporter with KSCrash#104

Draft
williazz wants to merge 2 commits into
mainfrom
fix/replace-plcrash-with-kscrash-backtrace
Draft

feat: Replace PLCrashReporter with KSCrash#104
williazz wants to merge 2 commits into
mainfrom
fix/replace-plcrash-with-kscrash-backtrace

Conversation

@williazz

Copy link
Copy Markdown
Collaborator

Summary

Closes #85

Remove the deprecated PLCrashReporter dependency and use KSCrash's captureBacktrace/symbolicate API for collecting live stack traces during hang detection. KSCrash is already a dependency so this removes an unnecessary third-party library while maintaining the same functionality.

Key changes:

  • Replace PLLiveStackTraceReporter with KSCrashLiveStackTraceReporter
  • Remove platform conditional (#if !os(watchOS)) since KSCrash works on all Apple platforms including watchOS
  • Remove plcrashreporter from Package.swift dependencies
  • Add Recording product dependency for KSCrashRecordingCore access
  • Update tests to cover the new implementation

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

williazz added 2 commits May 23, 2026 05:00
…races

Remove the deprecated PLCrashReporter dependency and use KSCrash's
captureBacktrace/symbolicate API for collecting live stack traces
during hang detection. KSCrash is already a dependency so this removes
an unnecessary third-party library while maintaining the same
functionality.

Key changes:
- Replace PLLiveStackTraceReporter with KSCrashLiveStackTraceReporter
- Remove platform conditional (#if !os(watchOS)) since KSCrash works
  on all Apple platforms including watchOS
- Remove plcrashreporter from Package.swift dependencies
- Add Recording product dependency for KSCrashRecordingCore access
- Update tests to cover the new implementation

Closes #85
Fix SwiftFormat lint violations (redundantSelf, consecutiveBlankLines,
conditionalAssignment) and add missing CHANGELOG.md entry for PR #104.
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.

PLCrashReporter is deprecated

1 participant