fix: Update vector store file chunking strategy to use StaticChunkingStrategy (64bit#230)

* fix: Update vector store file chunking strategy to use StaticChunkingStrategy

This targets the changes mentioned in 64bit#229, because OpenAI requires a non-null name when creating a vector store. This also fixes attaching a file to a vector store, where it would fail to deserialize. Also adds a test for both errors to catch them during development in the future!

64bit#229

* bugfix: Made optional name field consistent to OpenAI spec

Reverted according to recent comments on 64bit#229, reasoning can be found on thread.

* chore: Modified redudant code in test

Related details can be found in PR 64bit#230

* test: Update tests to double check for a failure in add file to vector store

Related PR: 64bit#230

Co-authored-by: Himanshu Neema <himanshun.iitkgp@gmail.com>

---------

Co-authored-by: Himanshu Neema <himanshun.iitkgp@gmail.com>

Author: hiibolt

async-openai/src/types/vector_store.rs

@@ -63,7 +63,7 @@ pub struct VectorStoreObject {
     /// The Unix timestamp (in seconds) for when the vector store was created.
     pub created_at: u32,
     /// The name of the vector store.
-    pub name: String,
+    pub name: Option<String>,
     /// The total number of bytes used by the files in the vector store.
     pub usage_bytes: u64,
     pub file_counts: VectorStoreFileCounts,
@@ -192,7 +192,7 @@ pub enum VectorStoreFileErrorCode {
 pub enum VectorStoreFileObjectChunkingStrategy {
     /// This is returned when the chunking strategy is unknown. Typically, this is because the file was indexed before the `chunking_strategy` concept was introduced in the API.
     Other,
-    Static(StaticChunkingStrategy),
+    Static{ r#static: StaticChunkingStrategy },
 }
 
 #[derive(Debug, Serialize, Default, Clone, Builder, PartialEq)]

async-openai/src/vector_store_files.rs

@@ -75,3 +75,55 @@ impl<'c, C: Config> VectorStoreFiles<'c, C> {
             .await
     }
 }
+
+#[cfg(test)]
+mod tests {
+    use crate::Client;
+    use crate::types::{CreateFileRequest, CreateVectorStoreFileRequest, CreateVectorStoreRequest, FileInput, FilePurpose};
+
+    #[tokio::test]
+    async fn vector_store_file_creation_and_deletion() -> Result<(), Box<dyn std::error::Error + Send + Sync>> {
+        let client = Client::new();
+
+        // Create a file
+        let file_handle = client
+            .files()
+            .create( CreateFileRequest {
+                file: FileInput::from_vec_u8(
+                    String::from("meow.txt"),
+                    String::from(":3").into_bytes()
+                ),
+                purpose: FilePurpose::Assistants
+            }).await?;
+
+        // Create a vector store
+        let vector_store_handle = client
+            .vector_stores()
+            .create( CreateVectorStoreRequest {
+                file_ids: Some(vec![file_handle.id.clone()]),
+                name: None,
+                expires_after: None,
+                chunking_strategy: None,
+                metadata: None
+            })
+            .await?;
+    let vector_store_file = client
+        .vector_stores()
+        .files(&vector_store_handle.id)
+        .retrieve(&file_handle.id)
+        .await?;
+
+    assert_eq!(vector_store_file.id, file_handle.id);
+        // Delete the vector store
+        client
+            .vector_stores()
+            .delete(&vector_store_handle.id).await?;
+
+        // Delete the file
+        client
+            .files()
+            .delete(&file_handle.id).await?;
+
+        Ok(())
+    }
+}