Skip to content

Publish PRs from conversations whose workspace diff is empty - #160

Closed
nishu-builder wants to merge 1 commit into
claude/pr-publishing-caos-agent-mil3g8-1-statusfrom
claude/pr-publishing-caos-agent-mil3g8-2-empty-diff
Closed

Publish PRs from conversations whose workspace diff is empty#160
nishu-builder wants to merge 1 commit into
claude/pr-publishing-caos-agent-mil3g8-1-statusfrom
claude/pr-publishing-caos-agent-mil3g8-2-empty-diff

Conversation

@nishu-builder

@nishu-builder nishu-builder commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Independent PR based on main (restacked from the earlier chain; merges cleanly in any order with #159 and #161).

Ctrl+P refused any conversation whose base-to-head patch was empty with "there are no conversation changes to publish". That blocked real publications: a conversation whose changes already landed in the base branch, or one whose value is the transcript itself. /publish-branch already accepts both, for the same reason.

The gate now only requires a completed turn to point the branch at — the same "no completed turn to publish" rule /publish-branch uses — and an empty diff proceeds to the base prompt as usual. The publish pipeline itself needed no change: it operates on the conversation head, not the patch.

Validated with cargo test -p caos-cli on this branch (132 tests, two new: empty diff opens the base prompt; no completed turn still errors) and cargo fmt --check.

Ctrl+P refused any conversation whose base-to-head patch was empty with
"there are no conversation changes to publish". That blocked real
publications: a conversation whose changes already landed in the base,
or one whose value is the transcript itself. /publish-branch already
accepts these for the same reason.

The gate now only requires a completed turn to point the branch at,
matching /publish-branch's "no completed turn" rule; an empty diff
proceeds to the base prompt as usual.

Co-Authored-By: Claude <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_019FKeyxNMPfqcUR2Ts9ghqq
@nishu-builder
nishu-builder force-pushed the claude/pr-publishing-caos-agent-mil3g8-2-empty-diff branch from 0d5bf65 to b5cc4db Compare August 28, 2026 21:44
@nishu-builder

Copy link
Copy Markdown
Contributor Author

Closing to re-open this against main: GitHub grouped the original chain into a managed stack and refuses base-branch changes on it. Same head branch and content; replacement PR link to follow.


Generated by Claude Code

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.

2 participants