@promptowl/contextnest-community 1.20.0 → 1.21.1

package/dist/{chunk-GVUBZTI3.js → chunk-NH7L2S6S.js}

@@ -4,19 +4,19 @@ import {
   resolveNestWideRoles,
   resolveStewardsForNode,
   stewardCoverageForUser
-} from "./chunk-O5DYL7ZG.js";
+} from "./chunk-LL2JU5AA.js";
 import {
   grantCoversNode,
   listUserGrants,
   resolveNodeGrant
-} from "./chunk-3LUUJUWS.js";
+} from "./chunk-IVXVOXYN.js";
 import {
   createVersion,
   getApprovedVersion,
   getApprovedVersions,
   getCurrentVersion,
   setApprovedVersion
-} from "./chunk-KHMSDTJJ.js";
+} from "./chunk-FXYFLUE7.js";
 import {
   buildDocContext,
   buildTitleMap,
@@ -36,7 +36,7 @@ import {
   resolveNestPermission,
   sendEmailToRecipient,
   titleForNode
-} from "./chunk-7CPP6FIU.js";
+} from "./chunk-G4AL7ETF.js";
 import {
   ConflictError,
   NotFoundError,
@@ -45,7 +45,7 @@ import {
 import {
   config,
   getDb
-} from "./chunk-XCFKS65Y.js";
+} from "./chunk-SKZ7YYT4.js";
 
 // src/governance/review-service.ts
 import { v4 as uuid2 } from "uuid";
@@ -119,6 +119,20 @@ var safeJson = (s) => {
 };
 
 // src/workflow/flow-graph.ts
+async function nodeTrail(db, nestId, ids) {
+  const unique = [...new Set(ids)];
+  const titles = /* @__PURE__ */ new Map();
+  try {
+    const rows = await db.all(
+      `SELECT node_id, title FROM node_index
+        WHERE nest_id = ? AND node_id IN (${unique.map(() => "?").join(",")})`,
+      [nestId, ...unique]
+    );
+    for (const r of rows) if (r.title) titles.set(r.node_id, r.title);
+  } catch {
+  }
+  return ids.map((id) => `"${titles.get(id) ?? id}"`).join(" \u2192 ");
+}
 async function assertFlowGraphAcyclic(nestId, extraFlowTypeId) {
   const db = getDb();
   const rows = await db.all(
@@ -133,7 +147,7 @@ async function assertFlowGraphAcyclic(nestId, extraFlowTypeId) {
   for (const r of rows) {
     if (r.from_node === r.to_node) {
       throw new ConflictError(
-        `cycle: ["${r.from_node}"] \u2014 a flow edge cannot be a self-loop`
+        `cycle: ${await nodeTrail(db, nestId, [r.from_node])} cannot flow into itself`
       );
     }
     nodes.add(r.from_node);
@@ -698,31 +712,60 @@ function withNodeWriteLock(key, fn) {
 // src/governance/review-service.ts
 async function submitForReview(params) {
   const db = getDb();
-  const existing = await db.get(
-    "SELECT id FROM review_requests WHERE nest_id = ? AND node_id = ? AND status = 'pending'",
-    [params.nestId, params.nodeId]
-  );
-  if (existing) {
-    throw new Error("A review is already pending for this node");
-  }
-  const id = uuid2();
-  await db.run(
-    `INSERT INTO review_requests
+  const { id, version } = await withNodeWriteLock(
+    nodeWriteKey(params.nestId, params.nodeId),
+    async () => {
+      const existing = await db.get(
+        "SELECT id FROM review_requests WHERE nest_id = ? AND node_id = ? AND status = 'pending'",
+        [params.nestId, params.nodeId]
+      );
+      if (existing) {
+        throw new Error("A review is already pending for this node");
+      }
+      const version2 = await getCurrentVersion(params.nestId, params.nodeId) || params.version;
+      const id2 = uuid2();
+      await db.run(
+        `INSERT INTO review_requests
      (id, nest_id, node_id, version, requested_by, request_note, priority)
      VALUES (?, ?, ?, ?, ?, ?, ?)`,
-    [
-      id,
-      params.nestId,
-      params.nodeId,
-      params.version,
-      params.requestedBy,
-      params.note || null,
-      params.priority || "normal"
-    ]
-  );
-  await db.run(
-    "UPDATE node_versions SET status = 'pending_review' WHERE nest_id = ? AND node_id = ? AND version = ?",
-    [params.nestId, params.nodeId, params.version]
+        [
+          id2,
+          params.nestId,
+          params.nodeId,
+          version2,
+          params.requestedBy,
+          params.note || null,
+          params.priority || "normal"
+        ]
+      );
+      await db.run(
+        "UPDATE node_versions SET status = 'pending_review' WHERE nest_id = ? AND node_id = ? AND version = ?",
+        [params.nestId, params.nodeId, version2]
+      );
+      try {
+        const { storage, versions: versionManager } = await engineCache.get(
+          params.nestId
+        );
+        const node = await storage.readDocument(params.nodeId);
+        const history = await versionManager.getHistory(params.nodeId);
+        const alreadySealed = (history?.versions ?? []).some(
+          (v) => v.version === (node.frontmatter.version ?? 1)
+        );
+        if (!alreadySealed) {
+          await versionManager.createVersion(node, params.requestedBy, {
+            note: params.note || "Submitted for review"
+          });
+        }
+      } catch (err) {
+        console.error(
+          "VersionManager.createVersion failed (submit for review)",
+          params.nestId,
+          params.nodeId,
+          err
+        );
+      }
+      return { id: id2, version: version2 };
+    }
   );
   const reviewCtx = await buildDocContext(params.nestId, params.nodeId, params.baseUrl);
   void dispatchEvent({
@@ -730,18 +773,18 @@ async function submitForReview(params) {
     nestId: params.nestId,
     subjectId: params.nodeId,
     actor: params.requestedBy,
-    message: `Review requested on *${reviewCtx.label}* (v${params.version}) by ${params.requestedBy}${params.note ? ` \u2014 "${params.note}"` : ""}`
+    message: `Review requested on *${reviewCtx.label}* (v${version}) by ${params.requestedBy}${params.note ? ` \u2014 "${params.note}"` : ""}`
   });
   void notifyNestEvent(
     params.nestId,
-    `:memo: Review requested on *${reviewCtx.label}* (v${params.version}) by ${params.requestedBy}${params.note ? ` \u2014 "${params.note}"` : ""}`,
+    `:memo: Review requested on *${reviewCtx.label}* (v${version}) by ${params.requestedBy}${params.note ? ` \u2014 "${params.note}"` : ""}`,
     {
       status: "pending_review",
       docTitle: reviewCtx.docTitle,
       path: reviewCtx.path,
       link: reviewCtx.link,
       actor: params.requestedBy,
-      version: params.version,
+      version,
       note: params.note || null
     }
   );
@@ -765,7 +808,7 @@ async function submitForReview(params) {
         path: reviewCtx.path,
         link: reviewCtx.link,
         actor: params.requestedBy,
-        version: params.version,
+        version,
         note: params.note || null
       });
     }
@@ -1106,9 +1149,12 @@ async function getPendingReview(nestId, nodeId) {
   );
   return row ? rowToReviewRequest(row) : null;
 }
+function isNestTarget(t) {
+  return t === "nest" || t === "nest_archive";
+}
 async function deletionTargetTitle(request, titles) {
   if (request.targetType === "folder") return prettyFolderPath(request.nodeId);
-  if (request.targetType === "nest") return await nestName(request.nestId);
+  if (isNestTarget(request.targetType)) return await nestName(request.nestId);
   return titles?.get(request.nodeId) ?? await titleForNode(request.nestId, request.nodeId);
 }
 async function nestAdminEmails(nestId) {
@@ -1145,7 +1191,7 @@ async function deletionTargetContext(request, baseUrl) {
   const name = await nestName(request.nestId);
   const label = request.targetType === "folder" ? prettyFolderPath(request.nodeId) : name;
   return {
-    noun: request.targetType,
+    noun: request.targetType === "nest_archive" ? "nest" : request.targetType,
     label,
     docTitle: label,
     path: name,
@@ -1210,7 +1256,8 @@ async function notifyDeletionRequested(request, baseUrl) {
 async function notifyDeletionResolved(params) {
   try {
     const { outcome, resolvedBy, note } = params;
-    const verb = outcome === "deleted" ? "deleted" : "rejected the deletion of";
+    const archivedInstead = outcome === "deleted" && params.targetType === "nest_archive";
+    const verb = archivedInstead ? "archived" : outcome === "deleted" ? "deleted" : "rejected the deletion of";
     const line = `${resolvedBy} ${verb} *${params.docTitle}*${note ? ` \u2014 "${note}"` : ""}`;
     const recipient = params.requestedBy.toLowerCase();
     const canInbox = !(params.targetType === "nest" && outcome === "deleted");
@@ -1220,7 +1267,7 @@ async function notifyDeletionResolved(params) {
         [recipient],
         `deletion_${outcome}`,
         params.requestId,
-        outcome === "deleted" ? `"${params.docTitle}" was deleted by ${resolvedBy} \u2014 your request was accepted` : `Your deletion request for "${params.docTitle}" was rejected by ${resolvedBy}${note ? ` \u2014 "${note}"` : ""}`
+        outcome === "deleted" ? `"${params.docTitle}" was ${archivedInstead ? "archived" : "deleted"} by ${resolvedBy} \u2014 your request was accepted` : `Your deletion request for "${params.docTitle}" was rejected by ${resolvedBy}${note ? ` \u2014 "${note}"` : ""}`
       );
     }
     void dispatchEvent({
@@ -1240,7 +1287,10 @@ async function notifyDeletionResolved(params) {
     if (recipient !== resolvedBy.toLowerCase()) {
       const link = params.baseUrl ? `${params.baseUrl}/?nest=${encodeURIComponent(params.nestId)}` : "";
       void sendEmailToRecipient(recipient, {
-        status: outcome === "deleted" ? "deletion_deleted" : "deletion_declined",
+        // The email has to agree with the inbox row and the channel line above:
+        // an honoured archive is not a deletion, and the deletion template says
+        // the nest is gone.
+        status: archivedInstead ? "deletion_archived" : outcome === "deleted" ? "deletion_deleted" : "deletion_declined",
         docTitle: params.docTitle,
         path: params.nestId,
         link: outcome === "deleted" ? link : void 0,
@@ -1261,12 +1311,13 @@ async function requestDeletion(params) {
   const db = getDb();
   const targetType = params.targetType || "document";
   const noun = targetType === "document" ? "node" : targetType === "folder" ? "folder" : "nest";
+  const kind = targetType === "nest_archive" ? "An archive request" : "A deletion request";
   const existing = await db.get(
     "SELECT id FROM deletion_requests WHERE nest_id = ? AND node_id = ? AND target_type = ? AND status = 'pending'",
     [params.nestId, params.nodeId, targetType]
   );
   if (existing) {
-    throw new ConflictError(`A deletion request is already pending for this ${noun}`);
+    throw new ConflictError(`${kind} is already pending for this ${noun}`);
   }
   const id = uuid2();
   try {
@@ -1289,7 +1340,7 @@ async function requestDeletion(params) {
       params.nodeId,
       err
     );
-    throw new ConflictError(`A deletion request is already pending for this ${noun}`);
+    throw new ConflictError(`${kind} is already pending for this ${noun}`);
   }
   const request = await getDeletionRequest(id);
   await notifyDeletionRequested(request, params.baseUrl);
@@ -1437,6 +1488,7 @@ export {
   canRequestDeletion,
   filterAccessible,
   safeJson,
+  nodeTrail,
   requireWorkflowPlane,
   seedDefaultEdgeTypes,
   parseConditionSchema,
@@ -1458,6 +1510,7 @@ export {
   getReviewQueue,
   getReviewHistory,
   getPendingReview,
+  isNestTarget,
   deletionTargetTitle,
   notifyDeletionResolved,
   requestDeletion,

package/dist/{chunk-XCFKS65Y.js → chunk-SKZ7YYT4.js}

@@ -1678,6 +1678,30 @@ function runMigrations(db) {
       db.exec("ALTER TABLE publish ADD COLUMN no_download INTEGER NOT NULL DEFAULT 0");
     }
   }
+  if (!hasMigration("038_nest_archive")) {
+    const cols = db.prepare("PRAGMA table_info(nests)").all().map((c) => c.name);
+    if (!cols.includes("archived_at")) {
+      db.exec("ALTER TABLE nests ADD COLUMN archived_at TEXT");
+    }
+    recordMigration("038_nest_archive");
+  }
+  if (!hasMigration("039_nest_pins")) {
+    db.transaction(() => {
+      db.exec(`
+        CREATE TABLE IF NOT EXISTS nest_pins (
+          -- BOTH sides cascade, as grants and nest_collaborators do. Without it
+          -- on user_id, deleting an account with a single pin fails the whole
+          -- transaction on a foreign-key violation (foreign_keys is ON).
+          user_id TEXT NOT NULL REFERENCES users(id) ON DELETE CASCADE,
+          nest_id TEXT NOT NULL REFERENCES nests(id) ON DELETE CASCADE,
+          created_at TEXT NOT NULL DEFAULT (datetime('now')),
+          PRIMARY KEY (user_id, nest_id)
+        );
+        CREATE INDEX IF NOT EXISTS idx_nest_pins_user ON nest_pins(user_id);
+      `);
+    })();
+    recordMigration("039_nest_pins");
+  }
 }
 function mergeCaseCollidingUsers(db) {
   const groups = db.prepare(
@@ -1894,8 +1918,8 @@ async function initDb() {
   if (adapter) return adapter;
   if (config.DB_DRIVER === "postgres") {
     const { Pool } = await import("pg");
-    const { PostgresAdapter } = await import("./adapter.postgres-YOODX2BI.js");
-    const { runPostgresMigrations } = await import("./migrations.postgres-L42OXR7K.js");
+    const { PostgresAdapter } = await import("./adapter.postgres-6VZCMPQL.js");
+    const { runPostgresMigrations } = await import("./migrations.postgres-JA2YGLS3.js");
     const pool = new Pool(buildPgConfig());
     adapter = new PostgresAdapter(pool);
     await runPostgresMigrations(adapter);

package/dist/{chunk-EY7WOKJR.js → chunk-UVOAYQTU.js}

@@ -1,13 +1,14 @@
 import {
   engineCache,
   indexedNodeIds,
+  isNestArchived,
   markIndexStale,
   resolveNestPath,
   setSuggestionFlag
-} from "./chunk-7CPP6FIU.js";
+} from "./chunk-G4AL7ETF.js";
 import {
   getDb
-} from "./chunk-XCFKS65Y.js";
+} from "./chunk-SKZ7YYT4.js";
 
 // src/governance/external-edit-service.ts
 import { readFile, readdir } from "fs/promises";
@@ -81,6 +82,10 @@ async function scanDocumentForDriftInternal(nestId, documentId, actor) {
   if (await bodyMatchesLatestVersion(storage, documentId, drift.actualHash)) {
     return null;
   }
+  const { hasUnsealedDraft } = await import("./version-service-GWSDSUZR.js");
+  if (await hasUnsealedDraft(nestId, documentId)) {
+    return null;
+  }
   const approved = await loadChainHead(storage, documentId);
   if (!approved) return null;
   const existing = await listSuggestions(storage, documentId);
@@ -110,6 +115,7 @@ async function refreshSuggestionFlag(nestId, storage, documentId) {
   }
 }
 async function scanNestForDrift(nestId, actor = "system:scanner") {
+  if (await isNestArchived(nestId)) return { scanned: 0, staged: 0 };
   const { storage, dropDiscoveryCache } = await engineCache.get(nestId);
   dropDiscoveryCache();
   const docs = await storage.discoverDocuments();
@@ -253,7 +259,7 @@ async function listExternalEditVerdicts(nestId, documentId) {
 async function mirrorVersion(input) {
   const { storage } = await engineCache.get(input.nestId);
   const node = await storage.readDocument(input.documentId);
-  const { upsertVersion, setApprovedVersion } = await import("./version-service-5PIFT2OY.js");
+  const { upsertVersion, setApprovedVersion } = await import("./version-service-GWSDSUZR.js");
   await upsertVersion({
     nestId: input.nestId,
     nodeId: input.documentId,
@@ -338,7 +344,9 @@ async function rejectExternalEdit(input) {
 var scannerTimer = null;
 async function scanAllNests() {
   const db = getDb();
-  const rows = await db.all("SELECT id FROM nests");
+  const rows = await db.all(
+    "SELECT id FROM nests WHERE archived_at IS NULL"
+  );
   await Promise.all(
     rows.map(
       ({ id }) => scanNestForDrift(id).catch(

package/dist/{client-7Z2IVCPW.js → client-KBK4VIWJ.js}

@@ -2,7 +2,7 @@ import {
   getDb,
   initDb,
   resetDb
-} from "./chunk-XCFKS65Y.js";
+} from "./chunk-SKZ7YYT4.js";
 import "./chunk-YB3LKF7U.js";
 export {
   getDb,

package/dist/{engine-VZTSDUPB.js → engine-WXWER2V7.js}

@@ -2,9 +2,9 @@ import {
   engineApi,
   engineCache,
   opContext
-} from "./chunk-7CPP6FIU.js";
+} from "./chunk-G4AL7ETF.js";
 import "./chunk-YVMSM7LS.js";
-import "./chunk-XCFKS65Y.js";
+import "./chunk-SKZ7YYT4.js";
 import "./chunk-YB3LKF7U.js";
 import "./chunk-FRQJWGN3.js";
 export {

package/dist/{external-edit-service-3FVK3OD7.js → external-edit-service-Y757EFMS.js}

@@ -11,10 +11,10 @@ import {
   scanNestForDrift,
   startDriftScanner,
   stopDriftScanner
-} from "./chunk-EY7WOKJR.js";
-import "./chunk-7CPP6FIU.js";
+} from "./chunk-UVOAYQTU.js";
+import "./chunk-G4AL7ETF.js";
 import "./chunk-YVMSM7LS.js";
-import "./chunk-XCFKS65Y.js";
+import "./chunk-SKZ7YYT4.js";
 import "./chunk-YB3LKF7U.js";
 import "./chunk-FRQJWGN3.js";
 export {

package/dist/{grants-service-WC27TRY3.js → grants-service-FKINSDLZ.js}

@@ -6,9 +6,9 @@ import {
   listGrants,
   listUserGrants,
   resolveNodeGrant
-} from "./chunk-3LUUJUWS.js";
+} from "./chunk-IVXVOXYN.js";
 import "./chunk-YVMSM7LS.js";
-import "./chunk-XCFKS65Y.js";
+import "./chunk-SKZ7YYT4.js";
 import "./chunk-YB3LKF7U.js";
 export {
   createGrant,