@factory/cli@0.250.0
Factory Droid CLI - AI-powered software engineering agent
AI Security Review
scanned 8d ago · by lpm-firewall-aiThe postinstall fallback has a weak destination check for forwarding NPM_TOKEN across redirects. The inspected source does not prove an attacker-controlled recipient or malicious intent, so this warrants a warning rather than a block.
Static reason
One or more suspicious static signals were detected.
Trigger
Postinstall runs when no platform binary is found and the fallback downloader is used.
Impact
A redirect to a matching but unrelated hostname could disclose the environment-held npm token.
Mechanism
The downloader follows redirect locations and adds NPM_TOKEN to requests whose hostname contains npmjs.org, without an exact-host check.
AI rationale
The postinstall fallback has a weak destination check for forwarding NPM_TOKEN across redirects. The inspected source does not prove an attacker-controlled recipient or malicious intent, so this warrants a warning rather than a block.
Evidence
package.jsoninstall.js
Network endpoints1
registry.npmjs.org
Decision evidence
public snapshotAI called this Suspicious at 84.0% confidence as Dangerous Capability with medium false-positive risk.
Evidence for warning
- package.json runs install.js during postinstall.
- The fallback downloader adds NPM_TOKEN as an Authorization header when the hostname contains npmjs.org.
- The downloader follows redirect locations by calling makeRequest with the supplied URL; the token check is repeated for each destination.
- The fallback download is used when no platform binary is found during postinstall.
Evidence against
- The fallback downloads the platform binary from the npm registry, matching the package’s declared platform-specific dependencies.
- The inspected source does not establish an attacker-selected redirect destination or a package-controlled recipient.
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.250.0.
See all versions of @factory/cli for other recorded verdicts.
- Review the evidence and your use of @factory/cli@0.250.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.