Skip to content

[build-tools] Normalize project archive permissions - #4202

Draft
AbbanMustafa wants to merge 1 commit into
mainfrom
mustafa/fix-archive-permissions
Draft

[build-tools] Normalize project archive permissions#4202
AbbanMustafa wants to merge 1 commit into
mainfrom
mustafa/fix-archive-permissions

Conversation

@AbbanMustafa

@AbbanMustafa AbbanMustafa commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Why

A regression review found 664 prebuild failures across 227 apps and 221 accounts with EACCES errors while reading project assets.

View the affected logs in Datadog

Project archives can preserve modes that leave files unreadable or directories unwritable by the build user.

Screenshot 2026-08-14 at 12 26 49 PM

How

After extracting the project archive, grant the build user read/write access while preserving executable bits.

Test Plan

  • Added a unit test verifying extraction is followed by chmod -R u+rwX.
  • Locally archived the app icon with mode 000; without the fix, Prebuild failed with EACCES while reading it.
  • Rebuilt the same corrupted archive with the fix enabled; Prebuild and the full iOS build succeeded.

@AbbanMustafa AbbanMustafa changed the title [eas build] Normalize project archive permissions [build-tools] Normalize project archive permissions Aug 14, 2026
@AbbanMustafa AbbanMustafa added the no changelog PR that doesn't require a changelog entry label Aug 14, 2026
@codecov

codecov Bot commented Aug 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.52%. Comparing base (a9d693d) to head (7b4093c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4202      +/-   ##
==========================================
+ Coverage   63.52%   63.52%   +0.01%     
==========================================
  Files        1028     1028              
  Lines       46918    46919       +1     
  Branches     9847     9847              
==========================================
+ Hits        29801    29802       +1     
  Misses      17015    17015              
  Partials      102      102              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@AbbanMustafa
AbbanMustafa force-pushed the mustafa/fix-archive-permissions branch from 642811e to 7b4093c Compare August 14, 2026 16:25
@github-actions

Copy link
Copy Markdown

⏩ The changelog entry check has been skipped since the "no changelog" label is present.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no changelog PR that doesn't require a changelog entry

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant