@blamejs/core 0.13.17 → 0.13.18

package/CHANGELOG.md

@@ -8,6 +8,8 @@ upgrading across more than a few patches at a time.
 
 ## v0.13.x
 
+- v0.13.18 (2026-05-27) — **`bodyParser` multipart can buffer uploads in memory — no tmp directory for serverless / read-only filesystems.** The multipart/form-data sub-parser previously streamed every file part to a tmp directory on disk (os.tmpdir() by default), which fails on a read-only or ephemeral serverless filesystem. A new multipart.storage option selects where file parts land: "disk" (default, unchanged — req.files[].path points at a tmp file cleaned up on response end) or "memory" (req.files[].buffer holds the assembled bytes, with no filesystem access at all). Both modes enforce the same per-file (fileSize), per-field, and total-request (totalSize) caps, so memory mode adds no new memory-exhaustion surface. The file object shape is stable across both modes — disk sets path with buffer null, memory sets buffer with path null — so a handler branches on whichever is non-null. An invalid storage value is rejected when the middleware is constructed. **Added:** *`bodyParser` multipart `storage: "memory"` — buffer uploads in RAM instead of a tmp directory* — `b.middleware.bodyParser({ multipart: { storage: "memory" } })` buffers each uploaded file part in memory and exposes it as `req.files[].buffer` (a Buffer), with no `os.tmpdir()` write and no tmp-file cleanup — the read-only / serverless path. The default `storage: "disk"` is unchanged: file parts stream to a tmp file, `req.files[].path` points at it, and it is removed when the response finishes. Both modes apply the existing `fileSize` / per-field `maxBytes` / `totalSize` caps and SHA3-512 hash each part during streaming, so memory mode is bounded by the same limits and adds no new DoS surface. The `req.files[]` shape is stable across modes (disk: `path` set, `buffer` null; memory: `buffer` set, `path` null). A `storage` value other than `"disk"` or `"memory"` throws a `TypeError` at construction.
+
 - v0.13.17 (2026-05-27) — **Template engine can render from a string with no views directory — for serverless / read-only filesystems.** b.template.create previously required a viewsDir that exists on disk, and rendering always read the template (and its layout/partials) from that directory — unusable on a read-only or ephemeral serverless filesystem where the templates aren't on disk. The engine now accepts a source string directly: viewsDir is optional, and the returned engine exposes renderString(source, data?, opts?) and compileString(source, opts?) that compile and render from a string with no disk read. {% extends %} and {{> partial}} in a string source resolve through an operator-supplied opts.resolve(name) -> string callback (without it, an extends throws a clear error and a missing partial inlines empty, matching the file path). The same HTML-escaping, expression grammar, and extends/partial-depth caps apply. The file-backed render / compile / precompileAll still work exactly as before when a viewsDir is configured, and now refuse with a clear error when one isn't. **Added:** *`engine.renderString` / `engine.compileString` — render templates from a string, no viewsDir* — `b.template.create({})` (no `viewsDir`) returns a string-only engine; `renderString(source, data?, { resolve })` and `compileString(source, { resolve })` compile and render from a source string with zero filesystem access — the read-only / serverless path. `{% extends %}` and `{{> partial}}` resolve through `opts.resolve(name) -> string`. The HTML escaping, grammar, and depth caps are identical to the file path. When a `viewsDir` IS configured, `render`/`compile`/`precompileAll` behave exactly as before; without one they refuse with `viewsDir not configured`. `renderString(source, { resolve })` may omit the data argument — an opts object carrying a function `resolve` is recognized as opts, not data. **Security:** *Vendored `@simplewebauthn/server` refreshed 13.3.0 → 13.3.1* — The vendored WebAuthn server bundle (`b.auth.passkey`'s registration/authentication verification) is refreshed to the latest upstream patch, with the MANIFEST version, CPE, and SHA-256 integrity hashes updated and the bundle re-verified.
 
 - v0.13.16 (2026-05-27) — **`b.mail.agent` docs now describe the facade accurately, and not-yet-wired verbs point to the primitive to use.** b.mail.agent's module documentation claimed it was "the standardization contract for every mail protocol" that JMAP / IMAP / POP3 all route through — but no protocol server actually dispatches through the agent (the framework's own JMAP EmailSubmission handler composes b.mail.send.deliver directly), and the compose / send / reply / forward, sieve.list / sieve.activate, identity / vacation / mdn.* and export / job / import verbs throw mail-agent/not-implemented. The docs are corrected to describe what the agent is: a mailbox-access facade (RBAC + posture + audit + dispatch around a mail store) whose read surface plus the mailbox-mutation and Sieve-upload methods are wired, with the remaining verbs not yet routed through it. Those verbs' error message now names the underlying primitive to compose directly (b.mail.send.deliver, b.mail.sieve, b.mailMdn, …) instead of citing a version tag that had long passed. The public WIRED_AT export (a method→version map that no longer reflected reality) is replaced by COMPOSE_HINT (a method→primitive-to-compose map). No behaviour change: the same methods are wired or throw exactly as before. **Changed:** *`b.mail.agent` documentation corrected; not-implemented errors point to the primitive to compose* — The `@module` / `@card` no longer claim the agent is the universal protocol-dispatch contract — it's documented as a mailbox-access facade with a wired read + mutation + Sieve-upload surface, and the compose/send/identity/vacation/MDN/export verbs documented as not yet routed through it (compose the underlying primitive directly until a protocol server adopts the agent). The `mail-agent/not-implemented` error now names that primitive (e.g. `b.mail.send.deliver`) rather than a passed version tag. **Removed:** *`b.mail.agent.WIRED_AT` export replaced by `COMPOSE_HINT`* — The `WIRED_AT` export mapped each method to a framework version that was supposed to "light it up" — versions that have all shipped without the wiring, so the map was misleading. It is replaced by `COMPOSE_HINT`, mapping each not-yet-wired method to the primitive an operator composes directly. Operators reading `b.mail.agent.WIRED_AT` should read `b.mail.agent.COMPOSE_HINT` instead (pre-1.0: no compatibility shim).

package/README.md

@@ -125,7 +125,7 @@ The framework bundles the surface a typical Node app reaches for. Every primitiv
   - Rate-limit
   - Security headers with `Permissions-Policy` defaults denying storage-access / browsing-topics / private-aggregation / controlled-frame
   - CSP nonce
-  - Body parser
+  - Body parser — JSON / urlencoded / text / multipart; multipart file parts stream to a tmp dir or buffer in memory (`storage: "memory"`) for read-only / serverless filesystems
   - Compression
   - SSE
   - Request log

package/lib/middleware/body-parser.js

@@ -37,7 +37,8 @@
  *     text:        { limit: b.constants.BYTES.mib(1), charset: "utf-8" },
  *     raw:         { limit: b.constants.BYTES.mib(10), contentTypes: ["application/octet-stream"] },
  *     multipart: {
- *       tmpDir:       os.tmpdir(),
+ *       storage:      "disk",                // "disk" → req.files[].path; "memory" → req.files[].buffer (serverless / read-only fs)
+ *       tmpDir:       os.tmpdir(),           // disk mode only
  *       fileSize:     b.constants.BYTES.mib(10),
  *       totalSize:    b.constants.BYTES.mib(50),
  *       fileCount:    20,
@@ -184,7 +185,8 @@ var DEFAULTS = Object.freeze({
     contentTypes: ["application/octet-stream"],
   },
   multipart: {
-    tmpDir:        null,             // resolved per-instance from os.tmpdir()
+    storage:       "disk",           // "disk" (tmp files) | "memory" (req.files[].buffer)
+    tmpDir:        null,             // resolved per-instance from os.tmpdir() (disk mode only)
     fileSize:      C.BYTES.mib(10),
     totalSize:     C.BYTES.mib(50),
     fileCount:     20,
@@ -680,16 +682,23 @@ async function _parseMultipart(req, opts, ctParams) {
       true, HTTP_STATUS.BAD_REQUEST
     );
   }
+  // storage: "memory" buffers file parts in RAM (capped by fileSize ×
+  // fileCount, the same DoS bound as disk mode) and exposes each file as
+  // req.files[].buffer with no filesystem touch — the read-only /
+  // serverless path. "disk" (default) streams to tmp files as before.
+  var useMemory = opts.storage === "memory";
   // Resolve tmpDir per-request so directory-creation failure surfaces as a
-  // structured error rather than a deferred fs throw.
-  var tmpDir = opts.tmpDir || nodePath.join(os.tmpdir(), "blamejs-uploads");
-  try { atomicFile.ensureDir(tmpDir, 0o700); }
-  catch (e) {
-    throw new BodyParserError(
-      "body-parser/multipart-tmpdir",
-      "could not create multipart tmp dir '" + tmpDir + "': " + ((e && e.message) || String(e)),
-      true, 500
-    );
+  // structured error rather than a deferred fs throw (disk mode only).
+  var tmpDir = useMemory ? null : (opts.tmpDir || nodePath.join(os.tmpdir(), "blamejs-uploads"));
+  if (!useMemory) {
+    try { atomicFile.ensureDir(tmpDir, 0o700); }
+    catch (e) {
+      throw new BodyParserError(
+        "body-parser/multipart-tmpdir",
+        "could not create multipart tmp dir '" + tmpDir + "': " + ((e && e.message) || String(e)),
+        true, 500
+      );
+    }
   }
 
   var boundaryBuf      = Buffer.from("--" + boundary);
@@ -721,7 +730,8 @@ async function _parseMultipart(req, opts, ctParams) {
   var currentFd = null;
   var currentSize = 0;
   var currentHash = null;
-  var currentBuf = null; // for fields (in-memory accumulator)
+  var currentBuf = null; // in-memory accumulator (text fields always; file parts when useMemory)
+  var currentIsFile = false; // file part (has a filename) vs text field — drives finalize shape
   var currentDiscarded = false; // true when fileFilter rejected the part — body bytes are
                                 // still consumed (we have to read past them to find the next
                                 // boundary) but never written to disk.
@@ -737,6 +747,7 @@ async function _parseMultipart(req, opts, ctParams) {
     currentSize = 0;
     currentHash = null;
     currentBuf = null;
+    currentIsFile = false;
     currentDiscarded = false;
     currentEffectiveLimit = 0;
   }
@@ -765,7 +776,8 @@ async function _parseMultipart(req, opts, ctParams) {
     if (currentFd !== null) { try { nodeFs.closeSync(currentFd); } catch (_e) { /* fd already closed */ } currentFd = null; }
     if (currentTmpPath) { try { nodeFs.unlinkSync(currentTmpPath); } catch (_e) { /* tmp file already removed */ } }
     for (var i = 0; i < files.length; i++) {
-      try { nodeFs.unlinkSync(files[i].path); } catch (_e) { /* tmp file already removed */ }
+      // Memory-mode files have no path (buffer only) — nothing to unlink.
+      if (files[i].path) { try { nodeFs.unlinkSync(files[i].path); } catch (_e) { /* tmp file already removed */ } }
     }
   }
 
@@ -943,17 +955,24 @@ async function _parseMultipart(req, opts, ctParams) {
                 }
               }
 
-              // Generate the tmp path — never derived from the
-              // operator-supplied filename.
-              var unique = bCrypto.generateToken(C.BYTES.bytes(16));
-              currentTmpPath = nodePath.join(tmpDir, "blamejs-up-" + unique);
-              try {
-                currentFd = nodeFs.openSync(currentTmpPath, "wx", 0o600);
-              } catch (e) {
-                done(new BodyParserError("body-parser/multipart-tmp-open",
-                  "could not open multipart tmp file: " + ((e && e.message) || String(e)),
-                  true, 500));
-                return;
+              currentIsFile = true;
+              if (useMemory) {
+                // Buffer the file in RAM — no filesystem touch. Bounded by
+                // currentEffectiveLimit (per-file) and totalSize (per-request).
+                currentBuf = [];
+              } else {
+                // Generate the tmp path — never derived from the
+                // operator-supplied filename.
+                var unique = bCrypto.generateToken(C.BYTES.bytes(16));
+                currentTmpPath = nodePath.join(tmpDir, "blamejs-up-" + unique);
+                try {
+                  currentFd = nodeFs.openSync(currentTmpPath, "wx", 0o600);
+                } catch (e) {
+                  done(new BodyParserError("body-parser/multipart-tmp-open",
+                    "could not open multipart tmp file: " + ((e && e.message) || String(e)),
+                    true, 500));
+                  return;
+                }
               }
               currentHash = nodeCrypto.createHash("sha3-512");
               currentSize = 0;
@@ -1004,8 +1023,10 @@ async function _parseMultipart(req, opts, ctParams) {
                     true, 413));
                   return;
                 }
-              } else if (currentFd !== null) {
-                // File part — write to disk.
+              } else if (currentIsFile) {
+                // File part — write to disk (currentFd) or accumulate in
+                // memory (useMemory). Same per-file + total-request caps
+                // either way, so memory mode adds no new DoS surface.
                 currentSize += bodyChunk.length;
                 if (currentSize > currentEffectiveLimit) {
                   var perFieldFile = (perField && perField[currentField] &&
@@ -1024,16 +1045,20 @@ async function _parseMultipart(req, opts, ctParams) {
                     true, 413));
                   return;
                 }
-                try {
-                  var written = 0;
-                  while (written < bodyChunk.length) {
-                    written += nodeFs.writeSync(currentFd, bodyChunk, written, bodyChunk.length - written);
+                if (currentFd !== null) {
+                  try {
+                    var written = 0;
+                    while (written < bodyChunk.length) {
+                      written += nodeFs.writeSync(currentFd, bodyChunk, written, bodyChunk.length - written);
+                    }
+                  } catch (e) {
+                    done(new BodyParserError("body-parser/multipart-tmp-write",
+                      "multipart tmp write failed: " + ((e && e.message) || String(e)),
+                      true, 500));
+                    return;
                   }
-                } catch (e) {
-                  done(new BodyParserError("body-parser/multipart-tmp-write",
-                    "multipart tmp write failed: " + ((e && e.message) || String(e)),
-                    true, 500));
-                  return;
+                } else {
+                  currentBuf.push(bodyChunk);
                 }
                 currentHash.update(bodyChunk);
               } else {
@@ -1067,17 +1092,27 @@ async function _parseMultipart(req, opts, ctParams) {
             if (currentDiscarded) {
               // fileFilter rejected — already recorded in filesRejected; no
               // tmp file was opened, nothing to clean up here.
-            } else if (currentFd !== null) {
-              try { nodeFs.closeSync(currentFd); } catch (_e) { /* fd already closed */ }
-              currentFd = null;
-              files.push({
+            } else if (currentIsFile) {
+              // Stable shape across both modes: disk gets path (buffer null),
+              // memory gets buffer (path null). Operators branch on whichever
+              // is non-null.
+              var fileEntry = {
                 field:    currentField,
                 filename: currentFilename,
                 mimeType: currentMime,
-                path:     currentTmpPath,
+                path:     null,
+                buffer:   null,
                 size:     currentSize,
                 hash:     currentHash.digest("hex"),
-              });
+              };
+              if (currentFd !== null) {
+                try { nodeFs.closeSync(currentFd); } catch (_e) { /* fd already closed */ }
+                currentFd = null;
+                fileEntry.path = currentTmpPath;
+              } else {
+                fileEntry.buffer = Buffer.concat(currentBuf);
+              }
+              files.push(fileEntry);
             } else {
               // Field part — flatten + decode UTF-8.
               var fbuf = Buffer.concat(currentBuf);
@@ -1106,6 +1141,7 @@ async function _parseMultipart(req, opts, ctParams) {
             currentSize = 0;
             currentHash = null;
             currentBuf = null;
+            currentIsFile = false;
             currentDiscarded = false;
             currentEffectiveLimit = 0;
             state = MP_AFTER_BD;
@@ -1159,11 +1195,13 @@ async function _parseMultipart(req, opts, ctParams) {
  * sub-parsers ship: JSON (via `safe-json` — POISONED_KEYS stripped,
  * depth + size caps), urlencoded, text, raw octet-stream, and
  * multipart/form-data. Multipart streams file parts to a tmp dir
- * with per-file + total-request size caps, filename sanitization,
- * SHA3-512 hashing during streaming, and tmp-file cleanup on
- * response end. Defends against RFC 9112 §6.1 request smuggling
- * before any body bytes are read. Each sub-parser can be disabled
- * by passing `false` in its slot.
+ * (`storage: "disk"`, default) or buffers them in RAM
+ * (`storage: "memory"` — for read-only / serverless filesystems,
+ * exposing `req.files[].buffer` instead of `.path`), with per-file +
+ * total-request size caps, filename sanitization, SHA3-512 hashing
+ * during streaming, and tmp-file cleanup on response end. Defends
+ * against RFC 9112 §6.1 request smuggling before any body bytes are
+ * read. Each sub-parser can be disabled by passing `false` in its slot.
  *
  * @opts
  *   {
@@ -1172,8 +1210,8 @@ async function _parseMultipart(req, opts, ctParams) {
  *     text:        false | { limit, charset, contentTypes },
  *     raw:         false | { limit, contentTypes },
  *     multipart:   false | {
- *       tmpDir, fileSize, totalSize, fileCount, fieldCount, fieldSize,
- *       mimeAllowlist, fileFilter, fields, audit, contentTypes,
+ *       storage, tmpDir, fileSize, totalSize, fileCount, fieldCount,
+ *       fieldSize, mimeAllowlist, fileFilter, fields, audit, contentTypes,
  *     },
  *     keepRawBody: boolean,    // expose req.bodyRaw for webhook signing
  *   }
@@ -1202,6 +1240,11 @@ function create(opts) {
   var textOpts        = _resolve("text");
   var rawOpts         = _resolve("raw");
   var multipartOpts   = _resolve("multipart");
+  if (multipartOpts && multipartOpts.storage !== "disk" && multipartOpts.storage !== "memory") {
+    throw new TypeError(
+      "middleware.bodyParser: multipart.storage must be \"disk\" or \"memory\" (got " +
+      JSON.stringify(multipartOpts.storage) + ")");
+  }
   var keepRawBody     = !!opts.keepRawBody;
 
   return async function bodyParser(req, res, next) {
@@ -1255,7 +1298,10 @@ function create(opts) {
           if (cleanedUp) return;
           cleanedUp = true;
           for (var i = 0; i < mpResult.files.length; i++) {
-            try { nodeFs.unlinkSync(mpResult.files[i].path); } catch (_e) { /* tmp file already removed */ }
+            // Memory-mode files (storage: "memory") have no path — nothing to unlink.
+            if (mpResult.files[i].path) {
+              try { nodeFs.unlinkSync(mpResult.files[i].path); } catch (_e) { /* tmp file already removed */ }
+            }
           }
         }
         res.on("finish", cleanup);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@blamejs/core",
-  "version": "0.13.17",
+  "version": "0.13.18",
   "description": "The Node framework that owns its stack.",
   "license": "Apache-2.0",
   "author": "blamejs contributors",

package/sbom.cdx.json

@@ -2,10 +2,10 @@
   "$schema": "http://cyclonedx.org/schema/bom-1.5.schema.json",
   "bomFormat": "CycloneDX",
   "specVersion": "1.5",
-  "serialNumber": "urn:uuid:2a708a4e-31ac-4ac7-abff-7e496d5700db",
+  "serialNumber": "urn:uuid:85f02e1d-6e57-4585-9067-a882fb442bc0",
   "version": 1,
   "metadata": {
-    "timestamp": "2026-05-27T19:07:43.710Z",
+    "timestamp": "2026-05-27T19:52:58.359Z",
     "lifecycles": [
       {
         "phase": "build"
@@ -19,14 +19,14 @@
       }
     ],
     "component": {
-      "bom-ref": "@blamejs/core@0.13.17",
+      "bom-ref": "@blamejs/core@0.13.18",
       "type": "application",
       "name": "blamejs",
-      "version": "0.13.17",
+      "version": "0.13.18",
       "scope": "required",
       "author": "blamejs contributors",
       "description": "The Node framework that owns its stack.",
-      "purl": "pkg:npm/%40blamejs/core@0.13.17",
+      "purl": "pkg:npm/%40blamejs/core@0.13.18",
       "properties": [],
       "externalReferences": [
         {
@@ -54,7 +54,7 @@
   "components": [],
   "dependencies": [
     {
-      "ref": "@blamejs/core@0.13.17",
+      "ref": "@blamejs/core@0.13.18",
       "dependsOn": []
     }
   ]