Fix tests, fix dependabot issues, 0.8.8

Author: casperwardensl

.github/workflows/test.yml

@@ -0,0 +1,21 @@
+name: Tests
+
+on:
+  push:
+  pull_request:
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        node-version: [20.x, 22.x, 24.x]
+    steps:
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+          cache: npm
+      - run: npm ci
+      - run: npm test

lib/LoginHandler.ts

@@ -12,7 +12,12 @@ import type { BotOptionFlags } from './enums/BotOptionFlags';
 import { URL } from 'url';
 import * as os from 'os';
 
-const packageJsonPath = path.join(__dirname, '..', '..', 'package.json');
+// Resolves from lib/ when running from source, or dist/lib/ when compiled
+const packageJsonCandidates = [
+    path.join(__dirname, '..', 'package.json'),
+    path.join(__dirname, '..', '..', 'package.json')
+];
+const packageJsonPath = packageJsonCandidates.find((p) => fs.existsSync(p)) ?? packageJsonCandidates[1];
 // eslint-disable-next-line @typescript-eslint/no-require-imports
 const packageJson = require(packageJsonPath);
 const version = packageJson.version;

lib/classes/EventQueueClient.ts

@@ -22,7 +22,7 @@ import { LandStatsEvent } from '../events/LandStatsEvent';
 import * as LLSD from '@caspertech/llsd';
 import type { CancelableRequest, Response as GotResponse } from 'got';
 import got from 'got';
-import * as Long from 'long';
+import { Long } from './LongUtil';
 
 export class EventQueueClient
 {

lib/classes/LLSettings.spec.ts

@@ -1,18 +1,17 @@
 import { describe, expect, it } from 'vitest';
-import * as fg from 'fast-glob';
 import * as fs from 'fs';
 import * as path from 'path';
 import { LLSettings } from './LLSettings';
-import { toDeeplyMatch } from '../../testing/TestingUtils.util.spec';
+import { toDeeplyMatch } from '../../testing/TestingUtils.util';
 
 expect.extend({
     toDeeplyMatch,
 });
 
 describe('LLSettings', () =>
 {
-    const filePath = path.join(__dirname, '..', '..', '..', '..', 'assets');
-    const filteredFileNames = fg.sync(filePath + '/*.bin');
+    const filePath = path.join(__dirname, '..', '..', 'testing');
+    const filteredFileNames = fs.readdirSync(filePath).filter((f) => f.endsWith('.bin')).map((f) => path.join(filePath, f));
     for(const file of filteredFileNames)
     {
         const baseName = path.basename(file);

lib/classes/LoginResponse.ts

@@ -2,7 +2,7 @@ import { UUID } from './UUID';
 import { Agent } from './Agent';
 import { Region } from './Region';
 import { Vector3 } from './Vector3';
-import * as Long from 'long';
+import { Long } from './LongUtil';
 import type { ClientEvents } from './ClientEvents';
 import { InventoryFolder } from './InventoryFolder';
 import type { BotOptionFlags} from '..';

lib/classes/LongUtil.ts

@@ -0,0 +1,9 @@
+import * as LongImport from 'long';
+
+// The 'long' package is CommonJS: tsc's CommonJS output binds the constructor
+// directly, while ESM-based tooling (such as vitest) exposes it on `default`.
+const LongCtor: LongImport.LongConstructor =
+    (LongImport as unknown as { default?: LongImport.LongConstructor }).default ?? LongImport;
+
+export { LongCtor as Long };
+export type Long = LongImport;

lib/classes/UUID.ts

@@ -1,5 +1,5 @@
 import validator from 'validator';
-import * as Long from 'long';
+import { Long } from './LongUtil';
 import type { XMLNode } from 'xmlbuilder';
 import * as uuid from 'uuid';
 

lib/classes/Utils.ts

@@ -1,4 +1,4 @@
-import * as Long from 'long';
+import { Long } from './LongUtil';
 import type { Subscription } from 'rxjs';
 import { Subject } from 'rxjs';
 import * as xml2js from 'xml2js';

lib/classes/llsd/LLSD.spec.ts

@@ -97,14 +97,14 @@ describe('LLSD', () =>
             expect(parsed.arr[12]).toBeInstanceOf(LLSDReal);
             expect(parsed.arr[12].valueOf()).toBe(-0);
 
-            const rebuilt = LLSD.toNotation(parsed as unknown as LLSDType);
+            const rebuilt = LLSD.toNotation(parsed as unknown as LLSDType, false);
             expect(rebuilt).toBe('{"string":"Hello!","map":{"nestedMap":{"nestedArray":["one",i2,r3.14,u00000000-1111-2222-3333-444444444444,l"https://www.blahblah.com",{"date":d"2024-11-22T20:35:36.638Z"},!]}},"arr":[1,0,!,!,d"2024-11-22T20:35:36.638Z",b64"dGhlIHF1aWNrIGJyb3duIGZveCBqdW1wcyBvdmVyIHRoZSBsYXp5IGRvZw==",b64"UGVlcGVycyBqZWVwZXJzIQ==",rNaN,rNaN,r-Infinity,rInfinity,r0,r-0]}');
         });
 
         it('escapes special characters', () =>
         {
             const str = '\'"ⳑ⣺⋺➍⣋▎⊻™⿨₀⏆⨣⑬⛪␹⩯\0⁇⯒';
-            const encoded= LLSD.toNotation(str);
+            const encoded = LLSD.toNotation(str, false);
             expect(encoded.valueOf()).toBe('"\'\\"ⳑ⣺⋺➍⣋▎⊻™⿨₀⏆⨣⑬⛪␹⩯\\x00⁇⯒"');
 
             const parsed = LLSD.parseNotation(encoded);

lib/classes/messages/AbortXfer.ts

@@ -1,6 +1,6 @@
 // This file has been automatically generated by writeMessageClasses.js
 
-import * as Long from 'long';
+import { Long } from '../LongUtil';
 import { MessageFlags } from '../../enums/MessageFlags';
 import { MessageBase } from '../MessageBase';
 import { Message } from '../../enums/Message';