@mrsf/cli 0.7.1 → 0.8.0

package/dist/commands/add.js

@@ -56,8 +56,8 @@ export function registerAdd(program) {
         .requiredOption("-t, --text <text>", "Comment text")
         .option("-l, --line <n>", "Line number (1-based)")
         .option("--end-line <n>", "End line (inclusive)")
-        .option("--start-column <n>", "Start column (0-based)")
-        .option("--end-column <n>", "End column (0-based)")
+        .option("--start-column <n>", "Inclusive UTF-16 column (0-based)")
+        .option("--end-column <n>", "Exclusive UTF-16 column (0-based)")
         .option("--type <type>", "Comment type (e.g. suggestion, issue)")
         .option("--severity <level>", "Severity: low | medium | high")
         .option("--reply-to <id>", "Reply to an existing comment")
@@ -116,7 +116,7 @@ export function registerAdd(program) {
         else if (comment.line != null) {
             try {
                 const lines = await readDocumentLines(docPath);
-                populateSelectedText(comment, lines);
+                populateSelectedText(comment, lines.slice(1));
             }
             catch {
                 // Document file may not exist yet, skip

package/dist/index.cjs

@@ -381,6 +381,7 @@ init_parser();
 var import_promises3 = require("node:fs/promises");
 var import_node_fs2 = require("node:fs");
 var import_node_crypto = require("node:crypto");
+var import_promises4 = require("node:timers/promises");
 var import_node_path4 = __toESM(require("node:path"), 1);
 var import_yaml2 = require("yaml");
 init_serialize();
@@ -401,7 +402,17 @@ async function atomicWriteFile(filePath, content) {
   const tmp = filePath + "." + (0, import_node_crypto.randomBytes)(6).toString("hex") + ".tmp";
   try {
     await (0, import_promises3.writeFile)(tmp, content, "utf-8");
-    await (0, import_promises3.rename)(tmp, filePath);
+    for (let attempt = 0; ; attempt++) {
+      try {
+        await (0, import_promises3.rename)(tmp, filePath);
+        break;
+      } catch (error) {
+        if (attempt >= 5 || !(error instanceof Error) || !("code" in error) || !["EPERM", "EACCES", "EBUSY"].some((code) => code === error.code)) {
+          throw error;
+        }
+        await (0, import_promises4.setTimeout)(50 * (attempt + 1));
+      }
+    }
   } catch (err) {
     try {
       await (0, import_promises3.unlink)(tmp);
@@ -711,7 +722,7 @@ async function writeSidecarInternal(abs, doc) {
 }
 
 // src/lib/validator.ts
-var import_promises4 = require("node:fs/promises");
+var import_promises5 = require("node:fs/promises");
 var import_node_path5 = __toESM(require("node:path"), 1);
 var import_node_url = require("node:url");
 
@@ -805,12 +816,12 @@ var mrsf_schema_default = {
           start_column: {
             type: "integer",
             minimum: 0,
-            description: "Starting column index (0-based) within the starting line."
+            description: "Inclusive starting UTF-16 code-unit offset (0-based) within the starting line."
           },
           end_column: {
             type: "integer",
             minimum: 0,
-            description: "Ending column index. MUST be >= start_column when on the same line."
+            description: "Exclusive ending UTF-16 code-unit offset (0-based) within the ending line. MUST be >= start_column when on the same line."
           },
           selected_text: {
             type: "string",
@@ -975,7 +986,7 @@ async function loadSchema() {
   ];
   for (const candidate of candidates) {
     try {
-      const raw = await (0, import_promises4.readFile)(candidate, "utf-8");
+      const raw = await (0, import_promises5.readFile)(candidate, "utf-8");
       _schemaCache = JSON.parse(raw);
       return _schemaCache;
     } catch {
@@ -2864,7 +2875,7 @@ async function reanchorDocument(doc, documentLines, opts = {}) {
   if (!opts.noGit && await isGitAvailable()) {
     const repoRoot = opts.repoRoot ?? await findRepoRoot(opts.cwd);
     if (repoRoot && opts.documentPath) {
-      const relPath = import_node_path6.default.relative(repoRoot, opts.documentPath);
+      const relPath = import_node_path6.default.relative(repoRoot, opts.documentPath).split(import_node_path6.default.sep).join("/");
       const head = await getCurrentCommit(repoRoot);
       const globalFrom = opts.fromCommit;
       const diffCache = /* @__PURE__ */ new Map();