Mark take and page as optional in Swagger for CreateCombinationDto

[Copilot]

The take and page fields in CreateCombinationDto were marked as optional at the validation layer but required in Swagger documentation, causing type generation tools and API consumers to treat them as mandatory.

Changes

• Added required: false to @ApiProperty() decorators for take and page fields in CreateCombinationDto

These fields already had @IsOptional() validators and the service handles them with defaults (take ?? 5, page for offset calculation), but the OpenAPI spec was incorrectly marking them as required.

@ApiProperty({
  type: Number,
  description: 'Número de prendas a buscar por categorías',
  required: false,  // Added
})
@IsOptional()
@IsNumber()
take: number;

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

• checkpoint.prisma.io
  • Triggering command: /usr/local/bin/node /usr/local/bin/node /home/REDACTED/work/StyleMind_Backend/StyleMind_Backend/node_modules/prisma/build/child {"product":"prisma","version":"6.18.0","cli_install_type":"local","information":"","local_timestamp":"2025-12-11T00:49:40Z","project_hash":"6b0c20ab","cli_path":"/home/REDACTED/work/StyleMind_Backend/StyleMind_Backend/node_modules/@prisma/client/node_module (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

• Configure Actions setup steps to set up my environment, which run before the firewall is enabled
• Add the appropriate URLs or hosts to the custom allowlist in this repository's Copilot coding agent settings (admins only)

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Propiedades page y take deberian ser opcionales</issue_title>
<issue_description>Creo que estas propiedades al generar un outfit deberían ser opcionales ya que no es algo principal a la hora de formar un outfit, ademas que este endpoint /api/combinations/generate se va a utilizar para generar rapidamente un outfit con prendas bases seleccionadas</issue_description>

<agent_instructions>Necesito que en mi DTO las propiedades take y page sean opcionales tanto para Swagger como para el tipo TypeScript. Por favor revisa también el controller para asegurar que estas propiedades efectivamente se traten como opcionales en las validaciones y en la lógica. Si no son opcionales actualmente, ajusta el DTO y el controller para que lo sean.</agent_instructions>

Comments on the Issue (you are @copilot in this section)

@IvanM9 Correcto, actualmente se encuentran como opcionales y sirven para el caso de que no se quiera buscar con todas las prendas de las categorías seleccionadas, sino un número limitado de prendas @josueJouvin > Correcto, actualmente se encuentran como opcionales y sirven para el caso de que no se quiera buscar con todas las prendas de las categorías seleccionadas, sino un número limitado de prendas

Segun el schema y ya cuando lo utilizo y veo el type están como obligatorio:

</comment_new>
<comment_new>@IvanM9
Veo el problema, en la documentación del swagger no se ha definido como opcional, pero a nivel de validación sí está</comment_new>
<comment_new>@josueJouvin

Veo el problema, en la documentación del swagger no se ha definido como opcional, pero a nivel de validación sí está

pondre a copilot a realizar la actualizacion del swagger y todo lo relacionado a ello</comment_new>

• Fixes Propiedades page y take deberian ser opcionales #62

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[IvanM9]

Piece of cake