openhands-sdk 1.44.0__tar.gz → 1.44.1__tar.gz

{openhands_sdk-1.44.0 → openhands_sdk-1.44.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openhands-sdk
-Version: 1.44.0
+Version: 1.44.1
 Summary: OpenHands SDK - Core functionality for building AI agents
 Project-URL: Source, https://github.com/OpenHands/software-agent-sdk
 Project-URL: Homepage, https://github.com/OpenHands/software-agent-sdk
@@ -22,7 +22,7 @@ Requires-Dist: python-frontmatter>=1.1.0
 Requires-Dist: python-json-logger>=3.3.0
 Requires-Dist: tenacity>=9.1.2
 Requires-Dist: websockets>=12
-Requires-Dist: lmnr<0.8.0,>=0.7.56
+Requires-Dist: lmnr<0.8.0,>=0.7.60
 Requires-Dist: tree-sitter>=0.25
 Requires-Dist: tree-sitter-bash>=0.25
 Provides-Extra: boto3

{openhands_sdk-1.44.0 → openhands_sdk-1.44.1}/openhands/sdk/agent/acp_agent.py

@@ -1699,6 +1699,21 @@ class ACPAgent(AgentBase):
         ),
     )
 
+    @field_validator("agent_context")
+    @classmethod
+    def _drop_project_skills(cls, value: AgentContext | None) -> AgentContext | None:
+        """Clear ``load_project_skills`` — ACP CLIs read the repo themselves.
+
+        Claude Code, Codex and Gemini already ingest ``AGENTS.md`` / ``CLAUDE.md``
+        and their own project skills from the session cwd, so loading them here
+        too would put that content in the prompt twice. Normalised rather than
+        rejected: callers legitimately set the flag on a shared context they also
+        use for OpenHands agents (#4019).
+        """
+        if value is None or not value.load_project_skills:
+            return value
+        return value.model_copy(update={"load_project_skills": False})
+
     def model_post_init(self, __context: object) -> None:
         super().model_post_init(__context)
         # Propagate the actual model name to the sentinel LLM and its

{openhands_sdk-1.44.0 → openhands_sdk-1.44.1}/openhands/sdk/llm/utils/verified_models.py

@@ -69,6 +69,7 @@ VERIFIED_DEEPSEEK_MODELS = [
     "deepseek-chat",
     "deepseek-v3.2-reasoner",
     "deepseek-v4-pro",
+    "deepseek-v4-flash",
 ]
 
 VERIFIED_MOONSHOT_MODELS = [
@@ -131,6 +132,7 @@ VERIFIED_OPENHANDS_MODELS = [
     "deepseek-chat",
     "deepseek-v3.2-reasoner",
     "deepseek-v4-pro",
+    "deepseek-v4-flash",
     "kimi-k3",
     "kimi-k2-thinking",
     "kimi-k2.6",

{openhands_sdk-1.44.0 → openhands_sdk-1.44.1}/openhands/sdk/profiles/agent_profile.py

@@ -224,9 +224,9 @@ class ACPAgentProfile(AgentProfileBase):
         ),
     )
     # No skill-selection field: ACP agents own their tooling and prompt
