dm.rs defines its own row_to_channel_record (line 487, ~30 lines) returning the same ChannelRecord type from channel.rs. The two implementations must stay in sync manually — any column added to ChannelRecord requires updating both mappers. Fix: make the channel module's mapper pub(crate) and import it in dm.rs, or apply #[derive(FromRow)] per #15. Priority: Low — small cleanup, risk of divergence.
🤖 AI review update (2026-08-23)
This is a good small cleanup or the first acceptance item in #15. Prefer one canonical mapper or one proven FromRow implementation, with a test that exercises the DM query shape against the shared ChannelRecord. Do not wait for the full lib.rs breakup if this can be removed safely now.
dm.rsdefines its ownrow_to_channel_record(line 487, ~30 lines) returning the sameChannelRecordtype fromchannel.rs. The two implementations must stay in sync manually — any column added toChannelRecordrequires updating both mappers. Fix: make the channel module's mapperpub(crate)and import it indm.rs, or apply#[derive(FromRow)]per #15. Priority: Low — small cleanup, risk of divergence.🤖 AI review update (2026-08-23)
This is a good small cleanup or the first acceptance item in #15. Prefer one canonical mapper or one proven FromRow implementation, with a test that exercises the DM query shape against the shared ChannelRecord. Do not wait for the full lib.rs breakup if this can be removed safely now.