Dev1

CHANGELOG.md

@@ -22,7 +22,35 @@ Changelog
 [Github master](https://github.com/bjones1/CodeChat_Editor)
 -----------------------------------------------------------
 
-* No changes.
+*  No changes.
+
+Version 0.1.53 -- 2026-Apr-14
+-----------------------------
+
+* Claude code reviews revealed the following issues, which were fixed manually:
+  * Fix multi-byte Unicode encoding bug.
+  * Fix incorrect websocket shutdown sequence.
+  * On websocket disconnect and reconnect, correctly retain timeouts for
+    responses.
+  * Change websocket response timeout to a reasonable value.
+  * Provide better error reporting when `hashLocations.json` file isn't found.
+  * Improve encoder performance.
+  * Correct bug in diff computation.
+  * Correctly handle CRLFs in files that begin with a LF.
+  * Fix several XSS vulnerabilities.
+  * Remove redundant reference counting on shared state.
+  * Avoid panics when validating passwords and when converting paths to strings.
+  * Fix bugs in nested comment parser.
+  * Improve CLI interface with better userid/password parsing, error reporting,
+    and ipv6 support.
+  * Include Windows drive letter Z when looking for available drives.
+  * Use block comment delimiter found in code, not a hard-coded C-only
+    delimiter.
+  * Correct many errors in the grammar for supported languages.
+  * Add lexer support for shell scripts, Swift, TOML, and Verilog, and VHDL.
+  * Ensure math is re-typeset after saving the document.
+  * Delay math un-typesetting until all typesetting is complete.
+  * Improve error reporting in VSCode extension.
 
 Version 0.1.52 -- 2026-Apr-09
 -----------------------------

builder/.gitignore

@@ -17,7 +17,7 @@
 # [http://www.gnu.org/licenses/](http://www.gnu.org/licenses/).
 #
 # `.gitignore` -- files for Git to ignore
-# ==============================================================================
+# =======================================
 #
 # Rust build output
 target/

builder/Cargo.toml

@@ -17,7 +17,7 @@
 # [http://www.gnu.org/licenses/](http://www.gnu.org/licenses/).
 #
 # `Cargo.toml` -- Rust build/package management config for the builder
-# ==============================================================================
+# ====================================================================
 [package]
 name = "builder"
 version = "0.1.0"

builder/src/main.rs

@@ -249,7 +249,7 @@ fn quick_copy_dir<P: AsRef<Path>>(src: P, dest: P, files: Option<P>) -> io::Resu
     // Per
     // [these docs](https://learn.microsoft.com/en-us/troubleshoot/windows-server/backup-and-storage/return-codes-used-robocopy-utility),
     // check the return code.
-    if cfg!(windows) && exit_code >= 8 || !cfg!(windows) && exit_code != 0 {
+    if (cfg!(windows) && exit_code >= 8) || (!cfg!(windows) && exit_code != 0) {
         Err(io::Error::other(format!(
             "Copy process return code {exit_code} indicates failure."
         )))
@@ -271,6 +271,8 @@ fn remove_dir_all_if_exists<P: AsRef<Path> + std::fmt::Display>(path: P) -> io::
     Ok(())
 }
 
+/// Search and replace a file using the regex. It's currently only used to
+/// update the version of a file; it does only one replacement.
 fn search_and_replace_file<
     P: AsRef<Path> + std::fmt::Display,
     S1: AsRef<str> + std::fmt::Display,
@@ -283,6 +285,8 @@ fn search_and_replace_file<
     let file_contents = fs::read_to_string(&path)?;
     let re = Regex::new(search_regex.as_ref())
         .map_err(|err| io::Error::other(format!("Error in search regex {search_regex}: {err}")))?;
+    // Note that this does only one replacement -- there should be exactly one
+    // version number to replace in a file.
     let file_contents_replaced = re.replace(&file_contents, replace_string.as_ref());
     assert_ne!(
         file_contents, file_contents_replaced,
@@ -413,6 +417,8 @@ fn run_install(dev: bool) -> io::Result<()> {
             cargo binstall cargo-outdated --no-confirm;
             info "cargo binstall cargo-sort";
             cargo binstall cargo-sort --no-confirm;
+            info "cargo binstall cargo-audit";
+            cargo binstall cargo-audit --no-confirm;
         )?;
     }
     Ok(())
@@ -425,7 +431,7 @@ fn run_update() -> io::Result<()> {
     run_cmd!(
         info "Builder: cargo update";
         cargo update --manifest-path=$BUILDER_PATH/Cargo.toml;
-        info "VSCoe extension: cargo update";
+        info "VSCode extension: cargo update";
         cargo update --manifest-path=$VSCODE_PATH/Cargo.toml;
         info "test_utils: cargo update"
         cargo update --manifest-path=$TEST_UTILS_PATH/Cargo.toml;
@@ -469,6 +475,16 @@ fn run_format_and_lint(check_only: bool) -> io::Result<()> {
         info "test_utils: cargo clippy and fmt"
         cargo clippy --all-targets --all-features --tests --manifest-path=$TEST_UTILS_PATH/Cargo.toml -- $clippy_check_only;
         cargo fmt --all $check --manifest-path=$TEST_UTILS_PATH/Cargo.toml;
+
+        info "cargo audit";
+        cargo audit;
+        info "Builder: cargo audit";
+        cargo audit --file=$BUILDER_PATH/Cargo.lock --no-fetch;
+        info "VSCode extension: cargo audit";
+        cargo audit --file=$VSCODE_PATH/Cargo.lock --no-fetch;
+        info "test_utils: cargo audit"
+        cargo audit --file=$TEST_UTILS_PATH/Cargo.lock --no-fetch;
+
         info "cargo sort";
         cargo sort $check;
         cd $BUILDER_PATH;
@@ -487,7 +503,9 @@ fn run_format_and_lint(check_only: bool) -> io::Result<()> {
         eslint_args.push(eslint_check)
     }
     run_script("npx", &eslint_args, CLIENT_PATH, true)?;
-    run_script("npx", &eslint_args, VSCODE_PATH, true)
+    run_script("npx", &eslint_args, VSCODE_PATH, true)?;
+    run_script("pnpm", &["audit", "--prod"], CLIENT_PATH, false)?;
+    run_script("pnpm", &["audit", "--prod"], VSCODE_PATH, false)
 }
 
 fn run_test() -> io::Result<()> {
@@ -564,7 +582,7 @@ fn run_client_build(
         true,
     )?;
 
-    // \<a id="#pdf.js>The PDF viewer for use with VSCode. Built it separately,
+    // \<a id="#pdf.js>The PDF viewer for use with VSCode. Build it separately,
     // since it's loaded apart from the rest of the Client.
     run_script(
         &esbuild,

client/eslint.config.js

@@ -2,8 +2,8 @@
 //
 // This file is part of the CodeChat Editor.
 //
-// The CodeChat Editor is free software: you can redistribute it and/or modify it
-// under the terms of the GNU General Public License as published by the Free
+// The CodeChat Editor is free software: you can redistribute it and/or modify
+// it under the terms of the GNU General Public License as published by the Free
 // Software Foundation, either version 3 of the License, or (at your option) any
 // later version.
 //
@@ -30,7 +30,9 @@ export default defineConfig(
     eslintPluginPrettierRecommended,
     defineConfig([
         {
-            // This must be the only key in this dict to be treated as a global ignore. Only global ignores can ignore directories. See the [docs](https://eslint.org/docs/latest/use/configure/configuration-files#globally-ignoring-files-with-ignores).
+            // This must be the only key in this dict to be treated as a global
+            // ignore. Only global ignores can ignore directories. See the
+            // [docs](https://eslint.org/docs/latest/use/configure/configuration-files#globally-ignoring-files-with-ignores).
             ignores: ["src/third-party/**"],
         },
         {

client/package.json5

@@ -43,7 +43,7 @@
         url: 'https://github.com/bjones1/CodeChat_editor',
     },
     type: 'module',
-    version: '0.1.52',
+    version: '0.1.53',
     dependencies: {
         '@codemirror/commands': '^6.10.3',
         '@codemirror/lang-cpp': '^6.0.3',
@@ -60,6 +60,8 @@
         '@codemirror/lang-sql': '^6.10.0',
         '@codemirror/lang-xml': '^6.1.0',
         '@codemirror/lang-yaml': '^6.1.3',
+        '@codemirror/language': '^6.12.3',
+        '@codemirror/legacy-modes': '^6.5.2',
         '@codemirror/state': '^6.6.0',
         '@codemirror/view': '6.38.8',
         '@hpcc-js/wasm-graphviz': '^1.21.2',
@@ -79,20 +81,20 @@
         '@types/mocha': '^10.0.10',
         '@types/node': '^24.12.2',
         '@types/toastify-js': '^1.12.4',
-        '@typescript-eslint/eslint-plugin': '^8.58.1',
-        '@typescript-eslint/parser': '^8.58.1',
+        '@typescript-eslint/eslint-plugin': '^8.58.2',
+        '@typescript-eslint/parser': '^8.58.2',
         chai: '^6.2.2',
         esbuild: '^0.28.0',
         eslint: '^10.2.0',
         'eslint-config-prettier': '^10.1.8',
         'eslint-plugin-import': '^2.32.0',
         'eslint-plugin-prettier': '^5.5.5',
-        globals: '^17.4.0',
+        globals: '^17.5.0',
         mocha: '^11.7.5',
-        'npm-check-updates': '^20.0.0',
-        prettier: '^3.8.1',
+        'npm-check-updates': '^20.0.2',
+        prettier: '^3.8.2',
         typescript: '^6.0.2',
-        'typescript-eslint': '^8.58.1',
+        'typescript-eslint': '^8.58.2',
     },
     scripts: {
         test: 'echo "Error: no test specified" && exit 1',