Skip to content

fix: file manager and cleanups#4922

Closed
Pavel910 wants to merge 3 commits intonextfrom
pavel/feat/module-resolution
Closed

fix: file manager and cleanups#4922
Pavel910 wants to merge 3 commits intonextfrom
pavel/feat/module-resolution

Conversation

@Pavel910
Copy link
Copy Markdown
Collaborator

@Pavel910 Pavel910 commented Feb 5, 2026

Summary

This PR cleans up legacy code and improves type safety across DynamoDB and GraphQL packages.

Changes

DynamoDB Toolbox Improvements

  • Extracted type definitions from dynamodb-toolbox internal paths to local definitions
  • Updated import paths to use public APIs instead of internal implementation details
  • Removed legacy createLegacyEntity helper that's no longer needed

Migrations Package Cleanup

  • Removed unused @webiny/logger dependency
  • Deleted obsolete utilities: forEachTenantLocale, createLocaleEntity
  • Simplified entity creation to only support current patterns

Type Cleanup

  • Removed unused type imports across API packages
  • Added proper type guards and annotations where needed
  • Fixed Lexical editor imports to use public APIs

GraphQL Schema Improvements

  • Standardized File Manager type naming (FmLocationFmFile_Location)
  • Changed folderId from String to ID! for better type safety
  • Simplified where input filters for ID-based queries

Other Improvements

  • Upgraded dotenv dependency to latest version
  • Removed deprecated test files for unfinished features
  • Cleaned up unused React components

Impact

395 lines removed, 72 lines added (net -323 lines) — cleaner codebase with better type safety.

🤖 Generated with Claude Code

@Pavel910 Pavel910 added this to the 6.0.0 milestone Feb 5, 2026
@Pavel910
Copy link
Copy Markdown
Collaborator Author

Pavel910 commented Feb 5, 2026

/vitest

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Feb 5, 2026

Vitest tests have been initiated (for more information, click here). ✨

@Pavel910 Pavel910 changed the title feat: set TS module resolution to bundler fix: file manager and cleanups Feb 5, 2026
@Pavel910 Pavel910 closed this Feb 5, 2026
@Pavel910 Pavel910 deleted the pavel/feat/module-resolution branch March 11, 2026 12:18
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