Bugfix: Allowing to log SObject without ID field (#912)

* Small scope creep - removed a few unnecessary method calls in LogBatchPurger_Tests

---------

Co-authored-by: Jonathan Gillespie <jonathan.c.gillespie@gmail.com>

Author: TrangOul

README.md

@@ -5,13 +5,13 @@
 
 The most robust observability solution for Salesforce experts. Built 100% natively on the platform, and designed to work seamlessly with Apex, Lightning Components, Flow, OmniStudio, and integrations.
 
-## Unlocked Package - v4.17.0
+## Unlocked Package - v4.17.1
 
-[![Install Unlocked Package in a Sandbox](./images/btn-install-unlocked-package-sandbox.png)](https://test.salesforce.com/packaging/installPackage.apexp?p0=04tg70000000rNhAAI)
-[![Install Unlocked Package in Production](./images/btn-install-unlocked-package-production.png)](https://login.salesforce.com/packaging/installPackage.apexp?p0=04tg70000000rNhAAI)
+[![Install Unlocked Package in a Sandbox](./images/btn-install-unlocked-package-sandbox.png)](https://test.salesforce.com/packaging/installPackage.apexp?p0=04tg7000000157JAAQ)
+[![Install Unlocked Package in Production](./images/btn-install-unlocked-package-production.png)](https://login.salesforce.com/packaging/installPackage.apexp?p0=04tg7000000157JAAQ)
 [![View Documentation](./images/btn-view-documentation.png)](https://github.com/jongpie/NebulaLogger/wiki)
 
-`sf package install --wait 20 --security-type AdminsOnly --package 04tg70000000rNhAAI`
+`sf package install --wait 20 --security-type AdminsOnly --package 04tg7000000157JAAQ`
 
 ---
 

nebula-logger/core/main/logger-engine/classes/LogEntryEventBuilder.cls

@@ -460,7 +460,15 @@ global with sharing class LogEntryEventBuilder {
       this.logEntryEvent.RecordSObjectClassification__c = 'Unknown';
       this.logEntryEvent.RecordSObjectType__c = 'Unknown';
     } else {
-      this.logEntryEvent.RecordId__c = record.Id;
+      try {
+        this.logEntryEvent.RecordId__c = record.Id;
+      } catch (System.SObjectException sobjectException) {
+        // Some SObject Types (such as Platform Events) do not have the ID field;
+        // that's OK, just skip setting it.
+        if (sobjectException.getMessage().contains('Invalid field Id') == false) {
+          throw sobjectException;
+        }
+      }
       this.logEntryEvent.RecordSObjectClassification__c = getSObjectClassification(record.getSObjectType());
       this.logEntryEvent.RecordSObjectType__c = record.getSObjectType().toString();
       this.logEntryEvent.RecordSObjectTypeNamespace__c = getSObjectTypeNamespace(record.getSObjectType());

nebula-logger/core/main/logger-engine/classes/Logger.cls

@@ -15,7 +15,7 @@
 global with sharing class Logger {
   // There's no reliable way to get the version number dynamically in Apex
   @TestVisible
-  private static final String CURRENT_VERSION_NUMBER = 'v4.17.0';
+  private static final String CURRENT_VERSION_NUMBER = 'v4.17.1';
   private static final System.LoggingLevel FALLBACK_LOGGING_LEVEL = System.LoggingLevel.DEBUG;
   private static final List<LogEntryEventBuilder> LOG_ENTRIES_BUFFER = new List<LogEntryEventBuilder>();
   private static final String MISSING_SCENARIO_ERROR_MESSAGE = 'No logger scenario specified. A scenario is required for logging in this org.';

nebula-logger/core/main/logger-engine/lwc/logger/loggerService.js

@@ -10,7 +10,7 @@ import LoggerServiceTaskQueue from './loggerServiceTaskQueue';
 import getSettings from '@salesforce/apex/ComponentLogger.getSettings';
 import saveComponentLogEntries from '@salesforce/apex/ComponentLogger.saveComponentLogEntries';
 
-const CURRENT_VERSION_NUMBER = 'v4.17.0';
+const CURRENT_VERSION_NUMBER = 'v4.17.1';
 
 const CONSOLE_OUTPUT_CONFIG = {
   messagePrefix: `%c  Nebula Logger ${CURRENT_VERSION_NUMBER}  `,

nebula-logger/core/tests/log-management/classes/LogBatchPurger_Tests.cls

@@ -30,13 +30,13 @@ private class LogBatchPurger_Tests {
 
     Date scheduledDeletionDate = System.today().addDays(-7);
     Log__c log = new Log__c(LogRetentionDate__c = scheduledDeletionDate);
-    LoggerMockDataCreator.createDataBuilder(log).populateRequiredFields().getRecord();
+    LoggerMockDataCreator.createDataBuilder(log).populateRequiredFields();
     insert log;
 
     List<LogEntry__c> logEntries = new List<LogEntry__c>();
     for (Integer i = 0; i < NUMBER_OF_LOG_ENTRIES_TO_CREATE; i++) {
       LogEntry__c logEntry = new LogEntry__c(Log__c = log.Id, LoggingLevel__c = System.LoggingLevel.INFO.name());
-      LoggerMockDataCreator.createDataBuilder(logEntry).populateRequiredFields().getRecord();
+      LoggerMockDataCreator.createDataBuilder(logEntry).populateRequiredFields();
       logEntries.add(logEntry);
     }
     insert logEntries;
@@ -47,7 +47,7 @@ private class LogBatchPurger_Tests {
     List<LogEntryTag__c> logEntryTags = new List<LogEntryTag__c>();
     for (LogEntry__c logEntry : logEntries) {
       LogEntryTag__c logEntryTag = new LogEntryTag__c(LogEntry__c = logEntry.Id, Tag__c = tag.Id);
-      LoggerMockDataCreator.createDataBuilder(logEntryTag).populateRequiredFields().getRecord();
+      LoggerMockDataCreator.createDataBuilder(logEntryTag).populateRequiredFields();
       logEntryTags.add(logEntryTag);
     }
     insert logEntryTags;

nebula-logger/core/tests/logger-engine/classes/LogEntryEventBuilder_Tests.cls

@@ -1461,7 +1461,7 @@ private class LogEntryEventBuilder_Tests {
   }
 
   @IsTest
-  static void it_should_set_record_fields_for_record_when_platform_event() {
+  static void it_should_set_record_fields_for_record_when_unpublished_platform_event() {
     LogEntryEventBuilder builder = new LogEntryEventBuilder(getUserSettings(), System.LoggingLevel.INFO, true);
     System.Assert.isNull(builder.getLogEntryEvent().RecordId__c);
     System.Assert.isNull(builder.getLogEntryEvent().RecordJson__c);
@@ -1470,13 +1470,37 @@ private class LogEntryEventBuilder_Tests {
     System.Assert.isNull(builder.getLogEntryEvent().RecordSObjectTypeNamespace__c);
 
     // To avoid creating a new platform event just for testing purposes, Nebula Logger's LogEntryEvent__e is reused
-    LogEntryEvent__e platformEvent = new LogEntryEvent__e();
-    builder.setRecordId(platformEvent);
+    LogEntryEvent__e unpublishedPlatformEvent = new LogEntryEvent__e();
+    builder.setRecordId(unpublishedPlatformEvent);
 
     System.Assert.areEqual(1, builder.getLogEntryEvent().RecordCollectionSize__c);
     System.Assert.areEqual('Single', builder.getLogEntryEvent().RecordCollectionType__c);
     System.Assert.isNull(builder.getLogEntryEvent().RecordId__c);
-    System.Assert.areEqual(System.JSON.serializePretty(platformEvent), builder.getLogEntryEvent().RecordJson__c);
+    System.Assert.areEqual(System.JSON.serializePretty(unpublishedPlatformEvent), builder.getLogEntryEvent().RecordJson__c);
+    System.Assert.areEqual('Platform Event Object', builder.getLogEntryEvent().RecordSObjectClassification__c);
+    System.Assert.areEqual(LogEntryEvent__e.SObjectType.toString(), builder.getLogEntryEvent().RecordSObjectType__c);
+  }
+
+  @IsTest
+  static void it_should_set_record_fields_for_record_when_published_platform_event() {
+    LogEntryEventBuilder builder = new LogEntryEventBuilder(getUserSettings(), System.LoggingLevel.INFO, true);
+    System.Assert.isNull(builder.getLogEntryEvent().RecordId__c);
+    System.Assert.isNull(builder.getLogEntryEvent().RecordJson__c);
+    System.Assert.isNull(builder.getLogEntryEvent().RecordSObjectClassification__c);
+    System.Assert.isNull(builder.getLogEntryEvent().RecordSObjectType__c);
+    System.Assert.isNull(builder.getLogEntryEvent().RecordSObjectTypeNamespace__c);
+
+    // To avoid creating a new platform event just for testing purposes, Nebula Logger's LogEntryEvent__e is reused
+    LogEntryEvent__e publishedPlatformEvent = new LogEntryEvent__e();
+    LoggerMockDataCreator.createDataBuilder(publishedPlatformEvent).populateRequiredFields();
+    System.EventBus.publish(publishedPlatformEvent);
+    System.Test.getEventBus().deliver();
+    builder.setRecordId(publishedPlatformEvent);
+
+    System.Assert.areEqual(1, builder.getLogEntryEvent().RecordCollectionSize__c);
+    System.Assert.areEqual('Single', builder.getLogEntryEvent().RecordCollectionType__c);
+    System.Assert.isNull(builder.getLogEntryEvent().RecordId__c);
+    System.Assert.areEqual(System.JSON.serializePretty(publishedPlatformEvent), builder.getLogEntryEvent().RecordJson__c);
     System.Assert.areEqual('Platform Event Object', builder.getLogEntryEvent().RecordSObjectClassification__c);
     System.Assert.areEqual(LogEntryEvent__e.SObjectType.toString(), builder.getLogEntryEvent().RecordSObjectType__c);
   }

nebula-logger/managed-package/sfdx-project.json

@@ -11,10 +11,10 @@
       "postInstallScript": "LoggerInstallHandler",
       "scopeProfiles": true,
       "ancestorVersion": "HIGHEST",
-      "versionNumber": "4.17.0.NEXT",
-      "versionName": "Summer '25 Release",
-      "versionDescription": "View the v4.17.0 milestone in GitHub for the list of changes - https://github.com/jongpie/NebulaLogger/milestone/17?closed=1",
-      "releaseNotesUrl": "https://github.com/jongpie/NebulaLogger/releases/tag/v4.17.0"
+      "versionNumber": "4.18.0.NEXT",
+      "versionName": "Winter '26 Release",
+      "versionDescription": "View the v4.18.0 milestone in GitHub for the list of changes - https://github.com/jongpie/NebulaLogger/milestone/18?closed=1",
+      "releaseNotesUrl": "https://github.com/jongpie/NebulaLogger/releases/tag/v4.18.0"
     }
   ],
   "packageAliases": {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "nebula-logger",
-  "version": "4.17.0",
+  "version": "4.17.1",
   "description": "The most robust logger for Salesforce. Works with Apex, Lightning Components, Flow, Process Builder & Integrations. Designed for Salesforce admins, developers & architects.",
   "author": "Jonathan Gillespie",
   "license": "MIT",

sfdx-project.json

@@ -9,9 +9,9 @@
       "path": "./nebula-logger/core",
       "definitionFile": "./config/scratch-orgs/base-scratch-def.json",
       "scopeProfiles": true,
-      "versionNumber": "4.17.0.NEXT",
-      "versionName": "Summer '25 Release",
-      "versionDescription": "Updated all metadata to API v64.0, added logEntryPageSection LWC to LogEntry__c page, improved testing in Apex & LWC",
+      "versionNumber": "4.17.1.NEXT",
+      "versionName": "Bugfix: Allowing to log SObject without ID field",
+      "versionDescription": "Updated LogEntryEventBuilder to handle the few types of SObjects that do not have an Id field (most notably, Platform Events)",
       "postInstallUrl": "https://github.com/jongpie/NebulaLogger/wiki",
       "releaseNotesUrl": "https://github.com/jongpie/NebulaLogger/releases",
       "unpackagedMetadata": {
@@ -222,6 +222,7 @@
     "Nebula Logger - Core@4.16.4-bugfix:-logger-lwc-initialization-message": "04tKe0000011MyWIAU",
     "Nebula Logger - Core@4.16.5-capture-apex-cursor-transaction-limits": "04tKe0000011N4KIAU",
     "Nebula Logger - Core@4.17.0-summer-'25-release": "04tg70000000rNhAAI",
+    "Nebula Logger - Core@4.17.1-bugfix:-allowing-to-log-sobject-without-id-field": "04tg7000000157JAAQ",
     "Nebula Logger - Core Plugin - Async Failure Additions": "0Ho5Y000000blO4SAI",
     "Nebula Logger - Core Plugin - Async Failure Additions@1.0.0": "04t5Y0000015lhiQAA",
     "Nebula Logger - Core Plugin - Async Failure Additions@1.0.1": "04t5Y0000015lhsQAA",