Skip to content

fix(cli): guard split()[0] against empty/whitespace strings#23017

Closed
KhanCold wants to merge 3 commits into
NousResearch:mainfrom
KhanCold:fix/cli-split-guard
Closed

fix(cli): guard split()[0] against empty/whitespace strings#23017
KhanCold wants to merge 3 commits into
NousResearch:mainfrom
KhanCold:fix/cli-split-guard

Conversation

@KhanCold

Copy link
Copy Markdown

Summary

Prevent IndexError when text/cmd_lower contains only whitespace.

Changes

  • Added guards for three split()[0] accesses in cli.py

Test Plan

@alt-glitch alt-glitch added type/bug Something isn't working comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have labels May 10, 2026
…t IndexError when text/cmd_lower contains only whitespace.\nAdded guards for three split()[0] accesses.\n\nFixes potential crash in command parsing.
@KhanCold KhanCold force-pushed the fix/cli-split-guard branch from 79ab2d9 to 5e6d361 Compare May 10, 2026 12:06
KhanCold added 2 commits May 11, 2026 07:40
…t IndexError when text/cmd_lower contains only whitespace.\nAdded guards for three split()[0] accesses.\n\nFixes potential crash in command parsing.
@KhanCold KhanCold closed this May 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants