Skip to content

fix(exec): stdout errors propagate instead of panicking - #16

Merged
gold-silver-copper merged 4 commits into
mainfrom
fix/meta-stable-dotmeta
Oct 28, 2026
Merged

gold-silver-copper merged 4 commits into
mainfrom
fix/meta-stable-dotmeta

Conversation

@gold-silver-copper

@gold-silver-copper gold-silver-copper commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor
  • A write to stdout that fails (EPIPE from meta exec -- seq 1 20000 | head -1) returns a StdoutError instead of panicking; meta exits quietly (141) on it.
  • loop's own lines (dry-run, "Failed to run …", the failure summary, no-directory, a malformed .looprc alias) show a command escaped; JSON output keeps the real argv.

Part of the meta stability set: gitkb/meta#128 has the full description and how it was verified. Merge together with it (same branch name, so CI builds them together).

Implements [[tasks/harmony-2793]]

Keep stdout write failures distinct from child stdin errors so callers can handle a closed output pipe quietly.

[[tasks/harmony-2793]]
@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 3a3f7ac8-f832-4207-9557-58367b52b4dc
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@gold-silver-copper
gold-silver-copper merged commit c939885 into main Oct 28, 2026
7 checks passed
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