Skip to content

fix(ci): skip docs-only changes for fuzz workflow - #94

Closed
seonghobae wants to merge 3 commits into
mainfrom
chore/paths-ignore-fix
Closed

seonghobae wants to merge 3 commits into
mainfrom
chore/paths-ignore-fix

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 3, 2026 •

Copy link
Copy Markdown
Contributor

What

The non-required fuzz workflow now skips documentation-only push/PR changes using docs/** plus **.md.

The recursive Markdown glob is deliberate. GitHub workflow filters treat * as not crossing /; the initial *.md only covered repository-root Markdown outside docs/. **.md covers Markdown at any repository depth, matching the stated docs-only optimization.

tests/test_ci_path_filter_contract.py is an executable regression requiring the recursive filter on both push and pull-request triggers and rejecting the root-only pattern.

Why

The Hypothesis and bounded Atheris fuzz jobs are useful for code changes but should not consume runner capacity for documentation-only changes. This is a queue-load optimization for an optional local workflow, not a required-gate shortcut.

Safety / acceptance

  • main required checks remain API integration suite, Analyze (python), and Analyze (actions); none is produced by .github/workflows/fuzz.yml.
  • The central required-workflow ruleset does not require this local fuzz workflow.
  • Any push/PR with at least one path outside docs/** and **.md still triggers fuzzing.
  • The regression test is itself a non-document change, so this PR's exact head remains eligible for repository CI/fuzz validation.
  • Merge requires terminal exact-head GREEN under then-live checks/reviews. Queued, pending, skipped-required, startup-failed, or predecessor evidence is non-passing.

Reference: https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#filter-pattern-cheat-sheet

Summary by CodeRabbit

  • 기타

    • 저장소 내 모든 경로의 마크다운 파일만 변경된 경우 퍼즈 테스트 워크플로가 실행되지 않도록 조정했습니다.
    • 문서 변경으로 인한 불필요한 자동 검증 실행을 줄였습니다.
  • 테스트

    • 다양한 디렉터리 깊이의 마크다운 변경이 퍼즈 워크플로를 건너뛰는지 검증하는 테스트를 추가했습니다.

Org-wide audit found fuzz.yml lacked paths-ignore, triggering full CI on every docs-only push/PR and contributing to Actions queue congestion across the org.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 3, 2026 •

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

Next included review available in 41 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 811c27a4-6ea6-408f-9221-c660a4976379

📥 Commits

Reviewing files that changed from the base of the PR and between 49abcc9 and 47b33a2.

📒 Files selected for processing (1)
  • tests/test_ci_path_filter_contract.py
📝 Walkthrough

Walkthrough

퍼즈 워크플로의 push 및 pull_request 트리거가 모든 디렉터리 깊이의 Markdown 변경을 무시하도록 수정되었습니다. 이 동작을 검증하는 계약 테스트가 추가되었습니다.

Changes

퍼즈 워크플로 트리거

Layer / File(s) Summary
트리거 경로 필터와 계약 테스트
.github/workflows/fuzz.yml, tests/test_ci_path_filter_contract.py
push 및 pull_request 트리거의 Markdown 제외 패턴을 **.md로 변경했습니다. 계약 테스트는 해당 패턴이 두 번 사용되고 *.md가 제거되었는지 확인합니다.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: 🔵 Low · up to 49abc

Docs-only changes are intended to skip fuzzing on both pushes and pull requests. The current behavior is present, but the regression test does not independently protect each trigger or the docs exclusion, leaving bounded CI-configuration drift risk.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (1 skipped: 1 …
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 fuzz workflow에서 문서 전용 변경을 건너뛰도록 CI 경로 필터를 수정한 내용을 정확하게 설명합니다. 실제 변경의 주요 목적과 일치합니다.
Full details: Docstring Coverage

Explanation

Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/paths-ignore-fix

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

devin-ai-integration[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

coderabbitai[bot]

This comment was marked as resolved.

@seonghobae

Copy link
Copy Markdown
Contributor Author

Superseded by #93, which includes the docs-only fuzz filter with the repository-wide concurrency audit.

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