Skip to content

Add auto-sync for binary repo and update integration docs#845

Merged
Kyle-Ye merged 1 commit intomainfrom
feature/auto-sync-binary-repo
Mar 25, 2026
Merged

Add auto-sync for binary repo and update integration docs#845
Kyle-Ye merged 1 commit intomainfrom
feature/auto-sync-binary-repo

Conversation

@Kyle-Ye
Copy link
Collaborator

@Kyle-Ye Kyle-Ye commented Mar 25, 2026

Summary

  • Add reusable codesign-xcframework action with individual checksum outputs
  • Add update-binary-repo job to release workflow that auto-updates OpenSwiftUI-spm Package.swift from template on each release
  • Add binary integration (recommended) section to INTEGRATION.md
  • Rename OpenSwiftUI-binary to OpenSwiftUI-spm

Setup required

  • BINARY_REPO_PAT org secret (already configured)

Test plan

  • Verify next tag release auto-updates OpenSwiftUI-spm

- Add individual checksum outputs to build-xcframework action
- Add update-binary-repo job to release workflow that auto-updates
  OpenSwiftUI-spm Package.swift from template on each release
- Add binary integration section to INTEGRATION.md
- Rename OpenSwiftUI-binary to OpenSwiftUI-spm
@Kyle-Ye Kyle-Ye merged commit 4b9e0a4 into main Mar 25, 2026
7 of 8 checks passed
@Kyle-Ye Kyle-Ye deleted the feature/auto-sync-binary-repo branch March 25, 2026 15:52
@github-actions github-actions bot added documentation Improvements or additions to documentation enhancement New feature or request labels Mar 25, 2026
@codecov
Copy link

codecov bot commented Mar 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 26.86%. Comparing base (6e47bd6) to head (61ed2d0).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #845      +/-   ##
==========================================
+ Coverage   26.85%   26.86%   +0.01%     
==========================================
  Files         671      671              
  Lines       43470    43470              
==========================================
+ Hits        11674    11679       +5     
+ Misses      31796    31791       -5     

☔ View full report in Codecov by Sentry.
📢 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.

@augmentcode
Copy link

augmentcode bot commented Mar 25, 2026

🤖 Augment PR Summary

Summary: Automates keeping the binary SwiftPM repo in sync with OpenSwiftUI releases.

Changes:

  • Expose per-XCFramework checksum outputs from the composite build action, and surface them as outputs of the release workflow job.
  • Add an update-binary-repo job that clones OpenSwiftUI-spm, generates Package.swift from a template using the release tag + checksums, then commits/tags/pushes.
  • Update INTEGRATION.md to recommend binary integration via OpenSwiftUI-spm and to clarify when to use source integration.
  • Adjust integration docs wording/section structure and replace the previews note with a link to Docs/Preview.md.

Technical Notes: The release workflow now publishes individual checksum outputs so downstream jobs can safely template Package.swift without re-computing checksums.

🤖 Was this summary useful? React with 👍 or 👎

Copy link

@augmentcode augmentcode bot left a comment

Choose a reason for hiding this comment

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

Review completed. 1 suggestion posted.

Fix All in Augment

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

git add Package.swift
git diff --cached --quiet && echo "No changes" && exit 0
git commit -m "Update to ${VERSION} with code-signed XCFrameworks"
git tag "${VERSION}"
Copy link

Choose a reason for hiding this comment

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

git clone will fetch existing tags, so on a workflow re-run for the same release git tag "${VERSION}" is likely to fail because the tag already exists locally; consider making this step resilient so re-runs don’t turn the workflow red unnecessarily.

Severity: medium

Fix This in Augment

🤖 Was this useful? React with 👍 or 👎, or 🚀 if it prevented an incident/outage.

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

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant