Moving APIs in PreviewViewModel to Controller classes

[davidjiagoogle]

Interfaces QuickSettingsController, CaptureController, CaptureScreenController, SnackBarController, ZoomController, and DebugController are created. They contain APIs that were originally implemented in the view models. A default implementation of their APIs is made for each interface so that view model can be simplified

PreviewViewModel now only contains instances of these controller and has no APIs itself.

[gemini-code-assist]

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

[temcguir]

/gemini review

[gemini-code-assist]

Code Review

The pull request successfully refactors the PreviewViewModel by extracting various UI event handling logic into dedicated callback data classes: QuickSettingsCallbacks, CaptureCallbacks, SnackBarCallbacks, and DebugCallbacks. This significantly simplifies the PreviewViewModel and improves modularity, making the code easier to understand and maintain. The changes are well-implemented, and the new callback structures enhance code organization and reusability. The test file PreviewViewModelTest.kt has also been updated to reflect these changes, ensuring continued test coverage.

[temcguir]

/gemini review

[gemini-code-assist]

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

[temcguir]

/gemini review

[gemini-code-assist]

Code Review

This pull request is a significant and well-executed refactoring that decomposes the large PreviewViewModel into several smaller, single-responsibility controller classes. This greatly improves the architecture by enhancing modularity, testability, and overall code clarity. The changes are consistently applied across the UI components, ViewModels, and tests. My review includes a few suggestions: fixing a minor copy-paste bug in CaptureControllerImpl, adding KDoc to the new controller APIs to improve documentation, and a couple of minor stylistic improvements in PreviewScreen.kt to make the code more idiomatic.

[temcguir]

This is a good start. There are definitely a few things that could be cleaned up later (such as the zoom controller). If you can address the comments I've added we could merge this and make smaller clean-up PRs to address those later.

[temcguir]

/gemini review

[gemini-code-assist]

Code Review

This pull request is a significant and beneficial refactoring that moves logic out of the ViewModels and into dedicated controller classes. This greatly improves the separation of concerns and aligns well with the project's MVVM architecture. The code is much cleaner and easier to follow. My review focuses on ensuring the new components adhere to the repository's documentation and visibility standards. I've identified several new interfaces and a class that are missing KDoc documentation, and one property that should have its visibility restricted.

[temcguir]

LGTM once all the nits and gemini comments are addressed.