mirror of
https://github.com/stablyai/orca.git
synced 2026-09-22 08:02:28 +00:00
* chore(mobile): pin mobile page source to LF so a Windows checkout keeps buildId The Phase C web bundle hashes every text byte under mobile/src and mobile/app into its asset digests and from there into buildId. There is no global text=auto, so a CRLF checkout on Windows would give the Windows release a different buildId for identical source, the same failure the src/mobile-web pin above exists for. All 1924 tracked files in those directories are already LF in the index, so the pin renormalises nothing. mobile/web-entry does not exist yet; the pin is forward-looking for the Phase C entry point. Claude-Session: https://claude.ai/code/session_01JNnE9qzUZMMnqpZWCqM3nb * ci: install mobile dependencies in every desktop packaging job Ten workflows reach build:release/build:desktop and none of them installs mobile/node_modules. Root has no react-native, react-native-web or expo, so once the mobile web bundle builds from mobile/ its packaging jobs would fail at electron-builder's beforePack with an unresolvable import. Extract the frozen mobile install that pr.yml's static analysis job already ran inline into .github/actions/install-mobile-dependencies, and invoke it from every job the packaging census enumerates, after the root install and before the build. Same --frozen-lockfile, same lockfile-drift guard, and still no --ignore-scripts: mobile's postinstall generates the gitignored webview engine modules that tracked source imports. pr.yml now uses the action too, so there is one definition. Where a packaging job's setup-node caches the pnpm store, mobile/pnpm-lock.yaml joins cache-dependency-path so a mobile lockfile change invalidates it. Two jobs (daemon-relocation-spike, win-update-survival-e2e) do not cache at all and are left alone. The census test grows a per-job assertion that the action is present, so a new packaging job has to add the install deliberately rather than discover it at beforePack. release-cut's composite-action restore is no longer Windows-only: every platform consumes this action now, so any of them can be the leg whose cut ref predates it. No job builds anything different; this only makes mobile/node_modules present. Claude-Session: https://claude.ai/code/session_01JNnE9qzUZMMnqpZWCqM3nb * test(ci): assert the mobile install contract on the shared action pr.yml's static analysis job no longer carries the install inline, so the scope test's findIndex by step name resolved to -1. Match the step by the action it uses, and read working-directory and --frozen-lockfile off the action itself so the job cannot keep the step while the action stops installing anything. Claude-Session: https://claude.ai/code/session_01JNnE9qzUZMMnqpZWCqM3nb * chore(mobile): exempt binary asset types from the mobile LF pin /mobile/{src,app,web-entry}/** text eol=lf would mark a future PNG or font as text and rewrite its bytes on a Windows checkout. Exempt the asset types an RN page carries, the same way src/mobile-web exempts its PNG. -text after text eol=lf wins: probed a CRLF-bearing .png under the pin, it stays i/crlf attr/-text while a sibling .ts still normalises to i/lf. No tracked file changes classification; the 1924 files under mobile/src and mobile/app stay i/lf. Claude-Session: https://claude.ai/code/session_01JNnE9qzUZMMnqpZWCqM3nb * ci: gate the mobile install with the build it feeds in the cached lanes win-crash-survival, win-update-survival and daemon-relocation-spike all skip electron-builder on an installer/unpacked cache hit, so an unconditional mobile install spent time on node_modules nothing then consumed. Move each `uses:` below its cache step and carry the same cache-hit condition as the build. Claude-Session: https://claude.ai/code/session_01JNnE9qzUZMMnqpZWCqM3nb
123 lines
4.4 KiB
YAML
123 lines
4.4 KiB
YAML
name: Windows Update-Survival E2E (branch build)
|
|
|
|
# Why: proves the Phase 1 daemon-host relocation actually makes terminal
|
|
# sessions SURVIVE a Windows update. Builds an (unsigned) installer FROM THIS
|
|
# BRANCH, then runs the win-update-e2e harness with --expect survival: install,
|
|
# open a terminal, silently update over it, relaunch, and assert the daemon PID
|
|
# is unchanged and the pre-update terminal is still interactive with no console
|
|
# flashing. Runs from the feature branch via push (workflow_dispatch only works
|
|
# on the default branch); main is never touched. A CI runner is the only safe
|
|
# place to install/update — see the relocation post-mortem.
|
|
|
|
on:
|
|
push:
|
|
branches:
|
|
- Jinwoo-H/windows-update-survival
|
|
paths:
|
|
- 'src/main/daemon/**'
|
|
- 'src/main/pty/**'
|
|
- 'tests/tools/win-update-e2e/**'
|
|
- 'config/electron-builder.config.cjs'
|
|
- '.github/workflows/win-update-survival-e2e.yml'
|
|
workflow_dispatch:
|
|
inputs:
|
|
expect:
|
|
description: Expected outcome profile
|
|
required: true
|
|
type: choice
|
|
default: survival
|
|
options:
|
|
- survival
|
|
- cold-restore
|
|
|
|
permissions:
|
|
contents: read
|
|
|
|
concurrency:
|
|
group: win-update-survival-e2e-${{ github.ref }}
|
|
cancel-in-progress: true
|
|
|
|
jobs:
|
|
survival:
|
|
name: survival (branch build over itself)
|
|
runs-on: windows-2022
|
|
timeout-minutes: 50
|
|
env:
|
|
EXPECT: ${{ inputs.expect || 'survival' }}
|
|
|
|
steps:
|
|
- name: Checkout
|
|
uses: actions/checkout@v6
|
|
with:
|
|
# This job only builds and runs the harness locally; it never pushes.
|
|
persist-credentials: false
|
|
|
|
- name: Setup Node.js
|
|
uses: actions/setup-node@v6
|
|
with:
|
|
node-version-file: package.json
|
|
|
|
- name: Setup pnpm
|
|
uses: pnpm/setup@v2
|
|
with:
|
|
install: false
|
|
|
|
- name: Install dependencies
|
|
run: pnpm install --frozen-lockfile
|
|
|
|
# Why: cache the built installer keyed on the inputs that affect it, so a
|
|
# harness-only edit skips the ~20 min electron-builder build. The daemon
|
|
# relocation code lives under src/, so changing it correctly rebuilds.
|
|
- name: Cache branch installer
|
|
id: cache-installer
|
|
uses: actions/cache@v4
|
|
with:
|
|
path: dist/orca-windows-setup.exe
|
|
key: branch-installer-${{ hashFiles('src/**', 'config/**', 'native/**', 'resources/win32/**', 'package.json', 'pnpm-lock.yaml') }}
|
|
|
|
# Why here: electron-builder's beforePack requires out/mobile-web, and the bundle
|
|
# build resolves React Native and Expo from mobile/node_modules. Gated with the
|
|
# build it feeds, so a cache hit does not pay for an install nothing consumes.
|
|
- uses: ./.github/actions/install-mobile-dependencies
|
|
if: steps.cache-installer.outputs.cache-hit != 'true'
|
|
|
|
- name: Build Windows installer (unsigned)
|
|
if: steps.cache-installer.outputs.cache-hit != 'true'
|
|
run: |
|
|
node config/scripts/ensure-native-runtime.mjs --runtime=electron
|
|
pnpm run build:desktop
|
|
pnpm exec electron-builder --config config/electron-builder.config.cjs --win --publish never
|
|
|
|
# Why: install the branch build, open a terminal, update the SAME build
|
|
# over it, and assert the relocated daemon survives. Same build on both
|
|
# sides isolates the survival mechanism (NSIS kill sweep + userData daemon
|
|
# + adoption) from cross-version staleness policy. No --install-dir: a CI
|
|
# runner has no real Orca to protect.
|
|
- name: Run survival harness
|
|
id: harness
|
|
shell: pwsh
|
|
env:
|
|
ORCA_E2E_DIAG_DIR: artifacts/diag
|
|
run: |
|
|
New-Item -ItemType Directory -Force artifacts | Out-Null
|
|
$exe = "dist/orca-windows-setup.exe"
|
|
if (-not (Test-Path $exe)) { throw "Installer not found at $exe" }
|
|
$log = "artifacts/survival-output.log"
|
|
node tests/tools/win-update-e2e/run.mjs `
|
|
--from "$exe" `
|
|
--to "$exe" `
|
|
--expect "$env:EXPECT" `
|
|
--soak-seconds 60 2>&1 | Tee-Object -FilePath $log
|
|
exit $LASTEXITCODE
|
|
|
|
- name: Upload survival output
|
|
if: always()
|
|
uses: actions/upload-artifact@v7
|
|
with:
|
|
name: win-update-survival-output
|
|
path: |
|
|
artifacts/survival-output.log
|
|
artifacts/diag/**
|
|
retention-days: 7
|
|
if-no-files-found: warn
|