@nuasite/cms 0.31.0 → 0.32.0

package/package.json

@@ -14,7 +14,7 @@
     "directory": "packages/astro-cms"
   },
   "license": "Apache-2.0",
-  "version": "0.31.0",
+  "version": "0.32.0",
   "module": "src/index.ts",
   "types": "src/index.ts",
   "type": "module",

package/src/editor/constants.ts

@@ -109,6 +109,9 @@ export const CSS = {
 	HIGHLIGHT_ELEMENT: 'cms-highlight-overlay',
 	/** Data attribute for background image elements */
 	BG_IMAGE_ATTRIBUTE: 'data-cms-bg-img',
+	/** Data attribute set during edit mode on elements whose manifest entry has no source path;
+	 *  suppresses hover outlines and blocks interaction so the user doesn't type into a dead-end. */
+	LOCKED_ATTRIBUTE: 'data-cms-locked',
 } as const
 
 /**

package/src/editor/dom.ts

@@ -76,6 +76,8 @@ export function getCmsElementAtPosition(
 		if (!el.hasAttribute(CSS.ID_ATTRIBUTE)) continue
 		// Skip component roots - they should be handled separately
 		if (el.hasAttribute(CSS.COMPONENT_ID_ATTRIBUTE)) continue
+		// Skip elements locked because their manifest entry has no source path
+		if (el.hasAttribute(CSS.LOCKED_ATTRIBUTE)) continue
 
 		const cmsId = el.getAttribute(CSS.ID_ATTRIBUTE)
 
@@ -96,6 +98,7 @@ export function getCmsElementAtPosition(
 		if (!(el instanceof HTMLElement)) continue
 		if (!el.hasAttribute(CSS.ID_ATTRIBUTE)) continue
 		if (el.hasAttribute(CSS.COMPONENT_ID_ATTRIBUTE)) continue
+		if (el.hasAttribute(CSS.LOCKED_ATTRIBUTE)) continue
 
 		const cmsId = el.getAttribute(CSS.ID_ATTRIBUTE)
 

package/src/editor/editor.ts

@@ -76,6 +76,7 @@ const INLINE_STYLE_ELEMENTS = [
 // merge a burst, short enough that the next deliberate action still explains itself.
 const FORMATTING_BLOCKED_TOAST_COOLDOWN_MS = 3000
 let lastFormattingBlockedToastAt = 0
+let lastLockedToastAt = 0
 
 // Signals listener cleanup on stopEditMode. Aborting removes every listener
 // attached with { signal } in the current edit session in one shot.
@@ -90,6 +91,21 @@ function notifyFormattingBlocked(): void {
 	signals.showToast("Formatting isn't available — this text is used as a plain value", 'info')
 }
 
+export function notifyLockedElement(): void {
+	const now = Date.now()
+	if (now - lastLockedToastAt < FORMATTING_BLOCKED_TOAST_COOLDOWN_MS) {
+		return
+	}
+	lastLockedToastAt = now
+	signals.showToast("This text can't be edited here — no source file is linked to it", 'info')
+}
+
+/** Test-only: reset toast throttle state between test cases. */
+export function _resetToastThrottles(): void {
+	lastFormattingBlockedToastAt = 0
+	lastLockedToastAt = 0
+}
+
 // Uses the Selection/Range API rather than the deprecated document.execCommand('insertText').
 export function insertPlainTextAtRange(range: Range, text: string): boolean {
 	if (!text) return false
@@ -224,6 +240,16 @@ export async function startEditMode(
 			return
 		}
 
+		// Without a source path, the writer has nowhere to persist text edits — lock
+		// the element so it can't be typed into and the user gets told why on click.
+		if (!manifestEntry?.sourcePath) {
+			logDebug(config.debug, 'Skipping element without source path:', cmsId)
+			makeElementNonEditable(el)
+			el.setAttribute(CSS.LOCKED_ATTRIBUTE, 'true')
+			el.addEventListener('click', notifyLockedElement, { signal: editModeSignal })
+			return
+		}
+
 		makeElementEditable(el)
 
 		const stylingAllowed = manifestEntry?.allowStyling !== false
@@ -434,6 +460,7 @@ export function stopEditMode(onStateChange?: () => void): void {
 
 	getAllCmsElements().forEach(el => {
 		makeElementNonEditable(el)
+		el.removeAttribute(CSS.LOCKED_ATTRIBUTE)
 	})
 }
 

package/src/field-types.ts

@@ -65,10 +65,24 @@ const toISODate = (v: unknown) => (v instanceof Date ? v.toISOString().slice(0,
 /** Normalize YAML Date objects to ISO datetime strings */
 const toISODatetime = (v: unknown) => (v instanceof Date ? v.toISOString() : v)
 
-/** Add a chainable `.orderBy()` method to a Zod schema. The scanner detects it from source code. */
+const WRAPPING_METHODS = ['default', 'optional', 'nullable', 'nullish'] as const
+
+/**
+ * Add a chainable `.orderBy()` method to a Zod schema. The scanner detects it from source code.
+ *
+ * Also wraps the Zod methods that produce a new schema instance (`.default()`, `.optional()`,
+ * `.nullable()`, `.nullish()`) so the marker survives those wrappers — chain order doesn't matter.
+ */
 function withOrderBy<T extends z.ZodTypeAny>(schema: T): WithOrderBy<T> {
 	const s = schema as WithOrderBy<T>
 	s.orderBy = (_direction?: OrderByDirection) => schema
+	for (const method of WRAPPING_METHODS) {
+		const original = (schema as unknown as Record<string, unknown>)[method]
+		if (typeof original !== 'function') continue
+		;(s as unknown as Record<string, unknown>)[method] = function(this: unknown, ...args: unknown[]) {
+			return withOrderBy((original as (...a: unknown[]) => z.ZodTypeAny).apply(this, args))
+		}
+	}
 	return s
 }