Skip to content

CLI: bump agent-sdk #11710

Merged
malhotra5 merged 4 commits intomainfrom
analyzer-fix
Nov 11, 2025
Merged

CLI: bump agent-sdk #11710
malhotra5 merged 4 commits intomainfrom
analyzer-fix

Conversation

@malhotra5
Copy link
Copy Markdown
Collaborator

@malhotra5 malhotra5 commented Nov 11, 2025 •
edited by github-actions Bot
Loading

Summary of PR

Change Type

  • Bug fix
  • New feature
  • Breaking change
  • Refactor
  • Other (dependency update, docs, typo fixes, etc.)

Checklist

  • I have read and reviewed the code and I understand what the code is doing.
  • I have tested the code to the best of my ability and ensured it works as expected.

Fixes

Fixes #11378

Resolves #(issue)

Release Notes

  • Include this change in the Release Notes.

To run this PR locally, use the following command:

GUI with Docker:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.openhands.dev/openhands/runtime:381c61f-nikolaik   --name openhands-app-381c61f   docker.openhands.dev/openhands/openhands:381c61f

CLI with uvx:

uvx --python 3.12 --from git+https://github.com/OpenHands/OpenHands@analyzer-fix#subdirectory=openhands-cli openhands

malhotra5 and others added 2 commits November 11, 2025 13:30
@malhotra5
use new analyzer api
e6b9613
@openhands-agent
Update openhands-cli tests to use new conversation.set_security_analy…
d17d430 
…zer() API

- Remove deprecated security_analyzer parameters from agent creation functions
- Update security analyzer tests to use conversation.set_security_analyzer() instead of agent-level parameters
- Update conversation runner tests to use new security analyzer API
- Clean up unused imports
- All 143 tests passing

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

github-actions Bot commented Nov 11, 2025 •
edited
Loading

Coverage report

This PR does not seem to contain any modification to coverable code.

@malhotra5 malhotra5 enabled auto-merge (squash) November 11, 2025 20:07
@malhotra5 malhotra5 merged commit 0a6b76c into main Nov 11, 2025
26 checks passed
@malhotra5 malhotra5 deleted the analyzer-fix branch November 11, 2025 20:29
chuckbutkus pushed a commit that referenced this pull request Nov 13, 2025
@malhotra5 @openhands-agent @chuckbutkus
CLI: bump agent-sdk (#11710)
d8c93fe 
Co-authored-by: openhands <openhands@all-hands.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xingyaoww xingyaoww xingyaoww approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

CLI crashes with FunctionCallValidationError: Missing required parameter 'security_risk' for execute_bash

3 participants

@malhotra5 @xingyaoww @openhands-agent