feat(stubs): Proper phpdoc-style comments in stubs #369 (#676)

Author: kakserpom

crates/cli/src/ext.rs

@@ -2,7 +2,7 @@ use std::path::PathBuf;
 
 use anyhow::{Context, Result};
 use ext_php_rs::describe::Description;
-use libloading::os::unix::{Library, Symbol};
+use libloading::os::unix::{Library, RTLD_LAZY, RTLD_LOCAL, Symbol};
 
 #[allow(improper_ctypes_definitions)]
 pub struct Ext {
@@ -17,7 +17,16 @@ pub struct Ext {
 impl Ext {
     /// Loads an extension.
     pub fn load(ext_path: PathBuf) -> Result<Self> {
-        let ext_lib = unsafe { Library::new(ext_path) }
+        // On macOS, add RTLD_FIRST for two-level namespace which properly defers
+        // resolution of symbols not needed for stub generation.
+        #[cfg(target_os = "macos")]
+        let ext_lib =
+            unsafe { Library::open(Some(ext_path), RTLD_LAZY | RTLD_LOCAL | libc::RTLD_FIRST) }
+                .with_context(|| "Failed to load extension library")?;
+
+        // On other Unix platforms, RTLD_LAZY | RTLD_LOCAL is sufficient
+        #[cfg(not(target_os = "macos"))]
+        let ext_lib = unsafe { Library::open(Some(ext_path), RTLD_LAZY | RTLD_LOCAL) }
             .with_context(|| "Failed to load extension library")?;
 
         let describe_fn = unsafe {

crates/cli/tests/snapshots/stubs_snapshot__hello_world_stubs.snap

@@ -1,13 +1,17 @@
 ---
 source: crates/cli/tests/stubs_snapshot.rs
-assertion_line: 69
+assertion_line: 74
 expression: stubs
 ---
 <?php
 
 // Stubs for ext-php-rs
 
 namespace {
+    const CONST_NAME = 100;
+
+    const HELLO_WORLD = 100;
+
     class TestClass {
         const NEW_CONSTANT_NAME = 5;
 
@@ -17,22 +21,43 @@ namespace {
 
         public $b;
 
+        /**
+         * @param int $a
+         * @param int $b
+         */
         public function __construct(int $a, int $b) {}
 
+        /**
+         * @return \TestClass
+         */
         public function builderPattern(): \TestClass {}
 
+        /**
+         * @param int $a
+         * @param int $test
+         * @return void
+         */
         public function testCamelCase(int $a = 5, int $test = 100): void {}
 
+        /**
+         * @return int
+         */
         public static function x(): int {}
     }
 
-    const CONST_NAME = 100;
-
-    const HELLO_WORLD = 100;
-
+    /**
+     * @param object $z
+     * @return int
+     */
     function get_zval_convert(object $z): int {}
 
+    /**
+     * @return string
+     */
     function hello_world(): string {}
 
+    /**
+     * @return \TestClass
+     */
     function new_class(): \TestClass {}
 }