Skip to content

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

Open
terraboops wants to merge 1 commit into
mainfrom
share-diff
Open

terraboops wants to merge 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
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