fix(sdk): logging null breaks script (#2140)

## Describe your changes

Fixes NAN-909

- Fix logging null should not throw an error
Added a test but `persistApi` is hard to mock so I had to use dryRun

packages/shared/lib/sdk/sync.ts

@@ -553,7 +553,7 @@ export class NangoAction {
         const lastArg = args[args.length - 1];
 
         const isUserDefinedLevel = (object: UserLogParameters): boolean => {
-            return typeof lastArg === 'object' && 'level' in object;
+            return lastArg && typeof lastArg === 'object' && 'level' in object;
         };
 
         const userDefinedLevel: UserLogParameters | undefined = isUserDefinedLevel(lastArg) ? lastArg : undefined;

packages/shared/lib/sdk/sync.unit.test.ts

@@ -395,3 +395,27 @@ describe('Pagination', () => {
         };
     };
 });
+
+describe('Log', () => {
+    it('should not log if no activityLogId', async () => {
+        const nangoAction = new NangoAction({
+            secretKey: '***',
+            providerConfigKey: 'github',
+            connectionId: 'connection-1'
+        });
+        await expect(async () => {
+            await nangoAction.log('top');
+        }).rejects.toThrowError(new Error('There is no current activity log stream to log to'));
+    });
+
+    it('should not fail on null', async () => {
+        const nangoAction = new NangoAction({
+            secretKey: '***',
+            providerConfigKey: 'github',
+            connectionId: 'connection-1',
+            dryRun: true,
+            activityLogId: 1
+        });
+        await nangoAction.log(null);
+    });
+});