reviewdeck 0.2.5 → 0.2.6

package/dist/reviewdeck.mjs

@@ -477,6 +477,23 @@ function formatIndexedChanges(changes) {
   }
   return lines.join("\n");
 }
+function collapseRanges(nums) {
+  if (nums.length === 0) return [];
+  const ranges = [];
+  let start = nums[0];
+  let end = start;
+  for (let i = 1; i < nums.length; i++) {
+    if (nums[i] === end + 1) {
+      end = nums[i];
+    } else {
+      ranges.push(start === end ? `${start}` : `${start}-${end}`);
+      start = nums[i];
+      end = start;
+    }
+  }
+  ranges.push(start === end ? `${start}` : `${start}-${end}`);
+  return ranges;
+}
 function validateMeta(meta, totalChanges) {
   const errors = [];
   const assigned = /* @__PURE__ */ new Set();
@@ -492,15 +509,27 @@ function validateMeta(meta, totalChanges) {
       errors.push(`Group ${g + 1}: ${e.message}`);
       continue;
     }
+    const outOfRange = [];
+    const duplicates = [];
     for (const idx of expanded) {
       if (idx < 0 || idx >= totalChanges) {
-        errors.push(`Group ${g + 1}: index ${idx} out of range [0, ${totalChanges - 1}]`);
+        outOfRange.push(idx);
       }
       if (assigned.has(idx)) {
-        errors.push(`Group ${g + 1}: index ${idx} already assigned to another group`);
+        duplicates.push(idx);
       }
       assigned.add(idx);
     }
+    if (outOfRange.length > 0) {
+      for (const range of collapseRanges(outOfRange)) {
+        errors.push(`Group ${g + 1}: index ${range} out of range [0, ${totalChanges - 1}]`);
+      }
+    }
+    if (duplicates.length > 0) {
+      for (const range of collapseRanges(duplicates)) {
+        errors.push(`Group ${g + 1}: index ${range} already assigned to another group`);
+      }
+    }
     const groupChanges = new Set(expanded);
     for (let c = 0; c < (group.draftComments?.length ?? 0); c++) {
       const comment = group.draftComments[c];
@@ -520,9 +549,13 @@ function validateMeta(meta, totalChanges) {
       }
     }
   }
+  const unassigned = [];
   for (let i = 0; i < totalChanges; i++) {
-    if (!assigned.has(i)) {
-      errors.push(`Change ${i} is not assigned to any group`);
+    if (!assigned.has(i)) unassigned.push(i);
+  }
+  if (unassigned.length > 0) {
+    for (const range of collapseRanges(unassigned)) {
+      errors.push(`Change ${range} is not assigned to any group`);
     }
   }
   return errors;

package/package.json

@@ -1,15 +1,15 @@
 {
   "name": "reviewdeck",
-  "version": "0.2.5",
+  "version": "0.2.6",
   "description": "Split large PR diffs into reviewable sub-patches via indexed groups.",
-  "repository": {
-    "type": "git",
-    "url": "https://github.com/neutree-ai/reviewdeck.git"
-  },
   "homepage": "https://github.com/neutree-ai/reviewdeck",
   "bugs": {
     "url": "https://github.com/neutree-ai/reviewdeck/issues"
   },
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/neutree-ai/reviewdeck.git"
+  },
   "bin": {
     "reviewdeck": "dist/reviewdeck.mjs"
   },