Get this on every PR automatically. Webhook fires, AI reviews, you approve.
Install on vercel/eve →
// reviewed by pulllight

PullLight found 1 issue

vercel/eve #706 — fix(slack): collect thread attachments across recent message… by @shiminshen Jul 12, 2026
1 high
🔍
// convert to your repo
PullLight caught 1 1 high on this PR.
Install on vercel/eve to get this on every PR automatically — 60 seconds.
Findings
1 issue
high logic
The old code had an intentional early-exit `return []` after the first non-bot message with no files, which prevented scanning further back in the thread. The new code removes that guard and continues scanning all older messages even when a more-recent non-bot message had no attachments. This is a behaviour change: previously, a text-only reply from the user would stop the lookback (treating it as a 'clean slate'), but now the bot will silently pick up files from arbitrarily old messages in the thread. This can cause stale files from a much earlier conversation turn to be unexpectedly re-processed. If the intent is to only look back until the first non-bot message that has no files, the `continue` on line 113 should be a `break` (or the old `return []` should be restored) for messages that are non-bot but have zero parts.
// packages/eve/src/public/channels/slack/attachments.ts:118
model: claude-sonnet-4-5   input: 3141 tokens   output: 308 tokens