From cfe4e298ca52402e558fa244e3d9296590e7e9a7 Mon Sep 17 00:00:00 2001 From: CyberL1 Date: Tue, 28 Oct 2025 16:43:45 +0100 Subject: [PATCH] fix: check if components are beetwen 1 to 5 in each row --- .../#interaction_token/callback.ts | 22 ++++++++++++++++++- 1 file changed, 21 insertions(+), 1 deletion(-) diff --git a/src/api/routes/interactions/#interaction_id/#interaction_token/callback.ts b/src/api/routes/interactions/#interaction_id/#interaction_token/callback.ts index 8554c41c..a91326db 100644 --- a/src/api/routes/interactions/#interaction_id/#interaction_token/callback.ts +++ b/src/api/routes/interactions/#interaction_id/#interaction_token/callback.ts @@ -19,13 +19,33 @@ import { InteractionCallbackSchema, InteractionCallbackType, MessageCreateSchema, MessageType } from "@spacebar/schemas"; import { route } from "@spacebar/api"; import { Request, Response, Router } from "express"; -import { emitEvent, InteractionSuccessEvent, Message, MessageCreateEvent, pendingInteractions, User } from "@spacebar/util"; +import { emitEvent, FieldErrors, InteractionSuccessEvent, Message, MessageCreateEvent, pendingInteractions, User } from "@spacebar/util"; const router = Router({ mergeParams: true }); router.post("/", route({}), async (req: Request, res: Response) => { const body = req.body as InteractionCallbackSchema; + // TODO: Only one error is ever returned. That's not the case on discord's side + const errors: Record = {}; + + for (const row of body.data.components || []) { + if (!row.components) { + continue; + } + + if (row.components.length < 1 || row.components.length > 5) { + errors[`data.components[${body.data.components!.indexOf(row)}].components`] = { + code: "BASE_TYPE_BAD_LENGTH", + message: `Must be between 1 and 5 in length.`, + }; + } + } + + if (Object.keys(errors).length > 0) { + throw FieldErrors(errors); + } + const interactionId = req.params.interaction_id; const interaction = pendingInteractions.get(req.params.interaction_id);