Open PRs by enyst and smolpaws in five OpenHands repositories. See what each changes, its current review, and the next step.
Showing all 28 pull requests.
No PR matches this search. Clear the search or choose another author.
20 open pull requests
| Pull request | What it does | Status | Review | Next step |
|---|---|---|---|---|
| OpenHands/OpenHands #18346 · fix(lint): use semantic colors for Git Sync feedback | Applies semantic success and error colors to Git Sync activity and error states. |
Draft
CI passes; Docker is skipped. |
No formal review yet. |
|
| OpenHands/OpenHands #18334 · docs(verify-openhands): backfill all existing open bugs | Backfills the open-bug feature map and adds explicit mapped, not-reproduced, and out-of-scope outcomes. |
Draft
CI and Docker workflows pass. |
No formal review yet. |
|
| OpenHands/OpenHands #18333 · fix(themes): make warning feedback theme-aware | Adds a readable warning text token and replaces six raw warning colors across Home, Apps, and onboarding. |
Review next
Ready and mergeable. CI passes; Docker is skipped. |
No formal review yet. |
|
| OpenHands/OpenHands #18330 · fix(themes): use readable validation and service errors | Replaces raw red styles in validation and service errors with semantic feedback tokens. |
Approved
Ready and mergeable. CI passes; Docker is skipped. |
Vasco approval · 12 OctApproved after checking the theme-aware error styles and host-override browser regression. |
|
| OpenHands/OpenHands #18327 · fix(themes): use readable backend focus and valid scrim opacity | Uses the focus role for backend outlines, improves contrast, and fixes the exported mobile drawer scrim opacity. |
Cannot merge
Ready. GitHub says it cannot merge. CI passes; Docker is skipped. |
No formal review yet. |
|
| OpenHands/OpenHands #18321 · fix(themes): make backend connection feedback readable | Uses host-overridable feedback tokens for backend connection status and error messages. |
Approved
Ready and mergeable. CI passes; Docker is skipped. |
Vasco + bot approvals · 12 OctBoth reviews approved the backend-feedback change; the bot verified the host override and contrast checks. |
|
| OpenHands/OpenHands #18320 · chore(lint): remove redundant text-medium color exception | Removes an obsolete text-medium exception from the raw-color lint rule without changing runtime styles. |
Approved
Ready and mergeable. CI passes; Docker is skipped. |
Vasco + bot approvals · 12 OctBoth reviews approved the narrow lint-rule change; the bot verified the current head. |
|
| OpenHands/OpenHands #18317 · docs(verify-openhands): add pstack's reconcile step and sharper live-pass rules | Adds a reconciliation phase and tighter recovery rules to the verify-openhands skill. |
Draft
CI and Docker workflows pass. |
No formal review yet. |
|
| OpenHands/OpenHands #18311 · fix: fail the launch when the automation backend exits during startup | The launcher now waits for automation health and stops the stack if that backend exits during startup. This makes an automation migration failure visible at once. |
Review next
Ready and mergeable. CI, Docker, and SDK version sync pass. |
No formal review yet. |
|
| OpenHands/OpenHands #18307 · fix: pin the automation KV secret so session-key rotation keeps KV data readable | The launchers now keep one automation encryption key when the session API key changes. This preserves access to saved automation KV data after key rotation. |
Review next
Ready and mergeable. Current Actions workflows pass; the latest formal approval predates this head. |
Bot approval · earlier head · 12 OctThe current head changed after this approval. Request a fresh review before merge. |
|
| OpenHands/OpenHands #18296 · docs(verify-openhands): carry over #18213's map corrections, re-checked on main | Carries forward feature-map corrections from a closed PR after checking them again on current main. It updates expected UI behavior and links known bugs. |
Cannot merge
Ready. GitHub says it cannot merge; CI and Docker workflows pass. |
No formal review yet. |
|
| OpenHands/OpenHands #17558 · feat: preserve Insider Cat context across Canvas views | Adds a persistent Insider Cat companion in Canvas and keeps its saved identity across views. It is part of the experimental voice project. |
Cannot merge
Draft. GitHub says it cannot merge; CI passes and Docker is skipped. |
No formal review yet. |
|
| OpenHands/software-agent-sdk #5660 · fix(agent-server): treat an empty OH_SECRET_KEY as unset | Treats an empty Agent Server encryption key as unset and uses the existing fallback. The change still accepts whitespace-only keys. |
Needs fixes
Ready. Workflows pass, but GitHub says it cannot merge. |
Bot comment · 11 OctThe review found that a whitespace-only OH_SECRET_KEY still becomes a weak encryption key; this gap remains open. |
|
| OpenHands/software-agent-sdk #5621 · Add verify-agent-server: an executable feature map and control CLI for the Agent Server API | Adds an executable map and command-line tool to test Agent Server features end to end. The first run found several bugs now tracked separately. |
Review next
Ready and mergeable. The latest workflow runs pass. |
Bot comment · 11 OctThe bot found two recipe issues on an earlier head; the author updated them afterward. The current head needs a fresh review. |
|
| OpenHands/software-agent-sdk #5180 · feat(agent-server): realtime voice broker for Insider controller conversations | Adds an experimental realtime voice broker so Insider Cat voice can use the same saved controller conversation as text. It also manages credentials, handoffs, and call teardown. |
Draft
The Agent Server and TypeScript integration workflows fail; other listed workflows pass. |
No formal review yet. |
|
| OpenHands/docs #928 · docs(agent-canvas): recover automations after the API key changed | Explains how to recover automations when an older Canvas install rotated the key used for encrypted KV data. It covers restoring the old key or clearing unreadable KV rows. |
Cannot merge
Ready. The broken-link check passes; GitHub says it cannot merge. |
No formal review yet. |
|
| OpenHands/docs #926 · docs(sdk): distinguish local fork tags from remote titles | Updates the SDK fork guide: local fork titles no longer live in tags, while remote titles remain separate metadata. It also explains the local title deprecation. |
Cannot merge
Ready. The broken-link check passes; GitHub says it cannot merge. |
No formal review yet. |
|
| OpenHands/docs #608 · docs: add Agent Canvas Docker observability guidance | Explains Agent Canvas Docker tracing and how to inspect conversations through the backend API. It includes collector networking and API-key location notes. |
Cannot merge
Ready. The broken-link check passes; GitHub says it cannot merge. |
No formal review yet. |
|
| OpenHands/automation #577 · feat: add verify-automation skill with agent-facing API feature map and control CLI | Adds a verification skill, control CLI, and runnable feature map for the automation API and background jobs. |
Approved
Ready and mergeable. CI, tests, and Docker workflows pass. |
Bot approvals · 12 OctThe latest approval rechecked the current head; the listed CI, test, and Docker workflows pass. |
|
| OpenHands/automation #576 · fix: make SQLite migrations atomic and repair partial 029/030 states | Makes SQLite automation migrations transactional and repairs databases left half-upgraded at revisions 029 or 030. This addresses the database failure in #18299. |
Approved
Ready and mergeable. CI, tests, and Docker workflows pass. |
Bot approvals · 12 OctThe latest approval covers the current head; the reviewer confirmed the SQLite recovery tests and green workflows. |
8 open pull requests
| Pull request | What it does | Status | Review | Next step |
|---|---|---|---|---|
| OpenHands/OpenHands #18313 · refactor(frontend): remove low-value tests and simplify | Remove tests that repeat source text or styling details, while keeping behavior tests and simplifying two settings navigation wrappers. |
Review next
Ready and mergeable. CI passes; Docker is skipped. |
No formal review yet. |
|
| OpenHands/OpenHands #18212 · fix(home): hide automation surfaces without interface manifest | Hide Home automation links and stop manifest-error toasts when no automation interface manifest is available. |
Cannot merge
Draft. GitHub says it cannot merge; CI passes and Docker is skipped. |
No formal review yet. |
|
| OpenHands/OpenHands #18208 · fix: render /btw API errors without raw transport JSON | Show a clear plain-text error in the /btw card instead of raw transport JSON and a broken link. |
Draft
CI passes; Docker is skipped. |
No formal review yet. |
|
| OpenHands/OpenHands #18205 · fix: focus provider connection menu actions after portal mount | Move keyboard focus into a provider connection menu when it opens so arrow keys work at once. |
Draft
CI passes; Docker is skipped. |
No formal review yet. |
|
| OpenHands/OpenHands #18197 · fix(a11y): contain focus in Canvas App trust confirmation | Keep keyboard focus inside the Canvas App trust dialog and return it to the opener after the dialog closes. |
Draft
CI passes; Docker is skipped. |
No formal review yet. |
|
| OpenHands/OpenHands #17585 · Weekly test sweep: remove low-value tests + simplify | Remove tests that only repeat source text and delete small helpers used only by those tests. |
Cannot merge
Ready. GitHub says it cannot merge; CI passes and Docker is skipped. |
Bot comment · 1 OctThe bot found no code defect, but flagged the title, empty human note, and missing visual proof. |
|
| OpenHands/OpenHands #16278 · fix: guard Automate tab against catalog entry with missing `requires` | Prevent a malformed automation catalog entry from blanking the Automate tab. |
Cannot merge
Ready. GitHub says it cannot merge; listed workflows pass. |
Bot comment · 4 AugThe bot supported the small fix, but GitHub recorded its review as COMMENTED, not APPROVED. |
|
| OpenHands/docs #913 · docs(sdk): explain prompt cache capability fallback | Explain automatic prompt cache markers, model fallback, and cache metrics in the SDK guide. |
Cannot merge
Draft. The broken-link check passes; GitHub says it cannot merge. |
No formal review yet. |
GitHub PR search used author:enyst is:open and author:smolpaws is:open in OpenHands/OpenHands, OpenHands/software-agent-sdk, OpenHands/docs, OpenHands/extensions, and OpenHands/automation. It returned 28 PRs open at the time shown above.
For each PR, I read its title and description, draft and mergeability state, submitted review events, and workflow run summaries for the current head commit. The Actions list is the first page returned by GitHub. “Approved” means GitHub recorded an APPROVED review; a COMMENTED review stays a comment. Skipped workflows are described as skipped, not passed. When a review predates the current head, the row says so.
This is a static snapshot and does not update itself. PR state, reviews, mergeability, and checks can change after the timestamp. “Ready” means the PR is not a draft; it does not mean a reviewer approved it. The short sentences follow the style of ASD-STE100 Simplified Technical English.