feat(chrono): refactor chrono support and fuzz testing

- Separate chrono and time feature support by moving chrono-specific code into dedicated modules/files and removing the shared Date component
- Implement conversion from DateTimeDecode to chrono::NaiveDateTime and remove the previous conversion from DateTimeEncode
- Remove support for chrono::DateTime<FixedOffset>
- Add fuzz testing for:
  - chrono::NaiveTime
  - chrono::NaiveDate
  - chrono::NaiveDateTime
  - chrono::DateTime<Utc>

This change improves modularity, eliminates unnecessary shared types, and strengthens testing coverage for the remaining chrono types.

Author: l-7-l

Cargo.toml

@@ -44,7 +44,6 @@ zstd = "0.13.0"
 derive = [ "dep:bitcode_derive" ]
 std = [ "serde?/std", "glam?/std", "arrayvec?/std" ]
 default = [ "derive", "std" ]
-chrono_datetime_fixedoffset = ["dep:chrono"]
 
 [package.metadata.docs.rs]
 features = [ "derive", "serde", "std" ]

fuzz/Cargo.toml

@@ -10,7 +10,7 @@ cargo-fuzz = true
 
 [dependencies]
 arrayvec = { version = "0.7", features = ["serde"] }
-bitcode = { path = "..", features = [ "arrayvec", "rust_decimal", "serde", "time" ] }
+bitcode = { path = "..", features = [ "arrayvec", "rust_decimal", "serde", "time" ,"chrono"] }
 libfuzzer-sys = "0.4"
 rust_decimal = "1.36.0"
 serde = { version ="1.0", features = [ "derive" ] }
@@ -24,4 +24,4 @@ members = ["."]
 name = "fuzz"
 path = "fuzz_targets/fuzz.rs"
 test = false
-doc = false
+doc = false

fuzz/fuzz_targets/fuzz.rs

@@ -3,14 +3,14 @@ use libfuzzer_sys::fuzz_target;
 extern crate bitcode;
 use arrayvec::{ArrayString, ArrayVec};
 use bitcode::{Decode, DecodeOwned, Encode};
+use rust_decimal::Decimal;
 use serde::de::DeserializeOwned;
 use serde::{Deserialize, Serialize};
 use std::collections::{BTreeMap, HashMap};
 use std::fmt::Debug;
+use std::net::{IpAddr, Ipv4Addr, Ipv6Addr, SocketAddr, SocketAddrV4, SocketAddrV6};
 use std::num::NonZeroU32;
 use std::time::Duration;
