Skip to content

初期コミット内容のレビュー

https://gitlab.com/piroor/alert-on-paste-private-urls/-/blob/main/content_scripts/content.js#L47 https://gitlab.com/piroor/alert-on-paste-private-urls/-/blob/main/content_scripts/content.js#L50

patedpastedの誤字と思われます。

https://gitlab.com/piroor/alert-on-paste-private-urls/-/blob/main/content_scripts/content.js#L52

text.matchはマッチするものがなかった場合nullを返すので、const urls = text.match(/https?:\/\/[^\s]+/gim);で一致しなかった場合、ここでnullアクセスでエラーになりそうです。(エラーになった結果、「何のダイアログも出ない」という結果の期待値は正しくなりそうですが)

https://gitlab.com/piroor/alert-on-paste-private-urls/-/blob/main/content_scripts/content.js#L49

余分な文字列を取得してしまうケースがありそうです。例えば[この行](https://gitlab.com/piroor/alert-on-paste-private-urls/-/blob/main/content_scripts/content.js#L47)を参照してください。というような文字列をコピペしたときに、ただしくないURLがアラートに抽出されそうな気がします(https://gitlab.com/piroor/alert-on-paste-private-urls/-/blob/main/content_scripts/content.js#L47)を参照してください。が抽出されそう)。ただ、警告したいURL自体は正しく判定できそうなので、このままでも問題なさそうに思いました。