diff --git a/hermes_cli/web_server.py b/hermes_cli/web_server.py index 31e28cc2996..32a8fc67a50 100644 --- a/hermes_cli/web_server.py +++ b/hermes_cli/web_server.py @@ -8105,7 +8105,8 @@ async def install_skill_hub(body: SkillInstallRequest, profile: Optional[str] = raise HTTPException(status_code=400, detail="identifier is required") try: proc = _spawn_hermes_action( - _profile_cli_args(body.profile or profile) + ["skills", "install", identifier], + _profile_cli_args(body.profile or profile) + + ["skills", "install", identifier, "--yes"], "skills-install", ) except HTTPException: @@ -8843,7 +8844,7 @@ async def create_profile_endpoint(body: ProfileCreate): continue try: proc = _spawn_hermes_action( - ["-p", body.name, "skills", "install", ident], + ["-p", body.name, "skills", "install", ident, "--yes"], "skills-install", ) hub_installs.append({"identifier": ident, "pid": proc.pid}) diff --git a/tests/hermes_cli/test_web_server.py b/tests/hermes_cli/test_web_server.py index 28b6ee3b019..4782176caf4 100644 --- a/tests/hermes_cli/test_web_server.py +++ b/tests/hermes_cli/test_web_server.py @@ -2690,7 +2690,12 @@ class TestNewEndpoints: assert data["hub_installs"] == [{"identifier": "someuser/some-skill", "pid": 4321}] # Hub install was scoped to the new profile. - assert spawned == [(["-p", "builder", "skills", "install", "someuser/some-skill"], "skills-install")] + assert spawned == [ + ( + ["-p", "builder", "skills", "install", "someuser/some-skill", "--yes"], + "skills-install", + ) + ] # Verify the writes landed in the NEW profile's config, not the root. prof_dir = get_hermes_home() / "profiles" / "builder" diff --git a/tests/hermes_cli/test_web_server_skills_profiles.py b/tests/hermes_cli/test_web_server_skills_profiles.py index 9a131bbb246..76325d628f2 100644 --- a/tests/hermes_cli/test_web_server_skills_profiles.py +++ b/tests/hermes_cli/test_web_server_skills_profiles.py @@ -178,7 +178,10 @@ class TestProfileScopedHubActions: ) assert resp.status_code == 200 assert calls == [ - (["-p", "worker_alpha", "skills", "install", "official/demo"], "skills-install") + ( + ["-p", "worker_alpha", "skills", "install", "official/demo", "--yes"], + "skills-install", + ) ] def test_hub_install_without_profile_keeps_legacy_argv( @@ -200,7 +203,7 @@ class TestProfileScopedHubActions: "/api/skills/hub/install", json={"identifier": "official/demo"} ) assert resp.status_code == 200 - assert calls == [["skills", "install", "official/demo"]] + assert calls == [["skills", "install", "official/demo", "--yes"]] def test_hub_install_unknown_profile_404(self, client, isolated_profiles): resp = client.post(