fix(api): silently filter invalid IDs from actor_ids query instead of 400

engine/packages/api-peer/src/actors/list.rs

@@ -1,5 +1,5 @@
 use anyhow::{Result, bail};
-use rivet_api_builder::{ApiBadRequest, ApiCtx};
+use rivet_api_builder::ApiCtx;
 use rivet_api_types::{actors::list::*, pagination::Pagination};
 
 #[utoipa::path(
@@ -20,17 +20,9 @@ pub async fn list(ctx: ApiCtx, _path: (), query: ListQuery) -> Result<ListRespon
 			.actor_ids
 			.map(|x| {
 				x.split(',')
-					.map(|s| {
-						s.trim().parse::<rivet_util::Id>().map_err(|e| {
-							ApiBadRequest {
-								reason: format!("invalid id in `actor_ids` query: {e}"),
-							}
-							.build()
-						})
-					})
-					.collect::<Result<Vec<_>>>()
+					.filter_map(|s| s.trim().parse::<rivet_util::Id>().ok())
+					.collect::<Vec<_>>()
 			})
-			.transpose()?
 			.unwrap_or_default(),
 	]
 	.concat();

engine/packages/api-public/src/actors/list.rs

@@ -1,7 +1,7 @@
 use anyhow::Result;
 use axum::response::{IntoResponse, Response};
 use rivet_api_builder::{
-	ApiBadRequest, ApiError,
+	ApiError,
 	extract::{Extension, Json, Query},
 };
 use rivet_api_types::{actors::list::*, pagination::Pagination};
@@ -63,17 +63,9 @@ async fn list_inner(ctx: ApiCtx, query: ListQuery) -> Result<ListResponse> {
 			.as_ref()
 			.map(|x| {
 				x.split(',')
-					.map(|s| {
-						s.trim().parse::<rivet_util::Id>().map_err(|e| {
-							ApiBadRequest {
-								reason: format!("invalid id in `actor_ids` query: {e}"),
-							}
-							.build()
-						})
-					})
-					.collect::<Result<Vec<_>>>()
+					.filter_map(|s| s.trim().parse::<rivet_util::Id>().ok())
+					.collect::<Vec<_>>()
 			})
-			.transpose()?
 			.unwrap_or_default(),
 	]
 	.concat();

engine/packages/engine/tests/envoy/api_actors_list.rs

@@ -1055,7 +1055,6 @@ fn list_default_limit_100() {
 // Broken legacy Pegboard Runner coverage: full `runner::` sweep times out with
 // `test timed out: Elapsed(())`.
 #[test]
-#[ignore = "API rejects mixed-validity actor_ids instead of filtering"]
 fn list_with_invalid_actor_id_format_in_comma_list() {
 	common::run(common::TestOpts::new(1).with_timeout(30), |ctx| async move {
 		let (namespace, _, _runner) =