fix clang-tidy noexcept issues

Author: serges147

include/libcyphal/application/node/heartbeat_producer.hpp

@@ -169,7 +169,7 @@ class HeartbeatProducer final  // NOSONAR cpp:S3624
     void publishMessage(const TimePoint approx_now)
     {
         // Publishing of heartbeats makes sense only if the local node ID is known.
-        if (presentation_.transport().getLocalNodeId() == cetl::nullopt)
+        if (!presentation_.transport().getLocalNodeId().has_value())
         {
             return;
         }

include/libcyphal/presentation/client.hpp

@@ -243,7 +243,7 @@ class Client final : public detail::ClientBase
                 {
                     return TooManyPendingRequestsError{};
                 }
-                const auto transfer_id = opt_transfer_id.value();
+                const auto transfer_id = *opt_transfer_id;
 
                 // Create and register a response promise object, which will be used to handle the response.
                 // Its done specifically before sending the request, so that we will be ready to handle a response
@@ -331,7 +331,7 @@ class RawServiceClient final : public detail::ClientBase
         {
             return TooManyPendingRequestsError{};
         }
-        const auto transfer_id = opt_transfer_id.value();
+        const auto transfer_id = *opt_transfer_id;
 
         // 2. Create and register a response promise object, which will be used to handle the response.
         //    Its done specifically before sending the request, so that we will be ready to handle a response

include/libcyphal/presentation/publisher_impl.hpp

@@ -54,7 +54,7 @@ class PublisherImpl final : public common::cavl::Node<PublisherImpl>,
         {
             if (const auto local_node_id = delegate.getLocalNodeId())
             {
-                const SessionSpec session_spec{subject_id_, local_node_id.value()};
+                const SessionSpec session_spec{subject_id_, *local_node_id};
                 next_transfer_id_ = transfer_id_map->getIdFor(session_spec);
             }
         }
@@ -108,7 +108,7 @@ class PublisherImpl final : public common::cavl::Node<PublisherImpl>,
         {
             if (const auto local_node_id = delegate_.getLocalNodeId())
             {
-                const SessionSpec session_spec{subject_id_, local_node_id.value()};
+                const SessionSpec session_spec{subject_id_, *local_node_id};
                 transfer_id_map->setIdFor(session_spec, next_transfer_id_);
             }
         }

include/libcyphal/transport/can/can_transport_impl.hpp

@@ -564,12 +564,11 @@ class TransportImpl final : private TransportDelegate, public ICanTransport
             return;
         }
         const auto& pop_success = cetl::get<IMedia::PopResult::Success>(pop_result);
-        if (!pop_success.has_value())
+        if (!pop_success)
         {
             return;
         }
-
-        const IMedia::PopResult::Metadata& pop_meta = pop_success.value();
+        const IMedia::PopResult::Metadata& pop_meta = *pop_success;
 
         const auto timestamp_us =
             std::chrono::duration_cast<std::chrono::microseconds>(pop_meta.timestamp.time_since_epoch());