fix: address review feedback on C ABI codec registry

- codec_name_to_cstring: use expect() instead of unwrap_or_default()
  to fail fast on programmer errors (matches lib.rs macro pattern)
- Add tracing::warn! when EncodedVideo null custom_type_id falls back
  to Binary (aids debugging of new plugins vs backward compat)
- Simplify map_err to forward original from_c_name() error messages
- Add doc comments clarifying intentional field loss (bitstream_format,
  codec_private, profile, level) through C ABI pin-type declarations

Signed-off-by: StreamKit Devin <devin@streamkit.dev>
Co-Authored-By: Claudio Costa <cstcld91@gmail.com>

Author: streamkit-devin

sdks/plugin-sdk/native/src/conversions.rs

@@ -74,8 +74,8 @@ pub fn packet_type_from_c(cpt_info: CPacketTypeInfo) -> Result<PacketType, Strin
                 Ok(PacketType::Binary)
             } else {
                 let name = unsafe { c_str_to_string(cpt_info.custom_type_id) }?;
-                let codec = VideoCodec::from_c_name(&name)
-                    .map_err(|e| format!("EncodedVideo: {e}"))?;
+                let codec =
+                    VideoCodec::from_c_name(&name).map_err(|e| format!("EncodedVideo: {e}"))?;
                 // Note: bitstream_format, codec_private, profile, and level
                 // are not carried through the C ABI — this conversion is used
                 // for pin-type declarations only, not runtime packet data.
@@ -96,8 +96,7 @@ pub fn packet_type_from_c(cpt_info: CPacketTypeInfo) -> Result<PacketType, Strin
                 AudioCodec::Opus
             } else {
                 let name = unsafe { c_str_to_string(cpt_info.custom_type_id) }?;
-                AudioCodec::from_c_name(&name)
-                    .map_err(|e| format!("EncodedAudio: {e}"))?
+                AudioCodec::from_c_name(&name).map_err(|e| format!("EncodedAudio: {e}"))?
             };
             // Note: codec_private is not carried through the C ABI — this
             // conversion is used for pin-type declarations only.