@bitsocial/bitsocial-cli 0.19.71 → 0.19.73

package/README.md

@@ -421,7 +421,7 @@ EXAMPLES
   $ bitsocial challenge install ./my-local-challenge
 ```
 
-_See code: [src/cli/commands/challenge/install.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/challenge/install.ts)_
+_See code: [src/cli/commands/challenge/install.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/challenge/install.ts)_
 
 ## `bitsocial challenge list`
 
@@ -447,7 +447,7 @@ EXAMPLES
   $ bitsocial challenge list -q
 ```
 
-_See code: [src/cli/commands/challenge/list.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/challenge/list.ts)_
+_See code: [src/cli/commands/challenge/list.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/challenge/list.ts)_
 
 ## `bitsocial challenge ls`
 
@@ -501,7 +501,7 @@ EXAMPLES
   $ bitsocial challenge remove @scope/my-challenge
 ```
 
-_See code: [src/cli/commands/challenge/remove.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/challenge/remove.ts)_
+_See code: [src/cli/commands/challenge/remove.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/challenge/remove.ts)_
 
 ## `bitsocial challenge rm NAME`
 
@@ -615,7 +615,7 @@ EXAMPLES
     $ bitsocial community create --jsonFile ./create-options.json
 ```
 
-_See code: [src/cli/commands/community/create.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/community/create.ts)_
+_See code: [src/cli/commands/community/create.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/community/create.ts)_
 
 ## `bitsocial community delete ADDRESSES`
 
@@ -640,7 +640,7 @@ EXAMPLES
   $ bitsocial community delete 12D3KooWG3XbzoVyAE6Y9vHZKF64Yuuu4TjdgQKedk14iYmTEPWu
 ```
 
-_See code: [src/cli/commands/community/delete.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/community/delete.ts)_
+_See code: [src/cli/commands/community/delete.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/community/delete.ts)_
 
 ## `bitsocial community edit ADDRESS`
 
@@ -710,7 +710,7 @@ EXAMPLES
     $ bitsocial community edit bitsocial.bso --jsonFile ./edit-options.json
 ```
 
-_See code: [src/cli/commands/community/edit.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/community/edit.ts)_
+_See code: [src/cli/commands/community/edit.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/community/edit.ts)_
 
 ## `bitsocial community export [ADDRESS]`
 
@@ -751,7 +751,7 @@ EXAMPLES
   $ bitsocial community export --publicKey 12D3KooWG3XbzoVyAE6Y9vHZKF64Yuuu4TjdgQKedk14iYmTEPWu
 ```
 
-_See code: [src/cli/commands/community/export.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/community/export.ts)_
+_See code: [src/cli/commands/community/export.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/community/export.ts)_
 
 ## `bitsocial community get [ADDRESS]`
 
@@ -782,7 +782,7 @@ EXAMPLES
   $ bitsocial community get --publicKey 12D3KooWG3XbzoVyAE6Y9vHZKF64Yuuu4TjdgQKedk14iYmTEPWu
 ```
 
-_See code: [src/cli/commands/community/get.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/community/get.ts)_
+_See code: [src/cli/commands/community/get.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/community/get.ts)_
 
 ## `bitsocial community list`
 
@@ -793,7 +793,7 @@ USAGE
   $ bitsocial community list --pkcRpcUrl <value> [-q]
 
 FLAGS
