@factory/cli@0.247.0
Factory Droid CLI - AI-powered software engineering agent
AI Security Review
scanned 14d ago · by lpm-firewall-aiThe postinstall behavior is otherwise consistent with platform-binary setup, but its fallback can send an existing npm token to a non-registry hostname that passes a substring check. This supports a conditional credential-exposure warning, not a confirmed theft finding.
Static reason
One or more suspicious static signals were detected.
Trigger
Package installation when the platform binary is unavailable and NPM_TOKEN is set.
Impact
A matching non-registry recipient could receive the caller’s npm token. The inspected source does not show that such a recipient was contacted.
Mechanism
The fallback reads a tarball URL from registry metadata and follows redirects; requests whose hostname contains “npmjs.org” receive the token.
AI rationale
The postinstall behavior is otherwise consistent with platform-binary setup, but its fallback can send an existing npm token to a non-registry hostname that passes a substring check. This supports a conditional credential-exposure warning, not a confirmed theft finding.
Evidence
package.jsoninstall.js
Network endpoints1
registry.npmjs.org
Decision evidence
public snapshotAI called this Suspicious at 86.0% confidence as Dangerous Capability with medium false-positive risk.
Evidence for warning
- The postinstall hook runs install.js, which falls back to downloading a platform binary when its optional dependency is missing.
- The fallback obtains its tarball URL from npm registry metadata and follows redirects recursively.
- When NPM_TOKEN is set, requests to any hostname containing “npmjs.org” receive it in an Authorization header; the check does not require the exact registry host.
Evidence against
- The token must already be present in the caller’s environment, and this path runs only when the platform binary is unavailable.
- The fallback serves the package’s platform-binary setup; the inspected source does not establish that a non-registry recipient was actually contacted.
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.247.0.
See all versions of @factory/cli for other recorded verdicts.
- Review the evidence and your use of @factory/cli@0.247.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.