Skip to content

share: upload a .diff/.patch as a diff pin, with --title and --desc - #5

Merged
terraboops merged 1 commit into
mainfrom
share-diff
Oct 28, 2026
Merged

terraboops merged 1 commit into
mainfrom
share-diff

Conversation

@terraboops

Copy link
Copy Markdown
Collaborator

pin share change.diff --title T --desc D uploads a patch as a diff pin (text/x-diff, X-Pin-Title, X-Pin-Description base64), which pin-api has rendered via diffview all along. Before this the CLI sent only HTML or MDX, and a .diff got 415.

Proven on production with this build: a diff pin described See (https://example.org/). Also see (https://en.wikipedia.org/wiki/Foo_(bar)). renders href="https://example.org/" and href="https://en.wikipedia.org/wiki/Foo_(bar)". That's pin#47's goldmark 1.8.6 linkify fix working on the deployed system. go vet and go test pass.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Sn3acTQsYfyvVidkmhCC5j

The server has rendered text/x-diff with an X-Pin-Title banner and an X-Pin-Description PR
description since diffview landed, but the CLI only sent HTML or MDX (a .diff got 415), so a diff
pin needed a hand-rolled request with a bearer token. --desc travels base64-encoded, as the server
documents, so it may span lines.

Used 2026-10-02 to prove pin#47 (goldmark 1.8.6) on production: a diff pin described
"See (https://example.org/)." renders href="https://example.org/", not the old "…/)".

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Sn3acTQsYfyvVidkmhCC5j
@terraboops
terraboops merged commit 5a788a1 into main Oct 28, 2026
1 check 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