-  -q, --quiet              Only display community addresses
+  -q, --quiet              Only display community addresses (much faster: skips the per-community 'started' lookup)
       --pkcRpcUrl=<value>  (required) [default: ws://localhost:9138/] URL to PKC RPC
 
 DESCRIPTION
@@ -805,7 +805,7 @@ EXAMPLES
   $ bitsocial community list
 ```
 
-_See code: [src/cli/commands/community/list.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/community/list.ts)_
+_See code: [src/cli/commands/community/list.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/community/list.ts)_
 
 ## `bitsocial community start ADDRESSES`
 
@@ -839,7 +839,7 @@ EXAMPLES
     $ bitsocial community start $(bitsocial community list -q) --concurrency 1
 ```
 
-_See code: [src/cli/commands/community/start.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/community/start.ts)_
+_See code: [src/cli/commands/community/start.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/community/start.ts)_
 
 ## `bitsocial community stop ADDRESSES`
 
@@ -864,7 +864,7 @@ EXAMPLES
   $ bitsocial community stop Qmb99crTbSUfKXamXwZBe829Vf6w5w5TktPkb6WstC9RFW
 ```
 
-_See code: [src/cli/commands/community/stop.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/community/stop.ts)_
+_See code: [src/cli/commands/community/stop.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/community/stop.ts)_
 
 ## `bitsocial daemon`
 
@@ -911,7 +911,7 @@ EXAMPLES
   $ bitsocial daemon --no-allowPrivateKeyExport
 ```
 
-_See code: [src/cli/commands/daemon.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/daemon.ts)_
+_See code: [src/cli/commands/daemon.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/daemon.ts)_
 
 ## `bitsocial help [COMMAND]`
 
@@ -977,7 +977,7 @@ EXAMPLES
   $ bitsocial logs --stdout -f
 ```
 
-_See code: [src/cli/commands/logs.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/logs.ts)_
+_See code: [src/cli/commands/logs.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/logs.ts)_
 
 ## `bitsocial update check`
 
@@ -994,7 +994,7 @@ EXAMPLES
   $ bitsocial update check
 ```
 
-_See code: [src/cli/commands/update/check.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/update/check.ts)_
+_See code: [src/cli/commands/update/check.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/update/check.ts)_
 
 ## `bitsocial update install [VERSION]`
 
@@ -1026,7 +1026,7 @@ EXAMPLES
   $ bitsocial update install --no-restart-daemons
 ```
 
-_See code: [src/cli/commands/update/install.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/update/install.ts)_
+_See code: [src/cli/commands/update/install.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/update/install.ts)_
 
 ## `bitsocial update versions`
 
@@ -1048,7 +1048,7 @@ EXAMPLES
   $ bitsocial update versions --limit 5
 ```
 
-_See code: [src/cli/commands/update/versions.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.71/src/cli/commands/update/versions.ts)_
+_See code: [src/cli/commands/update/versions.ts](https://github.com/bitsocialnet/bitsocial-cli/blob/v0.19.73/src/cli/commands/update/versions.ts)_
 <!-- commandsstop -->
 
 ## Contribution

package/bin/run

@@ -1,5 +1,13 @@
 #!/usr/bin/env node
 
+// Enable the V8 compile cache before any imports so the CLI's own dist/, oclif and all
+// deps get bytecode-cached across invocations. No-op on Node < 22.8, and on any failure
+// it reports via its return value rather than throwing. Cache dir defaults to
+// os.tmpdir()/node-compile-cache; override with NODE_COMPILE_CACHE, disable with
+// NODE_DISABLE_COMPILE_CACHE=1.
+const { enableCompileCache } = await import("node:module");
+enableCompileCache?.();
+
 // Save and strip DEBUG before any imports so the debug module
 // doesn't auto-enable stderr output. The daemon command will
 // re-enable namespaces programmatically and redirect to the log file.

package/dist/cli/commands/community/list.js

@@ -7,7 +7,10 @@ export default class List extends BaseCommand {
     static description = "List your communities";
     static examples = ["bitsocial community list -q", "bitsocial community list"];
     static flags = {
-        quiet: Flags.boolean({ char: "q", summary: "Only display community addresses" })
+        quiet: Flags.boolean({
+            char: "q",
+            summary: "Only display community addresses (much faster: skips the per-community 'started' lookup)"
+        })
     };
     async run() {
         const { flags } = await this.parse(List);

package/dist/common-utils/daemon-state.d.ts

@@ -37,23 +37,49 @@ export interface DaemonState {
 export declare function parseSystemdUnitFromCgroup(content: string): string | undefined;
 /** Read the systemd unit owning `pid` (or the current process when "self") from /proc, or undefined. */
 export declare function readSystemdUnit(pid: number | "self"): Promise<string | undefined>;
+/**
+ * MainPID of a systemd unit as reported by the system manager, or undefined when it cannot be
+ * determined (systemctl missing, unit unknown, MainPID=0). `systemctl --user` units are invisible
+ * to the system manager and resolve to undefined — the daemon is then treated as unsupervised,
+ * matching the restart path, which also only drives the system manager.
+ */
+export declare function readUnitMainPid(unit: string): Promise<number | undefined>;
+/** Parent PID of `pid` from /proc/<pid>/stat, or undefined. */
+export declare function readParentPid(pid: number): Promise<number | undefined>;
 /**
  * Detect whether THIS process was started by systemd, and under which unit. systemd sets
- * $INVOCATION_ID for every service it spawns; the unit name comes from this process's own cgroup.
- * `env`/`readUnit` are injectable for testing. Returns undefined when not systemd-supervised.
+ * $INVOCATION_ID for every service it spawns and the unit name comes from this process's own
+ * cgroup — but BOTH are inherited by every descendant of any service (e.g. all processes inside a
+ * CI runner's agent service, issue #92), so unit membership alone is not ownership. The daemon is
+ * only systemd-supervised when it is the unit's main process (or its direct child, covering
+ * `ExecStart=/bin/sh -c …` wrappers); otherwise restarting the unit would bounce an unrelated
+ * service. `env`/`readUnit`/`getMainPid`/`self` are injectable for testing. Returns undefined when
+ * not systemd-supervised.
  */
-export declare function detectSelfSupervisor(env?: NodeJS.ProcessEnv, readUnit?: (pid: number | "self") => Promise<string | undefined>): Promise<DaemonSupervisor | undefined>;
+export declare function detectSelfSupervisor(env?: NodeJS.ProcessEnv, readUnit?: (pid: number | "self") => Promise<string | undefined>, getMainPid?: (unit: string) => Promise<number | undefined>, self?: {
+    pid: number;
+    ppid: number;
+}): Promise<DaemonSupervisor | undefined>;
 /**
  * Resolve the supervisor for a daemon described by `state`. Prefers the `supervisor` it recorded
- * at startup; for legacy daemons that predate that field, falls back to inferring the unit from the
- * live process's cgroup. `readUnit` is injectable for testing.
+ * at startup (already ownership-verified by detectSelfSupervisor); for legacy daemons that predate
+ * that field, falls back to inferring the unit from the live process's cgroup, with the same
+ * MainPID ownership check as detectSelfSupervisor (issue #92).
+ * `readUnit`/`getMainPid`/`getParentPid` are injectable for testing.
  */
-export declare function resolveDaemonSupervisor(state: DaemonState, readUnit?: (pid: number | "self") => Promise<string | undefined>): Promise<DaemonSupervisor | undefined>;
+export declare function resolveDaemonSupervisor(state: DaemonState, readUnit?: (pid: number | "self") => Promise<string | undefined>, getMainPid?: (unit: string) => Promise<number | undefined>, getParentPid?: (pid: number) => Promise<number | undefined>): Promise<DaemonSupervisor | undefined>;
 /** Write a daemon state file atomically (write to .tmp then rename). */
 export declare function writeDaemonState(state: DaemonState): Promise<void>;
 /** Read all state files from the daemon states directory. */
 export declare function readAllDaemonStates(): Promise<DaemonState[]>;
-/** Delete a specific daemon's state file. Ignores ENOENT. */
+/**
+ * Delete a specific daemon's state file. Best-effort: pruning a stale file must never be fatal.
+ * Ignores ENOENT (already gone) and the Windows file-locking codes EPERM/EACCES/EBUSY — on Windows,
+ * unlinking a file another process still has open (or that is in "delete-pending" state) returns
+ * EPERM, unlike POSIX where it succeeds. Concurrent daemons sharing the .daemon_states dir race to
+ * prune the same dead-PID file; a loser there used to crash daemon startup. Another daemon will
+ * reclaim the file on its next prune.
+ */
 export declare function deleteDaemonState(pid: number): Promise<void>;
 /** Delete state files for dead or reused PIDs from disk. */
 export declare function pruneStaleStates(): Promise<void>;

package/dist/common-utils/daemon-state.js

@@ -42,27 +42,76 @@ export async function readSystemdUnit(pid) {
         return undefined; // no /proc (non-Linux) or unreadable — treat as unsupervised
     }
 }
+/**
+ * MainPID of a systemd unit as reported by the system manager, or undefined when it cannot be
+ * determined (systemctl missing, unit unknown, MainPID=0). `systemctl --user` units are invisible
+ * to the system manager and resolve to undefined — the daemon is then treated as unsupervised,
+ * matching the restart path, which also only drives the system manager.
+ */
+export async function readUnitMainPid(unit) {
+    try {
+        const { stdout } = await execFileAsync("systemctl", ["show", "--property=MainPID", "--value", unit]);
+        const mainPid = Number(String(stdout).trim());
+        return Number.isInteger(mainPid) && mainPid > 0 ? mainPid : undefined;
+    }
+    catch {
+        return undefined;
+    }
+}
+/** Parent PID of `pid` from /proc/<pid>/stat, or undefined. */
+export async function readParentPid(pid) {
+    try {
+        const stat = await fs.readFile(`/proc/${pid}/stat`, "utf-8");
+        // Format: `pid (comm) state ppid …` — comm may itself contain spaces or parens, so the
+        // fields are only parseable after the LAST closing paren.
+        const afterComm = stat.slice(stat.lastIndexOf(")") + 2);
+        const ppid = Number(afterComm.split(" ")[1]);
+        return Number.isInteger(ppid) && ppid > 0 ? ppid : undefined;
+    }
+    catch {
+        return undefined;
+    }
+}
 /**
  * Detect whether THIS process was started by systemd, and under which unit. systemd sets
- * $INVOCATION_ID for every service it spawns; the unit name comes from this process's own cgroup.
- * `env`/`readUnit` are injectable for testing. Returns undefined when not systemd-supervised.
+ * $INVOCATION_ID for every service it spawns and the unit name comes from this process's own
+ * cgroup — but BOTH are inherited by every descendant of any service (e.g. all processes inside a
+ * CI runner's agent service, issue #92), so unit membership alone is not ownership. The daemon is
+ * only systemd-supervised when it is the unit's main process (or its direct child, covering
+ * `ExecStart=/bin/sh -c …` wrappers); otherwise restarting the unit would bounce an unrelated
+ * service. `env`/`readUnit`/`getMainPid`/`self` are injectable for testing. Returns undefined when
+ * not systemd-supervised.
  */
-export async function detectSelfSupervisor(env = process.env, readUnit = readSystemdUnit) {
+export async function detectSelfSupervisor(env = process.env, readUnit = readSystemdUnit, getMainPid = readUnitMainPid, self = { pid: process.pid, ppid: process.ppid }) {
     if (!env.INVOCATION_ID)
         return undefined;
     const unit = await readUnit("self");
-    return unit ? { type: "systemd", unit } : undefined;
+    if (!unit)
+        return undefined;
+    const mainPid = await getMainPid(unit);
+    if (mainPid === undefined || (mainPid !== self.pid && mainPid !== self.ppid))
+        return undefined;
+    return { type: "systemd", unit };
 }
 /**
  * Resolve the supervisor for a daemon described by `state`. Prefers the `supervisor` it recorded
- * at startup; for legacy daemons that predate that field, falls back to inferring the unit from the
- * live process's cgroup. `readUnit` is injectable for testing.
+ * at startup (already ownership-verified by detectSelfSupervisor); for legacy daemons that predate
+ * that field, falls back to inferring the unit from the live process's cgroup, with the same
+ * MainPID ownership check as detectSelfSupervisor (issue #92).
+ * `readUnit`/`getMainPid`/`getParentPid` are injectable for testing.
  */
-export async function resolveDaemonSupervisor(state, readUnit = readSystemdUnit) {
+export async function resolveDaemonSupervisor(state, readUnit = readSystemdUnit, getMainPid = readUnitMainPid, getParentPid = readParentPid) {
     if (state.supervisor)
         return state.supervisor;
     const unit = await readUnit(state.pid);
-    return unit ? { type: "systemd", unit } : undefined;
+    if (!unit)
+        return undefined;
+    const mainPid = await getMainPid(unit);
+    if (mainPid === undefined)
+        return undefined;
+    if (mainPid !== state.pid && mainPid !== (await getParentPid(state.pid)))
+        return undefined;
+    return { type: "systemd", unit };
 }
 function stateFilePath(pid) {
     return path.join(DAEMON_STATES_DIR, `${pid}-daemon.state`);
@@ -145,13 +194,21 @@ export async function readAllDaemonStates() {
     }
     return states;
 }
-/** Delete a specific daemon's state file. Ignores ENOENT. */
+/**
+ * Delete a specific daemon's state file. Best-effort: pruning a stale file must never be fatal.
+ * Ignores ENOENT (already gone) and the Windows file-locking codes EPERM/EACCES/EBUSY — on Windows,
+ * unlinking a file another process still has open (or that is in "delete-pending" state) returns
+ * EPERM, unlike POSIX where it succeeds. Concurrent daemons sharing the .daemon_states dir race to
+ * prune the same dead-PID file; a loser there used to crash daemon startup. Another daemon will
+ * reclaim the file on its next prune.
+ */
 export async function deleteDaemonState(pid) {
     try {
         await fs.unlink(stateFilePath(pid));
     }
     catch (e) {
-        if (e.code !== "ENOENT")
+        const code = e.code;
+        if (code !== "ENOENT" && code !== "EPERM" && code !== "EACCES" && code !== "EBUSY")
             throw e;
     }
 }

package/oclif.manifest.json

@@ -639,7 +639,7 @@
         "quiet": {
           "char": "q",
           "name": "quiet",
-          "summary": "Only display community addresses",
+          "summary": "Only display community addresses (much faster: skips the per-community 'started' lookup)",
           "allowNo": false,
           "type": "boolean"
         }
@@ -871,5 +871,5 @@
       ]
     }
   },
-  "version": "0.19.71"
+  "version": "0.19.73"
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@bitsocial/bitsocial-cli",
-    "version": "0.19.71",
+    "version": "0.19.73",
     "description": "Command line interface to Bitsocial API",
     "types": "./dist/index.d.ts",
     "homepage": "https://github.com/bitsocialnet/bitsocial-cli",
@@ -119,7 +119,7 @@
         "@oclif/plugin-help": "6.2.36",
         "@oclif/plugin-not-found": "3.2.73",
         "@oclif/table": "0.5.1",
-        "@pkcprotocol/pkc-js": "0.0.45",
+        "@pkcprotocol/pkc-js": "0.0.48",
         "dataobject-parser": "1.2.22",
         "decompress": "4.2.1",
         "env-paths": "2.2.1",