Fix more documentation issues from PR review

- error-handling.md: Fix input fields (errorContext, recoverable instead of errorType, stack, context)
- error-handling.md: Fix output fields (suppressOutput, errorHandling, retryCount, userNotification)
- error-handling.md: Update examples to use correct field names
- user-prompt-submitted.md: Remove non-existent reject/rejectReason, add suppressOutput
- compatibility.md: Fix token usage event (assistant.usage not assistant.turn_end)
- compatibility.md: Fix usage fields (inputTokens/outputTokens not usage object)
- session-lifecycle.md: Fix Log Session Summary example to track startTime

Author: patniko

docs/compatibility.md

@@ -146,8 +146,11 @@ const session = await client.createSession({
 Instead of `/usage`, subscribe to usage events:
 
 ```typescript
-session.on("assistant.turn_end", (event) => {
-  console.log("Tokens used:", event.data.usage);
+session.on("assistant.usage", (event) => {
+  console.log("Tokens used:", {
+    input: event.data.inputTokens,
+    output: event.data.outputTokens,
+  });
 });
 ```
 

docs/hooks/error-handling.md

@@ -63,19 +63,19 @@ public delegate Task<ErrorOccurredHookOutput?> ErrorOccurredHandler(
 | `timestamp` | number | Unix timestamp when the error occurred |
 | `cwd` | string | Current working directory |
 | `error` | string | Error message |
-| `errorType` | string | Type/category of error |
-| `stack` | string | Stack trace (if available) |
-| `context` | object | Additional error context |
+| `errorContext` | string | Where the error occurred: `"model_call"`, `"tool_execution"`, `"system"`, or `"user_input"` |
+| `recoverable` | boolean | Whether the error can potentially be recovered from |
 
 ## Output
 
 Return `null` or `undefined` to use default error handling. Otherwise, return an object with:
 
 | Field | Type | Description |
 |-------|------|-------------|
-| `additionalContext` | string | Context to add to help recover |
-| `suppressError` | boolean | If true, don't show error to user |
-| `modifiedMessage` | string | Custom error message to display |
+| `suppressOutput` | boolean | If true, don't show error output to user |
+| `errorHandling` | string | How to handle: `"retry"`, `"skip"`, or `"abort"` |
+| `retryCount` | number | Number of times to retry (if errorHandling is `"retry"`) |
+| `userNotification` | string | Custom message to show the user |
 
 ## Examples
 
@@ -89,10 +89,8 @@ const session = await client.createSession({
   hooks: {
     onErrorOccurred: async (input, invocation) => {
       console.error(`[${invocation.sessionId}] Error: ${input.error}`);
-      console.error(`  Type: ${input.errorType}`);
-      if (input.stack) {
-        console.error(`  Stack: ${input.stack}`);
-      }
+      console.error(`  Context: ${input.errorContext}`);
+      console.error(`  Recoverable: ${input.recoverable}`);
       return null;
     },
   },
@@ -107,9 +105,8 @@ const session = await client.createSession({
 ```python
 async def on_error_occurred(input_data, invocation):
     print(f"[{invocation['session_id']}] Error: {input_data['error']}")
-    print(f"  Type: {input_data['errorType']}")
-    if input_data.get("stack"):
-        print(f"  Stack: {input_data['stack']}")
+    print(f"  Context: {input_data['error_context']}")
+    print(f"  Recoverable: {input_data['recoverable']}")
     return None
 
 session = await client.create_session({
@@ -127,10 +124,8 @@ session, _ := client.CreateSession(ctx, copilot.SessionConfig{
     Hooks: &copilot.SessionHooks{
         OnErrorOccurred: func(input copilot.ErrorOccurredHookInput, inv copilot.HookInvocation) (*copilot.ErrorOccurredHookOutput, error) {
             fmt.Printf("[%s] Error: %s\n", inv.SessionID, input.Error)
-            fmt.Printf("  Type: %s\n", input.ErrorType)
-            if input.Stack != "" {
-                fmt.Printf("  Stack: %s\n", input.Stack)
-            }
+            fmt.Printf("  Context: %s\n", input.ErrorContext)
+            fmt.Printf("  Recoverable: %v\n", input.Recoverable)
             return nil, nil
         },
     },

docs/hooks/session-lifecycle.md

@@ -384,12 +384,16 @@ const session = await client.createSession({
 #### Log Session Summary
 
 ```typescript
-const sessionData: Record<string, { prompts: number; tools: number }> = {};
+const sessionData: Record<string, { prompts: number; tools: number; startTime: number }> = {};
 
 const session = await client.createSession({
   hooks: {
-    onSessionStart: async (_, invocation) => {
-      sessionData[invocation.sessionId] = { prompts: 0, tools: 0 };
+    onSessionStart: async (input, invocation) => {
+      sessionData[invocation.sessionId] = { 
+        prompts: 0, 
+        tools: 0, 
+        startTime: input.timestamp 
+      };
       return null;
     },
     onUserPromptSubmitted: async (_, invocation) => {

docs/hooks/user-prompt-submitted.md

@@ -72,8 +72,7 @@ Return `null` or `undefined` to use the prompt unchanged. Otherwise, return an o
 |-------|------|-------------|
 | `modifiedPrompt` | string | Modified prompt to use instead of original |
 | `additionalContext` | string | Extra context added to the conversation |
-| `reject` | boolean | If true, reject the prompt entirely |
-| `rejectReason` | string | Explanation shown when prompt is rejected |
+| `suppressOutput` | boolean | If true, suppress the assistant's response output |
 
 ## Examples