diff --git a/gateway/platforms/api_server.py b/gateway/platforms/api_server.py index 68221498f3b7..559403d1af97 100644 --- a/gateway/platforms/api_server.py +++ b/gateway/platforms/api_server.py @@ -2705,11 +2705,7 @@ async def _handle_skills(self, request: "web.Request") -> "web.Response": category), the same set ``/skills list`` shows.""" try: from tools.skills_tool import _find_all_skills, _sort_skills - skills = _sort_skills( - _find_all_skills( - skip_disabled=False, include_editorial=True - ) - ) + skills = _sort_skills(_find_all_skills(skip_disabled=False)) except Exception: logger.exception("GET /v1/skills failed") return _error_response("Failed to enumerate skills", 500, err_type="server_error") diff --git a/tests/gateway/test_api_server.py b/tests/gateway/test_api_server.py index f1dc90c74038..aa6a2f7e2145 100644 --- a/tests/gateway/test_api_server.py +++ b/tests/gateway/test_api_server.py @@ -1034,8 +1034,12 @@ async def test_skills_returns_list_envelope(self, adapter): {"name": "github", "description": "GitHub workflow skill", "category": "github"}, {"name": "ascii-art", "description": "ASCII art generation", "category": "creative"}, ] + # autospec=True enforces the real _find_all_skills signature. Without it the + # kwargs-blind MagicMock swallowed a stale keyword argument and this test stayed + # green while the live route raised TypeError and answered 500 on every request. with patch( "tools.skills_tool._find_all_skills", + autospec=True, return_value=list(fake_skills), ): app = _create_app(adapter)