From 8c00a05b55b979a082872a1c7538bb16dc4fe944 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jean-Micha=C3=ABl=20Celerier?= Date: Fri, 9 Oct 2026 18:44:06 -0400 Subject: [PATCH] ci: name packaging artifacts after the tool, not "app" Every Suite SAT tool now follows one packaging convention: artifacts are named - (and -pro- where there is a pro edition), and the release job's download pattern is -*, which must match every edition's upload. spm diverged by using the generic app-* for both, which is a pattern that would silently collect any other artifact whose name starts with "app-" -- the same class of mistake that once put a basic Domeport build into a release labelled Pro. A tool-specific prefix makes that impossible. The wasm row also carried the whole upload path (artifact-path) rather than just its file glob. Overriding the path does not compose with a second edition -- one override would point both uploads at the same directory -- so the matrix now supplies artifact-file and each upload keeps its own output directory. spm has one edition today; the shape is shared so it stays correct if a pro build is added. Co-Authored-By: Claude Opus 5 Claude-Session: https://claude.ai/code/session_019QYyvhRXvSMSRQPaq12Teq --- .github/workflows/build.yml | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 363fcc3..aabd20b 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -38,7 +38,7 @@ jobs: - os: ubuntu-24.04 platform: wasm artifact-name: wasm - artifact-path: packages/*.zip + artifact-file: '*.zip' target-platform: wasm steps: @@ -92,8 +92,8 @@ jobs: - name: Upload packages uses: actions/upload-artifact@v7 with: - name: app-${{ matrix.artifact-name }} - path: ${{ matrix.artifact-path || 'packages/' }} + name: spm-${{ matrix.artifact-name }} + path: packages/${{ matrix.artifact-file || '' }} retention-days: 30 release: @@ -108,7 +108,7 @@ jobs: uses: actions/download-artifact@v8 with: path: all-packages - pattern: app-* + pattern: spm-* merge-multiple: true - name: Create Release (on tags)