overleaf-review 0.6.0 → 0.6.1

package/README.md

@@ -212,6 +212,13 @@ Grouping applies to new pushes; it does not consolidate or accept suggestions al
 ### Holistic paragraph rewrites over reviewed suggestions and comments
 
 Use this when a paragraph rewrite intentionally incorporates or supersedes existing suggestions.
+
+To regroup existing fragmented suggestions **without changing their proposed wording**, set
+`before` and `after` to the same paragraph text and explicitly list the suggestion IDs in
+`supersedes`. Keep the local file unchanged for that block. This is supported from v0.6.1;
+the proposal must still differ from the underlying text obtained by rejecting those suggestions.
+Affected comments still require explicit anchor mappings, and all stale-plan and verification
+checks still apply. Ordinary replacements without selected suggestions cannot be no-ops.
 It differs from consolidation: the proposed text may change, and selected older suggestion cards
 are replaced by a new pending revision under the authenticated account.
 

package/dist/{chunk-TWIDV42A.js → chunk-JRXROCS6.js}

@@ -924,11 +924,11 @@ function buildReviewEdits(source, target, options = {}) {
   }
   return groups;
 }
-function validateExplicitEdits(source, value) {
+function validateExplicitEdits(source, value, validation = {}) {
   if (!Array.isArray(value) || !value.length) throw new Error("Explicit blocks must be a nonempty array.");
   let previousEnd = -1;
   return value.map((edit) => {
-    if (!edit || !Number.isSafeInteger(edit.start) || !Number.isSafeInteger(edit.end) || edit.start < 0 || edit.end < edit.start || edit.end > source.length || typeof edit.text !== "string" || edit.start <= previousEnd || source.slice(edit.start, edit.end) === edit.text) {
+    if (!edit || !Number.isSafeInteger(edit.start) || !Number.isSafeInteger(edit.end) || edit.start < 0 || edit.end < edit.start || edit.end > source.length || typeof edit.text !== "string" || edit.start <= previousEnd || !validation.allowUnchanged && source.slice(edit.start, edit.end) === edit.text) {
       throw new Error("Explicit blocks must be valid, ordered, separated replacements with a text change.");
     }
     previousEnd = edit.end;
@@ -941,7 +941,7 @@ function applyTextEdits(source, edits) {
   }
   return source;
 }
-function parseReplacementManifest(base, value) {
+function parseReplacementManifest(base, value, validation = {}) {
   const manifest = value;
   if (!Array.isArray(manifest?.replacements) || !manifest.replacements.length) {
     throw new Error("Replacement manifest requires a nonempty replacements array.");
@@ -962,10 +962,10 @@ function parseReplacementManifest(base, value) {
     if (start === void 0) throw new Error("Replacement occurrence is absent from the saved base.");
     return { start, end: start + block.before.length, text: block.after };
   }).sort((a, b) => a.start - b.start);
-  return validateExplicitEdits(base, edits);
+  return validateExplicitEdits(base, edits, validation);
 }
-function bindExplicitEdits(base, local, live, edits, options) {
-  const checked = validateExplicitEdits(base, edits);
+function bindExplicitEdits(base, local, live, edits, options, validation = {}) {
+  const checked = validateExplicitEdits(base, edits, validation);
   if (applyTextEdits(base, checked) !== local) {
     throw new Error("Replacement manifest must describe every local change exactly.");
   }
@@ -2299,7 +2299,7 @@ ${doc.localPath} \u2192 ${doc.docPath} (v${doc.liveVersion}): ${doc.ops.length}
 }
 async function push(opts) {
   validatePushOptions(opts);
-  const { isReviewedManifest, createReviewedFilePlan, applyReviewedFilePlan, REVIEWED_FILE_KIND } = await import("./reviewed-replace-A7C4UNJS.js");
+  const { isReviewedManifest, createReviewedFilePlan, applyReviewedFilePlan, REVIEWED_FILE_KIND } = await import("./reviewed-replace-4PCULETF.js");
   if (opts.edits && isReviewedManifest(JSON.parse(readFileSync6(workspaceReadPath(opts.edits), "utf8")))) {
     await createReviewedFilePlan(opts);
     return;

package/dist/cli.js

@@ -39,7 +39,7 @@ import {
   uploadFile,
   validateSession,
   workspaceWritePath
-} from "./chunk-TWIDV42A.js";
+} from "./chunk-JRXROCS6.js";
 
 // src/commands/pull.ts
 import { writeFileSync, mkdirSync } from "fs";

package/dist/{reviewed-replace-A7C4UNJS.js → reviewed-replace-4PCULETF.js}

@@ -34,7 +34,7 @@ import {
   workspaceRelativePath,
   workspaceWritePath,
   writeJsonAtomic
-} from "./chunk-TWIDV42A.js";
+} from "./chunk-JRXROCS6.js";
 
 // src/commands/reviewed-replace.ts
 import { readFileSync } from "fs";
@@ -315,10 +315,13 @@ function resolveReviewedBlocks(base, local, live, value) {
   const manifest = value;
   if (!Array.isArray(manifest?.replacements) || !manifest.replacements.length) throw new Error("Reviewed manifest requires replacements.");
   const resolved = manifest.replacements.map((block) => {
-    const [edit] = parseReplacementManifest(base, { replacements: [{ before: block.before, after: block.after, occurrence: block.occurrence }] });
+    const allowUnchanged = Array.isArray(block.supersedes) && block.supersedes.length > 0;
+    const [edit] = parseReplacementManifest(base, {
+      replacements: [{ before: block.before, after: block.after, occurrence: block.occurrence }]
+    }, { allowUnchanged });
     return { ...edit, supersedes: block.supersedes ?? [], reason: block.reason, comments: block.comments ?? [] };
   }).sort((a, b) => a.start - b.start);
-  const mapped = bindExplicitEdits(base, local, live, resolved, {}).explicitEdits;
+  const mapped = bindExplicitEdits(base, local, live, resolved, {}, { allowUnchanged: true }).explicitEdits;
   return resolved.map((block, index) => ({ ...block, ...mapped[index] }));
 }
 async function createReviewedFilePlan(opts) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "overleaf-review",
-  "version": "0.6.0",
+  "version": "0.6.1",
   "description": "The missing review layer for Overleaf's Git bridge — sync comments and tracked changes between Overleaf and your local repo.",
   "type": "module",
   "license": "MIT",