docs: Modernize lift() pattern examples per Berni's feedback

Simplify pattern instance storage examples:
- Remove toSchema() wrapper - use simpler lift<Type, Return> syntax
- Remove [ID] field with Math.random() (anti-pattern for causal IDs)
- Remove unnecessary type casting (modern types handle it)
- Add note about proper ID generation in production

Addresses Berni's review comments on PR #1998:
- Line 505: toSchema not needed with current framework
- Line 515: [ID] + Math.random() is anti-pattern
- Line 536: Type casting unnecessary

Examples now show modern, simplified approach while maintaining
the core lesson (use lift() when storing pattern instances).

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>

Author: jkomoros

docs/common/HANDLERS.md

@@ -501,19 +501,17 @@ type Entry = {
 };
 
 // ✅ CORRECT: Wrap the push operation in a lift function
-const storePattern = lift(
-  toSchema<{
-    charm: any;
-    patternsList: Cell<Entry[]>;
-    isInitialized: Cell<boolean>;
-  }>(),
-  undefined,
+const storePattern = lift<{
+  charm: any;
+  patternsList: Cell<Entry[]>;
+  isInitialized: Cell<boolean>;
+}, undefined>(
   ({ charm, patternsList, isInitialized }) => {
     if (!isInitialized.get()) {
       // ✅ CORRECT: .push() happens inside lift, not handler
       patternsList.push({
-        [ID]: Math.random().toString(36).substring(2, 10),
-        local_id: Math.random().toString(36).substring(2, 10),
+        // Note: Using a stable ID here - in real code use proper ID generation
+        local_id: `pattern-${Date.now()}`,
         charm
       });
       isInitialized.set(true);
@@ -530,11 +528,11 @@ const createPattern = handler<
     const isInitialized = cell(false);
     const pattern = MyPattern({ title: "New" });
 
-    // ✅ CORRECT: Return the lift function call
+    // ✅ CORRECT: Return the lift function call (no casting needed with modern types)
     return storePattern({
       charm: pattern,
-      patternsList: patternsList as unknown as OpaqueRef<Entry[]>,
-      isInitialized: isInitialized as unknown as Cell<boolean>
+      patternsList,
+      isInitialized
     });
   }
 );

docs/common/PATTERNS.md

@@ -810,12 +810,15 @@ const createChat = handler<unknown, { chatsList: Cell<Entry[]> }>(
 );
 
 // ✅ CORRECT - Use lift() to wrap the push operation
-const storeChat = lift(
-  toSchema<{ charm: any; chatsList: Cell<Entry[]>; isInitialized: Cell<boolean> }>(),
-  undefined,
+const storeChat = lift<{
+  charm: any;
+  chatsList: Cell<Entry[]>;
+  isInitialized: Cell<boolean>;
+}, undefined>(
   ({ charm, chatsList, isInitialized }) => {
     if (!isInitialized.get()) {
-      chatsList.push({ [ID]: randomId, local_id: randomId, charm });
+      // Use stable ID - in production use proper ID generation
+      chatsList.push({ local_id: `chat-${Date.now()}`, charm });
       isInitialized.set(true);
     }
   }
@@ -826,7 +829,7 @@ const createChat = handler<unknown, { chatsList: Cell<Entry[]> }>(
     const isInitialized = cell(false);
     const chat = Chat({ title: "New Chat", messages: [] });
 
-    // ✅ Handler returns the lift call instead of pushing directly
+    // ✅ Handler returns the lift call (no casting needed with modern types)
     return storeChat({ charm: chat, chatsList, isInitialized });
   }
 );