@vtex/faststore-plugin-buyer-portal 1.1.115 → 1.1.116-experimental.20251010183930

package/.github/workflows/experimental.yaml

@@ -0,0 +1,70 @@
+# This Action runs on experimental branch and publishes an experimental version to npm
+name: CD Experimental
+
+on:
+  push:
+    branches:
+      - experimental
+
+jobs:
+  build:
+    name: BuyerPortal Experimental
+    timeout-minutes: 15
+    runs-on: ubuntu-latest
+
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v4
+        with:
+          fetch-depth: 2
+
+      - name: Setup Node.js environment
+        uses: actions/setup-node@v4
+        with:
+          node-version: 20
+          cache: "yarn"
+          registry-url: "https://registry.npmjs.org"
+
+      - name: Configure CI Git User
+        run: |
+          git config user.name vtexgithubbot
+          git config user.email vtexgithubbot@github.com
+
+      - name: Install dependencies
+        run: yarn
+
+      - name: Get current version
+        id: get_version
+        run: |
+          CURRENT_VERSION=$(node -p "require('./package.json').version")
+          echo "current_version=$CURRENT_VERSION" >> $GITHUB_OUTPUT
+
+      - name: Generate experimental version
+        id: generate_version
+        run: |
+          # Extract base version (remove any existing prerelease suffix)
+          BASE_VERSION=$(echo "${{ steps.get_version.outputs.current_version }}" | sed -E 's/(-.*)?$//')
+          # Generate timestamp-based identifier
+          TIMESTAMP=$(date +%Y%m%d%H%M%S)
+          # Create experimental version
+          EXPERIMENTAL_VERSION="${BASE_VERSION}-experimental.${TIMESTAMP}"
+          echo "experimental_version=$EXPERIMENTAL_VERSION" >> $GITHUB_OUTPUT
+          echo "Publishing experimental version: $EXPERIMENTAL_VERSION"
+
+      - name: Update package.json version
+        run: |
+          npm version ${{ steps.generate_version.outputs.experimental_version }} --no-git-tag-version
+
+      - name: Publish experimental version
+        run: yarn publish --non-interactive --tag experimental
+        env:
+          NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
+
+      - name: Push changes and tags
+        run: |
+          git config user.email "vtexgithubbot@github.com"
+          git config user.name "vtexgithubbot"
+          git add package.json
+          git commit -m "chore: publish experimental version ${{ steps.generate_version.outputs.experimental_version }}" || echo "No changes to commit"
+          git push origin experimental || echo "No changes to push"
+

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@vtex/faststore-plugin-buyer-portal",
-	"version": "1.1.115",
+	"version": "1.1.116-experimental.20251010183930",
 	"description": "A plugin for faststore with buyer portal",
 	"main": "index.js",
 	"scripts": {

package/src/features/shared/hooks/useCookieMonitor.ts

@@ -12,7 +12,6 @@ import { AUT_COOKIE_KEY } from "../utils/constants";
  */
 export const useCookieMonitor = () => {
   const [hasCookie, setHasCookie] = useState(false);
-  const [timedOut, setTimedOut] = useState(false);
 
   useEffect(() => {
     const checkCookie = () => {
@@ -59,10 +58,7 @@ export const useCookieMonitor = () => {
 
       // Cleanup after 5 seconds max
       const timeout = setTimeout(() => {
-        console.log(
-          "[useCookieMonitor] Polling timeout reached (5s) - cookie never appeared"
-        );
-        setTimedOut(true);
+        console.log("[useCookieMonitor] Polling timeout reached (5s)");
         clearInterval(interval);
       }, 5000);
 
@@ -73,5 +69,5 @@ export const useCookieMonitor = () => {
     }
   }, []);
 
-  return { hasCookie, timedOut };
+  return hasCookie;
 };

package/src/features/shared/types/AuthRouteProps.ts

@@ -3,9 +3,4 @@ import { ClientContext } from "../utils";
 export type AuthRouteProps<T> =
   | { authorized: true; data: T; clientContext: ClientContext; loading: false }
   | { authorized: false; loading: false }
-  | {
-      authorized: false;
-      loading: true;
-      data?: Partial<T>;
-      clientContext?: ClientContext;
-    };
+  | { authorized: false; loading: true };

package/src/features/shared/utils/withAuth.tsx

@@ -2,6 +2,7 @@ import { useEffect, useRef, type ComponentType } from "react";
 
 import { useRouter } from "next/router";
 
+import { PageLoader } from "../components";
 import { useCookieMonitor } from "../hooks/useCookieMonitor";
 
 import type { AuthRouteProps } from "../types";
@@ -15,7 +16,7 @@ export const withAuth = <T extends Record<string, unknown>>(
 ) => {
   return function AuthenticatedComponent(props: AuthRouteProps<T>) {
     const router = useRouter();
-    const { hasCookie, timedOut } = useCookieMonitor();
+    const hasCookie = useCookieMonitor();
     const reloadAttemptedRef = useRef(false);
 
     // Debug logging
@@ -24,10 +25,9 @@ export const withAuth = <T extends Record<string, unknown>>(
         authorized: props?.authorized,
         loading: props?.loading,
         hasCookie,
-        timedOut,
         reloadAttempted: reloadAttemptedRef.current,
       });
-    }, [props?.authorized, props?.loading, hasCookie, timedOut]);
+    }, [props?.authorized, props?.loading, hasCookie]);
 
     useEffect(() => {
       // If not authorized and not loading, reload the page
@@ -56,28 +56,14 @@ export const withAuth = <T extends Record<string, unknown>>(
       }
     }, [props?.loading, hasCookie, router]);
 
