fix-vietnamese-claude-code 1.0.2 → 1.0.3

package/README.md

@@ -1,75 +1,46 @@
 # Claude Code Vietnamese IME Fix
 
-Fix lỗi gõ tiếng Việt trong Claude Code CLI với các bộ gõ như OpenKey, EVKey, PHTV, Unikey... Hỗ trợ đa nền tảng (macOS và Windows).
+Fix lỗi gõ tiếng Việt trong Claude Code CLI cho các bộ gõ (OpenKey, EVKey, PHTV, Unikey...). Hỗ trợ cả phiên bản **npm** và **binary** (macOS, Windows, Linux).
 
-**Phiên bản đã test:** Claude Code v2.0.64 → v2.1.12
-
-## Vấn đề
-
-Khi gõ tiếng Việt trong Claude Code CLI, các bộ gõ sử dụng kỹ thuật "backspace rồi thay thế" để chuyển đổi ký tự (ví dụ: `a` + `s` → `á`). Claude Code xử lý phần backspace (ký tự DEL `\x7f`) nhưng không đưa ký tự thay thế vào đúng vị trí, dẫn đến:
-
-- Ký tự bị "nuốt" hoặc mất khi gõ.
-- Văn bản hiển thị không đúng với những gì đã gõ.
-- Gây khó khăn khi nhập liệu trực tiếp trong terminal.
-
-Script này patch tệp `cli.js` của Claude Code để xử lý đúng các ký tự tiếng Việt sau khi nhận tín hiệu xóa từ bộ gõ.
+**Phiên bản đã test:**
+- npm: v2.1.39
+- binary: v2.1.39
+(Chi tiết tại [CHANGELOG.md](./CHANGELOG.md))
 
 ## Cài đặt & Sử dụng
 
-> [!IMPORTANT]
-> **Yêu cầu:** Chỉ hỗ trợ phiên bản cài đặt qua **npm**. Nếu bạn cài Claude Code qua các đường dẫn khác (MSI installer, Homebrew binary), vui lòng gỡ cài đặt và cài lại qua npm:
-> ```bash
-> npm install -g @anthropic-ai/claude-code
-> ```
-
-### 2. Chạy patch
-
-Bạn có thể chạy trực tiếp bằng **npx** (không cần tải file):
+Gõ lệnh sau trong terminal để áp dụng bản vá:
 
 ```bash
 npx fix-vietnamese-claude-code
 ```
 
-Hoặc nếu bạn đã tải tệp `patch-cli-claude-code.js` về máy:
-
-```bash
-node patch-cli-claude-code.js
-```
-
-Script sẽ tự động tìm kiếm đường dẫn đến tệp `cli.js` của Claude Code trên hệ thống của bạn và áp dụng bản vá.
+Lệnh trên sẽ tự động tìm và vá tệp `cli.js` (npm) hoặc file binary của Claude Code trên hệ thống của bạn.
 
 ### Tùy chọn nâng cao
 
-Nếu script không tự động tìm thấy đường dẫn, bạn có thể chỉ định thủ công:
+Nếu cài đặt ở đường dẫn không mặc định, bạn có thể chỉ định thủ công:
 
 ```bash
-node patch-cli-claude-code.js -f "/đường/dẫn/đến/@anthropic-ai/claude-code/cli.js"
-```
+# Đối với bản npm cli.js
+npx fix-vietnamese-claude-code -f "/đường/dẫn/đến/cli.js"
 
-Xem hướng dẫn chi tiết:
-```bash
-node patch-cli-claude-code.js --help
+# Đối với bản binary
+npx fix-vietnamese-claude-code -f "/đường/dẫn/đến/claude"
 ```
 
 ## Lưu ý
+- **Cập nhật:** Bạn cần chạy lại lệnh patch mỗi khi Claude Code cập nhật phiên bản mới.
+- **Môi trường:** Đã kiểm tra tốt trên Windows (CMD/PowerShell), macOS và Linux.
 
