Upgrade to 0.43.0 by shabbir-shakudo · Pull Request #3 · devsentient/OpenHands

shabbir-shakudo · 2025-06-25T16:51:47Z

✍️ Custom Changes in Fork

All recent customizations are tracked in the upgrade-to-0.42.0 branch of the DevSentient OpenHands fork.

Key Modifications

Frontend UI Behavior
- File: frontend/src/hooks/query/use-settings.ts
- Change: Disabled a popup that was shown by default.
Agent Startup Timeout
- File: openhands/runtime/impl/action_execution/action_execution_client.py
- Change: Increased the wait time for agent startup to improve reliability.
Custom Docker Runtime Enhancements
- File: openhands/runtime/impl/docker/docker_runtime.py
- Custom functions and variables are prefixed with shak_
- Changes to OpenHands functions:
  - init_container: Custom initialization logic and dynamic workspace path.
  - vscode_url: Constructs VS Code exposure URL
  - web_hosts: Manages multi-host application exposure
  - _attach_to_container: Custom logic for container attachment
  - close: to cleanup the workspace on container

These changes support enhanced startup control, URL generation for multi-host apps, and better lifecycle management of containers.

Give a summary of what the PR does, explaining any non-trivial design decisions

DockerRuntime Enhancements:
- Added dynamic port assignment support for the application.
- Improved the handling and backup of app port configuration, including resetting logic and validation.
- Introduced a port conversion utility to better support VSCode URL construction.
- Updated the host URL format for app secrets and enhanced secret generation methods.
- Dynamic host mount workspace paths
- Clearing the workspace when containers are destroyed
Configuration Adjustments:
- Revised default values in configuration (e.g., changed default debug settings and the use_host_network flag in SandboxConfig).
- Updated the VSCODE_PORT_RANGE to correct its upper limit.
- Made refinements to dynamic workspace path construction and other runtime settings for better consistency.
Additional Fixes and Refactoring:
- Removed unused code segments – for example, unused backup references for app ports.
- Merged some upstream changes (from upgrade-to-0.41.0), ensuring consistency with the latest remote updates.
- Introduced improvements to error handling, especially around runtime connection and dependency checks.
- Carried out minor performance optimizations and refactorings to simplify the code.

Link of any specific issues this addresses

