@palettelab/cli 0.3.41 → 0.3.42

package/lib/commands/publish.js

@@ -23,6 +23,114 @@ function sha256(buf) {
   return crypto.createHash("sha256").update(buf).digest("hex")
 }
 
+function formatBytes(bytes) {
+  if (!Number.isFinite(bytes)) return null
+  const units = ["bytes", "KiB", "MiB", "GiB"]
+  let value = bytes
+  let unit = 0
+  while (value >= 1024 && unit < units.length - 1) {
+    value /= 1024
+    unit += 1
+  }
+  const fixed = unit === 0 ? String(value) : value.toFixed(value >= 10 ? 1 : 2)
+  const formatted = fixed.replace(/\.0+$|(\.\d*[1-9])0+$/, "$1")
+  return `${formatted} ${units[unit]}`
+}
+
+function plural(count, singular, pluralForm = `${singular}s`) {
+  return count === 1 ? singular : pluralForm
+}
+
+function routeLabel(route) {
+  if (!route?.path) return null
+  const methods = Array.isArray(route.methods) && route.methods.length ? route.methods.join(", ") : "ANY"
+  return `${methods} ${route.path}`
+}
+
+function extractMigrationHint(message) {
+  const match = String(message || "").match(
+    /^([^:]+): create_table\("([^"]+)"\) is missing ensure_org_rls\(op, "([^"]+)"\)/,
+  )
+  if (!match) return null
+  return { file: match[1], table: match[2] }
+}
+
+function explainPreflightFailure(result) {
+  const message = String(result?.message || "preflight check failed")
+  const fix = result?.fix ? String(result.fix) : null
+
+  if (result?.bytes && result?.limit && /bundle exceeds size limit/.test(message)) {
+    const kind = message.startsWith("frontend") ? "Frontend" : "Backend"
+    return {
+      title: `${kind} bundle is too large`,
+      details: [
+        `Current bundle: ${formatBytes(result.bytes)} (${result.bytes} bytes)`,
+        `Allowed limit: ${formatBytes(result.limit)} (${result.limit} bytes)`,
+      ],
+      fix,
+    }
+  }
+
+  if (result?.route && /has no require_permission\(\)/.test(message)) {
+    const label = routeLabel(result.route)
+    return {
+      title: "Backend route is missing a permission gate",
+      details: label ? [`Route: ${label}`] : [],
+      fix,
+    }
+  }
+
+  const migration = extractMigrationHint(message)
+  if (migration) {
+    return {
+      title: "Database migration is missing organization RLS",
+      details: [`File: ${migration.file}`, `Table: ${migration.table}`],
+      fix:
+        fix ||
+        `Call ensure_org_rls(op, "${migration.table}") after create_table, or add # palette:rls-ok only if the table is intentionally global.`,
+    }
+  }
+
+  return {
+    title: message,
+    details: [],
+    fix,
+  }
+}
+
+function printPreflightFailure(payload, fallbackOutput) {
+  const results = Array.isArray(payload?.results) ? payload.results : []
+  const failures = results.filter((result) => result?.status === "fail")
+  const warnings = results.filter((result) => result?.status === "warn")
+  const passed = results.filter((result) => result?.status === "ok").length
+
+  console.error("[pltt] preflight checks failed; publish cancelled before upload.")
+  if (!failures.length) {
+    if (fallbackOutput) console.error(fallbackOutput)
+    return
+  }
+
+  console.error(
+    `[pltt] ${failures.length} blocking ${plural(failures.length, "issue")} found. ` +
+      "Fix these and rerun the command.\n",
+  )
+  failures.forEach((failure, index) => {
+    const issue = explainPreflightFailure(failure)
+    console.error(`${index + 1}. ${issue.title}`)
+    for (const detail of issue.details) console.error(`   ${detail}`)
+    if (issue.fix) console.error(`   Fix: ${issue.fix}`)
+    console.error("")
+  })
+
+  if (passed > 0) {
+    console.error(`[pltt] ${passed} ${plural(passed, "check")} passed before the publish was blocked.`)
+  }
+  if (warnings.length) {
+    console.error(`[pltt] ${warnings.length} ${plural(warnings.length, "warning", "warnings")} were also reported by preflight.`)
+  }
+  console.error("[pltt] Need machine-readable details? Run `pltt test --json`.")
+}
+
 function runPreflight(cwd, json) {
   const cliBin = path.resolve(__dirname, "..", "..", "bin", "pltt.js")
   const res = spawnSync(process.execPath, [cliBin, "test", "--json"], {
@@ -46,8 +154,13 @@ function runPreflight(cwd, json) {
     }
     console.log(JSON.stringify({ ok: false, stage: "preflight", test: payload }, null, 2))
   } else {
-    console.error("[pltt] preflight contract tests failed; publish cancelled.")
-    if (output) console.error(output)
+    let payload = null
+    try {
+      payload = JSON.parse(res.stdout)
+    } catch {
+      payload = null
+    }
+    printPreflightFailure(payload, output)
   }
   process.exit(res.status || 1)
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@palettelab/cli",
-  "version": "0.3.41",
+  "version": "0.3.42",
   "description": "Developer CLI for building Palette platform plugins — no platform source access required.",
   "bin": {
     "pltt": "bin/pltt.js"