@factory/cli@0.245.0
Factory Droid CLI - AI-powered software engineering agent
AI Security Review
scanned 16d ago · by lpm-firewall-aiThe normal fallback fetches a same-version platform binary from the npm registry, but its redirect handling can forward NPM_TOKEN to a loosely matched hostname. This is a real conditional credential exposure warranting a warning, with no source evidence establishing active theft.
Static reason
One or more suspicious static signals were detected.
Trigger
Postinstall runs and the platform binary is missing, causing the npm fallback download.
Impact
A matching redirect destination could receive the installer's npm token.
Mechanism
install.js adds NPM_TOKEN based on a substring hostname check and follows redirect locations without restricting their host.
AI rationale
The normal fallback fetches a same-version platform binary from the npm registry, but its redirect handling can forward NPM_TOKEN to a loosely matched hostname. This is a real conditional credential exposure warranting a warning, with no source evidence establishing active theft.
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
- The postinstall hook runs install.js, which fetches a platform package from the npm registry when the optional binary is missing.
- install.js attaches NPM_TOKEN to requests when the hostname contains npmjs.org, and follows redirects by recursively requesting the supplied location.
- The redirect destination is not restricted before the next request, so a redirect to a hostname containing npmjs.org could receive NPM_TOKEN.
Evidence against
- The token is only added when NPM_TOKEN is set and the request hostname contains npmjs.org.
- The fallback download is used only when the platform binary from optionalDependencies is unavailable.
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.245.0.
See all versions of @factory/cli for other recorded verdicts.
- Review the evidence and your use of @factory/cli@0.245.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.