Peter Marsh

Results 12 comments of Peter Marsh

Here's all the cases: # normal chat ![image](https://github.com/ScottLogic/prompt-injection/assets/118171430/ce105af9-19a1-4d9a-a3c6-125973975712) # chat with transformation ![image](https://github.com/ScottLogic/prompt-injection/assets/118171430/54979cd2-d612-44bb-ab12-5cb2d0bcbfbe) # win level ![image](https://github.com/ScottLogic/prompt-injection/assets/118171430/cf8ca5dc-28c8-4a1e-9bf9-39bfbf8f9f8d) # defence trigger ![image](https://github.com/ScottLogic/prompt-injection/assets/118171430/31701cbb-08ea-4c68-b1d3-3b037b55bc08) # defence alert ![image](https://github.com/ScottLogic/prompt-injection/assets/118171430/6e394633-8267-4eab-97b9-601345fb2dc2) # openAI error paste the...

would like to do this after #761

- chat controller unit tests missing tests for alerted but not triggered defences

Guess we all need to be in agreement about which content needs to be headered, and which level each header should be at before any implemention...

On further research, this is default and expected behaviour of radio buttons. It is unfortunate that changing the option triggers a chat message and a network call. If we want...

Another idea is we could send the network request and add the info message on blur. So: - Keyboard naviage through the defences to get to focus the prompt enclosure...

This is likely to conflict with #705 if done at the same time

Same comments apply here as in the [testing comment for 705](https://github.com/ScottLogic/prompt-injection/issues/705#issuecomment-1906364108).

> I see typescript enums being used in a few places in the code, e.g. DEFENCE_TYPES. > > There is often a better way to achieve this using the power...

I'll leave it to @chriswilty to defend the ticket :)