Address Copilot review: dead code, validation, docs, pagination

- Remove dead mapNotification() from MoltbookProvider (no API endpoint)
- Fix malformed UUID examples in Handle.ts doc comments
- Fix all 9 social command README test paths (spaces → lowercase slashes)
- Fix Handles.expireStale() to paginate through all stale handles
- Add UUID format validation to HandleEntity.requestedBy
- Add orderBy to short ID resolution for deterministic collision handling
- Document in-memory rate limiting as ephemeral by design
- Document single-threaded safety for retryCount read-increment-write

Skipped by design: credential check-then-insert race (single-persona,
tiny window), apiKey format validation (unknown platform formats),
5xx retry logic (project policy: fail clearly, no fallbacks).

Author: Joel

src/debug/jtag/commands/social/comment/README.md

@@ -110,7 +110,7 @@ Test command logic in isolation using mock dependencies:
 
 ```bash
 # Run unit tests (no server required)
-npx tsx commands/Social Comment/test/unit/SocialCommentCommand.test.ts
+npx tsx commands/social/comment/test/unit/SocialCommentCommand.test.ts
 ```
 
 **What's tested:**
@@ -137,7 +137,7 @@ Test command with real client connections and system integration:
 npm start  # Wait 90+ seconds for deployment
 
 # Run integration tests
-npx tsx commands/Social Comment/test/integration/SocialCommentIntegration.test.ts
+npx tsx commands/social/comment/test/integration/SocialCommentIntegration.test.ts
 ```
 
 **What's tested:**

src/debug/jtag/commands/social/community/README.md

@@ -123,7 +123,7 @@ Test command logic in isolation using mock dependencies:
 
 ```bash
 # Run unit tests (no server required)
-npx tsx commands/Social Community/test/unit/SocialCommunityCommand.test.ts
+npx tsx commands/social/community/test/unit/SocialCommunityCommand.test.ts
 ```
 
 **What's tested:**
@@ -150,7 +150,7 @@ Test command with real client connections and system integration:
 npm start  # Wait 90+ seconds for deployment
 
 # Run integration tests
-npx tsx commands/Social Community/test/integration/SocialCommunityIntegration.test.ts
+npx tsx commands/social/community/test/integration/SocialCommunityIntegration.test.ts
 ```
 
 **What's tested:**

src/debug/jtag/commands/social/downvote/README.md

@@ -102,7 +102,7 @@ Test command logic in isolation using mock dependencies:
 
 ```bash
 # Run unit tests (no server required)
-npx tsx commands/Social Downvote/test/unit/SocialDownvoteCommand.test.ts
+npx tsx commands/social/downvote/test/unit/SocialDownvoteCommand.test.ts
 ```
 
 **What's tested:**
@@ -129,7 +129,7 @@ Test command with real client connections and system integration:
 npm start  # Wait 90+ seconds for deployment
 
 # Run integration tests
-npx tsx commands/Social Downvote/test/integration/SocialDownvoteIntegration.test.ts
+npx tsx commands/social/downvote/test/integration/SocialDownvoteIntegration.test.ts
 ```
 
 **What's tested:**

src/debug/jtag/commands/social/feed/README.md

@@ -111,7 +111,7 @@ Test command logic in isolation using mock dependencies:
 
 ```bash
 # Run unit tests (no server required)
-npx tsx commands/Social Feed/test/unit/SocialFeedCommand.test.ts
+npx tsx commands/social/feed/test/unit/SocialFeedCommand.test.ts
 ```
 
 **What's tested:**
@@ -138,7 +138,7 @@ Test command with real client connections and system integration:
 npm start  # Wait 90+ seconds for deployment
 
 # Run integration tests
-npx tsx commands/Social Feed/test/integration/SocialFeedIntegration.test.ts
+npx tsx commands/social/feed/test/integration/SocialFeedIntegration.test.ts
 ```
 
 **What's tested:**

src/debug/jtag/commands/social/notifications/README.md

@@ -110,7 +110,7 @@ Test command logic in isolation using mock dependencies:
 
 ```bash
 # Run unit tests (no server required)
-npx tsx commands/Social Notifications/test/unit/SocialNotificationsCommand.test.ts
+npx tsx commands/social/notifications/test/unit/SocialNotificationsCommand.test.ts
 ```
 
 **What's tested:**
@@ -137,7 +137,7 @@ Test command with real client connections and system integration:
 npm start  # Wait 90+ seconds for deployment
 
 # Run integration tests
-npx tsx commands/Social Notifications/test/integration/SocialNotificationsIntegration.test.ts
+npx tsx commands/social/notifications/test/integration/SocialNotificationsIntegration.test.ts
 ```
 
 **What's tested:**

src/debug/jtag/commands/social/post/README.md

@@ -105,7 +105,7 @@ Test command logic in isolation using mock dependencies:
 
 ```bash
 # Run unit tests (no server required)
-npx tsx commands/Social Post/test/unit/SocialPostCommand.test.ts
+npx tsx commands/social/post/test/unit/SocialPostCommand.test.ts
 ```
 
 **What's tested:**
@@ -132,7 +132,7 @@ Test command with real client connections and system integration:
 npm start  # Wait 90+ seconds for deployment
 
 # Run integration tests
-npx tsx commands/Social Post/test/integration/SocialPostIntegration.test.ts
+npx tsx commands/social/post/test/integration/SocialPostIntegration.test.ts
 ```
 
 **What's tested:**

src/debug/jtag/commands/social/profile/README.md

