Fix the permissions piping

Author: wheregmis

packages/permissions/permissions-macro/src/permission.rs

@@ -32,42 +32,20 @@ impl ToTokens for PermissionParser {
         self.expr.to_string().hash(&mut hash);
         let permission_hash = format!("{:016x}", hash.finish());
 
-        // Check if this is a Custom permission by examining the expression
-        // Custom permissions are built via PermissionBuilder::custom() or contain PermissionKind::Custom
-        let expr_str = self.expr.to_string();
-        let is_custom = expr_str.contains("custom()")
-            || expr_str.contains("Custom {")
-            || expr_str.contains("PermissionKind::Custom");
-
         let expr = &self.expr;
+        let link_section =
+            crate::linker::generate_link_section(quote!(__PERMISSION), &permission_hash);
 
-        if is_custom {
-            // For Custom permissions, skip linker section generation due to buffer size limitations
-            // Custom permissions can exceed the 4096 byte buffer limit when serialized
-            tokens.extend(quote! {
-                {
-                    // Create the permission instance directly for Custom permissions
-                    // Skip linker section generation due to buffer size limitations
-                    const __PERMISSION: permissions_core::Permission = #expr;
-                    __PERMISSION
-                }
-            });
-        } else {
-            // For regular permissions, use the normal serialization approach with linker sections
-            let link_section =
-                crate::linker::generate_link_section(quote!(__PERMISSION), &permission_hash);
-
-            tokens.extend(quote! {
-                {
-                    // Create the permission instance from the expression
-                    const __PERMISSION: permissions_core::Permission = #expr;
+        tokens.extend(quote! {
+            {
+                // Create the permission instance from the expression
+                const __PERMISSION: permissions::Permission = #expr;
 
-                    #link_section
+                #link_section
 
-                    // Return the permission
-                    __PERMISSION
-                }
-            });
-        }
+                // Return the permission
+                __PERMISSION
+            }
+        });
     }
 }

packages/permissions/permissions/README.md

@@ -17,16 +17,24 @@ This crate provides a unified API for declaring permissions across supported pla
 ### Basic Permission Declaration
 
 ```rust
-use permissions::{static_permission, Permission};
+use permissions::{static_permission, Permission, PermissionBuilder, PermissionKind};
 
-// Declare a camera permission
-const CAMERA: Permission = static_permission!(Camera, description = "Take photos");
+// Declare a camera permission using the builder
+const CAMERA: Permission = static_permission!(
+    Permission::new(PermissionKind::Camera, "Take photos")
+);
 
-// Declare a location permission with precision
-const LOCATION: Permission = static_permission!(Location(Fine), description = "Track your runs");
+// Declare a fine-grained location permission
+const LOCATION: Permission = static_permission!(
+    PermissionBuilder::location(permissions::LocationPrecision::Fine)
+        .with_description("Track your runs")
+        .build()
+);
 
 // Declare a microphone permission
-const MICROPHONE: Permission = static_permission!(Microphone, description = "Record audio");
+const MICROPHONE: Permission = static_permission!(
+    Permission::new(PermissionKind::Microphone, "Record audio")
+);
 ```
 
 ### Custom Permissions (For Untested or Special Use Cases)
@@ -35,16 +43,16 @@ For permissions that aren't yet tested or for special use cases, use the `Custom
 with platform-specific identifiers:
 
 ```rust
-use permissions::{static_permission, Permission};
+use permissions::{static_permission, Permission, PermissionBuilder};
 
 // Example: Request storage permission
 const STORAGE: Permission = static_permission!(
-    Custom { 
-        android = "android.permission.READ_EXTERNAL_STORAGE",
-        ios = "NSPhotoLibraryUsageDescription",
-        macos = "NSPhotoLibraryUsageDescription"
-    },
-    description = "Access files on your device"
+    PermissionBuilder::custom()
+        .with_android("android.permission.READ_EXTERNAL_STORAGE")
+        .with_ios("NSPhotoLibraryUsageDescription")
+        .with_macos("NSPhotoLibraryUsageDescription")
+        .with_description("Access files on your device")
+        .build()
 );
 ```
 
@@ -55,9 +63,13 @@ const STORAGE: Permission = static_permission!(
 ### Using Permissions
 
 ```rust
-use permissions::{static_permission, Permission, Platform};
+use permissions::{
+    static_permission, Permission, PermissionBuilder, PermissionKind, Platform,
+};
 
-const CAMERA: Permission = static_permission!(Camera, description = "Take photos");
+const CAMERA: Permission = static_permission!(
+    Permission::new(PermissionKind::Camera, "Take photos")
+);
 
 // Get the description
 println!("Description: {}", CAMERA.description());

packages/permissions/permissions/src/lib.rs

@@ -30,8 +30,8 @@
 //! > to preserve backward compatibility with existing code.
 
 pub use permissions_core::{
-    LocationPrecision, Permission, PermissionKind, PermissionManifest, Platform, PlatformFlags,
-    PlatformIdentifiers,
+    CustomPermissionBuilder, LocationPrecision, Permission, PermissionBuilder, PermissionKind,
+    PermissionManifest, Platform, PlatformFlags, PlatformIdentifiers,
 };
 pub use permissions_macro::{permission, static_permission};