hadris_fat/

dir.rs

io_transform! {

use core::mem::size_of;

use hadris_common::types::endian::Endian;

use crate::error::{Error, Result};
use crate::file::ShortFileName;
#[cfg(feature = "lfn")]
use crate::file::{LfnBuilder, LongFileName};
use crate::raw::{DirEntryAttrFlags, NtCaseFlags, RawDirectoryEntry};
use crate::time::FatDateTime;
use super::fs::FatVolume;
#[cfg(not(feature = "alloc"))]
use super::io::ReadExt;
use super::io::{Cluster, ClusterLike, Read, Seek, SeekFrom};
use super::read::FileReader;

/// Formats a stored 8.3 short name as its human-facing filename: the NT case
/// flags are applied, trailing space padding is trimmed from the base and
/// extension, and the `.` separator is dropped when there is no extension.
///
/// [`ShortFileName::as_str`] returns the padded on-disk field (`README  .TXT`);
/// this returns the logical name (`readme.txt`).
#[cfg(feature = "alloc")]
fn short_name_display(short: &ShortFileName, flags: NtCaseFlags) -> alloc::string::String {
    use alloc::string::ToString;

    // The "." and ".." directory entries are stored without 8.3 padding and
    // carry no case flags; the base/extension split below would corrupt them.
    let raw = short.as_str();
    if raw == "." || raw == ".." {
        return raw.to_string();
    }

    let cased = short.with_nt_case(flags);
    let raw = cased.as_str();
    let (base, ext) = match raw.find('.') {
        Some(dot) => (raw[..dot].trim_end(), raw[dot + 1..].trim_end()),
        None => (raw.trim_end(), ""),
    };
    let mut name = alloc::string::String::with_capacity(base.len() + 1 + ext.len());
    name.push_str(base);
    if !ext.is_empty() {
        name.push('.');
        name.push_str(ext);
    }
    name
}

/// A directory within a mounted FAT filesystem.
pub struct FatDir<'a, DATA: Read + Seek> {
    pub(crate) data: &'a FatVolume<DATA>,
    /// Cluster for subdirectories, or 0 (sentinel) for FAT12/16 fixed root
    pub(crate) cluster: Cluster,
    /// For FAT12/16 root: (start_byte, size_bytes), None for cluster-based dirs
    pub(crate) fixed_root: Option<(usize, usize)>,
}

