@applicaster/zapp-react-dom-app 13.0.0-alpha.7210171818 → 13.0.0-alpha.7846284820

package/App/InteractionManager/index.tsx

@@ -341,23 +341,24 @@ class InteractionManagerClass extends React.Component<Props, State> {
       this.longPressDetector.endKeyPressMonitoring(event.code);
     }
 
-    if (keyCode(event).matches(KEYS.Enter) || keyCode(event).matches(KEYS.X)) {
+    if (
+      keyCode(event).matches(KEYS.Enter) ||
+      (__DEV__ && keyCode(event).matches(KEYS.X))
+    ) {
       focusManager.pressOut();
     }
 
     if (
       keyCode(event).matches(KEYS.Forward) ||
-      keyCode(event).matches(KEYS.C)
+      (__DEV__ && keyCode(event).matches(KEYS.C))
     ) {
       setDisplayState(DISPLAY_STATES.HUD);
       KeyInputHandler.getInstance().onForwardPressOut();
-
-      return;
     }
 
     if (
       keyCode(event).matches(KEYS.Rewind) ||
-      keyCode(event).matches(KEYS.Z)
+      (__DEV__ && keyCode(event).matches(KEYS.Z))
     ) {
       setDisplayState(DISPLAY_STATES.HUD);
       KeyInputHandler.getInstance().onRewindPressOut();
@@ -365,9 +366,6 @@ class InteractionManagerClass extends React.Component<Props, State> {
   }
 
   onKeyDown(event) {
-    console.log("Anton event", event);
-    // console.log("Anton: event", event);
-    // debugger;
     const { displayState, setDisplayState, navigator } = this.props;
     const { isKeyboardVisible } = this.state;
 
@@ -388,8 +386,8 @@ class InteractionManagerClass extends React.Component<Props, State> {
       this.longPressDetector.monitorKeyPress(event.code, event);
     }
 
-    // None of the logic below should be executed for repeated keys
-    if (event.repeat) return;
+    // Only handle repeat events for arrow keys
+    if (event.repeat && !keyCode(event).matchesAny(...ARROW_KEYS)) return;
 
     /**
      When the player is displayed, we listen to Native Play / Pause / Stop
@@ -430,7 +428,7 @@ class InteractionManagerClass extends React.Component<Props, State> {
 
       if (
         keyCode(event).matches(KEYS.Forward) ||
-        keyCode(event).matches(KEYS.C)
+        (__DEV__ && keyCode(event).matches(KEYS.C))
       ) {
         setDisplayState(DISPLAY_STATES.HUD);
         KeyInputHandler.getInstance().onForwardPress();
@@ -440,7 +438,7 @@ class InteractionManagerClass extends React.Component<Props, State> {
 
       if (
         keyCode(event).matches(KEYS.Rewind) ||
-        keyCode(event).matches(KEYS.Z)
+        (__DEV__ && keyCode(event).matches(KEYS.Z))
       ) {
         setDisplayState(DISPLAY_STATES.HUD);
         KeyInputHandler.getInstance().onRewindPress();
@@ -481,7 +479,9 @@ class InteractionManagerClass extends React.Component<Props, State> {
             const isHookInHomescreen = entryData?.payload?.home;
 
             const isHookFlowBlocker =
-              entryData?.hookPlugin?.module?.isFlowBlocker();
+              typeof entryData?.hookPlugin?.module?.isFlowBlocker === "function"
+                ? entryData?.hookPlugin?.module?.isFlowBlocker()
+                : entryData?.hookPlugin?.module?.isFlowBlocker;
 
             if (isHookInHomescreen && isHookFlowBlocker) {
               this.platformBack();
@@ -503,7 +503,10 @@ class InteractionManagerClass extends React.Component<Props, State> {
       }
     }
 
-    if (keyCode(event).matches(KEYS.Enter) || keyCode(event).matches(KEYS.X)) {
+    if (
+      keyCode(event).matches(KEYS.Enter) ||
+      (__DEV__ && keyCode(event).matches(KEYS.X))
+    ) {
       if (displayState === DISPLAY_STATES.PLAYER) {
         setDisplayState(DISPLAY_STATES.HUD);
       } else {
@@ -620,6 +623,7 @@ class InteractionManagerClass extends React.Component<Props, State> {
 
     if (
       (target.tagName === "INPUT" && target.type === "text") ||
+      (target.tagName === "INPUT" && target.type === "password") ||
       target.tagName === "TEXTAREA"
     ) {
       this.setState({
@@ -718,27 +722,13 @@ class InteractionManagerClass extends React.Component<Props, State> {
     }, SEQUENCE_ENTRY_TIME);
   }
 
+  // TODO: Delete it on next PR if not needed
   private handleLongPress = (event: CustomEvent) => {
     const { keyCode } = event.detail;
-    const { displayState } = this.props;
-    const playerState = displayState === DISPLAY_STATES.PLAYER;
-    const hudState = displayState === DISPLAY_STATES.HUD;
-    const playbackState = playerState || hudState;
     this.longPressActive = true;
     this.cancelHudTimeout();
 
     switch (keyCode) {
-      case KEYS.Forward.code:
-      case KEYS.C.code:
-        // playbackState && KeyInputHandler.getInstance().onForwardLongPress();
-
-        break;
-
-      case KEYS.Rewind.code:
-      case KEYS.Z.code:
-        // playbackState && KeyInputHandler.getInstance().onRewindLongPress();
-        break;
-
       case KEYS.Enter.code:
       case KEYS.X.code:
         focusManager.longPress();
@@ -746,23 +736,13 @@ class InteractionManagerClass extends React.Component<Props, State> {
     }
   };
 
+  // TODO: Delete it on next PR if not needed
   private handleLongPressRelease = (event: CustomEvent) => {
     const { keyCode } = event.detail;
     this.longPressActive = false;
     this.resetHudTimeout();
-    // debugger;
 
     switch (keyCode) {
-      case KEYS.Forward.code:
-      case KEYS.C.code:
-        // KeyInputHandler.getInstance().onForwardPressOut();
-        break;
-
-      case KEYS.Rewind.code:
-      case KEYS.Z.code:
-        // KeyInputHandler.getInstance().onRewindPressOut();
-        break;
-
       case KEYS.Enter.code:
       case KEYS.X.code:
         // focusManager.pressOut();

package/App/Loader/__tests__/__snapshots__/loadDomPlugins.test.js.snap

@@ -1,15 +1,15 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`loadDomPlugins when there are plugins requiring to write to the dom skips plugins without init method 1`] = `
-Array [
-  Array [
+[
+  [
     "could not initialize plugin Plugin with Dom node & no init method - it doesn't have an init method",
-    Object {
-      "plugin": Object {
-        "configuration": Object {
+    {
+      "plugin": {
+        "configuration": {
           "foo": "bar",
         },
-        "module": Object {
+        "module": {
           "requiresAppendingDomNode": true,
         },
         "name": "Plugin with Dom node & no init method",

package/App/Loader/loadLocale.js

@@ -4,8 +4,8 @@ import { QuickBrickCommunicationModule } from "../../Polyfills/QuickBrickCommuni
 import { loadLocaleImpl, getLanguageCode } from "./utils";
 
 type Locale = {
-  language: string,
-  country: string,
+  language: string;
+  country: string;
 };
 
 async function getBrowserLocale() {

package/App/Loader/utils/platform.js

@@ -11,11 +11,28 @@ import {
   PLATFORMS,
 } from "./const";
 
+/**
+ * @typedef {"tv" | "web" | "other"} DeviceType
+ * @typedef {"samsung_tv" | "lg_tv" | unknown} CustomPlatformTypes
+ */
+
+/**
+ * Retrieves the user agent string from the browser or global navigator.
+ * @returns {string} The user agent string.
+ */
 export const getUserAgent = () =>
   window?.navigator?.userAgent || global?.navigator?.userAgent;
 
+/**
+ * Checks if the device is HD (1280 width).
+ * @returns {boolean} True if the device is HD, false otherwise.
+ */
 export const isHD = () => window.innerWidth === 1280;
 
+/**
+ * Checks if the platform is LG.
+ * @returns {boolean} True if the platform is LG, false otherwise.
+ */
 export const isLgPlatform = () => {
   const userAgent = getUserAgent();
 
@@ -23,10 +40,13 @@ export const isLgPlatform = () => {
     return false;
   }
 
-  // Note: LG uses the number 0 in their userAgent, possibly because of Palm WebOS
   return hasWebOS || userAgent.includes(PLATFORMS.webos);
 };
 
+/**
+ * Checks if the platform is Samsung.
+ * @returns {boolean} True if the platform is Samsung, false otherwise.
+ */
 export const isSamsungPlatform = () => {
   const userAgent = getUserAgent();
 
@@ -42,42 +62,39 @@ export const isSamsungPlatform = () => {
 };
 
 /**
- * Checks if we are on the Vizio platform by using the userAgent string
- *
- * Use this method when you need to identify the platform on runtime
- * before the VIZIO Companion Library has loaded
+ * Checks if the platform is Vizio by using the user agent string.
+ * @returns {boolean} True if the platform is Vizio, false otherwise.
  */
 export const isVizioPlatform = () => {
   const userAgent = getUserAgent();
 
-  const isVizioAgent =
+  return (
     userAgent.includes(PLATFORMS.vizio) ||
     userAgent.includes(PLATFORMS.smartcast) ||
-    userAgent.includes(PLATFORMS.conjure);
-
-  return isVizioAgent;
+    userAgent.includes(PLATFORMS.conjure)
+  );
 };
 
 /**
- * Checks if we are on the Vizio platform by checking for the global VIZIO object
- * and the VIZIO Companion Library. This method does not use the userAgent string,
- * because it mainly checks whether we have access to apis, which may not be available yet
+ * Checks if the Vizio APIs are available.
+ * @returns {boolean} True if the Vizio APIs are available, false otherwise.
  */
 export const hasVizioAPIs = () => {
-  const hasAPIs =
+  return (
     typeof window.VIZIO !== "undefined" &&
-    window?.applicaster?.vizioLibraryDidLoad;
-
-  return hasAPIs;
+    window?.applicaster?.vizioLibraryDidLoad
+  );
 };
 
 /**
- * platformSelect can't be used, because Platform.OS is overriden
- * by overloadReactNativePlatform and don't support 'web' value
- * @returns {"tv" | "web" | "other"} value
+ * Determines the device type based on the platform.
+ * @returns {DeviceType} The device type ("tv", "web", or "other").
  */
 export const getDeviceType = () => {
-  switch (Platform.OS) {
+  /** @type {CustomPlatformTypes} */
+  const platformOS = Platform.OS;
+
+  switch (platformOS) {
     case "samsung_tv":
       return isSamsungPlatform() ? "tv" : "web";
     case "lg_tv":
@@ -88,14 +105,15 @@ export const getDeviceType = () => {
 };
 
 /**
- * This function is used to rename keys
- * @param {string} key - key you would like to rename
+ * Renames keys based on a predefined map.
+ * @param {string} key - The key to rename.
+ * @returns {string} The renamed key.
  */
 export const renameKeys = (key) => desiredKeysMap?.[key] || key;
 
 /**
- * This function is used to get the connection info from WebOS
- * TODO: move to connection hook
+ * Retrieves the connection info from WebOS.
+ * @returns {Promise<{networkType: string} | null>} A promise that resolves with the connection info or null.
  */
 export const getWebOSConnectionInfo = () => {
   return new Promise((resolve, reject) => {
@@ -108,7 +126,6 @@ export const getWebOSConnectionInfo = () => {
     webOS.service.request("luna://com.palm.connectionmanager", {
       method: "getStatus",
       parameters: { subscribe: false },
-
       onSuccess: (args) => {
         const types = Object.keys(args)
           .filter((key) => args[key].state === "connected")
@@ -116,7 +133,6 @@ export const getWebOSConnectionInfo = () => {
 
         resolve({ networkType: types });
       },
-
       onFailure: (error) => {
         console.error(
           "getWebOSConnectionInfo: There was an error getting connection info",
@@ -129,6 +145,10 @@ export const getWebOSConnectionInfo = () => {
   });
 };
 
+/**
+ * Retrieves the connection info from Tizen.
+ * @returns {Promise<{networkType: string} | null>} A promise that resolves with the connection info or null.
+ */
 export const getTizenConnectionInfo = () => {
   return new Promise((resolve, reject) => {
     if (!isSamsungPlatform) {

package/Polyfills/Storage/__tests__/__snapshots__/storage.test.js.snap

@@ -1,7 +1,7 @@
 // Jest Snapshot v1, https://goo.gl/fbAQLP
 
 exports[`storageObject when the requested storage exists getAllItems returns all keys 1`] = `
-Object {
+{
   "baz_::_foo": "baz",
   "baz_::_otherKey": "otherValue",
   "foo": "bar",
@@ -10,7 +10,7 @@ Object {
 `;
 
 exports[`storageObject when the requested storage exists getAllItems returns all keys 2`] = `
-Object {
+{
   "baz_::_foo": "baz",
   "baz_::_otherKey": "otherValue",
   "foo": "bar",
@@ -19,14 +19,14 @@ Object {
 `;
 
 exports[`storageObject when the requested storage exists getAllItems returns all keys in the namespace if provided 1`] = `
-Object {
+{
   "foo": "baz",
   "otherKey": "otherValue",
 }
 `;
 
 exports[`storageObject when the requested storage exists getAllItems returns all keys in the namespace if provided 2`] = `
-Object {
+{
   "foo": "baz",
   "otherKey": "otherValue",
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@applicaster/zapp-react-dom-app",
-  "version": "13.0.0-alpha.7210171818",
+  "version": "13.0.0-alpha.7846284820",
   "description": "Zapp App Component for Applicaster's Quick Brick React Native App",
   "main": "index.js",
   "scripts": {
@@ -22,11 +22,11 @@
   },
   "homepage": "https://github.com/applicaster/zapp-react-dom-app#readme",
   "dependencies": {
-    "@applicaster/zapp-react-dom-ui-components": "13.0.0-alpha.7210171818",
-    "@applicaster/zapp-react-native-bridge": "13.0.0-alpha.7210171818",
-    "@applicaster/zapp-react-native-redux": "13.0.0-alpha.7210171818",
-    "@applicaster/zapp-react-native-ui-components": "13.0.0-alpha.7210171818",
-    "@applicaster/zapp-react-native-utils": "13.0.0-alpha.7210171818",
+    "@applicaster/zapp-react-dom-ui-components": "13.0.0-alpha.7846284820",
+    "@applicaster/zapp-react-native-bridge": "13.0.0-alpha.7846284820",
+    "@applicaster/zapp-react-native-redux": "13.0.0-alpha.7846284820",
+    "@applicaster/zapp-react-native-ui-components": "13.0.0-alpha.7846284820",
+    "@applicaster/zapp-react-native-utils": "13.0.0-alpha.7846284820",
     "abortcontroller-polyfill": "^1.7.5",
     "history": "^4.7.2",
     "react-router-dom": "^5.1.2",
@@ -36,9 +36,19 @@
     "videojs-contrib-quality-levels": "3.0.0"
   },
   "peerDependencies": {
+    "@applicaster/zapp-pipes-v2-client": "*",
+    "@react-native-community/netinfo": "*",
+    "@types/node": "*",
     "core-js": "3.29.1",
+    "immer": "*",
     "react": "17.0.2",
     "react-dom": "17.0.2",
-    "react-native-web": "0.18.11"
+    "react-native": "*",
+    "react-native-safe-area-context": "*",
+    "react-native-svg": "*",
+    "react-native-web": "0.18.11",
+    "uglify-js": "*",
+    "validate-color": "*",
+    "zustand": "*"
   }
 }