Skip to content

Return artifact source from show-artifact - #1943

Open
baggiiiie wants to merge 1 commit into
UsefulSoftwareCo:mainfrom
baggiiiie:fix/show-artifact-source
Open

Return artifact source from show-artifact#1943
baggiiiie wants to merge 1 commit into
UsefulSoftwareCo:mainfrom
baggiiiie:fix/show-artifact-source

Conversation

@baggiiiie

@baggiiiie baggiiiie commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Summary

when an agent calls show-artifact from a client without MCP Apps support, the result only includes a save confirmation and deep link. the model cannot read the current component source needed for edit-artifact.

this change returns the current source in both the text and structured MCP result channels for show-artifact, while create/edit success stays a save confirmation. it also updates the tool description and covers the Apps, fallback URL, and no-web-UI delivery paths.

Linked issue

Fixes #1941

Verification

  • format, lint, and typecheck
  • scoped artifact tests — 82 passed
  • e2e — Artifacts · create-artifact hands a non-Apps client a deep link that renders the live componentrecording

Checklist

  • Added a changeset (bun run changeset), or this change needs none.
  • Added or updated tests for the new behaviour.
  • No secrets, credentials, or private data in the diff.

…t render MCP Apps can edit from the tool result.
@baggiiiie
baggiiiie marked this pull request as ready for review September 4, 2026 06:07
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.

[bug] show-artifact does not return artifact source

1 participant