impl<'a, DATA: Read + Seek> FatDir<'a, DATA> {
    /// Creates an iterator over this directory's entries.
    #[cfg(feature = "lfn")]
    pub fn entries(&self) -> FatDirIter<'a, DATA> {
        FatDirIter {
            data: self.data,
            cluster: self.cluster,
            #[cfg(feature = "write")]
            dir_start_cluster: self.cluster,
            offset: 0,
            fixed_root_remaining: self.fixed_root.map(|(_, size)| size),
            fixed_root_start: self.fixed_root.map(|(start, _)| start),
            cluster_steps: 0,
            lfn_builder: LfnBuilder::new(),
            #[cfg(feature = "alloc")]
            cluster_buffer: None,
            #[cfg(feature = "alloc")]
            buffer_valid: false,
        }
    }

    /// Creates an iterator over this directory's entries.
    #[cfg(not(feature = "lfn"))]
    pub fn entries(&self) -> FatDirIter<'a, DATA> {
        FatDirIter {
            data: self.data,
            cluster: self.cluster,
            #[cfg(feature = "write")]
            dir_start_cluster: self.cluster,
            offset: 0,
            fixed_root_remaining: self.fixed_root.map(|(_, size)| size),
            fixed_root_start: self.fixed_root.map(|(start, _)| start),
            cluster_steps: 0,
            #[cfg(feature = "alloc")]
            cluster_buffer: None,
            #[cfg(feature = "alloc")]
            buffer_valid: false,
        }
    }

    /// Open a subdirectory from a file entry.
    ///
    /// The entry must be a directory.
    pub fn open_entry(&self, entry: &FileEntry) -> Result<FatDir<'a, DATA>> {
        if !entry.is_directory() {
            return Err(Error::NotADirectory);
        }
        Ok(FatDir {
            data: self.data,
            cluster: entry.cluster(),
            fixed_root: None, // Subdirectories are never fixed root
        })
    }

    /// Find an entry by name.
    ///
    /// When the `lfn` feature is enabled, this performs a case-sensitive match
    /// against long file names first, then falls back to case-insensitive short
    /// name matching.
    ///
    /// Without the `lfn` feature, only case-insensitive short name matching is used.
    pub async fn find(&self, name: &str) -> Result<Option<FileEntry>> {
        let mut iter = self.entries();
        loop {
            match iter.next_entry().await {
                Some(result) => {
                    let DirectoryEntry::Entry(file_entry) = result?;

                    // Check LFN match (case-sensitive)
                    #[cfg(feature = "lfn")]
                    if let Some(lfn) = file_entry.long_name()
                        && lfn.eq_str(name)
                    {
                        return Ok(Some(file_entry));
                    }
                    // Check short name match (case-insensitive, handles 8.3 padding)
                    if file_entry.short_name().matches(name) {
                        return Ok(Some(file_entry));
                    }
                }
                None => return Ok(None),
            }
        }
    }

    /// Open a subdirectory by name.
    ///
    /// Returns an error if the entry is not found or is not a directory.
    pub async fn open_dir(&self, name: &str) -> Result<FatDir<'a, DATA>> {
        let entry = self.find(name).await?.ok_or(Error::EntryNotFound)?;

        if !entry.is_directory() {
            return Err(Error::NotADirectory);
        }

        // Subdirectories always use cluster chains, never fixed root
        Ok(FatDir {
            data: self.data,
            cluster: entry.cluster(),
            fixed_root: None,
        })
    }

    /// Open a file for reading by name.
    ///
    /// Returns an error if the entry is not found or is a directory.
    pub async fn open_file(&self, name: &str) -> Result<FileReader<'a, DATA>> {
        let entry = self.find(name).await?.ok_or(Error::EntryNotFound)?;
        FileReader::new(self.data, &entry)
    }
}

/// Stateful iterator over the entries in a FAT directory.
pub struct FatDirIter<'a, DATA: Read + Seek> {
    data: &'a FatVolume<DATA>,
    /// Current cluster (or 0 for fixed root directory)
    cluster: Cluster,
    /// First cluster of the directory chain (or 0 for a fixed root).
    #[cfg(feature = "write")]
    dir_start_cluster: Cluster,
    /// Offset within current cluster (or within fixed root dir)
    offset: usize,
    /// For fixed root directory: remaining bytes to read (None for cluster-based)
    fixed_root_remaining: Option<usize>,
    /// For fixed root directory: start byte offset
    fixed_root_start: Option<usize>,
    /// Cluster transitions taken so far. A cluster-based directory chain
    /// longer than `max_cluster` clusters has to revisit one — that's a
    /// loop and we abort with `Error::ClusterLoop`.
    cluster_steps: u32,
    #[cfg(feature = "lfn")]
    lfn_builder: LfnBuilder,
    /// Buffered cluster data (reduces seeks by reading entire cluster at once)
    #[cfg(feature = "alloc")]
    cluster_buffer: Option<alloc::vec::Vec<u8>>,
    /// Whether the buffer is valid for the current cluster
    #[cfg(feature = "alloc")]
    buffer_valid: bool,
}

