@botbuddy/cli@1.64.1
BotBuddy — Swarm coordination CLI for multi-agent workflows
AI Security Review
scanned 4d ago · by lpm-firewall-aiA user-run `bb wait` with malformed conditions automatically sends bounded diagnostics to the package’s relay using an existing credential. The event is minimized and does not expose secrets, but the runtime telemetry has no user-facing disclosure or opt-out, so this warrants a warning rather than a publish block.
Static reason
No blocking static signals were detected.; previous stored version diff introduced dangerous source
Trigger
A user runs `bb wait` with invalid condition syntax and a usable profile or binding bearer.
Impact
Tenant attribution and diagnostic categories are sent to the vendor service without a user-facing disclosure or opt-out; the event omits credentials and raw condition text.
Mechanism
The CLI posts a minimized error event to `/sensor-ingest` on `BOTBUDDY_RELAY_URL`, defaulting to `https://api.bot-buddy.ai/functions/v1`.
AI rationale
A user-run `bb wait` with malformed conditions automatically sends bounded diagnostics to the package’s relay using an existing credential. The event is minimized and does not expose secrets, but the runtime telemetry has no user-facing disclosure or opt-out, so this warrants a warning rather than a publish block.
Evidence
src/wait.mjssrc/wait-core.mjs
Network endpoints2
api.bot-buddy.ai/functions/v1/sensor-ingestBOTBUDDY_RELAY_URL + /sensor-ingest
Decision evidence
public snapshotAI called this Suspicious at 94.0% confidence as Dangerous Capability with medium false-positive risk.
Evidence for warning
- Invalid `bb wait` condition syntax triggers a setup-error report automatically.
- The report posts to `/sensor-ingest` on the configured relay and uses an existing bearer in the Authorization header.
- The event contains bounded error diagnostics; condition details are reduced to attempted types and parse-error categories, excluding raw condition text and values.
Evidence against
- The package manifest has no lifecycle scripts.
- The staging-pool LaunchAgent setup is behind an explicit `install` subcommand, which requires a tenant and checks that the caller is human.
- The telemetry is limited to diagnostic fields and does not include the bearer in the request body.
Behavioral surface
ChildProcessCryptoEnvironmentVarsFilesystemNetworkWebSocket
HighEntropyStringsUrlStrings
Source & flagged code
7 flaggedsrc/update-check.mjsView file
16import { readFileSync } from "node:fs";
L17: import { spawn, spawnSync } from "node:child_process";
L18: import { homedir } from "node:os";
High
Child Process
Package source references child process execution.
src/update-check.mjsView on unpkg · L16src/staging-pool/refresh-runtime.mjsView file
26const jobPath = join(app, "job.mjs");
L27: async function owned(path, directory = false, privateFile = false) {
L28: const info = await fs.lstat(path);
...
L42: const before = await configBytes();
L43: const installed = validateInstalledConfig(JSON.parse(before));
L44: const previous = installed.poolBinding;
...
L61: const rows = await fetchRegisteredPools({
L62: exec: (file, args, callOptions) => exec(process.execPath, [file, ...args], callOptions),
L63: botbuddyBin: cliEntry, env, tenant, poolId, preserveAgentContext: true, requireTenantMatch: true,
...
L125: await fs.rename(configTemp, configPath);
L126: return { ok: true, command: "refresh-runtime", tenant, poolId, runtimeVersion: metadata.version, runtimeSha256: contentDigest, personaAliases: Object.keys(next.personas), producer:
Medium
Install Persistence
Source writes installer persistence such as shell profile or service configuration.
Findings
5 High5 Medium3 Low
HighChild Processsrc/update-check.mjs
HighSame File Env Network Executionsrc/pw/daemon.mjs
HighTrigger Reachable Persistencesrc/staging-pool/cli.mjs
HighSemantic Analysis Limitedsrc/agent-credential-store.mjs#virtual:normalized:round1
HighPrevious Version Dangerous Deltasrc/pw/staging-auth.mjs
MediumNetwork
MediumEnvironment Vars
MediumInstall Persistencesrc/staging-pool/refresh-runtime.mjs
MediumShips Native Binarydaemon/bin/darwin-arm64/botbuddy-daemon
MediumStructural Risk Force Deep Review
LowFilesystem
LowHigh Entropy Strings
LowUrl Strings
Affected versions and remediation
This report applies to @botbuddy/cli@1.64.1.
See all versions of @botbuddy/cli for other recorded verdicts.
- Review the evidence and your use of @botbuddy/cli@1.64.1 before allowing it. Restrict the permissions described in this report.
- Choose an independently verified alternative or release. This report does not establish that other versions are safe.
Evidence last updated: .
Think this verdict is wrong? Dispute it on GitHub or read how reports are produced.