fix(ci): resolve build failures from xo 3.0.2 upgrade

- Update CI actions from v3 to v4 (deprecated Node 20 runtime)
- Replace Node 18 with Node 22 in test matrix
- Fix lint errors: reorder class members, use unknown in catch
- Disable false-positive rules (no-unsafe-* for node:crypto types)
- Disable rules incompatible with Node 20 (prefer-uint8array-base64)
- Fix tsconfig: add node types, disable erasableSyntaxOnly for enums
- Update minimum engine to Node 20

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: AchoArnold

.github/workflows/main.yml

@@ -11,12 +11,12 @@ jobs:
             matrix:
                 node-version:
                     - latest
+                    - 22
                     - 20
-                    - 18
 
         steps:
-            - uses: actions/checkout@v3
-            - uses: actions/setup-node@v3
+            - uses: actions/checkout@v4
+            - uses: actions/setup-node@v4
               with:
                   node-version: ${{ matrix.node-version }}
             - run: npm install

index.ts

@@ -3,11 +3,11 @@ import MessageService from './src/message-service.js';
 import CipherService from './src/cipher-service.js';
 
 class HttpSms {
+	private readonly client: AxiosInstance;
+
 	public messages: MessageService;
 	public cipher: CipherService;
 
-	private readonly client: AxiosInstance;
-
 	constructor(apiKey: string, baseUrl = 'https://api.httpsms.com') {
 		this.client = axios.create({
 			// eslint-disable-next-line  @typescript-eslint/naming-convention

package.json

@@ -17,7 +17,7 @@
 	},
 	"types": "./dist/index.d.ts",
 	"engines": {
-		"node": ">=16"
+		"node": ">=20"
 	},
 	"scripts": {
 		"test": "xo --env=node && npm run build && mocha -r ts-node/register 'tests/**/*.js'",
@@ -56,7 +56,13 @@
 	},
 	"xo": {
 		"rules": {
-			"n/file-extension-in-import": 0
+			"n/file-extension-in-import": 0,
+			"jsdoc/require-asterisk-prefix": 0,
+			"unicorn/prefer-uint8array-base64": 0,
+			"@typescript-eslint/no-unsafe-return": 0,
+			"@typescript-eslint/no-unsafe-call": 0,
+			"@typescript-eslint/no-unsafe-member-access": 0,
+			"@typescript-eslint/no-unsafe-assignment": 0
 		}
 	}
 }

src/cipher-service.ts

@@ -7,13 +7,13 @@ import {
 import {Buffer} from 'node:buffer';
 
 class CipherService {
+	private hash(value: string): Uint8Array {
+		return createHash('sha256').update(value).digest();
+	}
+
 	public encrypt(key: string, message: string): string {
 		const iv = randomBytes(16);
-		const cipher = createCipheriv(
-			'aes-256-cfb',
-			this.hash(key),
-			iv,
-		).setAutoPadding(false);
+		const cipher = createCipheriv('aes-256-cfb', this.hash(key), iv).setAutoPadding(false);
 		return Buffer.concat([
 			iv,
 			cipher.update(message, 'utf8'),
@@ -24,21 +24,13 @@ class CipherService {
 	public decrypt(key: string, message: string): string {
 		const cipherBytes = Buffer.from(message, 'base64');
 		const iv = cipherBytes.subarray(0, 16);
-		const decipher = createDecipheriv(
-			'aes-256-cfb',
-			this.hash(key),
-			iv,
-		).setAutoPadding(false);
+		const decipher = createDecipheriv('aes-256-cfb', this.hash(key), iv).setAutoPadding(false);
 
 		return Buffer.concat([
 			decipher.update(cipherBytes.subarray(16, cipherBytes.length)),
 			decipher.final(),
 		]).toString();
 	}
-
-	private hash(value: string): Uint8Array {
-		return createHash('sha256').update(value).digest();
-	}
 }
 
 export default CipherService;

src/message-service.ts

@@ -11,8 +11,8 @@ class MessageService {
 				.then((response: AxiosResponse<MessageResponse>) => {
 					resolve(response.data.data);
 				})
-				.catch(async (error: AxiosError) => {
-					reject(error);
+				.catch(async (error: unknown) => {
+					reject(error as AxiosError);
 				});
 		});
 	}

tests/message-service.spec.js

@@ -9,14 +9,7 @@ const successMessageResponse = '{ "data":  { "id": "32b0774a-5094-4b58-9efa-57bc
 describe('MessageService', () => {
 	it('should respond with a valid message', async () => {
 		// Arrange
-		const server = setupServer(
-			// Describe network behavior with request handlers.
-			// Tip: move the handlers into their own module and
-			// import it across your browser and Node.js setups!
-			http.post('https://api.httpsms.com/v1/messages/send', () =>
-				HttpResponse.json(JSON.parse(successMessageResponse)),
-			),
-		);
+		const server = setupServer(http.post('https://api.httpsms.com/v1/messages/send', () => HttpResponse.json(JSON.parse(successMessageResponse))));
 		server.listen();
 
 		const client = new HttpSms('test key');
@@ -35,14 +28,7 @@ describe('MessageService', () => {
 	});
 	it('should respond with an error when there is a network error', async () => {
 		// Arrange
-		const server = setupServer(
-			// Describe network behavior with request handlers.
-			// Tip: move the handlers into their own module and
-			// import it across your browser and Node.js setups!
-			http.post('https://api.httpsms.com/v1/messages/send', () =>
-				HttpResponse.error(),
-			),
-		);
+		const server = setupServer(http.post('https://api.httpsms.com/v1/messages/send', () => HttpResponse.error()));
 		server.listen();
 
 		const client = new HttpSms('test key');

tsconfig.json

@@ -3,9 +3,11 @@
 	"compilerOptions": {
 		"outDir": "dist",
 		"experimentalDecorators": true,
-		"esModuleInterop": true
+		"esModuleInterop": true,
+		"erasableSyntaxOnly": false,
+		"types": ["node"]
 	},
-	"files": ["index.ts"],
+	"include": ["index.ts", "src/**/*.ts"],
 	"ts-node": {
 		"transpileOnly": true,
 		"esm": true