Skip to content

Conversation

@ChmaraX
Copy link
Contributor

@ChmaraX ChmaraX commented Nov 19, 2025

What changed? Why was the change needed?

This pull request introduces a new Biome plugin to help developers ensure proper OpenAPI configuration when using Record<string, ...> types in DTO files. The plugin automatically warns developers to verify that @ApiProperty includes the correct settings for SDK generation. Additionally, the plugin is registered in the Biome configuration.

Biome plugin addition and configuration:

  • Added biome-plugins/api-property-record-type.grit plugin to detect usage of Record<string, ...> types and warn developers to verify @ApiProperty includes type: 'object' and additionalProperties: true for SDK generation.
  • Registered the new plugin in biome.json by adding it to the plugins array.

@linear
Copy link

linear bot commented Nov 19, 2025

@netlify
Copy link

netlify bot commented Nov 19, 2025

Deploy Preview for dashboard-v2-novu-staging canceled.

Name Link
🔨 Latest commit bc0fbd8
🔍 Latest deploy log https://app.netlify.com/projects/dashboard-v2-novu-staging/deploys/691dc9c8da050f0008e25abc

@github-actions github-actions bot changed the title chore(root): add biome plugin to validate Record types in DTOs chore(root): add biome plugin to validate Record types in DTOs fixes NV-6905 Nov 19, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Nov 19, 2025

PR title automatically updated!

I found the Linear ticket ID chore(root): add biome plugin to validate Record types in DTOs fixes NV-6905 in your branch name and automatically added it to your PR title.

Updated title: chore(root): add biome plugin to validate Record types in DTOs fixes NV-6905

@ChmaraX ChmaraX merged commit efb53f9 into next Nov 19, 2025
28 checks passed
@ChmaraX ChmaraX deleted the nv-6905-biome-rule-to-detect-flexible-objects-in-dto branch November 19, 2025 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants