← Engel's Code Design Notebook

OpenHands / OpenHands · Maintainer review queue

PRs to keep or close

2 Keep candidates and 32 Close without merge candidates from the previously reviewed other-author PR queue. Each PR has its author and a specific reason.

Issue, PR and reference states checked through . Code assessment dates from the original audit on 10 October 2026; refreshed heads are compared with that audit.

Update : PR #18127 (errmakov) was merged after a live check, closing issue #17928. It moves from Keep to Already merged; PR #18152, the Close candidate for the same issue, now points to it. Other rows keep the snapshot above.

Update : PR #18152 (lx3133584) was closed without merge as redundant with #18127. It moves from Close to Already closed.

Update : Official API refresh reconciles #17995 merged and seven superseded alternatives already closed. Current queue: 2 Keep, 32 Close, 10 merged and 9 already closed, preserving the original 53. Keep #17970 is held for a verified controlled backend-switch rollback regression and a current-head enyst request-changes review; #17880 awaits independent live verification. The maintainer plans independent replacement fixes and no reuse for every Close submission.

Update : Official API recheck of the original 53-PR queue: 2 Keep, 32 Close, 10 merged and 9 closed without merge. No PR state or linked-issue state changed since the previous snapshot. #17962 has a new, independently unverified head and remains a draft. #17970 remains held at the tested 8a13bda5 head with enyst’s request-changes review; #17880 still awaits independent live verification. The Close policy remains independent replacement with no reuse.

Update : Official API recheck of the original 53-PR queue: 2 Keep, 32 Close, 10 merged and 9 closed without merge. 0 PR state changes, 1 linked-issue state changes and 3 head changes since the previous snapshot. Original evidence and audit decisions are retained; the Close policy remains independent replacement with no reuse.

The maintainer plans to solve the remaining issues independently. Every PR marked Close without merge is intended for closure and supersession by that work, with no code or implementation reused. “Keep” retains a candidate for review and does not mean approval. The issue inventory since Monday identifies the work and distinguishes enyst/smolpaws PRs from other proposals. This page records current GitHub states; the inventory refresh itself submits no closures or reviews.

Keep · 2 PRs

fix: preserve toggle click targets and roll back failed skill saves

Why keepHistorical assessment at head 8a13bda59017f0bf8bd1c4509e2191336e65de3a: Retained as a candidate for repair, with merge on hold. Independent live checks passed first-click, basic failed-save rollback, retry and shared MCP/plugin toggle behavior; 105 existing focused/consumer tests passed. A separate controlled real-hook regression shows a late backend-A save can replace backend B’s rollback target, disabling an untouched skill after B’s next save fails. The callback needs backend/org and dispatch-generation ownership. Current head a6dc5b17c33385f5cb94e9d365823dddc3c530c2 has not been independently reviewed by this metadata refresh.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed. The earlier independent verdict HOLD MERGE applies to head 8a13bda59017f0bf8bd1c4509e2191336e65de3a; current GitHub review decision is CHANGES_REQUESTED. The current head differs from the originally audited head and is independently unverified by this metadata refresh.

Evidence and review limits

Current GitHub head: a6dc5b17c33385f5cb94e9d365823dddc3c530c2.

Original audited head: 8a13bda59017f0bf8bd1c4509e2191336e65de3a. The current head differs from the earlier audit and was not independently reviewed during this metadata refresh.

  • The following source and runtime assessment limits belong to the originally audited head, not a fresh verification of the current head.
  • The original repository approval names an older SHA in its prose; its API commit is the then-current 8a13bda5 head.
  • Independent first-click and simple rollback checks passed on the exact head and representative fresh-main patch checks passed.
  • The blocking backend-switch finding is controlled service evidence through real hooks/context, not live two-server proof or evidence of a durable incorrect server write.
  • The same-backend delayed-refetch live attempts did not prove terminal failure before releasing the held genuine GET; they are not passes.

fix(home): scope launch targets by backend

Why keepScopes Home launch targets, workspace selections and recent repositories by backend/organization. Real Agent Server switch-and-launch recordings and a current-head review reporting 90 focused tests support keeping it; the failing description gate is the maintainer-owned ready-for-dev label.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: a80d41e08b6e81e78af35269fa55a549b7ab1179.

