carlos / todomvc-bakeoff Public

docs(rust): STATUS: review fixes live at f78a916

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

Paul Campbell pushed by claude 1 day ago 6a507169d9a44b22ec0e90a47ea8c661c790dc4c parent 6d3ad8a

1 file changed, +23 −2

diff --git a/apps/rust/STATUS.md b/apps/rust/STATUS.md

index 8cc8377..b832ba8 100644

--- a/apps/rust/STATUS.md

+++ b/apps/rust/STATUS.md

@@ -1,4 +1,4 @@

-LIVE https://todo-rust.bfj.oncarlos.com 4c742de

+LIVE https://todo-rust.bfj.oncarlos.com f78a916

## What's there

@@ -9,7 +9,7 @@ LIVE https://todo-rust.bfj.oncarlos.com 4c742de

- Shared suite on a local run: 67 passed, 3 skipped (the skips are the

JS-only tests in the JS-off run). Against

[https://todo-rust.bfj.oncarlos.com](https://todo-rust.bfj.oncarlos.com),

- two full runs after the last deploy: 67 passed, 3 skipped each time.

+ 67 passed, 3 skipped after the review-fix deploy (`f78a916`).

- `scripts/boot-check.sh dist/app-linux-arm64`: healthz 200, version 200,

clean exit on SIGTERM.

@@ -24,3 +24,24 @@ that drops the connection to the edge. The app now reads the whole body

those requests answered correctly (403 and 404). Any other app that

rejects a request without reading its body may hit the same thing on

Carloku.

+## Review fixes (`f78a916`)

+1. **An edit was lost when another action's response arrived.**

+ `static/app.js` used to replace the whole list on every response. If

+ an item is being edited and its edit hasn't been sent yet, the list is

+ now patched around that item instead. Its draft, focus and edit state

+ stay, and only its view (label, checkbox) is refreshed. Checked with a

+ Playwright script that delayed POST responses by 1.5 s: toggle one

+ item, start editing another and type a draft. When the toggle response

+ arrived, the toggle was applied and the draft, focus and edit state

+ were all still there. Pressing Enter then saved the draft.

+2. **`clean-deps` left Zig's compiled libraries in its shared cache.**

+ The Makefile now points `ZIG_GLOBAL_CACHE_DIR` and

+ `ZIG_LOCAL_CACHE_DIR` at `apps/rust/.cache/zig`, and `clean-deps`

+ removes that directory. A cold `make build-arm64` after `clean-deps`

+ took 1 min 29 s and refilled `.cache/zig` (79 MB).

+## Review findings not fixed

+None.