From 2993a14ac4c05a0f63b18512f6ab2113b30028e7 Mon Sep 17 00:00:00 2001 From: Nick Nisi Date: Wed, 24 Jun 2026 15:19:58 -0500 Subject: [PATCH] fix: update User test mocks for @workos-inc/node 10.4.0 The @workos-inc/node 10.4.0 bump (#107) made `User.name` a required field, but the User mocks in AuthKitProvider.spec.tsx and useAccessToken.spec.tsx were not updated. The publish job runs prepublishOnly -> typecheck (root tsconfig, which includes spec files), so it failed and 0.10.0 was tagged but never published to npm. PR CI only runs build via tsconfig.build.json (which excludes specs), so it never caught this. Add the missing name field to both mocks to restore a publishable build. Follow-up: add a `pnpm run typecheck` step to ci.yml so spec-only type errors are caught in PR CI instead of at publish time (parity with authkit-nextjs). --- src/client/AuthKitProvider.spec.tsx | 1 + src/client/useAccessToken.spec.tsx | 1 + 2 files changed, 2 insertions(+) diff --git a/src/client/AuthKitProvider.spec.tsx b/src/client/AuthKitProvider.spec.tsx index 317fe8a..9dc7dc3 100644 --- a/src/client/AuthKitProvider.spec.tsx +++ b/src/client/AuthKitProvider.spec.tsx @@ -18,6 +18,7 @@ describe('AuthKitProvider', () => { const mockUser: User = { id: 'user_123', email: 'test@example.com', + name: 'Test User', firstName: 'Test', lastName: 'User', emailVerified: true, diff --git a/src/client/useAccessToken.spec.tsx b/src/client/useAccessToken.spec.tsx index 95f3558..4167d4b 100644 --- a/src/client/useAccessToken.spec.tsx +++ b/src/client/useAccessToken.spec.tsx @@ -23,6 +23,7 @@ describe('useAccessToken', () => { const mockUser: User = { id: 'user_123', email: 'test@example.com', + name: 'Test User', firstName: 'Test', lastName: 'User', emailVerified: true,