Fix review

Author: FlickerSoul

Sources/BinaryParseKit/BinaryParseKit.swift

@@ -255,6 +255,10 @@ public macro parseRest(endianness: Endianness) = #externalMacro(
 /// - Proper error handling for parsing failures
 /// - A ``Printable`` conformance
 ///
+/// - Parameters:
+///   - parsingAccessor: The accessor level for the generated `Parsable` conformance (default is `.follow`)
+///   - printingAccessor: The accessor level for the generated `Printable` conformance (default is `.follow`)
+///
 /// - Note: All fields except those with accessors (`get` and `set`) must be parsed must be marked with `@parse`
 /// variants.
 ///
@@ -298,6 +302,10 @@ public macro ParseStruct(
 /// - A ``Parsable`` conformance
 /// - A ``Printable`` conformance
 ///
+/// - Parameters:
+///   - parsingAccessor: The accessor level for the generated `Parsable` conformance (default is `.follow`)
+///   - printingAccessor: The accessor level for the generated `Printable` conformance (default is `.follow`)
+///
 /// - Note: All enum cases must be marked with `@match` variants, which is intentional by design, which I don't think is
 /// necessary and is possible to be lifted int the future.
 /// - Note: Only one `@matchDefault` case is allowed per enum, and has to be declared at the end of all other cases.

Sources/BinaryParseKitCommons/ExtensionAccessor.swift

@@ -5,21 +5,33 @@
 //  Created by Larry Zeng on 11/26/25.
 //
 
+/// This enum represents the access level of an extension in Swift, with a few extra cases
 public enum ExtensionAccessor: ExpressibleByUnicodeScalarLiteral, Sendable, Codable {
     public typealias UnicodeScalarLiteralType = String
 
+    /// Same as public keyword
     case `public`
+    /// Same as package keyword
     case package
+    /// Same as internal keyword
     case `internal`
+    /// Same as fileprivate keyword
     case `fileprivate`
+    /// Same as private keyword
     case `private`
+    /// This specifier indicates that the extension should follow the access level of the type it extends
     case follow
+    /// Represents an unknown or unsupported access level, which will be raised as macro error.
     case unknown(String)
 
+    /// All allowed cases for ExtensionAccessor
+    ///
+    /// This includes all the defined access levels except for the ``ExtensionAccessor/unknown(_:)`` case.
     public static var allowedCases: [ExtensionAccessor] {
         [.public, .package, .internal, .fileprivate, .private, .follow]
     }
 
+    /// A textual representation of the access level.
     public var description: String {
         switch self {
         case .public: "public"
@@ -32,6 +44,10 @@ public enum ExtensionAccessor: ExpressibleByUnicodeScalarLiteral, Sendable, Coda
         }
     }
 
+    /// Initializes an `ExtensionAccessor` from a unicode scalar literal.
+    ///
+    /// - Note: It tries to match the provided string with valid levels specified in ``ExtensionAccessor/allowedCases``,
+    /// or default to ``ExtensionAccessor/unknown(_:)``.
     public init(unicodeScalarLiteral value: String) {
         self = ExtensionAccessor
             .allowedCases

Tests/BinaryParseKitMacroTests/BinaryParseKitStructTests.swift

@@ -761,7 +761,7 @@ extension BinaryParseKitMacroTests {
         }
 
         @Test
-        func `enum bad accessor`() {
+        func `struct bad accessor`() {
             assertMacro {
                 """
                 @ParseStruct(parsingAccessor: "invalid", printingAccessor: .invalid)