fix: upgrade Comms SDK to 2.0.1

package.json

@@ -52,7 +52,7 @@
     ],
     "dependencies": {
         "@doist/cli-core": "0.26.2",
-        "@doist/comms-sdk": "0.11.0",
+        "@doist/comms-sdk": "2.0.1",
         "@pnpm/tabtab": "0.5.4",
         "chalk": "5.6.2",
         "commander": "14.0.3",

src/commands/channel/members.test.ts

@@ -108,6 +108,27 @@ describe('tdc channel members list (default)', () => {
         ])
     })
 
+    it('omits email for restricted members', async () => {
+        refsMocks.resolveChannelRef.mockResolvedValue(createChannel([6]))
+        apiMocks.getCommsClient.mockResolvedValue({
+            workspaceUsers: {
+                getUserById: vi.fn().mockResolvedValue({
+                    id: 6,
+                    fullName: 'Restricted User',
+                    email: 'restricted@example.com',
+                    restricted: true,
+                }),
+            },
+        })
+        const consoleSpy = captureConsole('log')
+        const program = createProgram()
+
+        await program.parseAsync(['node', 'tdc', 'channel', 'members', 'General', '--json'])
+
+        const payload = JSON.parse(consoleSpy.mock.calls[0][0] as string)
+        expect(payload.members[0]).toEqual({ id: 6, name: 'Restricted User', email: null })
+    })
+
     it('falls back to user:<id> for unknown members', async () => {
         refsMocks.resolveChannelRef.mockResolvedValue(createChannel([99]))
         const consoleSpy = captureConsole('log')

src/commands/channel/members.ts

@@ -1,3 +1,4 @@
+import { isRestrictedWorkspaceUser } from '@doist/comms-sdk'
 import { getCurrentWorkspaceId, getWorkspaceGroups } from '../../lib/api.js'
 import type { ViewOptions } from '../../lib/options.js'
 import { colors, formatJson, formatNdjson, pluralize } from '../../lib/output.js'
@@ -21,7 +22,8 @@ export async function listChannelMembers(
 
     const members = userIds.map((id) => {
         const user = userMap.get(id)
-        return { id, name: user?.fullName ?? null, email: user?.email ?? null }
+        const email = user && !isRestrictedWorkspaceUser(user) ? (user.email ?? null) : null
+        return { id, name: user?.fullName ?? null, email }
     })
 
     const slimPayload = {

src/commands/channel/membership-helpers.ts

@@ -1,4 +1,4 @@
-import type { Channel, Group, WorkspaceUser } from '@doist/comms-sdk'
+import type { Channel, Group, VisibleWorkspaceUser } from '@doist/comms-sdk'
 import {
     addUsersToChannel,
     getCommsClient,
@@ -20,7 +20,7 @@ export function channelUserIds(channel: Channel): number[] {
 export async function fetchUsersByIds(
     workspaceId: number,
     userIds: number[],
-): Promise<Map<number, WorkspaceUser>> {
+): Promise<Map<number, VisibleWorkspaceUser>> {
     if (userIds.length === 0) return new Map()
     const client = await getCommsClient()
     // Per-member fetch keeps latency tied to channel size, not workspace size
@@ -35,7 +35,7 @@ export async function fetchUsersByIds(
             }
         }),
     )
-    const map = new Map<number, WorkspaceUser>()
+    const map = new Map<number, VisibleWorkspaceUser>()
     for (const entry of entries) {
         if (entry) map.set(entry[0], entry[1])
     }

src/commands/groups/groups.test.ts

@@ -232,6 +232,22 @@ describe('tdc groups view', () => {
         expect(output).not.toHaveProperty('version')
     })
 
+    it('omits email for restricted members', async () => {
+        mockGetUserById.mockResolvedValueOnce({
+            id: 1,
+            fullName: 'Restricted User',
+            email: 'restricted@example.com',
+            restricted: true,
+        })
+        const program = createProgram()
+        const consoleSpy = captureConsole('log')
+
+        await program.parseAsync(['node', 'tdc', 'groups', 'view', 'Frontend', '--json'])
+
+        const output = JSON.parse(consoleSpy.mock.calls[0][0])
+        expect(output.members[0]).toEqual({ id: 1, name: 'Restricted User', email: null })
+    })
+
     it('renders user:N for members whose lookup fails', async () => {
         mockGetUserById.mockImplementationOnce(async () => {
             throw new Error('User not found')

src/commands/groups/view.ts

@@ -1,3 +1,4 @@
+import { isRestrictedWorkspaceUser } from '@doist/comms-sdk'
 import { getCommsClient, getCurrentWorkspaceId } from '../../lib/api.js'
 import type { ViewOptions } from '../../lib/options.js'
 import { colors, formatJson, formatNdjson, pluralize } from '../../lib/output.js'
@@ -18,7 +19,8 @@ export async function viewGroup(ref: string, options: GroupViewOptions): Promise
                     workspaceId,
                     userId: id,
                 })
-                return { id, name: user.fullName, email: user.email ?? null }
+                const email = isRestrictedWorkspaceUser(user) ? null : (user.email ?? null)
+                return { id, name: user.fullName, email }
             } catch {
                 return { id, name: null as string | null, email: null as string | null }
             }