@contentrain/mcp 3.1.7 → 3.2.0

package/dist/{server-BRkKIO3r.mjs → server-ChpsLHlF.mjs}

@@ -1,25 +1,25 @@
-import { n as readContext } from "./context-CxpTATh4.mjs";
-import { D as contentDirPath, O as contentFilePath, T as writeContent, _ as resolveContentDir, b as resolveMdFilePath, c as listModels, d as validateModelDefinition, f as writeModel, i as countEntries, l as readModel, m as listContent, o as fieldDefZodSchema, p as deleteContent, r as checkReferences, v as resolveJsonFilePath, y as resolveLocaleStrategy } from "./model-manager-DqwL8ptm.mjs";
-import { a as planModelDelete, i as planModelSave, n as planVocabularySave, o as planContentSave, s as planContentDelete, t as planVocabularyDelete } from "./ops-BfrdZ6GK.mjs";
+import { n as readContext } from "./context-BHfO4WBv.mjs";
+import { A as contentDirPath, C as resolveMdFilePath, O as writeContent, S as resolveLocaleStrategy, _ as listContent, a as countEntries, b as resolveContentDir, d as readModel, g as deleteContent, h as writeModel, i as collectFieldPaths, j as contentFilePath, m as validateModelDefinition, r as checkReferences, s as fieldDefZodSchema, u as listModels, x as resolveJsonFilePath } from "./model-manager-CfwJWvz7.mjs";
+import { a as planModelDelete, i as planModelSave, n as planVocabularySave, o as planContentSave, s as planContentDelete, t as planVocabularyDelete } from "./ops-B2P11b8v.mjs";
 import { a as readJson, n as ensureDir, r as pathExists, s as writeJson, t as contentrainDir } from "./fs-DJv5G8uL.mjs";
-import { a as writeMeta, i as readMeta, o as writeMetaEntries, r as mergeEntryMeta, t as applyStatusChange } from "./meta-manager-g3shoZES.mjs";
+import { a as writeMeta, i as readMeta, o as writeMetaEntries, r as mergeEntryMeta, t as applyStatusChange } from "./meta-manager-Cd6AfuGS.mjs";
 import { t as OverlayReader } from "./overlay-reader-Bx1YxGXq.mjs";
 import { n as readVocabulary, t as readConfig } from "./config-B8a4JBT1.mjs";
-import { t as validateProject } from "./validator-B2GhPM3L.mjs";
+import { t as validateProject } from "./validator-CLugFhHw.mjs";
 import { r as createGit } from "./identity-CxAB9Qb8.mjs";
 import { a as contentBranchRelation, c as listRemoteCrBranches, i as cleanupMergedBranches, n as checkBranchHealth, o as deleteRemoteBranch, u as pruneMergedRemoteBranches } from "./branch-lifecycle-DUOoyNWn.mjs";
-import { i as ensureContentBranch, n as buildBranchName, o as mergeBranch, r as createTransaction } from "./transaction-BxDiiakv.mjs";
-import { n as isMerged, t as LocalProvider } from "./local-bnn6yrG7.mjs";
+import { i as ensureContentBranch, n as buildBranchName, o as mergeBranch, r as createTransaction } from "./transaction-BR2jg4uj.mjs";
+import { n as isMerged, t as LocalProvider } from "./local-DbPlR0Br.mjs";
 import { n as isToolAvailable, t as TOOL_REQUIREMENTS } from "./availability-DYHNoP0_.mjs";
 import { t as detectStack } from "./detect-CI32ynpq.mjs";
 import { t as TOOL_ANNOTATIONS } from "./annotations-CXHPnS-M.mjs";
 import { n as listTemplates, t as getTemplate } from "./templates-BZbohJc8.mjs";
 import { t as normalizeOperationError } from "./errors-BmLE-JSC.mjs";
-import { t as reconcileBranches } from "./reconcile-M7KBZtdn.mjs";
+import { t as reconcileBranches } from "./reconcile-EANDIp_T.mjs";
 import { t as buildGraph } from "./graph-builder-C9NS7y-a.mjs";
 import { n as scanSummary, t as scanCandidates } from "./scanner-CEiT8Tgx.mjs";
