chore(deps): bump tj-actions/changed-files from 47.0.5 to 47.0.6#344
Open
dependabot[bot] wants to merge 1 commit into
Open
chore(deps): bump tj-actions/changed-files from 47.0.5 to 47.0.6#344dependabot[bot] wants to merge 1 commit into
dependabot[bot] wants to merge 1 commit into
Claude / Claude Code Review
completed
Apr 20, 2026 in 8m 50s
Code review found 4 potential issues
Found 5 candidates, confirmed 4. See review comments for details.
Details
| Severity | Count |
|---|---|
| 🔴 Important | 0 |
| 🟡 Nit | 0 |
| 🟣 Pre-existing | 3 |
| Severity | File:Line | Issue |
|---|---|---|
| 🟣 Pre-existing | .github/workflows/auto-label.yml:18-21 |
Missing github-actions label causes Dependabot PRs to fail labeling |
| 🟣 Pre-existing | .github/workflows/docs-check.yml:22-28 |
docs-check.yml: path trigger and skip-check mechanism are both broken |
| 🟣 Pre-existing | .github/workflows/auto-label.yml:22-24 |
Catch block in label creation does not check error type |
Annotations
Check notice on line 21 in .github/workflows/auto-label.yml
claude / Claude Code Review
Missing github-actions label causes Dependabot PRs to fail labeling
This is a pre-existing issue: `.github/dependabot.yml` specifies the `github-actions` label for the github-actions package ecosystem, but this label does not exist in the repository. The `auto-label.yml` workflow's color map (which drives auto-creation) also does not include `github-actions`, so the label is never created — causing every Dependabot GitHub Actions PR (including this one) to emit the error: *'The following labels could not be found: github-actions'*. Fix: add `'github-actions': 'X
Check notice on line 28 in .github/workflows/docs-check.yml
claude / Claude Code Review
docs-check.yml: path trigger and skip-check mechanism are both broken
This is a pre-existing issue: the docs-check.yml workflow has two related logic bugs that render the documentation check completely non-functional. The on.pull_request.paths trigger only watches docs/**, README.md, and CHANGELOG.md — it never fires for src-only PRs, so the check-docs-updated job can never warn about missing documentation for source changes. Additionally, adding [skip-docs-check] to the PR title does not suppress the warning because the warning step runs before the skip step, emi
Check notice on line 24 in .github/workflows/auto-label.yml
claude / Claude Code Review
Catch block in label creation does not check error type
**Pre-existing issue:** The label existence check in `.github/workflows/auto-label.yml` uses a bare `catch (e)` block without checking `e.status === 404`, meaning any transient error (rate limit 429, permission error 403, server error 500) from `getLabel` will incorrectly trigger a `createLabel` call. The fix is to rethrow non-404 errors: `catch (e) { if (e.status \!== 404) throw e; ... createLabel ... }`.
Loading