impl<DATA: Read + Seek> FatDirIter<'_, DATA> {
    /// Read the next directory entry.
    pub async fn next_entry(&mut self) -> Option<Result<DirectoryEntry>> {
        let mut data = self.data.data.lock();
        let entry_size = size_of::<RawDirectoryEntry>();
        let cluster_size = data.cluster_size;

        loop {
            // Check bounds and handle cluster transitions
            if let Some(ref mut remaining) = self.fixed_root_remaining {
                // Fixed root directory (FAT12/16)
                if *remaining < entry_size {
                    return None; // End of fixed root directory
                }
            } else {
                // Cluster-based directory (FAT32 or subdirectory)
                // Check if we need to move to the next cluster
                if self.offset >= cluster_size {
                    self.cluster_steps = self.cluster_steps.saturating_add(1);
                    if self.cluster_steps > self.data.fat.max_cluster() {
                        return Some(Err(Error::ClusterLoop {
                            cluster: self.cluster.0 as u32,
                        }));
                    }
                    // Drop data lock so the routed helper can acquire cache
                    // first (canonical order) without deadlocking. Re-lock
                    // after.
                    drop(data);
                    let next = match self.data.next_cluster_routed(self.cluster.0).await {
                        Ok(n) => n,
                        Err(e) => return Some(Err(e)),
                    };
                    data = self.data.data.lock();
                    match next {
                        Some(cluster) => {
                            self.cluster.0 = cluster as usize;
                            self.offset = 0;
                            #[cfg(feature = "alloc")]
                            {
                                self.buffer_valid = false;
                            }
                        }
                        None => return None, // End of directory
                    }
                }
            }

            // Read the entry - use buffering when alloc is available
            #[cfg(feature = "alloc")]
            let raw_entry = {
                // Ensure buffer is filled
                if !self.buffer_valid || self.cluster_buffer.is_none() {
                    let buffer_size = if let Some(remaining) = self.fixed_root_remaining {
                        // For fixed root, buffer the remaining bytes (up to a reasonable size)
                        remaining.min(4096)
                    } else {
                        cluster_size
                    };

                    let seek_pos = if self.fixed_root_remaining.is_some() {
                        let start = self.fixed_root_start.unwrap();
                        start as u64
                    } else {
                        self.cluster
                            .to_bytes(self.data.info.data_start, cluster_size)
                            as u64
                    };

                    if let Err(e) = data.seek(SeekFrom::Start(seek_pos)).await {
                        return Some(Err(Error::Io(e.erase())));
                    }

                    let mut buffer = alloc::vec![0u8; buffer_size];
                    if let Err(e) = data.read_exact(&mut buffer).await {
                        return Some(Err(Error::Io(e.erase())));
                    }

                    self.cluster_buffer = Some(buffer);
                    self.buffer_valid = true;
                }

                // Read entry from buffer
                let buffer = self.cluster_buffer.as_ref().unwrap();
                let offset = self.offset;

                if offset + entry_size > buffer.len() {
                    // Buffer exhausted, need to handle this case
                    // For fixed root: we're done
                    // For cluster-based: handled by cluster transition above
                    if self.fixed_root_remaining.is_some() {
                        return None;
                    }
                    continue;
                }

                let entry_bytes: [u8; 32] = buffer[offset..offset + entry_size].try_into().unwrap();

                // Safety: RawDirectoryEntry is a union of properly aligned types
                // and entry_bytes has the correct size
                unsafe { core::mem::transmute::<[u8; 32], RawDirectoryEntry>(entry_bytes) }
            };

            #[cfg(not(feature = "alloc"))]
            let raw_entry = {
                // Calculate seek position
                let seek_pos = if self.fixed_root_remaining.is_some() {
                    let start = self.fixed_root_start.unwrap();
                    (start + self.offset) as u64
                } else {
                    self.cluster
                        .to_bytes(self.data.info.data_start, cluster_size)
                        as u64
                        + self.offset as u64
                };

                if let Err(e) = data.seek(SeekFrom::Start(seek_pos)).await {
                    return Some(Err(Error::Io(e.erase())));
                }

                // Read the directory entry
                match data.read_struct::<RawDirectoryEntry>().await {
                    Ok(e) => e,
                    Err(e) => return Some(Err(Error::Io(e))),
                }
            };

            let entry_bytes = unsafe { raw_entry.bytes };

            // Check for end of directory
            if entry_bytes[0] == 0 {
                #[cfg(feature = "lfn")]
                self.lfn_builder.reset();
                return None;
            }

            // Check for deleted entry
            if entry_bytes[0] == 0xE5 {
                self.offset += entry_size;
                if let Some(ref mut remaining) = self.fixed_root_remaining {
                    *remaining = remaining.saturating_sub(entry_size);
                }
                #[cfg(feature = "lfn")]
                self.lfn_builder.reset(); // Deleted entry breaks LFN sequence
                continue;
            }

            self.offset += entry_size;
            if let Some(ref mut remaining) = self.fixed_root_remaining {
                *remaining = remaining.saturating_sub(entry_size);
            }

            // Check if this is an LFN entry (attributes == LONG_NAME)
            #[cfg(feature = "lfn")]
            {
                let entry_attr = unsafe { raw_entry.file }.attributes;
                if entry_attr == DirEntryAttrFlags::LONG_NAME.bits() {
                    // This is an LFN entry
                    let lfn = unsafe { raw_entry.lfn };
                    let seq = lfn.sequence_number;

                    // Check if this is the start of a new LFN sequence (has 0x40 bit set)
                    if seq & LfnBuilder::LAST_ENTRY_MASK != 0 {
                        self.lfn_builder.start(seq, lfn.checksum);
                    }

                    if self.lfn_builder.building {
                        self.lfn_builder.add_entry(
                            seq,
                            lfn.checksum,
                            &lfn.name1,
                            &lfn.name2,
                            &lfn.name3,
                        );
                    }
                    continue;
                }
            }

            // This is a regular file/directory entry
            let file_entry = unsafe { raw_entry.file };

            let attr = DirEntryAttrFlags::from_bits_retain(file_entry.attributes);
            if attr.is_volume_label_entry() {
                #[cfg(feature = "lfn")]
                self.lfn_builder.reset();
                continue;
            }

            // Convert 0x05 back to 0xE5 for kanji compatibility
            let mut name_bytes = file_entry.name;
            if name_bytes[0] == 0x05 {
                name_bytes[0] = 0xE5;
            }

            let short_name = match ShortFileName::new(name_bytes) {
                Ok(n) => n,
                Err(_) => return Some(Err(Error::InvalidShortFilename)),
            };

            // Try to get the LFN if we've been building one
            #[cfg(feature = "lfn")]
            let long_name = self.lfn_builder.finish(&short_name);

            // For FAT12/16 with fixed root dir, parent_clus is 0 (sentinel)
            // For cluster-based dirs, parent_clus is the actual cluster
            let created = FatDateTime::from_raw(
                u16::from_le_bytes(file_entry.creation_date),
                u16::from_le_bytes(file_entry.creation_time),
                file_entry.creation_time_tenth,
            );
            let modified = FatDateTime::from_raw(
                u16::from_le_bytes(file_entry.last_write_date),
                u16::from_le_bytes(file_entry.last_write_time),
                0,
            );
            let last_access_date = u16::from_le_bytes(file_entry.last_access_date);

            return Some(Ok(DirectoryEntry::Entry(FileEntry {
                short_name,
                nt_case: NtCaseFlags::from_bits_truncate(file_entry.reserved),
                #[cfg(feature = "lfn")]
                long_name,
                attr,
                size: file_entry.size.get() as usize,
                #[cfg(feature = "write")]
                parent_dir_clus: self.dir_start_cluster,
                #[cfg(feature = "write")]
                parent_clus: self.cluster,
                #[cfg(feature = "write")]
                offset_within_cluster: self.offset - entry_size,
                cluster: Cluster::from_parts(
                    file_entry.first_cluster_high.get(),
                    file_entry.first_cluster_low.get(),
                ),
                created,
                last_access_date,
                modified,
            })));
        }
    }
}

