Skip to content

[nightly-telemetry] Upload Sentry debug symbols during release#819

Merged
r3dbars merged 1 commit into
mainfrom
codex/nightly-telemetry-sentry-dsyms
May 20, 2026
Merged

[nightly-telemetry] Upload Sentry debug symbols during release#819
r3dbars merged 1 commit into
mainfrom
codex/nightly-telemetry-sentry-dsyms

Conversation

@r3dbars
Copy link
Copy Markdown
Owner

@r3dbars r3dbars commented May 20, 2026

Summary

  • generate build/Transcripted.app.dSYM during beta/release builds
  • upload release dSYMs during Sentry release registration with --no-sources
  • document the release credential/symbolication contract and cover it in repo command tests

Evidence

  • Latest released build 1.1.42 has a current-release fatal crash (APPLE-MACOS-S) where Sentry had runtime context but missing app frames/debug symbols.
  • Recent fixes landed for the likely CoreML lifetime issue, but future current-release crashes would still be hard to diagnose without symbols.

Candidate score

  • Winner: missing Sentry release dSYM upload, 88/100
  • Runner-up: device-change rewarm failures, 69/100, mostly older releases
  • Runner-up: quiet/degraded meeting capture UX, 62/100, current-release telemetry did not show unrecovered quiet-mic hits

Verification

  • bash -n scripts/release/register-sentry-release.sh
  • bash -n scripts/entrypoints/build-beta.sh
  • TRANSCRIPTED_DISABLE_FILE_LOGGER=1 bash run-tests.sh (2251 passed)
  • sentry-cli debug-files upload --no-upload --org r3dbars --project apple-macos --type dsym --no-sources build/Transcripted.app.dSYM
  • thin packaging smoke with SKIP_NOTARIZATION=1 ... REGISTER_SENTRY_RELEASE=0 bash build-beta.sh nightly-smoke Codex

@r3dbars
Copy link
Copy Markdown
Owner Author

r3dbars commented May 20, 2026

verdict: no-findings-but-human

findings: none from this pass.
actions: human signoff recommended because this is draft release/Sentry symbolication plumbing and changes the shipped packaging path.
head_sha: e7a4ce0
fix_commit: none
checks: gh pr view, gh pr diff, release/Sentry docs read, git diff --check origin/main...origin/codex/nightly-telemetry-sentry-dsyms, bash -n on PR versions of scripts/entrypoints/build-beta.sh and scripts/release/register-sentry-release.sh; GitHub checks reported none.

manual_merge_handoff:
head_sha: e7a4ce0
verdict: no-findings-but-human
blockers: draft PR plus release/Sentry symbol upload path needs human signoff
checks: gh metadata/diff, local docs/static review, diff check, bash syntax checks

@r3dbars r3dbars marked this pull request as ready for review May 20, 2026 16:45
@r3dbars r3dbars merged commit d9771d6 into main May 20, 2026
@r3dbars r3dbars deleted the codex/nightly-telemetry-sentry-dsyms branch May 20, 2026 16:45
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.

1 participant