GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
GeezerVison is showing github.com — v0.0.14-beta.1 (5/5): Release v0.0.14-beta.1 and v0.0.14 by Obed0101 · Pull Request #141 · MendCode/MendCode Conversation
|
|
MendCode: @Obed0101's MendCode worked on this pull request on one of its devices; released 2026-11-10 21:31 UTC. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
No findings · nothing in this change was confirmed as a problem
head 7f1822ade15d
Merge risk: 🟢 Low · Only release documentation changes, using the same stable/beta structure as the preceding release. A regression would affect release-note presentation or accuracy rather than application execution.
Change: Docs · Review effort: 1/5, about 5 minutes
This changelog-only change records v0.0.14-beta.1 and v0.0.14, preserves an empty Unreleased section, and reorders existing release-note bullets. No defects were found in this pull request’s own diff; risk is low because it changes release documentation rather than runtime behavior. Review was read-only, and tests were not run.
Readers: 1 code file no reader could read went to the review pass whole (the diff is small: one reader call costs more than the readers' share of this review's cap, so the review pass reads it).
Stacked on #140: only this pull request's own commits since #140's head d608ea3.
The existing unreleased notes become the notes for v0.0.14-beta.1, with a new summary highlighting the release’s user-visible improvements. A separate v0.0.14 entry repeats that summary and states that it contains the same code as the beta, following the preceding release’s format. Existing bullets are retained, with several reordered, and Unreleased remains available for future changes.
Changes
| Layer / files | Summary |
|---|---|
Release documentation CHANGELOG.md |
Adds dated stable and beta release sections, release summaries, and a stable-release equivalence note while retaining and reordering the existing change descriptions. |
| Check | Status | Explanation |
|---|---|---|
| Title | ✅ Passed | The title identifies both release versions documented by the change. |
| Description | ✅ Passed | The description explains the changelog-only scope, its position in the stack, and the stable release’s equivalence to the beta. It also reports typecheck and full-suite results, which were not independently executed during this read-only review. |
| Out of scope | ✅ Passed | All changes are release-note additions or rearrangements consistent with the stated scope; no parent-stack implementation changes are included. |
CHANGELOG.mdComment @mendcode review to review again · @mendcode pause / resume · reply to a finding with explain, fix or learn · settings in .mendcode/review.yaml
Sorry, something went wrong.
There was an error while loading. Please reload this page.
…ession is live, starting or holding a fresh capture, so it never drops another session's references or window
There was a problem hiding this comment.
The reason will be displayed to describe this comment to others. Learn more.
No findings · nothing in this change was confirmed as a problem
head dee18c87e34b · only the commits since the last review at 7f1822a
Merge risk: 🟡 Medium · The change affects asynchronous cleanup around shared native resources. A regression could invalidate active observations or window handles, or delay restoring application accessibility state; ordinary expiry and authorization checks remain in place.
Change: Bug fix · Review effort: 1/5, about 5 minutes
The incremental diff changes Computer Use session cleanup to defer native resource release until sessions, starts, captures, and recent bootstrap captures are quiet, and updates regression coverage. No introduced defect was verified by source inspection; risk is medium because cleanup timing affects shared native window handles and accessibility references. Tests were not run in this read-only review.
Readers: 2 code files no reader could read went to the review pass whole (the diff is small: one reader call costs more than the readers' share of this review's cap, so the review pass reads it).
Incremental review: only what changed since the last review at 7f1822a; the rest of the pull request was reviewed then (read other files with your tools when a change depends on them).
This increment is a Computer Use lifecycle fix rather than the changelog-only release described in the pull request. Previously, a sweep could release an idle session while another operation still needed native resources. Sweeping now waits for global quiescence, and session starts are counted through a finally block so both successful and failed starts trigger another sweep. Session authorization, per-call expiry checks, and the native release implementation remain unchanged.
Changes
| Layer / files | Summary |
|---|---|
Computer Use session lifecycle packages/daemon/src/computer-sessions.ts |
Tracks pending session starts and postpones automatic cleanup while any live session, operation, capture, or usable bootstrap capture may depend on native resources. |
Lifecycle regression tests packages/daemon/test/b80-computer.test.ts |
Uses a failed start to trigger a quiet sweep and adds coverage confirming that a newer session prevents premature release of an idle session's application. |
Sequence diagram
sequenceDiagram
participant Caller
participant Sessions as Computer sessions
participant Native as Accessibility backend
Caller->>Sessions: Start session
Sessions->>Sessions: Increment pending starts
Sessions->>Native: Resolve or identify target
Sessions->>Sessions: Register session if successful
Sessions->>Sessions: Finally decrement pending starts and sweep
alt Sessions, operations, or captures still hold resources
Sessions->>Sessions: Schedule next sweep
else All resource holders are quiet
Sessions->>Native: Release expired sessions' resources
end
| Check | Status | Explanation |
|---|---|---|
| Title | The release title does not identify the Computer Use cleanup fix in this incremental diff. It may still describe the broader stacked release. | |
| Description | ❌ Failed | The description explicitly claims a changelog-only change, but this increment modifies session lifecycle code and tests. It reports prior checks without explaining this fix or its verification. |
| Out of scope | ❌ Failed | Both changed files implement a runtime cleanup fix that the release title and changelog-only description do not explain. |
Performance (perf-attrib, base d608ea3 against head dee18c8)
perf-attrib: baja: tui.after.heap.protected -1; tui.rest.fp_mb -5.2MB; total.rest.fp_mb -5.2MB
base d608ea3b230fa30b4d4bdc528a4bcb786ef25629 (d608ea3b) vs cand dee18c87e34bc15d8ae6b4a1dc68b9d522f2092e (dee18c87), median of 2 faux-engine session(s) per side; flagged = beyond the noise measured on 18 base-vs-base runs (x1.5).
| metric | base | cand | diff |
|---|---|---|---|
| tui.after.fp_mb | 57.6 | 57.2 | -0.4 |
| tui.after.peak_mb | 73.5 | 73.2 | -0.3 |
| tui.ready.fp_mb | 54.9 | 54.8 | -0.1 |
| tui.ready.peak_mb | 73.5 | 73.2 | -0.3 |
| tui.rest.fp_mb | 62 | 56.8 | -5.2 down |
| tui.rest.peak_mb | 73.5 | 73.2 | -0.3 |
| tui.after.rss_mb | 106.7 | 136.3 | +29.6 UP |
| tui.ready.rss_mb | 114.2 | 114.1 | -0.1 |
| tui.rest.rss_mb | 85.9 | 93.3 | +7.4 UP |
| tui.after.heap.extra_b | 5925520 | 6688136 | +762616 |
| tui.after.heap.objects | 239558 | 241097 | +1539 |
| tui.after.heap.protected | 20 | 19 | -1 down |
| tui.after.heap.size_b | 23442192 | 24339608 | +897416 |
| tui.ready.heap.extra_b | 2895389 | 2895394 | +5 |
| tui.ready.heap.objects | 217214 | 217211 | -3 |
| tui.ready.heap.protected | 18 | 18 | 0 |
| tui.ready.heap.size_b | 19330520 | 19330356 | -164 |
| tui.rest.heap.extra_b | 4625289 | 3734037 | -891252 |
| tui.rest.heap.objects | 238381 | 236024 | -2357 |
| tui.rest.heap.protected | 18 | 18 | 0 |
| tui.rest.heap.size_b | 22075721 | 20940053 | -1135668 |
| tui.rest.js_code_b | 5842385 | 4889545 | -952840 |
| tui.rest.js_live_b | 19285419 | 19252304 | -33115 |
| tui.cpu_ms.after | 6099 | 6216 | +117 |
| tui.cpu_ms.ready | 391 | 365 | -26 |
| tui.cpu_ms.rest | 540 | 533 | -7 |
| binary.bytes | 72290528 | 72290528 | 0 |
| disk.bytes | 4113 | 4111 | -2 |
| disk.bytes_per_session | 2057 | 2056 | -1 |
JS object types that rose (count):
JS heap bytes by constructor that rose:
Modules loaded at start: 230 -> 230, 4631481 -> 4631481 bytes of chunks (chunk names are hashes: the list says which changed in size, not by what name)
Reference (skill, 2026-11-03, real model): footprint 64 ready / 66 after / peak 80 MB, 2.6 KB per session. This run (faux engine, whole process tree): 54.8 / 57.2 / peak 73.2 MB, 2056 B per session.
packages/daemon/src/computer-sessions.tspackages/daemon/test/b80-computer.test.tsComment @mendcode review to review again · @mendcode pause / resume · reply to a finding with explain, fix or learn · settings in .mendcode/review.yaml
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.