registry / Remote code execution / @decocms/blocks-cli / 7.59.0

@decocms/blocks-cli@7.59.0

Deco codegen (generate-blocks, generate-schema, generate-invoke) and Fresh-to-TanStack migration tooling

AI Security Review

scanned 1mo ago · by lpm-firewall-ai

This release is not malware, but it contains concrete user-invoked command-injection vulnerabilities in published CLI code. The absence of lifecycle hooks limits the attack to explicit CLI use.

Static reason

One or more suspicious static signals were detected.; previous stored version diff introduced dangerous source

Trigger

Run scripts/deco-migrate-cli.ts against a Git URL with a crafted --branch argument.

Impact

Arbitrary commands can run with the privileges of the user invoking the migration CLI.

Mechanism

Unquoted user input interpolated into execSync shell commands.

AI rationale

This release is not malware, but it contains concrete user-invoked command-injection vulnerabilities in published CLI code. The absence of lifecycle hooks limits the attack to explicit CLI use.

Evidence

scripts/deco-migrate-cli.ts

Decision evidence

public snapshot

AI called this Suspicious at 96.0% confidence as Critical Vulnerability with low false-positive risk.

Evidence for warning

The migration CLI interpolates the user-controlled --branch value into a shell command passed to execSync, enabling command injection when that CLI is run.
The KV sync CLI also interpolates the user-controlled --since value into execSync without argument separation.
The package exposes mutable deployment and migration commands, but they are user-invoked rather than install-time.
The remaining flagged dynamic process execution is confined to test fixtures or an explicit audit CLI.

Evidence against

package.json contains no preinstall, install, or postinstall lifecycle hook.
No runtime self-dependency on @decocms/blocks-cli was found.
The Cloudflare purge request requires explicit purge URL and token options.

Behavioral surface

Source

ChildProcessCryptoDynamicRequireEnvironmentVarsFilesystemNetworkShell

Supply chain

HighEntropyStrings

Source & flagged code

6 flagged

scripts/audit-observability-config.test.tsView file

Lines 289-329typescript

289});

290

291describe("CLI gate hardness (D-16) — --mode warn|block + --github", () => {

292 let tmpdir: string;

293 const cliPath = path.resolve(__dirname, "audit-observability-config.ts");

294

295 beforeEach(() => {

296 tmpdir = fs.mkdtempSync(path.join(os.tmpdir(), "audit-mode-"));

297 });

298 afterEach(() => {

299 fs.rmSync(tmpdir, { recursive: true, force: true });

300 });

301

302 // Spawn the script via tsx in a child process so we exercise the real

303 // `process.exit()` paths instead of monkey-patching them. This is the

304 // contract storefront CI consumes, so it's the contract under test.

305 function runCli(args: string[]): {

306 status: number | null;

307 stdout: string;

308 stderr: string;

309 } {

310 const { spawnSync } = require("node:child_process") as typeof import(

311 "node:child_process"

High

Child Process

Package source references child process execution.

scripts/audit-observability-config.test.tsView on unpkg · L309

Medium

Dynamic Require

Package source references dynamic require/import behavior.

scripts/audit-observability-config.test.tsView on unpkg · L309

312 );

313 const result = spawnSync(

314 process.execPath,

315 [

316 require.resolve("tsx/cli"),

317 cliPath,

318 "--source",

319 tmpdir,

320 ...args,

321 ],

322 { encoding: "utf8" },

323 );

324 return {

325 status: result.status,

326 stdout: result.stdout,

327 stderr: result.stderr,

328 };

329 }

scripts/sync-blocks-to-kv.tsView file

Lines 57-97typescript

57import { readDecofileFromDir } from "./lib/read-decofile";

58import { changedBlockFiles, changedBlockKeys, purgePathsForChangedKeys } from "./lib/sync-helpers";

60const DEFAULT_RETAIN = 10;

scripts/generate-invoke.test.tsView file

Lines 200-240typescript

200 // `--apps-dir node_modules/@decocms/apps-vtex/src` by hand (granadobr's

201 // migration workaround). The default must probe <pkg>/invoke.ts first,

202 // then <pkg>/src/invoke.ts.

203

204 function makeSite(layout: "root" | "src")

scripts/audit-observability-config.ts#virtual:normalized:round1View file

•stage = ast_semantic_analysis; reason = ast_parse_error; limitedFiles = 9

High

Semantic Analysis Limited

A bounded semantic-analysis stage reached its safety limit; remaining detectors completed, but this package requires AI review.

scripts/audit-observability-config.ts#virtual:normalized:round1View on unpkg

scripts/deco-migrate-cli.tsView file

•matchType = previous_version_dangerous_delta matchedPackage = @decocms/blocks-cli@7.20.4 matchedIdentity = npm:QGRlY29jbXMvYmxvY2tzLWNsaQ:7.20.4 similarity = 0.537 summary = stored previous version shares package body but lacks this dangerous source file

Critical

Previous Version Dangerous Delta

This package version adds a dangerous source file absent from the previous stored version; route for source-aware review.

scripts/deco-migrate-cli.tsView on unpkg

Source & flagged code

6 flagged · loading source

scripts/audit-observability-config.test.tsView file

309} { L310: const { spawnSync } = require("node:child_process") as typeof import( L311: "node:child_process"

High

Child Process

Package source references child process execution.

scripts/audit-observability-config.test.tsView on unpkg · L309

309} { L310: const { spawnSync } = require("node:child_process") as typeof import( L311: "node:child_process"

Medium

Dynamic Require

Package source references dynamic require/import behavior.

scripts/audit-observability-config.test.tsView on unpkg · L309

scripts/sync-blocks-to-kv.tsView file

77purgeUrl: val("--purge-url", ""), L78: purgeToken: val("--purge-token", process.env.PURGE_TOKEN ?? ""), L79: }; ... L82: function gitChangedFiles(since: string, blocksDir: string): string[] { L83: const out = execSync(`git diff --name-only ${since} HEAD`, { encoding: "utf-8" }); L84: return changedBlockFiles(out, blocksDir); ... L87: async function purgeCache(origin: string, token: string, paths: string[]): Promise<void> { L88: const res = await fetch(new URL("/_cache/purge", origin).toString(), { L89: method: "POST",

Findings

1 Critical5 High4 Medium5 Low

CriticalPrevious Version Dangerous Deltascripts/deco-migrate-cli.ts

HighChild Processscripts/audit-observability-config.test.ts

HighShell

HighSame File Env Network Executionscripts/sync-blocks-to-kv.ts

HighRuntime Package Installscripts/generate-invoke.test.ts

HighSemantic Analysis Limitedscripts/audit-observability-config.ts#virtual:normalized:round1

MediumDynamic Requirescripts/audit-observability-config.test.ts

MediumNetwork

MediumEnvironment Vars

MediumStructural Risk Force Deep Review

LowScripts Present

LowFilesystem

LowHigh Entropy Strings

Affected versions and remediation

This report applies to @decocms/blocks-cli@7.59.0.

See all versions of @decocms/blocks-cli for other recorded verdicts.

Review the evidence and your use of @decocms/blocks-cli@7.59.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: 2026-09-08 19:00:44.171Z (UTC).

Think this verdict is wrong? Dispute it on GitHub or read how reports are produced.