#[derive(Debug)]
/// A parsed FAT directory record.
pub enum DirectoryEntry {
    /// A file or directory entry
    Entry(FileEntry),
}

impl DirectoryEntry {
    /// Get the display name of the entry.
    /// Returns the long filename if available, otherwise the short name.
    ///
    /// Requires the `alloc` feature. See [`FileEntry::name`].
    #[cfg(feature = "alloc")]
    pub fn name(&self) -> alloc::borrow::Cow<'_, str> {
        match self {
            Self::Entry(ent) => ent.name(),
        }
    }

    /// Get the file entry if this is an Entry variant
    pub fn as_entry(&self) -> Option<&FileEntry> {
        match self {
            Self::Entry(ent) => Some(ent),
        }
    }
}

#[derive(Debug)]
/// A parsed value together with non-fatal filesystem diagnostics.
pub struct ParseInfo<T> {
    /// Parsed value.
    pub data: T,
    /// Warnings encountered while parsing.
    pub warnings: FileSystemWarnings,
    /// Errors accumulated while parsing.
    pub errors: FileSystemErrors,
}

bitflags::bitflags! {
    /// Non-fatal warnings discovered while parsing a filesystem object.
    #[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord)]
    pub struct FileSystemWarnings: u64 {

    }

    /// Errors accumulated while parsing a filesystem object.
    #[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord)]
    pub struct FileSystemErrors: u64 {

    }
}