Original audited head: a80d41e08b6e81e78af35269fa55a549b7ab1179. The current head matches the earlier code audit.

  • This is a broader state migration: legacy unowned recent repositories are deliberately cleared. Cloud changes are regression-covered rather than live-recorded. The current review requests changes for readiness only, not a code defect; temporary .pr artifacts remain.

Close without merge · 32 PRs

Defect or redundant fix · 11 PRs

PR #18247Close without mergeOpen

fix(chat): resolve event group status for rejected and errored actions

Closure reasonLinked issue #17920 is closed by merged PR #17995 (errmakov). This still-open proposal is superseded; close without merge. No changes from this proposal will be reused.

GitHub’s latest issue-closing event identifies merged PR #17995 as the closer. The original audit and evidence remain historical; this refresh did not verify product behavior or submit a closure.

Evidence and review limits

Current GitHub head: 545da063406da7e90a41a360149f7f31430b9f93.

Original audited head: 545da063406da7e90a41a360149f7f31430b9f93. The current head matches the earlier code audit.

  • The earlier thought/grouping findings are source-derived, not a newly run UI reproduction. GitHub now confirms #17995 merged and closed #17920; this inventory refresh did not re-execute either branch.
PR #18242Close without mergeOpen

fix(chat): fit overflow and tools submenus to viewport at narrow widths

Closure reasonMeasures a zero-size wrapper around an absolutely positioned menu, then substitutes 220×200 instead of the real menu dimensions. A prior independent minimal Chromium fixture reproduced overflow that its mocked geometry tests hide.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 54b325b47ee4dc97d36f3dd9fb424c945bc0f4c4.

Original audited head: 54b325b47ee4dc97d36f3dd9fb424c945bc0f4c4. The current head matches the earlier code audit.

  • The proof is a minimal DOM/layout fixture, not a full running OpenHands test. Current-head approval and green CI do not address that geometry. #18115 is also an unfinished closure candidate, not a retained solution; #18063 stays open.
PR #18201Close without mergeDraft · open

fix: preserve App Settings edits after save errors

Closure reasonDraft with the same incomplete production change as #17945: controlled switch edits survive failure, but uncontrolled Git Username/Email fields still reset and retries can send stale values. The author explicitly supplies no running-Canvas evidence.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 317e6e12b1a5172b343c195e059166c21e33a0ec.

Original audited head: 317e6e12b1a5172b343c195e059166c21e33a0ec. The current head matches the earlier code audit.

  • The text-input failure was reproduced by an exact-head review of #17945; this draft shares that production diff, but was not independently run. Both candidates need completion; #17927 remains open.
PR #18186Close without mergeOpen

fix(automations): invalidate queries when dispatch fails with 404

Closure reasonMissing the running-Canvas recordings of 404 removal on dashboard, home and detail pages requested by the latest review. An earlier source audit also found a code-derived detail-toast suppression gap; the later review calls the code sound but did not test detail-query settlement. No direct runtime reproduction on this PR is claimed.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: d088ddec5a81284a5b05544f77d97c1d47c6b1ba.

Original audited head: d088ddec5a81284a5b05544f77d97c1d47c6b1ba. The current head matches the earlier code audit.

  • The delayed second toast was live-reproduced on the equivalent #18073 path, not on #18186. Current immutable source still lacks disableToast, so the concern is code-derived and unresolved, not a new direct reproduction. #18073 is also a closure candidate, not a preserved finished alternative; #18062 remains open.
PR #18156Close without mergeOpen

fix(launcher): honor --allow-lan-session-key, clean up port-in-use error, and align backend-only ingress description

Closure reasonHistorical assessment at head ccd2b2f5249cc993b2ba2a920c31642b8b66b914: The claimed LAN session-key fix does not cover the official npm/bin static-server path: startStaticFrontend neither passes allowLanSessionKey to policy application nor forwards --allow-lan-session-key to static-server. Explicit opt-in therefore still cannot work on that path. Current head 33b95c0abe9a404d284ab26d8df563378e083819 has not been independently reviewed by this metadata refresh.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed. The current head differs from the originally audited head and is independently unverified by this metadata refresh.