@@ -116,7 +116,7 @@ Test command logic in isolation using mock dependencies:
 
 ```bash
 # Run unit tests (no server required)
-npx tsx commands/Social Profile/test/unit/SocialProfileCommand.test.ts
+npx tsx commands/social/profile/test/unit/SocialProfileCommand.test.ts
 ```
 
 **What's tested:**
@@ -143,7 +143,7 @@ Test command with real client connections and system integration:
 npm start  # Wait 90+ seconds for deployment
 
 # Run integration tests
-npx tsx commands/Social Profile/test/integration/SocialProfileIntegration.test.ts
+npx tsx commands/social/profile/test/integration/SocialProfileIntegration.test.ts
 ```
 
 **What's tested:**

src/debug/jtag/commands/social/signup/README.md

@@ -108,7 +108,7 @@ Test command logic in isolation using mock dependencies:
 
 ```bash
 # Run unit tests (no server required)
-npx tsx commands/Social Signup/test/unit/SocialSignupCommand.test.ts
+npx tsx commands/social/signup/test/unit/SocialSignupCommand.test.ts
 ```
 
 **What's tested:**
@@ -135,7 +135,7 @@ Test command with real client connections and system integration:
 npm start  # Wait 90+ seconds for deployment
 
 # Run integration tests
-npx tsx commands/Social Signup/test/integration/SocialSignupIntegration.test.ts
+npx tsx commands/social/signup/test/integration/SocialSignupIntegration.test.ts
 ```
 
 **What's tested:**

src/debug/jtag/commands/social/trending/README.md

@@ -116,7 +116,7 @@ Test command logic in isolation using mock dependencies:
 
 ```bash
 # Run unit tests (no server required)
-npx tsx commands/Social Trending/test/unit/SocialTrendingCommand.test.ts
+npx tsx commands/social/trending/test/unit/SocialTrendingCommand.test.ts
 ```
 
 **What's tested:**
@@ -143,7 +143,7 @@ Test command with real client connections and system integration:
 npm start  # Wait 90+ seconds for deployment
 
 # Run integration tests
-npx tsx commands/Social Trending/test/integration/SocialTrendingIntegration.test.ts
+npx tsx commands/social/trending/test/integration/SocialTrendingIntegration.test.ts
 ```
 
 **What's tested:**

src/debug/jtag/system/core/shared/Handles.ts

@@ -137,9 +137,11 @@ export const Handles = {
 
     // Query all handles and filter by suffix
     // The $regex operator matches UUIDs ending with the short ID
+    // Order by createdAt desc so the most recent match wins on collision
     const result = await DataList.execute<HandleEntity>({
       collection: COLLECTIONS.HANDLES,
       filter: { id: { $regex: `${shortId}$` } },
+      orderBy: [{ field: 'createdAt', direction: 'desc' }],
       limit: 2, // Get 2 to detect ambiguity
     });
 
@@ -267,37 +269,44 @@ export const Handles = {
 
   /**
    * Expire all handles past their TTL. Call periodically (e.g., every 60s).
-   * Returns the number of handles expired.
+   * Processes in batches of 200 until all stale handles are expired.
+   * Returns the total number of handles expired.
    */
   async expireStale(): Promise<number> {
     const now = new Date().toISOString();
+    let totalExpired = 0;
+    const BATCH_SIZE = 200;
 
-    // Find active handles with expiresAt in the past
-    const result = await DataList.execute<HandleEntity>({
-      collection: COLLECTIONS.HANDLES,
-      filter: {
-        status: { $in: ['pending', 'processing'] },
-        expiresAt: { $lte: now },
-      },
-      limit: 200,
-    });
+    // Loop in batches until no more stale handles remain
+    while (true) {
+      const result = await DataList.execute<HandleEntity>({
+        collection: COLLECTIONS.HANDLES,
+        filter: {
+          status: { $in: ['pending', 'processing'] },
+          expiresAt: { $lte: now },
+        },
+        limit: BATCH_SIZE,
+      });
 
-    if (!result.success || !result.items?.length) return 0;
+      if (!result.success || !result.items?.length) break;
 
-    let expired = 0;
-    for (const entity of result.items) {
-      try {
-        await this._updateStatus((entity as HandleEntity).id, 'expired');
-        expired++;
-      } catch (err) {
-        log.warn(`Failed to expire handle ${(entity as HandleEntity).id}: ${err}`);
+      for (const entity of result.items) {
+        try {
+          await this._updateStatus((entity as HandleEntity).id, 'expired');
+          totalExpired++;
+        } catch (err) {
+          log.warn(`Failed to expire handle ${(entity as HandleEntity).id}: ${err}`);
+        }
       }
+
+      // If we got fewer than BATCH_SIZE, we've processed all of them
+      if (result.items.length < BATCH_SIZE) break;
     }
 
-    if (expired > 0) {
-      log.info(`Expired ${expired} stale handles`);
+    if (totalExpired > 0) {
+      log.info(`Expired ${totalExpired} stale handles`);
     }
-    return expired;
+    return totalExpired;
   },
 
   /**
@@ -317,7 +326,8 @@ export const Handles = {
     if (extra?.error !== undefined) updates.error = extra.error;
     if (extra?.params !== undefined) updates.params = extra.params;
     if (status === 'failed') {
-      // Increment retry count on failure — read current, increment, write back
+      // Increment retry count on failure — read current, increment, write back.
+      // Safe: Node.js is single-threaded, so no concurrent failures for the same handle.
       const current = await DataRead.execute<HandleEntity>({
         collection: COLLECTIONS.HANDLES,
         id,