Skip to content

fix(ci): restore app-host artifact rerun setup - #15029

Merged
teamleaderleo merged 4 commits into
mainfrom
fix/rerun-selected-binary-frameworks
Sep 27, 2026
Merged

teamleaderleo merged 4 commits into
mainfrom
fix/rerun-selected-binary-frameworks

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Sep 27, 2026 •

Copy link
Copy Markdown
Collaborator

App-host artifact reruns failed during compilation when every resolved binary framework variant became visible, then stopped before tests because diagnostic cleanup unconditionally required sudo. This repairs both setup failures without rebuilding the app or changing artifact identity, runner admission, or console-session checks.

Recover binary module search paths only for frameworks embedded in the adopted app or selected test bundle and missing usable archived module files. Preserve existing importable frameworks, select compatible macOS slices, and reject ambiguous matches. Collect this job’s existing xcodebuild and console captures, preserving rebuilt-run logs before the baseline, without deleting global debug logs or changing ownership.

Separate regression/fix commits cover unused binary variants and execute the workflow’s test/baseline/collection steps with sudo denied. All 49 helper tests and 12 applicable local static checks pass.

At e70c4a78469011034a7d520bd1b21365d77ff26c, native run36333697551 rebuilt the test bundle against the existing product and completed both selected executions: 71 tests passed with the repaired fixture (10.506 seconds), and 71 baseline tests passed (10.570 seconds). Log collection and cleanup also passed. The baseline’s earlier timing failures did not recur in this focused run.

The prior native compile passed after the module-selection correction but stopped before tests at the diagnostic sudo command fixed here.

Follows #15026. The original variant-cycle failure remains available.

Changelog

none

@coderabbitai

coderabbitai Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 112395de-3e7f-48bf-a1c4-09b9d4c47096

📥 Commits

Reviewing files that changed from the base of the PR and between c842f7d and e70c4a7.

📒 Files selected for processing (3)
  • .github/workflows/app-host-test-rerun.yml
  • scripts/ci/app_host_test_rerun.py
  • tests/test_app_host_test_rerun.py
 __________________
< I see dead code. >
 ------------------
  \
   \   (\__/)
       (•ㅅ•)
       /   づ
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@teamleaderleo teamleaderleo changed the title fix(ci): recover only selected binary framework modules fix(ci): restore app-host artifact rerun setup Sep 27, 2026
@teamleaderleo
teamleaderleo marked this pull request as ready for review September 27, 2026 16:41
@teamleaderleo
teamleaderleo enabled auto-merge (squash) September 27, 2026 16:41
@teamleaderleo
teamleaderleo merged commit 368ec9f into main Sep 27, 2026
71 of 72 checks passed
@teamleaderleo
teamleaderleo deleted the fix/rerun-selected-binary-frameworks branch September 27, 2026 16:42
@github-actions

Copy link
Copy Markdown
Contributor

Merge receipt for e70c4a7846: every check was green at merge (13 verified; 17 skipped by policy). Full suite runs on main after merge.

rustybret pushed a commit to rustybret/bmux that referenced this pull request Sep 27, 2026
8efe28d Add terminal.confirmUnsafePaste to confirm unsafe pastes in a window sheet (manaflow-ai#14951)
368ec9f fix(ci): restore app-host artifact rerun setup (manaflow-ai#15029)
e67ea0f perf: stop launching the cmux CLI for every queued Claude hook (manaflow-ai#14931)
badf9f6 test: give the tmux split mapping test its own portal authority (manaflow-ai#15028)
41a0c37 current-work: preserve remote machine kinds (manaflow-ai#14914)
c842f7d test(hermes): wait for the hook installer instead of racing a 1 s deadline (manaflow-ai#15027)
810ffba fix(ci): resolve binary modules in detached test reruns (manaflow-ai#15026)
d363290 test: await fork probe fixture start signals (manaflow-ai#15025)
8c98e64 Add cmux import for settings from other terminals (manaflow-ai#15004)
30aa6c1 Keep SSH workspace titles when cmux-tui creates the remote workspace (manaflow-ai#14976)
b33c467 Restore workspace group color and icon key handling from manaflow-ai#13877 (manaflow-ai#15000)

# Conflicts:
#	.github/workflows/app-host-test-rerun.yml
#	.github/workflows/ci-macos.yml
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