mirror of
https://github.com/AutoMaker-Org/automaker.git
synced 2026-03-16 21:53:07 +00:00
The setup-project action was force-installing Linux-specific npm binaries (@rollup/rollup-linux-x64-gnu, @tailwindcss/oxide-linux-x64-gnu) on ALL platforms including macOS and Windows. This overwrote the correct platform-native binaries, causing Vite builds to fail on those runners, which prevented any release assets from being uploaded. Also removes the redundant `draft == false` condition from the upload job (already guaranteed by `types: [published]` trigger) and adds an explicit checkout step to the upload job for correctness. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
74 lines
2.6 KiB
YAML
74 lines
2.6 KiB
YAML
name: 'Setup Project'
|
|
description: 'Common setup steps for CI workflows - checkout, Node.js, dependencies, and native modules'
|
|
|
|
inputs:
|
|
node-version:
|
|
description: 'Node.js version to use'
|
|
required: false
|
|
default: '22'
|
|
check-lockfile:
|
|
description: 'Run lockfile lint check for SSH URLs'
|
|
required: false
|
|
default: 'false'
|
|
rebuild-node-pty-path:
|
|
description: 'Working directory for node-pty rebuild (empty = root)'
|
|
required: false
|
|
default: ''
|
|
|
|
runs:
|
|
using: 'composite'
|
|
steps:
|
|
- name: Setup Node.js
|
|
uses: actions/setup-node@v4
|
|
with:
|
|
node-version: ${{ inputs.node-version }}
|
|
cache: 'npm'
|
|
cache-dependency-path: package-lock.json
|
|
|
|
- name: Check for SSH URLs in lockfile
|
|
if: inputs.check-lockfile == 'true'
|
|
shell: bash
|
|
run: npm run lint:lockfile
|
|
|
|
- name: Configure Git for HTTPS
|
|
shell: bash
|
|
# Convert SSH URLs to HTTPS for git dependencies (e.g., @electron/node-gyp)
|
|
# This is needed because SSH authentication isn't available in CI
|
|
run: git config --global url."https://github.com/".insteadOf "git@github.com:"
|
|
|
|
- name: Install dependencies
|
|
shell: bash
|
|
# Use npm install instead of npm ci to correctly resolve platform-specific
|
|
# optional dependencies (e.g., @tailwindcss/oxide, lightningcss binaries)
|
|
# Skip scripts to avoid electron-builder install-app-deps which uses too much memory
|
|
# Use --force to allow platform-specific dev dependencies like dmg-license on non-darwin platforms
|
|
run: npm install --ignore-scripts --force
|
|
|
|
- name: Install Linux native bindings
|
|
if: runner.os == 'Linux'
|
|
shell: bash
|
|
# Workaround for npm optional dependencies bug (npm/cli#4828)
|
|
# Explicitly install Linux bindings needed for build tools
|
|
run: |
|
|
npm install --no-save --force --ignore-scripts \
|
|
@rollup/rollup-linux-x64-gnu@4.53.3 \
|
|
@tailwindcss/oxide-linux-x64-gnu@4.1.17
|
|
|
|
- name: Build shared packages
|
|
shell: bash
|
|
# Build shared packages (types, utils, platform, etc.) before apps can use them
|
|
run: npm run build:packages
|
|
|
|
- name: Rebuild native modules (root)
|
|
if: inputs.rebuild-node-pty-path == ''
|
|
shell: bash
|
|
# Rebuild node-pty and other native modules for Electron
|
|
run: npm rebuild node-pty
|
|
|
|
- name: Rebuild native modules (workspace)
|
|
if: inputs.rebuild-node-pty-path != ''
|
|
shell: bash
|
|
# Rebuild node-pty and other native modules needed for server
|
|
run: npm rebuild node-pty
|
|
working-directory: ${{ inputs.rebuild-node-pty-path }}
|