registry / AI-agent capability abuse / @decocms/blocks-cli / 7.53.0

@decocms/blocks-cli@7.53.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 package is not established as malicious, but the generated persistent agent instruction creates a concrete destructive risk, and analytics provisioning has an unescaped SQL input. Both behaviors are conditional on a user-run migration, so the evidence supports a warning rather than a publication block.

Static reason

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

Trigger

A user runs deco-migrate outside dry-run mode; the SQL issue additionally requires SUPABASE_ACCESS_TOKEN to be set.

Impact

An AI agent following the generated rule may delete the working tree. A crafted package name may alter the SQL executed against the configured Supabase project.

Mechanism

The migration writes the Cursor rule into the project and removes src/AGENTS.md during cleanup. Analytics provisioning interpolates the project's package name into a SQL query and sends it to Supabase with the supplied token.

AI rationale

Evidence

scripts/migrate.tsscripts/migrate/phase-scaffold.tsscripts/migrate/phase-cleanup.tsscripts/migrate/templates/cursor-rules.ts.cursor/rules/migration-tooling-policy.mdcsrc/AGENTS.md

Network endpoints1

api.supabase.com

Decision evidence

public snapshot

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

Evidence for warning

The requested scaffold location writes a project-local Cursor policy pointer during migration.
The cleanup phase removes src/AGENTS.md when present unless running in dry-run mode.
The generated Cursor rule applies to every session in the migrated site and tells agents to delete the working tree after a failed migration.
A non-dry-run migration runs analytics provisioning; it reads the target project's package name and skips provisioning if SUPABASE_ACCESS_TOKEN is absent.
With that token present, the migration inserts the package name directly into SQL and sends the query to the fixed Supabase project using bearer authentication.
The manifest exposes migration as a CLI and lists ordinary package scripts, with no install lifecycle script.

Evidence against

Behavioral surface

Source

ChildProcessCryptoEnvironmentVarsFilesystemNetworkShell

Supply chain

HighEntropyStrings

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 (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/audit-secrets.tsView file

•matchType = previous_version_dangerous_delta matchedPackage = @decocms/blocks-cli@7.62.5 matchedIdentity = npm:QGRlY29jbXMvYmxvY2tzLWNsaQ:7.62.5 similarity = 0.750 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.53.0.

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

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

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