From c489452a2d26045e18ef1356127d3f001de79905 Mon Sep 17 00:00:00 2001 From: Bryan Van Deusen Date: Mon, 5 Oct 2026 14:28:50 -0400 Subject: [PATCH] test(moments): an install removal still drops its tool while the skill loader stays listed (milestone 458 step 5, #4923) Co-Authored-By: Claude Opus 5.5 --- tests/test_moment_delivery.py | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/tests/test_moment_delivery.py b/tests/test_moment_delivery.py index 5b663882..9242c48a 100644 --- a/tests/test_moment_delivery.py +++ b/tests/test_moment_delivery.py @@ -209,7 +209,8 @@ async def test_an_installs_own_mapping_and_removal_both_count(): own = SimpleNamespace(tool="mcp__deploy__ship", match="", moment="work.deliver", effect="add") assert "ship" in await _reachable({"work.deliver"}, [own]) gone = SimpleNamespace(tool="AskUserQuestion", match="", moment="reply.ask", effect="remove") - assert await _reachable({"reply.ask"}, [gone]) == [] + assert "askuserquestion" in await _reachable({"reply.ask"}) + assert "askuserquestion" not in await _reachable({"reply.ask"}, [gone]) # ── the plugin routes ───────────────────────────────────────────────────