Start app start up root span early to detect terminations

[bidetofevil]

Goal

Start the startup span as soon as it's reasonable that any terminations during it can result in it being captured as having failed.

Testing

Rely on existing tests for now to ensure there are no regressions. Additional tests will be added later to check for in-process termination when we're able to allow the finish to be triggered manaually

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

Package	Version	Score	Details

Scanned Files

[bidetofevil]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• Hook up API to allow manual app start trace ending #1957
• Give startup trace spans and attributes better names #1949
• Support manual ending of an app startup trace #1948
• Simplify logic to record app startup traces #1946
• Start app start up root span early to detect terminations #1944 👈 (View in Graphite)
• Re-add reverted app startup changes #1964
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codecov]

Codecov Report

Attention: Patch coverage is 76.36364% with 13 lines in your changes missing coverage. Please review.

Project coverage is 85.24%. Comparing base (44c1642) to head (2e79bb4).

Files with missing lines	Patch %	Lines
...internal/capture/startup/AppStartupTraceEmitter.kt	75.92%	3 Missing and 10 partials ⚠️

Additional details and impacted files

@@                       Coverage Diff                        @@
##           hho/reverted-startup-chnages    #1944      +/-   ##
================================================================
- Coverage                         85.25%   85.24%   -0.01%     
================================================================
  Files                               482      482              
  Lines                             11235    11221      -14     
  Branches                           1681     1682       +1     
================================================================
- Hits                               9578     9565      -13     
- Misses                              906      909       +3     
+ Partials                            751      747       -4     

Files with missing lines	Coverage Δ
...droid/embracesdk/internal/spans/EmbraceSpanImpl.kt	90.90% <100.00%> (+0.05%)	⬆️
...mbracesdk/internal/spans/PersistableEmbraceSpan.kt	100.00% <ø> (ø)
...internal/capture/startup/AppStartupTraceEmitter.kt	81.96% <75.92%> (+0.14%)	⬆️

... and 1 file with indirect coverage changes

[fractalwrench]

LGTM

[bidetofevil]

Merge activity

• Feb 26, 10:34 AM EST: A user started a stack merge that includes this pull request via Graphite.
• Feb 26, 10:34 AM EST: Graphite rebased this pull request as part of a merge.
• Feb 26, 11:06 AM EST: Graphite rebased this pull request as part of a merge.
• Feb 26, 11:40 AM EST: Graphite couldn't merge this PR because it was not satisfying all requirements (Failed CI: 'gradle-test').