-    useEffect(() => {
-      // If we're in loading state and timeout reached, redirect to home
-      if (props?.loading && timedOut && !reloadAttemptedRef.current) {
-        console.log("[withAuth] Cookie timeout - redirecting to home page");
-        reloadAttemptedRef.current = true;
-        router.push("/");
+    if (!props?.authorized) {
+      if (props?.loading) {
+        return <PageLoader />;
       }
-    }, [props?.loading, timedOut, router]);
-
-    // Always render the component, but pass loading state
-    // This allows components to handle their own loading states
-    if (!props?.authorized && !props?.loading) {
-      // Not authorized and not loading = permanently denied
       return null;
     }
 
-    // Render with whatever data we have (might be empty/partial if loading)
-    const componentProps = {
-      ...(props?.data || ({} as T)),
-      loading: props?.loading || false,
-    } as T & { loading: boolean };
-
-    return <Component {...componentProps} />;
+    // If authorized, render the component with the data
+    return <Component {...(props.data as T)} />;
   };
 };

package/src/features/shared/utils/withAuthProvider.tsx

@@ -2,7 +2,7 @@ import { useEffect, useRef, type ComponentType } from "react";
 
 import { useRouter } from "next/router";
 
-import { BuyerPortalProvider } from "../components";
+import { BuyerPortalProvider, PageLoader } from "../components";
 import { useCookieMonitor } from "../hooks/useCookieMonitor";
 
 import type { AuthRouteProps } from "../types";
@@ -18,7 +18,7 @@ export const withAuthProvider = <T,>(
 ) => {
   return function WrappedPage(props: AuthRouteProps<T>) {
     const router = useRouter();
-    const { hasCookie, timedOut } = useCookieMonitor();
+    const hasCookie = useCookieMonitor();
     const reloadAttemptedRef = useRef(false);
     const isProduction = process.env.NODE_ENV === "production";
 
@@ -29,7 +29,6 @@ export const withAuthProvider = <T,>(
           authorized: props?.authorized,
           loading: props?.loading,
           hasCookie,
-          timedOut,
           reloadAttempted: reloadAttemptedRef.current,
           routerPath: router.asPath,
         });
@@ -38,7 +37,6 @@ export const withAuthProvider = <T,>(
       props?.authorized,
       props?.loading,
       hasCookie,
-      timedOut,
       router.asPath,
       isProduction,
     ]);
@@ -73,35 +71,17 @@ export const withAuthProvider = <T,>(
       }
     }, [props?.loading, hasCookie, router]);
 
-    useEffect(() => {
-      // If we're in loading state and timeout reached, redirect to home
-      if (props?.loading && timedOut && !reloadAttemptedRef.current) {
-        console.log(
-          "[withAuthProvider] Cookie timeout - redirecting to home page"
-        );
-        reloadAttemptedRef.current = true;
-        router.push("/");
+    if (!props?.authorized) {
+      if (props?.loading) {
+        console.log("[withAuthProvider] Loading state");
+        return <PageLoader />;
       }
-    }, [props?.loading, timedOut, router]);
-
-    // Always render the component, but pass loading state
-    // This allows components to handle their own loading states
-    if (!props?.authorized && !props?.loading) {
-      // Not authorized and not loading = permanently denied
       return null;
     }
 
-    // Render with whatever data we have (might be empty/partial if loading)
-    const componentProps = {
-      ...(props?.data || ({} as T)),
-      loading: props?.loading || false,
-    } as T & { loading: boolean };
-
-    const clientContext = props?.clientContext || ({} as ClientContext);
-
     return (
-      <BuyerPortalProvider clientContext={clientContext}>
-        <Component {...componentProps} clientContext={clientContext} />
+      <BuyerPortalProvider clientContext={props?.clientContext}>
+        <Component {...props.data} clientContext={props?.clientContext} />
       </BuyerPortalProvider>
     );
   };