@elytracms/studio 0.0.13 → 0.0.14

package/convex/cms.ts

@@ -3,6 +3,7 @@ import { v } from 'convex/values'
 import { mutation, query } from './functions'
 import { requireRole } from './guard'
 import { maintainDocumentReferences, removeDocumentReferences } from './referenceMaintenance'
+import { scheduleRevalidate } from './revalidate'
 
 /**
  * CMS document persistence (EC-043). The backend stores **documents only**
@@ -174,6 +175,11 @@ export const publishDocumentVersion = mutation({
     // entries are extracted from the live draft content (matching the EC-173 guard
     // and `reindexDocumentReferences`), not the pinned snapshot.
     await maintainDocumentReferences(ctx, args.collection, args.docId, doc.document as CmsDocument, true)
+    // EC-146 instant publish: a CONTENT publish must invalidate the host cache just
+    // like a graph publish — without this, instant-publish silently never fires for
+    // a content-only site (the server-side webhook path previously covered only
+    // `publishing:publish`). Fire-and-forget; unconfigured deployments no-op.
+    await scheduleRevalidate(ctx, 'documents.publish')
     return ctx.db.get(doc._id)
   },
 })
@@ -191,6 +197,9 @@ export const unpublishDocument = mutation({
     await ctx.db.patch(doc._id, { publishedVersion: undefined, updatedAt: args.updatedAt })
     // EC-277: the live pin is cleared — drop this doc's published index rows.
     await maintainDocumentReferences(ctx, args.collection, args.docId, doc.document as CmsDocument, false)
+    // EC-146 instant publish: unpublishing content must drop the host's cached
+    // pages too (the page that baked this doc now serves stale published content).
+    await scheduleRevalidate(ctx, 'documents.unpublish')
     return ctx.db.get(doc._id)
   },
 })

package/convex/publishing.ts

@@ -1,50 +1,18 @@
-import { makeFunctionReference } from 'convex/server'
 import { v } from 'convex/values'
 import { mutation, query } from './functions'
-import type { MutationCtx } from './functions'
 import { requireRole } from './guard'
 import { maintainGraphPublishedReferences } from './referenceMaintenance'
-
-// EC-223: the studio's convex functions are package-shipped and deployment-agnostic
-// — they address each other by STRING ref (resolved against the consumer's deployment),
-// never the per-deployment `_generated/api`. Mirrors the lib's `queryRef`/`actionRef`.
-const webhooksDeliver = makeFunctionReference<'action'>('webhooks:deliver')
+import { scheduleRevalidate } from './revalidate'
 
 /**
  * Publishing state (EC-045). Single-environment (EC-179, vision AD-8): a
  * deployment IS one environment, so there is exactly ONE publishingState row —
  * see `env-is-deployment-boundary`.
- */
-
-/**
- * EC-180 / EC-146 — fire the host revalidation webhook on a publish-state change.
- *
- * A mutation cannot `fetch`, so we hand off to the `webhooks.deliver` internal
- * action (it signs + POSTs server-side; the secret never leaves Convex). A graph
- * publish/unpublish can affect any page, so the honest v1 invalidation is the
- * whole-project sweep tag `p:<scope>:project` — a mirror of
- * `projectSweepWebhookTag` (`src/lib/webhooks/tags.ts`) / `@elytracms/next`'s
- * `projectSweepTag`; the tests on those two sides pin the literal.
  *
- * `scope` must equal the host's `cache.scope` (its `ELYTRA_PROJECT_ID`) for the
- * tag to match what the host stamped on its cached fetches; it comes from the
- * `CANVAS_PROJECT_SCOPE` Convex env var. Unconfigured deployments (local dev,
- * no env) are a silent no-op — publishing must never depend on a reachable host.
+ * EC-180 / EC-146: a graph publish/unpublish fires the host revalidation webhook
+ * via `scheduleRevalidate` (`./revalidate`) — the SAME helper CMS document
+ * publish uses, so content-only sites invalidate too (see `cms.ts`).
  */
-async function scheduleRevalidate(
-  ctx: MutationCtx,
-  event: 'publishing.publish' | 'publishing.unpublish',
-): Promise<void> {
-  const env =
-    (globalThis as { process?: { env?: Record<string, string | undefined> } }).process?.env ?? {}
-  const scope = env['CANVAS_PROJECT_SCOPE']
-  if (!scope) return
-  await ctx.scheduler.runAfter(0, webhooksDeliver, {
-    projectId: scope,
-    event,
-    tags: [`p:${scope}:project`],
-  })
-}
 
 export const publish = mutation({
   args: {

package/convex/revalidate.ts

@@ -0,0 +1,42 @@
+import { makeFunctionReference } from 'convex/server'
+import type { MutationCtx } from './functions'
+
+// EC-223: the studio's convex functions address each other by STRING ref
+// (resolved against the consumer's deployment), never the per-deployment
+// `_generated/api`. Mirrors the lib's `queryRef`/`actionRef`.
+const webhooksDeliver = makeFunctionReference<'action'>('webhooks:deliver')
+
+/**
+ * EC-146 / EC-180 — schedule the host revalidation webhook after a publish-state
+ * change. A mutation cannot `fetch`, so it hands off to the `webhooks.deliver`
+ * internal action (it signs + POSTs server-side; the secret never leaves Convex).
+ *
+ * The v1 invalidation is the whole-project sweep tag `p:<scope>:project` — every
+ * cached host page carries it (`@elytracms/next` `nextCacheTags` stamps it on
+ * every entry), so it is GUARANTEED to drop the affected pages with no tag-format
+ * coupling that could silently miss. `scope` must equal the host's `cache.scope`
+ * (its `ELYTRA_PROJECT_ID`); it comes from the `CANVAS_PROJECT_SCOPE` Convex env
+ * var. An unconfigured deployment (local dev, no env) is a silent no-op —
+ * publishing must never depend on a reachable host.
+ *
+ * Shared by BOTH graph publish (`publishing.ts`) and CMS **document** publish
+ * (`cms.ts`). A content-only site never touches the graph, so without the
+ * document-publish call instant-publish silently never fires for it (the
+ * footgun this consolidates away).
+ *
+ * Doc/collection-scoped tags (matching the client dispatcher's
+ * `webhookTagsForEvent`) would be more surgical, but need the project locales —
+ * not available in the Convex env here. The sweep is the honest, reliable v1;
+ * threading scoped tags is a precision follow-up.
+ */
+export async function scheduleRevalidate(ctx: MutationCtx, event: string): Promise<void> {
+  const env =
+    (globalThis as { process?: { env?: Record<string, string | undefined> } }).process?.env ?? {}
+  const scope = env['CANVAS_PROJECT_SCOPE']
+  if (!scope) return
+  await ctx.scheduler.runAfter(0, webhooksDeliver, {
+    projectId: scope,
+    event,
+    tags: [`p:${scope}:project`],
+  })
+}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@elytracms/studio",
   "author": "finaldream <hi@finaldream.de>",
-  "version": "0.0.13",
+  "version": "0.0.14",
   "private": false,
   "description": "The Elytra studio engine (UI + lib + routes + data-only Convex schema) as an importable package. Extraction staged in EXTRACTION.md (EC-223); Stages 1-2 = UI primitives + router-free lib leaves.",
   "license": "MIT",
@@ -112,7 +112,7 @@
     "lucide-react": "^1.17.0",
     "tailwind-merge": "^3.6.0",
     "zod": "^4.0.0",
-    "@elytracms/core": "^0.0.13"
+    "@elytracms/core": "^0.0.14"
   },
   "peerDependencies": {
     "@tanstack/react-router": "^1.170.0",