From 33651acdd9aeb09a00aff36a6816005a9007e657 Mon Sep 17 00:00:00 2001
From: Emanuele Bartolesi <bartolesiemanuele@hotmail.com>
Date: Mon, 8 Sep 2025 17:05:35 +0200
Subject: [PATCH] feat: update GitHub Actions workflow to publish VS Code
 extension on merged PRs

- Changed trigger from push to pull_request for the main branch.
- Added condition to ensure the job runs only on merged PRs.
- Updated npm install command from `npm ci` to `npm install`.
- Added step to upload VSIX artifact after packaging.
- Adjusted environment variable for VSCE token to use VSCE_PAT.
---
 .github/workflows/publish-extension.yml | 23 +++++++++++++++++++----
 1 file changed, 19 insertions(+), 4 deletions(-)

diff --git a/.github/workflows/publish-extension.yml b/.github/workflows/publish-extension.yml
index 5d6eabe..b3f0b7e 100644
--- a/.github/workflows/publish-extension.yml
+++ b/.github/workflows/publish-extension.yml
@@ -1,12 +1,17 @@
 name: Publish VS Code Extension
 
 on:
-  push:
+  # Publish only when a PR into main is merged
+  pull_request:
     branches:
       - main
+    types:
+      - closed
 
 jobs:
   publish:
+    # Ensure we only run on merged PRs (not just closed)
+    if: github.event.pull_request.merged == true
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
@@ -18,7 +23,7 @@ jobs:
           node-version: '20'
 
       - name: Install dependencies
-        run: npm ci
+        run: npm install
 
       - name: Install vsce
         run: npm install -g vsce
@@ -26,7 +31,17 @@ jobs:
       - name: Package extension
         run: vsce package
 
+      - name: Upload VSIX artifact (optional)
+        uses: actions/upload-artifact@v4
+        with:
+          name: vsix
+          path: '*.vsix'
+          if-no-files-found: warn
+
       - name: Publish to Marketplace
-        run: vsce publish --pat ${{ secrets.VSCE_TOKEN }}
         env:
-          VSCE_TOKEN: ${{ secrets.VSCE_TOKEN }}
\ No newline at end of file
+          # Map your repo secret to the env var that vsce expects
+          VSCE_PAT: ${{ secrets.VSCE_TOKEN }}
+        run: |
+          # vsce reads token from VSCE_PAT; no need to pass --pat explicitly
+          vsce publish
\ No newline at end of file