From 2b9fed3bbdbf5c1681458f9a17017ad431825f8d Mon Sep 17 00:00:00 2001 From: chiri Date: Mon, 19 Jan 2026 19:23:51 +0300 Subject: [PATCH] [`FastAPI`] Document fix safety for `FAST001` (#22655) ## Summary ## Test Plan --- .../rules/fastapi/rules/fastapi_redundant_response_model.rs | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/crates/ruff_linter/src/rules/fastapi/rules/fastapi_redundant_response_model.rs b/crates/ruff_linter/src/rules/fastapi/rules/fastapi_redundant_response_model.rs index d31ffbf61e..a52b1c5bd9 100644 --- a/crates/ruff_linter/src/rules/fastapi/rules/fastapi_redundant_response_model.rs +++ b/crates/ruff_linter/src/rules/fastapi/rules/fastapi_redundant_response_model.rs @@ -58,6 +58,11 @@ use crate::{AlwaysFixableViolation, Fix}; /// async def create_item(item: Item) -> Item: /// return item /// ``` +/// +/// ## Fix safety +/// This fix is always unsafe, as removing the `response_model` argument can change +/// runtime behavior and API documentation generation. Additionally, comments inside +/// the decorator might be removed when the argument is deleted. #[derive(ViolationMetadata)] #[violation_metadata(stable_since = "0.8.0")] pub(crate) struct FastApiRedundantResponseModel;