Evidence and review limits

Current GitHub head: 33b95c0abe9a404d284ab26d8df563378e083819.

Original audited head: ccd2b2f5249cc993b2ba2a920c31642b8b66b914. The current head differs from the earlier audit and was not independently reviewed during this metadata refresh.

  • The following source and runtime assessment limits belong to the originally audited head, not a fresh verification of the current head.
  • Source-derived omission; this refresh did not run a LAN launcher or inspect served HTML. Port-conflict wording and ingress-description changes may be useful independently. No retained alternative; #17949 remains open.
PR #18154Close without mergeDraft · open

fix: make settings switches keyboard accessible

Closure reasonStill a draft and explicitly lacks full Canvas evidence. It repeats #17963's switch primitive change with fewer regression cases; #17963 is also an unfinished draft recommended for closure, so neither constitutes a completed solution.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: f160a34f47742198db31cb9a059c89fca370fd4c.

Original audited head: f160a34f47742198db31cb9a059c89fca370fd4c. The current head matches the earlier code audit.

  • Current code CI is green; the old local build failure is not a current CI blocker. No code defect is asserted. Issue #17900 remains open for a finished fix.
PR #17945Close without mergeOpen

fix(settings): preserve unsaved edits and keep save enabled on failed save

Closure reasonAn exact-head review reproduced Git Username/Email edits resetting after a failed save and a retry sending stored stale text. The new test covers only the controlled sound switch, so the advertised failed-save preservation remains incomplete.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 3226edb27dc052c133697c17eeb6cc2c29fdafa4.

Original audited head: 3226edb27dc052c133697c17eeb6cc2c29fdafa4. The current head matches the earlier code audit.

  • The empirical finding belongs to the existing reviewer's focused rendered-route test, with SettingsService mocked to reject; this refresh did not repeat it or run a live backend. Head is unchanged; #17927 remains open.
PR #17570Close without mergeOpen

fix(diff-viewer): dispose Monaco models after editor teardown

Closure reasonDisables the library's diff-model disposal but waits for IStandaloneDiffEditor.onDidDispose, which the locked Monaco 0.56 does not fire. The models therefore leak; the added fake-editor tests invent the callback behavior and cannot catch it.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: d7d4d01dea9419d0fd40cc344a88d2d4c6077f4c.

Original audited head: d7d4d01dea9419d0fd40cc344a88d2d4c6077f4c. The current head matches the earlier code audit.

  • Prior dependency-source inspection and an exact-head review's real-Monaco probe support the finding. This refresh did not run the app or repeat the probe. Running-Canvas lifecycle evidence and the author-owned description gate remain absent; #17567 stays open.
PR #17017Close without mergeOpen

