hanlint 0.0.14 → 0.0.15

package/data/version.json

@@ -1 +1 @@
-{"version": "0.0.14"}
+{"version": "0.0.15"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hanlint",
-  "version": "0.0.14",
+  "version": "0.0.15",
   "description": "한국어 글 검사기. Korean prose linter for Markdown: translationese, noun piles, double passives, weak structure. Zero dependencies",
   "type": "module",
   "bin": {

package/src/cli/main.js

@@ -1005,6 +1005,11 @@ function relativeLabel(path) {
   return label.split(sep).join("/").replaceAll("\\", "/");
 }
 
+/** 표의 자리 글자. 뜻은 파이썬 SheetRow.place 가 소유한다. @param {import("../report/sheet.js").SheetRow} row */
+function placeOf(row) {
+  return row.column > 0 ? `${row.file}:${row.line}:${row.column}` : `${row.file}:${row.line}`;
+}
+
 /** 표의 고침을 파일에 쓴다. 뜻은 파이썬 applySheet 가 소유한다. @param {string} sheetPath @param {boolean} dryRun @returns {[string[], string[]]} */
 function applySheet(sheetPath, dryRun) {
   const parsed = parseSheet(readFileSync(sheetPath, "utf-8"));
@@ -1020,21 +1025,33 @@ function applySheet(sheetPath, dryRun) {
   for (const file of [...byFile.keys()].sort()) {
     const rows = /** @type {import("../report/sheet.js").SheetRow[]} */ (byFile.get(file));
     if (!existsSync(file)) {
-      failed.push(...rows.map((row) => `${row.file}:${row.line}: 파일이 없다`));
+      failed.push(...rows.map((row) => `${placeOf(row)}: 파일이 없다`));
       continue;
     }
     const lines = readFileSync(file, "utf-8").split("\n");
     let changed = false;
-    for (const row of rows) {
-      const place = `${row.file}:${row.line}`;
+    // 한 줄에 고침이 여럿이면 뒤 칸부터. 앞 칸을 먼저 바꾸면 길이가 달라져 뒤 칸의 자리가 어긋난다.
+    for (const row of [...rows].sort((a, b) => a.line - b.line || b.column - a.column)) {
+      const place = placeOf(row);
       if (row.line < 1 || row.line > lines.length) {
         failed.push(`${place}: 그 줄이 없다`);
         continue;
       }
-      const [newLine, count] = replaceLiteral(lines[row.line - 1], row.text, row.fix);
-      if (count !== 1) {
-        failed.push(`${place}: 글이 그 줄에 ${count}번 있다. 한 번이어야 바꾼다`);
-        continue;
+      let newLine = lines[row.line - 1];
+      if (row.column > 0) {
+        const start = row.column - 1;
+        if (newLine.slice(start, start + row.text.length) !== row.text) {
+          failed.push(`${place}: 그 칸에 그 글이 없다. 표를 다시 뽑는다`);
+          continue;
+        }
+        newLine = newLine.slice(0, start) + row.fix + newLine.slice(start + row.text.length);
+      } else {
+        const [replaced, count] = replaceLiteral(newLine, row.text, row.fix);
+        if (count !== 1) {
+          failed.push(`${place}: 글이 그 줄에 ${count}번 있다. 한 번이어야 바꾼다`);
+          continue;
+        }
+        newLine = replaced;
       }
       lines[row.line - 1] = newLine;
       changed = true;
@@ -1078,7 +1095,7 @@ function runSheet(args) {
     const label = relativeLabel(file);
     for (const literal of sourceLiterals(readFileSync(file, "utf-8"), label)) {
       const findings = runAll(fingerprint(literal.plain, config), config).filter((finding) => finding.severity === "error");
-      if (findings.length || options["--all"]) rows.push({ file: label, line: literal.line, text: literal.text, findings, fix: "" });
+      if (findings.length || options["--all"]) rows.push({ file: label, line: literal.line, text: literal.text, findings, fix: "", column: literal.column });
     }
   }
   emit(format === "json" ? renderSheetJson(rows, config.preset, files.length) : renderSheet(rows, config.preset, files.length), output);

package/src/document/sourceText.js

@@ -20,6 +20,7 @@ const RUST_TEST_MARKER = "#[cfg(test)]";
  * @property {number} line 1부터 세는 줄 번호. 여러 줄을 이은 러스트 문자열은 시작 줄이다
  * @property {string} text 따옴표와 태그를 뺀 글. 파일에 있는 그대로라 되돌려 쓸 때 찾을 수 있다
  * @property {string} plain 검사에 쓰는 글. 식 (`${...}`, `{...}`) 을 비우고 공백을 하나로 모았다
+ * @property {number} column 1부터 세는 칸. 그 줄에서 text 가 시작하는 자리
  */
 
 /** 템플릿 리터럴의 `${ ... }` 식을 (중첩 괄호를 세어) 같은 길이의 빈칸으로 바꾼다. @param {string} source */
@@ -95,9 +96,9 @@ function tagCloses(line, index) {
   return "\"'}".includes(before) || /[A-Za-z0-9]/.test(before);
 }
 
-/** 한 줄의 글 마디를 나온 차례로. 뜻은 파이썬 lineLiterals 가 소유한다. @param {string} line @returns {string[]} */
+/** 한 줄의 글 마디를 [시작 자리, 글] 로 나온 차례로. 뜻은 파이썬 lineLiterals 가 소유한다. @param {string} line @returns {[number, string][]} */
 export function lineLiterals(line) {
-  /** @type {string[]} */
+  /** @type {[number, string][]} */
   const found = [];
   let index = 0;
   while (index < line.length) {
@@ -105,7 +106,7 @@ export function lineLiterals(line) {
     if (QUOTES.includes(char)) {
       const end = closingQuote(line, index);
       if (end < 0) break;
-      found.push(line.slice(index + 1, end));
+      found.push([index + 1, line.slice(index + 1, end)]);
       index = end + 1;
       continue;
     }
@@ -113,7 +114,7 @@ export function lineLiterals(line) {
       const end = line.indexOf("<", index + 1);
       if (end > index) {
         const inner = line.slice(index + 1, end);
-        found.push(inner);
+        found.push([index + 1, inner]);
         if (!inner.includes("{")) {
           index = end;
           continue;
@@ -130,10 +131,10 @@ export function sourceLiterals(source, path = "") {
   /** @type {SourceLiteral[]} */
   const found = [];
   userFacingSource(source, path).split("\n").forEach((line, index) => {
-    for (const raw of lineLiterals(line)) {
+    for (const [start, raw] of lineLiterals(line)) {
       const text = raw.trim();
       const plain = plainText(text);
-      if (text && KOREAN.test(plain)) found.push({ line: index + 1, text, plain });
+      if (text && KOREAN.test(plain)) found.push({ line: index + 1, text, plain, column: start + (raw.length - raw.trimStart().length) + 1 });
     }
   });
   return found;