Skip to content

fix(compute): adopt observer removal fix - #1107

Merged
Kyle-Ye merged 1 commit into
mainfrom
bugfix/remove_observer
Sep 27, 2026
Merged

Kyle-Ye merged 1 commit into
mainfrom
bugfix/remove_observer

Conversation

@Kyle-Ye

@Kyle-Ye Kyle-Ye commented Sep 27, 2026

Copy link
Copy Markdown
Member

Summary

  • Use Compute 0.5.2-bugfix.2 for source builds to prevent crashes when removing observers from invalidated subgraphs. Update the environment, CI, setup fallback, and stdout renderer references.
  • Default the Compute environment to source builds until a binary release includes the fix.

Dependency

Validation

  • Confirmed that the unchanged OpenAttributeGraph manifest selects Compute 0.5.2-bugfix.2 with the OpenSwiftUI environment settings.
  • Verified shell syntax and whitespace checks.

@github-actions github-actions Bot added area: graph ViewGraph, AttributeGraph, graph hosts, and graph internals. impact: crash Crash, assertion, or fatal runtime failure. type: bug Something is not working correctly. type: build Build system, packaging, toolchain, release, or compiler support. labels Sep 27, 2026
@tuist

tuist Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

🛠️ Tuist Run Report 🛠️

Tests 🧪

Scheme Status Ran test modules Commit
OpenSwiftUI ✅ 7 a8b5148c9

Builds 🔨

Scheme Status Duration Commit
OpenSwiftUI ✅ 52.8s a8b5148c9

@codecov

codecov Bot commented Sep 27, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 33.26%. Comparing base (5448ac1) to head (bf67e15).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1107      +/-   ##
==========================================
- Coverage   33.27%   33.26%   -0.01%     
==========================================
  Files         805      805              
  Lines       63992    63992              
==========================================
- Hits        21293    21290       -3     
- Misses      42699    42702       +3     

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

@Kyle-Ye
Kyle-Ye merged commit 2e0d99a into main Sep 27, 2026
6 checks passed
@Kyle-Ye
Kyle-Ye deleted the bugfix/remove_observer branch September 27, 2026 08:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: graph ViewGraph, AttributeGraph, graph hosts, and graph internals. impact: crash Crash, assertion, or fatal runtime failure. type: bug Something is not working correctly. type: build Build system, packaging, toolchain, release, or compiler support.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant