feat(app): add native file image actions - #4063
Closed
dwyanewang wants to merge 2 commits into
Closed
dwyanewang wants to merge 2 commits into
dwyanewang wants to merge 2 commits into
Conversation
Contributor
Author
|
Android visual and interaction QA for commit Manual Android QA confirmed that Copy, Save, and Share complete successfully, and that closing the compact preview returns to the parent session. |
dwyanewang
marked this pull request as ready for review
August 30, 2026 08:57
Collaborator
|
Closing this feature PR. Please discuss the workflow and shared need in Discussions, following CONTRIBUTING.md. |
Contributor
Author
|
Understood, thanks. Opened the workflow discussion for the image actions: #4534. Leaving this closed. If the discussion gains interest I'll reopen a narrower PR covering only the image actions — the compact close action is a separate concern and shouldn't have been bundled here. |
dwyanewang
added a commit
to dwyanewang/paseo
that referenced
this pull request
Sep 12, 2026
Refresh every overlay's reviewed coordinates to this round's main snapshot fa93c42 (was d7c7044) and add the new personal overlay feat/todo-plugin-hybrid. Semantic review of d7c7044..fa93c42 (9 upstream commits) against every overlay head: none of the features were absorbed upstream, so all entries stay. - feat/file-image-actions (956b2e3): PR getpaseo#4063 never merged; image-preview* files and expo-media-library still absent from main. Only i18n paths overlap. - feat/forge-plugin-infrastructure (a900da8 -> 7e3d1c7): rebased onto the new main. range-diff maps all 8 commits; the two conflict resolutions in plugin-process.ts and run-git-command.ts are semantic unions (upstream's releaseApi dispose and executeGitCommand<Output>/runGitCommandBytes refactor kept alongside the branch's forge cleanup and git URL redaction). One new test-only commit fixes a stale spawn-args assertion. - feat/todo-plugin-hybrid (faedbc1): new personal overlay, single commit on fa93c42. PluginSettingsDocument and plugins/agent-launch/** have no counterpart in main. Restructured at the source so it merges cleanly with rw-base and the forge overlay. - fix/android-keyboard-covers-input (8541863): PR getpaseo#2467 open; main still uses keyboardBehavior="extend" and has no android_keyboardInputMode. - fix/electron-explorer-tab-menu (99cb41d): PR getpaseo#3890 open; main exposes the configuration only through a right-click ContextMenu, not the toolbar trigger. - fix/skill-cache-invalidation-scoped (317f359): PR getpaseo#4463 open; headOid and worktreeRevision still absent from the checkout-status schemas. - refactor/provider-history-read (e1d2876): PR getpaseo#4473 open; readSessionHistory has zero hits in main, which still resumes archived sessions via purpose.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Linked issue
Follow-up to #4032.
Type of change
Reasoning
Image files can now be previewed and zoomed in place, but on mobile the preview does not expose the image actions users need to move the result into another workflow. A user currently has to leave the preview and find another route to copy, save, or share the image.
File previews are child tabs of the session that opened them. The compact header also needs a direct close action so a user can leave the preview and return to that parent session without opening the tab switcher.
This change adds the native image actions and the compact file-preview close path without replacing the shared zoom viewport.
Goals
Non-goals
QA
Automated checks:
PASEO_PASSWORDis removed for this command because the isolated E2E daemon inherits the parent environment; leaving a local daemon password set enables authentication that the seed client does not use.The pre-commit hook ran the repository checks against the committed change:
Platform coverage:
Android visual and interaction QA is complete for the current PR commit. iOS remains untested because no iOS environment is available on this Linux host; this gap is reported for review.
Checklist
npm run typecheckpassesnpm run lintpassesnpm run formatpasses