Skip to content

Return artifact source from show-artifact - #1943

Merged
RhysSullivan merged 3 commits into
UsefulSoftwareCo:mainfrom
baggiiiie:fix/show-artifact-source
Sep 15, 2026
Merged

RhysSullivan merged 3 commits 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.

@baggiiiie
baggiiiie marked this pull request as ready for review September 4, 2026 06:07
@baggiiiie
baggiiiie force-pushed the fix/show-artifact-source branch from 1d6a36f to 219dfd6 Compare September 13, 2026 09:18
@RhysSullivan
RhysSullivan merged commit b5d53cb into UsefulSoftwareCo:main Sep 15, 2026
40 checks passed
@baggiiiie
baggiiiie deleted the fix/show-artifact-source branch September 15, 2026 02:16
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

2 participants