fix processSignal onboarding issue if no edge fn exists

packages/signals/signals/src/core/processor/__tests__/sandbox-settings.test.ts

@@ -0,0 +1,54 @@
+import { SandboxSettings, SandboxSettingsConfig } from '../sandbox'
+
+describe(SandboxSettings, () => {
+  const edgeFnResponseBody = `function processSignal() { console.log('hello world') }`
+  const baseSettings: SandboxSettingsConfig = {
+    functionHost: undefined,
+    processSignal: undefined,
+    edgeFnDownloadURL: 'http://example.com/download',
+    edgeFnFetchClient: jest.fn().mockReturnValue(
+      Promise.resolve({
+        text: () => edgeFnResponseBody,
+      })
+    ),
+  }
+  test('initializes with provided settings', async () => {
+    const sandboxSettings = new SandboxSettings({ ...baseSettings })
+    expect(baseSettings.edgeFnFetchClient).toHaveBeenCalledWith(
+      baseSettings.edgeFnDownloadURL
+    )
+    expect(await sandboxSettings.processSignal).toEqual(edgeFnResponseBody)
+  })
+
+  test('normalizes edgeFnDownloadURL when functionHost is provided', async () => {
+    const settings: SandboxSettingsConfig = {
+      ...baseSettings,
+      processSignal: undefined,
+      functionHost: 'newHost.com',
+      edgeFnDownloadURL: 'https://original.com/download',
+    }
+    new SandboxSettings(settings)
+    expect(baseSettings.edgeFnFetchClient).toHaveBeenCalledWith(
+      'https://newHost.com/download'
+    )
+  })
+
+  test('creates default processSignal when parameters are missing', async () => {
+    const consoleWarnSpy = jest
+      .spyOn(console, 'warn')
+      .mockImplementation(() => {})
+    const settings: SandboxSettingsConfig = {
+      ...baseSettings,
+      processSignal: undefined,
+      edgeFnDownloadURL: undefined,
+    }
+    const sandboxSettings = new SandboxSettings(settings)
+    expect(await sandboxSettings.processSignal).toEqual(
+      'globalThis.processSignal = function processSignal() {}'
+    )
+    expect(baseSettings.edgeFnFetchClient).not.toHaveBeenCalled()
+    expect(consoleWarnSpy).toHaveBeenCalledWith(
+      expect.stringContaining('processSignal')
+    )
+  })
+})

packages/signals/signals/src/core/processor/sandbox.ts

@@ -149,7 +149,9 @@ export class SandboxSettings {
       this.processSignal = Promise.resolve(
         `globalThis.processSignal = function processSignal() {}`
       )
-      console.warn('edgeFnDownloadURL or processSignal must be defined')
+      console.warn(
+        `No processSignal function found. Have you written a processSignal function on app.segment.com?`
+      )
       return
     }