Skip to content

feat: add OpenHands CLI as an executor option#2

Open
anchapin wants to merge 1 commit intoTinyAGI:mainfrom
anchapin:feat/openhands-executor
Open

feat: add OpenHands CLI as an executor option#2
anchapin wants to merge 1 commit intoTinyAGI:mainfrom
anchapin:feat/openhands-executor

Conversation

@anchapin
Copy link
Copy Markdown

@anchapin anchapin commented Mar 6, 2026

This PR adds OpenHands CLI as a third executor option to the Fractals task orchestrator.

Changes

  • Add 'openhands' to ExecutorProvider type in types.ts
  • Add invokeOpenHands() function in executor.ts using 'openhands exec' command
  • Add 'openhands' to executor validation in server.ts
  • Add 'openhands' to Session interface in web/src/lib/api.ts
  • Add OpenHands button to executor selector in web UI
  • Update README with new executor documentation

- Add 'openhands' to ExecutorProvider type in types.ts
- Add invokeOpenHands() function in executor.ts using 'openhands exec' command
- Add 'openhands' to executor validation in server.ts
- Add 'openhands' to Session interface in web/src/lib/api.ts
- Add OpenHands button to executor selector in web UI
- Update README with new executor documentation

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

@xkonjin xkonjin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Quick review pass:

  • Main risk area here is auth/session state and stale credential handling.
  • I didn’t see targeted regression coverage in the diff; please add or point CI at a focused test for the changed path in README.md, executor.ts, server.ts (+3 more).
  • Before merge, I’d smoke-test the behavior touched by README.md, executor.ts, server.ts (+3 more) with malformed input / retry / rollback cases, since that’s where this class of change usually breaks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants