refactor: Split utils.ts into multiple files in a utils/ directory.

src/utils/byte.ts

@@ -0,0 +1,13 @@
+/** Determine if a number is a byte. */
+function isByte(b: number): boolean {
+	return (b & 0xff) === b
+}
+
+/** Ensure every number in bytes is an actual byte value. */
+export function checkBytes(bytes: readonly number[]): void {
+	for (const b of bytes) {
+		if (!isByte(b)) {
+			throw new RangeError('non-byte found')
+		}
+	}
+}

src/utils/pretty.ts

@@ -0,0 +1,11 @@
+/** Debug representation of a byte value. */
+function prettyByte(byte: number): string {
+	return ('0' + byte.toString(16).toUpperCase()).slice(-2)
+}
+
+/** Debug representation of a byte list. */
+export function prettyBytes(bytes: readonly number[] | Uint8Array): string {
+	// Explicitly apply the Array map function so this works on both arrays and
+	// typed arrays.
+	return `[${Array.prototype.map.call(bytes, prettyByte).join(' ')}]`
+}

src/utils/promise-with-resolvers.ts

@@ -0,0 +1,19 @@
+/** Return an unresolved promise and resolve/reject functions for it. */
+export function promiseWithResolvers<T>(): {
+	promise: Promise<T>
+	resolve: (value: T) => void
+	reject: (reason?: any) => void
+} {
+	let promiseResolve: (value: T) => void
+	let promiseReject: (reason?: any) => void
+	const promise = new Promise<T>((resolve: (value: T) => void, reject: (reason?: any) => void) => {
+		promiseResolve = resolve
+		promiseReject = reject
+	})
+
+	return {
+		promise,
+		resolve: promiseResolve!,
+		reject: promiseReject!,
+	}
+}

src/visca/__tests__/camera-interactions/matchers.ts

@@ -1,4 +1,4 @@
-import { prettyBytes } from '../../utils.js'
+import { prettyBytes } from '../../../utils/pretty.js'
 
 export function MatchVISCABytes(bytes: readonly number[]): RegExp {
 	return new RegExp(`\\[${prettyBytes(bytes).slice(1, -1)}\\]`)

src/visca/__tests__/camera-interactions/run-test.ts

@@ -2,7 +2,7 @@ import { assertNever, type CompanionOptionValues, InstanceStatus, type LogLevel
 import { type Interaction, type Match } from './interactions.js'
 import net from 'net'
 import { type MessageType, type PartialInstance, VISCAPort } from '../../port.js'
-import { prettyBytes } from '../../utils.js'
+import { prettyBytes } from '../../../utils/pretty.js'
 
 /** Turn on extra logging in performing test interactions to debug tests. */
 const DEBUG_LOGGING = true

src/visca/command.ts

@@ -1,5 +1,6 @@
 import type { CompanionOptionValues } from '@companion-module/base'
-import { checkBytes, prettyBytes } from './utils.js'
+import { checkBytes } from '../utils/byte.js'
+import { prettyBytes } from '../utils/pretty.js'
 
 // TERMINOLOGY NOTE:
 // Eight bits is a byte.  The upper or lower four-bit half of a byte is a

src/visca/port.ts

@@ -7,8 +7,8 @@ import {
 } from '@companion-module/base'
 import { checkCommandBytes, type Command, type Inquiry, type Response, responseMatches } from './command.js'
 import type { PtzOpticsInstance } from '../instance.js'
-import { prettyBytes } from './utils.js'
-import { promiseWithResolvers } from './utils.js'
+import { prettyBytes } from '../utils/pretty.js'
+import { promiseWithResolvers } from '../utils/promise-with-resolvers.js'
 
 const BLAME_MODULE =
 	'This is likely a bug in the ptzoptics-visca Companion module.  Please ' +