Skip to content

rust(fix): return artifact app links from the artifact tools - #832

Merged
bill-sift merged 6 commits into
mainfrom
return-artifact-app-links
Oct 14, 2026
Merged

bill-sift merged 6 commits into
mainfrom
return-artifact-app-links

Conversation

@bill-sift

@bill-sift bill-sift commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Other MCP tools return a convenient url field to make it easier for users to follow up in the app. Updated the various artifact tools to also return a url, with the version ID when relevant.

  • list_artifacts has a url since it's a standard list endpoint
  • list_artifact_versions has one top level artifact_url to avoid duplicated data, which matches list_rule_versions
  • download_artifact, create_artifact, update_artifact, archive_artifact, and unarchived_artifact return an artifact_url field
  • Fixed escaping brackets in url rendering for all tools, since that would break markdown links
  • Updated agent guidance to return markdown links instead of raw URLs

@alexa-sift

Copy link
Copy Markdown
Contributor

Two small nits/nice to have:

  • That tests provide mostly full coverage, but would be nice to have a screenshot of an agent showing the proper artifcat url link now
  • Do we need to worry about a display name containing a stray ], like results v2].csv, breaking the link?

@bill-sift

Copy link
Copy Markdown
Contributor Author
  • We need to merge the PR first to pick up the new version of the MCP for manual testing, so that will come later
  • Good catch! Fixed in the shared helper so it will apply to all callers

alexa-sift
alexa-sift previously approved these changes Oct 14, 2026
Comment thread rust/crates/sift_mcp/src/tool/artifacts/mod.rs
Comment thread rust/crates/sift_mcp/src/tool/artifacts/mod.rs
@bill-sift
bill-sift enabled auto-merge (squash) October 14, 2026 17:51
@bill-sift
bill-sift merged commit 0a10cf7 into main Oct 14, 2026
22 checks passed
@bill-sift
bill-sift deleted the return-artifact-app-links branch October 14, 2026 17:57
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.

3 participants