Do not generate fetchRequest

There is no need to generate this, it can be looked
up at runtime.
Also provide a convenience variant that takes a typed
sort descriptor.

Author: helje5

Sources/ManagedModelMacros/ModelMacro/ModelMembers.swift

@@ -15,7 +15,6 @@ import SwiftDiagnostics
  *   named(init),           // Initializers.
  *   named(schemaMetadata), // The metadata.
  *   named(entity),         // Override the `entity()` function.
- *   named(fetchRequest),   // The fetchRequest factory
  *   named(_$entity),       // The cached the Entity
  *   named(_$originalName),
  *   named(_$hashModifier)
@@ -49,23 +48,6 @@ extension ModelMacro: MemberMacro { // @attached(member, names:...)
       initializers: findInitializers(in: classDecl)
     )
 
-    if classDecl.findFunctionWithName("fetchRequest", isStaticOrClass: true,
-                                      numberOfParametersWithoutDefaults: 0)
-       == nil
-    {
-      let modelClassName = modelClassName.text
-      newMembers.append(
-        """
-        /// Returns an `NSFetchRequest` setup for the `\(raw: modelClassName)`.
-        @nonobjc \(raw: access)class func fetchRequest() -> CoreData.NSFetchRequest<\(raw: modelClassName)> {
-            let fetchRequest = CoreData.NSFetchRequest<\(raw: modelClassName)>(entityName: "\(raw: modelClassName)")
-            fetchRequest.entity = Self._$entity
-            return fetchRequest
-        }
-        """
-      )
-    }
-    
     let metadata = generateMetadataSlot(
       access: access,
       modelClassName: modelClassName,

Sources/ManagedModels/ModelMacroDefinition.swift

@@ -82,7 +82,6 @@ public macro _PersistedProperty() =
   named(init),           // Initializers.
   named(schemaMetadata), // The metadata.
   named(entity),         // Override the `entity()` function.
-  named(fetchRequest),   // The fetchRequest factory
   named(_$entity),       // The cached the Entity
   named(_$originalName),
   named(_$hashModifier)

Sources/ManagedModels/PersistentModel/PersistentModel.swift

@@ -59,3 +59,36 @@ extension PersistentModel {
   @inlinable
   public static var _$entity : NSEntityDescription { self.entity() }
 }
+
+public extension PersistentModel {
+
+  @inlinable
+  static func fetchRequest() -> NSFetchRequest<Self> {
+    NSFetchRequest<Self>(entityName: _$entity.name ?? NSStringFromClass(self))
+  }
+  
+  @inlinable
+  static func fetchRequest<T>(filter         : NSPredicate? = nil,
+                              sortBy keyPath : KeyPath<Self, T>,
+                              order: NSSortDescriptor.SortOrder = .forward,
+                              fetchOffset    : Int? = nil,
+                              fetchLimit     : Int? = nil)
+              -> NSFetchRequest<Self>
+  {
+    let fetchRequest = Self.fetchRequest()
+    fetchRequest.predicate = filter
+    if let meta = Self.schemaMetadata.first(where: { $0.keypath == keyPath }) {
+      fetchRequest.sortDescriptors = [
+        NSSortDescriptor(key: meta.name, ascending: order == .forward)
+      ]
+    }
+    else {
+      fetchRequest.sortDescriptors = [
+        NSSortDescriptor(keyPath: keyPath, ascending: order == .forward)
+      ]
+    }
+    if let fetchOffset { fetchRequest.fetchOffset = fetchOffset }
+    if let fetchLimit  { fetchRequest.fetchLimit  = fetchLimit  }
+    return fetchRequest
+  }
+}