Skip to content

fix: SC2038 — use find -print0 | xargs -0 in workflow scripts - #49756

Merged
pelikhan merged 2 commits into
mainfrom
copilot/static-analysis-report-2026-08-02
Aug 2, 2026
Merged

fix: SC2038 — use find -print0 | xargs -0 in workflow scripts#49756
pelikhan merged 2 commits into
mainfrom
copilot/static-analysis-report-2026-08-02

Conversation

Copilot AI commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Six find | xargs pipelines in workflow scripts lacked null-delimiter safety, causing silent argument mis-splitting on filenames with spaces or special characters (SC2038/actionlint).

Changes

  • repository-quality-improver.md (3 sites): find ... | xargs wc -lfind ... -print0 | xargs -0 wc -l
  • spec-extractor.md (1 site): find ... | xargs grep -hfind ... -print0 | xargs -0 grep -h
  • unbloat-docs.md (2 sites): -print | xargs grep -rL-print0 | xargs -0 grep -rL
  • Recompiled all three workflows; corresponding .lock.yml files updated
# Before
find . -type f -name "*.go" ! -name "*_test.go" ! -path "./.git/*" | xargs wc -l

# After
find . -type f -name "*.go" ! -name "*_test.go" ! -path "./.git/*" -print0 | xargs -0 wc -l

Copilot AI linked an issue Aug 2, 2026 that may be closed by this pull request
Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update static analysis report for 2026-08-02 fix: SC2038 — use find -print0 | xargs -0 in workflow scripts Aug 2, 2026
Copilot AI requested a review from pelikhan August 2, 2026 10:53
@pelikhan
pelikhan marked this pull request as ready for review August 2, 2026 11:20
Copilot AI review requested due to automatic review settings August 2, 2026 11:20
@pelikhan
pelikhan merged commit 8d0ce55 into main Aug 2, 2026
@pelikhan
pelikhan deleted the copilot/static-analysis-report-2026-08-02 branch August 2, 2026 11:20

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes SC2038 findings from #49714 by making six find | xargs pipelines null-delimiter safe.

Changes:

  • Adds -print0 and xargs -0 across three workflows.
  • Regenerates the corresponding compiled workflow files.
Show a summary per file
File Description
.github/workflows/repository-quality-improver.md Makes Go metric collection filename-safe.
.github/workflows/repository-quality-improver.lock.yml Updates compiled workflow output.
.github/workflows/spec-extractor.md Makes import extraction filename-safe.
.github/workflows/spec-extractor.lock.yml Updates compiled workflow output.
.github/workflows/unbloat-docs.md Makes documentation filtering filename-safe.
.github/workflows/unbloat-docs.lock.yml Updates compiled workflow output.

Review details

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 6/6 changed files
  • Comments generated: 0
  • Review effort level: Balanced

@github-actions

github-actions Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

🎉 This pull request is included in a new release.

Release: v0.84.3

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.

[static-analysis] Report - 2026-08-02

3 participants