Skip to content

Conversation

@fkloes
Copy link
Collaborator

@fkloes fkloes commented Nov 4, 2025

Description

This pull requests updates all Babel-related dependencies to the latest stable versions for improved compatibility, performance, and tooling support.

Additionally it updates Jest to the latest version that's compatible with the current Shopgate infrastructure.

Type of change

  • Bug Fix 🐛 (non-breaking change which fixes an issue)
  • Enhancement 🚀 (non-breaking change which adds functionality)
  • Breaking Change 💥 (fix or feature that would cause existing functionality to not work as expected)
  • Polish 💅 (Just some cleanups)
  • Internal 🏠 Only relates to internal processes.

How to test it

Since the Jest update to version 29 introduced a new snapshot format, every snapshot had to be updated. I would recommend to filter snapshots from the changed files.

@fkloes fkloes self-assigned this Nov 4, 2025
@fkloes fkloes added the internal Internal changes label Nov 4, 2025
@AylinUenal AylinUenal self-requested a review November 5, 2025 09:16
@fkloes fkloes changed the title Update Babel Update Babel and Jest Nov 10, 2025
@fkloes fkloes marked this pull request as ready for review November 10, 2025 08:04
# Conflicts:
#	extensions/@shopgate-product-reviews/frontend/package.json
#	libraries/tracking-core/package.json
@AylinUenal AylinUenal requested a review from Copilot November 14, 2025 14:52
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates Babel and Jest dependencies to their latest stable versions to improve compatibility, performance, and tooling support. The update includes migrating from Jest 24.9.0 to 29.7.0, which introduced a new snapshot format requiring all snapshots to be regenerated.

Key changes:

  • Upgraded Babel packages from v7.5.x to v7.28.x
  • Updated Jest from v24.9.0 to v29.7.0
  • Modernized Babel configuration with improved preset options
  • Removed obsolete Babel plugins and consolidated dependencies

Reviewed Changes

Copilot reviewed 241 out of 310 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
themes/theme-gmd/package.json Updated Babel and Jest dependencies to latest versions
themes/theme-gmd/babel.config.js Modernized Babel configuration with new presets and removed obsolete plugins
package.json Updated root-level Babel and Jest dependencies
themes/theme-gmd/.browserslistrc Added browser support targets for Babel compilation
themes/theme-gmd/jest.config.js Updated CSS module mock configuration
Snapshot files Regenerated snapshots due to Jest 29 format changes
Test files Updated mock function calls from require.requireActual to jest.requireActual
Files not reviewed (1)
  • themes/theme-gmd/package-lock.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Base automatically changed from CURB-5489-update-webpack to 2025-tooling-update November 20, 2025 14:53
# Conflicts:
#	extensions/@shopgate-product-reviews/frontend/package.json
#	libraries/tracking-core/package.json
* Creates a mocked document with iframes.
* @param {Array} srcs A list of video URLs.
* @return {Object}
* Creates a mocked document with iframes (parity with your original helper).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove "(parity with your original helper)", it is clearly an AI helper comment

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

const options = { trackerName: 'mock' };
const expectedObject = { expected: 'data' };

// initial call as in original test
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove comment. original test is gone now

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

# Conflicts:
#	extensions/@shopgate-product-reviews/frontend/package.json
#	libraries/tracking-core/package.json
@AylinUenal
Copy link
Contributor

AylinUenal commented Dec 10, 2025

I get this error when deleting or renaming a custom favorites list, can be reproduced every time:
Screenshot 2025-12-10 at 15 13 31

@AylinUenal
Copy link
Contributor

when i enter the PDP of a product with variants, often the variant picker doesn't show up and I get this error (tested with shop 31331 on iphone 15 plus iOS 17):
Screenshot 2025-12-16 at 16 04 01

@fkloes
Copy link
Collaborator Author

fkloes commented Jan 2, 2026

Comment 1:
I'll prepare an additional ticket to fix the issue in the favourites list. I don't think that it's cause by the babel update, but by the webpack update.

Comment 2:
I can't reproduce the issue, but i think that's caused by invalid code when we spread inside arrays. Since all browsers we support can handle spread syntax natively, i removed some babel plugins. Maybe we should just add the plugins again to the babel.config.js files. Let's talk about that when you're back from vacation.

[
      ['@babel/plugin-transform-spread', {
        loose: true,
      }],
      ['@babel/plugin-transform-classes', {
        loose: true,
      }],
]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

internal Internal changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants