@factory/cli@0.239.0
Factory Droid CLI - AI-powered software engineering agent
AI Security Review
scanned 23d ago · by lpm-firewall-aiThe package’s fallback is designed to obtain a same-version platform binary, but its hostname substring check does not bind NPM_TOKEN to the intended registry. This creates conditional credential exposure risk, so warn without blocking publication.
Static reason
One or more suspicious static signals were detected.
Trigger
The package is installed with lifecycle scripts enabled, its platform optional dependency is unavailable, and NPM_TOKEN is set.
Impact
A crafted metadata URL with a spoofed hostname could expose the installer’s npm token; the source does not establish that such a URL is used in this package’s normal install path.
Mechanism
The installer uses a substring hostname check for the bearer token and requests the tarball URL obtained from package metadata without verifying that its host is the intended registry.
AI rationale
The package’s fallback is designed to obtain a same-version platform binary, but its hostname substring check does not bind NPM_TOKEN to the intended registry. This creates conditional credential exposure risk, so warn without blocking publication.
Evidence
package.jsoninstall.jsbin/droid
Network endpoints1
registry.npmjs.org
Decision evidence
public snapshotAI called this Suspicious at 83.0% confidence as Dangerous Capability with medium false-positive risk.
Evidence for warning
- The package runs install.js automatically through its postinstall hook.
- The installer adds NPM_TOKEN as a bearer token when the request hostname merely contains npmjs.org.
- The fallback fetches a package tarball URL from npm registry metadata, then requests that URL.
- The fallback runs only when the platform-specific optional dependency is unavailable.
- The installer writes the downloaded binary into its bin directory.
Evidence against
- The fallback targets same-version @factory platform packages listed as optional dependencies.
- The installer’s first metadata request uses registry.npmjs.org.
Behavioral surface
ChildProcessEnvironmentVarsFilesystemNetworkShell
HighEntropyStringsUrlStrings
NoLicense
Source & flagged code
7 flaggedpackage.jsonView file
•scripts.postinstall = node install.js
High
Install Time Lifecycle Scripts
Package defines install-time lifecycle scripts.
package.jsonView on unpkg9scripts.postinstall = node install.js
Install-time code downloads an unverified remote native executable, stages it locally, activates an executable path, and exposes it to process execution.
L9: *
L10: * Based on esbuild's approach: https://github.com/evanw/esbuild/pull/1621
L11: * and Sentry's guide: https://sentry.engineering/blog/publishing-binaries-on-npm
L12: *
...
L19: const path = require('path');
L20: const https = require('https');
L21: const zlib = require('zlib');
...
L27: getBinaryName,
L28: getBinaryPath,
L29: detectAVX2Support,
...
L50:
L51: function validateBinaryVersion(binaryPath) {
High
Unverified Remote Native Payload Install
Install-time source downloads a native archive from a fixed external host without transport verification, extracts it, and installs an executable payload.
package.jsonView on unpkg · L9•scripts.postinstall = node install.js
Medium
Ambiguous Install Lifecycle Script
Install-time lifecycle script is not statically allowlisted and needs review.
Findings
6 High4 Medium5 Low
HighInstall Time Lifecycle Scriptspackage.json
HighChild Processinstall.js
HighShellinstall.js
HighSame File Env Network Executioninstall.js#virtual:normalized:round1
HighUnverified Remote Native Payload Installpackage.json
HighSemantic Analysis Limitedinstall.js#virtual:normalized:round1
MediumAmbiguous Install Lifecycle Scriptpackage.json
MediumNetwork
MediumEnvironment Vars
MediumStructural Risk Force Deep Review
LowScripts Present
LowFilesystem
LowHigh Entropy Strings
LowUrl Strings
Low
Affected versions and remediation
This report applies to @factory/cli@0.239.0.
See all versions of @factory/cli for other recorded verdicts.
- Review the evidence and your use of @factory/cli@0.239.0 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.