Skip to content

[deep-report] Sweep gh ... | head -n N SIGPIPE pattern across all workflow lock files #33788

Closed as not planned

Description

@github-actions

Context

#33769 identified that Matt Pocock Skills Reviewer fails its Pre-fetch PR diff step with exit code 141 (SIGPIPE) when the PR diff exceeds 3000 lines. The root cause is the set -o pipefail + gh pr diff ... | head -n 3000 shape: head closes its stdin early, gh receives SIGPIPE on its next write, and pipefail propagates 141.

That fix patches one workflow. The shape is likely present in other generated .lock.yml files — exactly the workflows that should be most defensive against large diffs.

Quick-win task

  1. grep -rnE '\| (head|tail)( |$)' .github/workflows/*.lock.yml .github/workflows/*.md and triage hits.
  2. For each match that runs under set -o pipefail, apply the same pattern: either { cmd || true; } | head -n N, or write to a temp file and truncate.
  3. Patch source .md files (not .lock.yml) and re-compile.

Success criteria

  • All | head / | tail usages either are not under pipefail or tolerate SIGPIPE.
  • No new step-exit-141 failures from large-input branches in the next 7 days.

References

  • Sibling issue: #33769
  • Source audit: [DeepReport 2026-05-21]
  • Estimated effort: < 2h (one PR touching ~3-6 workflows)

Generated by 🔬 DeepReport - Intelligence Gathering Agent · ● 17.9M · ◷

  • expires on May 23, 2026, 4:03 PM UTC

Activity

  1. github-actions commented on May 23, 2026

    @github-actions
    ContributorAuthor

    This issue was automatically closed because it expired on 2026-05-23T16:03:20.377Z.

    Closed by Workflow

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions