Added support for logging Map<Id, SObject> in core, added support for configurable fields in Slack plugin

[jongpie]

Core Package Changes

Apex Logging Improvements

• Added new instance method overload LogEntryEventBuilder.setRecord(Map<Id, SObject> recordIdToRecord)
  • For now, I've decided to not add overloads for this in Logger directly - I might in a future release, but for now, setRecord() can be called by chaining the method to one of Logger's methods, like Logger.info('some message').setRecord(recordIdToRecord)
• Updated System.debug() calls in Logger to start with 'Nebula Logger' instead of just 'Logger' for clarity

Admin Improvements

• Updated formula field LogEntry__c.LoggingLevelWithImage__c to use a different emoji for INFO (✅) vs lower levels (⚙️ for DEBUG, FINE, FINER, and FINEST)
• Added inline help text to all of the limits-related formula fields on LogEntry__c to describe the thresholds used to determine which emojis are shown in the formulas (⛔ ⚠️ ✅)
• Unlocked package-only: Added AnimationRule metadata for Log__c.Status__c to show confetti when the status changes 🥳 🎊

Bugfixes

• Fixed Null pointer Exception  #447 by updating LogEntryEventHandler to have an extra null check when processing LogEntryTagRule__mdt rules
• Fixed Default Logger Settings values change the results of unit tests in sandboxes vs. production #453 by simplifying Logger.getUserSettings() to always use default field values, and making LoggerSettingsController responsible for handling prod-specific settings
  • This keeps the behavior of Logger.getUserSettings() consistent in all orgs (instead of prod behaving differently from sandboxes/scratch orgs), while still providing the suggested defaults for a prod org via the loggerSettings LWC/custom tab
  • Some default field values on LoggerSettings__c have also been updated to provide the most detailed settings by default
  • This bugfix also led to the creation of the new public method Logger.createSettings() - this doesn't seem needed in the managed package right now, so public seems sufficient (instead of using global)

Slack Plugin Package Changes

• Updated SlackLoggerPlugin Apex class to leverage a new field set, Log__c.SlackNotificationFields, to support customization of the fields shown in Slack notifications
  • This field set can either be directly modified (although changes may be reverted during upgrades), or by creating & specifying another field set in the custom metadata record LoggerParameter.SlackLogNotificationFieldSet
  • The Slack remote site setting has also been renamed from Slack.remoteSite-meta.xml to LoggerSlack.remoteSite-meta.xml to ensure there's consistency in having some variation of the word "Log" in all metadata. Some orgs unintentionally had a previously-existing remote site setting (also named just "Slack") unintentionally overwritten - this name change should help avoid overwriting existing remote site settings for Slack.

[codecov]

Codecov Report

Patch coverage: 87.19% and project coverage change: -0.28 ⚠️

Comparison is base (4ca9535) 95.26% compared to head (bc8abe0) 94.99%.

❗ Current head bc8abe0 differs from pull request most recent head 040dbbd. Consider uploading reports for the commit 040dbbd to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #452      +/-   ##
==========================================
- Coverage   95.26%   94.99%   -0.28%     
==========================================
  Files          56       56              
  Lines        5623     5676      +53     
  Branches      114      114              
==========================================
+ Hits         5357     5392      +35     
- Misses        261      279      +18     
  Partials        5        5              

Flag	Coverage Δ
Apex	95.43% <87.19%> (-0.33%)	⬇️
LWC	92.49% <ø> (ø)

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

Impacted Files	Coverage Δ
...ore/main/configuration/classes/LoggerParameter.cls	100.00% <ø> (ø)
...s/slack/plugin/slack/classes/SlackLoggerPlugin.cls	85.02% <83.47%> (-10.31%)	⬇️
...ain/logger-engine/classes/LogEntryEventBuilder.cls	94.68% <94.11%> (+0.13%)	⬆️
...in/log-management/classes/LogEntryEventHandler.cls	90.39% <100.00%> (+0.06%)	⬆️
...og-management/classes/LoggerSettingsController.cls	98.40% <100.00%> (+0.04%)	⬆️
...-logger/core/main/logger-engine/classes/Logger.cls	95.82% <100.00%> (+0.09%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[jamessimone]

@jongpie one small question - no worries if you want to bump that API version in a subsequent release though. I think moving responsibility for what the default logging level is in a sandbox to the controller is a great change.

[jongpie]

@jamessimone awesome, thanks so much for reviewing!