fix: dirty fix for char width

.zed/tasks.json

@@ -1,9 +1,10 @@
-// Static tasks configuration.
-//
-// Example:
 [
   {
     "label": "Test",
     "command": "deno test"
+  },
+  {
+    "label": "Test - Watch",
+    "command": "deno test --watch"
   }
 ]

deno.json

@@ -1,8 +1,9 @@
 {
   "name": "@textplace/core",
-  "version": "0.3.0",
+  "version": "0.4.1",
   "exports": "./mod.ts",
   "imports": {
-    "@deno/dnt": "jsr:@deno/dnt@^0.41.3"
+    "@deno/dnt": "jsr:@deno/dnt@^0.41.3",
+    "@std/cli": "jsr:@std/cli@^1.0.11"
   }
 }

deno.lock

@@ -7,6 +7,7 @@
     "jsr:@std/assert@0.223": "0.223.0",
     "jsr:@std/assert@0.226": "0.226.0",
     "jsr:@std/bytes@0.223": "0.223.0",
+    "jsr:@std/cli@^1.0.11": "1.0.11",
     "jsr:@std/fmt@0.223": "0.223.0",
     "jsr:@std/fmt@1": "1.0.3",
     "jsr:@std/fs@0.223": "0.223.0",
@@ -54,6 +55,9 @@
     "@std/bytes@0.223.0": {
       "integrity": "84b75052cd8680942c397c2631318772b295019098f40aac5c36cead4cba51a8"
     },
+    "@std/cli@1.0.11": {
+      "integrity": "ec219619fdcd31bcf0d8e53bee1e2706ec9a02f70255365a094f69755dadd340"
+    },
     "@std/fmt@0.223.0": {
       "integrity": "6deb37794127dfc7d7bded2586b9fc6f5d50e62a8134846608baf71ffc1a5208"
     },
@@ -151,7 +155,8 @@
   },
   "workspace": {
     "dependencies": [
-      "jsr:@deno/dnt@~0.41.3"
+      "jsr:@deno/dnt@~0.41.3",
+      "jsr:@std/cli@^1.0.11"
     ]
   }
 }

logic/board.ts

@@ -106,9 +106,9 @@ export function renderFullBoard(data: BoardData): FullBoard {
   return {
     w: lineLength,
     h: totalLineCount,
-    ch: ([] as string[]).concat(...chLines).flat(),
+    ch: chLines.flat(),
-    color: ([] as string[]).concat(...colorLines).flat(),
+    color: colorLines.flat(),
-    bg_color: ([] as string[]).concat(...bgColorLines).flat(),
+    bg_color: bgColorLines.flat(),
-    width: ([] as number[]).concat(...widthLines).flat(),
+    width: widthLines.flat(),
   };
 }

logic/character.ts

@@ -1,7 +1,6 @@
+import { unicodeWidth } from "@std/cli/unicode-width";
+
 const segmenter = new Intl.Segmenter("en", { granularity: "grapheme" });
-const cjkRegex =
-  /[\p{Unified_Ideograph}\u30A0-\u30FF\u3040-\u309F\u31F0-\u31FF]/u;
-const printableASCIIRegex = /^[\x20-\x7E]$/;
 
 export function getCharacterWidth(ch: string): number {
   const segments = [...segmenter.segment(ch)];
@@ -11,11 +10,6 @@ export function getCharacterWidth(ch: string): number {
     );
   }
 
-  const matchesASCII = ch.match(printableASCIIRegex);
+  // TODO: Properly fix this.
-  const matchesCJK = ch.match(cjkRegex);
+  return Math.min(unicodeWidth(ch), 2);
-
-  if (!matchesASCII && !matchesCJK) throw new Error(`Invalid character: ${ch}`);
-
-  // TODO: Support Emojis.
-  return matchesCJK ? 2 : 1;
 }

logic/section.ts

