@switchbot/homebridge-switchbot 5.0.0 → 5.1.0-beta.0

package/src/utils.ts

@@ -1137,6 +1137,37 @@ export function normalizeConfig(raw?: PlatformConfig): SwitchBotPluginConfig {
   return { ...(raw as any) } as SwitchBotPluginConfig
 }
 
+/**
+ * Resolve the per-device external publish protocol from raw config.
+ * Accepts canonical values (`none`, `hap`, `matter`) and common aliases.
+ */
+export function resolveExternalPublishProtocol(rawDevice: any): 'none' | 'hap' | 'matter' {
+  const raw = String(
+    rawDevice?.externalPublishProtocol
+    ?? rawDevice?.externalDeviceProtocol
+    ?? rawDevice?.publishExternal
+    ?? rawDevice?.external
+    ?? 'none',
+  ).trim().toLowerCase()
+
+  if (raw === 'true') {
+    return 'hap'
+  }
+  if (raw === 'false') {
+    return 'none'
+  }
+  if (raw === 'homekit') {
+    return 'hap'
+  }
+  if (raw === 'none' || raw === 'off' || raw === 'disabled') {
+    return 'none'
+  }
+  if (raw === 'hap' || raw === 'matter') {
+    return raw
+  }
+  return 'none'
+}
+
 // Create a Proxy constructor that instantiates the right platform implementation at runtime.
 /**
  * Creates a proxy class that instantiates the correct platform implementation (HAP or Matter) at runtime.

package/test/utils/utils.spec.ts

@@ -5,6 +5,7 @@ import {
   MATTER_ATTRIBUTE_IDS,
   MATTER_CLUSTER_IDS,
   normalizeConfig,
+  resolveExternalPublishProtocol,
 } from '../../src/utils'
 
 describe('utils', () => {
@@ -50,4 +51,18 @@ describe('utils', () => {
     expect(HAP).toHaveBeenCalled()
     expect(Matter).not.toHaveBeenCalled()
   })
+
+  it('resolveExternalPublishProtocol normalizes known values and aliases', () => {
+    expect(resolveExternalPublishProtocol({ externalPublishProtocol: 'hap' })).toBe('hap')
+    expect(resolveExternalPublishProtocol({ externalPublishProtocol: 'matter' })).toBe('matter')
+    expect(resolveExternalPublishProtocol({ externalPublishProtocol: 'none' })).toBe('none')
+    expect(resolveExternalPublishProtocol({ externalPublishProtocol: 'homekit' })).toBe('hap')
+    expect(resolveExternalPublishProtocol({ publishExternal: true })).toBe('hap')
+    expect(resolveExternalPublishProtocol({ publishExternal: false })).toBe('none')
+  })
+
+  it('resolveExternalPublishProtocol defaults to none for unknown values', () => {
+    expect(resolveExternalPublishProtocol({ externalPublishProtocol: 'invalid' })).toBe('none')
+    expect(resolveExternalPublishProtocol({})).toBe('none')
+  })
 })