node-osc 9.1.1 → 9.1.3

package/.github/workflows/nodejs.yml

@@ -11,7 +11,7 @@ jobs:
   run-tests:
     strategy:
       matrix:
-        node-version: ['21', '20', '18']
+        node-version: ['22', '20', '18']
         os: [ubuntu-latest, macos-latest, windows-latest]
 
     runs-on: ${{ matrix.os }}

package/dist/test/test-getPort.js

@@ -0,0 +1,20 @@
+'use strict';
+
+var tap = require('tap');
+var util = require('./util.js');
+var node_net = require('node:net');
+
+tap.test('getPort function returns an available port', async (t) => {
+  const port = await util.getPort();
+  t.plan(2);
+  t.type(port, 'number', 'getPort should return a number');
+
+  const server = node_net.createServer();
+  server.listen(port, () => {
+    t.pass('Port is usable');
+  });
+  server.on('close', () => {
+    t.pass('Server closed');
+  });
+  server.close();
+});

package/dist/test/util.js

@@ -1,13 +1,27 @@
 'use strict';
 
+var node_net = require('node:net');
+
 async function bootstrap(t) {
-  const {default: getPorts, portNumbers} = await import('get-port');
-  const port = await getPorts({
-    port: portNumbers(3000, 3500)
-  });
+  const port = await getPort();
   t.context = {
     port
   };
 }
 
+function getPort() {
+  return new Promise((resolve, reject) => {
+    const server = node_net.createServer();
+    server.unref();
+    server.on('error', reject);
+    server.listen(() => {
+      const { port } = server.address();
+      server.close(() => {
+        resolve(port);
+      });
+    });
+  });
+}
+
 exports.bootstrap = bootstrap;
+exports.getPort = getPort;

package/eslint.config.mjs

@@ -0,0 +1,8 @@
+import globals from "globals";
+import pluginJs from "@eslint/js";
+
+
+export default [
+  {languageOptions: { globals: globals.node }},
+  pluginJs.configs.recommended,
+];

package/examples/client.js

@@ -1,5 +1,3 @@
-/* eslint-disable no-console */
-
 'use strict';
 const { Client, Message } = require('node-osc');
 

package/examples/server.js

@@ -1,5 +1,3 @@
-/* eslint-disable no-console */
-
 'use strict';
 var { Server } = require('node-osc');
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "node-osc",
   "description": "pyOSC inspired library for sending and receiving OSC messages",
-  "version": "9.1.1",
+  "version": "9.1.3",
   "exports": {
     "require": "./dist/lib/index.js",
     "default": "./lib/index.mjs"
@@ -17,7 +17,7 @@
     "email": "myles.borins@gmail.com"
   },
   "engines": {
-    "node": "^18.17.0 || >=20.5.0"
+    "node": "^18.17.0 || ^20.5.0 || >=22.0.0"
   },
   "license": "LGPL-3.0-or-later",
   "scripts": {
@@ -26,8 +26,8 @@
     "prepublishOnly": "npm run build",
     "lint": "eslint \"lib/**/*.mjs\" test/* examples/* rollup.config.mjs",
     "test": "npm run lint && npm run build && npm run test:esm && npm run test:cjs",
-    "test:esm": "tap -j1 test/test-*.mjs",
-    "test:cjs": "tap -j1 dist/test/test-*.js"
+    "test:esm": "tap test/test-*.mjs",
+    "test:cjs": "tap dist/test/test-*.js"
   },
   "contributors": [
     "Hans Hübner <hans.huebner@gmail.com>",
@@ -46,9 +46,10 @@
     "osc-min": "^1.1.1"
   },
   "devDependencies": {
-    "eslint": "^8.36.0",
-    "get-port": "^6.1.2",
-    "rollup": "^4.9.5",
-    "tap": "^18.4.2"
+    "@eslint/js": "^9.4.0",
+    "eslint": "^9.4.0",
+    "globals": "^15.3.0",
+    "rollup": "^4.18.0",
+    "tap": "^19.0.2"
   }
 }

package/rollup.config.mjs

@@ -1,26 +1,6 @@
 import { readdirSync as readdir, statSync as stat } from 'fs';
 import { join } from 'path';
 
-// Borrowed from the rollup docs
-// https://github.com/rollup/rollup/blob/d0db53459be43c5cc806cb91f14e82217950ba42/docs/05-plugin-development.md#renderdynamicimport
-function retainImportExpressionPlugin() {
-  return {
-    name: 'retain-import-expression',
-    resolveDynamicImport(specifier) {
-      if (specifier === 'get-port') return false;
-      return null;
-    },
-    renderDynamicImport({ targetModuleId }) {
-      if (targetModuleId === 'get-port') {
-        return {
-          left: 'import(',
-          right: ')'
-        };
-      }
-    }
-  };
-}
-
 function walk(root, result=[]) {
   const rootURL = new URL(root, import.meta.url);
   const paths = readdir(rootURL);
@@ -67,7 +47,7 @@ function walkTest(config) {
   tests.forEach(({input, dir}) => {
     config.push({
       input,
-      plugins: [retainImportExpressionPlugin()],
+      plugins: [],
       output: {
         entryFileNames: '[name].js',
         dir,
@@ -77,7 +57,7 @@ function walkTest(config) {
       },
       external: [
         'node:dgram',
-        'get-port',
+        'node:net',
         'node-osc',
         'osc-min',
         'tap',

package/test/test-getPort.mjs

@@ -0,0 +1,18 @@
+import { test } from 'tap';
+import { getPort } from './util.mjs';
+import { createServer } from 'node:net';
+
+test('getPort function returns an available port', async (t) => {
+  const port = await getPort();
+  t.plan(2);
+  t.type(port, 'number', 'getPort should return a number');
+
+  const server = createServer();
+  server.listen(port, () => {
+    t.pass('Port is usable');
+  });
+  server.on('close', () => {
+    t.pass('Server closed');
+  });
+  server.close();
+});

package/test/util.mjs

@@ -1,13 +1,27 @@
+import { createServer } from 'node:net';
+
 async function bootstrap(t) {
-  const {default: getPorts, portNumbers} = await import('get-port');
-  const port = await getPorts({
-    port: portNumbers(3000, 3500)
-  });
+  const port = await getPort();
   t.context = {
     port
   };
 }
 
+function getPort() {
+  return new Promise((resolve, reject) => {
+    const server = createServer();
+    server.unref();
+    server.on('error', reject);
+    server.listen(() => {
+      const { port } = server.address();
+      server.close(() => {
+        resolve(port);
+      });
+    });
+  });
+}
+
 export {
-  bootstrap
+  bootstrap,
+  getPort
 };

package/.eslintrc.yml

@@ -1,18 +0,0 @@
-env:
-  commonjs: true
-  es6: true
-  node: true
-extends: 'eslint:recommended'
-globals:
-  Atomics: readonly
-  SharedArrayBuffer: readonly
-parserOptions:
-  ecmaVersion: 2020
-  sourceType: module
-rules:
-  quotes:
-    - error
-    - single
-  semi:
-    - error
-    - always