ref: Use slice instead of vec for assemble artifact request

[szokeasaurusrex]

Split off from #2408

[Copilot]

Pull Request Overview

This PR updates the way projects are passed to the assemble artifact request by replacing allocations via Vec with slice references. The changes affect three files to reduce unnecessary allocations when constructing the projects list.

• Changed file_upload.rs to use slice literals instead of vectors when calling the API.
• Updated src/api/mod.rs to accept slice parameters and pass empty slice literals.
• Modified the ChunkedArtifactRequest in src/api/data_types/chunking/artifact.rs to hold a slice reference instead of a Vec.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
src/utils/file_upload.rs	Replaced vector construction with slice literal for the projects parameter
src/api/mod.rs	Adjusted the API method signature and usage to accept a slice, including empty slice usage
src/api/data_types/chunking/artifact.rs	Updated the projects field type and its serializer condition for slice references

Comments suppressed due to low confidence (3)

src/utils/file_upload.rs:461

• Using a slice literal instead of a vector reduces heap allocations. Ensure that the temporary String created by to_string() has a suitable lifetime for the API call.

+                &[context.project.unwrap().to_string()],

src/api/mod.rs:999

• Switching to an empty slice literal improves efficiency by avoiding unnecessary vector construction. Verify that the changes align with the expected lifetime requirements of the API.

+                projects: &[],

src/api/data_types/chunking/artifact.rs:10

• Updating the skip_serializing_if condition to use the slice pattern is appropriate for the new type change. Confirm that this serializer condition correctly handles empty slices.

+    #[serde(skip_serializing_if = "<[_]>::is_empty")]