Description
Multiple locations access str.split()[0] without guarding against the case where split() returns an empty list (e.g., whitespace-only or empty strings). This causes an unhandled IndexError.
Affected Files
| File |
Line |
Code |
tools/file_operations.py |
814 |
linter_cmd.split()[0] |
gateway/platforms/slack.py |
797 |
text.split()[0] if text else "" — doesn't handle whitespace-only text |
agent/anthropic_adapter.py |
79 |
result.stdout.strip().split()[0] |
Steps to Reproduce
- In
file_operations.py: configure a linter command as an empty string
- In
slack.py: send a Slack message with only whitespace
- In
anthropic_adapter.py: have claude --version return empty/whitespace output
Fix
Guard each split()[0] with a check that the resulting list is non-empty before indexing.
Description
Multiple locations access
str.split()[0]without guarding against the case wheresplit()returns an empty list (e.g., whitespace-only or empty strings). This causes an unhandledIndexError.Affected Files
tools/file_operations.pylinter_cmd.split()[0]gateway/platforms/slack.pytext.split()[0] if text else ""— doesn't handle whitespace-onlytextagent/anthropic_adapter.pyresult.stdout.strip().split()[0]Steps to Reproduce
file_operations.py: configure a linter command as an empty stringslack.py: send a Slack message with only whitespaceanthropic_adapter.py: haveclaude --versionreturn empty/whitespace outputFix
Guard each
split()[0]with a check that the resulting list is non-empty before indexing.