-use std::net::{IpAddr, Ipv4Addr, Ipv6Addr, SocketAddr, SocketAddrV4, SocketAddrV6};
-use rust_decimal::Decimal;
 
 #[inline(never)]
 fn test_derive<T: Debug + PartialEq + Encode + DecodeOwned>(data: &[u8]) {
@@ -148,10 +148,20 @@ fuzz_target!(|data: &[u8]| {
         A,
         B,
         C(u16),
-        D { a: u8, b: u8, #[serde(skip)] #[bitcode(skip)] c: u8 },
+        D {
+            a: u8,
+            b: u8,
+            #[serde(skip)]
+            #[bitcode(skip)]
+            c: u8,
+        },
         E(String),
         F,
-        G(#[bitcode(skip)] #[serde(skip)] i16),
+        G(
+            #[bitcode(skip)]
+            #[serde(skip)]
+            i16,
+        ),
         P(BTreeMap<u16, u8>),
     }
 
@@ -233,5 +243,9 @@ fuzz_target!(|data: &[u8]| {
         SocketAddrV6,
         SocketAddr,
         time::Time,
+        chrono::NaiveTime,
+        chrono::NaiveDate,
+        chrono::NaiveDateTime,
+        chrono::DateTime<Utc>,
     );
 });

src/ext/chrono.rs

@@ -0,0 +1,20 @@
+mod date_time_utc;
+mod naive_date;
+mod naive_date_time;
+mod naive_time;
+
+use crate::int::ranged_int;
+
+ranged_int!(Hour, u8, 0, 23);
+ranged_int!(Minute, u8, 0, 59);
+ranged_int!(Second, u8, 0, 59);
+ranged_int!(Nanosecond, u32, 0, 1_999_999_999);
+
+type TimeEncode = (u8, u8, u8, u32);
+type TimeDecode = (Hour, Minute, Second, Nanosecond);
+
+type DateEncode = i32;
+type DateDecode = i32;
+
+type DateTimeEncode = (DateEncode, TimeEncode);
+type DateTimeDecode = (DateEncode, TimeDecode);

src/ext/date/chrono/date_time_utc.rs src/ext/chrono/date_time_utc.rssrc/ext/date/chrono/date_time_utc.rs renamed to src/ext/chrono/date_time_utc.rs

@@ -2,7 +2,7 @@ use chrono::{DateTime, NaiveDateTime, Utc};
 
 use crate::{
     convert::{impl_convert, ConvertFrom},
-    ext::date::{DateTimeDecode, DateTimeEncode},
+    ext::chrono::{DateTimeDecode, DateTimeEncode},
 };
 
 impl_convert!(DateTime<Utc>, DateTimeEncode, DateTimeDecode);
@@ -13,8 +13,8 @@ impl ConvertFrom<&DateTime<Utc>> for DateTimeEncode {
     }
 }
 
-impl ConvertFrom<DateTimeEncode> for DateTime<Utc> {
-    fn convert_from(enc: DateTimeEncode) -> Self {
+impl ConvertFrom<DateTimeDecode> for DateTime<Utc> {
+    fn convert_from(enc: DateTimeDecode) -> Self {
         let naive = NaiveDateTime::convert_from(enc);
 
         DateTime::from_naive_utc_and_offset(naive, Utc)

src/ext/date/chrono/naive_date.rs src/ext/chrono/naive_date.rssrc/ext/date/chrono/naive_date.rs renamed to src/ext/chrono/naive_date.rs

@@ -2,20 +2,20 @@ use chrono::{Datelike, NaiveDate};
 
 use crate::{
     convert::{impl_convert, ConvertFrom},
-    ext::date::{DateDecode, DateEncode},
+    ext::chrono::{DateDecode, DateEncode},
 };
 
 impl_convert!(NaiveDate, DateEncode, DateDecode);
 
 impl ConvertFrom<&NaiveDate> for DateEncode {
     fn convert_from(days: &NaiveDate) -> Self {
-        days.num_days_from_ce() - 719_163 // 1970-1-1
+        days.num_days_from_ce()
     }
 }
 
 impl ConvertFrom<DateDecode> for NaiveDate {
     fn convert_from(days: DateDecode) -> Self {
-        NaiveDate::from_num_days_from_ce_opt(days + 719_163).unwrap() // 1970-1-1
+        NaiveDate::from_num_days_from_ce_opt(days).unwrap()
     }
 }
 
@@ -28,6 +28,7 @@ mod tests {
             NaiveDate::from_ymd_opt(2025, 10, 6).unwrap(),
             NaiveDate::from_ymd_opt(1, 1, 1).unwrap(),
             NaiveDate::from_ymd_opt(-44, 3, 15).unwrap(), // BCE
+            NaiveDate::from_ymd_opt(-44, 3, 15).unwrap(), // BCE
             NaiveDate::from_ymd_opt(9999, 12, 31).unwrap(),
         ];
 

src/ext/date/chrono/naive_date_time.rs src/ext/chrono/naive_date_time.rssrc/ext/date/chrono/naive_date_time.rs renamed to src/ext/chrono/naive_date_time.rs

@@ -2,12 +2,13 @@ use chrono::{NaiveDate, NaiveDateTime, NaiveTime};
 
 use crate::{
     convert::{impl_convert, ConvertFrom},
-    ext::date::{DateEncode, DateTimeDecode, DateTimeEncode, TimeEncode},
+    ext::chrono::{DateEncode, DateTimeDecode, DateTimeEncode, TimeEncode},
 };
 
 impl_convert!(NaiveDateTime, DateTimeEncode, DateTimeDecode);
 
 impl ConvertFrom<&NaiveDateTime> for DateTimeEncode {
+    #[inline(always)]
     fn convert_from(x: &NaiveDateTime) -> Self {
         (
             DateEncode::convert_from(&x.date()),
@@ -16,13 +17,10 @@ impl ConvertFrom<&NaiveDateTime> for DateTimeEncode {
     }
 }
 
-impl ConvertFrom<DateTimeEncode> for NaiveDateTime {
-    fn convert_from((date, time): DateTimeEncode) -> Self {
-        NaiveDateTime::new(
-            NaiveDate::convert_from(date),
-            NaiveTime::from_hms_nano_opt(time.0 as u32, time.1 as u32, time.2 as u32, time.3)
-                .unwrap(),
-        )
+impl ConvertFrom<DateTimeDecode> for NaiveDateTime {
+    #[inline(always)]
+    fn convert_from((date, time): DateTimeDecode) -> Self {
+        NaiveDateTime::new(NaiveDate::convert_from(date), NaiveTime::convert_from(time))
     }
 }
 

src/ext/date/chrono/naive_time.rs src/ext/chrono/naive_time.rssrc/ext/date/chrono/naive_time.rs renamed to src/ext/chrono/naive_time.rs

@@ -1,12 +1,13 @@
 use crate::{
     convert::{impl_convert, ConvertFrom},
-    ext::date::{TimeDecode, TimeEncode},
+    ext::chrono::{TimeDecode, TimeEncode},
 };
 use chrono::{NaiveTime, Timelike};
 
 impl_convert!(NaiveTime, TimeEncode, TimeDecode);
 
 impl ConvertFrom<&NaiveTime> for TimeEncode {
+    #[inline(always)]
     fn convert_from(value: &NaiveTime) -> Self {
         (
             value.hour() as u8,
@@ -18,12 +19,15 @@ impl ConvertFrom<&NaiveTime> for TimeEncode {
 }
 
 impl ConvertFrom<TimeDecode> for NaiveTime {
+    #[inline(always)]
     fn convert_from(value: TimeDecode) -> Self {
+        let (hour, min, sec, nano) = value;
+
         NaiveTime::from_hms_nano_opt(
-            value.0.into_inner() as u32,
-            value.1.into_inner() as u32,
-            value.2.into_inner() as u32,
-            value.3.into_inner(),
+            hour.into_inner() as u32,
+            min.into_inner() as u32,
+            sec.into_inner() as u32,
+            nano.into_inner(),
         )
         .unwrap()
     }