Fix CI

Adapt JS CI in SDK tests
Merge pull request #6041 from meilisearch/fix-workflow-injection
2025-12-13 07:57:02 +00:00 · 2025-12-11 15:25:41 +01:00 · 2025-12-11 15:20:32 +01:00 · 2025-12-09 17:04:58 +00:00 · 2025-12-09 17:06:41 +01:00
1 changed files with 19 additions and 13 deletions
--- a/.github/workflows/sdks-tests.yml
+++ b/.github/workflows/sdks-tests.yml
@@ -25,14 +25,18 @@ jobs:
      - uses: actions/checkout@v5
      - name: Define the Docker image we need to use
        id: define-image
+        env:
+          EVENT_NAME: ${{ github.event_name }}
+          DOCKER_IMAGE_INPUT: ${{ github.event.inputs.docker_image }}
        run: |
-          event=${{ github.event_name }}
          echo "docker-image=nightly" >> $GITHUB_OUTPUT
-          if [[ $event == 'workflow_dispatch' ]]; then
-            echo "docker-image=${{ github.event.inputs.docker_image }}" >> $GITHUB_OUTPUT
+          if [[ "$EVENT_NAME" == 'workflow_dispatch' ]]; then
+            echo "docker-image=$DOCKER_IMAGE_INPUT" >> $GITHUB_OUTPUT
          fi
      - name: Docker image is ${{ steps.define-image.outputs.docker-image }}
-        run: echo "Docker image is ${{ steps.define-image.outputs.docker-image }}"
+        env:
+          DOCKER_IMAGE: ${{ steps.define-image.outputs.docker-image }}
+        run: echo "Docker image is $DOCKER_IMAGE"

 ##########
 ## SDKs ##
@@ -159,24 +163,26 @@ jobs:
      - uses: actions/checkout@v5
        with:
          repository: meilisearch/meilisearch-js
+      - name: Install pnpm
+        uses: pnpm/action-setup@v4
      - name: Setup node
-        uses: actions/setup-node@v5
+        uses: actions/setup-node@v6
        with:
-          cache: 'yarn'
+          cache: 'pnpm'
      - name: Install dependencies
-        run: yarn --dev
+        run: pnpm install
      - name: Run tests
-        run: yarn test
+        run: pnpm test
      - name: Build project
-        run: yarn build
+        run: pnpm build
      - name: Run ESM env
-        run: yarn test:env:esm
+        run: pnpm test:env:esm
      - name: Run Node.js env
-        run: yarn test:env:nodejs
+        run: pnpm test:env:nodejs
      - name: Run node typescript env
-        run: yarn test:env:node-ts
+        run: pnpm test:env:node-ts
      - name: Run Browser env
-        run: yarn test:env:browser
+        run: pnpm test:env:browser

  meilisearch-php-tests:
    needs: define-docker-image
Author	SHA1	Message	Date
curquiza	367a3cdac9	Fix CI	2025-12-11 15:25:41 +01:00
curquiza	11dd0fd432	Adapt JS CI in SDK tests	2025-12-11 15:20:32 +01:00
Clément Renault	26e368b116	Merge pull request #6041 from meilisearch/fix-workflow-injection Remove risk of command injection	2025-12-09 17:04:58 +00:00
curquiza	ba95ac0915	Remove risk of command injection	2025-12-09 17:06:41 +01:00