fix(ploeg): keep a valid Claude review when its stdout is malformed #145

Merged
ryangr0 merged 1 commit from ryangr0/ploeg-claude-outcome-preservation into development 2026-10-03 00:03:05 +00:00 AGit
Owner

The Claude Code adapter returned the drop box together with the envelope
decode error, and the generic runner discards every report field on a parse
error, so a banner or truncated JSON on stdout lost a finished review. When
the drop box holds an outcome or verdict it is now returned without the
error; an empty drop box still reports the decode error. Based on the
2026-10-02 Execution Pack patch.

Differs from the pack's patch: the error is suppressed only when the drop box actually holds an outcome or verdict, so a malformed envelope with no drop box still logs its decode error.

Verified with mise run verify on the pinned toolchain (all gates passed).

Ticket: https://vikunja.webgrip.dev/tasks/1746

🤖 Generated with Claude Code

The Claude Code adapter returned the drop box together with the envelope decode error, and the generic runner discards every report field on a parse error, so a banner or truncated JSON on stdout lost a finished review. When the drop box holds an outcome or verdict it is now returned without the error; an empty drop box still reports the decode error. Based on the 2026-10-02 Execution Pack patch. Differs from the pack's patch: the error is suppressed only when the drop box actually holds an outcome or verdict, so a malformed envelope with no drop box still logs its decode error. Verified with `mise run verify` on the pinned toolchain (all gates passed). Ticket: https://vikunja.webgrip.dev/tasks/1746 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix(ploeg): keep a valid Claude review when its stdout is malformed
Some checks failed
[Workflow] On Pull Request / checks (pull_request) Has been cancelled
[Workflow] On Pull Request / warnings (pull_request) Has been cancelled
[Workflow] On Pull Request / release-policy (pull_request) Has been cancelled
4497c86164
The Claude Code adapter returned the drop box together with the envelope
decode error, and the generic runner discards every report field on a parse
error, so a banner or truncated JSON on stdout lost a finished review. When
the drop box holds an outcome or verdict it is now returned without the
error; an empty drop box still reports the decode error. Based on the
2026-10-02 Execution Pack patch.

VIK-1746

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
ryangr0 merged commit c046898f89 into development 2026-10-03 00:03:05 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
webgrip/unfold!145
No description provided.