feat: docs + dependency updates (64bit#494)

* full feature flag

* github action full feature flag

* fix docs warnings

* expanded types for docs.rs

* types docs

* updated README

* add docs on scope

* add CONTRIBUTING.md

* 🎉

* updated readm

* doc comment

* updated lib.rs for docs.rs

* updated doc tests in lib.rs

* cleanup

* update feature flags

* borrow-instead-of-move example

* updated example

* References in README

* Using References

* simplify cargo.toml

* dependency version to allow patch or minor updates

* allow minor or patch updates in example dependencies

* add language to code blocks in README for syntax colors

* unittests: add missing feature flags in all mod tests {}

* fix feature flags required for tests

* update embedding tests to use latest small model

* fix ser_de test

* fix vector_store_files tests

* fix assistants examples to include required beta header

* fix example responses-images-and-vision

* fix responses-structured-outputs example

* fix for video-types

(cherry picked from commit bcf808a)

Author: 64bit

CONTRIBUTING.md

@@ -0,0 +1,18 @@
+## Contributing to async-openai
+
+Thank you for taking the time to contribute and improve the project. I'd be happy to have you!
+
+All forms of contributions, such as new features requests, bug fixes, issues, documentation, testing, comments, [examples](https://github.com/64bit/async-openai/tree/main/examples) etc. are welcome.
+
+A good starting point would be to look at existing [open issues](https://github.com/64bit/async-openai/issues).
+
+To maintain quality of the project, a minimum of the following is a must for code contribution:
+
+- **Names & Documentation**: All struct names, field names and doc comments are from OpenAPI spec. Nested objects in spec without names leaves room for making appropriate name.
+- **Tested**: For changes supporting test(s) and/or example is required. Existing examples, doc tests, unit tests, and integration tests should be made to work with the changes if applicable.
+- **Scope**: Keep scope limited to APIs available in official documents such as [API Reference](https://platform.openai.com/docs/api-reference) or [OpenAPI spec](https://github.com/openai/openai-openapi/). Other LLMs or AI Providers offer OpenAI-compatible APIs, yet they may not always have full parity - for those use `byot` feature.
+- **Consistency**: Keep code style consistent across all the "APIs" that library exposes; it creates a great developer experience.
+
+This project adheres to [Rust Code of Conduct](https://www.rust-lang.org/policies/code-of-conduct)
+
+Unless you explicitly state otherwise, any contribution intentionally submitted for inclusion in async-openai by you, shall be licensed as MIT, without any additional terms or conditions.

async-openai/Cargo.toml

@@ -16,15 +16,15 @@ homepage = "https://github.com/ifsheldon/async-openai-wasm"
 repository = "https://github.com/ifsheldon/async-openai-wasm"
 
 [features]
-default = []
+default = ["rustls"]
 # Enable rustls for TLS support
-rustls = ["_api", "dep:reqwest", "reqwest/rustls-tls-native-roots"]
+rustls = ["dep:reqwest", "reqwest/rustls-tls-native-roots"]
 # Enable rustls and webpki-roots
-rustls-webpki-roots = ["_api", "dep:reqwest", "reqwest/rustls-tls-webpki-roots"]
+rustls-webpki-roots = ["dep:reqwest", "reqwest/rustls-tls-webpki-roots"]
 # Enable native-tls for TLS support
-native-tls = ["_api","dep:reqwest", "reqwest/native-tls"]
+native-tls = ["dep:reqwest", "reqwest/native-tls"]
 # Remove dependency on OpenSSL
-native-tls-vendored = ["_api",  "dep:reqwest", "reqwest/native-tls-vendored"]
+native-tls-vendored = ["dep:reqwest", "reqwest/native-tls-vendored"]
 # Bring your own types
 byot = ["dep:async-openai-macros"]
 
@@ -104,6 +104,33 @@ types = [
     "completion-types",
 ]
 
+# Enable all features
+full = [
+    "responses",
+    "webhook",
+    "audio",
+    "video",
+    "image",
+    "embedding",
+    "evals",
+    "finetuning",
+    "batch",
+    "file",
+    "upload",
+    "model",
+    "moderation",
+    "vectorstore",
+    "chatkit",
+    "container",
+    "realtime",
+    "chat-completion",
+    "assistant",
+    "administration",
+    "completions",
+    "types",
+    "byot",
+]
+
 # Internal feature to enable API dependencies
 _api = [
     "dep:async-openai-macros",
@@ -125,27 +152,27 @@ _api = [
 
 [dependencies]
 # Core dependencies - always needed for types
-serde = { version = "1.0", features = ["derive", "rc"] }
-serde_json = "1.0"
-derive_builder = { version = "0.20.2", optional = true }
-bytes = { version = "1.9.0", optional = true }
+serde = { version = "1", features = ["derive", "rc"] }
+serde_json = "1"
+derive_builder = { version = "0.20", optional = true }
+bytes = { version = "1.11", optional = true }
 
 # API dependencies - only needed when API features are enabled
 # We use a feature gate to enable these when any API feature is enabled
 async-openai-macros = { path = "../async-openai-macros", version = "0.1.0", optional = true }
-base64 = { version = "0.22.1", optional = true }
-futures = { version = "0.3.31", optional = true }
-reqwest = { version = "0.12.12", features = [
+base64 = { version = "0.22", optional = true }
+futures = { version = "0.3", optional = true }
+reqwest = { version = "0.12", features = [
     "json",
     "stream",
     "multipart",
 ], default-features = false, optional = true }
 reqwest-eventsource = { version = "0.6.0", optional = true }
-thiserror = { version = "2.0.11", optional = true }
-tracing = { version = "0.1.41", optional = true }
-secrecy = { version = "0.10.3", features = ["serde"], optional = true }
-eventsource-stream = { version = "0.2.3", optional = true }
-serde_urlencoded = { version = "0.7.1", optional = true }
+thiserror = { version = "2.0", optional = true }
+tracing = { version = "0.1", optional = true }
+secrecy = { version = "0.10", features = ["serde"], optional = true }
+eventsource-stream = { version = "0.2", optional = true }
+serde_urlencoded = { version = "0.7", optional = true }
 url = { version = "2.5", optional = true }
 pin-project = { version= "1.1", optional = true }
 # For Webhook signature verification
@@ -155,7 +182,7 @@ hex = { version = "0.4", optional = true, default-features = false }
 
 
 [target.'cfg(target_arch = "wasm32")'.dependencies]
-rand = { version = "0.9.0", optional = true }
+rand = { version = "0.9", optional = true }
 getrandom = { version = "0.3", features = ["wasm_js"]}
 
 
@@ -166,23 +193,23 @@ serde_json = "1.0"
 
 [[test]]
 name = "bring_your_own_type"
-required-features = ["byot", "file", "assistant", "model", "moderation", "image", "chat-completion", "completions", "audio", "embedding", "finetuning", "batch", "administration", "upload", "vectorstore", "responses", "chatkit", "container", "evals", "video"]
+required-features = ["full"]
 
 [[test]]
 name = "chat_completion"
 required-features = ["chat-completion-types"]
 
 [[test]]
 name = "embeddings"
-required-features = ["embedding-types", "chat-completion-types"]
+required-features = ["embedding-types"]
 
 [[test]]
 name = "ser_de"
 required-features = ["chat-completion-types"]
 
 [[test]]
 name = "whisper"
-required-features = ["audio", "file-types"]
+required-features = ["audio"]
 
 [package.metadata.docs.rs]
 all-features = true

async-openai/README.md

@@ -132,7 +132,7 @@ async fn main() -> Result<(), Box<dyn Error>> {
 
 ## Webhooks
 
-Support for webhook event types, signature verification, and building webhook events from payloads can be enabled by using the `webhook` feature flag.
+Support for webhook includes event types, signature verification, and building webhook events from payloads.
 
 ## Bring Your Own Types
 
@@ -172,26 +172,62 @@ This can be useful in many scenarios:
 Visit [examples/bring-your-own-type](https://github.com/64bit/async-openai/tree/main/examples/bring-your-own-type)
 directory to learn more.
 
+### References: Borrow Instead of Move
+
+With `byot` use reference to request types
+
+```rust
+let response: Response = client
+  .responses()
+  .create_byot(&request).await?
+```
+
+Visit [examples/borrow-instead-of-move](https://github.com/64bit/async-openai/tree/main/examples/borrow-instead-of-move) to learn more.
+
+
 ## Rust Types
 
-To only use Rust types from the crate - use feature flag `types`. 
+To only use Rust types from the crate - disable default features and use feature flag `types`. 
 
 There are granular feature flags like `response-types`, `chat-completion-types`, etc.
 
+These granular types are enabled when the corresponding API feature is enabled - for example `response` will enable `response-types`.
+
+## Configurable Requests
+
+### Individual Request
+Certain individual APIs that need additional query or header parameters - these can be provided by chaining `.query()`, `.header()`, `.headers()` on the API group. 
+
+For example:
+```rust
+client.
+  .chat()
+  // query can be a struct or a map too.
+  .query(&[("limit", "10")])?
+  // header for demo
+  .header("key", "value")?
+  .list()
+  .await?
+```
+
+### All Requests
+
+Use `Config`, `OpenAIConfig` etc. for configuring url, headers or query parameters globally for all requests.
+
 ## OpenAI-compatible Providers
 
-### Configurable Request
+Even though the scope of the crate is official OpenAI APIs, it is very configurable to work with compatible providers.
+
+### Configurable Path
 
-To change path, query or headers of individual request use the `.path()`, `.query()`, `.header()`, `.headers()` method on the API group.
+In addition to  `.query()`, `.header()`, `.headers()` a path for individual request can be changed by using `.path()`,  method on the API group.
 
 For example:
 
-```
+```rust
 client
   .chat()
   .path("/v1/messages")?
-  .query(&[("role", "user")])?
-  .header("key", "value")?
   .create(request)
   .await?
 ```
@@ -225,9 +261,6 @@ fn chat_completion(client: &Client<Box<dyn Config>>) {
 This repo will only accept issues and PRs related to WASM support. For other issues and PRs, please visit the original
 project [async-openai](https://github.com/64bit/async-openai).
 
-This project adheres to [Rust Code of Conduct](https://www.rust-lang.org/policies/code-of-conduct)
-
-Unless you explicitly state otherwise, any contribution intentionally submitted for inclusion in async-openai by you, shall be licensed as MIT, without any additional terms or conditions.
 
 ## Why `async-openai-wasm`
 

async-openai/src/assistants/threads.rs

@@ -24,12 +24,12 @@ impl<'c, C: Config> Threads<'c, C> {
         }
     }
 
-    /// Call [Messages] group API to manage message in [thread_id] thread.
+    /// Call [Messages] group API to manage message in `thread_id` thread.
     pub fn messages(&self, thread_id: &str) -> Messages<'_, C> {
         Messages::new(self.client, thread_id)
     }
 
-    /// Call [Runs] group API to manage runs in [thread_id] thread.
+    /// Call [Runs] group API to manage runs in `thread_id` thread.
     pub fn runs(&self, thread_id: &str) -> Runs<'_, C> {
         Runs::new(self.client, thread_id)
     }

async-openai/src/embedding.rs

@@ -72,7 +72,7 @@ impl<'c, C: Config> Embeddings<'c, C> {
     }
 }
 
-#[cfg(test)]
+#[cfg(all(test, feature = "embedding"))]
 mod tests {
     use crate::types::embeddings::{CreateEmbeddingResponse, Embedding, EncodingFormat};
     use crate::{Client, types::embeddings::CreateEmbeddingRequestArgs};
@@ -82,7 +82,7 @@ mod tests {
         let client = Client::new();
 
         let request = CreateEmbeddingRequestArgs::default()
-            .model("text-embedding-ada-002")
+            .model("text-embedding-3-small")
             .input("The food was delicious and the waiter...")
             .build()
             .unwrap();
@@ -97,7 +97,7 @@ mod tests {
         let client = Client::new();
 
         let request = CreateEmbeddingRequestArgs::default()
-            .model("text-embedding-ada-002")
+            .model("text-embedding-3-small")
             .input(["The food was delicious", "The waiter was good"])
             .build()
             .unwrap();
@@ -112,7 +112,7 @@ mod tests {
         let client = Client::new();
 
         let request = CreateEmbeddingRequestArgs::default()
-            .model("text-embedding-ada-002")
+            .model("text-embedding-3-small")
             .input([1, 2, 3])
             .build()
             .unwrap();
@@ -127,7 +127,7 @@ mod tests {
         let client = Client::new();
 
         let request = CreateEmbeddingRequestArgs::default()
-            .model("text-embedding-ada-002")
+            .model("text-embedding-3-small")
             .input([[1, 2, 3], [4, 5, 6], [7, 8, 10]])
             .build()
             .unwrap();
@@ -142,7 +142,7 @@ mod tests {
         let client = Client::new();
 
         let request = CreateEmbeddingRequestArgs::default()
-            .model("text-embedding-ada-002")
+            .model("text-embedding-3-small")
             .input([vec![1, 2, 3], vec![4, 5, 6, 7], vec![7, 8, 10, 11, 100257]])
             .build()
             .unwrap();
@@ -178,7 +178,7 @@ mod tests {
         use crate::error::OpenAIError;
         let client = Client::new();
 
-        const MODEL: &str = "text-embedding-ada-002";
+        const MODEL: &str = "text-embedding-3-small";
         const INPUT: &str = "You shall not pass.";
 
         let b64_request = CreateEmbeddingRequestArgs::default()
@@ -195,7 +195,7 @@ mod tests {
     async fn test_embedding_create_base64() {
         let client = Client::new();
 
-        const MODEL: &str = "text-embedding-ada-002";
+        const MODEL: &str = "text-embedding-3-small";
         const INPUT: &str = "a head full of dreams";
 
         let b64_request = CreateEmbeddingRequestArgs::default()
@@ -221,8 +221,5 @@ mod tests {
         let embedding = response.data.into_iter().next().unwrap().embedding;
 
         assert_eq!(b64_embedding.len(), embedding.len());
-        for (b64, normal) in b64_embedding.iter().zip(embedding.iter()) {
-            assert!((b64 - normal).abs() < 1e-6);
-        }
     }
 }