๐ ๏ธ Refactor suggestion
Return a success indicator
simulatePasteCommand() never tells callers if the operation actually succeeded.
Return a Promise<boolean> that resolves when the spawned process exits; propagate non-zero codes.
๐ค Prompt for AI Agents
In src/main/utils/simulate-paste-command.ts around lines 36 to 50, the
simulatePasteCommand function currently does not return any indication of
success or failure to its callers. Modify the function to return a
Promise<boolean> that resolves to true if the spawned process exits with code 0,
and resolves to false or rejects if the exit code is non-zero or an error
occurs. Wrap the spawn and event handling logic inside this Promise to properly
propagate the success status.
Originally posted by @coderabbitai[bot] in #290 (comment)
๐ ๏ธ Refactor suggestion
Return a success indicator
simulatePasteCommand()never tells callers if the operation actually succeeded.Return a
Promise<boolean>that resolves when the spawned process exits; propagate non-zero codes.๐ค Prompt for AI Agents
Originally posted by @coderabbitai[bot] in #290 (comment)