This let the bot only respond to comments which contain a bot command. If so it will either process the command if the sender is authorised to send commands to the bot or it will just respond with a note that the sender is not authorised.
The PR essentially adds a new function contains_any_bot_command and rearranges the processing in handle_issue_comment_event by first checking if the comment was newly created (only newly created comments are considered for processing bot commands), then checking if it contains any bot command, then proceeding with the processing of the commands (incl checking if the sender has permission to send a bot command).
This let the bot only respond to comments which contain a bot command. If so it will either process the command if the sender is authorised to send commands to the bot or it will just respond with a note that the sender is not authorised.
The PR has been tested by letting two different accounts (one authorised, one not authorised) send comments with and without bot commands. See comment https://github.com/trz42/software-layer/pull/72#issuecomment-1965222781 and below.
The PR essentially adds a new function
contains_any_bot_commandand rearranges the processing inhandle_issue_comment_eventby first checking if the comment was newly created (only newly created comments are considered for processing bot commands), then checking if it contains any bot command, then proceeding with the processing of the commands (incl checking if the sender has permission to send a bot command).