@nordicsemiconductor/pc-nrfconnect-shared 230.0.0 → 231.0.0

package/src/Parsers/shellParser.ts

@@ -52,7 +52,7 @@ export const xTerminalShellParserWrapper = (terminal: Terminal) => ({
     clear: () => terminal.clear(),
     getLastLine: () => {
         const lastLine = terminal.buffer.active.getLine(
-            terminal.buffer.active.cursorY
+            terminal.buffer.active.cursorY,
         );
         if (typeof lastLine === 'undefined') {
             return '';
@@ -60,7 +60,7 @@ export const xTerminalShellParserWrapper = (terminal: Terminal) => ({
 
         return lastLine.translateToString().trim();
     },
-    write: (data: string, callback: () => void | undefined) =>
+    write: (data: string, callback: () => void) =>
         terminal.write(data, callback),
 });
 
@@ -74,7 +74,7 @@ export const shellParser = async (
         timeout: 1000,
         columnWidth: 80,
     },
-    shellEchos = true
+    shellEchos = true,
 ) => {
     const eventEmitter = new EventEmitter();
 
@@ -89,7 +89,7 @@ export const shellParser = async (
 
     if (await serialPort.isOpen()) {
         serialPort.write(
-            `${String.fromCharCode(12)}${String.fromCharCode(21)}`
+            `${String.fromCharCode(12)}${String.fromCharCode(21)}`,
         );
     }
 
@@ -113,7 +113,7 @@ export const shellParser = async (
                         if (callback.onTimeout)
                             callback.onTimeout(
                                 `Callback timeout after ${elapsedTime}ms`,
-                                command.command
+                                command.command,
                             );
                     });
 
@@ -156,7 +156,7 @@ export const shellParser = async (
     const responseCallback = (commandAndResponse: string) => {
         if (settings.shellPromptUart == null) {
             logger.error(
-                'Shell Parser: responseCallback: Shell prompt has not been set'
+                'Shell Parser: responseCallback: Shell prompt has not been set',
             );
             return;
         }
@@ -216,12 +216,12 @@ export const shellParser = async (
                     .trim();
                 if (isError) {
                     commandQueue[0].callbacks.forEach(callback =>
-                        callback.onError(response, commandQueue[0].command)
+                        callback.onError(response, commandQueue[0].command),
                     );
                     callbackFound = true;
                 } else {
                     commandQueue[0].callbacks.forEach(callback =>
-                        callback.onSuccess(response, commandQueue[0].command)
+                        callback.onSuccess(response, commandQueue[0].command),
                     );
                     callbackFound = true;
                 }
@@ -285,7 +285,7 @@ export const shellParser = async (
 
         if (settings.shellPromptUart == null) {
             logger.debug(
-                'Shell Parser: loadToBuffer: Shell prompt has not been set'
+                'Shell Parser: loadToBuffer: Shell prompt has not been set',
             );
             return;
         }
@@ -295,7 +295,7 @@ export const shellParser = async (
                 'shellLogging',
                 commandBuffer
                     .replace(settings.shellPromptUart.trim(), '')
-                    .trim()
+                    .trim(),
             ); // TODO Consider improving tests instead of cleaning this here
             commandBuffer = '';
             return;
@@ -318,13 +318,13 @@ export const shellParser = async (
             if (match) {
                 settings.shellPromptUart = `${match[0]} `;
                 logger.debug(
-                    `Shell Parser: found shell prompt: "${settings.shellPromptUart}"`
+                    `Shell Parser: found shell prompt: "${settings.shellPromptUart}"`,
                 );
             }
         } else {
             commandBuffer = parseShellCommands(
                 commandBuffer,
-                settings.shellPromptUart.trim()
+                settings.shellPromptUart.trim(),
             );
         }
 
@@ -336,7 +336,7 @@ export const shellParser = async (
     const unregisterOnClosed = serialPort.onClosed(() => {
         if (settings.shellPromptUart == null) {
             logger.debug(
-                'Shell Parser: unregisterOnClosed: Shell prompt has not been set'
+                'Shell Parser: unregisterOnClosed: Shell prompt has not been set',
             );
             return;
         }
@@ -347,7 +347,7 @@ export const shellParser = async (
 
         commandBuffer = parseShellCommands(
             commandBuffer,
-            settings.shellPromptUart.trim()
+            settings.shellPromptUart.trim(),
         );
 
         xTerminalShellParser.clear();
@@ -431,7 +431,7 @@ export const shellParser = async (
                 command: string;
                 response: string;
                 error: boolean;
-            }) => void
+            }) => void,
         ) => {
             eventEmitter.on('anyCommandResponse', handler);
             return () => {
@@ -448,13 +448,13 @@ export const shellParser = async (
             command: string,
             callbacks?: Callbacks,
             timeout?: number,
-            unique = false
+            unique = false,
         ) => {
             command = command.trim();
 
             if (unique) {
                 const existingCommand = commandQueue.find(
-                    item => item.command === command
+                    item => item.command === command,
                 );
                 if (existingCommand) {
                     if (callbacks) existingCommand.callbacks.push(callbacks);
@@ -462,11 +462,11 @@ export const shellParser = async (
                     if (timeout !== undefined && existingCommand.sent) {
                         console.warn(
                             `Timeout of ${timeout} for command ${command} has been ignored as command
-                            has already been sent. Timeout of ${existingCommand.timeout} was used.`
+                            has already been sent. Timeout of ${existingCommand.timeout} was used.`,
                         );
                     } else if (timeout !== undefined) {
                         console.warn(
-                            `Timeout for command ${command} has been updated to ${timeout}`
+                            `Timeout for command ${command} has been updated to ${timeout}`,
                         );
                         existingCommand.timeout = timeout;
                     }
@@ -491,7 +491,7 @@ export const shellParser = async (
         registerCommandCallback: (
             command: string,
             onSuccess: (data: string, command: string) => void,
-            onError: (error: string, command: string) => void
+            onError: (error: string, command: string) => void,
         ) => {
             // Add Callbacks to the queue for future responses
             const callbacks = { onSuccess, onError };

package/src/PseudoButton/PseudoButton.tsx

@@ -19,7 +19,7 @@ const invokeIfSpaceOrEnterPressed =
 
 const blurAndInvoke =
     (
-        onClick: React.MouseEventHandler<HTMLElement>
+        onClick: React.MouseEventHandler<HTMLElement>,
     ): React.MouseEventHandler<HTMLElement> =>
     (event: React.MouseEvent<HTMLElement>) => {
         event.stopPropagation();

package/src/SerialPort/ConflictingSettingsDialog.tsx

@@ -19,7 +19,7 @@ import {
 
 const getCurrentOptions = async (
     portPath: string,
-    setSettings: (options: SerialPortOpenOptions<AutoDetectTypes>) => void
+    setSettings: (options: SerialPortOpenOptions<AutoDetectTypes>) => void,
 ) => {
     try {
         const options = await getSerialPortOptions(portPath);
@@ -57,7 +57,7 @@ const ConflictingSettingsDialog = ({
 
     const connectToSelectedSerialPort = async (
         overwrite: boolean,
-        newSettings: SerialPortOpenOptions<AutoDetectTypes>
+        newSettings: SerialPortOpenOptions<AutoDetectTypes>,
     ) => {
         try {
             const port = await createSerialPort(newSettings, {
@@ -71,7 +71,7 @@ const ConflictingSettingsDialog = ({
                 onCancel();
             } else {
                 console.error(
-                    'Port could not be opened. Verify it is not used by other applications.'
+                    'Port could not be opened. Verify it is not used by other applications.',
                 );
             }
         }
@@ -93,7 +93,7 @@ const ConflictingSettingsDialog = ({
                     connectToSelectedSerialPort(true, activeSettings);
                 } else {
                     logger.error(
-                        'Could not get the active serial port settings.'
+                        'Could not get the active serial port settings.',
                     );
                 }
             }}
@@ -151,7 +151,7 @@ const DisplayConflictingSettings = ({
         ? allKeys.filter(
               key =>
                   activeSettings[key as keyof typeof activeSettings] !==
-                  localSettings[key as keyof typeof localSettings]
+                  localSettings[key as keyof typeof localSettings],
           )
         : [];
 
@@ -167,14 +167,14 @@ const DisplayConflictingSettings = ({
                                 className={`tw-p-1 ${classNames(
                                     conflictingSettings.includes(key)
                                         ? 'tw-bg-red-100'
-                                        : 'tw-bg-green-100'
+                                        : 'tw-bg-green-100',
                                 )}`}
                             >
                                 <span className="tw-font-semibold">{`${key}: `}</span>
                                 {prettifyValue(
                                     activeSettings[
                                         key as keyof typeof activeSettings
-                                    ]
+                                    ],
                                 )}
                             </li>
                         ))}
@@ -191,13 +191,15 @@ const DisplayConflictingSettings = ({
                             className={`tw-p-1 ${classNames(
                                 conflictingSettings.includes(key)
                                     ? 'tw-bg-red-100'
-                                    : 'tw-bg-green-100'
+                                    : 'tw-bg-green-100',
                             )}`}
                             key={key}
                         >
                             <span className="tw-font-semibold">{`${key}: `}</span>
                             {prettifyValue(
-                                localSettings[key as keyof typeof localSettings]
+                                localSettings[
+                                    key as keyof typeof localSettings
+                                ],
                             )}
                         </li>
                     ))}

package/src/SerialPort/SerialPort.test.ts

@@ -17,7 +17,7 @@ test('SerialPort is initialized with the correct setup', async () => {
     expect(ipcRenderer.invoke).toHaveBeenCalledWith(
         'serialport:open',
         defaultOptions,
-        { overwrite: false, settingsLocked: false }
+        { overwrite: false, settingsLocked: false },
     );
 
     expect(port.path).toBe(defaultOptions.path);
@@ -29,7 +29,7 @@ test('SerialPort may be initialized with overwrite and settingsLocked', async ()
     expect(ipcRenderer.invoke).toHaveBeenCalledWith(
         'serialport:open',
         defaultOptions,
-        overwriteOptions
+        overwriteOptions,
     );
 
     overwriteOptions = { settingsLocked: true };
@@ -37,7 +37,7 @@ test('SerialPort may be initialized with overwrite and settingsLocked', async ()
     expect(ipcRenderer.invoke).toHaveBeenCalledWith(
         'serialport:open',
         defaultOptions,
-        overwriteOptions
+        overwriteOptions,
     );
 
     overwriteOptions = { settingsLocked: true, overwrite: true };
@@ -45,6 +45,6 @@ test('SerialPort may be initialized with overwrite and settingsLocked', async ()
     expect(ipcRenderer.invoke).toHaveBeenCalledWith(
         'serialport:open',
         defaultOptions,
-        overwriteOptions
+        overwriteOptions,
     );
 });

package/src/SerialPort/SerialPort.ts

@@ -22,17 +22,17 @@ export const createSerialPort = async (
     overwriteOptions: OverwriteOptions = {
         overwrite: false,
         settingsLocked: false,
-    }
+    },
 ) => {
     const { path } = options;
     const eventEmitter = new EventEmitter();
     let closed = false;
 
     forMain.registerOnDataReceived(path)(data =>
-        eventEmitter.emit('onData', data)
+        eventEmitter.emit('onData', data),
     );
     forMain.registerOnDataWritten(path)(data =>
-        eventEmitter.emit('onDataWritten', data)
+        eventEmitter.emit('onDataWritten', data),
     );
 
     forMain.registerOnClosed(path)(() => {
@@ -40,29 +40,33 @@ export const createSerialPort = async (
         closed = true;
     });
     forMain.registerOnUpdated(path)(newOptions =>
-        eventEmitter.emit('onUpdate', newOptions)
+        eventEmitter.emit('onUpdate', newOptions),
     );
     forMain.registerOnSet(path)(newOptions =>
-        eventEmitter.emit('onSet', newOptions)
+        eventEmitter.emit('onSet', newOptions),
     );
     forMain.registerOnChanged(path)(newOptions =>
-        eventEmitter.emit('onChange', newOptions)
+        eventEmitter.emit('onChange', newOptions),
     );
 
     const openWithRetries = async (retryCount: number) => {
         try {
-            return await inMain.open(options, overwriteOptions);
+            await inMain.open(options, overwriteOptions);
+            return undefined;
         } catch (error) {
             if (
                 (error as Error).message.includes(
-                    'PORT_IS_ALREADY_BEING_OPENED'
+                    'PORT_IS_ALREADY_BEING_OPENED',
                 ) &&
                 retryCount > 0
             ) {
-                return new Promise<void | string>(resolve => {
-                    setTimeout(async () => {
-                        resolve(await openWithRetries(retryCount - 1));
-                    }, 50 + Math.random() * 100);
+                return new Promise<string | undefined>(resolve => {
+                    setTimeout(
+                        async () => {
+                            resolve(await openWithRetries(retryCount - 1));
+                        },
+                        50 + Math.random() * 100,
+                    );
                 });
             }
             return (error as Error).message;
@@ -76,7 +80,7 @@ export const createSerialPort = async (
         try {
             if (await inMain.isOpen(path)) {
                 logger.info(
-                    `Port ${options.path} still in use by other window(s)`
+                    `Port ${options.path} still in use by other window(s)`,
                 );
             }
         } catch {
@@ -95,11 +99,11 @@ export const createSerialPort = async (
     if (error) {
         if (error.includes('FAILED_DIFFERENT_SETTINGS')) {
             logger.warn(
-                `Failed to connect to port: ${path}. Port is open with different settings.`
+                `Failed to connect to port: ${path}. Port is open with different settings.`,
             );
         } else {
             logger.error(
-                `Failed to connect to port: ${path}. Make sure the port is not already taken, if you are not sure, try to power cycle the device and try to connect again.`
+                `Failed to connect to port: ${path}. Make sure the port is not already taken, if you are not sure, try to power cycle the device and try to connect again.`,
             );
         }
         throw new Error(error);
@@ -144,8 +148,8 @@ export const createSerialPort = async (
         },
         onChange: (
             handler: (
-                newOptions: SerialPortOpenOptions<AutoDetectTypes>
-            ) => void
+                newOptions: SerialPortOpenOptions<AutoDetectTypes>,
+            ) => void,
         ) => {
             eventEmitter.on('onChange', handler);
             return () => {
@@ -166,7 +170,7 @@ export const getSerialPortOptions = async (path: string) => {
         console.log('will fetch options from path=', path);
 
         return await inMain.getOptions(path);
-    } catch (error) {
+    } catch {
         logger.error(`Failed to get options for port: ${path}`);
     }
 };

package/src/Slider/Handle.tsx

@@ -54,7 +54,7 @@ const Handle: FC<Props> = ({
     const onChangeCompleteRef = useAutoupdatingRef(onChangeComplete);
 
     const grabHandle: MouseEventHandler<HTMLDivElement> = (
-        event: React.MouseEvent
+        event: React.MouseEvent,
     ) => {
         const sliderWidthStillUnknown = sliderWidth == null;
         if (sliderWidthStillUnknown) return;
@@ -78,7 +78,7 @@ const Handle: FC<Props> = ({
 
         const oldPercentage = onMouseDragStart.current?.percentage as number;
         const newPercentage = constrainedToPercentage(
-            oldPercentage - percentageChange
+            oldPercentage - percentageChange,
         );
 
         const lastValue = onMouseDragStart.current?.lastValue as number;
@@ -87,7 +87,7 @@ const Handle: FC<Props> = ({
             lastValue,
             newPercentage,
             range,
-            event.movementX > 0
+            event.movementX > 0,
         );
         onMouseDragStart.current = {
             mousePosition: oldMousePosition,

package/src/Slider/Slider.test.tsx

@@ -18,8 +18,8 @@ describe('Slider component', () => {
                     values={[2]}
                     range={{ min: 0, max: 5 }}
                     onChange={[() => {}]}
-                />
-            ).baseElement
+                />,
+            ).baseElement,
         ).toMatchSnapshot();
     });
 });

package/src/Slider/Slider.tsx

@@ -18,14 +18,14 @@ import './slider.scss';
 
 const useValidatedArraySizes = (
     values: readonly number[],
-    onChange: ((v: number) => void)[]
+    onChange: ((v: number) => void)[],
 ) => {
     useEffect(() => {
         if (values.length === 0)
             console.error('"values" must contain at least on element');
         if (values.length !== onChange.length)
             console.error(
-                `Props 'values' and 'onChange' must have the same size but were ${values} and ${onChange}`
+                `Props 'values' and 'onChange' must have the same size but were ${values} and ${onChange}`,
             );
     }, [onChange, values]);
 };

package/src/Slider/Ticks.tsx

@@ -21,7 +21,7 @@ const Ticks: FC<Props> = ({ valueRange, range: rangeOrValues }) => {
     useEffect(() => {
         if (isValues(rangeOrValues)) {
             console.error(
-                'Ticks are not yet implemented for explicit values. Not showing ticks.'
+                'Ticks are not yet implemented for explicit values. Not showing ticks.',
             );
         }
     }, [rangeOrValues]);
@@ -43,7 +43,7 @@ const Ticks: FC<Props> = ({ valueRange, range: rangeOrValues }) => {
                     key={String(value)}
                     className={classNames(
                         'tick',
-                        isSelected(value) && 'selected'
+                        isSelected(value) && 'selected',
                     )}
                 />
             ))}

package/src/Slider/percentage.ts

@@ -14,7 +14,7 @@ export const constrainedToPercentage = (percentage: number) => {
 
 export const toPercentage = (
     valueOrIndex: number,
-    rangeOrValues: RangeOrValues
+    rangeOrValues: RangeOrValues,
 ) => {
     if (isValues(rangeOrValues)) {
         const max = Math.max(0, rangeOrValues.length - 1);
@@ -32,7 +32,7 @@ export const fromPercentage = (
     lastValue: number,
     value: number,
     rangeOrValues: RangeOrValues,
-    directionForward: boolean
+    directionForward: boolean,
 ) => {
     const min = getMin(rangeOrValues);
     const max = getMax(rangeOrValues);

package/src/Slider/range.ts

@@ -19,7 +19,7 @@ export type Range = {
 export type RangeOrValues = Range | Values;
 
 export const isValues = (
-    rangeOrValues: RangeOrValues
+    rangeOrValues: RangeOrValues,
 ): rangeOrValues is Values => Array.isArray(rangeOrValues);
 
 export const getMin = (rangeOrValues: RangeOrValues) =>
@@ -42,30 +42,30 @@ const assert = (expectedToBeTrue: boolean, warning: string) => {
 const validateRange = (range: Range) => {
     assert(
         !(`explicitRange` in range),
-        'Using `explicitRange` is not supported anymore, use only an array instead of a range object.'
+        'Using `explicitRange` is not supported anymore, use only an array instead of a range object.',
     );
 
     assert(
         range.min < range.max,
-        `range.min must not be higher than range.max: ${JSON.stringify(range)}`
+        `range.min must not be higher than range.max: ${JSON.stringify(range)}`,
     );
 
     if (range.decimals != null)
         assert(
             range.decimals >= 0 && Number.isInteger(range.decimals),
-            `range.decimals must be non-negative integer but is ${range.decimals}`
+            `range.decimals must be non-negative integer but is ${range.decimals}`,
         );
 
     if (range.step != null) {
         assert(
             range.step > 0,
-            `range.step must be larger than zero but is ${range.step}`
+            `range.step must be larger than zero but is ${range.step}`,
         );
         assert(
             isFactor(range.max - range.min, range.step),
             `range.step must be a factor of range.max - range.min: ${JSON.stringify(
-                range
-            )}`
+                range,
+            )}`,
         );
     }
 };

package/src/StartStopButton/StartStopButton.tsx

@@ -47,7 +47,7 @@ const StartStopButton: FC<Props> = ({
             className={classNames(
                 'start-stop',
                 `${started ? 'active-animation' : ''}`,
-                className
+                className,
             )}
             disabled={disabled}
             size={large ? 'lg' : 'sm'}

package/src/StateSelector/StateSelector.tsx

@@ -47,7 +47,7 @@ const SwitchButton: React.FC<{
                 'tw-border-gray-700 tw-bg-white tw-text-gray-700',
             variant === 'unset' &&
                 'tw-border-white tw-bg-gray-700 tw-text-white',
-            active && 'active tw-z-[1]'
+            active && 'active tw-z-[1]',
         )}`}
         onClick={onClick}
         disabled={disabled}

package/src/Stepper/Stepper.tsx

@@ -47,7 +47,7 @@ const isActionCaption = (caption: StepCaption): caption is ActionCaption =>
     (caption as ActionCaption).action !== undefined;
 
 const isStringCaption = (
-    caption: string | StringCaption
+    caption: string | StringCaption,
 ): caption is string | StringCaption =>
     typeof caption === 'string' ||
     (!isToolTipCaption(caption) && !isActionCaption(caption));
@@ -107,7 +107,7 @@ export default ({ title, steps }: Steppers) => (
                 key={step.id}
                 className={classNames(
                     'step',
-                    step.state && `step-${step.state}`
+                    step.state && `step-${step.state}`,
                 )}
             >
                 <div>

package/src/Toggle/Toggle.tsx

@@ -74,7 +74,7 @@ export const Toggle: FC<Props> = ({
                         isPrimary &&
                             `toggle-bar-primary${isToggled ? '-toggled' : ''}`,
                         isSecondary &&
-                            `toggle-bar-secondary${isToggled ? '-toggled' : ''}`
+                            `toggle-bar-secondary${isToggled ? '-toggled' : ''}`,
                     )}
                     style={{
                         backgroundColor: isToggled ? barColorToggled : barColor,
@@ -99,7 +99,7 @@ export const Toggle: FC<Props> = ({
                             isSecondary &&
                                 `toggle-handle-secondary${
                                     isToggled ? '-toggled' : ''
-                                }`
+                                }`,
                         )}
                         style={{
                             backgroundColor: isToggled

package/src/logging/appTransport.test.ts

@@ -26,7 +26,7 @@ describe('AppTransport', () => {
         expect(onLogEntry).toHaveBeenCalledWith(
             expect.objectContaining({
                 message,
-            })
+            }),
         );
     });
 
@@ -40,7 +40,7 @@ describe('AppTransport', () => {
         expect(onLogEntry).toHaveBeenCalledWith(
             expect.objectContaining({
                 level,
-            })
+            }),
         );
     });
 

package/src/logging/appTransport.ts

@@ -20,7 +20,7 @@ export default class AppTransport extends Transport {
 
         if (!options.onLogEntry) {
             throw new Error(
-                'Property onLogEntry was not provided to AppTransport'
+                'Property onLogEntry was not provided to AppTransport',
             );
         }
 

package/src/logging/describeError.test.ts

@@ -32,7 +32,7 @@ describe('a reasonable string representation for an error', () => {
             describeError({
                 message: 'a message',
                 origin: SyntaxError('something'),
-            })
+            }),
         ).toBe('a message (Origin: SyntaxError: something)');
     });
 });

package/src/logging/describeError.ts

@@ -10,7 +10,7 @@ interface MaybeWithMessageAndOrigin {
 }
 
 function hasMaybeMessageAndOrigin(
-    error: unknown
+    error: unknown,
 ): error is MaybeWithMessageAndOrigin {
     return error != null;
 }