@planningcenter/chat-react-native 3.37.0-rc.1 → 3.37.0-rc.3

package/src/utils/native_adapters/__tests__/log.test.ts

@@ -0,0 +1,62 @@
+import { LogAdapter } from '../log'
+
+describe('LogAdapter', () => {
+  describe('reportError', () => {
+    it('forwards the error and context to the host callback', () => {
+      const reportError = jest.fn()
+      const adapter = new LogAdapter({ reportError })
+      const error = new Error('boom')
+      const context = { scope: 'screen' as const, screenName: 'ConversationScreen' }
+
+      adapter.reportError(error, context)
+
+      expect(reportError).toHaveBeenCalledWith(error, context)
+    })
+
+    it('fires even when the adapter is disabled', () => {
+      const reportError = jest.fn()
+      const adapter = new LogAdapter({ enabled: false, reportError })
+
+      adapter.reportError(new Error('boom'))
+
+      expect(reportError).toHaveBeenCalledTimes(1)
+    })
+
+    it('falls back to console.error when no host callback is configured', () => {
+      const spy = jest.spyOn(console, 'error').mockImplementation(() => {})
+      const adapter = new LogAdapter()
+      const error = new Error('boom')
+
+      adapter.reportError(error, { scope: 'http' })
+
+      expect(spy).toHaveBeenCalledWith(error, { scope: 'http' })
+      spy.mockRestore()
+    })
+  })
+
+  describe('info and error', () => {
+    it('drops info and error calls when disabled', () => {
+      const info = jest.fn()
+      const error = jest.fn()
+      const adapter = new LogAdapter({ enabled: false, info, error })
+
+      adapter.info('hi')
+      adapter.error('bye')
+
+      expect(info).not.toHaveBeenCalled()
+      expect(error).not.toHaveBeenCalled()
+    })
+
+    it('forwards info and error calls when enabled', () => {
+      const info = jest.fn()
+      const error = jest.fn()
+      const adapter = new LogAdapter({ enabled: true, info, error })
+
+      adapter.info('hi', { a: 1 })
+      adapter.error('bye', { b: 2 })
+
+      expect(info).toHaveBeenCalledWith('hi', { a: 1 })
+      expect(error).toHaveBeenCalledWith('bye', { b: 2 })
+    })
+  })
+})

package/src/utils/native_adapters/log.ts

@@ -1,18 +1,31 @@
+export type ReportErrorScope = 'screen' | 'component' | 'http'
+
+export type ReportErrorContext = {
+  componentStack?: string
+  scope?: ReportErrorScope
+  screenName?: string
+  tags?: Record<string, string>
+  extra?: Record<string, unknown>
+}
+
 interface LogAdapterConfig {
   enabled: boolean
   error: (message: string, ...args: any[]) => void
   info: (message: string, ...args: any[]) => void
+  reportError: (error: Error, context?: ReportErrorContext) => void
 }
 
 export class LogAdapter {
   enabled: boolean
   onError?: LogAdapterConfig['error']
   onInfo?: LogAdapterConfig['info']
+  onReportError?: LogAdapterConfig['reportError']
 
   constructor(config: Partial<LogAdapterConfig> = {}) {
     this.enabled = config.enabled ?? false
     this.onError = config.error
     this.onInfo = config.info
+    this.onReportError = config.reportError
   }
 
   error(message: string, ...args: any[]) {
@@ -26,4 +39,13 @@ export class LogAdapter {
 
     this.onInfo?.(message, ...args)
   }
+
+  reportError(error: Error, context?: ReportErrorContext) {
+    if (this.onReportError) {
+      this.onReportError(error, context)
+      return
+    }
+
+    console.error(error, context)
+  }
 }