Skip to content

Bump gpui-pre to 0.3.6 - #18

Merged
madcodelife merged 1 commit into
longbridge:mainfrom
yuanzhuang-svg:bump-gpui-pre-0.3.6
Sep 22, 2026
Merged

madcodelife merged 1 commit into
longbridge:mainfrom
yuanzhuang-svg:bump-gpui-pre-0.3.6

Conversation

@yuanzhuang-svg

Copy link
Copy Markdown

What

  • Adapt the iOS FallbackAtlas impl to gpui-pre 0.3.6, which changed
    PlatformAtlas::get_or_insert_with to take AtlasKey by value (was &AtlasKey).
  • Bump the pinned gpui-pre / gpui-pre-wgpu / gpui-pre-reqwest-client deps
    (lib + example) from =0.3.5 to =0.3.6.

Why

gpui-kit 0.6.5 requires gpui-pre =0.3.6, but gpui-mobile pins =0.3.5 — the two
exact pins are irreconcilable. Any downstream on gpui-kit 0.6.5 can't resolve
alongside gpui-mobile. Concretely, the ai-chat iOS example stopped building
after panel moved to gpui-kit's Message/Marker APIs (ai-chat #73, #82): the iOS
workspace is stuck on gpui-kit 0.6.2 because gpui-mobile only allows gpui-pre 0.3.5.

Verified

cargo check --lib --target aarch64-apple-ios-sim is clean after the change.

🤖 Generated with Claude Code

@yuanzhuang-svg
yuanzhuang-svg force-pushed the bump-gpui-pre-0.3.6 branch 2 times, most recently from 5052cc2 to d790aed Compare September 22, 2026 11:48
gpui-pre 0.3.6 changed `PlatformAtlas::get_or_insert_with` to take the
`AtlasKey` by value (was `&AtlasKey`). Adapt both the iOS and Android
`FallbackAtlas` impls accordingly, and bump the pinned `gpui-pre` /
`gpui-pre-wgpu` / `gpui-pre-reqwest-client` deps (lib + example) from
=0.3.5 to =0.3.6. The example's pinned `gpui-kit` rev is moved from
9504b46 to 13c716b6 to match gpui-pre 0.3.6 (the old rev's inspector API
expects a different `register_inspector_element` closure arity).

Without this, downstreams that pull gpui-kit 0.6.5 (which requires
gpui-pre =0.3.6) cannot resolve alongside gpui-mobile (pinned =0.3.5) —
e.g. the ai-chat iOS example no longer builds after it moved panel to
gpui-kit's `Message`/`Marker` APIs.

Verified: `cargo check --target aarch64-apple-ios-sim` is clean for both
the lib and the example.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@madcodelife
madcodelife merged commit da372e9 into longbridge:main Sep 22, 2026
5 checks passed
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.

2 participants