@faststore/core 3.100.3 → 3.100.4

package/.turbo/turbo-build.log

@@ -1,23 +1,23 @@
 
-> @faststore/core@3.100.2 prebuild /home/runner/work/faststore/faststore/packages/core
+> @faststore/core@3.100.3 prebuild /home/runner/work/faststore/faststore/packages/core
 > na run partytown && na run generate
 
 
-> @faststore/core@3.100.2 partytown /home/runner/work/faststore/faststore/packages/core
+> @faststore/core@3.100.3 partytown /home/runner/work/faststore/faststore/packages/core
 > partytown copylib ./public/~partytown
 
 Partytown lib copied to: /home/runner/work/faststore/faststore/packages/core/public/~partytown
 
-> @faststore/core@3.100.2 generate /home/runner/work/faststore/faststore/packages/core
+> @faststore/core@3.100.3 generate /home/runner/work/faststore/faststore/packages/core
 > na run generate:schema && na run generate:codegen && na run format:generated
 
 
-> @faststore/core@3.100.2 generate:schema /home/runner/work/faststore/faststore/packages/core
+> @faststore/core@3.100.3 generate:schema /home/runner/work/faststore/faststore/packages/core
 > tsx src/server/generator/generateGraphQLSchemaFile.ts
 
 Schema GraphQL file generated successfully
 
-> @faststore/core@3.100.2 generate:codegen /home/runner/work/faststore/faststore/packages/core
+> @faststore/core@3.100.3 generate:codegen /home/runner/work/faststore/faststore/packages/core
 > graphql-codegen
 
 [STARTED] Parse Configuration
@@ -37,11 +37,11 @@ Running lifecycle hook "afterStart" scripts...
 [CLI] Loading Documents
 [CLI] Generating output
 
-> @faststore/core@3.100.2 format:generated /home/runner/work/faststore/faststore/packages/core
+> @faststore/core@3.100.3 format:generated /home/runner/work/faststore/faststore/packages/core
 > prettier --write "@generated/**/*.{ts,js,tsx,jsx,json}" --loglevel error
 
 
-> @faststore/core@3.100.2 build /home/runner/work/faststore/faststore/packages/core
+> @faststore/core@3.100.3 build /home/runner/work/faststore/faststore/packages/core
 > next build
 
 ⚠ No build cache found. Please configure build caching for faster rebuilds. Read more: https://nextjs.org/docs/messages/no-cache
@@ -58,7 +58,7 @@ Browserslist: caniuse-lite is outdated. Please run:
    Creating an optimized production build ...
    Disabled SWC as replacement for Babel because of custom Babel configuration ".babelrc.js" https://nextjs.org/docs/messages/swc-disabled
    Using external babel configuration from /home/runner/work/faststore/faststore/packages/core/.babelrc.js
-Browserslist: browsers data (caniuse-lite) is 17 months old. Please run:
+Browserslist: browsers data (caniuse-lite) is 19 months old. Please run:
   npx update-browserslist-db@latest
   Why you should do it regularly: https://github.com/browserslist/update-db#readme
 <w> [webpack.cache.PackFileCacheStrategy] Skipped not serializable cache item 'Compilation/modules|/home/runner/work/faststore/faststore/node_modules/.pnpm/next@13.5.11_@babel+core@7.26.7_@opentelemetry+api@1.4.1_react-dom@18.3.1_react@18.3.1__react@18.3.1_sass@1.83.4/node_modules/next/dist/build/webpack/loaders/css-loader/src/index.js??ruleSet[1].rules[7].oneOf[9].use[1]!/home/runner/work/faststore/faststore/node_modules/.pnpm/next@13.5.11_@babel+core@7.26.7_@opentelemetry+api@1.4.1_react-dom@18.3.1_react@18.3.1__react@18.3.1_sass@1.83.4/node_modules/next/dist/build/webpack/loaders/postcss-loader/src/index.js??ruleSet[1].rules[7].oneOf[9].use[2]!/home/runner/work/faststore/faststore/node_modules/.pnpm/next@13.5.11_@babel+core@7.26.7_@opentelemetry+api@1.4.1_react-dom@18.3.1_react@18.3.1__react@18.3.1_sass@1.83.4/node_modules/next/dist/build/webpack/loaders/resolve-url-loader/index.js??ruleSet[1].rules[7].oneOf[9].use[3]!/home/runner/work/faststore/faststore/node_modules/.pnpm/next@13.5.11_@babel+core@7.26.7_@opentelemetry+api@1.4.1_react-dom@18.3.1_react@18.3.1__react@18.3.1_sass@1.83.4/node_modules/next/dist/compiled/sass-loader/cjs.js??ruleSet[1].rules[7].oneOf[9].use[4]!/home/runner/work/faststore/faststore/packages/core/src/components/common/Toast/section.module.scss': No serializer registered for Warning