…ates (OpenHands#8617) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: amanape <83104063+amanape@users.noreply.github.com>

…e" (OpenHands#8624)

Co-authored-by: openhands <openhands@all-hands.dev>

…config (OpenHands#8620) Co-authored-by: openhands <openhands@all-hands.dev>

Co-authored-by: openhands <openhands@all-hands.dev> Co-authored-by: Robert Brennan <contact@rbren.io>

… api key conversations (OpenHands#8626)

Co-authored-by: openhands <openhands@all-hands.dev> Co-authored-by: Engel Nyst <enyst@users.noreply.github.com>

…in the version-all group (OpenHands#8611) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Co-authored-by: openhands <openhands@all-hands.dev>

…rmation (OpenHands#8558) Co-authored-by: openhands <openhands@all-hands.dev> Co-authored-by: mamoodi <mamoodiha@gmail.com>

…nds#8634) Co-authored-by: openhands <openhands@all-hands.dev>

Co-authored-by: Engel Nyst <enyst@users.noreply.github.com>

…evstral (OpenHands#8642)

Co-authored-by: tofarr <tofarr@gmail.com> Co-authored-by: Rohit Malhotra <rohitvinodmalhotra@gmail.com>

…8264) Co-authored-by: openhands <openhands@all-hands.dev> Co-authored-by: Engel Nyst <enyst@users.noreply.github.com> Co-authored-by: Engel Nyst <engel.nyst@gmail.com>

Co-authored-by: openhands <openhands@all-hands.dev>

…penHands#8635) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: amanape <83104063+amanape@users.noreply.github.com>

…nHands#8660) Co-authored-by: openhands <openhands@all-hands.dev>

Co-authored-by: czlll <gangda@huaihe.usc.edu> Co-authored-by: Hoang Tran <descience.thh10@gmail.com>

…xt dependency (OpenHands#8659) Co-authored-by: openhands <openhands@all-hands.dev>

Co-authored-by: openhands <openhands@all-hands.dev>

…ervation when visualized in frontend (OpenHands#8644) Co-authored-by: openhands <openhands@all-hands.dev>

… converted port string

…erRuntime

…get_hosts

…mapping logic

…ing logic

…figuration

…me class

…Runtime class

…ersion utility

… DockerRuntime class

…leaner output

…mic port assignment

malhotra5 and others added 30 commits May 21, 2025 19:17

[Fix]: make mcp config optional in settings (OpenHands#8622)

8b08958

Revert "Fix for issue where initial env vars are not passed to runtim…

2bd10de

…e" (OpenHands#8624)

specify condenser config for evals (OpenHands#8177)

637cb07

Co-authored-by: openhands <openhands@all-hands.dev>

fix(mcp): fix the config conflict between microagent MCP tools & MCP …

70573dc

…config (OpenHands#8620) Co-authored-by: openhands <openhands@all-hands.dev>

Nested Conversation Support (OpenHands#8588)

a3d1a92

Co-authored-by: openhands <openhands@all-hands.dev> Co-authored-by: Robert Brennan <contact@rbren.io>

[Fix]: Don't start conversations without user instructions for remote…

ac87ff8

… api key conversations (OpenHands#8626)

[Refactor]: Update resolver instructions (OpenHands#8601)

097f757

Co-authored-by: openhands <openhands@all-hands.dev> Co-authored-by: Engel Nyst <enyst@users.noreply.github.com>

Split IssueResolver class into a separate file (OpenHands#8619)

cfa2725

chore(deps-dev): bump swagger-ui-dist from 5.21.0 to 5.22.0 in /docs …

66f33bf

…in the version-all group (OpenHands#8611) Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

fix(frontend): MCP tool name rendering (OpenHands#8623)

2906a90

Co-authored-by: openhands <openhands@all-hands.dev>

Fix Cloud documentation structure and consolidate issue resolver info…

13f9acd

…rmation (OpenHands#8558) Co-authored-by: openhands <openhands@all-hands.dev> Co-authored-by: mamoodi <mamoodiha@gmail.com>

docs: add documentation for adding user settings and tooltips (OpenHa…

be78cc0

…nds#8634) Co-authored-by: openhands <openhands@all-hands.dev>

Add option to run patch evaluation on Modal (OpenHands#8607)

3980ba5

Co-authored-by: Engel Nyst <enyst@users.noreply.github.com>

add claude 4 to prompt caching and fn call list; do not add icl for d…

926b425

…evstral (OpenHands#8642)

Release 0.39.1 (OpenHands#8608)

6d3c217

Co-authored-by: tofarr <tofarr@gmail.com> Co-authored-by: Rohit Malhotra <rohitvinodmalhotra@gmail.com>

Fix gitlab header error in console (OpenHands#8631)

f7cdf47

Add CLIRuntime implementation for local command execution (OpenHands#…

50d4c79

…8264) Co-authored-by: openhands <openhands@all-hands.dev> Co-authored-by: Engel Nyst <enyst@users.noreply.github.com> Co-authored-by: Engel Nyst <engel.nyst@gmail.com>

Downgrade info to debug log (OpenHands#8643)

8cc0399

Fix unlocalized elements in frontend (OpenHands#8650)

f3a127a

Co-authored-by: openhands <openhands@all-hands.dev>

fix(frontend): Fix action buttons display bugs in chat interface (Ope…

fa5b522

…nHands#8660) Co-authored-by: openhands <openhands@all-hands.dev>

integrate LocAgent into OpenHands (OpenHands#7371)

efe287c

Co-authored-by: czlll <gangda@huaihe.usc.edu> Co-authored-by: Hoang Tran <descience.thh10@gmail.com>

refactor(frontend): Simplify useConversation hook by removing conte…

20983a2

…xt dependency (OpenHands#8659) Co-authored-by: openhands <openhands@all-hands.dev>

feat(agent): First-class Search API support via MCP (OpenHands#8638)

a40443f

Co-authored-by: openhands <openhands@all-hands.dev>

docs: fix debug command (OpenHands#8654)

ca9fe7c

Fix tip link (OpenHands#8666)

2bb1b7b

Fix so redirect URL is in OAuth state (OpenHands#8653)

92f8061

Expose partial MCP server addition failure logs (OpenHands#8655)

b0b5a6c

feat(frontend): Display MCP tool name and arguments as JSON in MCPObs…

5e43dba

…ervation when visualized in frontend (OpenHands#8644) Co-authored-by: openhands <openhands@all-hands.dev>

shabbir-shakudo added 29 commits June 13, 2025 09:53

feat: dynamically construct workspace path using vscode port string

66f109b

fix: update vscode URL construction to use converted port string

37d0f39

fix: update dynamic workspace path and vscode URL construction to use…

3836cc3

… converted port string

fix: update dynamic workspace path configuration for DockerRuntime

f1f0f80

fix: remove unused workspace_mount_path_in_sandbox assignment in Dock…

76e6b55

…erRuntime

fix: add cleanup for host workspace directory on container stop

f978b7e

fix: update logging and dynamic workspace path configuration for Shakudo

0baaf48

fix: change default value of use_host_network to True in SandboxConfig

c59552d

fix: change default value of use_host_network to False in SandboxConfig

80d4574

fix: update VSCODE_PORT_RANGE to correct upper limit

b7ff6e0

fix: update VSCODE_PORT_RANGE to correct upper limit

bca0ee4

fix: change default value of debug to True in OpenHandsConfig

a57825c

add logging for active host and runtime details in useActiveHost and …

402c1ce

…get_hosts

fix: initialize port_mapping as an empty dictionary and restore port …

be70b87

…mapping logic

fix: change port_mapping initialization to None and restore port mapp…

09daf75

…ing logic

fix: update app port initialization logic to respect host network con…

a278b8f

…figuration

add logging for container app ports in DockerRuntime class

7df1a45

add logging for available port discovery and selection in DockerRunti…

f3ae7c2

…me class

fix: backup app_ports before logging and enhance logging for web hosts

3dad613

fix: add backup for app_ports in DockerRuntime class

3136cd9

fix: enhance logging with Shakudo prefix for better clarity in Docker…

2224915

…Runtime class

fix: reset and backup app_ports in DockerRuntime class; add port conv…

c0868dd

…ersion utility

fix: update shak_convert_any_port_to_string method to use self reference

ecc3aa9

fix: remove unused backup for app_ports in DockerRuntime class

8c0a1a6

feat: add secret generation for app ports in DockerRuntime class

d0aa9e0

fix: improve secret generation method to ensure valid token format in…

423c775

… DockerRuntime class

fix: update host URL format in DockerRuntime class for app secrets

845a6c8

refactor: remove debug logging statements in various components for c…

14bdf4f

…leaner output

fix: enhance app port handling in DockerRuntime class to support dyna…

661e2cc

…mic port assignment

shabbir-shakudo requested a review from yijie-shakudo June 25, 2025 16:51

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Upgrade to 0.43.0#3

Upgrade to 0.43.0#3
shabbir-shakudo wants to merge 1259 commits intomainfrom
upgrade-to-0.42.0

shabbir-shakudo commented Jun 25, 2025 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

Conversation

shabbir-shakudo commented Jun 25, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

✍️ Custom Changes in Fork

Key Modifications

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

20 participants

shabbir-shakudo commented Jun 25, 2025 •

edited

Loading