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

@decocms/blocks-cli@7.65.0

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

AI Security Review

scanned 5d ago · by lpm-firewall-ai

The source confirms shell command injection through the CLI's `--since` argument, so this is a real vulnerability rather than a scanner-only signal. It is reachable through explicit CLI use, and the source does not show automatic install-time execution.

Static reason

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

Trigger

Run `deco-sync-blocks-to-kv` without `--all`, providing a shell expression as the value for `--since`.

Impact

An attacker who can influence the CLI arguments can execute arbitrary local commands as the user running the sync command.

Mechanism

The CLI parses `--since` from its arguments and interpolates it into a `git diff` command executed through `execSync`.

AI rationale

Evidence

scripts/sync-blocks-to-kv.tspackage.json

Decision evidence

public snapshot

AI called this Suspicious at 95.0% confidence as Dangerous Capability with medium false-positive risk.

Evidence for warning

The package exposes the KV sync script as a CLI command.
The CLI takes `--since` from its arguments and passes it to `gitChangedFiles`.
`gitChangedFiles` interpolates `since` into a shell command executed with `execSync`.
Unless `--all` is set, the sync flow calls `gitChangedFiles`; this creates a command-injection vulnerability when `--since` contains shell syntax.

Evidence against

The vulnerable code is in an explicit CLI command, and `--all` skips the affected git command.

Behavioral surface

Source

ChildProcessCryptoEnvironmentVarsFilesystemNetworkShell

Supply chain

HighEntropyStringsUrlStrings

Manifest

NoLicense

scanned 160 file(s), 1.49 MB of source, external domains: analytics.tiktok.com, api.cloudflare.com, api.example.com, api.supabase.com, bat.bing.com, cdn.example.com, connect.facebook.net, deco-otel-ingest.deco-cx.workers.dev, deco-otel-ingest.example, decoims.com, fonts.googleapis.com, fonts.gstatic.com, github.com, graph.instagram.com, ingest.example, ingest.example.com, s.lilstts.com, script.hotjar.com, scripts.clarity.ms, sp.vtex.com, static.hotjar.com, storage.googleapis.com, tailwindcss.com, www.clarity.ms, www.googletagmanager.com, www.minhaloja.com.br, x.com

Source & flagged code

5 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

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 (a production

201 // site's 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/audit-secrets.tsView file

•matchType = previous_version_dangerous_delta matchedPackage = @decocms/blocks-cli@7.62.5 matchedIdentity = npm:QGRlY29jbXMvYmxvY2tzLWNsaQ:7.62.5 similarity = 0.758 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/audit-secrets.tsView on unpkg

Source & flagged code

5 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

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",

High

Same File Env Network Execution

A single source file combines environment access, network access, and code or shell execution; review context before blocking.

scripts/sync-blocks-to-kv.tsView on unpkg · L77

Findings

1 Critical5 High3 Medium5 Low

CriticalPrevious Version Dangerous Deltascripts/audit-secrets.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

MediumNetwork

MediumEnvironment Vars

MediumStructural Risk Force Deep Review

LowScripts Present

LowFilesystem

LowHigh Entropy Strings

LowUrl Strings

LowNo License

Affected versions and remediation

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

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

Review the evidence and your use of @decocms/blocks-cli@7.65.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-10-18 03:06:18.507Z (UTC).

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