revert: restore matrix testing strategy for Node 20 and 22

Reverted CI workflow changes to restore matrix testing across Node 20
and 22 as requested by maintainer. Strapi plugins must support multiple
Node versions.

Changes:
- Restored matrix strategy in lint and test jobs
- Removed Cypress retries configuration
- Removed scrollIntoView workaround in navigateToInterface command
- Removed || true fallbacks from yalc scripts

Per maintainer feedback, the matrix testing approach is required for
proper multi-version support. Test workarounds have been removed to
return to the original testing approach.

Author: MSACC

.github/workflows/tests.yml

@@ -14,11 +14,14 @@ jobs:
   lint:
     name: 'lint'
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        node: [20, 22]
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
-          node-version-file: '.nvmrc'
+          node-version: ${{ matrix.node }}
           cache: 'yarn'
       - name: Install dependencies
         run: yarn --frozen-lockfile
@@ -28,36 +31,31 @@ jobs:
     name: 'test'
     needs: [lint]
     runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        node: [20, 22]
     steps:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
-          node-version-file: '.nvmrc'
+          node-version: ${{ matrix.node }}
           cache: 'yarn'
       - name: Install dependencies plugin
         run: yarn --no-lockfile --unsafe-perm
-      - name: Build plugin
-        run: yarn build
-      - name: Link plugin to playground
+      - name: Build and pack plugin
         run: |
-          # Try yalc first
-          yarn playground:yalc-add || true
-
-          # Check if yalc succeeded
-          if [ ! -d "playground/.yalc/strapi-plugin-config-sync" ]; then
-            echo "Yalc failed, installing from tarball as fallback"
-            # Create tarball and install it
-            yarn pack --filename plugin.tgz
-            cd playground
-            # Remove yalc link from package.json and install from tarball
-            npm pkg delete dependencies.strapi-plugin-config-sync
-            yarn add ../plugin.tgz
-          else
-            cd playground
-            yarn install --unsafe-perm
-          fi
+          yarn build
+          yarn pack --filename plugin.tgz
+      - name: Install plugin in playground
+        run: |
+          cd playground
+          # Remove yalc link, install from tarball
+          sed -i 's|"strapi-plugin-config-sync": "link:.yalc/strapi-plugin-config-sync"|"strapi-plugin-config-sync": "file:../plugin.tgz"|' package.json
+          # Clean install everything
+          rm -rf node_modules yarn.lock
+          yarn install --unsafe-perm
       - name: Build playground
-        run: yarn playground:build
+        run: cd playground && yarn build
       # - name: Run unit tests
       #   run: yarn test:unit
       - name: Run integration tests

cypress.config.js

@@ -8,10 +8,6 @@ module.exports = defineConfig({
     video: true,
     defaultCommandTimeout: 30000,
     requestTimeout: 30000,
-    retries: {
-      runMode: 2,
-      openMode: 0,
-    },
     setupNodeEvents(on, config) {
       // implement node event listeners here.
       // eslint-disable-next-line global-require

cypress/support/commands.js

@@ -73,8 +73,6 @@ Cypress.Commands.add('navigateToInterface', (path) => {
     url: '/config-sync/diff',
   }).as('getConfigDiff');
 
-  // Scroll settings link into view to handle position:fixed overflow issues
-  cy.get('a[href="/admin/settings"]').scrollIntoView();
   cy.get('a[href="/admin/settings"]').should('be.visible').click();
   cy.get('a[href="/admin/settings/config-sync"]').should('be.visible').click();
 

package.json

@@ -31,15 +31,15 @@
   "scripts": {
     "develop": "strapi-plugin watch:link",
     "watch": "pack-up watch",
-    "build": "pack-up build && (yalc push --publish || true)",
+    "build": "pack-up build",
     "eslint": "eslint --max-warnings=0 './**/*.{js,jsx}'",
     "eslint:fix": "eslint --fix './**/*.{js,jsx}'",
     "test:unit": "jest --verbose",
     "test:integration": "cd playground && node_modules/.bin/jest --verbose  --forceExit --detectOpenHandles",
     "test:e2e": "cypress open",
-    "playground:install": "(yarn playground:yalc-add-link || true) && cd playground && yarn install",
-    "playground:yalc-add": "cd playground && (yalc add strapi-plugin-config-sync || true)",
-    "playground:yalc-add-link": "cd playground && (yalc add --link strapi-plugin-config-sync || true)",
+    "playground:install": "yarn playground:yalc-add-link && cd playground && yarn install",
+    "playground:yalc-add": "cd playground && yalc add strapi-plugin-config-sync",
+    "playground:yalc-add-link": "cd playground && yalc add --link strapi-plugin-config-sync",
     "playground:build": "cd playground && yarn build",
     "playground:develop": "cd playground && yarn develop",
     "playground:start": "cd playground && yarn start"