Skip to content

Add per-package diffs to update PR body - #173

Draft
samestep wants to merge 2 commits into
mainfrom
update-package-diffs
Draft

Add per-package diffs to update PR body#173
samestep wants to merge 2 commits into
mainfrom
update-package-diffs

Conversation

@samestep

@samestep samestep commented Jul 6, 2026

Copy link
Copy Markdown
Owner

No description provided.

samestep and others added 2 commits July 5, 2026 10:46
Build the current and updated closures for each machine and run
`nix store diff-closures` to produce the same fine-grained version
diffs that `nh {os,home} switch` shows, then include them in the
update PR body (and the run's step summary).

Also adds a `pull_request` trigger scoped to this workflow so the
rendered Markdown can be previewed in the step summary before merge;
on PR previews the create-pull-request step is skipped.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015MdeUZ7q1EdciK4Bax5RYu
`nh` renders its package diffs with dix, so use the same tool here for
output matching what `nh {os,home} switch` shows. dix is packaged in
nixpkgs, and `--color never` keeps ANSI escape codes out of the diff
artifacts -- unlike `nix store diff-closures`, which emits color even
when writing to a pipe.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015MdeUZ7q1EdciK4Bax5RYu
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