Skip to content

refactor: remove DTOs#85

Draft
tomast1337 wants to merge 27 commits intodevelopfrom
feature/admin-tools
Draft

refactor: remove DTOs#85
tomast1337 wants to merge 27 commits intodevelopfrom
feature/admin-tools

Conversation

@tomast1337
Copy link
Copy Markdown
Member

No description provided.

…ator

- Add env, Discord strategy, and song form schemas to @nbw/validation
- Wire ConfigModule to validateEnv; parse user/song/query payloads with Zod
- Replace DTO class helpers with mappers in song.util; re-export validation from @nbw/database for consumers
- Frontend: import song form schemas from @nbw/validation; remove local SongForm.zod
- Remove class-validator/class-transformer and global ValidationPipe from backend
- Update specs; note bun test may need workspace resolution fixes for @nbw/config
- Introduce a new jsonStringField function to validate and parse JSON strings, ensuring invalid JSON surfaces as a Zod issue.
- Update thumbnailData and customInstruments fields to utilize jsonStringField for improved validation.
…instructions

- Change package name to @nbw/validation and clarify its role as the source of shared Zod schemas and TypeScript types.
- Add detailed layout and script sections for building, developing, testing, linting, and cleaning the package.
- Include instructions for consumers on how to import and use the package after building.
- Delete jest.config.js as part of the cleanup process for the validation package.
…TO names

Remove common/types.ts and song/types.ts (PageQueryDTOType and *DtoType aliases).
Add song/uploadMeta.ts for VisibilityType, CategoryType, LicenseType, TimespanType.
Export SongsFolder from SongPage.dto.ts next to SongPageDto.

Point SongPreview/SongView/UploadSongDto at uploadMeta; update package index exports.
Rename frontend imports from *DtoType to the real DTO types (via @nbw/database).
…ckage dependencies

- Introduced new UI components: Button, Input, Label, and Textarea for consistent styling and functionality across the application.
- Added @radix-ui/react-label dependency to package.json for label component integration.
- Updated existing components to utilize the new UI components for improved user experience.
- Refactored ProfileBioEditor and ProfilePublicNameEditor to use the new Input and Button components.
- Changed the order field in PageQuery DTO from a union of boolean and string to an enum with 'asc' and 'desc' options for improved clarity and validation.
…ewSource type

- Updated song retrieval logic to use a more specific type for the uploader field in the song model.
- Introduced the SongPreviewSource type to standardize the structure of song preview data.
- Refactored songPreviewFromSongDocumentWithUser function to accept the new type for improved type safety and clarity.
- Changed various imports across the frontend application to source DTOs and types from the new @nbw/validation package instead of @nbw/database.
- This refactor aims to centralize validation logic and improve type safety throughout the application.
…e SongSearchParams

- Updated the SearchSongPage component to utilize the new SongSearchParams type for improved type safety.
- Refactored search logic to use constants from the SEARCH_SONGS configuration for pagination and placeholder counts.
- Cleaned up unused interfaces and enums, streamlining the codebase for better maintainability.
- Introduced a new SongSearchParams DTO to standardize search parameters across the application.
… from @nbw/config

- Removed the config-shim.ts file and updated imports in various DTOs to directly source constants from @nbw/config.
- This change simplifies the import structure and enhances clarity across the validation package.
…ring

- Modified the UserController to return paginated user data filtered by email, ID, or username.
- Updated the UserService to handle pagination and filtering logic, ensuring consistent query handling.
- Adjusted tests to reflect the new functionality and ensure proper coverage for paginated user retrieval.
- Removed deprecated lookup mode handling for a more streamlined approach to user queries.
- Removed unnecessary import of UserIndexQueryDto and updated the getUserIndex method to directly use UserIndexPageQueryInput.
- Streamlined the user retrieval logic for improved clarity and maintainability.
…constraints

- Introduced maximum length constraints for title, originalAuthor, and description fields in the Song entity using UPLOAD_CONSTANTS.
- Simplified the songPreviewFromSongDocumentWithUser function to default the description to an empty string if not provided.
- Updated the SongPreview DTO to reflect the new maximum length validations for title, originalAuthor, and description fields.
…pdate package dependencies"

This reverts commit fb03997.
- Updated import statements across multiple DTO files to remove the '.js' extension for consistency and clarity.
- This change enhances maintainability and aligns with best practices for module imports.
- Removed the mongoose dependency from the validation package to streamline dependencies.
- Updated import statements in SongSearchQuery.dto.ts to reference the correct path without the file extension, enhancing consistency and maintainability.
Initialize @nbw/admin with a Bun.serve backend and React SPA static delivery, plus monorepo wiring.
Add explicit env validation, @nbw/database model registration, modular S3 client scaffolding, and Tailwind/shadcn-style UI baseline.
@tomast1337 tomast1337 force-pushed the feature/admin-tools branch from 67b93fe to d72d3ad Compare April 21, 2026 16:47
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.

1 participant