-import { n as applyExtract, r as applyReuse } from "./apply-manager-DsNq7Zhm.mjs";
-import { t as runDoctor } from "./doctor-B7G0oFxO.mjs";
+import { n as applyExtract, r as applyReuse } from "./apply-manager-eZ6zA99Y.mjs";
+import { t as runDoctor } from "./doctor-VnOzDTbk.mjs";
 import { join } from "node:path";
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { CONTENTRAIN_BRANCH } from "@contentrain/types";
@@ -243,8 +243,8 @@ function registerContextTools(server, provider, projectRoot) {
 						updated_by: "Author identifier string",
 						approved_by: "Optional: approver identifier",
 						version: "Optional: version string",
-						publish_at: "Optional: ISO 8601 date for scheduled publishing",
-						expire_at: "Optional: ISO 8601 date for scheduled expiry (must be after publish_at)"
+						publish_at: "Optional: ISO 8601 date for scheduled publishing. A delivery gate on top of status \"published\" — it never changes status, so a draft with a past publish_at stays a draft. Meta only; never written to the content file.",
+						expire_at: "Optional: ISO 8601 date for scheduled expiry (must be after publish_at). Same gate semantics as publish_at."
 					}
 				},
 				locale_strategies: {
@@ -902,7 +902,7 @@ function registerModelTools(server, provider, projectRoot) {
 			}],
 			isError: true
 		};
-		const { errors, warnings: schemaWarnings } = validateModel(input);
+		const { errors, warnings: schemaWarnings } = validateModel(input, { existingFieldPaths: collectFieldPaths((await readModel(provider, input.id))?.fields) });
 		if (errors.length > 0) return {
 			content: [{
 				type: "text",
@@ -1110,8 +1110,8 @@ function registerContentTools(server, provider, projectRoot) {
 			slug: z.string().optional().describe("Slug (document only)"),
 			locale: z.string().optional().describe("Locale code (defaults to config default)"),
 			data: z.record(z.string(), z.unknown()).describe("Content data. For documents, include \"body\" key for markdown content."),
-			publish_at: z.string().optional().describe("ISO 8601 date for scheduled publishing (stored in meta)"),
-			expire_at: z.string().optional().describe("ISO 8601 date for scheduled expiry (stored in meta, must be after publish_at)")
+			publish_at: z.string().nullable().optional().describe("Scheduled publish date, ISO 8601. Stored in meta only — never in the content file — and it does NOT change status: a draft stays a draft, and a published entry is delivered only once this date has passed. Publish with contentrain_bulk update_status. Pass null to clear a previously set date; omit to leave it unchanged."),
+			expire_at: z.string().nullable().optional().describe("Scheduled expiry, ISO 8601, must be after publish_at. Same rules as publish_at: meta only, never changes status, null clears, omit leaves unchanged.")
 		})).describe("Content entries to save")
 	}, TOOL_ANNOTATIONS["contentrain_content_save"], async (input) => {
 		const config = await readConfig(provider);
@@ -1138,7 +1138,7 @@ function registerContentTools(server, provider, projectRoot) {
 				}],
 				isError: true
 			};
