fix lint

Author: friggeri

python/e2e/test_skills.py

@@ -7,7 +7,7 @@
 import tempfile
 import pytest
 
-from .testharness import E2ETestContext, get_final_assistant_message
+from .testharness import E2ETestContext
 
 pytestmark = pytest.mark.asyncio(loop_scope="module")
 
@@ -63,7 +63,8 @@ async def test_not_apply_skill_when_disabled_via_disabled_skills(
     ):
         """Test that disabledSkills prevents skill from being applied"""
         session = await ctx.client.create_session(
-            {"skill_directories": [skills_dir], "disabled_skills": ["test-skill"]}
+            {"skill_directories": [skills_dir],
+                "disabled_skills": ["test-skill"]}
         )
 
         assert session.session_id is not None
@@ -89,9 +90,7 @@ async def test_apply_skill_on_session_resume_with_skill_directories(
         assert SKILL_MARKER not in message1.data.content
 
         # Resume with skillDirectories - skill should now be active
-        session2 = await ctx.client.resume_session(
-            session_id, {"skill_directories": [skills_dir]}
-        )
+        session2 = await ctx.client.resume_session(session_id, {"skill_directories": [skills_dir]})
 
         assert session2.session_id == session_id
 

test/snapshots/session/should_abort_a_session.yaml

@@ -6,6 +6,8 @@ conversations:
         content: ${system}
       - role: user
         content: run the shell command 'sleep 100' (note this works on both bash and PowerShell)
+      - role: assistant
+        content: I'll run the sleep command for 100 seconds.
       - role: assistant
         tool_calls:
           - id: toolcall_0
@@ -19,13 +21,14 @@ conversations:
             type: function
             function:
               name: ${shell}
-              arguments: '{"description":"Run sleep 100 command","command":"sleep 100","mode":"sync","initial_wait":105}'
+              arguments: '{"command":"sleep 100","description":"Run sleep 100 command","mode":"sync","initial_wait":105}'
   - messages:
       - role: system
         content: ${system}
       - role: user
         content: run the shell command 'sleep 100' (note this works on both bash and PowerShell)
       - role: assistant
+        content: I'll run the sleep command for 100 seconds.
         tool_calls:
           - id: toolcall_0
             type: function
@@ -36,7 +39,7 @@ conversations:
             type: function
             function:
               name: ${shell}
-              arguments: '{"description":"Run sleep 100 command","command":"sleep 100","mode":"sync","initial_wait":105}'
+              arguments: '{"command":"sleep 100","description":"Run sleep 100 command","mode":"sync","initial_wait":105}'
       - role: tool
         tool_call_id: toolcall_0
         content: Intent logged

test/snapshots/skills/apply_skill_on_session_resume_with_skill_directories.yaml

@@ -24,7 +24,7 @@ conversations:
         content: >-
           <skill-context name="test-skill">
 
-          Base directory for this skill: /tmp/copilot-skills-test-e_oppucn/test-skill
+          Base directory for this skill: /tmp/copilot-skills-test-brp0xbfl/test-skill
 
 
           ---

test/snapshots/skills/load_and_apply_skill_from_skill_directories.yaml

@@ -20,7 +20,7 @@ conversations:
         content: >-
           <skill-context name="test-skill">
 
-          Base directory for this skill: /tmp/copilot-skills-test-e_oppucn/test-skill
+          Base directory for this skill: /tmp/copilot-skills-test-brp0xbfl/test-skill
 
 
           ---
@@ -41,4 +41,4 @@ conversations:
 
           </skill-context>
       - role: assistant
-        content: Hello! PINEAPPLE_COCONUT_42 - the test skill is active and ready to go.
+        content: Hello! PINEAPPLE_COCONUT_42 - Hope you're having a great day!

test/snapshots/skills/should_apply_skill_on_session_resume_with_skilldirectories.yaml

@@ -24,7 +24,7 @@ conversations:
         content: >-
           <skill-context name="test-skill">
 
-          Base directory for this skill: /tmp/copilot-skills-test-eSUOPo/test-skill
+          Base directory for this skill: /tmp/copilot-skills-test-f1ef743d-a194-4aa4-9809-43f2ce8086cb/test-skill
 
 
           ---
@@ -45,4 +45,4 @@ conversations:
 
           </skill-context>
       - role: assistant
-        content: Hello again! PINEAPPLE_COCONUT_42 - I'm here and ready to assist you with your development tasks!
+        content: Hello again! PINEAPPLE_COCONUT_42 - I'm here and ready to help with your tasks!

test/snapshots/skills/should_load_and_apply_skill_from_skilldirectories.yaml

@@ -20,7 +20,7 @@ conversations:
         content: >-
           <skill-context name="test-skill">
 
-          Base directory for this skill: /tmp/copilot-skills-test-eSUOPo/test-skill
+          Base directory for this skill: /tmp/copilot-skills-test-21c50037-7c7d-406b-b336-3a384641e588/test-skill
 
 
           ---
@@ -41,4 +41,4 @@ conversations:
 
           </skill-context>
       - role: assistant
-        content: Hello! PINEAPPLE_COCONUT_42 - the test skill is active and working.
+        content: Hello! PINEAPPLE_COCONUT_42 - Hope you're having a great day!