#[derive(Debug)]
/// Metadata for a file or subdirectory entry.
pub struct FileEntry {
    pub(crate) short_name: ShortFileName,
    /// Windows NT 8.3 name case flags (`DIR_NTRes`); applied to the display
    /// name when no long name is present.
    pub(crate) nt_case: NtCaseFlags,
    #[cfg(feature = "lfn")]
    pub(crate) long_name: Option<LongFileName>,
    pub(crate) attr: DirEntryAttrFlags,
    pub(crate) size: usize,
    /// First cluster of the containing directory (0 for a fixed root).
    #[cfg(feature = "write")]
    pub(crate) parent_dir_clus: Cluster<usize>,
    /// Cluster containing this short directory entry.
    #[cfg(feature = "write")]
    pub(crate) parent_clus: Cluster<usize>,
    /// Offset of this entry within the parent cluster (used for write operations)
    #[cfg(feature = "write")]
    pub(crate) offset_within_cluster: usize,
    pub(crate) cluster: Cluster<usize>,
    /// Creation time (date + time + 10ms-units).
    pub(crate) created: FatDateTime,
    /// Last-access date (FAT stores no time component for access).
    pub(crate) last_access_date: u16,
    /// Last-modified time (date + time; `time_tenth` is 0).
    pub(crate) modified: FatDateTime,
}

impl FileEntry {
    /// Get the file's display name.
    ///
    /// Returns the long filename if available, otherwise the short name. The
    /// `Cow` is borrowed for short names and owned (allocated) for long names,
    /// since long names are stored internally as UTF-16 and require decoding
    /// before they can be exposed as a `&str`.
    ///
    /// Requires the `alloc` feature. Without `alloc`, use [`Self::short_name`]
    /// and [`Self::long_name`] (with [`LongFileName::chars`] /
    /// [`LongFileName::eq_str`]) directly.
    #[cfg(feature = "alloc")]
    pub fn name(&self) -> alloc::borrow::Cow<'_, str> {
        #[cfg(feature = "lfn")]
        {
            use alloc::string::ToString;
            if let Some(ref lfn) = self.long_name {
                return alloc::borrow::Cow::Owned(lfn.to_string());
            }
        }
        alloc::borrow::Cow::Owned(short_name_display(&self.short_name, self.nt_case))
    }

    /// Get the short (8.3) filename, in its canonical on-disk (uppercase) form.
    ///
    /// Use [`Self::nt_case`] with [`ShortFileName::with_nt_case`] to recover the
    /// original lowercase presentation without allocating.
    pub fn short_name(&self) -> &ShortFileName {
        &self.short_name
    }

    /// Windows NT 8.3 name case flags (`DIR_NTRes`) for this entry.
    ///
    /// Records whether the short name's base and/or extension were originally
    /// lowercase. Meaningful only when the entry has no long file name.
    pub fn nt_case(&self) -> NtCaseFlags {
        self.nt_case
    }

    /// Get the long filename, if available
    #[cfg(feature = "lfn")]
    pub fn long_name(&self) -> Option<&LongFileName> {
        self.long_name.as_ref()
    }

    /// Get the file attributes
    pub fn attributes(&self) -> DirEntryAttrFlags {
        self.attr
    }

    /// Check if this entry is a directory
    pub fn is_directory(&self) -> bool {
        self.attr.contains(DirEntryAttrFlags::DIRECTORY)
    }

    /// Check if this entry is a regular file
    pub fn is_file(&self) -> bool {
        !self.is_directory()
    }

    /// Returns the file length in bytes (zero for directories).
    pub fn len(&self) -> u64 {
        self.size as u64
    }

    /// Returns whether this entry has a zero byte length.
    pub fn is_empty(&self) -> bool {
        self.len() == 0
    }

    /// Creation timestamp (date + time + 10-ms units).
    pub fn created(&self) -> FatDateTime {
        self.created
    }

    /// Last-access date. FAT does not store an access time, only a date.
    /// Returns the raw FAT-encoded date `(year-1980)<<9 | month<<5 | day`.
    pub fn accessed_date(&self) -> u16 {
        self.last_access_date
    }

    /// Last-modified timestamp (date + time; sub-2-second precision is not
    /// preserved by FAT for the modified field).
    pub fn modified(&self) -> FatDateTime {
        self.modified
    }

    /// Get the first cluster of the file data
    pub fn cluster(&self) -> Cluster<usize> {
        self.cluster
    }
}

} // end io_transform!

