Skip to content

Commit 091e35d

Browse files
committed
fix: add web build extraction and cleanup for Player web app in CI workflow
1 parent a665a86 commit 091e35d

1 file changed

Lines changed: 19 additions & 17 deletions

File tree

‎.github/workflows/agent.yml‎

Lines changed: 19 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -121,6 +121,14 @@ jobs:
121121
if (!asset) throw new Error(`Asset ${assetName} not found`);
122122
core.setOutput('asset_url', asset.browser_download_url);
123123
124+
const webZipName = `PhiZone.Player_${version}.zip`;
125+
const webZipAsset = release.assets.find(a => a.name === webZipName);
126+
if (webZipAsset) {
127+
core.setOutput('web_zip_url', webZipAsset.browser_download_url);
128+
} else {
129+
core.warning(`Web build ZIP ${webZipName} not found in release`);
130+
}
131+
124132
- name: Download Player executable
125133
run: |
126134
curl -L "${{ steps.get_url.outputs.asset_url }}" -o player
@@ -303,27 +311,21 @@ jobs:
303311
with:
304312
node-version: 22
305313

306-
- name: Setup PNPM
307-
if: ${{ github.event.inputs.useBrowser == 'true' }}
308-
uses: pnpm/action-setup@v4
309-
with:
310-
version: 10
311-
312-
- name: Clone and build Player web app
313-
if: ${{ github.event.inputs.useBrowser == 'true' }}
314+
- name: Download and extract Player web build
315+
if: ${{ github.event.inputs.useBrowser == 'true' && steps.get_url.outputs.web_zip_url }}
314316
run: |
315317
set -e
316-
git clone --depth 1 https://github.com/PhiZone/player.git player-app
317-
cd player-app
318-
pnpm install --frozen-lockfile
319-
pnpm build:only
320-
echo "Player web app built successfully."
318+
curl -L "${{ steps.get_url.outputs.web_zip_url }}" -o player-web.zip
319+
mkdir -p ./player-web
320+
unzip -q player-web.zip -d ./player-web
321+
rm player-web.zip
322+
echo "Player web build extracted successfully."
321323
322324
- name: Start local web server
323-
if: ${{ github.event.inputs.useBrowser == 'true' }}
325+
if: ${{ github.event.inputs.useBrowser == 'true' && steps.get_url.outputs.web_zip_url }}
324326
run: |
325327
set -e
326-
node serve.mjs ./player-app/build 3000 &
328+
node serve.mjs ./player-web 3000 &
327329
echo $! > ./server_pid
328330
# Wait for server to be ready
329331
for i in $(seq 1 30); do
@@ -550,8 +552,8 @@ jobs:
550552
rm -f ./server_pid
551553
fi
552554
553-
# Clean up cloned Player web app
554-
rm -rf ./player-app 2>/dev/null || true
555+
# Clean up Player web build
556+
rm -rf ./player-web 2>/dev/null || true
555557
556558
# Clean up downloaded files
557559
echo "Cleaning downloaded files..."

0 commit comments

Comments
 (0)