@@ -83,7 +83,7 @@ Learn more: https://nextjs.org/docs/api-reference/edge-runtime
 
 Import trace for requested module:
 ../../node_modules/.pnpm/jose@6.2.3/node_modules/jose/dist/webapi/lib/deflate.js
-../../node_modules/.pnpm/jose@6.2.3/node_modules/jose/dist/webapi/jwe/flattened/decrypt.js
+../../node_modules/.pnpm/jose@6.2.3/node_modules/jose/dist/webapi/jwe/flattened/encrypt.js
 ../../node_modules/.pnpm/jose@6.2.3/node_modules/jose/dist/webapi/index.js
 ./src/server/password-protection-service.ts
 
@@ -93,7 +93,7 @@ Learn more: https://nextjs.org/docs/api-reference/edge-runtime
 
 Import trace for requested module:
 ../../node_modules/.pnpm/jose@6.2.3/node_modules/jose/dist/webapi/lib/deflate.js
-../../node_modules/.pnpm/jose@6.2.3/node_modules/jose/dist/webapi/jwe/flattened/decrypt.js
+../../node_modules/.pnpm/jose@6.2.3/node_modules/jose/dist/webapi/jwe/flattened/encrypt.js
 ../../node_modules/.pnpm/jose@6.2.3/node_modules/jose/dist/webapi/index.js
 ./src/server/password-protection-service.ts
 
@@ -101,8 +101,8 @@ Import trace for requested module:
    Generating static pages (0/8) ...
 
   Generating static pages (2/8) 
 Search not found. Add a new component for this section or remove it from the CMS
-Warning: Dynamic Content not found for the page: home. Refer to the Dynamic Content documentation at https://developers.vtex.com/docs/guides/faststore/dynamic-content-overview for mapping the page and the corresponding data-fetching function.
 Search not found. Add a new component for this section or remove it from the CMS
+Warning: Dynamic Content not found for the page: home. Refer to the Dynamic Content documentation at https://developers.vtex.com/docs/guides/faststore/dynamic-content-overview for mapping the page and the corresponding data-fetching function.
 
   Generating static pages (4/8) 
 Search not found. Add a new component for this section or remove it from the CMS
 Search not found. Add a new component for this section or remove it from the CMS
@@ -154,7 +154,7 @@ Route (pages)                              Size     First Load JS
   ├ chunks/framework-d514426edf885c68.js   45.4 kB
   ├ chunks/main-a2d1cfe626f2fef1.js        33.4 kB
   ├ chunks/pages/_app-35e8dda638ee2672.js  26.6 kB
-  ├ chunks/webpack-25c67512e1907efd.js     3.9 kB
+  ├ chunks/webpack-6a844a3f6a390f62.js     3.9 kB
   └ css/146a8678babc61bf.css               9.2 kB
 
 ƒ Middleware                               44.8 kB

package/.turbo/turbo-test.log

@@ -1,20 +1,21 @@
 
-> @faststore/core@3.100.2 test /home/runner/work/faststore/faststore/packages/core
+> @faststore/core@3.100.3 test /home/runner/work/faststore/faststore/packages/core
 > jest
 
-PASS test/utils/clearCookies.test.ts (9.746 s)
+PASS test/utils/clearCookies.test.ts (11.489 s)
+PASS test/utils/cookieCacheBusting.test.ts (13.647 s)
 PASS test/utils/multipleTemplates.test.ts
-PASS test/server/password-protection-service.test.ts (16.656 s)
-PASS test/utils/cookieCacheBusting.test.ts (15.853 s)
-PASS test/server/cms/global.test.ts
+PASS test/server/password-protection-service.test.ts (16.602 s)
+PASS test/sdk/redirects/index.test.ts
 PASS test/utils/validateSessionRefreshToken.test.ts
