Skip to content

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

Merged
nishu-builder merged 1 commit into
mainfrom
claude/pr-publishing-caos-agent-mil3g8-2-empty-diff
Aug 31, 2026
Merged

Publish PRs from conversations whose workspace diff is empty#165
nishu-builder merged 1 commit into
mainfrom
claude/pr-publishing-caos-agent-mil3g8-2-empty-diff

Conversation

@nishu-builder

Copy link
Copy Markdown
Contributor

Independent PR based on main (replaces #160, which GitHub had locked into a managed stack; merges cleanly in any order with #159 and the .caos/agent.json PR).

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.


Generated by Claude Code

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 b5cc4db to b241c7b Compare August 28, 2026 22:28
@nishu-builder

Copy link
Copy Markdown
Contributor Author

Full caos suite run (caos-cli run dev/run-tests on a stack built from these sources, on the merge of #168 + #165 + #166 over current main): 46/47 passed. The one failure is tests/world-guard, which SPEC documents as expected when the suite runs against a host stack rather than inside the test container.


Generated by Claude Code

@nishu-builder
nishu-builder merged commit a365a55 into main Aug 31, 2026
1 check passed
@nishu-builder
nishu-builder deleted the claude/pr-publishing-caos-agent-mil3g8-2-empty-diff branch August 31, 2026 19:01
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