-- **Cập nhật:** Mỗi khi Claude Code được cập nhật phiên bản mới, bạn cần chạy lại script patch này vì tệp `cli.js` sẽ bị ghi đè.
-- **Môi trường:** Đã kiểm tra và hoạt động tốt trên Windows (CMD/PowerShell) và macOS.
-
-## Phát triển (Dành cho Developer)
-
-Dự án sử dụng **Vitest** để kiểm tra tính đúng đắn của bản vá trên nhiều phiên bản Claude Code khác nhau.
+## Phát triển
+Dự án sử dụng **Vitest** để kiểm tra tính đúng đắn trên nhiều phiên bản.
 
 ```bash
-# Cài đặt dependencies
 npm install
-
-# Chạy test
 npm test
 ```
 
-Script test sẽ tự động tải các phiên bản thực tế của Claude Code từ npm (từ `2.0.64` trở đi) để đảm bảo regex luôn khớp.
-
 ## Credits
 
 Dự án tham khảo và cải tiến từ:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "fix-vietnamese-claude-code",
-  "version": "1.0.2",
+  "version": "1.0.3",
   "description": "Fix Vietnamese IME compatibility issues in Claude Code.",
   "bin": {
     "fix-vietnamese-claude-code": "patch-cli-claude-code.js"

package/patch-cli-claude-code.js

@@ -5,37 +5,7 @@ const fs = require('fs');
 const path = require('path');
 const os = require('os');
 
-/**
- * Original method stolen from this
- * https://github.com/manhit96/claude-code-vietnamese-fix
- * This patch fixes Vietnamese input method editors (IMEs) compatibility issues in Claude.
- * By modifying `cli.js` code from this:
- * ```js
- * if (!S.equals(CA)) {
- *     if (S.text !== CA.text) Q(CA.text);
- *     T(CA.offset)
- * }
- * ct1(), lt1();
- * return
- * ```
- * To this:
- * ```js
- * /* Vietnamese IME fix * /
- * let _vn = n.replace(/\x7f/g, "");
- * if (_vn.length > 0) {
- *   for (const _c of _vn) CA = CA.insert(_c);
- *   if (!S.equals(CA)) {
- *     if (S.text !== CA.text) Q(CA.text);
- *     T(CA.offset)
- *   }
- * }
- * Oe1(), Me1();
- * return
- * ```
- * Note: I have no idea how this works, just ported python logic to JS with make it better performance.
- */
-
-const DORK = '/* Vietnamese IME fix */';
+const DORK = '/* _0x0a0d_ime_fix_ */';
 
 function usage() {
     console.log(`
@@ -43,112 +13,261 @@ Usage:
   fix-claude-code-vn [options]
 
 Options:
-  -f, --file <path>   Path to Claude's cli.js file
+  -f, --file <_path_>   Path to cli.js or claude file
+  -d, --dry-run       Test without overwriting the file
+  -o, --output <path>  Write patched content to a new file
   -h, --help          Show this help message
 
 Description:
-  This script patches Claude Code's cli.js to fix Vietnamese IME issues.
+  This script patches Claude Code CLI tool to fix Vietnamese IME issues.
   If no file is specified, it will try to find it automatically.
     `);
 }
 
 function findClaudePath() {
-    // 1. Try which/where
-    try {
-        const cmd = os.platform() === 'win32' ? 'where claude' : 'which claude';
-        const binPath = execSync(cmd).toString().split('\n')[0].trim();
-        if (binPath) {
-            // On Unix, it might be a symlink, resolve it
-            let realPath = binPath;
-            if (os.platform() !== 'win32') {
+    const isWin = os.platform() === "win32";
+
+    const run = (cmd) => {
+        try {
+            return execSync(cmd, { stdio: ["ignore", "pipe", "ignore"] })
+              .toString()
+              .split(/\r?\n/)[0]
+              .trim();
+        } catch {
+            return "";
+        }
+    };
+
+    const exists = (p) => p && fs.existsSync(p);
+
+    // 1) which / where / bun which
+    for (const cmd of [
+        isWin ? "where claude" : "which claude",
+        "bun which claude",
+    ]) {
+        const p = run(cmd);
+        if (exists(p)) {
+            if (!isWin) {
                 try {
-                    realPath = execSync(`realpath "${binPath}"`).toString().trim();
-                } catch (e) { /* ignore */ }
+                    return execSync(`realpath "${ p }"`).toString().trim();
+                } catch {}
             }
+            return p;
+        }
+    }
 
-            if (realPath.endsWith('.js')) return realPath;
-            
-            // If it's a binary/shell script, try to find the node_modules
-            const dir = path.dirname(realPath);
-            const npmPath = path.join(dir, 'node_modules', '@anthropic-ai', 'claude-code', 'cli.js');
-            if (fs.existsSync(npmPath)) return npmPath;
+    // 2) Bun global paths
+    const bunInstall =
+      process.env.BUN_INSTALL ||
+      (isWin
+        ? path.join(process.env.USERPROFILE || "", ".bun")
+        : path.join(process.env.HOME || "", ".bun"));
+
+    const bunPaths = [
+        path.join(bunInstall, "bin", isWin ? "claude.exe" : "claude"),
+        path.join(bunInstall, "bin", isWin ? "claude.cmd" : "claude"),
+        path.join(
+          bunInstall,
+          "install",
+          "global",
+          "node_modules",
+          "@anthropic-ai",
+          "claude-code",
+          "cli.js"
+        ),
+    ];
+
+    for (const p of bunPaths) {
+        if (exists(p)) {
+            return p;
         }
-    } catch (e) { /* ignore */ }
+    }
 
-    // 2. Try npm root -g
+    // 3) npm global
     try {
-        const npmRoot = execSync('npm root -g').toString().trim();
-        const cliPath = path.join(npmRoot, '@anthropic-ai', 'claude-code', 'cli.js');
-        if (fs.existsSync(cliPath)) return cliPath;
-    } catch (e) { /* ignore */ }
-
-    // 3. Common paths for Windows
-    if (os.platform() === 'win32') {
-        const commonPaths = [
-            path.join(process.env.APPDATA || '', 'npm', 'node_modules', '@anthropic-ai', 'claude-code', 'cli.js'),
-            path.join(process.env.LOCALAPPDATA || '', 'npm', 'node_modules', '@anthropic-ai', 'claude-code', 'cli.js')
+        const npmRoot = execSync("npm root -g").toString().trim();
+
+        const cliPath = path.join(
+          npmRoot,
+          "@anthropic-ai",
+          "claude-code",
+          "cli.js"
+        );
+        if (exists(cliPath)) {
+
+            return cliPath;
+        }
+    } catch (e) {
+
+    }
+
+    // 4) Windows fallbacks
+    if (isWin) {
+
+        const paths = [
+            path.join(
+              process.env.APPDATA || "",
+              "npm",
+              "node_modules",
+              "@anthropic-ai",
+              "claude-code",
+              "cli.js"
+            ),
+            path.join(
+              process.env.LOCALAPPDATA || "",
+              "npm",
+              "node_modules",
+              "@anthropic-ai",
+              "claude-code",
+              "cli.js"
+            ),
         ];
-        
-        // nvm-windows
+
         if (process.env.NVM_HOME) {
             try {
-                const dirs = fs.readdirSync(process.env.NVM_HOME);
-                for (const d of dirs) {
-                    const p = path.join(process.env.NVM_HOME, d, 'node_modules', '@anthropic-ai', 'claude-code', 'cli.js');
-                    if (fs.existsSync(p)) commonPaths.push(p);
+                for (const d of fs.readdirSync(process.env.NVM_HOME)) {
+                    paths.push(
+                      path.join(
+                        process.env.NVM_HOME,
+                        d,
+                        "node_modules",
+                        "@anthropic-ai",
+                        "claude-code",
+                        "cli.js"
+                      )
+                    );
                 }
-            } catch (e) { /* ignore */ }
+            } catch (e) {
+
+            }
         }
 
-        for (const p of commonPaths) {
-            if (fs.existsSync(p)) return p;
+        for (const p of paths) {
+            if (exists(p)) {
+
+                return p;
+            }
         }
     }
 
+
     return null;
 }
 
-function patchContent(fileContent) {
+// stolen fixed solution from manhit96/claude-code-vietnamese-fix
+function patchContentJs(fileContent) {
     if (fileContent.includes(DORK)) {
-        return { success: true, alreadyPatched: true, content: fileContent };
+        return { success: true, alreadyPatched: true };
     }
 
     // Pattern matching:
     // match this: l.match(/\x7f/g)...if(!S.equals(CA)){if(S.text!==CA.text)Q(CA.text);T(CA.offset)}ct1(),lt1();return
     // We use a regex that captures variable and function names dynamically.
-    const re = /((?<var0>[\w$]+)\.match\(\/\\x7f\/g\).*?)if\(!(?<var1>[\w$]+)\.equals\((?<var2>[\w$]+)\)\){if\(\k<var1>\.text!==\k<var2>\.text\)(?<func1>[\w$]+)\(\k<var2>\.text\);(?<func2>[\w$]+)\(\k<var2>\.offset\)}(?<remain>(?:[\w$]+\(\),?\s*)*;?\s*return)/;
-    
-    const newContent = fileContent.replace(re, (match, m0, var0, var1, var2, func1, func2, remain) => {
+    const re = /(?<m0>(?<var0>[\w$]+)\.match\(\/\\x7f\/g\).*?)(?<m1>if\(!(?<var1>[\w$]+)\.equals\((?<var2>[\w$]+)\)\){if\(\k<var1>\.text!==\k<var2>\.text\)(?<func1>[\w$]+)\(\k<var2>\.text\);(?<func2>[\w$]+)\(\k<var2>\.offset\)})(?<m2>(?:[\w$]+\(\),?\s*)*;?\s*return)/g;
+
+    const newContent = fileContent.replace(re, (...args) => {
+        const { m0, m1, var0, var2, m2 } = args[args.length - 1];
         return `
 ${DORK}
 ${m0}
 let _vn = ${var0}.replace(/\\x7f/g, "");
 if (_vn.length > 0) {
     for (const _c of _vn) ${var2} = ${var2}.insert(_c);
-    if (!${var1}.equals(${var2})) {
-        if (${var1}.text !== ${var2}.text) ${func1}(${var2}.text);
-        ${func2}(${var2}.offset);
-    }
+    ${m1}
 }
-${remain}
+${m2}
 `;
     });
 
-    if (newContent === fileContent) {
-        return { success: false, content: fileContent };
+    if (newContent.length === fileContent.length) {
+        return { success: false, message: 'Patch failed: no match found' };
     }
 
     return { success: true, alreadyPatched: false, content: newContent };
 }
 
+function patchContentBinary(binaryContent) {
+    if (binaryContent.includes(DORK)) {
+        return { success: true, alreadyPatched: true };
+    }
+
+    const re = /(?<m0>(?<var0>[\w$]+)\.match\(\/\\x7f\/g\).*?)(?<m1>if\(!(?<var1>[\w$]+)\.equals\((?<var2>[\w$]+)\)\){if\(\k<var1>\.text!==\k<var2>\.text\)(?<func1>[\w$]+)\(\k<var2>\.text\);(?<func2>[\w$]+)\(\k<var2>\.offset\)})(?<m2>(?:[\w$]+\(\),?\s*)*;?\s*return)/g;
+
+    const matches = [];
+    binaryContent = binaryContent.replace(re, (...args) => {
+        const groups = args[args.length - 1];
+        const offset = args[args.length - 3];
+        const { m0, m1, var0, var2, m2 } = groups;
+
+        const patchedContent = `${DORK}
+${m0}
+let _vn = ${var0}.replace(/\\x7f/g, "");
+if (_vn.length > 0) {
+    for (const _c of _vn) ${var2} = ${var2}.insert(_c);
+    ${m1}
+}
+${m2}`.replace(/^\s+/gm, '');
+        matches.push({ diff: patchedContent.length - args[0].length, index: offset });
+        return patchedContent;
+    });
+
+    if (matches.length === 0) {
+        return { success: false, message: 'Patch failed: no match found' };
+    }
+
+    // now from index, we must look back for `\x00// @bun `
+    const pragma = `// @bun `
+    const pragmaLength = pragma.length
+    for (let i = 0; i < matches.length; i++) {
+        for (let j = matches[i].index - 1; j >= (i === 0 ? 0 : matches[i - 1].index); j--) {
+            if (binaryContent[j] === '\x00') {
+                // pragma test
+                if (binaryContent.slice(j + 1, j + 1 + pragmaLength).toString() === pragma) {
+                    // look next to find first `\n//`
+                    let found = false;
+                    for (let k = j + 1 + pragmaLength; k < matches[i].index; k++) {
+                        if (binaryContent[k] === '\n' && binaryContent[k + 1] === '/' && binaryContent[k + 2] === '/') {
+                            // remove from binaryContent after `//` exactly `diff` bytes
+                            const diff = matches[i].diff;
+                            const sliceStart = k + 3;
+                            binaryContent = binaryContent.slice(0, sliceStart) + binaryContent.slice(sliceStart + diff);
+                            found = true;
+                            break;
+                        }
+                    }
+                    if (found) {
+                        matches[i].found = true;
+                        break;
+                    };
+                }
+            }
+        }
+        if (!matches[i].found) {
+            break;
+        }
+    }
+
+    if (matches.every(m => !m.found)) {
+        return { success: false, message: 'Patch failed: pragma' };
+    }
+
+    return { success: true, alreadyPatched: false, content: binaryContent };
+}
+
 // Main execution
 if (require.main === module) {
     let targetPath = null;
+    let isDryRun = false;
+    let outputPath = null;
     const args = process.argv.slice(2);
 
     for (let i = 0; i < args.length; i++) {
         if (args[i] === '-f' || args[i] === '--file') {
             targetPath = args[++i];
+        } else if (args[i] === '-d' || args[i] === '--dry-run') {
+            isDryRun = true;
+        } else if (args[i] === '-o' || args[i] === '--output') {
+            outputPath = args[++i];
         } else if (args[i] === '-h' || args[i] === '--help') {
             usage();
             process.exit(0);
@@ -160,7 +279,7 @@ if (require.main === module) {
     }
 
     if (!targetPath || !fs.existsSync(targetPath)) {
-        console.error('Error: Could not find Claude Code cli.js.');
+        console.error('Error: Could not find Claude Code CLI.');
         if (targetPath) console.error(`Path tried: ${targetPath}`);
         usage();
         process.exit(1);
@@ -168,13 +287,9 @@ if (require.main === module) {
 
     console.log(`Target: ${targetPath}`);
 
-    if (!targetPath.endsWith('.js')) {
-        console.error('Error: Target file must be a .js file.');
-        process.exit(1);
-    }
-
-    const content = fs.readFileSync(targetPath, 'utf-8');
-    const result = patchContent(content);
+    const result = targetPath.endsWith('.js')
+      ? patchContentJs(fs.readFileSync(targetPath, 'latin1'))
+      : patchContentBinary(fs.readFileSync(targetPath, 'latin1'));
 
     if (result.alreadyPatched) {
         console.log('Claude is already patched for Vietnamese IME.');
@@ -182,13 +297,23 @@ if (require.main === module) {
     }
 
     if (!result.success) {
-        console.error('Error: Failed to patch Claude. The code structure might have changed.');
+        console.error(result.message);
         process.exit(1);
     }
 
-    fs.writeFileSync(targetPath, result.content, 'utf-8');
-    console.log('Success: Claude has been patched for Vietnamese IME.');
+    if (isDryRun) {
+        console.log('Dry run: patch applied successfully (not saved).');
+        process.exit(0);
+    }
+
+    const finalPath = outputPath || targetPath;
+    fs.writeFileSync(finalPath, result.content, 'latin1');
+    console.log(`Success: Claude has been patched at ${finalPath}`);
 }
 
 // Export for testing
-module.exports = { patchContent };
+module.exports = {
+    DORK,
+    patchContentJs,
+    patchContentBinary,
+};