+PASS test/server/cms/global.test.ts
 PASS test/utils/isLocalHost.test.ts
-PASS test/server/cms/index.test.ts
 PASS test/utils/isLocalHost.browser.test.ts
-PASS test/server/index.test.ts (11.788 s)
+PASS test/server/cms/index.test.ts (5.293 s)
+PASS test/server/index.test.ts (10.491 s)
 
-Test Suites: 10 passed, 10 total
-Tests:       85 passed, 85 total
+Test Suites: 11 passed, 11 total
+Tests:       92 passed, 92 total
 Snapshots:   0 total
-Time:        27.32 s
+Time:        27.151 s
 Ran all test suites.

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [3.100.4](https://github.com/vtex/faststore/compare/v3.100.3...v3.100.4) (2026-08-28)
+
+### Bug Fixes
+
+- **core:** resolve redirect matcher module explicitly ([#3468](https://github.com/vtex/faststore/issues/3468)) ([fe3a3c7](https://github.com/vtex/faststore/commit/fe3a3c7c19237d53061be7b205838b626e225786)), closes [#3467](https://github.com/vtex/faststore/issues/3467) [#1450148](https://github.com/vtex/faststore/issues/1450148)
+
 ## [3.100.3](https://github.com/vtex/faststore/compare/v3.100.2...v3.100.3) (2026-06-30)
 
 ### Bug Fixes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@faststore/core",
-  "version": "3.100.3",
+  "version": "3.100.4",
   "license": "MIT",
   "repository": {
     "type": "git",
@@ -127,5 +127,5 @@
     "ts-jest": "29.1.1",
     "typescript": "5.3.2"
   },
-  "gitHead": "bf4812f3e3a53108f22ab987098b74f7eed9bfa3"
+  "gitHead": "b30af2d573626311c245c445860a45cb26478798"
 }

package/src/sdk/redirects/index.ts

@@ -1,5 +1,5 @@
 import storeConfig from 'discovery.config'
-import { matcher } from 'src/customizations/src/redirects'
+import { matcher } from 'src/customizations/src/redirects/index'
 
 type GetRedirectArgs = {
   pathname: string
@@ -19,6 +19,23 @@ const PERMANENT_STATUS = 308
 
 const ASSET_FILE_REGEX = /\.(js|css|png|jpg|jpeg|svg|gif|webp|ico|json|map)$/i
 
+let hasWarnedInvalidMatcher = false
+
+function resolveLocalMatch(pathname: string) {
+  if (typeof matcher === 'function') {
+    return matcher({ pathname })
+  }
+
+  if (!hasWarnedInvalidMatcher) {
+    hasWarnedInvalidMatcher = true
+    console.warn(
+      `[redirects] Expected src/redirects to export a \`matcher\` function, got ${typeof matcher}. Falling back to the platform redirect lookup.`
+    )
+  }
+
+  return null
+}
+
 export async function getRedirect({
   pathname,
 }: GetRedirectArgs): Promise<GetRedirectReturn> {
@@ -29,7 +46,8 @@ export async function getRedirect({
   }
 
   try {
-    const redirectMatch = matcher({ pathname })
+    const redirectMatch = resolveLocalMatch(pathname)
+
     if (redirectMatch) {
       return {
         destination: encodeURI(redirectMatch.destination),

package/test/sdk/redirects/index.test.ts

@@ -0,0 +1,134 @@
+const STORE_ID = 'teststore'
+const REWRITER_URL = `https://${STORE_ID}.myvtex.com/_v/public/redirect-evaluate`
+
+const MATCHER_MODULE = 'src/customizations/src/redirects/index'
+
+type GetRedirect = typeof import('../../../src/sdk/redirects').getRedirect
+
+async function loadGetRedirect(matcherExport?: unknown): Promise<GetRedirect> {
+  jest.resetModules()
+
+  jest.doMock('discovery.config', () => ({
+    __esModule: true,
+    default: { api: { storeId: STORE_ID } },
+  }))
+
+  if (matcherExport === undefined) {
+    jest.dontMock(MATCHER_MODULE)
+  } else {
+    jest.doMock(MATCHER_MODULE, () => ({
+      __esModule: true,
+      matcher: matcherExport,
+    }))
+  }
+
+  const { getRedirect } = await import('../../../src/sdk/redirects')
+
+  return getRedirect
+}
+
+function mockRewriter({ ok, body }: { ok: boolean; body?: unknown }) {
+  const fetchMock = jest
+    .fn()
+    .mockResolvedValue({ ok, json: async () => body } as Response)
+
+  globalThis.fetch = fetchMock as unknown as typeof fetch
+
+  return fetchMock
+}
+
+describe('getRedirect', () => {
+  const originalFetch = globalThis.fetch
+
+  beforeEach(() => {
+    jest.spyOn(console, 'error').mockImplementation(() => undefined)
+    jest.spyOn(console, 'warn').mockImplementation(() => undefined)
+  })
+
+  afterEach(() => {
+    globalThis.fetch = originalFetch
+    jest.dontMock('discovery.config')
+    jest.dontMock(MATCHER_MODULE)
+    jest.restoreAllMocks()
+    jest.resetModules()
+  })
+
+  it('skips asset paths without querying the rewriter', async () => {
+    const fetchMock = mockRewriter({ ok: false })
+    const getRedirect = await loadGetRedirect()
+
+    await expect(getRedirect({ pathname: '/icons.json' })).resolves.toBeNull()
+    expect(fetchMock).not.toHaveBeenCalled()
+  })
+
+  it('returns a custom matcher result without querying the rewriter', async () => {
+    const fetchMock = mockRewriter({ ok: false })
+    const getRedirect = await loadGetRedirect(() => ({
+      destination: '/promoção',
+      permanent: false,
+    }))
+
+    await expect(getRedirect({ pathname: '/sale' })).resolves.toEqual({
+      destination: encodeURI('/promoção'),
+      permanent: false,
+    })
+    expect(fetchMock).not.toHaveBeenCalled()
+    expect(console.warn).not.toHaveBeenCalled()
+  })
+
+  it('falls back to the rewriter when the matcher export is not a function', async () => {
+    const fetchMock = mockRewriter({
+      ok: true,
+      body: { location: '/eletrodomesticos', status: 308 },
+    })
+    // A store shipping `src/redirects.json` can shadow the customization
+    // module, which makes the imported `matcher` an array instead of a function.
+    const getRedirect = await loadGetRedirect([])
+
+    await expect(getRedirect({ pathname: '/old-page' })).resolves.toEqual({
+      destination: '/eletrodomesticos',
+      permanent: true,
+    })
+    expect(fetchMock).toHaveBeenCalledWith(`${REWRITER_URL}/old-page`)
+    expect(console.warn).toHaveBeenCalledTimes(1)
+  })
+
+  it('warns only once about an invalid matcher export', async () => {
+    mockRewriter({ ok: false })
+    const getRedirect = await loadGetRedirect([])
+
+    await getRedirect({ pathname: '/first' })
+    await getRedirect({ pathname: '/second' })
+
+    expect(console.warn).toHaveBeenCalledTimes(1)
+  })
+
+  it('does not leak a mocked matcher into a later default load', async () => {
+    mockRewriter({ ok: false })
+
+    const withInvalidMatcher = await loadGetRedirect([])
+    await expect(withInvalidMatcher({ pathname: '/first' })).resolves.toBeNull()
+
+    const withStubMatcher = await loadGetRedirect()
+    await expect(withStubMatcher({ pathname: '/second' })).resolves.toBeNull()
+
+    expect(console.warn).toHaveBeenCalledTimes(1)
+  })
+
+  it('marks non-permanent rewriter redirects as temporary', async () => {
+    mockRewriter({ ok: true, body: { location: '/new', status: 307 } })
+    const getRedirect = await loadGetRedirect()
+
+    await expect(getRedirect({ pathname: '/old' })).resolves.toEqual({
+      destination: '/new',
+      permanent: false,
+    })
+  })
+
+  it('returns null when the rewriter has no rule for the path', async () => {
+    mockRewriter({ ok: false })
+    const getRedirect = await loadGetRedirect()
+
+    await expect(getRedirect({ pathname: '/no-rule' })).resolves.toBeNull()
+  })
+})