Bugfix: Removed most of the usage of UserInfo.getSessionId()

[jongpie]

Core Unlocked Package Changes

Fixed #761 and fixed #964 by updating Logger Apex class to remove most of the usage of System.UserInfo.getSessionId() - it can't cause problems if we don't use it 😆

• Now the only place that the session ID is referenced is when using the REST save method. The user session ID is synchronously passed to the internal class RestApiSaver - and passing it like this will (intentionally) force a synchronous runtime exception when there isn't a session ID (instead of it silently failing async)

To help validate that this change doesn't cause any regressions, I've added some extra metadata (used only in the pipeline & testing) to recreate previously reported issues related to quiddity & user session ID:

• Issue Logging from an Email Service does not generate a log #155 - Added an example email service, ExampleInboundEmailService, to have a full solution to re-test. Previously, I only had the Apex class ExampleInboundEmailHandler, but I needed the service itself to do a full end-to-end test.
• Issue Logging from auth provider class in Experience Cloud throws an error #224 - Added Experience Cloud metadata SSO logins, auth provider, and custom registration handlers, to have a full solution to re-test.

With the extra metadata in place & the changes in Logger, everything still seems to be working as expected (without the use of System.UserInfo.getSessionId()). More testing will be done prior to merging this.

Pipeline Changes

Pipeline overhaul: made some big restructuring of .github/workflows/build.yml for maintainability and speed improvements

• Made two composite actions (setup-npm and authorize-dev-hub) for some stuff that's used throughout the build (and was previously duplicated a bunch)
• Pulled out the scratch-org test logic into a reusable workflow (test-scratch-org.yml), with parameters to control scratch def file, OmniStudio toggle, optional extra-metadata directory, and Codecov toggle
• Updated build.yml to use the new actions & workflows, which eliminates a ton of duplication
• Moved the scratch org tests a single matrix-driven caller. Adding a new scratch org now just requires a few new lines to be added to the matrix.
• All scratch orgs now run in parallel instead of in batches of 3 (a huge thanks to thanks @jamessimone and @mitchspano for getting the dev hub limit increased 🥳)
• Added a new state-and-country-picklists scratch org as a small follow-up to issue Testing Error in Orgs with State & Country Code Picklists #976
• Added a new hourly cleanup workflow (delete-stale-scratch-orgs.yml) to delete orphaned pipeline scratch orgs older than 2 hours (it'll skip any scratch orgs tied to GitHub Actions that are still running)

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.79%. Comparing base (05ce16c) to head (39d4b0f).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #995      +/-   ##
==========================================
+ Coverage   93.76%   93.79%   +0.03%     
==========================================
  Files          58       58              
  Lines        6335     6319      -16     
  Branches      204      204              
==========================================
- Hits         5940     5927      -13     
+ Misses        393      390       -3     
  Partials        2        2              

Flag	Coverage Δ
Apex	93.12% <100.00%> (+0.03%)	⬆️
LWC	96.27% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.