Skip to content

fix: skip broken untracked symlinks in review context#66

Open
zylar06 wants to merge 2 commits intoopenai:mainfrom
zylar06:fix/skip-broken-untracked-symlinks
Open

fix: skip broken untracked symlinks in review context#66
zylar06 wants to merge 2 commits intoopenai:mainfrom
zylar06:fix/skip-broken-untracked-symlinks

Conversation

@zylar06
Copy link
Copy Markdown

@zylar06 zylar06 commented Mar 31, 2026

Summary

  • skip broken or unreadable untracked paths during working-tree review context collection
  • keep review context generation from crashing on broken untracked symlinks
  • add a regression test covering broken untracked symlinks

Closes #65

Testing

  • node --test tests/git.test.mjs

@zylar06 zylar06 requested a review from a team March 31, 2026 10:40
@stoneHee99
Copy link
Copy Markdown

Heads up — my PR #48 also modifies formatUntrackedFile in git.mjs (adds a stat.isDirectory() guard for issue #14). Since both PRs touch the same function, there will likely be a merge conflict depending on which one lands first. Just flagging so the maintainers can coordinate. 👍

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.

Review context collection crashes on broken untracked symlinks

2 participants