ucu-mcp 0.3.4 → 0.3.6

package/CHANGELOG.md

@@ -5,6 +5,38 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.3.5] - 2026-06-06  *(Yanked — see 0.3.6)*
+
+### Tests
+
+- `macos-platform`: new regression test for the value-field regex pre-validation (`findElement({value:"[", textMode:"regex"})` throws `PlatformError` with an `Invalid regex pattern` message). Pins the 0.3.2 commit `0710eca` behavior change.
+- `macos-platform`: new regression test for the near-sort bounds fallback (elements without `bounds` are pushed to the end of the sorted result, instead of implicitly being centered at (0,0)). Pins the 0.3.2 commit `0710eca` second behavior change.
+- `tools-layer`: new regression test that `find_element({value:""})` returns `isError: true`. Pins the 0.3.2 commit `46d4ddd` schema tightening (`z.string().min(1).optional()`).
+
+### Changed
+
+- JXA `textMatches` regex branch now compiles the `RegExp` once per element instead of once per source (name / value / description) — three fewer compilations per matched element when `textMode="regex"`. The TS-side pre-validation in `findElement` guarantees the pattern is valid, so the `RegExp` constructor cannot throw here. (Herschel review perf Minor)
+
+### Fixed
+
+- Comment on the JXA `matchesValue` helper no longer claims "JXA function declarations are order-sensitive" (they aren't — JXA hoists them like any ES engine). The comment now correctly notes that the leading placement is for readability. (Herschel review comment Minor)
+
+## [0.3.6] - 2026-06-06
+
+### Re-publish of 0.3.5
+
+0.3.5 was published with a test that didn't account for the McpServer
+schema-validation wrapper. The test called `handler({value:""})`
+directly, which bypasses the schema validation, so the assertion
+`r.isError === true` saw `undefined` instead. The actual fix
+(asserting the zod schema constraint directly) was a one-test rewrite;
+0.3.6 carries the same code as 0.3.5 plus the test fix.
+
+### Yanked
+
+`ucu-mcp@0.3.5` was unpublished (yanked) within minutes of release
+due to the broken test. Users on `@latest` are now on 0.3.6.
+
 ## [0.3.4] - 2026-06-06
 
 ### Re-publish of 0.3.3
@@ -25,7 +57,7 @@ contents and CHANGELOG entry as 0.3.3 plus this note.
 `ucu-mcp@0.3.3` was unpublished (yanked) shortly after release due to
 the test file compile error. Users on `@latest` are now on 0.3.4.
 
-## [0.3.3] - 2026-06-06
+## [0.3.3] - 2026-06-06  *(Yanked — see 0.3.4)*
 
 ### Tests
 

package/dist/src/platform/macos.js

@@ -866,9 +866,9 @@ export class MacOSPlatform {
       // Shared filter helper. textMatches and valueMatches used to be near
       // copies of the same three-branch dispatch (contains / exact / regex);
       // this consolidates the logic so the two callers only differ in which
-      // sources they iterate. Declared before textMatches/valueMatches
-      // because JXA function declarations are order-sensitive within a
-      // script. (Singer Nit)
+      // sources they iterate. Declared before textMatches/valueMatches for
+      // readability — JXA's function declarations are hoisted, so order
+      // doesn't affect callability. (Singer Nit + Herschel comment fix)
       function matchesValue(filter, value, mode) {
         if (filter === null) return true;
         if (mode === "exact") {
@@ -886,8 +886,18 @@ export class MacOSPlatform {
       function textMatches(elemName, elemValue, elemDesc) {
         if (textFilter === null) return true;
         var sources = [elemName, elemValue, elemDesc];
-        for (var i = 0; i < sources.length; i++) {
-          if (matchesValue(textFilter, sources[i], textMode)) return true;
+        // Hoist regex compilation out of the loop. The TS-side pre-validation
+        // in findElement guarantees the pattern is valid, so the RegExp
+        // constructor cannot throw here. (Herschel perf Minor)
+        if (textMode === "regex") {
+          var reText = new RegExp(textFilter, "i");
+          for (var i = 0; i < sources.length; i++) {
+            if (reText.test(sources[i])) return true;
+          }
+          return false;
+        }
+        for (var j = 0; j < sources.length; j++) {
+          if (matchesValue(textFilter, sources[j], textMode)) return true;
         }
         return false;
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "ucu-mcp",
-  "version": "0.3.4",
+  "version": "0.3.6",
   "description": "MCP server for Universal Computer Use — desktop automation for AI agents via Model Context Protocol",
   "type": "module",
   "bin": {