Skip to content

Merge development → main (v1.3.3) - #43

Merged
setoelkahfi merged 5 commits into
mainfrom
development
Oct 17, 2026
Merged

setoelkahfi merged 5 commits into
mainfrom
development

Conversation

@setoelkahfi

Copy link
Copy Markdown
Collaborator

Promotes development to main at v1.3.3.

Since main's last promotion this brings in the watchOS fix that switches reqwest to native-tls so arm64_32 builds (#41) and the 1.3.3 version bump (#42).

setoelkahfi and others added 5 commits October 16, 2026 23:38
reqwest 0.13's rustls feature pulls in aws-lc-sys, whose C code fails on
arm64_32 because it assumes 64-bit limbs on aarch64. ring fails there
too, on its pointer-size asserts.

On watchOS, ed-acp and ed-onde-account now use reqwest's native-tls
(Security.framework) and ed-mcp uses rmcp's reqwest-native-tls, so no
rustls crypto provider is built. Other platforms keep rustls with
aws-lc. The workspace reqwest dependency no longer picks a TLS backend;
each crate chooses one per target.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Use native-tls for reqwest on watchOS so arm64_32 builds
Co-Authored-By: siGit Code v1.6.6-acp <noreply@sigit.si>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>

@sigit-code-review sigit-code-review 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.

This change promotes development to main at v1.3.3, bringing in a watchOS fix and a version bump. The main technical change is switching reqwest's TLS backend from rustls (aws-lc) to native-tls for watchOS targets, due to build failures and pointer size assertions in aws-lc/ring on arm64_32. This is implemented by conditionalizing reqwest and rmcp dependencies in ed-acp, ed-mcp, and ed-onde-account Cargo.toml files. The workspace version and all internal crate versions are bumped to 1.3.3. The reqwest dependency in the workspace root drops the rustls feature, leaving only json and stream, as TLS is now specified per-target. No code changes outside of Cargo.toml files are shown here.


Automated review by siGit Code Review · commit 8092ffb · see the review dashboard

@setoelkahfi setoelkahfi self-assigned this Oct 17, 2026
@setoelkahfi
setoelkahfi merged commit 683f138 into main Oct 17, 2026
4 checks passed
@setoelkahfi
setoelkahfi deleted the development branch October 17, 2026 08:03
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