@apocaliss92/scrypted-reolink-native 0.1.24 → 0.1.26

package/package.json

@@ -1,6 +1,6 @@
 {
    "name": "@apocaliss92/scrypted-reolink-native",
-   "version": "0.1.24",
+   "version": "0.1.26",
    "description": "Use any reolink camera with Scrypted, even older/unsupported models without HTTP protocol support",
    "author": "@apocaliss92",
    "license": "Apache",

package/src/baichuan-base.ts

@@ -248,11 +248,10 @@ export abstract class BaseBaichuanClass extends ScryptedDeviceBase {
                     username: config.username,
                     password: config.password,
                     uid: config.uid,
-                    logger: logger as Console,
+                    logger,
                     debugOptions: config.debugOptions,
                 },
                 transport: config.transport,
-                logger: logger as Console,
             });
 
             await api.login();

package/src/camera-battery.ts

@@ -120,8 +120,7 @@ export class ReolinkNativeBatteryCamera extends CommonCameraMixin {
 
         logger.log('Starting periodic tasks for battery camera');
 
-        // Check sleeping state every 5 seconds (non-blocking)
-        if (!this.nvrDevice) {
+        if (!this.nvrDevice && !this.multiFocalDevice) {
             this.sleepCheckTimer = setInterval(async () => {
                 try {
                     const api = this.baichuanApi;
@@ -375,18 +374,12 @@ export class ReolinkNativeBatteryCamera extends CommonCameraMixin {
         }
     }
 
-    async withBaichuanRetry<T>(fn: () => Promise<T>): Promise<T> {
-        return await fn();
-    }
-
     protected async withBaichuanClient<T>(fn: (api: ReolinkBaichuanApi) => Promise<T>): Promise<T> {
         const client = await this.ensureClient();
         return fn(client);
     }
 
     async createStreamClient(): Promise<ReolinkBaichuanApi> {
-        // Reuse the main Baichuan client connection instead of creating a new one
-        // This ensures we use a single session for everything (general + streams)
         return await this.ensureClient();
     }
 }

package/src/camera.ts

@@ -31,8 +31,8 @@ export class ReolinkNativeCamera extends CommonCameraMixin {
 
 
     constructor(
-        nativeId: string, 
-        public plugin: ReolinkNativePlugin, 
+        nativeId: string,
+        public plugin: ReolinkNativePlugin,
         nvrDevice?: ReolinkNativeNvrDevice,
         multiFocalDevice?: ReolinkNativeMultiFocalDevice
     ) {
@@ -67,27 +67,6 @@ export class ReolinkNativeCamera extends CommonCameraMixin {
         }
     }
 
-    async withBaichuanRetry<T>(fn: () => Promise<T>): Promise<T> {
-        try {
-            return await fn();
-        }
-        catch (e) {
-            if (!this.isRecoverableBaichuanError(e)) {
-                throw e;
-            }
-
-            // Reset client and clear cache on recoverable error
-            await this.resetBaichuanClient(e);
-
-            // Important: callers must re-acquire the client inside fn.
-            try {
-                return await fn();
-            } catch (retryError) {
-                throw retryError;
-            }
-        }
-    }
-
 
     async init() {
         this.startPeriodicTasks();
@@ -97,6 +76,7 @@ export class ReolinkNativeCamera extends CommonCameraMixin {
 
     async createStreamClient(): Promise<ReolinkBaichuanApi> {
         const { ipAddress, username, password } = this.storageSettings.values;
+        const logger = this.getBaichuanLogger();
 
         const debugOptions = this.getBaichuanDebugOptions();
         const api = await createBaichuanApi(
@@ -105,11 +85,10 @@ export class ReolinkNativeCamera extends CommonCameraMixin {
                     host: ipAddress,
                     username: username,
                     password: password,
-                    logger: this.console,
+                    logger,
                     debugOptions
                 },
                 transport: 'tcp',
-                logger: this.console,
             },
         );
         await api.login();

package/src/common.ts

@@ -1,14 +1,15 @@
 import type { DeviceCapabilities, PtzCommand, PtzPreset, ReolinkBaichuanApi, ReolinkSimpleEvent, ReolinkSupportedStream, StreamSamplingSelection } from "@apocaliss92/reolink-baichuan-js" with { "resolution-mode": "import" };
 import sdk, { BinarySensor, Brightness, Camera, Device, DeviceProvider, Intercom, MediaObject, MediaStreamUrl, ObjectDetectionTypes, ObjectDetector, ObjectsDetected, OnOff, PanTiltZoom, PanTiltZoomCommand, RequestMediaStreamOptions, ScryptedDeviceBase, ScryptedDeviceType, ScryptedInterface, ScryptedMimeTypes, Setting, Settings, SettingValue, VideoCamera, VideoTextOverlay, VideoTextOverlays } from "@scrypted/sdk";
 import { StorageSettings } from "@scrypted/sdk/storage-settings";
+import path from 'path';
 import type { UrlMediaStreamOptions } from "../../scrypted/plugins/rtsp/src/rtsp";
 import { BaseBaichuanClass, type BaichuanConnectionCallbacks, type BaichuanConnectionConfig } from "./baichuan-base";
 import { normalizeUid, type BaichuanTransport } from "./connect";
 import { convertDebugLogsToApiOptions, DebugLogDisplayNames, DebugLogOption, getApiRelevantDebugLogs, getDebugLogChoices } from "./debug-options";
 import { ReolinkBaichuanIntercom } from "./intercom";
 import ReolinkNativePlugin from "./main";
-import { ReolinkNativeNvrDevice } from "./nvr";
 import { ReolinkNativeMultiFocalDevice } from "./multiFocal";
+import { ReolinkNativeNvrDevice } from "./nvr";
 import { ReolinkPtzPresets } from "./presets";
 import {
     createRfc4571MediaObjectFromStreamManager,
@@ -17,10 +18,9 @@ import {
     selectStreamOption,
     StreamManager
 } from "./stream-utils";
-import { getDeviceInterfaces, updateDeviceInfo } from "./utils";
-import path from 'path';
+import { floodlightSuffix, getDeviceInterfaces, pirSuffix, sirenSuffix, updateDeviceInfo } from "./utils";
 
-export type CameraType = 'battery' | 'regular';
+export type CameraType = 'battery' | 'regular' | 'multi-focal' | 'multi-focal-battery';
 
 export interface CommonCameraMixinOptions {
     type: CameraType;
@@ -197,6 +197,12 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
                 await this.credentialsChanged();
             }
         },
+        debugEvents: {
+            title: 'Debug Events',
+            type: 'boolean',
+            immediate: true,
+            hide: true,
+        },
         username: {
             type: 'string',
             title: 'Username',
@@ -220,7 +226,7 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
             json: true,
             hide: true,
         },
-        operationChannels: {
+        multifocalInfo: {
             json: true,
             hide: true,
         },
@@ -515,7 +521,6 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
     // Abstract init method that subclasses must implement
     abstract init(): Promise<void>;
 
-    abstract withBaichuanRetry<T>(fn: () => Promise<T>): Promise<T>;
     protected withBaichuanClient?<T>(fn: (api: ReolinkBaichuanApi) => Promise<T>): Promise<T>;
     motionTimeout?: NodeJS.Timeout;
     doorbellBinaryTimeout?: NodeJS.Timeout;
@@ -527,18 +532,20 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
     thisDevice: Settings
 
     constructor(
-        nativeId: string, 
-        public plugin: ReolinkNativePlugin, 
+        nativeId: string,
+        public plugin: ReolinkNativePlugin,
         public options: CommonCameraMixinOptions
     ) {
         super(nativeId);
-        this.protocol = !options.nvrDevice && !options.multiFocalDevice && options.type === 'battery' ? 'udp' : 'tcp';
 
         // Store NVR device reference if provided
         this.nvrDevice = options.nvrDevice;
         this.multiFocalDevice = options.multiFocalDevice;
         this.thisDevice = sdk.systemManager.getDeviceById<Settings>(this.id);
 
+        const isBattery = options.type === 'battery' || options.type === 'multi-focal-battery';
+        this.protocol = isBattery ? 'udp' : 'tcp';
+
         setTimeout(async () => {
             await this.parentInit();
         }, 2000);
@@ -601,7 +608,31 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
         return this.name || 'Camera';
     }
 
-    private async runDiagnostics(): Promise<void> {
+    async withBaichuanRetry<T>(fn: () => Promise<T>): Promise<T> {
+        if (this.protocol === 'udp') {
+            return await fn();
+        } else {
+            try {
+                return await fn();
+            } catch (e) {
+                if (!this.isRecoverableBaichuanError(e)) {
+                    throw e;
+                }
+
+                // Reset client and clear cache on recoverable error
+                await this.resetBaichuanClient(e);
+
+                // Important: callers must re-acquire the client inside fn.
+                try {
+                    return await fn();
+                } catch (retryError) {
+                    throw retryError;
+                }
+            }
+        }
+    }
+
+    async runDiagnostics(): Promise<void> {
         const logger = this.getBaichuanLogger();
         const outputPath = this.storageSettings.values.diagnosticsOutputPath || process.env.SCRYPTED_PLUGIN_VOLUME || "";
 
@@ -654,8 +685,8 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
     }
 
     public getAbilities(): DeviceCapabilities {
-        if(this.options.multiFocalDevice) {
-            return this.options.multiFocalDevice.getInterfaces(this.storageSettings.values.rtspChannel).capabilities;
+        if (this.multiFocalDevice) {
+            return this.multiFocalDevice.getInterfaces(this.storageSettings.values.rtspChannel).capabilities;
         } else {
             return this.storageSettings.values.capabilities;
         }
@@ -690,9 +721,7 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
 
     // Event subscription methods
     unsubscribedToEvents(): void {
-        // Use base class unsubscribe
         this.unsubscribeFromEvents().catch(() => {
-            // ignore
         });
 
         if (this.motionDetected) {
@@ -1024,20 +1053,17 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
         this.binaryState = false;
     }
 
-    // Report devices (siren, floodlight, PIR)
     async reportDevices(): Promise<void> {
-        if (!this.nativeId || !this.name) {
-            return;
-        }
+        const abilities = this.getAbilities();
 
-        const { hasSiren, hasFloodlight, hasPir } = this.getAbilities();
+        const { hasSiren, hasFloodlight, hasPir } = abilities;
 
         const devices: Device[] = [];
 
         if (hasSiren) {
-            const sirenNativeId = `${this.nativeId}-siren`;
+            const sirenNativeId = `${this.nativeId}${sirenSuffix}`;
             devices.push({
-                providerNativeId: this.plugin?.nativeId,
+                providerNativeId: this.nativeId,
                 name: `${this.name} Siren`,
                 nativeId: sirenNativeId,
                 info: {
@@ -1049,9 +1075,9 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
         }
 
         if (hasFloodlight) {
-            const floodlightNativeId = `${this.nativeId}-floodlight`;
+            const floodlightNativeId = `${this.nativeId}${floodlightSuffix}`;
             devices.push({
-                providerNativeId: this.plugin?.nativeId,
+                providerNativeId: this.nativeId,
                 name: `${this.name} Floodlight`,
                 nativeId: floodlightNativeId,
                 info: {
@@ -1063,9 +1089,9 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
         }
 
         if (hasPir) {
-            const pirNativeId = `${this.nativeId}-pir`;
+            const pirNativeId = `${this.nativeId}${pirSuffix}`;
             devices.push({
-                providerNativeId: this.plugin?.nativeId,
+                providerNativeId: this.nativeId,
                 name: `${this.name} PIR`,
                 nativeId: pirNativeId,
                 info: {
@@ -1116,6 +1142,11 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
     async updateDeviceInfo(): Promise<void> {
         const logger = this.getBaichuanLogger();
 
+        if (this.multiFocalDevice) {
+            this.info = this.multiFocalDevice.info;
+            return;
+        }
+
         const { ipAddress, rtspChannel } = this.storageSettings.values;
         try {
             const api = await this.ensureClient();
@@ -1125,9 +1156,9 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
                 device: this,
                 ipAddress,
                 deviceData,
+                logger,
             });
 
-            logger.log(`Device info updated: ${JSON.stringify(deviceData)}`);
         } catch (e) {
             logger.warn('Failed to fetch device info', e);
         }
@@ -1135,24 +1166,24 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
 
     // Device provider methods
     async getDevice(nativeId: string): Promise<any> {
-        if (nativeId.endsWith('-siren')) {
+        if (nativeId.endsWith(sirenSuffix)) {
             this.siren ||= new ReolinkCameraSiren(this, nativeId);
             return this.siren;
-        } else if (nativeId.endsWith('-floodlight')) {
+        } else if (nativeId.endsWith(floodlightSuffix)) {
             this.floodlight ||= new ReolinkCameraFloodlight(this, nativeId);
             return this.floodlight;
-        } else if (nativeId.endsWith('-pir')) {
+        } else if (nativeId.endsWith(pirSuffix)) {
             this.pirSensor ||= new ReolinkCameraPirSensor(this, nativeId);
             return this.pirSensor;
         }
     }
 
     async releaseDevice(id: string, nativeId: string): Promise<void> {
-        if (nativeId.endsWith('-siren')) {
+        if (nativeId.endsWith(sirenSuffix)) {
             this.siren = undefined;
-        } else if (nativeId.endsWith('-floodlight')) {
+        } else if (nativeId.endsWith(floodlightSuffix)) {
             this.floodlight = undefined;
-        } else if (nativeId.endsWith('-pir')) {
+        } else if (nativeId.endsWith(pirSuffix)) {
             this.pirSensor = undefined;
         }
     }
@@ -1352,7 +1383,8 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
         }
 
         if (streams.length) {
-            logger.log('Fetched video stream options', { streams });
+            logger.log('Fetched video stream options', streams.map((s) => s.name).join(', '));
+            logger.debug(JSON.stringify(streams));
             this.cachedVideoStreamOptions = streams;
             return streams;
         }
@@ -1452,7 +1484,6 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
             });
             this.classes = objects;
             this.presets = presets;
-            this.storageSettings.values.capabilities = capabilities;
             this.ptzPresets.setCachedPtzPresets(presets);
 
             try {
@@ -1463,21 +1494,25 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
 
                 const device: Device = {
                     nativeId: this.nativeId,
-                    providerNativeId: this.nvrDevice?.nativeId ?? this.plugin?.nativeId,
+                    providerNativeId: this.nvrDevice?.nativeId ??
+                        this.multiFocalDevice?.nativeId ??
+                        this.plugin?.nativeId,
                     name: this.name,
                     interfaces,
                     type,
                     info: this.info,
                 };
 
-                logger.log(`Updating device interfaces: ${JSON.stringify(device)}`);
-
                 await sdk.deviceManager.onDeviceDiscovered(device);
+
+                logger.log(`Device interfaces updated`);
+                logger.debug(`${JSON.stringify(device)}`);
             } catch (e) {
                 logger.error('Failed to update device interfaces', e);
             }
 
-            logger.log(`Refreshed device capabilities: ${JSON.stringify({ capabilities, abilities, support, presets })}`);
+            logger.log(`Refreshed device capabilities: ${JSON.stringify(capabilities)}`);
+            logger.debug(`Refreshed device capabilities: ${JSON.stringify({ abilities, support, presets, objects })}`);
         }
         catch (e) {
             logger.error('Failed to refresh abilities', e);
@@ -1497,65 +1532,28 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
             logger.warn('Failed to update device info during init', e);
         }
 
-        try {
-            await this.refreshDeviceState();
-        }
-        catch (e) {
-            logger.warn('Failed to connect/refresh during init', e);
-        }
-
-        try {
-            await this.reportDevices();
-        }
-        catch (e) {
-            logger.warn('Failed to report devices during init', e);
-        }
-
-        const { hasIntercom, hasPtz } = this.getAbilities();
-
-        if (hasIntercom) {
-            this.intercom = new ReolinkBaichuanIntercom(this);
-        }
-
-        if (hasPtz) {
-            const choices = (this.presets || []).map((preset: any) => preset.id + '=' + preset.name);
-
-            this.storageSettings.settings.presets.choices = choices;
-            this.storageSettings.settings.ptzSelectedPreset.choices = choices;
-
-            this.storageSettings.settings.presets.hide = false;
-            this.storageSettings.settings.ptzMoveDurationMs.hide = false;
-            this.storageSettings.settings.ptzZoomStep.hide = false;
-            this.storageSettings.settings.ptzCreatePreset.hide = false;
-            this.storageSettings.settings.ptzSelectedPreset.hide = false;
-            this.storageSettings.settings.ptzUpdateSelectedPreset.hide = false;
-            this.storageSettings.settings.ptzDeleteSelectedPreset.hide = false;
-
-            this.updatePtzCaps();
+        if (!this.multiFocalDevice) {
+            try {
+                await this.refreshDeviceState();
+                await this.reportDevices();
+            }
+            catch (e) {
+                logger.warn('Failed to connect/refresh during init', e);
+            }
         }
 
-        const isBattery = this.options.type === 'battery';
         const { username, password } = this.storageSettings.values;
+        const isCamera = this.options.type === 'regular' || this.options.type === 'battery';
+        const isBatteryCamera = this.options.type === 'battery';
+        const isBatteryMultiFocal = this.options.type === 'multi-focal-battery';
+        const isBattery = isBatteryCamera || isBatteryMultiFocal;
 
-        this.streamManager = new StreamManager({
-            createStreamClient: () => this.createStreamClient(),
-            getLogger: () => logger as Console,
-            credentials: {
-                username,
-                password
-            },
-            // For battery cameras, we use a shared connection
-            sharedConnection: isBattery,
-        });
-
-
-        // this.storageSettings.settings.snapshotCacheMinutes.hide = !isBattery;
         this.storageSettings.settings.uid.hide = !isBattery;
         this.storageSettings.settings.batteryUpdateIntervalMinutes.hide = !isBattery;
         this.storageSettings.settings.lowThresholdBatteryRecording.hide = !isBattery;
         this.storageSettings.settings.highThresholdBatteryRecording.hide = !isBattery;
 
-        if (isBattery && !this.storageSettings.values.mixinsSetup) {
+        if (isBatteryCamera && !this.storageSettings.values.mixinsSetup) {
             try {
                 const device = sdk.systemManager.getDeviceById<Settings>(this.id);
                 if (device) {
@@ -1577,30 +1575,54 @@ export abstract class CommonCameraMixin extends BaseBaichuanClass implements Vid
             logger.warn('Failed to subscribe to Baichuan events', e);
         }
 
+        if (isCamera) {
+            this.streamManager = new StreamManager({
+                createStreamClient: () => this.createStreamClient(),
+                getLogger: () => logger as Console,
+                credentials: {
+                    username,
+                    password
+                },
+                sharedConnection: isBattery,
+            });
 
-        if (this.nvrDevice) {
-            this.storageSettings.settings.username.hide = true;
-            this.storageSettings.settings.password.hide = true;
-            this.storageSettings.settings.ipAddress.hide = true;
-            this.storageSettings.settings.uid.hide = true;
+            const { hasIntercom, hasPtz } = this.getAbilities();
 
-            this.storageSettings.settings.username.defaultValue = this.nvrDevice.storageSettings.values.username;
-            this.storageSettings.settings.password.defaultValue = this.nvrDevice.storageSettings.values.password;
-            this.storageSettings.settings.ipAddress.defaultValue = this.nvrDevice.storageSettings.values.ipAddress;
+            if (hasIntercom) {
+                this.intercom = new ReolinkBaichuanIntercom(this);
+            }
+
+            if (hasPtz) {
+                const choices = (this.presets || []).map((preset: any) => preset.id + '=' + preset.name);
+
+                this.storageSettings.settings.presets.choices = choices;
+                this.storageSettings.settings.ptzSelectedPreset.choices = choices;
+
+                this.storageSettings.settings.presets.hide = false;
+                this.storageSettings.settings.ptzMoveDurationMs.hide = false;
+                this.storageSettings.settings.ptzZoomStep.hide = false;
+                this.storageSettings.settings.ptzCreatePreset.hide = false;
+                this.storageSettings.settings.ptzSelectedPreset.hide = false;
+                this.storageSettings.settings.ptzUpdateSelectedPreset.hide = false;
+                this.storageSettings.settings.ptzDeleteSelectedPreset.hide = false;
+
+                this.updatePtzCaps();
+            }
         }
 
-        if (this.multiFocalDevice) {
+        if (this.nvrDevice || this.multiFocalDevice) {
             this.storageSettings.settings.username.hide = true;
             this.storageSettings.settings.password.hide = true;
             this.storageSettings.settings.ipAddress.hide = true;
             this.storageSettings.settings.uid.hide = true;
 
-            this.storageSettings.settings.username.defaultValue = this.multiFocalDevice.storageSettings.values.username;
-            this.storageSettings.settings.password.defaultValue = this.multiFocalDevice.storageSettings.values.password;
-            this.storageSettings.settings.ipAddress.defaultValue = this.multiFocalDevice.storageSettings.values.ipAddress;
+            this.storageSettings.settings.username.defaultValue = this.nvrDevice.storageSettings.values.username;
+            this.storageSettings.settings.password.defaultValue = this.nvrDevice.storageSettings.values.password;
+            this.storageSettings.settings.ipAddress.defaultValue = this.nvrDevice.storageSettings.values.ipAddress;
         }
 
         await this.init();
+
         this.initComplete = true;
     }
 }

package/src/connect.ts

@@ -19,16 +19,16 @@ export function normalizeUid(uid?: string): string | undefined {
 export async function createBaichuanApi(props: {
     inputs: BaichuanConnectInputs,
     transport: BaichuanTransport,
-    logger: Console,
 }): Promise<ReolinkBaichuanApi> {
-    const { inputs, transport, logger } = props;
+    const { inputs, transport } = props;
+    const { logger } = inputs;
     const { ReolinkBaichuanApi } = await import("@apocaliss92/reolink-baichuan-js");
 
     const base: BaichuanClientOptions = {
         host: inputs.host,
         username: inputs.username,
         password: inputs.password,
-        logger: logger, // Use the logger passed to createBaichuanApi, not inputs.logger
+        logger,
         debugOptions: inputs.debugOptions ?? {}
     };
 

package/src/main.ts

@@ -4,10 +4,9 @@ import { ReolinkNativeCamera } from "./camera";
 import { ReolinkNativeBatteryCamera } from "./camera-battery";
 import { CommonCameraMixin } from "./common";
 import { createBaichuanApi } from "./connect";
-import { ReolinkNativeMultiFocalDevice } from "./multiFocal";
 import { ReolinkNativeNvrDevice } from "./nvr";
-import { batteryCameraSuffix, cameraSuffix, getDeviceInterfaces, multifocalSuffix, nvrSuffix } from "./utils";
-import { BaichuanTransport } from "./connect";
+import { batteryCameraSuffix, batteryMultifocalSuffix, cameraSuffix, getDeviceInterfaces, multifocalSuffix, nvrSuffix } from "./utils";
+import { ReolinkNativeMultiFocalDevice } from "./multiFocal";
 
 class ReolinkNativePlugin extends ScryptedDeviceBase implements DeviceProvider, DeviceCreator {
     devices = new Map<string, BaseBaichuanClass>();
@@ -65,18 +64,28 @@ class ReolinkNativePlugin extends ScryptedDeviceBase implements DeviceProvider,
             const deviceInfo = detection.deviceInfo || {};
             const name = deviceInfo.name || 'Reolink Multi-Focal';
             const serialNumber = deviceInfo.serialNumber || deviceInfo.itemNo || `multifocal-${Date.now()}`;
-            nativeId = `${serialNumber}${multifocalSuffix}`;
+            const isBattery = detection.transport === 'udp';
+            nativeId = `${serialNumber}${isBattery ? batteryMultifocalSuffix : multifocalSuffix}`;
 
             settings.newCamera ||= name;
 
+            const interfaces = [
+                ScryptedInterface.Settings,
+                ScryptedInterface.DeviceProvider,
+                ScryptedInterface.Reboot,
+            ];
+
+            if (isBattery) {
+                interfaces.push(
+                    ScryptedInterface.Battery,
+                    ScryptedInterface.Sleep
+                );
+            }
+
             await sdk.deviceManager.onDeviceDiscovered({
                 nativeId,
                 name,
-                interfaces: [
-                    ScryptedInterface.Settings,
-                    ScryptedInterface.DeviceProvider,
-                    ScryptedInterface.Reboot,
-                ],
+                interfaces,
                 type: ScryptedDeviceType.DeviceProvider,
                 providerNativeId: this.nativeId,
             });
@@ -89,7 +98,6 @@ class ReolinkNativePlugin extends ScryptedDeviceBase implements DeviceProvider,
             device.storageSettings.values.username = username;
             device.storageSettings.values.password = password;
             device.storageSettings.values.uid = detection.uid || '';
-            device.storageSettings.values.protocol = detection.transport || 'tcp' as BaichuanTransport;
 
             return nativeId;
         }
@@ -151,15 +159,12 @@ class ReolinkNativePlugin extends ScryptedDeviceBase implements DeviceProvider,
                 logger: this.console,
             },
             transport: detection.transport,
-            logger: this.console,
         });
 
         try {
             await api.login();
             const rtspChannel = 0;
-            const { abilities, capabilities, objects, presets } = await api.getDeviceCapabilities(rtspChannel);
-
-            this.console.log(nativeId, JSON.stringify({ abilities, capabilities, deviceInfo }));
+            const { capabilities, objects, presets } = await api.getDeviceCapabilities(rtspChannel);
 
             const { interfaces, type } = getDeviceInterfaces({
                 capabilities,
@@ -175,7 +180,6 @@ class ReolinkNativePlugin extends ScryptedDeviceBase implements DeviceProvider,
             });
 
             const device = await this.getDevice(nativeId) as CommonCameraMixin;
-            this.console.log(name, interfaces, type, device);
 
             device.info = deviceInfo;
             device.classes = objects;
@@ -214,10 +218,12 @@ class ReolinkNativePlugin extends ScryptedDeviceBase implements DeviceProvider,
                 key: 'ip',
                 title: 'IP Address',
                 placeholder: '192.168.2.222',
+                value: '192.168.',
             },
             {
                 key: 'username',
                 title: 'Username',
+                value: 'admin',
             },
             {
                 key: 'password',
@@ -237,8 +243,10 @@ class ReolinkNativePlugin extends ScryptedDeviceBase implements DeviceProvider,
             return new ReolinkNativeBatteryCamera(nativeId, this);
         } else if (nativeId.endsWith(nvrSuffix)) {
             return new ReolinkNativeNvrDevice(nativeId, this);
+        } else if (nativeId.endsWith(batteryMultifocalSuffix)) {
+            return new ReolinkNativeMultiFocalDevice(nativeId, this, "multi-focal-battery");
         } else if (nativeId.endsWith(multifocalSuffix)) {
-            return new ReolinkNativeMultiFocalDevice(nativeId, this);
+            return new ReolinkNativeMultiFocalDevice(nativeId, this, "multi-focal");
         } else {
             return new ReolinkNativeCamera(nativeId, this);
         }