Skip to content

fix(ci): remove platform-dependent build failures - #21

Merged
johnnyhuy merged 2 commits into
mainfrom
fix/ci-workflows
Jul 30, 2026
Merged

fix(ci): remove platform-dependent build failures#21
johnnyhuy merged 2 commits into
mainfrom
fix/ci-workflows

Conversation

@johnnyhuy

Copy link
Copy Markdown
Contributor

Root causes

  • The pipeline test sourced generate.sh, which executed the generation loops on a clean Ubuntu checkout. The macOS development checkout passed only because ignored generated assets caused those loops to skip.
  • The iOS build queried CoreSimulator even though CI only needs compilation. Hosted macOS runners intermittently time out before xcodebuild starts.

Fix

  • guard the video generation entry point so functions can be sourced without executing the pipeline
  • test the real source path instead of extracting a partial script
  • add ios:build:ci using the generic iOS Simulator destination with signing disabled
  • keep device-specific local build and UI-test tasks unchanged
  • update checkout to v7.0.1 and mise-action to v4.2.3 using pinned commit SHAs

Verification

  • 23 pipeline checks pass
  • 95 Swift tests pass
  • mise run ios:build:ci succeeds
  • shell syntax, workflow YAML parsing, and git diff checks pass

Co-authored-by: opencode-agent noreply@opencode.ai

johnnyhuy and others added 2 commits July 30, 2026 22:50
Keep video pipeline functions sourceable on clean Linux runners and build iOS against a generic simulator destination when no device execution is required.

Co-authored-by: opencode-agent <noreply@opencode.ai>
Merge persisted workouts with in-memory completions and synchronize history access so asynchronous initialization cannot discard newly completed workouts.

Co-authored-by: opencode-agent <noreply@opencode.ai>
@johnnyhuy
johnnyhuy merged commit b496e3a into main Jul 30, 2026
7 checks passed
@johnnyhuy
johnnyhuy deleted the fix/ci-workflows branch July 30, 2026 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant