fix: initialize next_arg with an empty string if match[2] is unde…

[Matobi98]

Description

This PR fixes a bug in the commandArgs middleware where commands sent without arguments resulted in the first argument being the literal string "undefined".

Root Cause

The RegExp.prototype.exec() method expects a string. When no arguments were present, match[2] was undefined. JavaScript coerced this undefined value into the string "undefined", which was then matched as a valid argument by the secondary regex.

Changes

• Updated bot/middleware/commands.ts to initialize the argument parsing string with an empty string '' if match[2] is null/undefined.

Summary by CodeRabbit

• Bug Fixes
  • Improved robustness of command argument parsing to prevent potential failures when processing arguments.

[coderabbitai]

Walkthrough

A single robustness improvement in the command middleware parsing logic: the initial next_arg value now defaults to an empty string when the regex match third element is undefined, preventing potential undefined propagation through subsequent argument parsing operations.

Changes

Cohort / File(s)	Summary
Argument Parsing Robustness bot/middleware/commands.ts	Added null-coalescing fallback (match[2] || '') to ensure next_arg is always a defined string during regex-based argument extraction.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A tiny fix, so neat and small,
One empty string to catch them all,
No undefined shall slip away,
The arguments flow more safely today! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title accurately describes the primary fix: initializing next_arg with an empty string when match[2] is undefined, matching the core change in bot/middleware/commands.ts.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[mostronator]

LGTM ✅ Clean one-liner fix. When match[2] is undefined, the string concatenation would produce the literal string "undefined" which would then be parsed as an argument. Using || '' correctly defaults to an empty string. Good catch.

[grunch]

LGTM