Address PR review comments for create_session refactor

- Add validation for on_permission_request in create_session to fail fast
  when handler is missing/invalid
- Fix lambda signatures to accept two args (request, invocation) in test
  scenario and docs
- Fix permissionDecision key to use camelCase in pre_tool_use hook

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: brettcannon

docs/guides/skills.md

@@ -49,7 +49,7 @@ async def main():
     await client.start()
 
     session = await client.create_session(
-        lambda req: {"kind": "approved"},
+        lambda req, inv: {"kind": "approved"},
         "gpt-4.1",
         skill_directories=[
             "./skills/code-review",

python/copilot/client.py

@@ -493,6 +493,12 @@ async def create_session(
             ...     streaming=True,
             ... )
         """
+        if not on_permission_request or not callable(on_permission_request):
+            raise ValueError(
+                "A valid on_permission_request handler is required. "
+                "Use PermissionHandler.approve_all or provide a custom handler."
+            )
+
         if not self._client:
             if self.options["auto_start"]:
                 await self.start()

test/scenarios/tools/skills/python/main.py

@@ -15,11 +15,11 @@ async def main():
         skills_dir = str(Path(__file__).resolve().parent.parent / "sample-skills")
 
         session = await client.create_session(
-            lambda _: {"kind": "approved"},
+            lambda _, __: {"kind": "approved"},
             "claude-haiku-4.5",
             skill_directories=[skills_dir],
             hooks={
-                "on_pre_tool_use": lambda _: {"permission_decision": "allow"},
+                "on_pre_tool_use": lambda _, __: {"permissionDecision": "allow"},
             },
         )