-			if (entry.publish_at !== void 0) {
+			if (typeof entry.publish_at === "string") {
 				const d = new Date(entry.publish_at);
 				if (Number.isNaN(d.getTime())) return {
 					content: [{
@@ -1148,7 +1148,7 @@ function registerContentTools(server, provider, projectRoot) {
 					isError: true
 				};
 			}
-			if (entry.expire_at !== void 0) {
+			if (typeof entry.expire_at === "string") {
 				const d = new Date(entry.expire_at);
 				if (Number.isNaN(d.getTime())) return {
 					content: [{
@@ -1158,7 +1158,7 @@ function registerContentTools(server, provider, projectRoot) {
 					isError: true
 				};
 			}
-			if (entry.publish_at !== void 0 && entry.expire_at !== void 0) {
+			if (typeof entry.publish_at === "string" && typeof entry.expire_at === "string") {
 				if (new Date(entry.expire_at) <= new Date(entry.publish_at)) return {
 					content: [{
 						type: "text",
@@ -1167,9 +1167,8 @@ function registerContentTools(server, provider, projectRoot) {
 					isError: true
 				};
 			}
-			if (entry.publish_at !== void 0) entry.data["publish_at"] = entry.publish_at;
-			if (entry.expire_at !== void 0) entry.data["expire_at"] = entry.expire_at;
 		}
+		const schedulingTouched = input.entries.some((e) => e.publish_at !== void 0 || e.expire_at !== void 0);
 		const readiness = await provider.checkWriteReadiness?.();
 		if (readiness?.blocked) return {
 			content: [{
@@ -1262,6 +1261,7 @@ function registerContentTools(server, provider, projectRoot) {
 					advisory_note: "Save succeeded. Review these warnings and consider consolidating duplicate values."
 				} : {},
 				...warnings.length > 0 ? { warnings } : {},
+				...schedulingTouched ? { scheduling_note: "publish_at/expire_at were written to meta only — not to the content file — and status is unchanged: a draft is still a draft. They gate delivery of a *published* entry (served once publish_at has passed, until expire_at). To publish, call contentrain_bulk update_status." } : {},
 				validation: {
 					valid: true,
 					errors: []
@@ -1271,6 +1271,7 @@ function registerContentTools(server, provider, projectRoot) {
 					...divergenceNextSteps(commit),
 					...allAdvisories.length > 0 ? ["ADVISORY: Duplicate values detected — review advisories above"] : [],
 					...warnings.length > 0 ? ["REVIEW: see warnings above — the save was not blocked by them"] : [],
+					...schedulingTouched ? ["PUBLISH: publish_at did not change status — use contentrain_bulk update_status if this entry should go live"] : [],
 					...model.kind === "collection" ? ["Use contentrain_content_list to verify", "Add more entries or publish"] : ["Use contentrain_content_list to verify"]
 				]
 			}, null, 2)
@@ -1661,9 +1662,9 @@ async function resolveMergeBranch(git, input) {
 	}))).toSorted((left, right) => right.t - left.t)[0].b };
 }
 function registerWorkflowTools(server, provider, projectRoot) {
-	server.tool("contentrain_validate", "Validate project content against model schemas. Detects required field violations, type mismatches, broken relations, secret leaks, i18n parity issues, and more. If fix:true, auto-fixes structural issues (canonical sort, orphan meta, missing locale files, stray non-i18n meta layout) and backfills a missing model title_field — do NOT manually edit .contentrain/ files.", {
+	server.tool("contentrain_validate", "Validate project content against model schemas. Detects required field violations, type mismatches, broken relations, secret leaks, i18n parity issues, and more. If fix:true, auto-fixes structural issues (canonical sort, orphan meta, missing locale files, stray non-i18n meta layout, scheduling keys an older content_save leaked into document frontmatter) and backfills a missing model title_field — do NOT manually edit .contentrain/ files.", {
 		model: z.string().optional().describe("Model ID to validate (omit for all models)"),
-		fix: z.boolean().optional().describe("Auto-fix structural issues (canonical sort, orphan meta, missing locale files, stray non-i18n meta layout) and backfill a missing model title_field. Each backfilled title_field is reported as a notice naming the rule that chose it. Default: false")
+		fix: z.boolean().optional().describe("Auto-fix structural issues (canonical sort, orphan meta, missing locale files, stray non-i18n meta layout, publish_at/expire_at copies an older content_save left in document frontmatter when meta holds the same key) and backfill a missing model title_field. Each backfilled title_field is reported as a notice naming the rule that chose it. Default: false")
 	}, TOOL_ANNOTATIONS["contentrain_validate"], async (input) => {
 		if (input.fix && !projectRoot) return capabilityError("contentrain_validate", "localWorktree");
 		if (!await readConfig(provider)) return {
@@ -2421,6 +2422,7 @@ function registerBulkTools(server, _provider, projectRoot) {
 		source_locale: z.string().optional().describe("Source locale for copy_locale operation"),
 		target_locale: z.string().optional().describe("Target locale for copy_locale operation"),
 		entry_ids: z.array(z.string()).optional().describe("Entry IDs for update_status (collection models) or delete_entries"),
+		slugs: z.array(z.string()).optional().describe("Document slugs for update_status (document models) — a document is addressed by slug, the same identity contentrain_content_save uses, not by entry ID"),
 		locale: z.string().optional().describe("Scope update_status to a single locale (i18n models only; defaults to every supported locale)"),
 		status: z.enum([
 			"draft",
@@ -2571,22 +2573,39 @@ function registerBulkTools(server, _provider, projectRoot) {
 					}],
 					isError: true
 				};
-				if (model.kind === "document") return {
+				const keyedByEntry = model.kind === "collection";
+				const keyedBySlug = model.kind === "document";
+				const hasEntryIds = Boolean(input.entry_ids && input.entry_ids.length > 0);
+				const hasSlugs = Boolean(input.slugs && input.slugs.length > 0);
+				if (keyedBySlug && hasEntryIds) return {
 					content: [{
 						type: "text",
-						text: JSON.stringify({ error: `Model "${input.model}" is a document model. update_status does not support documents yet — their meta is keyed by slug, not entry ID.` })
+						text: JSON.stringify({ error: `Model "${input.model}" is a document model — documents are keyed by slug, not entry ID. Pass slugs instead of entry_ids.` })
 					}],
 					isError: true
 				};
-				const keyedByEntry = model.kind === "collection";
-				if (keyedByEntry && (!input.entry_ids || input.entry_ids.length === 0)) return {
+				if (keyedBySlug && !hasSlugs) return {
+					content: [{
+						type: "text",
+						text: JSON.stringify({ error: `update_status requires slugs for document model "${input.model}".` })
+					}],
+					isError: true
+				};
+				if (!keyedBySlug && hasSlugs) return {
+					content: [{
+						type: "text",
+						text: JSON.stringify({ error: `slugs only apply to document models — "${input.model}" is a ${model.kind} model. ${keyedByEntry ? "Pass entry_ids." : "Omit both entry_ids and slugs."}` })
+					}],
+					isError: true
+				};
+				if (keyedByEntry && !hasEntryIds) return {
 					content: [{
 						type: "text",
 						text: JSON.stringify({ error: `update_status requires entry_ids for collection model "${input.model}".` })
 					}],
 					isError: true
 				};
-				if (!keyedByEntry && input.entry_ids && input.entry_ids.length > 0) return {
+				if (!keyedByEntry && !keyedBySlug && hasEntryIds) return {
 					content: [{
 						type: "text",
 						text: JSON.stringify({ error: `Model "${input.model}" is a ${model.kind} model — it has one meta record per locale, so entry_ids do not apply. Omit entry_ids.` })
@@ -2624,6 +2643,24 @@ function registerBulkTools(server, _provider, projectRoot) {
 								defaultLocale: config.locales.default
 							}, updates);
 							if (written.length > 0) updatedPerLocale[locale] = written;
+						} else if (keyedBySlug) {
+							const written = [];
+							for (const slug of input.slugs) {
+								const existing = await readMeta(wt, model, {
+									locale,
+									slug,
+									defaultLocale: config.locales.default
+								});
+								if (!existing) continue;
+								await writeMeta(wt, model, {
+									locale,
+									slug,
+									defaultLocale: config.locales.default
+								}, applyStatusChange(existing, input.status));
+								written.push(slug);
+								foundIds.add(slug);
+							}
+							if (written.length > 0) updatedPerLocale[locale] = written;
 						} else {
 							const existing = await readMeta(wt, model, {
 								locale,
@@ -2638,14 +2675,15 @@ function registerBulkTools(server, _provider, projectRoot) {
 						}
 					});
 					const updatedCount = Object.values(updatedPerLocale).reduce((n, ids) => n + ids.length, 0);
-					const notFound = keyedByEntry ? input.entry_ids.filter((id) => !foundIds.has(id)) : [];
+					const requested = keyedByEntry ? input.entry_ids : keyedBySlug ? input.slugs : [];
+					const notFound = requested.filter((key) => !foundIds.has(key));
 					if (updatedCount === 0) {
 						await tx.cleanup();
 						return {
 							content: [{
 								type: "text",
 								text: JSON.stringify({
-									error: keyedByEntry ? `No meta records found for the given entry_ids in model "${input.model}" across locales [${targetLocales.join(", ")}]. Nothing was changed.` : `No meta record found for model "${input.model}" across locales [${targetLocales.join(", ")}]. Nothing was changed.`,
+									error: keyedByEntry ? `No meta records found for the given entry_ids in model "${input.model}" across locales [${targetLocales.join(", ")}]. Nothing was changed.` : keyedBySlug ? `No meta records found for the given slugs in model "${input.model}" across locales [${targetLocales.join(", ")}]. Nothing was changed. Check the slugs with contentrain_content_list.` : `No meta record found for model "${input.model}" across locales [${targetLocales.join(", ")}]. Nothing was changed.`,
 									not_found: notFound.length > 0 ? notFound : void 0
 								}, null, 2)
 							}],
@@ -2655,7 +2693,7 @@ function registerBulkTools(server, _provider, projectRoot) {
 					await tx.commit(`[contentrain] bulk: update status → ${input.status} for ${input.model}`, {
 						tool: "contentrain_bulk",
 						model: input.model,
-						...keyedByEntry ? { entries: input.entry_ids } : {}
+						...requested.length > 0 ? { entries: requested } : {}
 					});
 					const gitResult = await tx.complete();
 					return { content: [{
@@ -2921,7 +2959,7 @@ function registerMediaTools(server, provider, _projectRoot) {
 }
 //#endregion
 //#region package.json
-var version = "3.1.7";
+var version = "3.2.0";
 //#endregion
 //#region src/server.ts
 /**
@@ -2996,4 +3034,4 @@ function resolveServerContext(input) {
 //#endregion
 export { createServer as n, DEFAULT_INSTRUCTIONS as t };
 
-//# sourceMappingURL=server-BRkKIO3r.mjs.map
+//# sourceMappingURL=server-ChpsLHlF.mjs.map