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

@decocms/blocks-cli@7.56.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

The package does not show malicious install-time behavior, but its shell interpolation creates a concrete user-invoked command-execution vulnerability. It should be warned and remediated by using argument arrays or strict validation.

Static reason

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

Trigger

A user runs deco-sync-blocks-to-kv with a crafted --since value, or runs the migration CLI with crafted command arguments.

Impact

Commands can run with the permissions of the user invoking the CLI and can alter the target project or access local data.

Mechanism

Shell command construction with interpolated CLI input.

AI rationale

Evidence

scripts/sync-blocks-to-kv.tsscripts/deco-migrate-cli.tsscripts/migrate.ts

Decision evidence

public snapshot

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

Evidence for warning

The sync CLI interpolates caller-controlled --since directly into an execSync shell command.
The migration CLI constructs shell commands from caller-controlled repository, branch, source, and destination values.
A migration run automatically invokes bun install in the target project.

Evidence against

package.json has no preinstall, install, postinstall, or other lifecycle hook.
The observed network and file-mutating features are exposed as named user-invoked CLI tools, not import-time behavior.
The observability command reads a local wrangler configuration and emits findings rather than sending its contents.

Behavioral surface

Source

ChildProcessCryptoDynamicRequireEnvironmentVarsFilesystemNetworkShell

Supply chain

HighEntropyStringsUrlStrings

Manifest

NoLicense

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.56.0.

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

Review the evidence and your use of @decocms/blocks-cli@7.56.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:57.110Z (UTC).

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