refactor: source canvas event source/status contracts from typescript-client (#16952)

Closure reasonThe purported type migration adds an over-strict runtime filter that silently drops valid local events with optional source/id/timestamp fields, while leaving Cloud behavior inconsistent. An unchanged-head review identifies the same wire-contract mismatch.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 381f65b23c38d1399e479c098d8406540d67268b.

Original audited head: 381f65b23c38d1399e479c098d8406540d67268b. The current head matches the earlier code audit.

  • Source/contract assessment and existing focused review, not a new runtime replay. Green CI does not cover the valid legacy fixtures that are filtered away. It currently conflicts with main; #16952 stays open.
PR #17005Close without mergeDraft · open

fix(settings): keep unsaved SDK settings edits across background refetches

Closure reasonA successful save deletes every overlay entry whose key was submitted, including a newer edit to that same key made while the save was in flight. Its test edits a different key and misses the preservation gap; the submission remains a draft.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: a1a22037da90833d3e40519c7304eb4a73bede07.

Original audited head: a1a22037da90833d3e40519c7304eb4a73bede07. The current head matches the earlier code audit.

  • This is incomplete fulfillment of the claimed preservation behavior, not an established newly introduced main regression. It follows unchanged-head source assessment; no new live branch run was performed. #16948 stays open.
PR #16992Close without mergeOpen

fix(mcp-health): scope health keys by active backend and connection revision

Closure reasonAsync install/update/delete completion reads the current backend scope instead of the operation's originating scope, so switching backends can clear or seed the wrong health bucket. The unchanged-head review identifies this race and missing mutation-switch coverage; its screenshot is unrelated.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: ef301c4ec476a03aaf045c20f21e3e923cd2a3a1.

Original audited head: ef301c4ec476a03aaf045c20f21e3e923cd2a3a1. The current head matches the earlier code audit.

  • Existing exact-head source review and passing focused tests are not a new live race reproduction. It also conflicts with main, and #16954 lacks valid evidence/readiness. Do not close solely for the maintainer-owned label; the scope defect is the material reason.

Incomplete work or evidence · 21 PRs

PR #18260Close without mergeDraft · open

fix(automations): validate a cleared time of day before saving

Closure reasonHistorical assessment at head 324a1127200111b4fd372cd55e9d09c8ccf3b93c: Intentional partial draft: fixes only item 1 of the four-item automation issue. The required HUMAN note and description check remain unfinished; it does not claim to solve the other three items. Current head 0f812ac68902d2c71896568311bffb8aa6f216a6 has not been independently reviewed by this metadata refresh.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed. The current head differs from the originally audited head and is independently unverified by this metadata refresh.

Evidence and review limits

Current GitHub head: 0f812ac68902d2c71896568311bffb8aa6f216a6.

Original audited head: 324a1127200111b4fd372cd55e9d09c8ccf3b93c. The current head differs from the earlier audit and was not independently reviewed during this metadata refresh.

  • The following source and runtime assessment limits belong to the originally audited head, not a fresh verification of the current head.
  • Mock-Canvas before/after screenshots and 26 focused tests are supplied; do not say no evidence exists. No code defect established.
PR #18166Close without mergeOpen

fix(mcp): do not classify failed Slack probe tool calls as credential failures

Closure reasonOnly the original Slack probe failure screenshot is attached. There is no branch-specific install/Test/Retry evidence showing fetch failures and timeouts become connectivity-only, with credentials explicitly not verified.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: f500de4b326cef594d88f47be9db090402c2347e.

Original audited head: f500de4b326cef594d88f47be9db090402c2347e. The current head matches the earlier code audit.

  • Focused tests and code CI pass; this is an evidence/readiness closure, not a demonstrated credential-classification defect.
PR #18164Close without mergeOpen

fix(query): disable error toasts for free models hydrator and backend version probe

Closure reasonNo running-branch evidence distinguishes quiet bootstrap recovery from genuine post-bootstrap errors. The change silences these model/version queries in every app state, and the configured-backend failure test now expects silence.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: c485c41b4296491870b9c4c9fd6ef387afbf9615.

Original audited head: c485c41b4296491870b9c4c9fd6ef387afbf9615. The current head matches the earlier code audit.

  • The global error handler is unchanged; no global error-reporting regression has been established. The screenshot shows the original first-run contrast, not repaired returning-user recovery.
PR #18153Close without mergeOpen

fix(automations): render empty state on template search and exclude optional integrations from connect count

Closure reasonThe only screenshot repeats the original blank template-search result. There is no running-branch evidence for the no-results message, Clear action, optional-integration count or rendered singular/plural wording.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 3441b93aa8ebf47da89b9d11716e1d9a670a8230.

Original audited head: 3441b93aa8ebf47da89b9d11716e1d9a670a8230. The current head matches the earlier code audit.

  • Bounded code and route/component regression tests look plausible, and current code CI passes. No introduced defect established.
PR #18147Close without mergeOpen

fix(test): bound tab wait and record diagnostics on mock-llm sidebar cmd-click

Closure reasonThe click diagnostic snapshots defaultPrevented in capture phase, before later handlers can cancel it; click failures also bypass its diagnostic catch. Actual cross-connect success and a deliberately forced no-tab failure have not been demonstrated.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 07f0bfbe0fe09767e4aba683e2eb4e43ebe3b1f1.

Original audited head: 07f0bfbe0fe09767e4aba683e2eb4e43ebe3b1f1. The current head matches the earlier code audit.

  • A prior local jsdom control confirmed the capture-phase cancellation mismatch; click/wait ordering is source-reviewed. No new runtime tests were run for this refresh. Current code/description CI passes.
PR #18115Close without mergeOpen

fix(chat): keep the overflow menu's submenus inside the viewport

Closure reasonLocal before/after composer screenshots exist, but this is a partial fix: the expanded Tools-menu scope is untouched, and the viewport matrix, Code/Plan, profile/link activation and visible keyboard-focus behavior are not established.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: f8890f07e433bbd55e702c73d2f0941f76484bf6.

Original audited head: f8890f07e433bbd55e702c73d2f0941f76484bf6. The current head matches the earlier code audit.

  • The original report allowed Macros to be split out; the later AI triage explicitly includes Tools menus. A maintainer could narrow scope and retain this bounded fix. Local before/after screenshots exist; do not call all evidence absent.
PR #18073Close without mergeOpen

fix(automations): refetch automations when Run now finds the automation deleted

Closure reasonUbuntu CI still fails because an existing test mock lacks getErrorStatus. The detail-toast correction has unit coverage, but its author explicitly says the requested real-stack replay/recording is unfinished; earlier live review tested the previous head.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 9a831278b5fd79bfdd7646db33bb81f824beadc8.

Original audited head: 9a831278b5fd79bfdd7646db33bb81f824beadc8. The current head matches the earlier code audit.

  • The test-mock failure is a specific CI repair, not proof of a production regression. Prior real-stack dashboard/home and non-404 checks succeeded before the correction.
PR #17965Close without mergeOpen

fix(settings): persist displayed active llm settings when creating profile

Closure reasonLinked issue #17955 is closed by merged PR #18422 (enyst). This still-open proposal is superseded; close without merge. No changes from this proposal will be reused.

GitHub’s latest issue-closing event identifies merged PR #18422 as the closer. The original audit and evidence remain historical; this refresh did not verify product behavior or submit a closure.

Evidence and review limits

Current GitHub head: feffce0c4a4a6e3950b9550ea66f5b770e09eb72.

Original audited head: feffce0c4a4a6e3950b9550ea66f5b770e09eb72. The current head matches the earlier code audit.

  • Same-head review found the displayed-value fix sound and treats hidden-field inheritance as non-blocking. Do not imply demonstrated credential leakage or data corruption.
PR #17963Close without mergeOpen

fix: restore keyboard access to settings switches

Closure reasonStill Draft with the requested HUMAN note unanswered and description validation failing. Close under the review-ready policy; the switch implementation already has published real-stack Tab/Space and focus evidence.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: cb43883c4a0aec10958ae37c3847ffbc5cbb4357.

Original audited head: cb43883c4a0aec10958ae37c3847ffbc5cbb4357. The current head matches the earlier code audit.

  • Real AgentServer browser recordings, DOM/keyboard/state data and focused tests are supplied. No code defect established; absence of human review is not evidence the account is non-human.
PR #17962Close without mergeOpen

fix: honor Do Not Track before telemetry initialization

Closure reasonHistorical assessment at head 0cf65d442003f7ecd52971d266903bc746f99130: Still Draft with the requested HUMAN note unanswered and description validation failing. Close under the review-ready policy; the DNT fix already has published real-stack network/storage comparison evidence. Current head d7e78af478ec07586876d4847556b5436860635b has not been independently reviewed by this metadata refresh.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed. The current head differs from the originally audited head and is independently unverified by this metadata refresh.

Evidence and review limits

Current GitHub head: d7e78af478ec07586876d4847556b5436860635b.

Original audited head: 0cf65d442003f7ecd52971d266903bc746f99130. The current head differs from the earlier audit and was not independently reviewed during this metadata refresh.

  • The following source and runtime assessment limits belong to the originally audited head, not a fresh verification of the current head.
  • The author supplies base/head recordings and real AgentServer runs claiming six analytics attempts before versus zero after. These were not independently replayed in this refresh. No code defect established.
  • Then-current GitHub head d7e78af478ec07586876d4847556b5436860635b differs from original audited head 0cf65d442003f7ecd52971d266903bc746f99130; this refresh did not review or execute the new code.
PR #17694Close without mergeOpen

fix(automations): make dashboard header responsive on mobile viewports

Closure reasonCurrently conflicts with main and has not supplied the requested after-fix browser captures at 320px, 390px and desktop. Its jsdom class assertions cannot prove readable wrapping, reachable buttons or no viewport overflow.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 2562990c8604f3f90bed90ee7c9a4c1c8d9cc165.

Original audited head: 2562990c8604f3f90bed90ee7c9a4c1c8d9cc165. The current head matches the earlier code audit.

  • The HUMAN note claims those viewport checks, but the PR attaches only the original Before screenshot. Same-head review found no code defect; code CI passes on its unchanged head.
PR #17654Close without mergeOpen

fix: isolate embedded modal portal hosts

Closure reasonIncomplete portal acceptance: CSS variables supplied through the public style prop do not reach the body-level modal host. The required one-root/two-theme browser evidence is absent and description validation fails.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: f0474d5411281e1ec422832df7e7d0fb77ac7095.

Original audited head: f0474d5411281e1ec422832df7e7d0fb77ac7095. The current head matches the earlier code audit.

  • Same-head review labels the style-prop omission non-blocking and reports 13 focused tests passing. The author claims manual two-root verification but explicitly attaches no artifact.
PR #17575Close without mergeOpen

fix(automations): keep edit dialog usable on mobile

Closure reasonDescription validation still fails with an empty HUMAN note, and the author explicitly provides no post-fix mobile browser capture. Scrollable fields and close/save/cancel reachability at 390×844 remain unproven in a running app.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: f6457ec3a49ea242c18a5409293659cb40c189a5.

Original audited head: f6457ec3a49ea242c18a5409293659cb40c189a5. The current head matches the earlier code audit.

  • Same-head review found the CSS change correct and minimal and mutation-checked its regression test. No introduced defect established.
PR #17573Close without mergeOpen

fix(terminal): restore history after reload

Closure reasonCurrently conflicts with main, fails description validation and has no post-fix reload capture. Hydration only handles the initial history tail and stops once any terminal entry exists; older-page and later REST-history restoration need completion or an explicitly narrowed scope.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 148f8ce66610fbb3eed828715f5e2a08ed0629a9.

Original audited head: 148f8ce66610fbb3eed828715f5e2a08ed0629a9. The current head matches the earlier code audit.

  • Older-page omission is source-reviewed; the later-REST/WebSocket race is a source-derived sequence, not a reproduced runtime failure. Existing tests and same-head review support recent-tail reload/dedup behavior.
PR #17560Close without mergeOpen

fix: rebuild chat UI projection when older history pages arrive

Closure reasonAlready approved by koriyoshi2041, with strong deterministic tests, successful negative controls and green CI. Closure here is solely under the required running-conversation evidence policy: its before/after artifacts are unit-test output, and live page-boundary replay is listed as optional.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 543cd5afdf58e7a0c151534bdc876a21201768ea.

Original audited head: 543cd5afdf58e7a0c151534bdc876a21201768ea. The current head matches the earlier code audit.

  • There is an existing approval from koriyoshi2041, strong deterministic store tests, successful negative controls and green CI. Same-head review found the code sound and explicitly says maintainers may accept that proof; this closure is policy-based, not proof of a defect.
PR #17428Close without mergeDraft · open

fix: keep embedded provider instances stable across Strict Mode and sibling roots

Closure reasonExplicit partial Draft: it leaves stale client/i18n globals after both sibling roots unmount and excludes async-origin ownership and the full instance-scoping work. The HUMAN note and description validation remain unfinished.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 607971c0def84d28b09949f535f6af2eb95f465a.

Original audited head: 607971c0def84d28b09949f535f6af2eb95f465a. The current head matches the earlier code audit.

  • The residual global-state limit is openly disclosed and is not claimed as a new regression. The issue is marked Part of, not Fixes. No UI recording is required merely for this nonvisual provider fix.
PR #17376Close without mergeOpen

feat: surface ask_oracle as an Oracle profile

Closure reasonCurrently conflicts with main, and the core Configure → save → ask_oracle → delete/ghost flow remains unverified against a real Agent Server. Mock screenshots only show the entry/editor, and the author explicitly defers the save/tool payoff.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 5934d9ce1f72b9c42d85395743024ebb628e30b9.

Original audited head: 5934d9ce1f72b9c42d85395743024ebb628e30b9. The current head matches the earlier code audit.

  • The newer same-head review confirms the earlier reserved-name guard-test gap is fixed and reports 155 focused tests passing. Do not repeat the older absent-tests finding or stale CI warnings. No code/security defect established.
PR #17244Close without mergeOpen

fix: keep Files and Changes views source-driven after refresh (#16949)

Closure reasonCurrently conflicts with main. Its explicit unit-only verification and terminal SVG do not show running Cloud/Local Files refresh, file-edit invalidation or the retained-path Changes status update before and after.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: 034a003ed04200c06dd4a272e3000020f638f575.

Original audited head: 034a003ed04200c06dd4a272e3000020f638f575. The current head matches the earlier code audit.

  • Same-head review finds both source-driven fixes sound and reports focused tests/CI green. An old failure comment references a previous head and is not a current code-CI failure.
PR #16639Close without mergeDraft · open

fix(agent-canvas): gate onboarding on active backend readiness

Closure reasonStill Draft, currently conflicts with main and lacks issue ready-for-dev; the sessionStorage-versus-memory dismissal choice remains for maintainer decision. Published configured-Local screenshots and mock-LLM E2E do not establish the full unconfigured/Cloud/backend-switch runtime matrix.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: fa05e8b1c94993c12c4df9b68e696703ebc0600b.

Original audited head: fa05e8b1c94993c12c4df9b68e696703ebc0600b. The current head matches the earlier code audit.

  • There is real configured-Local before/after evidence and 62 successful mock-LLM E2E cases; do not claim no runtime evidence exists. Broad scope or a competing implementation alone is not a code-defect finding.
PR #16390Close without mergeOpen

feat(conversation-panel): group conversations by workspace by default

Closure reasonCurrently conflicts with main; the required PR template and screenshot remain absent, and description/artifact checks fail. The small default preference change needs a refreshed submission before merge review.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: d3e6b0ba744ebf9d12035825e9081b7f441c2ed5.

Original audited head: d3e6b0ba744ebf9d12035825e9081b7f441c2ed5. The current head matches the earlier code audit.

  • Same-head review found no changed-line defect and reports 66 focused tests passing. The artifact-check failure is a GitHub API permissions error, not a product-code failure.
PR #16262Close without mergeDraft · open

fix: avoid import-time signal handlers

Closure reasonStill Draft, currently conflicts with main and has unresolved HUMAN/issue/validator readiness. Close the unfinished submission; its signal-lifecycle fix already has real child-process shutdown and mutation-test support.

The earlier audit recommendation and its evidence limits are retained. Current issue/PR state, reviews and checks were read from GitHub; no code or runtime verification was performed.

Evidence and review limits

Current GitHub head: ff38ad9c2b4fea247f5f8508a3de5b27d42f1ac7.

Original audited head: ff38ad9c2b4fea247f5f8508a3de5b27d42f1ac7. The current head matches the earlier code audit.

  • The latest validator failure is an unsupported --files-file argument, not a failed lifecycle test. Earlier substantive SIGHUP/import gaps were repaired and independently checked; no UI recording is needed for this nonvisual change.

Already merged or closed · 19 PRs

Removed from the active recommendations.

Review scope

This page follows the original 53 open or draft other-author PRs reviewed for issues filed by enyst or smolpaws. It is a separate queue from the creation-date reports: some issues here predate the Monday issue inventory.

Reasons come from the earlier current-head production/test diff reviews, issue requirements, CI and available functional evidence. Fresh metadata confirms PR and issue states and whether the reviewed code head has changed. Existing links to reviews or contributor recordings are evidence, not a claim that this auditor reran every branch.

The original audit independently ran focused tests and a fail-before mutation check for PR #18151, and a minimal Chromium geometry reproduction for PR #18242. Later independent checks are identified in the relevant rows. No branch runtime verification was performed during this inventory refresh.

Checked triage dataset (JSON) · Active recommendations (CSV)

OpenHands/OpenHands · checked report snapshots.