@akinon/next 1.76.0-rc.0 → 2.0.0-beta.0

package/components/input.tsx

@@ -1,8 +1,6 @@
 import clsx from 'clsx';
 import { forwardRef, FocusEvent, useState, Ref } from 'react';
 import { Controller } from 'react-hook-form';
-
-// @ts-ignore
 import { PatternFormat, PatternFormatProps } from 'react-number-format';
 import { InputProps } from '../types';
 import { twMerge } from 'tailwind-merge';

package/components/link.tsx

@@ -10,9 +10,7 @@ type LinkProps = Omit<
   React.AnchorHTMLAttributes<HTMLAnchorElement>,
   keyof NextLinkProps
 > &
-  NextLinkProps & {
-    href: string;
-  };
+  NextLinkProps;
 
 export const Link = ({ children, href, ...rest }: LinkProps) => {
   const { locale, defaultLocaleValue, localeUrlStrategy } = useLocalization();
@@ -28,21 +26,19 @@ export const Link = ({ children, href, ...rest }: LinkProps) => {
       return href;
     }
 
-    if (typeof href === 'string' && !href.startsWith('http')) {
-      const pathnameWithoutLocale = href.replace(urlLocaleMatcherRegex, '');
-      const hrefWithLocale = `/${locale}${pathnameWithoutLocale}`;
-
-      if (localeUrlStrategy === LocaleUrlStrategy.ShowAllLocales) {
-        return hrefWithLocale;
-      } else if (
-        localeUrlStrategy === LocaleUrlStrategy.HideDefaultLocale &&
-        locale !== defaultLocaleValue
-      ) {
-        return hrefWithLocale;
-      }
+    const pathnameWithoutLocale = href.replace(urlLocaleMatcherRegex, '');
+    const hrefWithLocale = `/${locale}${pathnameWithoutLocale}`;
+
+    if (localeUrlStrategy === LocaleUrlStrategy.ShowAllLocales) {
+      return hrefWithLocale;
+    } else if (
+      localeUrlStrategy === LocaleUrlStrategy.HideDefaultLocale &&
+      locale !== defaultLocaleValue
+    ) {
+      return hrefWithLocale;
     }
 
-    return href;
+    return href || '#';
   }, [href, defaultLocaleValue, locale, localeUrlStrategy]);
 
   return (

package/data/server/flatpage.ts

@@ -7,8 +7,7 @@ import { ServerVariables } from '../../utils/server-variables';
 const getFlatPageDataHandler = (
   pk: number,
   locale: string,
-  currency: string,
-  headers?: Record<string, string>
+  currency: string
 ) => {
   return async function () {
     const data = await appFetch<FlatPage>({
@@ -18,8 +17,7 @@ const getFlatPageDataHandler = (
       init: {
         headers: {
           Accept: 'application/json',
-          'Content-Type': 'application/json',
-          ...(headers ?? {})
+          'Content-Type': 'application/json'
         }
       }
     });
@@ -31,17 +29,15 @@ const getFlatPageDataHandler = (
 export const getFlatPageData = ({
   pk,
   locale = ServerVariables.locale,
-  currency = ServerVariables.currency,
-  headers
+  currency = ServerVariables.currency
 }: {
   pk: number;
   locale?: string;
   currency?: string;
-  headers?: Record<string, string>;
 }) => {
   return Cache.wrap(
     CacheKey.FlatPage(pk),
     locale,
-    getFlatPageDataHandler(pk, locale, currency, headers)
+    getFlatPageDataHandler(pk, locale, currency)
   );
 };

package/data/server/form.ts

@@ -5,12 +5,7 @@ import appFetch from '../../utils/app-fetch';
 import { ServerVariables } from '../../utils/server-variables';
 import { form } from '../urls';
 
-const getFormDataHandler = (
-  pk: number,
-  locale: string,
-  currency: string,
-  headers?: Record<string, string>
-) => {
+const getFormDataHandler = (pk: number, locale: string, currency: string) => {
   return async function () {
     const data = await appFetch<FormType>({
       url: form.getForm(pk),
@@ -19,8 +14,7 @@ const getFormDataHandler = (
       init: {
         headers: {
           Accept: 'application/json',
-          'Content-Type': 'application/json',
-          ...(headers ?? {})
+          'Content-Type': 'application/json'
         }
       }
     });
@@ -32,17 +26,15 @@ const getFormDataHandler = (
 export const getFormData = ({
   pk,
   locale = ServerVariables.locale,
-  currency = ServerVariables.currency,
-  headers
+  currency = ServerVariables.currency
 }: {
   pk: number;
   locale?: string;
   currency?: string;
-  headers?: Record<string, string>;
 }) => {
   return Cache.wrap(
     CacheKey.Form(pk),
     locale,
-    getFormDataHandler(pk, locale, currency, headers)
+    getFormDataHandler(pk, locale, currency)
   );
 };

package/data/server/landingpage.ts

@@ -7,8 +7,7 @@ import { ServerVariables } from '../../utils/server-variables';
 const getLandingPageHandler = (
   pk: number,
   locale: string,
-  currency: string,
-  headers?: Record<string, string>
+  currency: string
 ) => {
   return async function () {
     const data = await appFetch<LandingPage>({
@@ -18,8 +17,7 @@ const getLandingPageHandler = (
       init: {
         headers: {
           Accept: 'application/json',
-          'Content-Type': 'application/json',
-          ...(headers ?? {})
+          'Content-Type': 'application/json'
         }
       }
     });
@@ -31,17 +29,15 @@ const getLandingPageHandler = (
 export const getLandingPageData = ({
   pk,
   locale = ServerVariables.locale,
-  currency = ServerVariables.currency,
-  headers
+  currency = ServerVariables.currency
 }: {
   pk: number;
   locale?: string;
   currency?: string;
-  headers?: Record<string, string>;
 }) => {
   return Cache.wrap(
     CacheKey.LandingPage(pk),
     locale,
-    getLandingPageHandler(pk, locale, currency, headers)
+    getLandingPageHandler(pk, locale, currency)
   );
 };

package/data/server/menu.ts

@@ -13,7 +13,6 @@ interface MenuHandlerParams {
   currency?: string;
   depth?: number;
   parent?: string;
-  headers?: Record<string, string>;
 }
 
 const DEFAULT_DEPTH = 3;
@@ -23,17 +22,13 @@ const getMenuHandler =
     locale = ServerVariables.locale,
     currency = ServerVariables.currency,
     depth,
-    parent,
-    headers
+    parent
   }: MenuHandlerParams = {}) =>
   async () => {
     const response = await appFetch<MenuResponse>({
       url: misc.menus(depth ?? DEFAULT_DEPTH, parent),
       locale,
-      currency,
-      init: {
-        headers
-      }
+      currency
     });
 
     return response?.menu;

package/data/server/product.ts

@@ -11,7 +11,6 @@ type GetProduct = {
   currency?: string;
   searchParams?: URLSearchParams;
   groupProduct?: boolean;
-  headers?: Record<string, string>;
 };
 
 const getProductDataHandler = ({
@@ -19,8 +18,7 @@ const getProductDataHandler = ({
   locale,
   currency,
   searchParams,
-  groupProduct,
-  headers
+  groupProduct
 }: GetProduct) => {
   return async function () {
     let url = groupProduct
@@ -42,8 +40,7 @@ const getProductDataHandler = ({
       init: {
         headers: {
           Accept: 'application/json',
-          'Content-Type': 'application/json',
-          ...(headers ?? {})
+          'Content-Type': 'application/json'
         }
       }
     });
@@ -98,8 +95,7 @@ export const getProductData = async ({
   locale = ServerVariables.locale,
   currency = ServerVariables.currency,
   searchParams,
-  groupProduct,
-  headers
+  groupProduct
 }: GetProduct) => {
   return Cache.wrap(
     CacheKey[groupProduct ? 'GroupProduct' : 'Product'](
@@ -107,14 +103,7 @@ export const getProductData = async ({
       searchParams ?? new URLSearchParams()
     ),
     locale,
-    getProductDataHandler({
-      pk,
-      locale,
-      currency,
-      searchParams,
-      groupProduct,
-      headers
-    }),
+    getProductDataHandler({ pk, locale, currency, searchParams, groupProduct }),
     {
       expire: 300
     }

package/data/server/seo.ts

@@ -7,10 +7,9 @@ interface SeoDataParams {
   url: string;
   locale?: string;
   currency?: string;
-  headers?: Record<string, string>;
 }
 
-function getSeoDataHandler({ url, locale, currency, headers }: SeoDataParams) {
+function getSeoDataHandler({ url, locale, currency }: SeoDataParams) {
   return async function () {
     let data = {} as {
       title: string;
@@ -20,12 +19,7 @@ function getSeoDataHandler({ url, locale, currency, headers }: SeoDataParams) {
     };
 
     try {
-      data = await appFetch({
-        url: misc.cmsSeo(url),
-        locale,
-        currency,
-        init: { headers }
-      });
+      data = await appFetch({ url: misc.cmsSeo(url), locale, currency });
     } catch (error) {
       // logger.error('Error while fetching seo data', { url, error });
     }
@@ -37,12 +31,11 @@ function getSeoDataHandler({ url, locale, currency, headers }: SeoDataParams) {
 export const getSeoData = async (
   url,
   locale = ServerVariables.locale,
-  currency = ServerVariables.currency,
-  headers?: Record<string, string>
+  currency = ServerVariables.currency
 ) => {
   return Cache.wrap(
     CacheKey.Seo(url),
     locale,
-    getSeoDataHandler({ url, locale, currency, headers })
+    getSeoDataHandler({ url, locale, currency })
   );
 };

package/data/server/widget.ts

@@ -6,44 +6,29 @@ import { widgets } from '../urls';
 import { ServerVariables } from '../../utils/server-variables';
 
 const getWidgetDataHandler =
-  (
-    slug: string,
-    locale: string,
-    currency: string,
-    headers?: Record<string, string>
-  ) =>
-  async () => {
+  (slug: string, locale: string, currency: string) => async () => {
     if (!slug) {
       return null;
     }
 
-    return await appFetch({
-      url: widgets.getWidget(slug),
-      locale,
-      currency,
-      init: {
-        headers
-      }
-    });
+    return await appFetch({ url: widgets.getWidget(slug), locale, currency });
   };
 
 export const getWidgetData = async <T>({
   slug,
   locale = ServerVariables.locale,
   currency = ServerVariables.currency,
-  cacheOptions,
-  headers
+  cacheOptions
 }: {
   slug: string;
   locale?: string;
   currency?: string;
   cacheOptions?: CacheOptions;
-  headers?: Record<string, string>;
 }): Promise<WidgetResultType<T>> => {
   return Cache.wrap(
     CacheKey.Widget(slug),
     locale,
-    getWidgetDataHandler(slug, locale, currency, headers),
+    getWidgetDataHandler(slug, locale, currency),
     cacheOptions
   );
 };

package/hocs/server/with-segment-defaults.tsx

@@ -72,13 +72,10 @@ const addRootLayoutProps = async (componentProps: RootLayoutProps) => {
 const checkRedisVariables = () => {
   const requiredVariableValues = [
     process.env.CACHE_HOST,
-    process.env.CACHE_PORT
+    process.env.CACHE_PORT,
+    process.env.CACHE_SECRET
   ];
 
-  if (!settings.usePrettyUrlRoute) {
-    requiredVariableValues.push(process.env.CACHE_SECRET);
-  }
-
   if (
     !requiredVariableValues.every((v) => v) &&
     process.env.NODE_ENV === 'production'

package/instrumentation/index.ts

@@ -1,12 +1,5 @@
-import { initSentry } from '../sentry';
-
 export async function register() {
   if (process.env.NEXT_RUNTIME === 'nodejs') {
     await import('./node');
-    initSentry('Server');
-  }
-
-  if (process.env.NEXT_RUNTIME === 'edge') {
-    initSentry('Edge');
   }
 }

package/lib/cache-handler.mjs

@@ -23,10 +23,10 @@ CacheHandler.onCreation(async () => {
     timeoutMs: 5000
   });
 
-  // const localHandler = createLruHandler();
+  const localHandler = createLruHandler();
 
   return {
-    handlers: [redisHandler]
+    handlers: [redisHandler, localHandler]
   };
 });
 

package/lib/cache.ts

@@ -31,8 +31,6 @@ export const CacheKey = {
     `category_${pk}_${encodeURIComponent(
       JSON.stringify(searchParams)
     )}${hashCacheKey(headers)}`,
-  Basket: (namespace?: string) => `basket${namespace ? `_${namespace}` : ''}`,
-  AllBaskets: () => 'all_baskets',
   CategorySlug: (slug: string) => `category_${slug}`,
   SpecialPage: (
     pk: number,

package/middlewares/complete-gpay.ts

@@ -145,8 +145,7 @@ const withCompleteGpay =
       logger.info('Redirecting to order success page', {
         middleware: 'complete-gpay',
         redirectUrlWithLocale,
-        ip,
-        setCookie: request.headers.get('set-cookie')
+        ip
       });
 
       // Using POST method while redirecting causes an error,

package/middlewares/complete-masterpass.ts

@@ -145,8 +145,7 @@ const withCompleteMasterpass =
       logger.info('Redirecting to order success page', {
         middleware: 'complete-masterpass',
         redirectUrlWithLocale,
-        ip,
-        setCookie: request.headers.get('set-cookie')
+        ip
       });
 
       // Using POST method while redirecting causes an error,

package/middlewares/default.ts

@@ -285,9 +285,7 @@ const withPzDefault =
                                 url.pathname =
                                   url.pathname +
                                   (/\/$/.test(url.pathname) ? '' : '/') +
-                                  `searchparams|${url.searchParams
-                                    .toString()
-                                    .replace(/%26/g, '--amp--')}`;
+                                  `searchparams|${url.searchParams.toString()}`;
                               }
 
                               if (

package/middlewares/redirection-payment.ts

@@ -146,8 +146,7 @@ const withRedirectionPayment =
       logger.info('Redirecting to order success page', {
         middleware: 'redirection-payment',
         redirectUrlWithLocale,
-        ip,
-        setCookie: request.headers.get('set-cookie')
+        ip
       });
 
       // Using POST method while redirecting causes an error,

package/middlewares/saved-card-redirection.ts

@@ -145,8 +145,7 @@ const withSavedCardRedirection =
       logger.info('Redirecting to order success page', {
         middleware: 'saved-card-redirection',
         redirectUrlWithLocale,
-        ip,
-        setCookie: request.headers.get('set-cookie')
+        ip
       });
 
       // Using POST method while redirecting causes an error,

package/middlewares/three-d-redirection.ts

@@ -145,8 +145,7 @@ const withThreeDRedirection =
       logger.info('Redirecting to order success page', {
         middleware: 'three-d-redirection',
         redirectUrlWithLocale,
-        ip,
-        setCookie: request.headers.get('set-cookie')
+        ip
       });
 
       // Using POST method while redirecting causes an error,

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@akinon/next",
   "description": "Core package for Project Zero Next",
-  "version": "1.76.0-rc.0",
+  "version": "2.0.0-beta.0",
   "private": false,
   "license": "MIT",
   "bin": {
@@ -20,7 +20,7 @@
     "@opentelemetry/sdk-trace-node": "1.19.0",
     "@opentelemetry/semantic-conventions": "1.19.0",
     "@reduxjs/toolkit": "1.9.7",
-    "@neshca/cache-handler": "1.9.0",
+    "@neshca/cache-handler": "1.5.1",
     "cross-spawn": "7.0.3",
     "generic-pool": "3.9.0",
     "react-redux": "8.1.3",
@@ -30,7 +30,7 @@
     "set-cookie-parser": "2.6.0"
   },
   "devDependencies": {
-    "@akinon/eslint-plugin-projectzero": "1.76.0-rc.0",
+    "@akinon/eslint-plugin-projectzero": "2.0.0-beta.0",
     "@types/react-redux": "7.1.30",
     "@types/set-cookie-parser": "2.4.7",
     "@typescript-eslint/eslint-plugin": "6.7.4",

package/types/commerce/order.ts

@@ -114,7 +114,6 @@ export interface Order {
     pk: number;
     name: string;
     slug: string;
-    logo: string;
     [key: string]: any;
   };
 }

package/utils/app-fetch.ts

@@ -43,12 +43,12 @@ const appFetch = async <T>({
     const requestURL = `${decodeURIComponent(commerceUrl)}${url}`;
 
     init.headers = {
-      cookie: nextCookies.toString(),
       ...(init.headers ?? {}),
       ...(ServerVariables.globalHeaders ?? {}),
       'Accept-Language': currentLocale.apiValue,
       'x-currency': currency,
-      'x-forwarded-for': ip
+      'x-forwarded-for': ip,
+      cookie: nextCookies.toString()
     };
 
     init.next = {

package/with-pz-config.js

@@ -64,7 +64,7 @@ const defaultConfig = {
   },
   sentry: {
     hideSourceMaps: true
-  } // TODO: This section will be reviewed again in the Sentry 8 update.
+  }
 };
 
 const withPzConfig = (

package/data/server/basket.ts

@@ -1,72 +0,0 @@
-import { Cache, CacheKey } from '../../lib/cache';
-import { basket } from '../../data/urls';
-import { Basket } from '../../types';
-import appFetch from '../../utils/app-fetch';
-import { ServerVariables } from '../../utils/server-variables';
-import logger from '../../utils/log';
-
-type GetBasketParams = {
-  locale?: string;
-  currency?: string;
-  namespace?: string;
-};
-
-const getBasketDataHandler = ({
-  locale,
-  currency,
-  namespace
-}: GetBasketParams) => {
-  return async function () {
-    try {
-      const url = namespace
-        ? basket.getBasketDetail(namespace)
-        : basket.getBasket;
-
-      const basketData = await appFetch<{ basket: Basket }>({
-        url,
-        locale,
-        currency,
-        init: {
-          headers: {
-            Accept: 'application/json',
-            'Content-Type': 'application/json'
-          }
-        }
-      });
-
-      if (!basketData?.basket) {
-        logger.warn('Basket data is undefined', {
-          handler: 'getBasketDataHandler',
-          namespace
-        });
-      }
-
-      return basketData;
-    } catch (error) {
-      logger.error('Error fetching basket data', {
-        handler: 'getBasketDataHandler',
-        error,
-        namespace
-      });
-      throw error;
-    }
-  };
-};
-
-export const getBasketData = async ({
-  locale = ServerVariables.locale,
-  currency = ServerVariables.currency,
-  namespace
-}: GetBasketParams = {}) => {
-  return Cache.wrap(
-    CacheKey.Basket(namespace),
-    locale,
-    getBasketDataHandler({ locale, currency, namespace }),
-    {
-      expire: 0,
-      cache: false
-    }
-  );
-};
-
-