Да, вы абсолютно правы, что это предложение вводит изменение кода - по сути, это ошибка. В конечном счете, нет встроенного декларативного способа сказать, что метод в .NET является чистой функцией. В некоторых случаях такое изменение будет совершенно корректным, тогда как в других (например, таких) оно будет плохим, и нет способа узнать, что это такое, в любом случае (без запуска поддерживаемого вручную чита - список времен, когда это известно - плохо или известно - хорошо). Итак: он предлагает эту функциональность для тех случаев, когда это будет полезно, без навязывания ее вам (она не отображается как рекомендуемое изменение, видимое по умолчанию, например - только появляется, когда вы активно выбираете переменную).
Итак, в данном случае: не используйте его.