Command Sender Fails With Unclear Error when Single Quotes are Used

[shonigmann]

Describe the bug

A member on our team recently ran into issues where the Command Sender would reject cmd() calls using single quotes instead of double quotes. The team member expected both single and double quotes to be treated equivalently and was surprised to see errors confusingly specifying that not enough inputs are provided (e.g. Error sending undefined due to RuntimeError: ERROR: Invalid number of arguments (0) passed to get_cmd_hazardous()).

The syntax highlighting appears treat both single and double quotes as strings, and we expect python and ruby to treat simple strings with single or double quotes equivalently as well

From claude's review of 7.2.0:

The bug is clear. At CommandSender.vue:401-403, when stripping the cmd("...") wrapper, the code only looks for double quotes ("). With single quotes, indexOf('"') returns -1, so firstQuote + 1 = 0 and lastQuote - firstQuote - 1 = -1 - (-1) - 1 = -1... 
let me verify what substr(0, -1) does:
  - firstQuote = -1 → firstQuote + 1 = 0
  - lastQuote = -1 → lastQuote - firstQuote - 1 = -1 - (-1) - 1 = -1
  - command.substr(0, -1) → empty string ""
So sendCmd("") is called, which explains why get_cmd_hazardous() gets 0 arguments — the command string is empty.
The fix is to handle single quotes in the same block

To Reproduce

cmd('SC NOOP')

fails

cmd("SC NOOP")

succeeds

Expected behavior

either:

• both single and double quotes are supported
• syntax highlighting or error messages clearly inform users that single quotes are not acceptable

Screenshots

No response

OS

No response

OpenC3 COSMOS Version

6.4.2 and 7.2.0

Browser

No response

[crawlchat]

Great — the bug is confirmed and well-understood. Here's a concise summary and a suggested fix:

---

🐛 Bug Confirmed — Single Quotes Not Supported in cmd() Wrapper

The root cause is in CommandSender.vue at lines 407–409. The code that strips the cmd("...") wrapper only looks for double quotes ("):

// CommandSender.vue ~L406-409
let firstQuote = command.indexOf('"')
let lastQuote = command.lastIndexOf('"')
command = command.substr(firstQuote + 1, lastQuote - firstQuote - 1)

When a single-quoted string like cmd('SC NOOP') is entered, indexOf('"') returns -1, causing command.substr(0, -1) which results in an empty string — hence the confusing 0 arguments error.

---

Suggested Fix:

Replace the hardcoded double-quote lookup with logic that handles both quote types:

// Detect which quote type is used
const quoteChar = command.includes('"') ? '"' : "'"
let firstQuote = command.indexOf(quoteChar)
let lastQuote = command.lastIndexOf(quoteChar)
command = command.substr(firstQuote + 1, lastQuote - firstQuote - 1)

⚠️ Note: The queueMatch regex on line 378 also only handles double-quoted queue names ("([^"]+)"), so if single-quote support is added broadly, that regex may need updating too.

---

Could one of the engineers confirm whether the intent is to fully support single quotes or to surface a clearer error when single quotes are used? The fix above enables single-quote support, but improving the error message (e.g., "Use double quotes in cmd() calls") could also be a valid alternative if single quotes are intentionally unsupported.