Repository navigation
Record that the vanishing Layer 1 is Photoshop's own Place - #176
Merged
Merged
Conversation
The 2026-10-15 and 2026-11-05 smoke runs both reported that importing the first result into a new document removes its empty Layer 1, with the cause unknown. Nothing in OpenLayer deletes it: the import only places, renames and positions the layer. A control run in Photoshop 2025 settled it. File > Place Embedded of a PNG into a new transparent document, with no plugin involved, removed the empty Layer 1 the same way (History: New, then Place Embedded Smart Object). With Layer 1 filled first, the same place kept it. Mehran decided to keep the behaviour. Smoke step 1.2 now says it is expected so future runs stop reporting it, the beta gate records the decision under G4, and the v0.41.0 known-limitations line names the real cause. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
3 tasks done
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.
Records the decision on finding (3) from the 2026-11-05 G4 smoke run: "Importing the first result into a new document still removes its empty Layer 1." Docs only, no code change.
What changed and why
The finding isn't caused by OpenLayer. It's Photoshop's own Place, and Mehran decided to keep it as is.
Evidence
importGeneratedImageAsLayer(src/photoshop/photoshopAdapter.ts) only places, renames and positions it.deleteLayerByIdis used only for temporary mask layers and for rolling back a failed import.Edits
docs/uxp-smoke-checklist.mdstep 1.2 now says the empty Layer 1 disappearing on the first import into a new transparent document is expected, so future runs don't report it.docs/beta-gate.mdG4 status records the cause and the keep decision.CHANGELOG.mdv0.41.0 known limitations: the line keeps its wording and adds that it turned out to be Photoshop's own behaviour.The Results rows in the smoke checklist are a historical record, so I left them as they were.
What was verified
The Photoshop control run is described above. The test document was closed without saving.
What could NOT be verified
Open question
The published v0.41.0 release notes on GitHub still list this as an OpenLayer limitation. I haven't touched the release. If you want it corrected there too, say so and I'll edit the release body.
What Mehran should check in Photoshop
Optional, since the behaviour isn't changing. To see it yourself:
Smoke checklist blocks to re-check (
docs/uxp-smoke-checklist.md)🤖 Generated with Claude Code