[ricardovfreixo]

Summary

Implements the README's native image viewer roadmap item for published commit attachments, replacing the in-app browser sheet with an EDGE full-screen modal.

• Uses the native image, gesture-area and SharedValue primitives on iOS and Android: whole-image fit, pinch zoom from 1× to 4× and pan, without webviews or new dependencies.
• Keeps the image on screen with the gesture area's new bounded mode and the image's real aspect ratio. Zoomed in, the image's edges stop at the screen's edges, and it stays centred on an axis where it fits. A drag past that meets resistance and springs back on release, so at 1× the image always returns to the centre. A pinch zooms around the fingers.
• Opens the attachment that was tapped; accessible previous/next controls and an image counter navigate the commit's attachments, resetting zoom/pan for each image.
• Keeps the modal mounted for dismissal, handles close/Android back, retains the underlying screen, and shares the existing image cache.
• Rejects unsupported URL schemes on both initial opening and gallery navigation; other browser links keep their existing behavior.

Refs #53. This is the image-viewing slice; the replies/undocking interaction requested there is not included.

Latest-main sync and SDK companions

Merged upstream main (up to #29) without rewriting this branch's history. The README's obsolete camera-branch note was removed; image-viewer implementation and security tests are unchanged. Upstream's composer-sheet handle adjustment and signed-binary .env.example changes are retained.

bounded and content-ratio are new on the gesture area, so this branch requires the branches that add them:

• Core: Let a gesture area keep zoomed content on screen mobile-air#603 (feat/gesture-area-bounded), the element and its props.
• UI: Keep a bounded gesture area's content on screen mobile-ui#173 (feat/gesture-area-bounded), the iOS and Android renderers.

Both are based on current main, which already builds without the tab-row mismatch, so the earlier pins to the #588 and #162 fork branches are gone. The viewer doesn't use anything from them. Only the two SDK package records differ from upstream's lock. Swap both back to dev-main once #603 and #173 are merged.

Verification

• Full Pest suite on this branch with the #603/#173 pair: 1,211 passed (11,709 assertions).
• vendor/bin/pint --dirty --format agent: passed.
• git diff --check: passed.
• Independent subagent review of the PHP/EDGE contract against both installed native renderers, followed by fixes for dismissal lifecycle, loading-comment accuracy, and a gallery-navigation URL guard.

Tests cover both platform wire trees, selected attachment index, gallery boundaries and reset state, unsafe sibling URLs, dismissal/reopening, Log/Profile/CommitDetail/Starred hosts, preservation of browser link behavior, and the bounds the gesture area gets (an image's real ratio, uncapped by the feed, and none when its size is unknown).

Native verification

Built and run on an iPhone 16 Pro and a moto g24 (Debug builds). On the moto, a one-finger drag at 1× trails the finger with resistance and springs back to the centre when released. No screenshots or recordings yet. Still worth a check by hand: close/back, safe-area layout, and opening the viewer above an already-open composer sheet.

Known limits: loading/failure presentation comes from the platform image loader because EDGE exposes no app callbacks. Draft-photo previews are unchanged. The implemented image-viewer item is removed from the README roadmap.

The camera dependency is updated to stable 1.1.0 because the previous fix/present-over-sheets source ref is no longer fetchable. No native runtime-lock changes are included.

[simonhamp]

Thanks for this, Ricardo. I've pushed two commits on top:

• The viewer now keeps the image on screen. Zoomed in, it stops at the screen edges, and at 1x it springs back to the centre after a drag. That needed new support in the gesture area, so it's in Let a gesture area keep zoomed content on screen mobile-air#603 and Keep a bounded gesture area's content on screen mobile-ui#173. The viewer just passes bounded and the image's real aspect ratio.
• composer.json now requires those two branches instead of your fork pins. They're based on current main, and the viewer didn't need anything from #588 or #162. We'll switch back to dev-main once both are merged.

I've updated the description to match. Shout if any of this gets in the way of #79.