@@ -16,17 +16,17 @@ export function createSection(
   const offsetX = sx * boardConfig.sectionWidth;
   const offsetY = sy * boardConfig.sectionHeight;
 
-  const ch: string[][] = Array(boardConfig.sectionHeight).fill(
+  const ch: string[][] = Array(boardConfig.sectionHeight).fill([]).map(() =>
-    Array(boardConfig.sectionWidth).fill(boardConfig.defaultCh),
+    Array(boardConfig.sectionWidth).fill(boardConfig.defaultCh)
   );
-  const color: string[][] = Array(boardConfig.sectionHeight).fill(
+  const color: string[][] = Array(boardConfig.sectionHeight).fill([]).map(() =>
-    Array(boardConfig.sectionWidth).fill(boardConfig.defaultColor),
+    Array(boardConfig.sectionWidth).fill(boardConfig.defaultColor)
   );
-  const bgColor: string[][] = Array(boardConfig.sectionHeight).fill(
+  const bgColor: string[][] = Array(boardConfig.sectionHeight).fill([]).map(
-    Array(boardConfig.sectionWidth).fill(boardConfig.defaultBgColor),
+    () => Array(boardConfig.sectionWidth).fill(boardConfig.defaultBgColor),
   );
-  const width: number[][] = Array(boardConfig.sectionHeight).fill(
+  const width: number[][] = Array(boardConfig.sectionHeight).fill([]).map(() =>
-    Array(boardConfig.sectionWidth).fill(boardConfig.defaultWidth),
+    Array(boardConfig.sectionWidth).fill(boardConfig.defaultWidth)
   );
 
   return { offsetX, offsetY, ch, color, bgColor, width };
@@ -44,7 +44,7 @@ export function applyChange(change: BoardChange, section: SectionData) {
 
   if (change.ch) {
     const chWidth = getCharacterWidth(change.ch);
-    const xCharacterOffset = xInSection % 2;
+    const xCharacterOffset = xInSection % chWidth;
     const offsetAdjustedXInSection = xInSection - xCharacterOffset;
     section.ch[yInSection][offsetAdjustedXInSection] = change.ch;
     section.width[yInSection][offsetAdjustedXInSection] = chWidth;

tests/board.test.ts

@@ -51,11 +51,12 @@ Deno.test("board", async (t) => {
     applyChangeOnBoard({ x: 4, y: 0, ch: "B" }, board);
     applyChangeOnBoard({ x: 0, y: 3, ch: "C" }, board);
     applyChangeOnBoard({ x: 4, y: 3, ch: "D" }, board);
+    applyChangeOnBoard({ x: 5, y: 3, ch: "E" }, board);
 
     assertEquals(board.sections[0][0].ch[0][0], "A");
     assertEquals(board.sections[0][1].ch[0][0], "B");
     assertEquals(board.sections[1][0].ch[0][0], "C");
-    assertEquals(board.sections[1][1].ch[0][0], "D");
+    assertEquals(board.sections[1][1].ch[0], ["D", "E", " ", " "]);
 
     applyChangeOnBoard({ x: 0, y: 1, ch: "你" }, board);
     applyChangeOnBoard({ x: 4, y: 2, ch: "好" }, board);
@@ -65,7 +66,10 @@ Deno.test("board", async (t) => {
     assertEquals(board.sections[0][0].ch[1][0], "你");
     assertEquals(board.sections[0][1].ch[2][0], "好");
     assertEquals(board.sections[1][0].ch[1][0], "嗎");
-    assertEquals(board.sections[1][1].ch[2][0], "嘛");
+    assertEquals(board.sections[1][1].ch[1], ["嘛", " ", " ", " "]);
+
+    applyChangeOnBoard({ x: 5, y: 4, ch: "啊" }, board);
+    assertEquals(board.sections[1][1].ch[1], ["啊", " ", " ", " "]);
   });
 
   await t.step("getSectionOnBoard: existing section", () => {
@@ -74,10 +78,10 @@ Deno.test("board", async (t) => {
     const section = getSectionOnBoard({ sx: 1, sy: 1 }, board, {
       readOnly: true,
     });
-    assertEquals(section.ch[0][0], "嘛");
+    assertEquals(section.ch[0], ["D", "E", " ", " "]);
     assertEquals(section.color[0][0], "F");
     assertEquals(section.bgColor[0][0], "0");
-    assertEquals(section.width[0][0], 2);
+    assertEquals(section.width[0], [1, 1, 1, 1]);
   });
 
   await t.step("getSectionOnBoard: non-existing row", () => {
@@ -114,7 +118,7 @@ Deno.test("board", async (t) => {
   await t.step("on-demand creation: only changed sections are saved", () => {
     assert(board);
 
-    assertEquals(board.sections.length, 2);
+    assertEquals(board.sections[2], undefined);
-    assertEquals(board.sections[0].length, 2);
+    assertEquals(board.sections[0][2], undefined);
   });
 });

tests/character.test.ts

@@ -26,14 +26,18 @@ Deno.test("getCharacterWidth CJK", () => {
   assertEquals(getCharacterWidth("グ"), 2);
   assertEquals(getCharacterWidth("ソ"), 2);
 
-  assertThrows(() => getCharacterWidth("？"));
+  assertEquals(getCharacterWidth("？"), 2);
-  assertThrows(() => getCharacterWidth("！"));
+  assertEquals(getCharacterWidth("！"), 2);
   assertThrows(() => getCharacterWidth("你好"));
   assertThrows(() => getCharacterWidth("ヨスガノ"));
 });
 
+Deno.test("getCharacterWidth Emoji", () => {
+  assertEquals(getCharacterWidth("👋"), 2);
+  assertEquals(getCharacterWidth("🌲️"), 2);
+  assertEquals(getCharacterWidth("👨‍👩‍👧‍👦"), 2);
+});
+
 Deno.test("getCharacterWidth previously faulty cases", () => {
   assertEquals(getCharacterWidth("𤲶"), 2);
-
-  assertThrows(() => getCharacterWidth("𤲶"[0]));
 });

tests/checkFullBoard.ts

@@ -47,6 +47,11 @@ export function checkFullBoard(board: FullBoard) {
       console.error("width:   ", widthLine);
     };
 
+    if (typeof cCh !== "string") {
+      printSituation();
+      throw new Error("cCh is not string");
+    }
+
     if (!isValidColor(cCo) || !isValidColor(cBg)) {
       printSituation();
       throw new Error("cCo or cBg is not valid");

tests/section.test.ts

@@ -90,27 +90,33 @@ Deno.test("section", async (t) => {
     assert(section);
 
     applyChange({ x: 0, y: 0, ch: "t" }, section);
-    assertEquals(section.ch[0][0], "t");
+    assertEquals(section.ch[0], ["t", " ", " ", " "]);
-    assertEquals(section.ch[0][1], " ");
+    assertEquals(section.ch[1], [" ", " ", " ", " "]);
-    assertEquals(section.width[0][0], 1);
+    assertEquals(section.width[0], [1, 1, 1, 1]);
+  });
+
+  await t.step("applyChange 1-width at odd position", () => {
+    assert(section);
+
+    applyChange({ x: 1, y: 0, ch: "t" }, section);
+    assertEquals(section.ch[0], ["t", "t", " ", " "]);
+    assertEquals(section.width[0], [1, 1, 1, 1]);
   });
 
   await t.step("applyChange 2-width at a correct position", () => {
     assert(section);
 
     applyChange({ x: 0, y: 0, ch: "あ" }, section);
-    assertEquals(section.ch[0][0], "あ");
+    assertEquals(section.ch[0], ["あ", "t", " ", " "]);
-    assertEquals(section.ch[0][1], " ");
+    assertEquals(section.width[0], [2, 1, 1, 1]);
-    assertEquals(section.width[0][0], 2);
   });
 
   await t.step("applyChange 2-width at an alternate position", () => {
     assert(section);
 
     applyChange({ x: 1, y: 0, ch: "あ" }, section);
-    assertEquals(section.ch[0][0], "あ");
+    assertEquals(section.ch[0], ["あ", "t", " ", " "]);
-    assertEquals(section.ch[0][1], " ");
+    assertEquals(section.width[0], [2, 1, 1, 1]);
-    assertEquals(section.width[0][0], 2);
   });
 
   await t.step("applyChange incorrect section", () => {