@factory/cli@0.239.1
Factory Droid CLI - AI-powered software engineering agent
AI Security Review
scanned 23d ago · by lpm-firewall-aiThe installer is aligned with the package’s platform-binary setup, but its fallback downloader has a conditional npm token exposure path because it accepts metadata-provided URLs and uses a hostname substring check. This supports a warning, not a confirmed attack or publish block.
Static reason
One or more suspicious static signals were detected.
Trigger
The postinstall runs and the platform binary is unavailable, causing the fallback download; NPM_TOKEN must also be set for token disclosure.
Impact
A matching but untrusted destination could receive the npm token. The source confirms this conditional exposure path but not an actual hostile destination.
Mechanism
The installer takes the tarball URL from registry metadata, follows redirects recursively, and adds the bearer token based on a hostname substring check rather than an exact destination allowlist.
AI rationale
The installer is aligned with the package’s platform-binary setup, but its fallback downloader has a conditional npm token exposure path because it accepts metadata-provided URLs and uses a hostname substring check. This supports a warning, not a confirmed attack or publish block.
Evidence
package.jsoninstall.jsbin/droid
Network endpoints1
registry.npmjs.org
Decision evidence
public snapshotAI called this Suspicious at 82.0% confidence as Dangerous Capability with medium false-positive risk.
Evidence for warning
- The postinstall fallback retrieves a tarball URL from npm metadata and uses it for a download request.
- When NPM_TOKEN is set, install.js attaches it to requests whose hostname contains “npmjs.org”; redirects are followed recursively through the same request function.
- The package’s ordinary installer purpose is to fetch a matching platform binary and link it into its own CLI shim.
Evidence against
- The fallback selects same-version platform packages named in this package’s optional dependencies.
- The inspected source does not establish that a hostile tarball URL or redirect is returned during normal installation.
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.1.
See all versions of @factory/cli for other recorded verdicts.
- Review the evidence and your use of @factory/cli@0.239.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.