@superdoc/sdk 2.21.0-next.8 → 2.21.0

package/dist/agent/actions.cjs

@@ -518,6 +518,17 @@ function evaluateChecks(pre, post, checks) {
     }
     return results;
 }
+function assertReplacementTextPayloads(edits) {
+    for (const [index, edit] of edits.entries()) {
+        const text = edit.replace ?? '';
+        for (let offset = 0; offset < text.length; offset += 1) {
+            const code = text.charCodeAt(offset);
+            if (code < 0x20 && code !== 0x09 && code !== 0x0a) {
+                throw new errors.SuperDocCliError(`edits[${index}].replace contains unsupported control character U+${code.toString(16).padStart(4, '0')}. Use plain text, tabs or preserved manual line breaks.`, { code: 'INVALID_PAYLOAD' });
+            }
+        }
+    }
+}
 function preserveRewriteStyle() {
     return {
         inline: { mode: 'preserve' },
@@ -964,6 +975,7 @@ async function runReplaceTextUnchecked(doc, args) {
         ? await docSnapshot.buildMutationSnapshot(doc, { includeDomains: selectorDomains ?? ['blocks'] })
         : snapshotFromIdentity(preIdentity);
     try {
+        assertReplacementTextPayloads(args.edits);
         if (args.edits.length === 0) {
             return failedReceipt('replace_text', new Error('edits must be non-empty'), pre);
         }
@@ -2493,115 +2505,6 @@ async function applyInlineQuiet(formatFn, params) {
         throw err;
     }
 }
-/** Best-effort read of a block's current run layout (null when unavailable). */
-async function captureBlockRuns(doc, nodeId, text) {
-    const queryFn = maybeMethod(doc, ['query', 'match']);
-    const trimmed = text.trim();
-    if (!queryFn || !trimmed || text.includes('\n'))
-        return null;
-    try {
-        const runs = await docSnapshot.matchRunsForBlock(queryFn, trimmed, nodeId);
-        return runs.length > 0 ? runs : null;
-    }
-    catch {
-        return null;
-    }
-}
-/**
- * Restore scoped replacements from their known spans: each untouched interval
- * keeps its run formatting, and replacements inherit the marks at their start.
- * `spans` are FULL-TEXT coordinates, ascending, non-overlapping.
- */
-async function preserveScopedRunPattern(doc, nodeId, oldText, oldRuns, spans) {
-    if (!oldRuns || oldRuns.length === 0 || spans.length === 0)
-        return null;
-    if (!oldRuns.some((run) => Object.keys(inlineFromSnapshotRun(run, new Set())).length > 0))
-        return null;
-    const formatFn = maybeMethod(doc, ['format', 'apply']);
-    if (!formatFn)
-        return null;
-    const normalize = new Set(RUN_BOOL_MARKS.filter((key) => oldRuns.some((run) => run[key])));
-    // Run offsets (captureBlockRuns) are relative to the TRIMMED text; spans are
-    // full-text. Convert spans to trimmed coordinates and clamp.
-    const oldTrimStart = oldText.length - oldText.trimStart().length;
-    const oldTrimLen = oldText.trim().length;
-    const trimSpans = spans
-        .map((s) => ({
-        start: Math.max(0, s.start - oldTrimStart),
-        end: Math.min(oldTrimLen, s.end - oldTrimStart),
-        replaceLen: (s.replace ?? '').length,
-    }))
-        .filter((s) => s.end > s.start || s.replaceLen > 0);
-    if (trimSpans.length === 0)
-        return null;
-    const newText = spans.reduceRight((text, span) => text.slice(0, span.start) + span.replace + text.slice(span.end), oldText);
-    const newTrimStart = newText.length - newText.trimStart().length;
-    // Cumulative old→new offset shift for positions AT OR AFTER a given old offset.
-    const deltaBefore = (oldPos) => {
-        let d = 0;
-        for (const s of trimSpans) {
-            if (s.end <= oldPos)
-                d += s.replaceLen - (s.end - s.start);
-            else
-                break;
-        }
-        return d;
-    };
-    let applied = 0;
-    const applyRange = async (start, end, inline) => {
-        if (end <= start || Object.keys(inline).length === 0)
-            return;
-        // Always direct, even under tracked mode. This pass RESTORES the marks the
-        // rewrite flattened; it is not new authorial formatting, and the text
-        // revision already owns the span's accept/reject fate. Applying it tracked
-        // adds a second reviewable change per rewrite (SD-4309's conformance matrix
-        // counts one per action), and rejecting that change alone would strand the
-        // text in the flattened look the rewrite produced.
-        const changed = await applyInlineQuiet(formatFn, {
-            blockId: nodeId,
-            start: newTrimStart + start,
-            end: newTrimStart + end,
-            inline,
-        });
-        if (changed)
-            applied += 1;
-    };
-    try {
-        // Untouched intervals between/around the spans, in old trimmed coords.
-        const intervals = [];
-        let cursor = 0;
-        for (const s of trimSpans) {
-            if (s.start > cursor)
-                intervals.push({ start: cursor, end: s.start });
-            cursor = Math.max(cursor, s.end);
-        }
-        if (cursor < oldTrimLen)
-            intervals.push({ start: cursor, end: oldTrimLen });
-        for (const run of oldRuns) {
-            const inline = inlineFromSnapshotRun(run, normalize);
-            for (const iv of intervals) {
-                const a = Math.max(run.start, iv.start);
-                const b = Math.min(run.end, iv.end);
-                if (b <= a)
-                    continue;
-                const d = deltaBefore(a); // constant across an untouched interval
-                await applyRange(a + d, b + d, inline);
-            }
-        }
-        // Each replacement span inherits the marks of the run it starts in.
-        for (const s of trimSpans) {
-            if (s.replaceLen === 0)
-                continue;
-            const anchorRun = oldRuns.find((run) => s.start >= run.start && s.start < run.end) ?? oldRuns[oldRuns.length - 1];
-            const newStart = s.start + deltaBefore(s.start);
-            await applyRange(newStart, newStart + s.replaceLen, inlineFromSnapshotRun(anchorRun, normalize));
-        }
-        return applied > 0 ? { restoredRunRanges: applied } : null;
-    }
-    catch (err) {
-        return { skipped: reasonOf(err) };
-    }
-}
 const NAMED_COLORS = {
     black: '000000',
     blue: '0000FF',
@@ -4397,6 +4300,7 @@ function parseScopedReplaceArgs(args, changeMode) {
     };
 }
 async function runScopedReplaceUnchecked(doc, args) {
+    assertReplacementTextPayloads(args.edits);
     const pre = await docSnapshot.buildMutationSnapshot(doc, { includeDomains: ['blocks', 'tables'] });
     const matched = docSnapshot.resolveSnapshotSelector(pre, args.selector);
     if (matched.length !== 1) {
@@ -4462,12 +4366,8 @@ async function runScopedReplaceUnchecked(doc, args) {
     // "requires a ref produced by query.match/find".
     const expected = planned.reduceRight((text, span) => text.slice(0, span.start) + span.replace + text.slice(span.end), block.text);
     const tracked = (args.changeMode ?? 'direct') === 'tracked';
-    // Tracked mode: redline ONLY the spans. A whole-block rewrite in tracked
-    // mode records the entire paragraph as deleted+reinserted, which buries a
-    // one-word edit in a full-paragraph redline. Each span becomes a
-    // select-scoped rewrite pinned to this block via `within`; duplicate find
-    // strings would re-resolve ambiguously against tracked-visible deleted
-    // text, so those fall back to the whole-block path below.
+    // Unique tracked finds can be resolved within the block. Repeated finds
+    // use one whole-block rewrite, whose engine diff preserves unchanged runs.
     let result = null;
     // Dedupe on the same basis the selector will match on: under a
     // case-sensitive request `Foo` and `foo` are distinct targets, and folding
@@ -4487,26 +4387,18 @@ async function runScopedReplaceUnchecked(doc, args) {
                 require: 'first',
                 within: { kind: 'block', nodeType: block.nodeType, nodeId },
             },
-            args: { replacement: { text: span.replace } },
+            args: { replacement: { text: span.replace }, style: preserveRewriteStyle() },
         }));
-        // Restore the known edit spans directly so the formatting pass adds no
-        // separate tracked change.
-        const trackedPreRuns = await captureBlockRuns(doc, nodeId, block.text);
         try {
             result = await executeMutations(doc, spanSteps, args.changeMode, pre.revision);
-            if (result != null)
-                await preserveScopedRunPattern(doc, nodeId, block.text, trackedPreRuns, planned);
         }
         catch {
             result = null; // select lane unavailable (e.g. case mismatch) — fall back
         }
     }
     if (result == null) {
-        // Direct mode (or tracked fallback): rewrite the whole block once, then
-        // restore the intra-paragraph run pattern span-aware — every interval the
-        // spans did NOT touch keeps its original run formatting at the mapped
-        // offsets, and each replacement inherits the marks of the run it starts in.
-        const preRuns = await captureBlockRuns(doc, nodeId, block.text);
+        // The engine preserves runs while applying the text diff. A separate
+        // formatting pass would restyle unchanged text and split the undo unit.
         const steps = [
             {
                 id: 'scoped-replace-1',
@@ -4516,7 +4408,6 @@ async function runScopedReplaceUnchecked(doc, args) {
             },
         ];
         result = await executeMutations(doc, steps, args.changeMode, pre.revision);
-        await preserveScopedRunPattern(doc, nodeId, block.text, preRuns, planned);
     }
     const post = await docSnapshot.buildMutationSnapshot(doc, { includeDomains: ['blocks', 'tables'] });
     const postBlock = findTextTarget(post, nodeId);

package/dist/agent/actions.js

@@ -515,6 +515,17 @@ function evaluateChecks(pre, post, checks) {
     }
     return results;
 }
+function assertReplacementTextPayloads(edits) {
+    for (const [index, edit] of edits.entries()) {
+        const text = edit.replace ?? '';
+        for (let offset = 0; offset < text.length; offset += 1) {
+            const code = text.charCodeAt(offset);
+            if (code < 0x20 && code !== 0x09 && code !== 0x0a) {
+                throw new SuperDocCliError(`edits[${index}].replace contains unsupported control character U+${code.toString(16).padStart(4, '0')}. Use plain text, tabs or preserved manual line breaks.`, { code: 'INVALID_PAYLOAD' });
+            }
+        }
+    }
+}
 function preserveRewriteStyle() {
     return {
         inline: { mode: 'preserve' },
@@ -961,6 +972,7 @@ async function runReplaceTextUnchecked(doc, args) {
         ? await buildMutationSnapshot(doc, { includeDomains: selectorDomains ?? ['blocks'] })
         : snapshotFromIdentity(preIdentity);
     try {
+        assertReplacementTextPayloads(args.edits);
         if (args.edits.length === 0) {
             return failedReceipt('replace_text', new Error('edits must be non-empty'), pre);
         }
@@ -2490,115 +2502,6 @@ async function applyInlineQuiet(formatFn, params) {
         throw err;
     }
 }
-/** Best-effort read of a block's current run layout (null when unavailable). */
-async function captureBlockRuns(doc, nodeId, text) {
-    const queryFn = maybeMethod(doc, ['query', 'match']);
-    const trimmed = text.trim();
-    if (!queryFn || !trimmed || text.includes('\n'))
-        return null;
-    try {
-        const runs = await matchRunsForBlock(queryFn, trimmed, nodeId);
-        return runs.length > 0 ? runs : null;
-    }
-    catch {
-        return null;
-    }
-}
-/**
- * Restore scoped replacements from their known spans: each untouched interval
- * keeps its run formatting, and replacements inherit the marks at their start.
- * `spans` are FULL-TEXT coordinates, ascending, non-overlapping.
- */
-async function preserveScopedRunPattern(doc, nodeId, oldText, oldRuns, spans) {
-    if (!oldRuns || oldRuns.length === 0 || spans.length === 0)
-        return null;
-    if (!oldRuns.some((run) => Object.keys(inlineFromSnapshotRun(run, new Set())).length > 0))
-        return null;
-    const formatFn = maybeMethod(doc, ['format', 'apply']);
-    if (!formatFn)
-        return null;
-    const normalize = new Set(RUN_BOOL_MARKS.filter((key) => oldRuns.some((run) => run[key])));
-    // Run offsets (captureBlockRuns) are relative to the TRIMMED text; spans are
-    // full-text. Convert spans to trimmed coordinates and clamp.
-    const oldTrimStart = oldText.length - oldText.trimStart().length;
-    const oldTrimLen = oldText.trim().length;
-    const trimSpans = spans
-        .map((s) => ({
-        start: Math.max(0, s.start - oldTrimStart),
-        end: Math.min(oldTrimLen, s.end - oldTrimStart),
-        replaceLen: (s.replace ?? '').length,
-    }))
-        .filter((s) => s.end > s.start || s.replaceLen > 0);
-    if (trimSpans.length === 0)
-        return null;
-    const newText = spans.reduceRight((text, span) => text.slice(0, span.start) + span.replace + text.slice(span.end), oldText);
-    const newTrimStart = newText.length - newText.trimStart().length;
-    // Cumulative old→new offset shift for positions AT OR AFTER a given old offset.
-    const deltaBefore = (oldPos) => {
-        let d = 0;
-        for (const s of trimSpans) {
-            if (s.end <= oldPos)
-                d += s.replaceLen - (s.end - s.start);
-            else
-                break;
-        }
-        return d;
-    };
-    let applied = 0;
-    const applyRange = async (start, end, inline) => {
-        if (end <= start || Object.keys(inline).length === 0)
-            return;
-        // Always direct, even under tracked mode. This pass RESTORES the marks the
-        // rewrite flattened; it is not new authorial formatting, and the text
-        // revision already owns the span's accept/reject fate. Applying it tracked
-        // adds a second reviewable change per rewrite (SD-4309's conformance matrix
-        // counts one per action), and rejecting that change alone would strand the
-        // text in the flattened look the rewrite produced.
-        const changed = await applyInlineQuiet(formatFn, {
-            blockId: nodeId,
-            start: newTrimStart + start,
-            end: newTrimStart + end,
-            inline,
-        });
-        if (changed)
-            applied += 1;
-    };
-    try {
-        // Untouched intervals between/around the spans, in old trimmed coords.
-        const intervals = [];
-        let cursor = 0;
-        for (const s of trimSpans) {
-            if (s.start > cursor)
-                intervals.push({ start: cursor, end: s.start });
-            cursor = Math.max(cursor, s.end);
-        }
-        if (cursor < oldTrimLen)
-            intervals.push({ start: cursor, end: oldTrimLen });
-        for (const run of oldRuns) {
-            const inline = inlineFromSnapshotRun(run, normalize);
-            for (const iv of intervals) {
-                const a = Math.max(run.start, iv.start);
-                const b = Math.min(run.end, iv.end);
-                if (b <= a)
-                    continue;
-                const d = deltaBefore(a); // constant across an untouched interval
-                await applyRange(a + d, b + d, inline);
-            }
-        }
-        // Each replacement span inherits the marks of the run it starts in.
-        for (const s of trimSpans) {
-            if (s.replaceLen === 0)
-                continue;
-            const anchorRun = oldRuns.find((run) => s.start >= run.start && s.start < run.end) ?? oldRuns[oldRuns.length - 1];
-            const newStart = s.start + deltaBefore(s.start);
-            await applyRange(newStart, newStart + s.replaceLen, inlineFromSnapshotRun(anchorRun, normalize));
-        }
-        return applied > 0 ? { restoredRunRanges: applied } : null;
-    }
-    catch (err) {
-        return { skipped: reasonOf(err) };
-    }
-}
 const NAMED_COLORS = {
     black: '000000',
     blue: '0000FF',
@@ -4394,6 +4297,7 @@ function parseScopedReplaceArgs(args, changeMode) {
     };
 }
 async function runScopedReplaceUnchecked(doc, args) {
+    assertReplacementTextPayloads(args.edits);
     const pre = await buildMutationSnapshot(doc, { includeDomains: ['blocks', 'tables'] });
     const matched = resolveSnapshotSelector(pre, args.selector);
     if (matched.length !== 1) {
@@ -4459,12 +4363,8 @@ async function runScopedReplaceUnchecked(doc, args) {
     // "requires a ref produced by query.match/find".
     const expected = planned.reduceRight((text, span) => text.slice(0, span.start) + span.replace + text.slice(span.end), block.text);
     const tracked = (args.changeMode ?? 'direct') === 'tracked';
-    // Tracked mode: redline ONLY the spans. A whole-block rewrite in tracked
-    // mode records the entire paragraph as deleted+reinserted, which buries a
-    // one-word edit in a full-paragraph redline. Each span becomes a
-    // select-scoped rewrite pinned to this block via `within`; duplicate find
-    // strings would re-resolve ambiguously against tracked-visible deleted
-    // text, so those fall back to the whole-block path below.
+    // Unique tracked finds can be resolved within the block. Repeated finds
+    // use one whole-block rewrite, whose engine diff preserves unchanged runs.
     let result = null;
     // Dedupe on the same basis the selector will match on: under a
     // case-sensitive request `Foo` and `foo` are distinct targets, and folding
@@ -4484,26 +4384,18 @@ async function runScopedReplaceUnchecked(doc, args) {
                 require: 'first',
                 within: { kind: 'block', nodeType: block.nodeType, nodeId },
             },
-            args: { replacement: { text: span.replace } },
+            args: { replacement: { text: span.replace }, style: preserveRewriteStyle() },
         }));
-        // Restore the known edit spans directly so the formatting pass adds no
-        // separate tracked change.
-        const trackedPreRuns = await captureBlockRuns(doc, nodeId, block.text);
         try {
             result = await executeMutations(doc, spanSteps, args.changeMode, pre.revision);
-            if (result != null)
-                await preserveScopedRunPattern(doc, nodeId, block.text, trackedPreRuns, planned);
         }
         catch {
             result = null; // select lane unavailable (e.g. case mismatch) — fall back
         }
     }
     if (result == null) {
-        // Direct mode (or tracked fallback): rewrite the whole block once, then
-        // restore the intra-paragraph run pattern span-aware — every interval the
-        // spans did NOT touch keeps its original run formatting at the mapped
-        // offsets, and each replacement inherits the marks of the run it starts in.
-        const preRuns = await captureBlockRuns(doc, nodeId, block.text);
+        // The engine preserves runs while applying the text diff. A separate
+        // formatting pass would restyle unchanged text and split the undo unit.
         const steps = [
             {
                 id: 'scoped-replace-1',
@@ -4513,7 +4405,6 @@ async function runScopedReplaceUnchecked(doc, args) {
             },
         ];
         result = await executeMutations(doc, steps, args.changeMode, pre.revision);
-        await preserveScopedRunPattern(doc, nodeId, block.text, preRuns, planned);
     }
     const post = await buildMutationSnapshot(doc, { includeDomains: ['blocks', 'tables'] });
     const postBlock = findTextTarget(post, nodeId);

package/dist/generated/client.cjs

@@ -108,7 +108,9 @@ function createDocApi(runtime) {
         },
         styles: {
             apply: (params = {}, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.apply"], params, options),
+            create: (params, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.create"], params, options),
             getCatalog: (params = {}, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.getCatalog"], params, options),
+            getTheme: (params = {}, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.getTheme"], params, options),
             paragraph: {
                 setStyle: (params, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.paragraph.setStyle"], params, options),
                 setStyleRef: (params, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.paragraph.setStyleRef"], params, options),
@@ -689,7 +691,9 @@ function createBoundDocApi(runtime) {
         },
         styles: {
             apply: (params = {}, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.apply"], params, options),
+            create: (params, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.create"], params, options),
             getCatalog: (params = {}, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.getCatalog"], params, options),
+            getTheme: (params = {}, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.getTheme"], params, options),
             paragraph: {
                 setStyle: (params, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.paragraph.setStyle"], params, options),
                 setStyleRef: (params, options) => runtime.invoke(contract.CONTRACT.operations["doc.styles.paragraph.setStyleRef"], params, options),