Skip to content

Conversation

webfiltered
Copy link
Contributor

@webfiltered webfiltered commented Sep 27, 2025

Summary

Migrate Zod schemas to a shared @comfyorg/schemas package to enable reuse across workspace packages while maintaining backward compatibility through re-exports.

Changes

  • What: Created new @comfyorg/schemas workspace package and migrated all Zod schemas from src/schemas/ to the new package
  • Breaking: None - all existing imports work through re-exports

Review Focus

The migration uses re-exports to maintain 100% backward compatibility - no import changes needed in consuming code. The git history is preserved through git mv operations.

The diff of this PR is massive, as the files were moved then replaced by re-export files (tracking issue will be created to remove these). The links below show the actual changes in the PR much more cleanly.

@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Sep 27, 2025
Copy link

github-actions bot commented Sep 27, 2025

🎭 Playwright Test Results

⚠️ Tests passed with flaky tests

⏰ Completed at: 09/27/2025, 02:35:14 AM UTC

📈 Summary

  • Total Tests: 463
  • Passed: 432 ✅
  • Failed: 0
  • Flaky: 2 ⚠️
  • Skipped: 29 ⏭️

📊 Test Reports by Browser

  • chromium: View Report • ✅ 425 / ❌ 0 / ⚠️ 2 / ⏭️ 29
  • chromium-2x: View Report • ✅ 2 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • chromium-0.5x: View Report • ✅ 1 / ❌ 0 / ⚠️ 0 / ⏭️ 0
  • mobile-chrome: View Report • ✅ 4 / ❌ 0 / ⚠️ 0 / ⏭️ 0

🎉 Click on the links above to view detailed test results for each browser configuration.

@webfiltered webfiltered marked this pull request as draft September 27, 2025 02:46
@webfiltered
Copy link
Contributor Author

Changed this to draft. There are translations being hard-coded in signInSchema.ts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:XXL This PR changes 1000+ lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant