@odla-ai/chapter 0.42.0 → 0.42.1

package/dist/worker/index.d.cts

@@ -185,6 +185,13 @@ interface ChapterNetworkTarget {
      * this follower. Omitted by default: a target never gains write permission
      * merely because its records are browsable. */
     sharedNotes?: readonly string[];
+    /** Per-record-type upper bound on profile fields this leader permits its
+     * admins to edit remotely. The follower must independently delegate the
+     * same field; readable fields never become writable implicitly. */
+    editableFields?: Record<string, readonly string[]>;
+    /** Record types on which this leader permits remote lifecycle transitions.
+     * The follower must independently opt in and remains pipeline authority. */
+    stageTransitions?: readonly string[];
 }
 /** One signed peer allowed to browse a privacy-bounded projection of this
  * site's CRM. The follower owns this allowlist; a leader target declaration
@@ -199,6 +206,13 @@ interface ChapterNetworkReader {
      * Omitted by default: browse permission never silently grants write access.
      * Shared notes land in the follower CRM activity feed. */
     sharedNotes?: readonly string[];
+    /** Exact local CRM fields this reader may edit remotely, by record type.
+     * Every field must also be readable so the peer can perform an optimistic
+     * edit against the value and revision it actually reviewed. */
+    editableFields?: Record<string, readonly string[]>;
+    /** Local CRM record types on which this reader may request a lifecycle
+     * transition. Chapter still enforces the local pipeline and gates. */
+    stageTransitions?: readonly string[];
 }
 /** Leader/follower network configuration. A follower needs no code config to
  * receive pushed records. It must explicitly declare `readers` before any peer
@@ -241,12 +255,16 @@ interface ResolvedNetworkTarget {
     secretName: string;
     fields?: Record<string, readonly string[]>;
     sharedNotes: readonly string[];
+    editableFields: Record<string, readonly string[]>;
+    stageTransitions: readonly string[];
 }
 /** A validated follower-owned record projection for one signed reader. */
 interface ResolvedNetworkReader {
     id: string;
     fields: Record<string, readonly string[]>;
     sharedNotes: readonly string[];
+    editableFields: Record<string, readonly string[]>;
+    stageTransitions: readonly string[];
 }
 /** Fully-resolved leader network configuration. */
 interface ResolvedNetwork {

package/dist/worker/index.d.ts

@@ -185,6 +185,13 @@ interface ChapterNetworkTarget {
      * this follower. Omitted by default: a target never gains write permission
      * merely because its records are browsable. */
     sharedNotes?: readonly string[];
+    /** Per-record-type upper bound on profile fields this leader permits its
+     * admins to edit remotely. The follower must independently delegate the
+     * same field; readable fields never become writable implicitly. */
+    editableFields?: Record<string, readonly string[]>;
+    /** Record types on which this leader permits remote lifecycle transitions.
+     * The follower must independently opt in and remains pipeline authority. */
+    stageTransitions?: readonly string[];
 }
 /** One signed peer allowed to browse a privacy-bounded projection of this
  * site's CRM. The follower owns this allowlist; a leader target declaration
@@ -199,6 +206,13 @@ interface ChapterNetworkReader {
      * Omitted by default: browse permission never silently grants write access.
      * Shared notes land in the follower CRM activity feed. */
     sharedNotes?: readonly string[];
+    /** Exact local CRM fields this reader may edit remotely, by record type.
+     * Every field must also be readable so the peer can perform an optimistic
+     * edit against the value and revision it actually reviewed. */
+    editableFields?: Record<string, readonly string[]>;
+    /** Local CRM record types on which this reader may request a lifecycle
+     * transition. Chapter still enforces the local pipeline and gates. */
+    stageTransitions?: readonly string[];
 }
 /** Leader/follower network configuration. A follower needs no code config to
  * receive pushed records. It must explicitly declare `readers` before any peer
@@ -241,12 +255,16 @@ interface ResolvedNetworkTarget {
     secretName: string;
     fields?: Record<string, readonly string[]>;
     sharedNotes: readonly string[];
+    editableFields: Record<string, readonly string[]>;
+    stageTransitions: readonly string[];
 }
 /** A validated follower-owned record projection for one signed reader. */
 interface ResolvedNetworkReader {
     id: string;
     fields: Record<string, readonly string[]>;
     sharedNotes: readonly string[];
+    editableFields: Record<string, readonly string[]>;
+    stageTransitions: readonly string[];
 }
 /** Fully-resolved leader network configuration. */
 interface ResolvedNetwork {

package/dist/worker/index.js

@@ -5348,12 +5348,22 @@ var handleAdminNetworkTargets = async (req, url, env, ctx) => {
   const got = await gate5(req, env, ctx);
   if ("response" in got) return got.response;
   return json({
-    targets: ctx.chapter.network.targets.map(({ id, name, url: targetUrl, fields, sharedNotes }) => ({
+    targets: ctx.chapter.network.targets.map(({
+      id,
+      name,
+      url: targetUrl,
+      fields,
+      sharedNotes,
+      editableFields,
+      stageTransitions
+    }) => ({
       id,
       name,
       url: targetUrl,
       types: Object.keys(fields ?? DEFAULT_SHARE_FIELDS),
-      sharedNotes
+      sharedNotes,
+      editableFields,
+      stageTransitions
     }))
   });
 };
@@ -5471,19 +5481,20 @@ var handleAdminNetworkPush = async (req, url, env, ctx) => {
   return json({ ok: results.every((result) => result.ok), results });
 };
 
-// src/worker-routes-network-records.ts
-import { listRecords } from "@odla-ai/crm";
-import { signFederatedRequest as signFederatedRequest3, verifyFederatedRequest as verifyFederatedRequest6 } from "@odla-ai/db";
+// src/worker-routes-network-records-follower.ts
+import { CRM_NS, CrmInputError, getRecord as getRecord2, listRecords, setStage as setStage2, updateRecord as updateRecord3 } from "@odla-ai/crm";
+import { verifyFederatedRequest as verifyFederatedRequest6 } from "@odla-ai/db";
+
+// src/worker-routes-network-records-common.ts
 var IDENTIFIER = /^[a-z][a-zA-Z0-9_]*$/;
 var MAX_PAGE = 50;
 var MAX_OFFSET = 1e4;
 var MAX_SEARCH = 200;
-async function gate6(req, env, ctx) {
-  const db = ctx.makeDb(env);
-  const user = await ctx.verifyUser(req, env);
-  if (!user) return { response: json({ error: "unauthorized" }, 401) };
-  if (!await ctx.isAdmin(db, user)) return { response: json({ error: "forbidden" }, 403) };
-  return { db, user };
+var MAX_BODY = 32768;
+var RECORD_ID = /^[a-zA-Z0-9][a-zA-Z0-9:_-]{0,191}$/;
+var MUTATION_ID = /^[a-zA-Z0-9][a-zA-Z0-9:._-]{7,127}$/;
+function bodyTooLarge(body) {
+  return new TextEncoder().encode(body).byteLength > MAX_BODY;
 }
 function integer(value, fallback, min, max) {
   if (value == null || value === "") return fallback;
@@ -5491,10 +5502,22 @@ function integer(value, fallback, min, max) {
   const parsed = Number(value);
   return Number.isSafeInteger(parsed) && parsed >= min && parsed <= max ? parsed : null;
 }
-function projectRecord(record4, allowed2) {
+function projectRecord(record4, allowed2, editable, stagesEnabled, crm) {
   const fields = Object.fromEntries(
     allowed2.flatMap((name) => Object.prototype.hasOwnProperty.call(record4.fields, name) ? [[name, record4.fields[name]]] : [])
   );
+  const def = crm.type(record4.type);
+  const editFields = editable.map((id) => {
+    const field = def.fields[id];
+    return {
+      id,
+      label: field.label ?? id,
+      type: field.type,
+      required: field.required ?? false,
+      ...field.options ? { options: field.options } : {}
+    };
+  });
+  const allowedStages = stagesEnabled ? (def.pipeline?.stages ?? []).flatMap((stage) => crm.canTransition(record4.type, record4, stage.id) ? [] : [{ id: stage.id, label: stage.label ?? stage.id }]) : [];
   return {
     id: record4.id,
     type: record4.type,
@@ -5502,11 +5525,66 @@ function projectRecord(record4, allowed2) {
     ...record4.stage ? { stage: record4.stage } : {},
     fields,
     createdAt: record4.createdAt,
-    updatedAt: record4.updatedAt
+    updatedAt: record4.updatedAt,
+    revision: record4.updatedAt,
+    ...editFields.length || stagesEnabled ? { edit: { fields: editFields, allowedStages } } : {}
   };
 }
+function readerRecord(record4, reader, crm) {
+  return projectRecord(
+    record4,
+    reader.fields[record4.type] ?? [],
+    reader.editableFields[record4.type] ?? [],
+    reader.stageTransitions.includes(record4.type),
+    crm
+  );
+}
+function restrictedRecord(record4, target) {
+  if (!record4.edit) return record4;
+  const fields = record4.edit.fields.filter((field) => target.editableFields[record4.type]?.includes(field.id));
+  const allowedStages = target.stageTransitions.includes(record4.type) ? record4.edit.allowedStages : [];
+  const edit = fields.length || target.stageTransitions.includes(record4.type) ? { fields, allowedStages } : void 0;
+  return { ...record4, ...edit ? { edit } : { edit: void 0 } };
+}
+
+// src/worker-routes-network-records-follower.ts
+var NETWORK_MUTATION_META = "chapterNetworkMutation";
+function mutationFingerprint(input) {
+  const fields = input.fields ? Object.fromEntries(Object.entries(input.fields).sort(([a], [b]) => a.localeCompare(b))) : void 0;
+  return JSON.stringify({
+    recordId: input.recordId,
+    type: input.type,
+    expectedRevision: input.expectedRevision,
+    expectedStage: input.expectedStage,
+    mutationId: input.mutationId,
+    fields,
+    stage: input.stage,
+    actor: input.actor
+  });
+}
+function mutationError(error) {
+  if (error?.code === "transact_guard_failed") {
+    return json({ error: "record changed since it was reviewed", code: "stale_revision" }, 409);
+  }
+  if (error instanceof CrmInputError) {
+    return json({ error: error.message, code: "invalid_transition_or_fields", ...error.fields ? { fields: error.fields } : {} }, 422);
+  }
+  return json({ error: "follower mutation failed", code: "mutation_failed" }, 500);
+}
+async function priorMutation(db, id) {
+  const result = await db.query({
+    [CRM_NS.activity]: { $: { where: { id }, limit: 1 } }
+  });
+  const row = (result[CRM_NS.activity] ?? [])[0];
+  const meta = row?.meta;
+  const receipt = meta?.[NETWORK_MUTATION_META];
+  return typeof receipt?.fingerprint === "string" ? { fingerprint: receipt.fingerprint } : null;
+}
 var handleNetworkRecords = async (req, url, env, ctx) => {
-  if (req.method !== "GET" || url.pathname !== "/api/network/records") return null;
+  if (url.pathname !== "/api/network/records" || req.method !== "GET" && req.method !== "PATCH") return null;
+  if (req.method === "PATCH" && Number(req.headers.get("content-length") ?? 0) > MAX_BODY) {
+    return json({ error: "mutation body is too large", code: "body_too_large" }, 413);
+  }
   const db = ctx.makeDb(env);
   const secret = await getVaultSecret(db, "network_share_secret");
   const readers = ctx.chapter.network.readers;
@@ -5517,8 +5595,105 @@ var handleNetworkRecords = async (req, url, env, ctx) => {
   });
   if (!verified.ok) return json({ error: "unauthorized", reason: verified.reason }, 401);
   const reader = readers.find((candidate) => candidate.id === verified.sender);
+  if (!reader) return json({ error: "unauthorized" }, 401);
+  if (req.method === "PATCH") {
+    if (bodyTooLarge(verified.body)) {
+      return json({ error: "mutation body is too large", code: "body_too_large" }, 413);
+    }
+    let input;
+    try {
+      input = JSON.parse(verified.body);
+    } catch {
+      return json({ error: "invalid JSON body", code: "invalid_json" }, 422);
+    }
+    const fields = input?.fields;
+    const hasFields = Boolean(fields && typeof fields === "object" && !Array.isArray(fields));
+    const hasStage = typeof input?.stage === "string";
+    const actor = input?.actor;
+    if (!input || !IDENTIFIER.test(input.type) || !RECORD_ID.test(input.recordId) || !Number.isSafeInteger(input.expectedRevision) || input.expectedRevision < 0 || !MUTATION_ID.test(input.mutationId) || !actor || typeof actor.id !== "string" || !RECORD_ID.test(actor.id) || hasFields === hasStage) {
+      return json({ error: "invalid network mutation", code: "invalid_mutation" }, 422);
+    }
+    const editable = reader.editableFields[input.type];
+    if (hasFields) {
+      const names = Object.keys(fields);
+      if (!editable || names.length === 0 || names.length > 32 || names.some((name) => !editable.includes(name))) {
+        return json({ error: "profile fields are not delegated for this reader", code: "field_forbidden" }, 403);
+      }
+    } else if (!reader.stageTransitions.includes(input.type)) {
+      return json({ error: "stage transitions are not delegated for this reader", code: "stage_forbidden" }, 403);
+    }
+    const activityId = `network-edit:${verified.sender}:${input.mutationId}`;
+    const fingerprint = mutationFingerprint(input);
+    const prior = await priorMutation(db, activityId);
+    if (prior) {
+      if (prior.fingerprint !== fingerprint) {
+        return json({ error: "mutationId was already used for different input", code: "mutation_conflict" }, 409);
+      }
+      const replayed = await getRecord2({ crm: ctx.chapter.crm, db }, input.recordId);
+      if (!replayed || replayed.type !== input.type) return json({ error: "record not found", code: "not_found" }, 404);
+      const result = {
+        version: 1,
+        site: { id: ctx.chapter.id, name: ctx.chapter.name },
+        record: readerRecord(replayed, reader, ctx.chapter.crm),
+        duplicate: true
+      };
+      return json(result);
+    }
+    const record4 = await getRecord2({ crm: ctx.chapter.crm, db }, input.recordId);
+    if (!record4 || record4.type !== input.type) return json({ error: "record not found", code: "not_found" }, 404);
+    if (record4.updatedAt !== input.expectedRevision) {
+      return json({ error: "record changed since it was reviewed", code: "stale_revision" }, 409);
+    }
+    if (hasStage && (input.expectedStage ?? null) !== (record4.stage ?? null)) {
+      return json({ error: "stage changed since it was reviewed", code: "stale_stage" }, 409);
+    }
+    const actorEmail = typeof actor.email === "string" && actor.email.length <= 320 ? actor.email : void 0;
+    const auditMeta = {
+      [NETWORK_MUTATION_META]: {
+        version: 1,
+        sourceId: verified.sender,
+        actorId: actor.id,
+        ...actorEmail ? { actorEmail } : {},
+        mutationId: input.mutationId,
+        fingerprint
+      }
+    };
+    try {
+      const deps = { crm: ctx.chapter.crm, db };
+      const updated = hasFields ? await updateRecord3(deps, {
+        id: record4.id,
+        input: fields,
+        mutationId: activityId,
+        expectedUpdatedAt: input.expectedRevision,
+        audit: {
+          id: activityId,
+          body: `Profile edited remotely by ${verified.sender}`,
+          authorId: `network:${verified.sender}`,
+          meta: auditMeta
+        }
+      }) : await setStage2(deps, {
+        id: record4.id,
+        to: input.stage,
+        authorId: `network:${verified.sender}`,
+        mutationId: activityId,
+        expectedUpdatedAt: input.expectedRevision,
+        expectedStage: input.expectedStage ?? null,
+        activityId,
+        activityMeta: auditMeta
+      });
+      const result = {
+        version: 1,
+        site: { id: ctx.chapter.id, name: ctx.chapter.name },
+        record: readerRecord(updated.record, reader, ctx.chapter.crm),
+        duplicate: updated.duplicate
+      };
+      return json(result);
+    } catch (error) {
+      return mutationError(error);
+    }
+  }
   const type = url.searchParams.get("type") ?? "";
-  const allowed2 = reader?.fields[type];
+  const allowed2 = reader.fields[type];
   if (!IDENTIFIER.test(type) || !allowed2) return json({ error: "record type is not shared with this reader" }, 403);
   const limit = integer(url.searchParams.get("limit"), 25, 1, MAX_PAGE);
   const offset = integer(url.searchParams.get("offset"), 0, 0, MAX_OFFSET);
@@ -5540,13 +5715,23 @@ var handleNetworkRecords = async (req, url, env, ctx) => {
     version: 1,
     site: { id: ctx.chapter.id, name: ctx.chapter.name },
     type,
-    records: page.records.map((record4) => projectRecord(record4, allowed2)),
+    records: page.records.map((record4) => readerRecord(record4, reader, ctx.chapter.crm)),
     total: page.total,
     limit: page.limit,
     offset: page.offset
   };
   return json(response);
 };
+
+// src/worker-routes-network-records-admin.ts
+import { signFederatedRequest as signFederatedRequest3 } from "@odla-ai/db";
+async function gate6(req, env, ctx) {
+  const db = ctx.makeDb(env);
+  const user = await ctx.verifyUser(req, env);
+  if (!user) return { response: json({ error: "unauthorized" }, 401) };
+  if (!await ctx.isAdmin(db, user)) return { response: json({ error: "forbidden" }, 403) };
+  return { db, user };
+}
 function targetFor(ctx, id) {
   return ctx.chapter.network.targets.find((target) => target.id === id);
 }
@@ -5556,12 +5741,7 @@ async function readTargetRecords(db, env, ctx, target, query) {
   const destination = new URL("/api/network/records", target.url);
   destination.search = query.toString();
   try {
-    const headers = await signFederatedRequest3({
-      secret,
-      sender: ctx.chapter.id,
-      method: "GET",
-      url: destination
-    });
+    const headers = await signFederatedRequest3({ secret, sender: ctx.chapter.id, method: "GET", url: destination });
     const response = await fetchNetworkTarget(env, target, destination, {
       headers,
       signal: AbortSignal.timeout(1e4)
@@ -5574,15 +5754,90 @@ async function readTargetRecords(db, env, ctx, target, query) {
         error: upstream ?? (response.ok ? "invalid follower record response" : `follower returned ${response.status}`)
       };
     }
-    return { ok: true, page: body };
+    return {
+      ok: true,
+      page: { ...body, records: body.records.map((record4) => restrictedRecord(record4, target)) }
+    };
   } catch (error) {
     return { ok: false, error: error instanceof Error ? error.message : "record read failed" };
   }
 }
+async function mutateTargetRecord(db, env, ctx, target, input) {
+  const fields = input.fields;
+  if (fields) {
+    const allowed2 = target.editableFields[input.type];
+    if (!allowed2 || Object.keys(fields).some((name) => !allowed2.includes(name))) {
+      return json({ error: "profile fields are not enabled for this target", code: "field_forbidden" }, 403);
+    }
+  } else if (!target.stageTransitions.includes(input.type)) {
+    return json({ error: "stage transitions are not enabled for this target", code: "stage_forbidden" }, 403);
+  }
+  const secret = await getVaultSecret(db, target.secretName);
+  if (!secret) return json({ error: "edge secret is missing", code: "edge_unavailable" }, 502);
+  const destination = new URL("/api/network/records", target.url);
+  const body = JSON.stringify(input);
+  try {
+    const headers = await signFederatedRequest3({
+      secret,
+      sender: ctx.chapter.id,
+      method: "PATCH",
+      url: destination,
+      body
+    });
+    const response = await fetchNetworkTarget(env, target, destination, {
+      method: "PATCH",
+      headers: { ...headers, "content-type": "application/json" },
+      body,
+      signal: AbortSignal.timeout(1e4)
+    });
+    const payload = await response.json().catch(() => null);
+    const site = payload && "site" in payload ? payload.site : void 0;
+    if (!response.ok || !payload || !("record" in payload) || site && site.id !== target.id) {
+      const status = [401, 403, 404, 409, 422].includes(response.status) ? response.status : 502;
+      return json({
+        error: payload && "error" in payload && typeof payload.error === "string" ? payload.error : `follower returned ${response.status}`,
+        ...payload && "code" in payload && typeof payload.code === "string" ? { code: payload.code } : {}
+      }, status);
+    }
+    return json({ ...payload, record: restrictedRecord(payload.record, target) });
+  } catch {
+    return json({ error: "follower mutation request failed", code: "follower_unavailable" }, 502);
+  }
+}
 var handleAdminNetworkRecords = async (req, url, env, ctx) => {
-  if (req.method !== "GET" || url.pathname !== "/api/admin/network/records") return null;
+  if (url.pathname !== "/api/admin/network/records" || req.method !== "GET" && req.method !== "PATCH") return null;
+  if (req.method === "PATCH" && Number(req.headers.get("content-length") ?? 0) > MAX_BODY) {
+    return json({ error: "mutation body is too large", code: "body_too_large" }, 413);
+  }
   const got = await gate6(req, env, ctx);
   if ("response" in got) return got.response;
+  if (req.method === "PATCH") {
+    let input;
+    try {
+      const body = await req.text();
+      if (bodyTooLarge(body)) {
+        return json({ error: "mutation body is too large", code: "body_too_large" }, 413);
+      }
+      input = JSON.parse(body);
+    } catch {
+      return json({ error: "invalid JSON body", code: "invalid_json" }, 422);
+    }
+    const target2 = targetFor(ctx, input?.targetId);
+    const hasFields = Boolean(input?.fields && typeof input.fields === "object" && !Array.isArray(input.fields));
+    const hasStage = typeof input?.stage === "string";
+    if (!target2 || !IDENTIFIER.test(input.type) || !RECORD_ID.test(input.recordId) || !Number.isSafeInteger(input.expectedRevision) || input.expectedRevision < 0 || !MUTATION_ID.test(input.mutationId) || hasFields === hasStage) {
+      return json({ error: "invalid network mutation", code: "invalid_mutation" }, 422);
+    }
+    return mutateTargetRecord(got.db, env, ctx, target2, {
+      type: input.type,
+      recordId: input.recordId,
+      expectedRevision: input.expectedRevision,
+      mutationId: input.mutationId,
+      ...input.expectedStage !== void 0 ? { expectedStage: input.expectedStage } : {},
+      ...hasFields ? { fields: input.fields } : { stage: input.stage },
+      actor: { id: got.user.userId, ...got.user.email ? { email: got.user.email } : {} }
+    });
+  }
   const targetId = url.searchParams.get("targetId") ?? "";
   const type = url.searchParams.get("type") ?? "";
   const target = targetFor(ctx, targetId);
@@ -5595,19 +5850,13 @@ var handleAdminNetworkRecords = async (req, url, env, ctx) => {
   }
   const query = new URLSearchParams({ type, limit: String(limit), offset: String(offset) });
   if (search) query.set("search", search);
-  const result = await readTargetRecords(
-    got.db,
-    env,
-    ctx,
-    target,
-    query
-  );
+  const result = await readTargetRecords(got.db, env, ctx, target, query);
   return result.ok ? json(result.page) : json({ error: result.error }, 502);
 };
 
 // src/worker-routes-network-notes.ts
 var IDENTIFIER2 = /^[a-z][a-zA-Z0-9_]*$/;
-var RECORD_ID = /^[A-Za-z0-9][A-Za-z0-9_.:-]{0,255}$/;
+var RECORD_ID2 = /^[A-Za-z0-9][A-Za-z0-9_.:-]{0,255}$/;
 var MAX_NOTE2 = 4e3;
 async function gate7(req, env, ctx) {
   const db = ctx.makeDb(env);
@@ -5621,7 +5870,7 @@ function noteIdentity(url, ctx) {
   const recordType = url.searchParams.get("type") ?? "";
   const recordId = url.searchParams.get("recordId") ?? "";
   const configured = ctx.chapter.network.targets.some((target) => target.id === targetId);
-  if (!configured || !IDENTIFIER2.test(recordType) || !RECORD_ID.test(recordId)) return null;
+  if (!configured || !IDENTIFIER2.test(recordType) || !RECORD_ID2.test(recordId)) return null;
   return {
     targetId,
     recordType,
@@ -5692,13 +5941,13 @@ var handleAdminNetworkNotes = async (req, url, env, ctx) => {
 // src/worker-routes-network-shared-notes.ts
 import {
   addActivity,
-  CRM_NS,
-  getRecord as getRecord2
+  CRM_NS as CRM_NS2,
+  getRecord as getRecord3
 } from "@odla-ai/crm";
 import { signFederatedRequest as signFederatedRequest4, verifyFederatedRequest as verifyFederatedRequest7 } from "@odla-ai/db";
 var IDENTIFIER3 = /^[a-z][a-zA-Z0-9_]*$/;
-var RECORD_ID2 = /^[A-Za-z0-9][A-Za-z0-9_.:-]{0,255}$/;
-var MUTATION_ID = /^[A-Za-z0-9_-]{16,128}$/;
+var RECORD_ID3 = /^[A-Za-z0-9][A-Za-z0-9_.:-]{0,255}$/;
+var MUTATION_ID2 = /^[A-Za-z0-9_-]{16,128}$/;
 var MAX_NOTE3 = 4e3;
 var NETWORK_META = "chapterNetwork";
 async function gate8(req, env, ctx) {
@@ -5724,7 +5973,7 @@ function sharedNote(activity, sourceId) {
 }
 async function listSharedNotes(db, sourceId, recordId) {
   const result = await db.query({
-    [CRM_NS.activity]: {
+    [CRM_NS2.activity]: {
       $: {
         where: {
           and: [
@@ -5737,7 +5986,7 @@ async function listSharedNotes(db, sourceId, recordId) {
       }
     }
   });
-  return (result[CRM_NS.activity] ?? []).flatMap((activity) => {
+  return (result[CRM_NS2.activity] ?? []).flatMap((activity) => {
     const note = sharedNote(activity, sourceId);
     return note ? [note] : [];
   });
@@ -5762,10 +6011,10 @@ var handleNetworkSharedNotes = async (req, url, env, ctx) => {
   const reader = readers.find((candidate) => candidate.id === verified.sender);
   const recordType = url.searchParams.get("type") ?? "";
   const recordId = url.searchParams.get("recordId") ?? "";
-  if (!IDENTIFIER3.test(recordType) || !RECORD_ID2.test(recordId) || !reader?.sharedNotes.includes(recordType)) {
+  if (!IDENTIFIER3.test(recordType) || !RECORD_ID3.test(recordId) || !reader?.sharedNotes.includes(recordType)) {
     return json({ error: "shared notes are not enabled for this reader and record type" }, 403);
   }
-  const record4 = await getRecord2({ crm: ctx.chapter.crm, db }, recordId);
+  const record4 = await getRecord3({ crm: ctx.chapter.crm, db }, recordId);
   if (!record4 || record4.type !== recordType) return json({ error: "record not found" }, 404);
   if (req.method === "GET") {
     return json({
@@ -5782,7 +6031,7 @@ var handleNetworkSharedNotes = async (req, url, env, ctx) => {
   }
   const body = typeof input.body === "string" ? input.body.trim() : "";
   const mutationId = typeof input.mutationId === "string" ? input.mutationId : "";
-  if (!body || body.length > MAX_NOTE3 || !MUTATION_ID.test(mutationId)) {
+  if (!body || body.length > MAX_NOTE3 || !MUTATION_ID2.test(mutationId)) {
     return json({ error: `body must be 1\u2013${MAX_NOTE3} characters and mutationId must be valid` }, 400);
   }
   const createdAt = Date.now();
@@ -5831,7 +6080,7 @@ var handleAdminNetworkSharedNotes = async (req, url, env, ctx) => {
   const recordType = url.searchParams.get("type") ?? "";
   const recordId = url.searchParams.get("recordId") ?? "";
   const target = targetFor2(ctx, targetId);
-  if (!target || !target.sharedNotes.includes(recordType) || !IDENTIFIER3.test(recordType) || !RECORD_ID2.test(recordId)) {
+  if (!target || !target.sharedNotes.includes(recordType) || !IDENTIFIER3.test(recordType) || !RECORD_ID3.test(recordId)) {
     return json({ error: "configured targetId, type, and recordId are required" }, 400);
   }
   let body = "";