sync_only! {

impl<DATA: Read + Seek> Iterator for FatDirIter<'_, DATA> {
    type Item = Result<DirectoryEntry>;

    fn next(&mut self) -> Option<Self::Item> {
        let mut data = self.data.data.lock();
        let entry_size = size_of::<RawDirectoryEntry>();
        let cluster_size = data.cluster_size;

        loop {
            // Check bounds and handle cluster transitions
            if let Some(ref mut remaining) = self.fixed_root_remaining {
                // Fixed root directory (FAT12/16)
                if *remaining < entry_size {
                    return None; // End of fixed root directory
                }
            } else {
                // Cluster-based directory (FAT32 or subdirectory)
                // Check if we need to move to the next cluster
                if self.offset >= cluster_size {
                    self.cluster_steps = self.cluster_steps.saturating_add(1);
                    if self.cluster_steps > self.data.fat.max_cluster() {
                        return Some(Err(Error::ClusterLoop {
                            cluster: self.cluster.0 as u32,
                        }));
                    }
                    // Drop data lock so next_cluster_routed can acquire
                    // cache+data in canonical order; re-lock afterwards.
                    drop(data);
                    let next = match self.data.next_cluster_routed(self.cluster.0) {
                        Ok(n) => n,
                        Err(e) => return Some(Err(e)),
                    };
                    data = self.data.data.lock();
                    match next {
                        Some(cluster) => {
                            self.cluster.0 = cluster as usize;
                            self.offset = 0;
                            #[cfg(feature = "alloc")]
                            {
                                self.buffer_valid = false;
                            }
                        }
                        None => return None, // End of directory
                    }
                }
            }

            // Read the entry - use buffering when alloc is available
            #[cfg(feature = "alloc")]
            let raw_entry = {
                // Ensure buffer is filled
                if !self.buffer_valid || self.cluster_buffer.is_none() {
                    let buffer_size = if let Some(remaining) = self.fixed_root_remaining {
                        // For fixed root, buffer the remaining bytes (up to a reasonable size)
                        remaining.min(4096)
                    } else {
                        cluster_size
                    };

                    let seek_pos = if self.fixed_root_remaining.is_some() {
                        let start = self.fixed_root_start.unwrap();
                        start as u64
                    } else {
                        self.cluster
                            .to_bytes(self.data.info.data_start, cluster_size)
                            as u64
                    };

                    if let Err(e) = data.seek(SeekFrom::Start(seek_pos)) {
                        return Some(Err(Error::Io(e.erase())));
                    }

                    let mut buffer = alloc::vec![0u8; buffer_size];
                    if let Err(e) = data.read_exact(&mut buffer) {
                        return Some(Err(Error::Io(e.erase())));
                    }

                    self.cluster_buffer = Some(buffer);
                    self.buffer_valid = true;
                }

                // Read entry from buffer
                let buffer = self.cluster_buffer.as_ref().unwrap();
                let offset = self.offset;

                if offset + entry_size > buffer.len() {
                    // Buffer exhausted, need to handle this case
                    // For fixed root: we're done
                    // For cluster-based: handled by cluster transition above
                    if self.fixed_root_remaining.is_some() {
                        return None;
                    }
                    continue;
                }

                let entry_bytes: [u8; 32] = buffer[offset..offset + entry_size].try_into().unwrap();

                // Safety: RawDirectoryEntry is a union of properly aligned types
                // and entry_bytes has the correct size
                unsafe { core::mem::transmute::<[u8; 32], RawDirectoryEntry>(entry_bytes) }
            };

            #[cfg(not(feature = "alloc"))]
            let raw_entry = {
                // Calculate seek position
                let seek_pos = if self.fixed_root_remaining.is_some() {
                    let start = self.fixed_root_start.unwrap();
                    (start + self.offset) as u64
                } else {
                    self.cluster
                        .to_bytes(self.data.info.data_start, cluster_size)
                        as u64
                        + self.offset as u64
                };

                if let Err(e) = data.seek(SeekFrom::Start(seek_pos)) {
                    return Some(Err(Error::Io(e.erase())));
                }

                // Read the directory entry
                match data.read_struct::<RawDirectoryEntry>() {
                    Ok(e) => e,
                    Err(e) => return Some(Err(Error::Io(e))),
                }
            };

            let entry_bytes = unsafe { raw_entry.bytes };

            // Check for end of directory
            if entry_bytes[0] == 0 {
                #[cfg(feature = "lfn")]
                self.lfn_builder.reset();
                return None;
            }

            // Check for deleted entry
            if entry_bytes[0] == 0xE5 {
                self.offset += entry_size;
                if let Some(ref mut remaining) = self.fixed_root_remaining {
                    *remaining = remaining.saturating_sub(entry_size);
                }
                #[cfg(feature = "lfn")]
                self.lfn_builder.reset(); // Deleted entry breaks LFN sequence
                continue;
            }

            self.offset += entry_size;
            if let Some(ref mut remaining) = self.fixed_root_remaining {
                *remaining = remaining.saturating_sub(entry_size);
            }

            // Check if this is an LFN entry (attributes == LONG_NAME)
            #[cfg(feature = "lfn")]
            {
                let entry_attr = unsafe { raw_entry.file }.attributes;
                if entry_attr == DirEntryAttrFlags::LONG_NAME.bits() {
                    // This is an LFN entry
                    let lfn = unsafe { raw_entry.lfn };
                    let seq = lfn.sequence_number;

                    // Check if this is the start of a new LFN sequence (has 0x40 bit set)
                    if seq & LfnBuilder::LAST_ENTRY_MASK != 0 {
                        self.lfn_builder.start(seq, lfn.checksum);
                    }

                    if self.lfn_builder.building {
                        self.lfn_builder.add_entry(
                            seq,
                            lfn.checksum,
                            &lfn.name1,
                            &lfn.name2,
                            &lfn.name3,
                        );
                    }
                    continue;
                }
            }

            // This is a regular file/directory entry
            let file_entry = unsafe { raw_entry.file };

            let attr = DirEntryAttrFlags::from_bits_retain(file_entry.attributes);
            if attr.is_volume_label_entry() {
                #[cfg(feature = "lfn")]
                self.lfn_builder.reset();
                continue;
            }

            // Convert 0x05 back to 0xE5 for kanji compatibility
            let mut name_bytes = file_entry.name;
            if name_bytes[0] == 0x05 {
                name_bytes[0] = 0xE5;
            }

            let short_name = match ShortFileName::new(name_bytes) {
                Ok(n) => n,
                Err(_) => return Some(Err(Error::InvalidShortFilename)),
            };

            // Try to get the LFN if we've been building one
            #[cfg(feature = "lfn")]
            let long_name = self.lfn_builder.finish(&short_name);

            // For FAT12/16 with fixed root dir, parent_clus is 0 (sentinel)
            // For cluster-based dirs, parent_clus is the actual cluster
            let created = FatDateTime::from_raw(
                u16::from_le_bytes(file_entry.creation_date),
                u16::from_le_bytes(file_entry.creation_time),
                file_entry.creation_time_tenth,
            );
            let modified = FatDateTime::from_raw(
                u16::from_le_bytes(file_entry.last_write_date),
                u16::from_le_bytes(file_entry.last_write_time),
                0,
            );
            let last_access_date = u16::from_le_bytes(file_entry.last_access_date);

            return Some(Ok(DirectoryEntry::Entry(FileEntry {
                short_name,
                nt_case: NtCaseFlags::from_bits_truncate(file_entry.reserved),
                #[cfg(feature = "lfn")]
                long_name,
                attr,
                size: file_entry.size.get() as usize,
                #[cfg(feature = "write")]
                parent_dir_clus: self.dir_start_cluster,
                #[cfg(feature = "write")]
                parent_clus: self.cluster,
                #[cfg(feature = "write")]
                offset_within_cluster: self.offset - entry_size,
                cluster: Cluster::from_parts(
                    file_entry.first_cluster_high.get(),
                    file_entry.first_cluster_low.get(),
                ),
                created,
                last_access_date,
                modified,
            })));
        }
    }
}

} // end sync_only!