Skip to content

fix(ci): allow organization members to run mobile previews - #41

Merged
khoi merged 1 commit into
mainfrom
supacode/investigate-skipped-ci-checks
Oct 4, 2026
Merged

khoi merged 1 commit into
mainfrom
supacode/investigate-skipped-ci-checks

Conversation

@khoi

@khoi khoi commented Oct 4, 2026

Copy link
Copy Markdown
Member

Problem

GitHub reports organization maintainers as MEMBER, so the mobile preview workflow's OWNER requirement skips their PRs even when the mobile deployment label is present. For example, PR #37 reports author khoi with author_association: MEMBER.

Change

Accept MEMBER alongside OWNER in the existing preview eligibility condition. The PR must still come from this repository and carry 🚀 Mobile Continuous Deployment; unrelated label events still skip.

Scope and approval

This is a focused fix to an established opt-in preview workflow, requested by the maintainer. It corrects the author-association check for an organization-owned repository.

Verification

mise exec -- actionlint .github/workflows/mobile-eas-preview.yml, vp fmt --check .github/workflows/mobile-eas-preview.yml, and git diff --check passed. Evaluating the workflow condition against eight representative contexts allowed owners and organization members with the preview label, and rejected collaborators, contributors, forks, missing labels, and unrelated label events. The native build and deployment were not run.

Implemented with gpt-6.1-sol through the Codex harness in Supacode.

@khoi khoi added the bug Something is broken or behaving incorrectly. label Oct 4, 2026
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:XS labels Oct 4, 2026
@github-actions

github-actions Bot commented Oct 4, 2026

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 4.9 KiB 4.9 KiB +40 B (+0.8%) 6.8 KiB ✅
Codex Thread snapshot wire 3.7 KiB 3.7 KiB 0 B (0.0%) 4.9 KiB ✅
Codex Live turn WebSocket wire 1.1 KiB 1.2 KiB +40 B (+3.4%) 2.0 KiB ✅
Codex Live turn WebSocket decoded 20.4 KiB 20.4 KiB +41 B (+0.2%) 29.3 KiB ✅
Codex Live turn messages 1 2 +1 (+100.0%) 8 ✅
Claude Total thread wire 4.9 KiB 4.9 KiB +41 B (+0.8%) 6.8 KiB ✅
Claude Thread snapshot wire 3.7 KiB 3.7 KiB 0 B (0.0%) 4.9 KiB ✅
Claude Live turn WebSocket wire 1.2 KiB 1.2 KiB +41 B (+3.5%) 2.0 KiB ✅
Claude Live turn WebSocket decoded 20.7 KiB 20.8 KiB +41 B (+0.2%) 29.3 KiB ✅
Claude Live turn messages 1 2 +1 (+100.0%) 8 ✅

Baseline: 5a64a46 · PR result: 2b0a0f7 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 106.1 KiB
  • Claude decoded thread snapshot: 106.4 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

@khoi
khoi merged commit 9921b84 into main Oct 4, 2026
27 checks passed
@khoi
khoi deleted the supacode/investigate-skipped-ci-checks branch October 6, 2026 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something is broken or behaving incorrectly. size:XS vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant