Skip to content

ci: show the Currents session's screenshots and screencast in the pull request - #7

Merged
agoldis merged 1 commit into
mainfrom
currents/embed-session-media
Nov 12, 2026
Merged

agoldis merged 1 commit into
mainfrom
currents/embed-session-media

Conversation

@agoldis

@agoldis agoldis commented Nov 12, 2026 •

Copy link
Copy Markdown

Owner: @agoldis

What

The screenshot section in the pull request description shows the Currents session's media instead of only a link:

  • each screenshot inline, from the session's copy
  • the trace screencast as an animated WebP, inline
  • the WebM video as a link
  • the session link, as before

Why

This fork has no image bucket, so the section had no images. GitHub drops <video> tags that point at outside hosts (checked with the /markdown API), so the video can only be a link. The animated screencast is the inline view of what the agent did.

How

  • The recording step reads each attachment's link from the ## Attachments section of the share page's Markdown (the CLI prints no link per file) and writes them to $RUNNER_TEMP/currents-media.tsv.
  • The description step swaps the placeholder image URLs for those links and adds the screencast, video and session lines.
  • COMMIT_INFO_REMOTE uses head_repository.html_url; clone_url is not in the workflow_run payload, so it was empty.

Tested

Ran both snippets from this file in ubuntu:24.04 against the share Markdown of session e046136bb2bbccf3 (PR #3). Output: the screenshot points at s.crts.sh/at/…, a screenshot with no Currents copy is dropped, and the screencast, video and session lines are added. GitHub's image proxy loads both the PNG and the animated WebP.

Links expire after 7 days, so these images break in the PR history after that.

🤖 Generated with Claude Code

https://claude.ai/code/session_011baD3iKo43DsfhRB8dkWCG


Autofix with [code]smith
Open in [code]smith · Need help on this PR? Tag @codesmith-bot with what you need.

…l request

The fork has no image bucket, so the screenshot section showed only a link.
The recording step now reads each attachment's link from the share page's
Markdown, and the description uses them: screenshots inline, the trace
screencast as an animated image, and the WebM as a link (GitHub drops <video>
tags with outside sources). Also sends the head repository's URL as the
commit remote, which was empty.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011baD3iKo43DsfhRB8dkWCG
@agoldis
agoldis merged commit 1fb4afa into main Nov 12, 2026
27 checks passed
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.

1 participant