feat(jni): handle async operation failure

Add nativeAsyncOperationFailed JNI callback to each interface client.
When Java reports a failed async operation, log a warning with error
details and fulfill the pending promise with a default value.

Add FullfillPromiseWithDefaultValue to MethodHelper. Look up the promise
by GUID, remove it from the map, and call FulfillWithDefaultValue on the
IPromiseHolder base.

Without this, a failed Java async operation leaves its TPromise
unfulfilled, deadlocking any game-thread code that calls TFuture::Get().

Author: alwa-siili

goldenmaster/Plugins/Counter/Source/CounterJni/Private/Generated/Detail/CounterMethodHelper.h

@@ -48,6 +48,41 @@ class FCounterMethodHelper
 		}
 	}
 
+	bool FulfillPromiseWithDefaultValue(const FGuid& InId)
+	{
+		UE_LOG(
+			LogCounterMethodHelper_JNI,
+			Verbose,
+			TEXT("Resolving promise %s with default value for %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		TUniquePtr<ICounterPromiseHolder> PromiseHolderPtr;
+		{
+			FScopeLock Lock(&ReplyPromisesMapCS);
+			if (auto* Found = ReplyPromisesMap.Find(InId))
+			{
+				PromiseHolderPtr = MoveTemp(*Found);
+				ReplyPromisesMap.Remove(InId);
+			}
+		}
+
+		if (PromiseHolderPtr)
+		{
+			PromiseHolderPtr->FulfillWithDefaultValue();
+			return true;
+		}
+
+		UE_LOG(
+			LogCounterMethodHelper_JNI,
+			Verbose,
+			TEXT("No promise found for id %s in %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		return false;
+	}
+
 	template <typename ResultType>
 	FGuid StorePromise(TPromise<ResultType>&& InPromise);
 

goldenmaster/Plugins/Counter/Source/CounterJni/Private/Generated/Jni/CounterCounterJniClient.cpp

@@ -1347,4 +1347,23 @@ JNI_METHOD void Java_counter_counterjniclient_CounterJniClient_nativeIsReady(JNI
 	}
 	localJniAccessor->notifyIsReady(value);
 }
+
+JNI_METHOD void Java_counter_counterjniclient_CounterJniClient_nativeAsyncOperationFailed(JNIEnv* Env, jclass Clazz, jstring callId, jstring errorMessage, jint errorCode)
+{
+	FString callIdString = FJavaHelper::FStringFromParam(Env, callId);
+	static const TCHAR* errorMsgId = TEXT("failed to convert call id string in nativeAsyncOperationFailed for counter/counterjniclient/CounterJniClient");
+	if (CounterDataJavaConverter::CheckJniErrorOccurred(errorMsgId))
+	{
+		return;
+	}
+	FString errorMsgString = FJavaHelper::FStringFromParam(Env, errorMessage);
+
+	UE_LOG(LogCounterCounterClient_JNI, Warning,
+		TEXT("Async operation failed for id %s: %s (code: %d)"),
+		*callIdString, *errorMsgString, static_cast<int32>(errorCode));
+
+	FGuid guid;
+	FGuid::Parse(callIdString, guid);
+	gUCounterCounterJniClientmethodHelper.FulfillPromiseWithDefaultValue(guid);
+}
 #endif

goldenmaster/Plugins/CustomTypes/Source/CustomTypesJni/Private/Generated/Detail/CustomTypesMethodHelper.h

@@ -48,6 +48,41 @@ class FCustomTypesMethodHelper
 		}
 	}
 
+	bool FulfillPromiseWithDefaultValue(const FGuid& InId)
+	{
+		UE_LOG(
+			LogCustomTypesMethodHelper_JNI,
+			Verbose,
+			TEXT("Resolving promise %s with default value for %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		TUniquePtr<ICustomTypesPromiseHolder> PromiseHolderPtr;
+		{
+			FScopeLock Lock(&ReplyPromisesMapCS);
+			if (auto* Found = ReplyPromisesMap.Find(InId))
+			{
+				PromiseHolderPtr = MoveTemp(*Found);
+				ReplyPromisesMap.Remove(InId);
+			}
+		}
+
+		if (PromiseHolderPtr)
+		{
+			PromiseHolderPtr->FulfillWithDefaultValue();
+			return true;
+		}
+
+		UE_LOG(
+			LogCustomTypesMethodHelper_JNI,
+			Verbose,
+			TEXT("No promise found for id %s in %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		return false;
+	}
+
 	template <typename ResultType>
 	FGuid StorePromise(TPromise<ResultType>&& InPromise);
 

goldenmaster/Plugins/ExternTypes/Source/ExternTypesJni/Private/Generated/Detail/ExternTypesMethodHelper.h

@@ -48,6 +48,41 @@ class FExternTypesMethodHelper
 		}
 	}
 
+	bool FulfillPromiseWithDefaultValue(const FGuid& InId)
+	{
+		UE_LOG(
+			LogExternTypesMethodHelper_JNI,
+			Verbose,
+			TEXT("Resolving promise %s with default value for %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		TUniquePtr<IExternTypesPromiseHolder> PromiseHolderPtr;
+		{
+			FScopeLock Lock(&ReplyPromisesMapCS);
+			if (auto* Found = ReplyPromisesMap.Find(InId))
+			{
+				PromiseHolderPtr = MoveTemp(*Found);
+				ReplyPromisesMap.Remove(InId);
+			}
+		}
+
+		if (PromiseHolderPtr)
+		{
+			PromiseHolderPtr->FulfillWithDefaultValue();
+			return true;
+		}
+
+		UE_LOG(
+			LogExternTypesMethodHelper_JNI,
+			Verbose,
+			TEXT("No promise found for id %s in %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		return false;
+	}
+
 	template <typename ResultType>
 	FGuid StorePromise(TPromise<ResultType>&& InPromise);
 

goldenmaster/Plugins/TbEnum/Source/TbEnumJni/Private/Generated/Detail/TbEnumMethodHelper.h

@@ -48,6 +48,41 @@ class FTbEnumMethodHelper
 		}
 	}
 
+	bool FulfillPromiseWithDefaultValue(const FGuid& InId)
+	{
+		UE_LOG(
+			LogTbEnumMethodHelper_JNI,
+			Verbose,
+			TEXT("Resolving promise %s with default value for %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		TUniquePtr<ITbEnumPromiseHolder> PromiseHolderPtr;
+		{
+			FScopeLock Lock(&ReplyPromisesMapCS);
+			if (auto* Found = ReplyPromisesMap.Find(InId))
+			{
+				PromiseHolderPtr = MoveTemp(*Found);
+				ReplyPromisesMap.Remove(InId);
+			}
+		}
+
+		if (PromiseHolderPtr)
+		{
+			PromiseHolderPtr->FulfillWithDefaultValue();
+			return true;
+		}
+
+		UE_LOG(
+			LogTbEnumMethodHelper_JNI,
+			Verbose,
+			TEXT("No promise found for id %s in %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		return false;
+	}
+
 	template <typename ResultType>
 	FGuid StorePromise(TPromise<ResultType>&& InPromise);
 

goldenmaster/Plugins/TbEnum/Source/TbEnumJni/Private/Generated/Jni/TbEnumEnumInterfaceJniClient.cpp

@@ -1456,4 +1456,23 @@ JNI_METHOD void Java_tbEnum_tbEnumjniclient_EnumInterfaceJniClient_nativeIsReady
 	}
 	localJniAccessor->notifyIsReady(value);
 }
+
+JNI_METHOD void Java_tbEnum_tbEnumjniclient_EnumInterfaceJniClient_nativeAsyncOperationFailed(JNIEnv* Env, jclass Clazz, jstring callId, jstring errorMessage, jint errorCode)
+{
+	FString callIdString = FJavaHelper::FStringFromParam(Env, callId);
+	static const TCHAR* errorMsgId = TEXT("failed to convert call id string in nativeAsyncOperationFailed for tbEnum/tbEnumjniclient/EnumInterfaceJniClient");
+	if (TbEnumDataJavaConverter::CheckJniErrorOccurred(errorMsgId))
+	{
+		return;
+	}
+	FString errorMsgString = FJavaHelper::FStringFromParam(Env, errorMessage);
+
+	UE_LOG(LogTbEnumEnumInterfaceClient_JNI, Warning,
+		TEXT("Async operation failed for id %s: %s (code: %d)"),
+		*callIdString, *errorMsgString, static_cast<int32>(errorCode));
+
+	FGuid guid;
+	FGuid::Parse(callIdString, guid);
+	gUTbEnumEnumInterfaceJniClientmethodHelper.FulfillPromiseWithDefaultValue(guid);
+}
 #endif

goldenmaster/Plugins/TbIfaceimport/Source/TbIfaceimportJni/Private/Generated/Detail/TbIfaceimportMethodHelper.h

@@ -48,6 +48,41 @@ class FTbIfaceimportMethodHelper
 		}
 	}
 
+	bool FulfillPromiseWithDefaultValue(const FGuid& InId)
+	{
+		UE_LOG(
+			LogTbIfaceimportMethodHelper_JNI,
+			Verbose,
+			TEXT("Resolving promise %s with default value for %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		TUniquePtr<ITbIfaceimportPromiseHolder> PromiseHolderPtr;
+		{
+			FScopeLock Lock(&ReplyPromisesMapCS);
+			if (auto* Found = ReplyPromisesMap.Find(InId))
+			{
+				PromiseHolderPtr = MoveTemp(*Found);
+				ReplyPromisesMap.Remove(InId);
+			}
+		}
+
+		if (PromiseHolderPtr)
+		{
+			PromiseHolderPtr->FulfillWithDefaultValue();
+			return true;
+		}
+
+		UE_LOG(
+			LogTbIfaceimportMethodHelper_JNI,
+			Verbose,
+			TEXT("No promise found for id %s in %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		return false;
+	}
+
 	template <typename ResultType>
 	FGuid StorePromise(TPromise<ResultType>&& InPromise);
 

goldenmaster/Plugins/TbNames/Source/TbNamesJni/Private/Generated/Detail/TbNamesMethodHelper.h

@@ -48,6 +48,41 @@ class FTbNamesMethodHelper
 		}
 	}
 
+	bool FulfillPromiseWithDefaultValue(const FGuid& InId)
+	{
+		UE_LOG(
+			LogTbNamesMethodHelper_JNI,
+			Verbose,
+			TEXT("Resolving promise %s with default value for %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		TUniquePtr<ITbNamesPromiseHolder> PromiseHolderPtr;
+		{
+			FScopeLock Lock(&ReplyPromisesMapCS);
+			if (auto* Found = ReplyPromisesMap.Find(InId))
+			{
+				PromiseHolderPtr = MoveTemp(*Found);
+				ReplyPromisesMap.Remove(InId);
+			}
+		}
+
+		if (PromiseHolderPtr)
+		{
+			PromiseHolderPtr->FulfillWithDefaultValue();
+			return true;
+		}
+
+		UE_LOG(
+			LogTbNamesMethodHelper_JNI,
+			Verbose,
+			TEXT("No promise found for id %s in %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		return false;
+	}
+
 	template <typename ResultType>
 	FGuid StorePromise(TPromise<ResultType>&& InPromise);
 

goldenmaster/Plugins/TbNames/Source/TbNamesJni/Private/Generated/Jni/TbNamesNamEsJniClient.cpp

@@ -960,4 +960,23 @@ JNI_METHOD void Java_tbNames_tbNamesjniclient_NamEsJniClient_nativeIsReady(JNIEn
 	}
 	localJniAccessor->notifyIsReady(value);
 }
+
+JNI_METHOD void Java_tbNames_tbNamesjniclient_NamEsJniClient_nativeAsyncOperationFailed(JNIEnv* Env, jclass Clazz, jstring callId, jstring errorMessage, jint errorCode)
+{
+	FString callIdString = FJavaHelper::FStringFromParam(Env, callId);
+	static const TCHAR* errorMsgId = TEXT("failed to convert call id string in nativeAsyncOperationFailed for tbNames/tbNamesjniclient/NamEsJniClient");
+	if (TbNamesDataJavaConverter::CheckJniErrorOccurred(errorMsgId))
+	{
+		return;
+	}
+	FString errorMsgString = FJavaHelper::FStringFromParam(Env, errorMessage);
+
+	UE_LOG(LogTbNamesNamEsClient_JNI, Warning,
+		TEXT("Async operation failed for id %s: %s (code: %d)"),
+		*callIdString, *errorMsgString, static_cast<int32>(errorCode));
+
+	FGuid guid;
+	FGuid::Parse(callIdString, guid);
+	gUTbNamesNamEsJniClientmethodHelper.FulfillPromiseWithDefaultValue(guid);
+}
 #endif

goldenmaster/Plugins/TbRefIfaces/Source/TbRefIfacesJni/Private/Generated/Detail/TbRefIfacesMethodHelper.h

@@ -48,6 +48,41 @@ class FTbRefIfacesMethodHelper
 		}
 	}
 
+	bool FulfillPromiseWithDefaultValue(const FGuid& InId)
+	{
+		UE_LOG(
+			LogTbRefIfacesMethodHelper_JNI,
+			Verbose,
+			TEXT("Resolving promise %s with default value for %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		TUniquePtr<ITbRefIfacesPromiseHolder> PromiseHolderPtr;
+		{
+			FScopeLock Lock(&ReplyPromisesMapCS);
+			if (auto* Found = ReplyPromisesMap.Find(InId))
+			{
+				PromiseHolderPtr = MoveTemp(*Found);
+				ReplyPromisesMap.Remove(InId);
+			}
+		}
+
+		if (PromiseHolderPtr)
+		{
+			PromiseHolderPtr->FulfillWithDefaultValue();
+			return true;
+		}
+
+		UE_LOG(
+			LogTbRefIfacesMethodHelper_JNI,
+			Verbose,
+			TEXT("No promise found for id %s in %s"),
+			*(InId.ToString(EGuidFormats::Digits)),
+			*OwnerName);
+
+		return false;
+	}
+
 	template <typename ResultType>
 	FGuid StorePromise(TPromise<ResultType>&& InPromise);