Skip to content

Update Compute backend configuration#228

Merged
Kyle-Ye merged 3 commits into
mainfrom
bugfix/compute
Jun 15, 2026
Merged

Update Compute backend configuration#228
Kyle-Ye merged 3 commits into
mainfrom
bugfix/compute

Conversation

@Kyle-Ye

@Kyle-Ye Kyle-Ye commented Jun 15, 2026

Copy link
Copy Markdown
Member

Summary

  • Update Compute backend configuration to support source and binary modes with explicit repo/version/branch settings.
  • Allow Tuist manifests to receive OPENSWIFTUI_* and OPENATTRIBUTEGRAPH_* environment values so generated projects can resolve the intended backend settings.

@Kyle-Ye Kyle-Ye marked this pull request as ready for review June 15, 2026 17:50
@Kyle-Ye Kyle-Ye changed the title [codex] Support Compute backend configuration Update Compute backend configuration Jun 15, 2026
@codecov

codecov Bot commented Jun 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 32.17%. Comparing base (2f3beaf) to head (4cf6c33).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #228   +/-   ##
=======================================
  Coverage   32.17%   32.17%           
=======================================
  Files          71       71           
  Lines        2601     2601           
=======================================
  Hits          837      837           
  Misses       1764     1764           
Flag Coverage Δ
ios 32.19% <ø> (ø)
macos 10.15% <ø> (ø)
ubuntu 12.03% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@augmentcode

augmentcode Bot commented Jun 15, 2026

Copy link
Copy Markdown
🤖 Augment PR Summary

Summary: This PR makes the Compute-backed OpenAttributeGraphShims setup configurable via environment variables and ensures Tuist forwards relevant environment values into manifest evaluation.

Changes:

  • Refactors the Compute dependency setup to support binary vs source modes with configurable repo/version/branch inputs.
  • Updates Tuist project generation to pass through OPENSWIFTUI_* and OPENATTRIBUTEGRAPH_* environment variables to manifests.
  • Refreshes Package.resolved to a newer DarwinPrivateFrameworks revision to match the updated configuration flow.

🤖 Was this summary useful? React with 👍 or 👎

@augmentcode augmentcode Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review completed. 2 suggestions posted.

Fix All in Augment

Comment augment review to trigger a new review at any time.

Comment thread Package.swift Outdated
Comment thread Package.swift
Comment thread Package.swift
if computeBinary {
let binary = envBoolValue("OPENATTRIBUTESHIMS_COMPUTE_BINARY", default: false)
if binary {
// FIXME: This version is broken and will have runtime crash

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cc @jcmosc

@Kyle-Ye Kyle-Ye merged commit f20328e into main Jun 15, 2026
10 checks passed
@Kyle-Ye Kyle-Ye deleted the bugfix/compute branch June 15, 2026 18:05
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