feat: Changes suggested in feedback

Author: KShivendu

supertokens_python/recipe/session/interfaces.py

@@ -602,12 +602,9 @@ class SessionClaimValidator(ABC):
     def __init__(
         self,
         id_: str,
-        claim: Optional[
-            SessionClaim[_T]  # pyright: ignore[reportInvalidTypeVarUse]
-        ] = None,
     ) -> None:
         self.id = id_
-        self.claim = claim
+        self.claim: Optional[SessionClaim[Any]] = None
 
     @abstractmethod
     async def validate(

supertokens_python/recipe/session/recipe.py

@@ -66,9 +66,6 @@ class SessionRecipe(RecipeModule):
     recipe_id = "session"
     __instance = None
 
-    claims_added_by_other_recipes: List[SessionClaim[Any]] = []
-    claim_validators_added_by_other_recipes: List[SessionClaimValidator] = []
-
     def __init__(
         self,
         recipe_id: str,
@@ -156,6 +153,9 @@ def __init__(
             else self.config.override.apis(api_implementation)
         )
 
+        self.claims_added_by_other_recipes: List[SessionClaim[Any]] = []
+        self.claim_validators_added_by_other_recipes: List[SessionClaimValidator] = []
+
     def is_error_from_this_recipe_based_on_instance(self, err: Exception) -> bool:
         return isinstance(err, SuperTokensError) and (
             isinstance(err, SuperTokensSessionError)

tests/sessions/claims/test_assert_claims.py

@@ -49,7 +49,8 @@ async def test_should_call_validate_with_the_same_payload_object():
 
     class DummyClaimValidator(SessionClaimValidator):
         def __init__(self, claim: SessionClaim[Any]):
-            super().__init__("claim_validator_id", claim)
+            super().__init__("claim_validator_id")
+            self.claim = claim
             self.validate_calls: Dict[str, int] = {}
 
         async def validate(

tests/sessions/claims/test_verify_session.py

@@ -87,7 +87,8 @@ async def new_get_global_claim_validators(
 
 class AlwaysValidValidator(SessionClaimValidator):
     def __init__(self):
-        super().__init__("always-valid-validator", TrueClaim)
+        super().__init__("always-valid-validator")
+        self.claim = TrueClaim
 
     async def validate(
         self, payload: JSONObject, user_context: Union[Dict[str, Any], None] = None
@@ -100,7 +101,8 @@ def should_refetch(self, payload: JSONObject, user_context: Dict[str, Any]):
 
 class AlwaysInvalidValidator(SessionClaimValidator):
     def __init__(self, reason: Optional[Dict[str, Any]]):
-        super().__init__("always-invalid-validator", TrueClaim)
+        super().__init__("always-invalid-validator")
+        self.claim = TrueClaim
         self.reason = reason
 
     async def validate(
@@ -169,7 +171,8 @@ async def refetched_claim2(  # type: ignore
 
     class CustomValidator(SessionClaimValidator):
         def __init__(self, is_valid: bool):
-            super().__init__("test_id", TrueClaim)
+            super().__init__("test_id")
+            self.claim = TrueClaim
             self.is_valid = is_valid
 
         async def validate(