chore(elements): Remove unused contexts

clerk · Oct 4, 2024 · 300168c · 300168c
1 parent 4641be0
commit 300168c
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 24 deletions.
diff --git a/packages/elements/src/react/sign-in/choose-session/choose-session.tsx b/packages/elements/src/react/sign-in/choose-session/choose-session.tsx
@@ -1,7 +1,6 @@
 import { Slot } from '@radix-ui/react-slot';
 import * as React from 'react';
 
-import { createContextForDomValidation } from '~/react/utils/create-context-for-dom-validation';
 import { isValidComponentType } from '~/react/utils/is-valid-component-type';
 
 import {
@@ -26,21 +25,13 @@ export type SignInSessionListItemProps = Omit<React.HTMLAttributes<HTMLLIElement
   children: ({ session }: { session: SignInActiveSessionListItem }) => React.ReactNode;
 };
 
-// ---------------------------------- CONTEXT -----------------------------------
-
-export const SignInChooseSessionCtx = createContextForDomValidation('SignInChooseSessionCtx');
-
 // --------------------------------- COMPONENTS ---------------------------------
 
 export function SignInChooseSession({ asChild, children, ...props }: SignInChooseSessionProps) {
   const activeState = useSignInChooseSessionIsActive();
   const Comp = asChild ? Slot : 'div';
 
-  return activeState ? (
-    <SignInChooseSessionCtx.Provider>
-      <Comp {...props}>{children}</Comp>
-    </SignInChooseSessionCtx.Provider>
-  ) : null;
+  return activeState ? <Comp {...props}>{children}</Comp> : null;
 }
 
 export function SignInSessionList({ asChild, children, includeCurrentSession, ...props }: SignInSessionListProps) {

diff --git a/packages/elements/src/react/sign-in/choose-strategy.tsx b/packages/elements/src/react/sign-in/choose-strategy.tsx
@@ -7,7 +7,6 @@ import { SignInCurrentStrategy, SignInFactors } from '~/internals/machines/sign-
 
 import { useActiveTags } from '../hooks';
 import { ActiveTagsMode } from '../hooks/use-active-tags.hook';
-import { createContextForDomValidation } from '../utils/create-context-for-dom-validation';
 import { SignInRouterCtx, SignInStrategyContext } from './context';
 
 // --------------------------------- HELPERS ---------------------------------
@@ -41,18 +40,12 @@ export type SignInForgotPasswordProps = React.HTMLAttributes<HTMLDivElement> & {
   asChild?: boolean;
 };
 
-export const SignInChooseStrategyCtx = createContextForDomValidation('SignInChooseStrategyCtx');
-
 export function SignInChooseStrategy({ asChild, children, ...props }: SignInChooseStrategyProps) {
   const routerRef = SignInRouterCtx.useActorRef();
   const activeState = useActiveTags(routerRef, 'step:choose-strategy');
   const Comp = asChild ? Slot : 'div';
 
-  return activeState ? (
-    <SignInChooseStrategyCtx.Provider>
-      <Comp {...props}>{children}</Comp>
-    </SignInChooseStrategyCtx.Provider>
-  ) : null;
+  return activeState ? <Comp {...props}>{children}</Comp> : null;
 }
 
 export function SignInForgotPassword({ asChild, children, ...props }: SignInForgotPasswordProps) {
@@ -64,11 +57,7 @@ export function SignInForgotPassword({ asChild, children, ...props }: SignInForg
   );
   const Comp = asChild ? Slot : 'div';
 
-  return activeState ? (
-    <SignInChooseStrategyCtx.Provider>
-      <Comp {...props}>{children}</Comp>
-    </SignInChooseStrategyCtx.Provider>
-  ) : null;
+  return activeState ? <Comp {...props}>{children}</Comp> : null;
 }
 
 const SUPPORTED_STRATEGY_NAME = 'SignInSupportedStrategy';

diff --git a/packages/elements/src/react/utils/create-context-for-dom-validation.ts b/packages/elements/src/react/utils/create-context-for-dom-validation.ts
@@ -19,7 +19,7 @@ export function createContextForDomValidation(displayName: string) {
 
   Provider.displayName = displayName;
 
-  function useContext(allowMissingContext: boolean = false) {
+  function useContext(allowMissingContext = false) {
     const context = React.useContext(ReactContext);
 
     if (!allowMissingContext && !context) {