Skip to content

[build-tools] Add Argent remote session artifact uploads#3880

Merged
sjchmiela merged 2 commits into
mainfrom
stanley/argent-artifact-upload
Jun 19, 2026
Merged

[build-tools] Add Argent remote session artifact uploads#3880
sjchmiela merged 2 commits into
mainfrom
stanley/argent-artifact-upload

Conversation

@sjchmiela

@sjchmiela sjchmiela commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • bump the minimum supported Argent remote-session version to the latest published @swmansion/argent version (0.12.0)
  • enable Argent's artifact inventory endpoint before starting the tool-server
  • poll the local Argent tool-server every 5 seconds for new artifacts and stream each new artifact to the EAS device-run-session artifact upload backend
  • add focused tests for version validation, artifact inventory listing, and signed upload streaming

Testing

CI should pass.

@github-actions

Copy link
Copy Markdown

Subscribed to pull request

File Patterns Mentions
**/* @douglowder

Generated by CodeMention

@codecov

codecov Bot commented Jun 18, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 58.75000% with 33 lines in your changes missing coverage. Please review.
✅ Project coverage is 58.92%. Comparing base (0986682) to head (e827ee6).

Files with missing lines Patch % Lines
...ges/build-tools/src/steps/utils/argentArtifacts.ts 52.55% 28 Missing ⚠️
...ls/src/steps/functions/startArgentRemoteSession.ts 50.00% 3 Missing ⚠️
...tools/src/steps/utils/deviceRunSessionArtifacts.ts 86.67% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3880      +/-   ##
==========================================
+ Coverage   58.91%   58.92%   +0.01%     
==========================================
  Files         932      934       +2     
  Lines       40744    40823      +79     
  Branches     8593     8606      +13     
==========================================
+ Hits        24002    24051      +49     
- Misses      16646    16676      +30     
  Partials       96       96              

☔ 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.

Base automatically changed from stanley/argent-remote-session to main June 18, 2026 10:13
@sjchmiela sjchmiela force-pushed the stanley/argent-artifact-upload branch from ce6edfe to c298bb5 Compare June 18, 2026 10:36
@sjchmiela sjchmiela marked this pull request as draft June 18, 2026 10:37
@sjchmiela sjchmiela force-pushed the stanley/argent-artifact-upload branch 5 times, most recently from 148eea5 to 6ea566a Compare June 18, 2026 11:07
@sjchmiela sjchmiela added the no changelog PR that doesn't require a changelog entry label Jun 18, 2026
@sjchmiela sjchmiela marked this pull request as ready for review June 18, 2026 11:09
@sjchmiela sjchmiela requested a review from hSATAC June 18, 2026 11:09
@sjchmiela sjchmiela changed the title Add Argent remote session artifact uploads [build-tools] Add Argent remote session artifact uploads Jun 18, 2026
@sjchmiela sjchmiela force-pushed the stanley/argent-artifact-upload branch from 6ea566a to e827ee6 Compare June 19, 2026 07:15
@github-actions

Copy link
Copy Markdown

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

@sjchmiela

Copy link
Copy Markdown
Contributor Author

Tested manually locally.

Zrzut ekranu 2026-06-19 o 09 40 39

@sjchmiela sjchmiela merged commit b2388fd into main Jun 19, 2026
10 checks passed
@sjchmiela sjchmiela deleted the stanley/argent-artifact-upload branch June 19, 2026 07:41
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.

2 participants