@blamejs/core 0.9.18 → 0.9.19

package/CHANGELOG.md

@@ -8,6 +8,7 @@ upgrading across more than a few patches at a time.
 
 ## v0.9.x
 
+- v0.9.19 (2026-05-14) — **First slice of the blamepost mail-stack sequence — `b.mailStore` + `b.safeMime` + `b.guardMessageId` substrates.** Byte-level mail-store foundation that every above-the-wire mail primitive composes (agent at v0.9.20, MX listener at v0.9.23, submission listener at v0.9.24, JMAP/IMAP/POP3 at v0.9.26-29, ManageSieve at v0.9.30, DAV at v0.9.32). (1) **`b.safeMime`** — RFC 5322 + 2045/2046/2047/EAI MIME parser. Bounded: total parts cap (default 64), nesting-depth cap (default 16), boundary length cap (default 70 per RFC 2046 §5.1.1), header-bytes cap (default 64 KiB), header-line cap (default 998 per RFC 5322 §2.1.1), body-bytes cap (default 25 MiB), message cap (default 50 MiB), charset allowlist (UTF-8 / US-ASCII / common legacy 8-bit), transfer-encoding allowlist (7bit/8bit/binary/qp/base64). Surface: `parse(bytes, opts) → tree`, `walk(tree, visitor)`, `findFirst(tree, predicate)`, `extractText(tree, opts)` (RFC 2046 §5.1.4 last-wins for `multipart/alternative`), `extractAttachments(tree, opts)`. Includes RFC 2047 Q + B encoded-word decoding for `Subject:` / `From:` etc. + RFC 2231 charset'lang'value filename decoding. Throws `safe-mime/<code>` on every cap exceeded / malformed boundary / unknown charset / unknown CTE / control chars in headers / NUL bytes. **Defends CVE-2024-39929** (Exim MIME multipart parser) and **CVE-2025-30258** (gnumail truncated-MIME-tree class). Fuzz harness ships in `fuzz/safe-mime.fuzz.js`. (2) **`b.guardMessageId`** — RFC 5322 §3.6.4 Message-Id validator. Gates Message-Id / In-Reply-To / References at the mail-store append boundary, the MX inbound boundary (v0.9.23), and the submission outbound path (v0.9.24). Refuses oversized (>998 bytes), bare CR/LF/NUL/C0-control/DEL (header-injection defense — defends `From:` / `Bcc:` smuggling via folded Message-Id continuation), unbracketed under strict profile, empty value, missing `@`, nested angle brackets, bidi codepoints (CVE-2021-42574 RTLO class in mail-header context). Profile family: strict (default) / balanced / permissive. Posture family: hipaa / pci-dss / gdpr / soc2 → all pin profile to strict. Surface: `validate(value, opts)`, `validateList(value, opts)` (References-chain cap = 100), `compliancePosture(posture)`. Fuzz harness ships in `fuzz/guard-message-id.fuzz.js`. (3) **`b.mailStore`** — byte-level mail-store substrate with pluggable backend (sqlite default; operator's `b.externalDb` Postgres or any `{ prepare(sql) → { run, get, all } }`-shaped object). Surface: `create(opts)` returning `{ appendMessage, fetchByObjectId, queryByModseq, setFlags, createFolder, listFolders, threadFor, quota, setLegalHold }`. **Sealed by default** via `b.cryptoField.sealRow` — `subject` / `from_addr` / `to_addrs` / `body_text` / `body_html` route through vault-managed AEAD envelope on insert + unseal on fetch. Plaintext (forensic-queryable without unsealing): `objectid` / `modseq` / `internal_date` / `received_at` / `size_bytes` / `flags` / `legal_hold` / `from_hash` / `message_id_hash`. Per-folder monotonic `modseq` counter (RFC 7162 CONDSTORE substrate). Per-message `objectid` (RFC 8474 JMAP cross-protocol identity). Threading at append time via In-Reply-To + References chain walk (cryptoField.lookupHash for hash-aware threading on sealed columns). Quota substrate (per-folder `used_bytes` + `used_count` maintained atomically). Legal-hold flag composes existing `b.legalHold`. Schema bootstraps at construction with six IMAP4rev2 default folders (INBOX / Sent / Drafts / Trash / Junk / Archive) and JMAP role mapping. Append composes `b.safeMime.parse` (bounded inbound) + `b.guardMessageId.validate` (header-injection gate). **Per the operator-confirmed blamepost roadmap** (`memory/specs/blamepost-roadmap.md`); next slice v0.9.20 wires `b.mail.agent` on top of this substrate.
 - v0.9.18 (2026-05-14) — **18 CodeQL alerts closed across 4 rule classes + SECURITY.md hardening checklist additions for v0.9.13+ primitives + MIGRATING.md out-of-band breaking-changes section.** Post-v0.9.17 audit identified 18 pre-existing CodeQL security findings on `main` — accumulated over many releases, surfaced explicitly when v0.9.15's rename sweep changed line content. v0.9.18 closes them all. (1) **`js/file-system-race` (6 sites)** — TOCTOU between `fs.existsSync()` / `fs.statSync()` and a subsequent file op. Fixed via the framework's canonical TOCTOU-safe-read scaffold (open fd first → `fstatSync` → `readSync` loop → `closeSync` in `finally`) at `lib/atomic-file.js` (`_readSyncCore`), `lib/restore-rollback.js` (marker write switched to exclusive-create `wx` + EEXIST-tolerant), `lib/network-tls.js` (`_readPathFile` extraction with per-file ENOENT tolerance), `lib/backup/bundle.js` (open-fd-first plus required-vs-skip branch routing), `lib/static.js` (request-serve hot path narrowed to single fd). `lib/vault/seal-pem-file.js` retained as-is with a CodeQL suppression — the site has an in-line `lstat.ino === fstat.ino` inode-equality defense (line 290) that refuses with `seal-pem-file/toctou-detected` if an attacker swaps the file between `lstat` and `open`. (2) **`js/insecure-temporary-file` (6 sites)** — predictable temp paths. `lib/vault/rotate.js` now uses `mkdtempSync` for a per-rotation random scratch dir + plain filenames inside (replaces the predictable `_blamejs_rotate.tmp.db` / `_blamejs_verify.tmp.db` paths in `stagingDir`). `lib/mtls-ca.js` switched to exclusive-create `openSync(..., "wx", 0o600)` + `writeSync` + `fsyncSync` so an attacker pre-creating the path is refused at `EEXIST`. `lib/atomic-file.js` (`fsyncDir`), `lib/vault/rotate.js` (`_fsyncFileByPath`), `lib/http-client.js` (atomic tmp path) retained as-is with suppressions — `dirPath` / `p` are operator-supplied framework data paths (not `os.tmpdir`-reachable), and `tmpPath` carries 16 hex chars of crypto-random suffix (line 1802 `dest + ".tmp-" + bCrypto.generateToken(8)`). (3) **`js/path-injection` (2 sites in `lib/static.js`)** — `nodeFs.createReadStream(absPath)` in `_readMeta` (line 161) and the request-serve hot path (line 1115). Suppression comments added referencing the upstream `_resolveSafe` lexical-resolve + `startsWith(rootResolved + nodePath.sep)` + realpath escape check at lines 181-207 — `absPath` is sandbox-validated against `root` before reaching these lines. (4) **`js/remote-property-injection` (4 sites)** — `lib/websocket.js` (`ext.params: {}` → `Object.create(null)`), `lib/middleware/csrf-protect.js` (`var out = {}` → `Object.create(null)` for cookie-parse output). `lib/middleware/body-parser.js` (multipart `fields[currentField] = ...`) retained as-is with suppression — `currentField` is gated upstream at line 867 by `POISONED_KEYS = new Set(["__proto__", "constructor", "prototype"])` refusing the field BEFORE assignment with a 400 BodyParserError. **Plus: SECURITY.md hardening checklist** gains 5 lines covering `b.middleware.idempotencyKey.dbStore` (hash + seal defaults), `b.metrics.snapshot` (out-of-process metrics export), `b.selfUpdate.standaloneVerifier` (zero-dep install-pipeline verifier), `b.pqcAgent.reload` (TLS-posture refresh without restart), `b.crypto.hashFilesParallel` (parallel SBOM/integrity-sweep hashing). **Plus: MIGRATING.md** now carries an "Out-of-band breaking changes" section (the v0.9.15 dbStore schema break is the first entry); `scripts/gen-migrating.js` extended with an `OUT_OF_BAND_BREAKS` table so future schema/on-disk format breaks land in MIGRATING.md without operators needing to grep CHANGELOG.
 - v0.9.17 (2026-05-14) — **Two new `codebase-patterns` detectors + 192-site cleanup sweep — `node:` prefix consistency + internal-binding leak prevention.** Post-v0.9.16 audit surfaced two enforceable invariants the existing detectors didn't cover. (1) **`node-builtin-prefix` detector** — every `require("<X>")` of a Node built-in (`fs`, `path`, `crypto`, `stream`, `tls`, `url`, `os`, `net`, `http`, `http2`, `https`, `zlib`, `dgram`, `events`, `child_process`, `readline`, …) must use the modern `require("node:<X>")` form. Three reasons: (a) userland packages on npm CAN be named after built-ins, so without the `node:` prefix a typo or `npm install` accident could shadow the built-in; (b) the prefix is a clearer at-a-glance signal that the dependency is on Node, not on a userland module; (c) bundler / SEA static-trace passes treat `node:` prefix as an unambiguous Node-builtin marker. Sweep: 153 `require()` rewrites across 79 framework files (2 parallel agents). The detector skips JSDoc `@example` block continuation lines (`*`-prefixed), so operator-facing examples that show `var fs = require("fs")` aren't rewritten — operators write their own bindings however they prefer. (2) **`internal-binding-in-prose` detector** — internal binding names (`nodeFs` / `nodePath` / `nodeCrypto` / `nodeStream` / `nodeTls` / `nodeUrl` / `bCrypto` / `retryHelper`) must NOT appear in operator-facing surface: JSDoc/comment continuation lines or string literals (error messages, audit metadata). Operators see the public API name (`path` / `fs` / `crypto` / `retry` / …), never the framework's internal alias. Sweep: 39 prose-leak fixes across 16 files — comments rewritten to use the operator-facing word (`nodePath` → `path`, `nodeFs.watch failed` → `fs.watch failed`, debug-log `"op": "nodeFs.unlinkSync"` → `"op": "fs.unlinkSync"`). (3) **2 follow-on require-binding canonicalizations** surfaced by the node-prefix sweep — `lib/ws-client.js` now destructures `var { EventEmitter } = require("node:events")` (was binding the entire `events` module to a class-shaped name) and `lib/process-spawn.js` renames inline `nodeChild` → `childProcess` (matches the module-level `childProcess` lazyRequire in `lib/dev.js`).
 - v0.9.16 (2026-05-14) — **Operator-facing prose cleanup + `require-binding-name` detector now covers `lazyRequire` wrappers + dbStore seal round-trip test added.** Post-v0.9.15 audit surfaced three classes of follow-up. (1) **Operator-facing prose leaks (7 sites)** — the v0.9.15 mechanical rename pattern `<OLD>.` → `<NEW>.` also caught occurrences inside JSDoc `@opts` comments and error-message string literals, so operators reading `b.keychain.create(opts)` saw `// absolute nodePath; required if file fallback may engage` instead of `// absolute path`. Fixed: `lib/db.js` (stream-limit error), `lib/keychain.js` (fallback-file error + 3 JSDoc lines), `lib/restore-bundle.js` (staging-dir error), `lib/watcher.js` (fs.watch failure error). Operators see plain English; internal binding names stay internal. (2) **`require-binding-name` detector extended to cover `lazyRequire`** — the v0.9.15 detector only matched plain `var X = require("M")` and missed the framework's `var X = lazyRequire(function () { return require("M"); })` pattern (used to break load cycles). 34 additional inconsistencies surfaced (`auditFwk` / `auditMod` / `auditModule` / `lazyAudit` → `audit`, `crypto` / `fwCrypto` → `bCrypto`, `dbMod` / `dbModule` → `db`, etc.) — every minority site renamed per the same canonical-name map. (3) **dbStore seal round-trip test** — the v0.9.15 test suite covered seal-falls-back-when-vault-not-ready and cross-process-sealed-row-preserved, but did NOT exercise the actual default-ON seal/unseal path because the test environment didn't `b.vault.init(...)`. New `testDbStoreSealRoundTripWithVault` bootstraps a plaintext vault, builds a dbStore with `seal: true`, writes a record + reads it back, and asserts (a) `headers` + `body` columns carry the `vault:` envelope on disk, (b) the round-trip restores the original values, and (c) `status_code` stays plaintext so forensic SELECTs still work without unsealing.

package/index.js

@@ -86,6 +86,8 @@ var session = require("./lib/session");
 var storage = require("./lib/storage");
 var safeJson = require("./lib/safe-json");
 var safeJsonPath = require("./lib/safe-jsonpath");
+var safeMime = require("./lib/safe-mime");
+var mailStore = require("./lib/mail-store");
 var ntpCheck = require("./lib/ntp-check");
 var auditSign = require("./lib/audit-sign");
 var objectStore = require("./lib/object-store");
@@ -157,6 +159,7 @@ var guardCsv = require("./lib/guard-csv");
 var guardHtml = require("./lib/guard-html");
 var guardSvg = require("./lib/guard-svg");
 var guardFilename = require("./lib/guard-filename");
+var guardMessageId = require("./lib/guard-message-id");
 var guardArchive = require("./lib/guard-archive");
 var guardJson = require("./lib/guard-json");
 var guardYaml = require("./lib/guard-yaml");
@@ -394,6 +397,7 @@ module.exports = {
   guardHtml:        guardHtml,
   guardSvg:         guardSvg,
   guardFilename:    guardFilename,
+  guardMessageId:   guardMessageId,
   guardArchive:     guardArchive,
   guardJson:        guardJson,
   guardYaml:        guardYaml,
@@ -473,6 +477,8 @@ module.exports = {
   flag:             flag,
   safeJson:         safeJson,
   safeJsonPath:     safeJsonPath,
+  safeMime:         safeMime,
+  mailStore:        mailStore,
   safeSchema:       safeSchema,
   pagination:       pagination,
   metrics:          metrics,

package/lib/guard-message-id.js

@@ -0,0 +1,241 @@
+"use strict";
+/**
+ * @module     b.guardMessageId
+ * @nav        Guards
+ * @title      Guard Message-Id
+ * @order      420
+ *
+ * @intro
+ *   RFC 5322 §3.6.4 Message-Id validator. Gates Message-Id /
+ *   In-Reply-To / References header values at the entry to
+ *   `b.mailStore.appendMessage` (v0.9.19), `b.mail.server.mx` (v0.9.23),
+ *   and the outbound submission path (v0.9.25).
+ *
+ *   Refuses:
+ *
+ *     - oversized (default 998-byte cap per RFC 5322 §2.1.1 line cap)
+ *     - bare CR / LF / NUL / C0 control chars (header-injection
+ *       defense — defends `From:` / `Bcc:` smuggling via folded
+ *       Message-Id continuation)
+ *     - DEL (0x7F) anywhere
+ *     - unbracketed under `strict` profile (the wire form per RFC
+ *       5322 §3.6.4 is `<unique-token@domain>` — operator with
+ *       legacy mail can opt down to `balanced` to accept bare tokens)
+ *     - empty value
+ *     - bidi codepoints in the local-part / domain (RFC 5322 + EAI
+ *       allow non-ASCII per RFC 6532 + RFC 5335 but bidi-marker
+ *       codepoints are operator-unfriendly and refused outright)
+ *
+ *   Profile vocabulary follows the existing guard-family convention:
+ *
+ *     - `strict` (default) — bracketed `<token@domain>`, length cap,
+ *       no control chars, no bidi
+ *     - `balanced` — accepts unbracketed tokens (legacy mail compat)
+ *     - `permissive` — minimal validation (NUL + CR/LF refused; rest
+ *       passes); use only for forensic-only flows
+ *
+ *   Posture vocabulary:
+ *
+ *     - `hipaa` / `pci-dss` / `gdpr` / `soc2` — each pins the
+ *       active profile to `strict` regardless of operator's profile
+ *       opt; refuses to relax under regulated postures.
+ *
+ *   Composes the framework's existing guard-family pattern via
+ *   `b.gateContract` (the same shape `b.guardEmail` / `b.guardCsv` /
+ *   `b.guardArchive` use). Registers in `b.guardAll`'s
+ *   `STANDALONE_GUARDS` map.
+ *
+ * @card
+ *   RFC 5322 §3.6.4 Message-Id validator — bounded length, no CRLF/NUL/control chars, bracketed shape under strict profile. Gates header-injection at the mail-store / MX / submission entry points.
+ */
+
+var { defineClass } = require("./framework-error");
+
+var GuardMessageIdError = defineClass("GuardMessageIdError", { alwaysPermanent: true });
+
+var DEFAULT_PROFILE = "strict";
+
+var PROFILES = Object.freeze({
+  strict:     { requireBrackets: true,  maxBytes: 998 },                                          // allow:raw-byte-literal
+  balanced:   { requireBrackets: false, maxBytes: 998 },                                          // allow:raw-byte-literal
+  permissive: { requireBrackets: false, maxBytes: 4096 },                                         // allow:raw-byte-literal — permissive cap, not bytes-as-storage
+});
+
+var COMPLIANCE_POSTURES = Object.freeze({
+  hipaa:     "strict",
+  "pci-dss": "strict",
+  gdpr:      "strict",
+  soc2:      "strict",
+});
+
+// Bidi codepoints refused — same set the framework's address-bidi
+// defense uses (RFC 5322 §3.6.4 doesn't speak EAI codepoints, but RTL
+// codepoints in Message-Ids are operator-unfriendly + defend the
+// CVE-2021-42574 RTLO class in mail header context).
+var BIDI_RE = /[؜‎‏‪-‮⁦-⁩]/;
+
+/**
+ * @primitive b.guardMessageId.validate
+ * @signature b.guardMessageId.validate(value, opts?)
+ * @since     0.9.19
+ * @status    stable
+ * @related   b.guardMessageId.validateList, b.safeMime.parse, b.guardEmail
+ *
+ * Validate a Message-Id / In-Reply-To / References header value.
+ * Returns the input value on success; throws `GuardMessageIdError`
+ * on refusal.
+ *
+ * @opts
+ *   profile:    "strict" | "balanced" | "permissive",     // default "strict"
+ *   posture:    "hipaa" | "pci-dss" | "gdpr" | "soc2",    // pins profile to strict
+ *   maxBytes:   number,                                    // per-profile default
+ *
+ * @example
+ *   b.guardMessageId.validate("<abc@example.com>");
+ *   // → "<abc@example.com>"
+ *
+ *   try { b.guardMessageId.validate("abc@example.com"); }
+ *   catch (e) { e.code; }
+ *   // → "message-id/unbracketed" (strict profile)
+ */
+function validate(value, opts) {
+  opts = opts || {};
+  var profileName = _resolveProfile(opts);
+  var profile = PROFILES[profileName];
+  var maxBytes = typeof opts.maxBytes === "number" ? opts.maxBytes : profile.maxBytes;
+
+  if (typeof value !== "string") {
+    throw new GuardMessageIdError("message-id/bad-input",
+      "guardMessageId.validate: value must be a string (got " + typeof value + ")");
+  }
+  if (value.length === 0) {
+    throw new GuardMessageIdError("message-id/empty",
+      "guardMessageId.validate: empty Message-Id refused");
+  }
+  if (Buffer.byteLength(value, "utf8") > maxBytes) {
+    throw new GuardMessageIdError("message-id/oversize",
+      "guardMessageId.validate: " + Buffer.byteLength(value, "utf8") +
+      " bytes exceeds maxBytes=" + maxBytes + " (RFC 5322 §2.1.1)");
+  }
+
+  // C0 control chars + NUL + DEL — always refused at every profile
+  // (defends mail-header-injection class — operator can't smuggle
+  // CR/LF into a Message-Id to fold an attacker-chosen From: line).
+  for (var i = 0; i < value.length; i += 1) {
+    var c = value.charCodeAt(i);
+    if (c < 0x20 || c === 0x7F) {                                                                 // allow:raw-byte-literal — C0 + DEL refusal
+      throw new GuardMessageIdError("message-id/control-char",
+        "guardMessageId.validate: control char 0x" + c.toString(16) + " at offset " + i);
+    }
+  }
+
+  // Bidi codepoints — refused at strict + balanced; permissive lets
+  // them through. Length-bounded by the maxBytes check above so a
+  // hostile input can't burn regex-engine CPU; the bidi codepoint set
+  // is tiny so the test is constant-time anyway.
+  if (profileName !== "permissive" && BIDI_RE.test(value)) {                                       // allow:regex-no-length-cap — value length-bounded by Buffer.byteLength check above
+    throw new GuardMessageIdError("message-id/bidi",
+      "guardMessageId.validate: bidi codepoint refused (CVE-2021-42574 RTLO class in mail-header context)");
+  }
+
+  // Bracketed shape — required under strict.
+  if (profile.requireBrackets) {
+    if (value.charAt(0) !== "<" || value.charAt(value.length - 1) !== ">") {
+      throw new GuardMessageIdError("message-id/unbracketed",
+        "guardMessageId.validate: strict profile requires `<token@domain>` shape (RFC 5322 §3.6.4)");
+    }
+    var inner = value.slice(1, -1);
+    var at = inner.indexOf("@");
+    if (at <= 0 || at === inner.length - 1) {
+      throw new GuardMessageIdError("message-id/no-at",
+        "guardMessageId.validate: Message-Id must contain `@` between local-part and domain");
+    }
+    if (inner.indexOf("<") >= 0 || inner.indexOf(">") >= 0) {
+      throw new GuardMessageIdError("message-id/nested-brackets",
+        "guardMessageId.validate: nested angle brackets refused");
+    }
+  }
+
+  return value;
+}
+
+/**
+ * @primitive b.guardMessageId.validateList
+ * @signature b.guardMessageId.validateList(value, opts?)
+ * @since     0.9.19
+ * @status    stable
+ * @related   b.guardMessageId.validate
+ *
+ * Validate a Message-Id-list header value (References / In-Reply-To
+ * may carry multiple ids separated by whitespace per RFC 5322 §3.6.4).
+ * Returns the array of validated Message-Ids; throws on any single
+ * refusal.
+ *
+ * @opts
+ *   profile:        same as validate
+ *   posture:        same as validate
+ *   maxBytes:       per-id cap
+ *   maxIds:         number,    // default 100 — References-chain cap
+ *
+ * @example
+ *   b.guardMessageId.validateList("<a@x> <b@x> <c@x>");
+ *   // → ["<a@x>", "<b@x>", "<c@x>"]
+ */
+function validateList(value, opts) {
+  opts = opts || {};
+  var maxIds = typeof opts.maxIds === "number" ? opts.maxIds : 100;                              // allow:raw-byte-literal — References-chain cap, not bytes
+  if (typeof value !== "string") {
+    throw new GuardMessageIdError("message-id/bad-input",
+      "guardMessageId.validateList: value must be a string");
+  }
+  var ids = value.split(/\s+/).filter(function (s) { return s.length > 0; });
+  if (ids.length > maxIds) {
+    throw new GuardMessageIdError("message-id/chain-too-long",
+      "guardMessageId.validateList: " + ids.length + " ids exceeds maxIds=" + maxIds);
+  }
+  for (var i = 0; i < ids.length; i += 1) {
+    validate(ids[i], opts);
+  }
+  return ids;
+}
+
+/**
+ * @primitive b.guardMessageId.compliancePosture
+ * @signature b.guardMessageId.compliancePosture(posture)
+ * @since     0.9.19
+ * @status    stable
+ *
+ * Return the effective profile for a given compliance posture.
+ * Composed by `b.compliance.set` to surface "what posture is active
+ * for which guard" in audit rows.
+ *
+ * @example
+ *   b.guardMessageId.compliancePosture("hipaa");      // → "strict"
+ *   b.guardMessageId.compliancePosture("unknown");    // → null
+ */
+function compliancePosture(posture) {
+  return COMPLIANCE_POSTURES[posture] || null;
+}
+
+function _resolveProfile(opts) {
+  if (opts.posture && COMPLIANCE_POSTURES[opts.posture]) {
+    return COMPLIANCE_POSTURES[opts.posture];
+  }
+  var p = opts.profile || DEFAULT_PROFILE;
+  if (!PROFILES[p]) {
+    throw new GuardMessageIdError("message-id/bad-profile",
+      "guardMessageId: unknown profile '" + p + "' (use strict / balanced / permissive)");
+  }
+  return p;
+}
+
+module.exports = {
+  validate:           validate,
+  validateList:       validateList,
+  compliancePosture:  compliancePosture,
+  PROFILES:           PROFILES,
+  COMPLIANCE_POSTURES: COMPLIANCE_POSTURES,
+  GuardMessageIdError: GuardMessageIdError,
+  NAME:               "messageId",
+  KIND:               "identifier",
+};