arisa 4.0.14 → 4.0.16

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "arisa",
-  "version": "4.0.14",
+  "version": "4.0.16",
   "description": "Telegram + Pi Agent modular assistant",
   "type": "module",
   "main": "src/index.js",

package/src/index.js

@@ -8,6 +8,7 @@ import { createLogger } from "./runtime/logger.js";
 import { getServiceStatus, registerServiceProcess, startService, stopService, unregisterServiceProcess } from "./runtime/service-manager.js";
 import { flushArisaHome } from "./runtime/flush.js";
 import { arisaPackageDir } from "./runtime/paths.js";
+import { parseProcessIds } from "./runtime/process-ids.js";
 
 process.env.ARISA_PACKAGE_DIR = arisaPackageDir;
 
@@ -49,11 +50,11 @@ async function runCommand(commandName, args = []) {
 async function getListeningPids(port) {
   const lsofOutput = await runCommand("lsof", ["-tiTCP", `-sTCP:LISTEN`, `-iTCP:${port}`]);
   if (lsofOutput) {
-    return lsofOutput.split(/\s+/).map(Number).filter(Number.isFinite);
+    return parseProcessIds(lsofOutput);
   }
 
   const fuserOutput = await runCommand("fuser", ["-n", "tcp", String(port)]);
-  return fuserOutput.split(/\s+/).map(Number).filter(Number.isFinite);
+  return parseProcessIds(fuserOutput);
 }
 
 async function getProcessCommand(pid) {

package/src/runtime/process-ids.js

@@ -0,0 +1,9 @@
+export function parseProcessIds(commandOutput) {
+  if (!commandOutput) return [];
+
+  return commandOutput
+    .split(/\s+/)
+    .filter((token) => /^\d+$/.test(token))
+    .map(Number)
+    .filter((pid) => Number.isInteger(pid) && pid > 0);
+}

package/test/process-ids.test.js

@@ -0,0 +1,14 @@
+import assert from "node:assert/strict";
+import test from "node:test";
+import { parseProcessIds } from "../src/runtime/process-ids.js";
+
+test("ignores empty PID command output", () => {
+  assert.deepEqual(parseProcessIds(""), []);
+  assert.deepEqual(parseProcessIds("   \n\t"), []);
+});
+
+test("parses only positive integer PID tokens", () => {
+  assert.deepEqual(parseProcessIds("123\n456"), [123, 456]);
+  assert.deepEqual(parseProcessIds("11970/tcp: 1234"), [1234]);
+  assert.deepEqual(parseProcessIds("0 nope -12 42"), [42]);
+});