Merge pull request #863 from Telegram-Mini-Apps/bugix/is-tma-problem

Bugix/is tma problem

Author: heyqbnk

.changeset/yellow-adults-attack.md

@@ -0,0 +1,5 @@
+---
+"@tma.js/bridge": patch
+---
+
+Rework the `hasWebviewProxy` function considering that the passed value is more likely to be window object

packages/bridge/src/env/hasWebviewProxy.test.ts

@@ -7,6 +7,7 @@ it('should return true if passed object contains path property "TelegramWebviewP
   expect(hasWebviewProxy({ TelegramWebviewProxy: {} })).toBe(false);
   expect(hasWebviewProxy({ TelegramWebviewProxy: { postEvent: [] } })).toBe(false);
   expect(hasWebviewProxy({
+    a: 'b',
     TelegramWebviewProxy: {
       postEvent: () => {
       },

packages/bridge/src/env/hasWebviewProxy.ts

@@ -1,4 +1,4 @@
-import { looseObject, function as fn, is } from 'valibot';
+import * as v from 'valibot';
 
 /**
  * Returns true in case, passed value contains path `TelegramWebviewProxy.postEvent` property and
@@ -10,8 +10,8 @@ export function hasWebviewProxy<T>(value: T): value is T & {
     postEvent: (...args: unknown[]) => unknown;
   };
 } {
-  return is(
-    looseObject({ TelegramWebviewProxy: looseObject({ postEvent: fn() }) }),
-    value,
+  return typeof value === 'object' && !!value && !Array.isArray(value) && v.is(
+    v.looseObject({ TelegramWebviewProxy: v.looseObject({ postEvent: v.function() }) }),
+    { TelegramWebviewProxy: (value as Record<string, unknown>).TelegramWebviewProxy },
   );
 }