test: cover invite session-drive selection

Extract inviteSessionDrive so a child invite cannot land on an unread
parent. Unit tests pin drive-level vs child-resource behavior.

Co-authored-by: joepmeindertsma <joepmeindertsma@gmail.com>

Author: cursoragent

TESTING_COVERAGE.md

@@ -139,6 +139,9 @@ Both matter because `iroh_transport` holds the router and node identity in
 |---|---|---|
 | Extract agent mentions from TipTap JSON / chat text | glue | `browser/lib/src/mentions.test.ts` |
 | Populate includes NotificationItem / watches ontology | unit | `populate::notifications_ontology_is_populated` |
+| Child invite keeps session on private drive | unit | `inviteSessionDrive.test.ts` |
+| Drive-level invite activates the granted host | unit | same |
+| Chatroom invite sidebar is `{name}'s Drive` | flow | `browser/e2e/tests/e2e.spec.ts` (`chatroom` @smoke) |
 | Sidebar Notifications + empty inbox | flow | `browser/e2e/tests/notifications.spec.ts` |
 | `/app/dev-drive` workspace ≠ personal inbox drive | flow | `notifications.spec.ts` (`dev-drive workspace is not the personal inbox drive`) |
 | Seeded NotificationItem appears with unread badge | flow | `notifications.spec.ts` |

browser/data-browser/src/helpers/inviteSessionDrive.test.ts

@@ -0,0 +1,40 @@
+import { describe, expect, it } from 'vitest';
+import { inviteSessionDrive } from './inviteSessionDrive';
+
+const PRIVATE = 'did:ad:private';
+const HOST = 'did:ad:host';
+
+describe('inviteSessionDrive', () => {
+  it('keeps a child invite on the private drive even when ancestry named a host', () => {
+    expect(
+      inviteSessionDrive({
+        privateDrive: PRIVATE,
+        hostDrive: HOST,
+        destinationIsDrive: false,
+      }),
+    ).toBe(PRIVATE);
+  });
+
+  it('lands a drive-level invite on the host', () => {
+    expect(
+      inviteSessionDrive({
+        privateDrive: PRIVATE,
+        hostDrive: HOST,
+        destinationIsDrive: true,
+      }),
+    ).toBe(HOST);
+  });
+
+  it('falls back to the private drive if a drive invite has no host bookmark', () => {
+    expect(
+      inviteSessionDrive({
+        privateDrive: PRIVATE,
+        destinationIsDrive: true,
+      }),
+    ).toBe(PRIVATE);
+  });
+
+  it('is undefined when neither drive resolved', () => {
+    expect(inviteSessionDrive({ destinationIsDrive: false })).toBeUndefined();
+  });
+});

browser/data-browser/src/helpers/inviteSessionDrive.ts

@@ -0,0 +1,22 @@
+/** Drives persistAgentAfterInvite hands to activateDrive. */
+export type InviteSessionDrives = {
+  privateDrive?: string;
+  hostDrive?: string;
+  destinationIsDrive?: boolean;
+};
+
+/**
+ * Session drive after accepting an invite.
+ *
+ * Drive-level invites land on the granted host. Child invites (chatroom,
+ * document) grant the destination, not the parent — stay on the invitee's
+ * private drive rather than an unreadable host (`Unauthorized` / truncated
+ * DID in the sidebar).
+ */
+export function inviteSessionDrive(
+  drives: InviteSessionDrives,
+): string | undefined {
+  return drives.destinationIsDrive
+    ? (drives.hostDrive ?? drives.privateDrive)
+    : drives.privateDrive;
+}

browser/data-browser/src/views/InvitePage.tsx

@@ -29,6 +29,7 @@ import { Logo } from '../components/Logo';
 import { useId, useState, type JSX } from 'react';
 import { useNavigate } from '@tanstack/react-router';
 import { getResourcesDrive } from '@helpers/getResourcesDrive';
+import { inviteSessionDrive } from '@helpers/inviteSessionDrive';
 import { fetchPrivateDriveSubject } from '@helpers/privateDrive';
 import { saveAgentToIDB } from '@helpers/agentStorage';
 import { Dialog, useDialog } from '@components/Dialog';
@@ -236,9 +237,7 @@ function InvitePage({ resource }: ResourcePageProps): JSX.Element {
     hostDrive?: string;
     destinationIsDrive?: boolean;
   }): boolean => {
-    const target = drives.destinationIsDrive
-      ? (drives.hostDrive ?? drives.privateDrive)
-      : drives.privateDrive;
+    const target = inviteSessionDrive(drives);
 
     if (!target) {
       return false;