Skip to content

Fix diff-mode returning empty changed files#15

Merged
thejefflarson merged 1 commit intomainfrom
fix/diff-mode-empty
Apr 24, 2026
Merged

Fix diff-mode returning empty changed files#15
thejefflarson merged 1 commit intomainfrom
fix/diff-mode-empty

Conversation

@thejefflarson
Copy link
Copy Markdown
Owner

Summary

  • git diff --name-only --diff-filter=ACMR -- <sha> treats <sha> as a path, not a ref — always returns empty
  • Every PR- and push-triggered soundcheck run has been short-circuiting with 'No changed files. Nothing to review.'
  • Drop the -- so the SHA is parsed as a commit ref

Test plan

  • git diff --name-only --diff-filter=ACMR HEAD~1 returns the changed files (verified locally)
  • git diff --name-only --diff-filter=ACMR -- HEAD~1 returns empty (verified locally)

🤖 Generated with Claude Code

git diff --name-only --diff-filter=ACMR -- <sha> treats <sha> as a
path (not a ref) because of the --. The match returns empty, so
diff-mode always short-circuited with 'No changed files. Nothing to
review.' Every PR-triggered and push-triggered run under the action
was a silent no-op.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@thejefflarson thejefflarson merged commit f4e386a into main Apr 24, 2026
3 checks passed
@thejefflarson thejefflarson deleted the fix/diff-mode-empty branch April 24, 2026 08:56
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