Visitar URL original
Accept input-interface destinations when marshalling by i-am-tom · Pull Request #24577 · pulumi/pulumi · GitHub
Skip to content

Accept input-interface destinations when marshalling - #24577

Merged
i-am-tom merged 1 commit into
masterfrom
claude/practical-keller-05d153
Sep 9, 2026
Merged

i-am-tom merged 1 commit into
masterfrom
claude/practical-keller-05d153

Conversation

@i-am-tom

@i-am-tom i-am-tom commented Sep 8, 2026 •

Copy link
Copy Markdown
Contributor

Marshalling fails when checking types against interfaces. This PR adds another case to handle this.

Fixes #13226.

🤖 Generated with Claude Code

@i-am-tom
i-am-tom marked this pull request as ready for review September 8, 2026 13:49
@i-am-tom
i-am-tom requested a review from a team as a code owner September 8, 2026 13:49
Marshalling a value into a destination whose static type is an input
interface (e.g. `pulumi.StringInput`) compared the value's *element*
type against the interface, which never matches, so any output or
prompt value assigned to such a field failed with "cannot marshal an
input of type pulumi.StringOutput ... as a value of type
pulumi.StringInput".

When the destination is an interface that the value itself satisfies,
marshal the value as its own element type instead. The rest of the
marshalling path already collapses interface destinations this way.

Fixes #13226

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
@pulumi-bot
pulumi-bot force-pushed the claude/practical-keller-05d153 branch from 5c3a716 to d1c17a9 Compare September 8, 2026 13:49
@i-am-tom
i-am-tom added this pull request to the merge queue Sep 9, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 9, 2026
@i-am-tom
i-am-tom added this pull request to the merge queue Sep 9, 2026
Merged via the queue into master with commit 814c846 Sep 9, 2026
245 of 247 checks passed
@i-am-tom
i-am-tom deleted the claude/practical-keller-05d153 branch September 9, 2026 17:09
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.

error in go component construct copying inputs onto component outputs

2 participants