feat grpc: enforce move semantics in stream Write

Removed lvalue Write overloads in grpc-proto-structs to prevent unsafe
usage patterns where modified objects could be reused after writes. This
addresses potential bugs when middlewares mutate objects
commit_hash:29b87e821f6a9e64cd810a8fd2ea056eee57756c

Author: ivan-skryabin

libraries/grpc-proto-structs/include/userver/grpc-proto-structs/client/stream.hpp

@@ -72,26 +72,30 @@ class Writer final {
     Writer(Writer&&) = default;
     Writer& operator=(Writer&&) = default;
 
-    /// @brief Write the next outgoing message
-    [[nodiscard]] bool Write(const StructsRequest& request) {
-        return writer_.Write(proto_structs::StructToMessage(request));
-    }
-
-    /// @brief Write the next outgoing message
+    /// @brief Write the next outgoing message.
+    /// @note This version may move some fields from the request.
     [[nodiscard]] bool Write(StructsRequest&& request) {
         return writer_.Write(proto_structs::StructToMessage(std::move(request)));
     }
 
-    /// @brief Write the next outgoing message and check result
-    void WriteAndCheck(const StructsRequest& request) {
-        writer_.WriteAndCheck(proto_structs::StructToMessage(request));
+    /// @brief Write the next outgoing message.
+    /// @note This version preserves the original request object by copying necessary data.
+    [[nodiscard]] bool WriteCopy(const StructsRequest& request) {
+        return writer_.Write(proto_structs::StructToMessage(request));
     }
 
-    /// @brief Write the next outgoing message and check result
+    /// @brief Write the next outgoing message and check result.
+    /// @note This version may move some fields from the request.
     void WriteAndCheck(StructsRequest&& request) {
         writer_.WriteAndCheck(proto_structs::StructToMessage(std::move(request)));
     }
 
+    /// @brief Write the next outgoing message and check result.
+    /// @note This version preserves the original request object by copying necessary data.
+    void WriteCopyAndCheck(const StructsRequest& request) {
+        writer_.WriteAndCheck(proto_structs::StructToMessage(request));
+    }
+
     /// @brief Complete the RPC successfully
     StructsResponse Finish() { return proto_structs::MessageToStruct<StructsResponse>(writer_.Finish()); }
 

libraries/grpc-proto-structs/include/userver/grpc-proto-structs/server/stream.hpp

@@ -65,23 +65,32 @@ class Writer {
 
     /// @{
     /// @brief Write the next outgoing message.
+    /// @note This version may move some fields from the request.
     ///
     /// Convert response to corresponding protobuf message and pass it to @ref ugrpc::server::Writer::Write.
     ///
     /// @see @ref ugrpc::server::Writer::Write method for details.
-    void Write(Response& response) { writer_.Write(proto_structs::StructToMessage(response)); }
-
-    void Write(Response& response, const grpc::WriteOptions& options) {
-        writer_.Write(proto_structs::StructToMessage(response), options);
-    }
-
     void Write(Response&& response) { writer_.Write(proto_structs::StructToMessage(std::move(response))); }
 
     void Write(Response&& response, const grpc::WriteOptions& options) {
         writer_.Write(proto_structs::StructToMessage(std::move(response)), options);
     }
     /// @}
 
+    /// @{
+    /// @brief Write the next outgoing message.
+    /// @note This version preserves the original request object by copying necessary data.
+    ///
+    /// Convert response to corresponding protobuf message and pass it to @ref ugrpc::server::Writer::Write.
+    ///
+    /// @see @ref ugrpc::server::Writer::Write method for details.
+    void WriteCopy(const Response& response) { writer_.Write(proto_structs::StructToMessage(response)); }
+
+    void WriteCopy(const Response& response, const grpc::WriteOptions& options) {
+        writer_.Write(proto_structs::StructToMessage(response), options);
+    }
+    /// @}
+
 private:
     ProtobufMessageWriter& writer_;
 };