Skip to content

[skill-drift] fix(sentry-cocoa-sdk): app start limit + SentryObjC docs#149

Merged
philprime merged 2 commits into
mainfrom
skill-drift/sentry-cocoa-sdk-skill-drift-fix-sentry-cocoa-sdk-app-sta-1780325408
Jun 15, 2026
Merged

[skill-drift] fix(sentry-cocoa-sdk): app start limit + SentryObjC docs#149
philprime merged 2 commits into
mainfrom
skill-drift/sentry-cocoa-sdk-skill-drift-fix-sentry-cocoa-sdk-app-sta-1780325408

Conversation

@github-actions

@github-actions github-actions Bot commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Warning

Auto-generated from external SDK content. Review all links and code suggestions before acting on them.

Summary

  • Fix incorrect app start duration limit: changed from 5 seconds to 180 seconds (3 minutes) in tracing.md
  • Add documentation for experimental SentryObjC wrapper SDK (introduced in 9.16.0-alpha.1, PR #7918)

Changes

1. App Start Duration Limit Fix

File: skills/sentry-cocoa-sdk/references/tracing.md

Corrected the maximum app start duration from "5 seconds" to "180 seconds (3 minutes)" to match the actual SDK implementation (SentryAppStartTracker.maxAppStartDuration = 180.0). The warning note correctly stated that this limit is skipped for standalone app start tracing.

2. SentryObjC Documentation

File: skills/sentry-cocoa-sdk/SKILL.md

Added Option 4 documenting the new experimental SentryObjC wrapper SDK. This is for pure Objective-C/C++ projects that cannot enable Clang modules (e.g., with -fmodules=NO). The documentation includes:

  • When to use SentryObjC vs. regular Sentry
  • SPM installation instructions
  • Migration steps from regular Sentry
  • Link to xcframework download
  • Clear experimental status warning

PRs Reviewed

  • #7918: feat(objc): SentryObjC wrapper SDK (merged 2026-05-29)
  • #7949: feat: skip max app start duration limit for standalone tracing (merged 2026-05-27)
  • #7936: feat: Add extendAppLaunch/finishExtendedAppLaunch API (merged 2026-05-27, already documented)

Verification

  • Confirmed 180-second limit by reading SentryAppStartTracker.swift source at merge SHA bbb8fdc7
  • Verified SentryObjC feature details from PR #7918 and CHANGELOG.md
  • Checked that extended app launch API (PR #7936) is already documented in tracing.md lines 171-236

Source PRs

Automated drift-fix run.

Co-Authored-By: Claude (claude-sonnet-4-5) <noreply@anthropic.com>
Comment thread skills/sentry-cocoa-sdk/references/tracing.md
Comment thread skills/sentry-cocoa-sdk/SKILL.md Outdated
Comment thread skills/sentry-cocoa-sdk/references/tracing.md
…tryObjC docs

- Fix inconsistent 5s vs 180s app start threshold in troubleshooting table
- Remove experimental tag from SentryObjC (stable since 9.17.0)
- Bump SentryObjC min version to 9.17.1 for debug symbol support

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@philprime philprime requested a review from a team as a code owner June 15, 2026 12:52
@philprime philprime merged commit e620e8e into main Jun 15, 2026
14 checks passed
@philprime philprime deleted the skill-drift/sentry-cocoa-sdk-skill-drift-fix-sentry-cocoa-sdk-app-sta-1780325408 branch June 15, 2026 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants