Resolve auth FFI bindings and rename package to gopher-mcp-rust (#2)

FFI fixes:
- Fix library name: libgopher_auth -> libgopher-orch (the actual library)
- Fix gopher_auth_client_create parameter order to match C API
  (output param first, not last)
- Fix symbol name: gopher_auth_set_option -> gopher_auth_client_set_option
- Add DYLD_LIBRARY_PATH/LD_LIBRARY_PATH to library search paths

Package rename:
- Rename package from gopher-orch to gopher-mcp-rust
- Update lib name to gopher_mcp_rust
- Update all imports in examples/auth to use new crate name
- Use path dependency for local development

Author: RahulHere

Cargo.toml

@@ -1,16 +1,16 @@
 [package]
-name = "gopher-orch"
-version = "0.1.0"
+name = "gopher-mcp-rust"
+version = "0.1.2-9"
 edition = "2021"
 authors = ["GopherSecurity"]
 description = "Rust SDK for Gopher Orch - AI Agent orchestration framework"
-license = "MIT"
+license = "Apache-2.0"
 repository = "https://github.com/GopherSecurity/gopher-mcp-rust"
 keywords = ["ai", "agent", "mcp", "llm", "orchestration"]
 categories = ["api-bindings", "development-tools"]
 
 [lib]
-name = "gopher_orch"
+name = "gopher_mcp_rust"
 path = "src/lib.rs"
 
 [[example]]

examples/auth/Cargo.toml

@@ -2,7 +2,7 @@
 name = "auth-mcp-server"
 version = "0.1.0"
 edition = "2021"
-description = "OAuth-protected MCP server example using gopher-orch SDK"
+description = "OAuth-protected MCP server example using gopher-mcp-rust SDK"
 license = "MIT"
 
 [dependencies]
@@ -21,9 +21,9 @@ serde_json = "1"
 # HTTP types
 http = "1"
 
-# Gopher Orch SDK with auth FFI
-# Use git dependency for standalone example
-gopher-orch = { git = "https://github.com/GopherSecurity/gopher-mcp-rust.git", features = ["auth"] }
+# Gopher MCP Rust SDK with auth FFI
+# Use path dependency for local development
+gopher-mcp-rust = { path = "../..", features = ["auth"] }
 
 # Logging
 tracing = "0.1"

examples/auth/src/error.rs

@@ -11,8 +11,8 @@ use axum::{
 use serde::Serialize;
 use thiserror::Error;
 
-// Re-export gopher_orch error for convenience
-pub use gopher_orch::Error as GopherOrchError;
+// Re-export gopher_mcp_rust error for convenience
+pub use gopher_mcp_rust::Error as GopherOrchError;
 
 /// Application error type.
 #[derive(Error, Debug)]

examples/auth/src/ffi/mod.rs

@@ -1,19 +1,19 @@
 //! FFI bindings module.
 //!
-//! Re-exports gopher-auth types from the gopher-orch library with a thin
+//! Re-exports gopher-auth types from the gopher-mcp-rust library with a thin
 //! wrapper to support testing without the native library.
 
 use crate::error::AppError;
 
 // Re-export payload types directly
-pub use gopher_orch::TokenPayload;
-pub use gopher_orch::ValidationResult;
+pub use gopher_mcp_rust::TokenPayload;
+pub use gopher_mcp_rust::ValidationResult;
 
-/// Wrapper around gopher-orch's GopherAuthClient.
+/// Wrapper around gopher-mcp-rust's GopherAuthClient.
 ///
 /// Provides the same interface but allows creating dummy instances for testing.
 pub struct GopherAuthClient {
-    inner: Option<gopher_orch::GopherAuthClient>,
+    inner: Option<gopher_mcp_rust::GopherAuthClient>,
 }
 
 unsafe impl Send for GopherAuthClient {}
@@ -22,7 +22,7 @@ unsafe impl Sync for GopherAuthClient {}
 impl GopherAuthClient {
     /// Create a new client.
     pub fn new(jwks_uri: &str, issuer: &str) -> Result<Self, AppError> {
-        let inner = gopher_orch::GopherAuthClient::new(jwks_uri, issuer)?;
+        let inner = gopher_mcp_rust::GopherAuthClient::new(jwks_uri, issuer)?;
         Ok(Self { inner: Some(inner) })
     }
 

src/ffi/auth.rs

@@ -73,9 +73,9 @@ pub struct TokenPayload {
 // FFI function type definitions
 type GopherAuthInitFn = unsafe extern "C" fn() -> c_int;
 type GopherAuthClientCreateFn = unsafe extern "C" fn(
+    out: *mut *mut c_void,
     jwks_uri: *const c_char,
     issuer: *const c_char,
-    out: *mut *mut c_void,
 ) -> c_int;
 type GopherAuthClientDestroyFn = unsafe extern "C" fn(client: *mut c_void);
 type GopherAuthSetOptionFn =
@@ -167,7 +167,7 @@ impl GopherAuthClient {
                 })?;
 
             let mut handle: *mut c_void = ptr::null_mut();
-            let result = create(jwks_uri_c.as_ptr(), issuer_c.as_ptr(), &mut handle);
+            let result = create(&mut handle, jwks_uri_c.as_ptr(), issuer_c.as_ptr());
 
             if result != 0 || handle.is_null() {
                 return Err(Error::auth(format!(
@@ -184,23 +184,59 @@ impl GopherAuthClient {
 
     /// Load the native library from known locations.
     fn load_library() -> Result<Library, Error> {
-        // Try multiple library names and locations
+        // Try multiple library names - the auth functions are in libgopher-orch
         let lib_names = if cfg!(target_os = "macos") {
-            vec!["libgopher_auth.dylib", "gopher_auth.dylib"]
+            vec![
+                "libgopher-orch.dylib",
+                "libgopher-orch.0.dylib",
+                "libgopher_orch.dylib",
+            ]
         } else if cfg!(target_os = "windows") {
-            vec!["gopher_auth.dll", "libgopher_auth.dll"]
+            vec!["gopher-orch.dll", "libgopher-orch.dll", "gopher_orch.dll"]
         } else {
-            vec!["libgopher_auth.so", "gopher_auth.so"]
+            vec![
+                "libgopher-orch.so",
+                "libgopher-orch.so.0",
+                "libgopher_orch.so",
+            ]
         };
 
-        // Try standard paths
-        let search_paths = vec![
+        // Build search paths including environment-specified locations
+        let mut search_paths = vec![
             String::new(), // Current directory / system paths
             String::from("./"),
-            String::from("/usr/local/lib/"),
-            String::from("/usr/lib/"),
+            String::from("./native/lib/"),
+            String::from("../native/lib/"),
         ];
 
+        // Add paths from DYLD_LIBRARY_PATH / LD_LIBRARY_PATH
+        if let Ok(lib_path) = std::env::var("DYLD_LIBRARY_PATH") {
+            for path in lib_path.split(':') {
+                if !path.is_empty() {
+                    let mut p = path.to_string();
+                    if !p.ends_with('/') {
+                        p.push('/');
+                    }
+                    search_paths.push(p);
+                }
+            }
+        }
+        if let Ok(lib_path) = std::env::var("LD_LIBRARY_PATH") {
+            for path in lib_path.split(':') {
+                if !path.is_empty() {
+                    let mut p = path.to_string();
+                    if !p.ends_with('/') {
+                        p.push('/');
+                    }
+                    search_paths.push(p);
+                }
+            }
+        }
+
+        // Add standard system paths
+        search_paths.push(String::from("/usr/local/lib/"));
+        search_paths.push(String::from("/usr/lib/"));
+
         for path in &search_paths {
             for name in &lib_names {
                 let full_path = format!("{}{}", path, name);
@@ -431,7 +467,7 @@ impl GopherAuthClient {
         unsafe {
             let set_option: Symbol<GopherAuthSetOptionFn> = self
                 .library
-                .get(b"gopher_auth_set_option\0")
+                .get(b"gopher_auth_client_set_option\0")
                 .map_err(|e| Error::auth(format!("Failed to load set_option: {}", e)))?;
 
             let result = set_option(self.handle, key_c.as_ptr(), value_c.as_ptr());