-    # construction, so no user/public discovered skills are injected. (Only
-    # repo-scoped project skills reach an ACP agent, via the resolver's
-    # ``load_project_skills`` — see #4019 for whether even those should.)
+    # construction. Project skills never reach one (the CLI reads the repo
+    # itself, #4019); whether any managed skill does is a deployment choice the
+    # caller expresses through ``resolve_agent_profile``'s ``available_skills``.
     acp_server: ACPServerKind = Field(
         default="claude-code",
         description=(

{openhands_sdk-1.44.0 → openhands_sdk-1.44.1}/openhands/sdk/profiles/resolver.py

@@ -261,18 +261,21 @@ def _build_openhands_settings(
 def _build_acp_settings(
     profile: ACPAgentProfile,
     mcp_config: dict[str, MCPServer],
+    managed_skills: list[Skill],
 ) -> AgentSettingsConfig:
     """Compose the resolved ``ACPAgentSettings`` from a profile.
 
     ``acp_command`` is stored as a shell string and split into the settings'
     token list. No credential is set — provider creds ride
-    ``state.secret_registry``. ACP profiles carry no user/public skills (the ACP
-    subprocess owns its context), so ``agent_context`` has no discovered skills;
-    it is always built (never ``None``) only so ``load_project_skills=True``
-    reaches ``LocalConversation``'s lazy load (``current_datetime=None`` matches
-    ACP's no-timestamp convention). Caveat: an ACP CLI that already ingests repo
-    files (e.g. AGENTS.md) may then see that content twice (#4019). A ``custom``
-    server has no default command, so one must be supplied.
+    ``state.secret_registry``.
+
+    ``load_project_skills`` stays ``False``: an ACP CLI reads ``AGENTS.md`` /
+    ``CLAUDE.md`` and its own project skills from the session cwd, so loading
+    them here would duplicate that content in the prompt (#4019).
+    ``managed_skills`` is what the *deployment* chose to inject — empty when the
+    ACP CLI can reach its own host configuration, non-empty in a container where
+    it cannot. ``current_datetime=None`` matches ACP's no-timestamp convention.
+    A ``custom`` server has no default command, so one must be supplied.
     """
     command = shlex.split(profile.acp_command) if profile.acp_command else []
     if profile.acp_server == "custom" and not command:
@@ -281,7 +284,7 @@ def _build_acp_settings(
             "default launch command to fall back to"
         )
     agent_context = AgentContext(
-        skills=[], current_datetime=None, load_project_skills=True
+        skills=managed_skills, current_datetime=None, load_project_skills=False
     )
     payload = {
         "schema_version": AGENT_SETTINGS_SCHEMA_VERSION,
@@ -313,10 +316,10 @@ def resolve_agent_profile(
     decrypted by the caller (the agent-server runs settings decryption
     before calling). ``available_skills`` is the server-discovered skill catalog
     (the agent-server caller passes the result of ``load_all_skills``); an
-    OpenHands profile keeps all of it except the names in ``disabled_skills``.
-    ``None`` means discovery was not run or failed: no catalog, so the resolved
-    agent gets no user/public skills (project skills, loaded separately by
-    ``LocalConversation``, are unaffected). Unlike the ``mcp_server_refs``
+    OpenHands profile keeps all of it except the names in ``disabled_skills``,
+    and an ACP profile keeps all of it (it has no deny-list). ``None`` means the
+    caller injected no catalog — discovery was not run, failed, or, for ACP, the
+    deployment leaves skill sourcing to the CLI. Unlike the ``mcp_server_refs``
     allow-list, the ``disabled_skills`` deny-list can never dangle, so this
     never raises for skills. ``cipher`` decrypts the referenced LLM profile.
 
@@ -340,7 +343,9 @@ def resolve_agent_profile(
             ) from e
         return _build_openhands_settings(profile, llm, filtered_mcp, filtered_skills)
 
-    return _build_acp_settings(profile, filtered_mcp)
+    return _build_acp_settings(
+        profile, filtered_mcp, _apply_disabled_skills(available_skills, [])
+    )
 
 
 def resolve_agent_profile_dry_run(
@@ -375,15 +380,17 @@ def resolve_agent_profile_dry_run(
             "MCP server(s) not configured: " + ", ".join(dangling)
         )
 
-    # Skill selection report (OpenHands only; ACP injects no user/public skills).
-    # Deny-list semantics: the catalog minus disabled names, never dangling.
+    # Skill selection report. Deny-list semantics: the catalog minus disabled
+    # names, never dangling. An ACP profile has no deny-list of its own — its
+    # catalog is whatever the deployment injects (empty unless the caller passes
+    # one), and never includes project skills (#4019).
     if isinstance(profile, OpenHandsAgentProfile):
         filtered_skills = _apply_disabled_skills(
             available_skills, profile.disabled_skills
         )
         diagnostics.disabled_skills = profile.disabled_skills
     else:
-        filtered_skills = []
+        filtered_skills = _apply_disabled_skills(available_skills, [])
     diagnostics.resolved_skills = [s.name for s in filtered_skills]
 
     llm: LLM | None = None
@@ -431,7 +438,7 @@ def resolve_agent_profile_dry_run(
                     profile, llm, filtered_mcp, filtered_skills
                 )
             else:
-                settings = _build_acp_settings(profile, filtered_mcp)
+                settings = _build_acp_settings(profile, filtered_mcp, filtered_skills)
             # No expose context => secrets redacted (mcp env/headers, llm api_key).
             diagnostics.resolved_settings = settings.model_dump(mode="json")
         except Exception as e:

{openhands_sdk-1.44.0 → openhands_sdk-1.44.1}/openhands/sdk/tool/registry.py

@@ -153,7 +153,12 @@ def resolve_tool(
         resolver = _REG.get(tool_spec.name)
 
     if resolver is None:
-        raise KeyError(f"ToolDefinition '{tool_spec.name}' is not registered")
+        from openhands.sdk.tool.builtins import BUILT_IN_TOOL_CLASSES
+
+        tool_class = BUILT_IN_TOOL_CLASSES.get(tool_spec.name)
+        if tool_class is None:
+            raise KeyError(f"ToolDefinition '{tool_spec.name}' is not registered")
+        resolver = _resolver_from_subclass(tool_spec.name, tool_class)
 
     params = dict(tool_spec.params)
     response_schema = params.pop("response_schema", None)

{openhands_sdk-1.44.0 → openhands_sdk-1.44.1}/openhands_sdk.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: openhands-sdk
-Version: 1.44.0
+Version: 1.44.1
 Summary: OpenHands SDK - Core functionality for building AI agents
 Project-URL: Source, https://github.com/OpenHands/software-agent-sdk
 Project-URL: Homepage, https://github.com/OpenHands/software-agent-sdk
@@ -22,7 +22,7 @@ Requires-Dist: python-frontmatter>=1.1.0
 Requires-Dist: python-json-logger>=3.3.0
 Requires-Dist: tenacity>=9.1.2
 Requires-Dist: websockets>=12
-Requires-Dist: lmnr<0.8.0,>=0.7.56
+Requires-Dist: lmnr<0.8.0,>=0.7.60
 Requires-Dist: tree-sitter>=0.25
 Requires-Dist: tree-sitter-bash>=0.25
 Provides-Extra: boto3

{openhands_sdk-1.44.0 → openhands_sdk-1.44.1}/openhands_sdk.egg-info/requires.txt

@@ -13,7 +13,7 @@ python-frontmatter>=1.1.0
 python-json-logger>=3.3.0
 tenacity>=9.1.2
 websockets>=12
-lmnr<0.8.0,>=0.7.56
+lmnr<0.8.0,>=0.7.60
 tree-sitter>=0.25
 tree-sitter-bash>=0.25
 

{openhands_sdk-1.44.0 → openhands_sdk-1.44.1}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "openhands-sdk"
-version = "1.44.0"
+version = "1.44.1"
 description = "OpenHands SDK - Core functionality for building AI agents"
 
 requires-python = ">=3.12"
@@ -20,7 +20,7 @@ dependencies = [
     "python-json-logger>=3.3.0",
     "tenacity>=9.1.2",
     "websockets>=12",
-    "lmnr>=0.7.56,<0.8.0",
+    "lmnr>=0.7.60,<0.8.0",
     "tree-sitter>=0.25",
     "tree-sitter-bash>=0.25",
 ]